fix(console): slice 7 self-review — Debug drawer visible-when-hidden bug + polish

[brettchien]

Slice 7/7 of console-navigation-ia ADR (#83) — usercron adr83-impl runbook. Self-review now that all six implementation slices (#84–#89) are merged.

Per the runbook, this reads the full post-slice-6 console/ (not per-slice diffs) against the ADR §7 mockups and checks for the #82 class of bug — this time with a Playwright render check (headless Chromium + screenshots) rather than reading alone, since "does hidden actually hide it" isn't reliably answerable from source.

Found

• #debug-drawer rendered from first paint, hidden attribute notwithstanding. .debug-drawer { display: flex; ... } (needed for its flex-column layout) and the [hidden] UA-stylesheet rule (display: none) have equal CSS specificity — the later author rule won the cascade, so the drawer (position: fixed, z-index: 50) covered the whole app before any [⚙] was ever clicked. Confirmed via getComputedStyle in a headless render (screenshot in the commit description reproduces it), fixed with .debug-drawer[hidden] { display: none }. Checked every other hidden-toggled element in the console (#deploy-wrap, #fleet-detail, #config-editor, #mcpio, #debug-config) the same way — none of them set their own display on the toggled element, so none have this problem.
• The drawer also visually covered the topbar's right-side controls (cluster/poll status, theme/update buttons) while open — same render check. Gave .topbar its own stacking context (position: relative; z-index: 60, above the drawer's 50) so it stays on top, per Part E ("persistent chrome... spans every screen").
• No duplicate ids or dangling getElementById references anywhere in console/ — checked exhaustively (every id= vs every getElementById("...") call), all clean (the two duplicate-id bugs of this class were already caught and fixed in feat(console): collapse Activity/MCP/Config into a Debug drawer (slice 6/6) #89).

Also cleaned up

Small residue from reading the whole file instead of per-slice diffs: dead .logs CSS (nothing has used that class since Activity/MCP/Config moved into the drawer), and two stale "Compose tab"/"Config tab" code comments left over from before Compose stopped being a tab and Config moved into the drawer.

Checked, no issues

Persistent management chat across all three drill-down depths, [+ New fleet]/[+ Add instance]/[⚙] wiring at every mockup location, class-naming reuse across the six independently-implemented slices (cfg-btn, fd-btn/fd-gear reused consistently rather than redefined per-slice).

Verification

• tsc --noEmit — clean
• vitest run — 107/107 passing
• vite build — clean
• Playwright (headless Chromium, manually run — not part of CI): confirmed drawer hidden ⇒ display: none again after the fix, confirmed topbar stays visually on top when the drawer opens

Progress table: this is 7/7 — all slices land after this merges.

🤖 Generated by Orca (ecs-claude), usercron adr83-impl.