bvandeusen
15b59a214d
New client-reported playback-error log. Surfaces zero-duration
tracks (and future load_failed / stalled kinds) into an admin
inbox so the operator can hide / delete / re-request the
offending track.
Schema (migration 0032):
- playback_errors table with CHECK constraints on the kind +
resolution enums (per the standing rule that new enum values
need a migration to add)
- Partial index on unresolved rows for fast inbox lookup
- ON DELETE CASCADE from tracks + users so cleanup is automatic
Endpoints:
- POST /api/playback-errors: any signed-in user reports. Body
validates track existence + kind whitelist; client_id required
so support can correlate reports from the same device.
- GET /api/admin/playback-errors?resolved=false&offset=&limit=:
admin list with join to track/album/artist for table render
without per-row round-trips. Pagination capped at 200/page.
- POST /api/admin/playback-errors/{id}/resolve: admin marks
resolved with a resolution enum string.
Auto-resolve on Hide/Delete/Re-request from the inbox row is
driven from the web client (two sequential calls) — keeps the
existing track-action endpoints unchanged.
64 lines
2.6 KiB
SQL
64 lines
2.6 KiB
SQL
-- name: InsertPlaybackError :one
|
|
-- Records a client-reported playback failure. The handler validates
|
|
-- the kind enum before this runs; the CHECK constraint is a
|
|
-- belt-and-braces guard.
|
|
INSERT INTO playback_errors (track_id, user_id, client_id, kind, detail)
|
|
VALUES ($1, $2, $3, $4, $5)
|
|
RETURNING id, track_id, user_id, client_id, kind, detail, occurred_at,
|
|
resolved_at, resolved_by, resolution;
|
|
|
|
-- name: ListAdminPlaybackErrors :many
|
|
-- Admin inbox query. Returns one row per playback_errors row joined
|
|
-- with track / album / artist metadata so the SPA can render without
|
|
-- a second round-trip. Filter by resolved status via the sqlc.arg —
|
|
-- pass true for the Resolved tab, false for Unresolved (the default).
|
|
SELECT
|
|
pe.id AS id,
|
|
pe.track_id AS track_id,
|
|
pe.user_id AS user_id,
|
|
u.username AS username,
|
|
pe.client_id AS client_id,
|
|
pe.kind AS kind,
|
|
pe.detail AS detail,
|
|
pe.occurred_at AS occurred_at,
|
|
pe.resolved_at AS resolved_at,
|
|
pe.resolution AS resolution,
|
|
t.title AS track_title,
|
|
t.file_path AS track_file_path,
|
|
ar.name AS artist_name,
|
|
al.title AS album_title,
|
|
al.id AS album_id
|
|
FROM playback_errors pe
|
|
JOIN users u ON u.id = pe.user_id
|
|
JOIN tracks t ON t.id = pe.track_id
|
|
JOIN albums al ON al.id = t.album_id
|
|
JOIN artists ar ON ar.id = t.artist_id
|
|
WHERE (sqlc.arg(resolved_filter)::bool AND pe.resolved_at IS NOT NULL)
|
|
OR (NOT sqlc.arg(resolved_filter)::bool AND pe.resolved_at IS NULL)
|
|
ORDER BY pe.occurred_at DESC
|
|
LIMIT sqlc.arg(lim)::int OFFSET sqlc.arg(off)::int;
|
|
|
|
-- name: ResolvePlaybackError :one
|
|
-- Marks a single error resolved. Idempotent over (id, resolution) —
|
|
-- a second resolve with the same resolution is a no-op rewrite. Returns
|
|
-- the row so the handler can echo it. The CHECK constraint enforces
|
|
-- the resolution enum.
|
|
UPDATE playback_errors
|
|
SET resolved_at = now(),
|
|
resolved_by = $2,
|
|
resolution = $3
|
|
WHERE id = $1
|
|
RETURNING id, track_id, user_id, client_id, kind, detail, occurred_at,
|
|
resolved_at, resolved_by, resolution;
|
|
|
|
-- name: ResolveAllPlaybackErrorsForTrack :execrows
|
|
-- Bulk-resolve every unresolved error for a track when an admin acts
|
|
-- on it from a non-inbox surface (existing quarantine flow, etc.).
|
|
-- Returns the affected row count so the caller can surface "N reports
|
|
-- auto-resolved" in the response if useful.
|
|
UPDATE playback_errors
|
|
SET resolved_at = now(),
|
|
resolved_by = $2,
|
|
resolution = $3
|
|
WHERE track_id = $1 AND resolved_at IS NULL;
|