Deprecate /flow-reviewer in favour of /flow-models

[vriesd]

Why

/flow-reviewer shipped in 8.4.0, when review was the only selectable role. 8.5.0 generalised the same picker to cover planning and added /flow-models over it, but left the old entry point in place. Both commands now reach one open() handler in src/tui.ts with the same active-work check, staleness check and configuration write — the alias saves one keypress.

The surface is also asymmetric. There is no /flow-planner, so a second review-only command implies the reviewer is special when it is not, and hides planning-model routing from anyone who learned the 8.4.0 command first.

What changes

• src/tui.ts marks the alias deprecated in the palette. Behaviour is unchanged.
• tests/reviewer-picker.test.ts reaches review through /flow-models and the role menu, so the suite no longer depends on the alias. One test still exercises the alias directly and asserts its title marks it deprecated — that test and the command block are all 9.0.0 has to delete.
• .agents/plans/10-reviewer-command-deprecation/README.md records the decision.

Removal waits for 9.0.0 per the cadence rule in docs/release-qualification.md, alongside the opencode-plugin-flow.reviewer-picker module id. A major needs full qualification for other reasons, so the removal carries no qualification cost of its own.

No version bump, deliberately

Once src/tui.ts diverges from the reviewed commit c0064dc, the 8.5.0 feature record can no longer re-verify, and .github/workflows/ci.yml runs that verification exactly when a change touches the record or package.json. Neither is touched here.

There is also no patch release available to carry this on its own: assertPatchScope requires a baseline sharing the candidate's major.minor with a sealed bundle and canary, and only 8.3.0 has those. The CHANGELOG entry is written at the next version bump, since ## [Unreleased] is asserted never to appear.

Documentation corrected alongside

• docs/quickstart.md is what README.md sends new users to, and it predated 8.4.0 — it taught only the opencode.json tuple route, never mentioned either picker, and contradicted README.md on precedence. It now leads with /flow-models and states the real order.
• docs/troubleshooting.md pinned opencode-plugin-flow@8.1.3 through four releases. It now refers to the README, and a new contract test keeps the asserted README pins the only version pins in maintained prose.

Verification

bun run check — 1209 pass, 1 skip, 0 fail. Maintained prose 98,993 of 99,000; src TypeScript 285,152 of 285,696.

🤖 Generated with Claude Code

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.