feat(api): channel on-air / now-playing state endpoint (#97) #112

Merged

timothy merged 7 commits from issue-97-channel-state-api into docs/59-ui-redesign-brief 2026-07-04 07:48:47 +02:00

Conversation 0 Commits 7 Files Changed 11

+693

7 Commits

Author	SHA1	Message	Date
timothyandClaude Fable 5	73455aae28	fix(api): surface programs not filler in channel state, tighten queries ... Adversarial review fixes for the #97 endpoint: - nowPlaying now resolves like the XMLTV guide: when the wall-clock item is pre/mid-roll filler, surface the guide group's program item (title + full program bounds) instead of the filler; a guide entry with no program item (e.g. fallback loop) reports null. Previously an ad break would retitle the channel and reset progress every 30 seconds. - covering-item lookup now runs one cheap projected query per distinct playout offset instead of hydrating the full metadata include tree for a window widened by the largest offset across ALL channels (up to 24h); metadata is fetched in a second query scoped to the covering guide groups, time-bounded to stay safe against GuideGroup recycling (mod 10000). Regression caught while testing: covering items are keyed by (source channel, lookup time) so a source channel no longer inherits its offset mirror's lookup result. - GetChannelStatesForApi now carries Now explicitly (controller passes DateTime.UtcNow), making boundary semantics testable: new tests pin now == Start (playing) and now == Finish (not playing), plus mid-roll, filler-only, and mirror-source-row cases. - ChannelNowPlayingResponseModel title is non-nullable in the schema (#nullable enable) — GetDisplayTitle always returns a string. - GetState uses the typed-return controller convention; Playouts.Mapper reverted to internal (handler lives in the same assembly). - Spec doc notes the OnDemand drift caveat and filler semantics. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>	2026-07-02 21:53:21 +02:00
timothyandClaude Fable 5	e66f49e54a	fix(api): make channel state NowPlaying nullable in DTO ... An off-air channel has no current playout item, so the handler returns null for NowPlaying; the DTO now declares that (file-scoped #nullable — ErsatzTV.Core has NRT disabled project-wide). Regenerated OpenAPI and web types: nowPlaying keeps its nullable oneOf schema (matching the schema contract test) and channelNumber loses a spurious nullable annotation left over from an earlier draft — the field is required. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>	2026-07-02 21:37:00 +02:00
timothy	07e9d9838d	fix(api): resolve mirror channel now playing state	2026-07-02 20:02:01 +02:00
timothy	f01ffc6254	test(api): enforce nullable channel now playing schema	2026-07-02 19:50:19 +02:00
timothy	6057bcea7d	docs(api): document channel state endpoint	2026-07-02 19:45:53 +02:00
timothy	7b903d4bb4	feat(api): return channel runtime state	2026-07-02 19:37:14 +02:00
timothy	7c1a1d5672	feat(api): add channel state endpoint contract	2026-07-02 19:29:59 +02:00