test(tui): extend coverage gate to Ink components + App.tsx (closes #1501)

[cliffhall]

Closes #1501.

Summary

Lifts the interim React-surface coverage exclusion in the TUI client (the follow-up to #1484 / #1532). The whole clients/tui/src tree now clears the uniform 90/90/90/90 per-file gate with no React-surface exclusions — App.tsx, every Ink component, and the useSelectableList hook are all exercised under coverage.

Test-only change: no source files were modified.

How it's tested

Added ink-testing-library and renderer-based tests that mount components in-process and drive keypresses through stdin. Two shared passthrough doubles in __tests__/helpers/ make the components mountable under the non-TTY test renderer:

• inkScrollViewMock.tsx — the real ScrollView renders a placeholder minimap and never mounts its children in a non-TTY env; the double renders children directly and stubs the imperative ref API (scrollBy/scrollTo/getViewportHeight).
• inkFormMock.tsx — the real ink-form Form is hard to drive field-by-field; the double fires onSubmit on Enter.

Per-component approach:

• Tabs + hook (Info/Auth/Resources/Prompts/Tools/Notifications/History/Requests, useSelectableList, SelectableItem, Tabs): assert on rendered frame text and drive list navigation / scroll keys.
• Modals (Tool/Resource/Prompt test modals, DetailsModal): these render position="absolute", which produces an empty frame under the renderer, so their tests assert on behavior — injected InspectorClient fakes being called, onClose, and state transitions — while the passthrough ScrollView still mounts the inner JSX for coverage.
• App.tsx (~1885 lines): mounts against a controllable mock of the entire @inspector/core surface (InspectorClient, the 7 managed-state classes, the 8 react hooks, transport, auth) plus the callback server. 67 tests drive the keyboard state machine, per-tab content, modal-open flows, OAuth (guided/quick/clear, success + error), and connect/disconnect. Each render is unmounted in afterEach (concurrent ink mounts share raw-mode stdin and interfere with useInput).

Verification

• clients/tui npm run validate — clean (format / lint / build / test).
• clients/tui npm run test:coverage — green, 260 tests, every file ≥90 on all four dimensions, no ERROR lines. App.tsx: 94.6 stmts / 90.6 branch / 94.5 funcs / 96.6 lines.

Config

clients/tui/vitest.config.ts: removed all React-surface entries from the coverage exclude (only the pure re-export tui-servers.ts and test/type globs remain) and widened the include glob to *.test.tsx. The interim-scope comment from #1484 is updated to reflect the completed lift.

🤖 Generated with Claude Code

[cliffhall]

Follow-up: bumped App.tsx branch coverage from 90.6% → 91.48% for a safer margin above the 90 gate — added tests for the OAuth handlers' non-Error throw arms (quick onCallback, guided start/advance, run-to-completion) and a re-entrant quick-auth guard.

Also hardened the App renderer tests against the heavier v8 coverage instrumentation (where React+ink's multi-macrotask render scheduling races fixed ticks): press() now settles two ticks per key, and frame/async-state assertions poll via expectFrame/waitUntil instead of asserting after a single tick. Plain npm test (the CI path) was already stable; this makes the local coverage gate stable too — 13/13 clean runs vs ~2/8 flaky before.