Comment on: ADR: Headless Mode as GSD's Programmable Surface for the Agent Ecosystem

## Independent ADR Review Thorough review grounding each claim against the current codebase and external ecosystem state. --- ### Overall Assessment This is a well-structured ADR with genuine strategic vision. The phased approach is sound, the dependency graph is correct, and the "What We're NOT Doing" section shows good discipline. That said, there are factual inaccuracies, missing risks, and architectural gaps that should be addressed before this moves to "Accepted." --- ### Factual Accuracy Audit #### Verified ✅ | Claim | Finding | |---|---| | "~30 commands" in RPC server | **31 commands** — close enough. Accurate. | | Typed union types in `rpc-types.ts` | Confirmed. Discriminated unions for `RpcCommand`, `RpcResponse`, `RpcExtensionUIRequest`. | | `waitForIdle()`, `collectEvents()`, `promptAndWait()` on RpcClient | All three exist. | | String-matching completion detection | Confirmed in `headless-events.ts`: matches `'auto-mode stopped'`, `'step-mode stopped'`, `'blocked:'`...