SPA media browse/detail: add-to-* affordances, per-show scan, per-episode info/troubleshoot entries missing (blocks #91 phase b) #209

New Issue

2026-07-09T21:40:11+02:00

timothy commented

2026-07-09 21:40:11 +02:00

From the #203 mutation-depth sweep (2026-07-09). Read-path parity of the media surface is genuinely solid (browse/pagination/search/detail/drill-in per the accepted parentId decision; /app/media/images/browser is fully parity-OK). Mutation-depth parity is systematically absent:

Tier 1 — capability lost outright (no SPA path at all):

Per-show Quick/Deep Scan: POST /api/libraries/{id}/scan-show exists (LibrariesController.cs:29) with zero SPA callers (Blazor TelevisionSeasonList.razor:62–73,308–320).
Per-episode Media Info dialog: API works for episodes (GET /api/media-items/{id}/info), no UI path invokes it (episode cards have no actions, no detail page).
Per-episode Troubleshoot Playback entry point: hand-built ?mediaItem={id} URL only (parity doc already admits this).
Cross-show season search/browse (/media/tv/seasons?query=…): MediaBrowseScreen KINDS deliberately has no seasons entry; drill-in only lists one show's seasons.

Tier 2 — direction inverted, not recorded as a decision:

Every add-to-collection / add-to-playlist / add-to-schedule affordance (per-item and multi-select on all 9 list pages + 4 detail pages) exists only from the inverse direction (collection/playlist/schedule editor pickers). Real workflow regression ("I'm looking at this movie, add it to X"). Either build the shared "Add to…" affordance (same component as the search-screen issue) or record an explicit accepted-deviation entry in docs/decisions.md.
Stale pointer: MediaDetailScreen.tsx:524 claims these are "tracked by #153/#155" — both closed without adding them; nothing tracked this until now.

Tier 3 — minor: LetterBar A–Z jump; metadata chips/actor links rendered as dead tags; reduced episode/music-video card metadata. (Also: parity doc understates /media/tv/episodes — browse is actually FULL via kind=episodes.)

Severity: MUST-FIX for Tier 1 + a decision (build or waive) on Tier 2 before Blazor removal. Refs: #91, #203.

From the #203 mutation-depth sweep (2026-07-09). Read-path parity of the media surface is genuinely solid (browse/pagination/search/detail/drill-in per the accepted `parentId` decision; `/app/media/images/browser` is fully parity-OK). Mutation-depth parity is systematically absent: **Tier 1 — capability lost outright (no SPA path at all):** - **Per-show Quick/Deep Scan**: `POST /api/libraries/{id}/scan-show` exists (`LibrariesController.cs:29`) with **zero SPA callers** (Blazor `TelevisionSeasonList.razor:62–73,308–320`). - **Per-episode Media Info dialog**: API works for episodes (`GET /api/media-items/{id}/info`), no UI path invokes it (episode cards have no actions, no detail page). - **Per-episode Troubleshoot Playback entry point**: hand-built `?mediaItem={id}` URL only (parity doc already admits this). - **Cross-show season search/browse** (`/media/tv/seasons?query=…`): `MediaBrowseScreen` KINDS deliberately has no seasons entry; drill-in only lists one show's seasons. **Tier 2 — direction inverted, not recorded as a decision:** - Every add-to-collection / add-to-playlist / add-to-schedule affordance (per-item and multi-select on all 9 list pages + 4 detail pages) exists only from the inverse direction (collection/playlist/schedule editor pickers). Real workflow regression ("I'm looking at this movie, add it to X"). Either build the shared "Add to…" affordance (same component as the search-screen issue) or record an explicit accepted-deviation entry in docs/decisions.md. - Stale pointer: `MediaDetailScreen.tsx:524` claims these are "tracked by #153/#155" — both closed without adding them; nothing tracked this until now. **Tier 3 — minor:** LetterBar A–Z jump; metadata chips/actor links rendered as dead tags; reduced episode/music-video card metadata. (Also: parity doc understates `/media/tv/episodes` — browse is actually FULL via `kind=episodes`.) **Severity: MUST-FIX for Tier 1 + a decision (build or waive) on Tier 2** before Blazor removal. Refs: #91, #203.

timothy referenced this issue

2026-07-09 21:42:12 +02:00

Parity doc is unreliable as the deletion checklist: "SPA-READY" conflates screen-exists with functional parity #203

timothy referenced this issue

2026-07-09 21:42:29 +02:00

ChicoryTV SPA: Blazor → SPA cutover (retire old UI) #91

timothy commented

2026-07-09 23:47:27 +02:00

Work started (session 2026-07-09, branch feat/208-209-add-to). Recon complete. Plan: shared Add-to dialogs (collection/playlist/schedule) as reusable SPA components + two small new endpoints (POST /api/playlists/{id}/items wrapping the existing AddItemsToPlaylist command; GET /api/search/all-items exposing QuerySearchIndexAllItems for the query-based Add All flow — mirrors Blazor's two-step materialize-then-add, no new business logic). Per-show scan reuses POST /api/libraries/{id}/scan-show. Wiring: search cards (nav + per-card add + multi-select + add-all + save-as-smart-collection), browse tiles, all four detail screens, per-episode info/troubleshoot/add.

Work started (session 2026-07-09, branch `feat/208-209-add-to`). Recon complete. Plan: shared Add-to dialogs (collection/playlist/schedule) as reusable SPA components + two small new endpoints (`POST /api/playlists/{id}/items` wrapping the existing `AddItemsToPlaylist` command; `GET /api/search/all-items` exposing `QuerySearchIndexAllItems` for the query-based Add All flow — mirrors Blazor's two-step materialize-then-add, no new business logic). Per-show scan reuses `POST /api/libraries/{id}/scan-show`. Wiring: search cards (nav + per-card add + multi-select + add-all + save-as-smart-collection), browse tiles, all four detail screens, per-episode info/troubleshoot/add.