refactor(web): extract Dashboard screen #360

Merged

timothy merged 1 commits from codex/246-dashboard-screen into main 2026-07-15 21:30:52 +02:00

Conversation 1 Commits 1 Files Changed 4

+592 -485

timothy commented 2026-07-15 21:19:00 +02:00

Owner

Copy Link

Copy Source

Refs #246

Summary

• extract Dashboard rendering and dashboard-specific helpers into DashboardScreen.tsx
• move detailed Dashboard coverage beside the screen while keeping a thin App route smoke
• keep App-owned health querying because the shell/sidebar also consumes it
• preserve API, routes, CSS, server state, and behavior

Verification

• npm test — 85 files, 762 tests passed
• npm run lint
• npm run typecheck
• npm run build
• npm run check:api

This is a pure SPA leaf refactor, so live E2E was not required.

Review and docs posture

No docs update is needed because no route or convention changed. A current-head cold-review verdict will be posted separately.

Refs #246 ## Summary - extract Dashboard rendering and dashboard-specific helpers into `DashboardScreen.tsx` - move detailed Dashboard coverage beside the screen while keeping a thin App route smoke - keep App-owned health querying because the shell/sidebar also consumes it - preserve API, routes, CSS, server state, and behavior ## Verification - `npm test` — 85 files, 762 tests passed - `npm run lint` - `npm run typecheck` - `npm run build` - `npm run check:api` This is a pure SPA leaf refactor, so live E2E was not required. ## Review and docs posture No docs update is needed because no route or convention changed. A current-head cold-review verdict will be posted separately.

timothy added 1 commit 2026-07-15 21:19:00 +02:00

refactor(web): extract Dashboard screen ... bceffef856

Move Dashboard rendering, health and on-air helpers, and detailed behavior tests into a screen-owned module while retaining only shell composition coverage in App.

Refs #246

Co-Authored-By: OpenAI Codex <codex@openai.com>

timothy commented 2026-07-15 21:23:15 +02:00

Author

Owner

Copy Link

Copy Source

Independent cold-context adversarial review found no issues. App retains shell-wide health ownership and the thin route smoke; Dashboard owns its rendering, helpers, and detailed tests. API behavior, routes, and CSS are preserved.

Review-verdict: MERGEABLE @ bceffef856

Independent cold-context adversarial review found no issues. App retains shell-wide health ownership and the thin route smoke; Dashboard owns its rendering, helpers, and detailed tests. API behavior, routes, and CSS are preserved. Review-verdict: MERGEABLE @ bceffef856bb43efb7d638511ea1e066afa3075c

timothy merged commit bceffef856 into main 2026-07-15 21:30:52 +02:00

timothy deleted branch codex/246-dashboard-screen 2026-07-15 21:30:52 +02:00

timothy referenced this pull request 2026-07-15 21:59:38 +02:00

Extract Dashboard and Guide screens from App.tsx #246

timothy referenced this pull request 2026-07-15 22:00:02 +02:00

ChicoryTV rewrite — queue tracker (goal, arc, session protocol) #237

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

ad-hoc

One-off / ad-hoc work not tracked by a dedicated issue

api

REST API / HTTP endpoints

bug

Something isn't working

ci-cd

Build, test, deploy pipeline

content

Channel content / schedules / playlists

dependencies

Dependency updates (Renovate)

enhancement

New feature or improvement

frontend

ChicoryTV React SPA frontend

in-progress

Claimed by an active session — do not pick up

jellyfin

Jellyfin tuner / IPTV integration

parked

Excluded from automatic queue pickup; work only when explicitly selected

priority: high

priority: low

priority: medium

review

Adversarial review finding

security

Security / vulnerability fix

No labels

Milestone

No items

No Milestone

Projects

Clear projects

No projects

Assignees

Clear assignees

renovate timothy (Timothy)

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: timothy/ersatztv#360