Summary
Low-priority frontend issues found in the review, batched together.
Items
- Blank
/sdoh route (pages/Sdoh.tsx:149-150): resource cards render only for a selected patient; the patientless /sdoh route (reachable from the mobile "Resources" tab, MobileNav.tsx:38) shows category tabs over an empty page with no explanation.
- Stale "today" (
pages/TaskQueue.tsx:66): TODAY_ISO is a module-load constant — overdue/"today" filters go stale across midnight in a long-lived tab.
- Unsupported
urgent priority (pages/TaskManagement.tsx:102-122): the New Task form offers urgent, which isn't in the TaskListEntry['priority'] union — PRIORITY_LABEL['urgent'] renders an empty pill, and the created task gets a local- id that vanishes on the next refetch while the toast claims "Task created".
- Role-inconsistent navigation (
components/Sidebar.tsx:74): "Cost/ROI" is shown to all roles but the route is director-only (App.tsx:97-104) — non-directors get bounced to their home.
- Fabricated data without a demo badge:
components/Header.tsx:70 hardcodes "Last synced 42s ago" and a badge count of "3"; pages/CostROI.tsx is 100% fabricated financial/HEDIS numbers with no DemoFallbackBadge, contradicting the codebase's own documented honesty rule (G4) that every other page follows.
- Unreachable fallbacks (
pages/Governance.tsx:179-202): the MOCK_AUDIT_TRAIL/MOCK_MODEL_PERFORMANCE/MOCK_PARITY fallbacks can never render — the component is gated on !isError, so on error the user sees the red error line, never the fallback the variables were built for.
Summary
Low-priority frontend issues found in the review, batched together.
Items
/sdohroute (pages/Sdoh.tsx:149-150): resource cards render only for a selected patient; the patientless/sdohroute (reachable from the mobile "Resources" tab,MobileNav.tsx:38) shows category tabs over an empty page with no explanation.pages/TaskQueue.tsx:66):TODAY_ISOis a module-load constant — overdue/"today" filters go stale across midnight in a long-lived tab.urgentpriority (pages/TaskManagement.tsx:102-122): the New Task form offersurgent, which isn't in theTaskListEntry['priority']union —PRIORITY_LABEL['urgent']renders an empty pill, and the created task gets alocal-id that vanishes on the next refetch while the toast claims "Task created".components/Sidebar.tsx:74): "Cost/ROI" is shown to all roles but the route is director-only (App.tsx:97-104) — non-directors get bounced to their home.components/Header.tsx:70hardcodes "Last synced 42s ago" and a badge count of "3";pages/CostROI.tsxis 100% fabricated financial/HEDIS numbers with noDemoFallbackBadge, contradicting the codebase's own documented honesty rule (G4) that every other page follows.pages/Governance.tsx:179-202): theMOCK_AUDIT_TRAIL/MOCK_MODEL_PERFORMANCE/MOCK_PARITYfallbacks can never render — the component is gated on!isError, so on error the user sees the red error line, never the fallback the variables were built for.