docs(planning): finalize phase21d commit references and smoke results

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: lhcaps

.planning/ROADMAP.md

@@ -1163,7 +1163,7 @@ This patch addresses accumulated traceability drift — several phases completed
 **Phase 21A commits:** `86416bf` (extraction) + `27d78bb` (cleanup round 1) + `2f3a1c4` (cleanup round 2 — leftovers)
 **Phase 21B commit:** `3693061` (runtime sync) + finalization commit
 **Phase 21C commit:** `d667962e` (prop surface cleanup) + `2227632e` (groundTruth overlay restored, Overview text neutralized)
-**Phase 21D commit:** [pending] (inspector routing boundary — fake state removed)
+**Phase 21D commit:** `4aa564a5` (inspector routing boundary — fake state removed)
 **Phase 21A status:** Done — App composition boundary, AppRoutes extraction, panel extractions, import cleanup, dead code removal. Full verification gate passed.
 **Phase 21B status:** Done (10/10) — FE/BE runtime sync, /api/health/runtime/status endpoint, useRuntimeStatus hook, ReadinessStrip reads real state, 3 controllers extracted. App.tsx reduced from 529 to ~144 lines. runtimeState.health derives from backend truth. Browser smoke passed. Dev boot text corrected.
 **Phase 21C status:** Done — AppRoutes prop surface reduced from 23 to 17 props (26% reduction). Threshold/annotation/pipeline state moved to feature-local components. JobsPanel owns threshold locally, AnnotationEnginePanel owns all annotation state, ShellHeader cleaned up. groundTruth overlay regression fixed, Overview neutral pipeline text restored. 65/65 tests pass. 21D pending.

.planning/STATE.md

@@ -33,7 +33,7 @@ Last updated: 2026-05-05.
 - Phase 21A ✅ Done — App composition boundary, AppRoutes extraction, panel extractions, import cleanup, dead code removal, full verification gate passed.
 - Phase 21B ✅ Done (10/10) — FE/BE runtime sync, /api/health/runtime/status endpoint, useRuntimeStatus hook, ReadinessStrip reads real state, useDatasetsController/useInferenceJobController/useEvaluationController extracted. App.tsx reduced from 529 to ~144 lines. runtimeState.health now derives from backend truth. Browser smoke passed.
 - Phase 21C ✅ Done — AppRoutes prop surface reduced from 23 to 17 props (26% reduction). Threshold/annotation/pipeline state moved to feature-local components. JobsPanel owns threshold, AnnotationEnginePanel owns annotation state, ShellHeader cleaned up. 65/65 tests pass, all runtime truth invariants preserved. groundTruth overlay regression fixed, Overview neutral pipeline text restored. 21D pending.
-- Phase 21D ✅ Done — InspectorRouter props refactored. Fake annotation/pipeline state removed from AppRoutes. AnnotationInspector and PipelineInspector rendered as honest embedded-notice panels for their respective sections. 4 unused types removed from inspector.types.ts. Option A1 chosen (section-owned inspector). Phase 21 fully closed.
+- Phase 21D ✅ Done — Commit `4aa564a5`. InspectorRouter props refactored. Fake annotation/pipeline state removed from AppRoutes. AnnotationInspector and PipelineInspector rendered as honest embedded-notice panels for their respective sections. 4 unused types removed from inspector.types.ts. Option A1 chosen (section-owned inspector). Phase 21 fully closed.
 - Phase 21 (Frontend split completion) Phase 21A + 21B + 21C + 21D all complete.
 - Phase 22A (Test harness) pending
 - Phase 22B (Production test suite) pending

.planning/phases/phase-21-frontend-feature-split-completion/21D-REVIEW.md

@@ -81,6 +81,7 @@ All checks passed:
 | `grep "pipelineDefinition={demoSnapshot"` | 0 results |
 | `grep "pipelineValidation={validatePipelineDefinition"` | 0 results |
 | `grep "useRuntimeStatus" (only App.tsx)` | PASS |
+| Playwright nav smoke (10 checks) | **PASS** — all 8 sections navigate without errors, no console errors |
 
 ---
 

.planning/phases/phase-21-frontend-feature-split-completion/21D-SUMMARY.md

@@ -3,7 +3,7 @@
 **Phase:** 21D
 **Date:** 2026-05-05
 **Status:** Complete
-**Commit:** [pending]
+**Commit:** `4aa564a5`
 
 ---
 
@@ -142,7 +142,16 @@ Not Option B because:
 
 ## Browser Smoke
 
-Not run — verification limited to typecheck/test/lint/build. The change is structural refactor only (no visual/behavioral changes to panel layouts). The honest embedded-notice rendering is strictly internal to `InspectorRouter`.
+**PASS — Playwright navigation smoke (2026-05-05)**
+
+Ran `pnpm playwright test e2e/navigation.spec.ts --project=chromium` against `http://localhost:5173`.
+
+All 10 checks passed:
+- 8 nav sections navigate without errors (Command, Media, Versions, Annotate, Pipeline, Jobs, Replay, Diff)
+- No console errors on initial load
+- All 8 nav buttons are visible
+
+This confirms the `InspectorRouter` refactor does not break navigation or introduce console errors.
 
 ---