refactor: optimize ship.md + review.md — compress + remove decouple-leftover dup

[JFK]

Problem

commands/ship.md (663 lines) and commands/review.md (543 lines) carry ~1,350 tokens of intra-file redundancy plus leftover duplication from the recent ship/review decouple (#83/#84).

Scope (CTO/Explore review 2026-06-13)

• ship.md: the binary-gate concept is re-explained ~4x (e.g. step 1b, step 5, step 6, step 7 intro). Collapse to one statement + references. Convert the --review dispatch (~ship.md:207-217) and advisor-verdict semantics (~ship.md:461-475 prose+table) to a single decision table.
• review.md: convert the per-thread reply/resolve prose (~review.md:424-447) and the state-write schema prose (~review.md:474-506) to tables/bullets.
• decouple leftovers: the Trust-boundary block, Output-language block, AUTONOMOUS explanation, and jq-filter note are near-verbatim duplicated between the two files. Keep ONE authoritative copy (the file that owns the logic) and reference from the other; the jq filter stays guarded by copilot loop: file allow-list and max-diff-per-iteration guard #2.

Precision tightening

• State AUTONOMOUS->HITL suppression once, explicitly (currently scattered across ~ship.md:101/383/416).
• Specify "last-wins" verdict scan (coordinate with copilot loop: file allow-list and max-diff-per-iteration guard #2).

Acceptance criteria

• Existing ship/review tests + copilot loop: file allow-list and max-diff-per-iteration guard #2 contract tests green.
• Token baseline (verdict parser: replace heuristic with structured ## Verdict: <token> contract #1) shows reduction; no behavior change to the gate/loop.

Depends on

#1, #2.

[JFK]

Closing as largely phantom / infeasible after a re-audit against the actual files (see #96 for the milestone wind-down).

Verified findings:

• "binary-gate re-explained ~4x" — FALSE. There are 2 distinct explanations (operational at ship.md:255-273, formal intro at ship.md:304-310), serving different purposes — not redundant.
• "jq-filter dedup between ship.md and review.md" — FALSE. ship.md contains no jq filter at all; review.md owns the Copilot-detection filter self-contained.
• "cross-file dedup (Trust boundary / Output language / AUTONOMOUS): keep one copy + reference" — INFEASIBLE. Slash commands load independently with no runtime include; a cross-file reference would force a Read per invocation (no runtime token saving) or leave a standalone /review invocation with a dangling reference.
• The AUTONOMOUS block is ~85% duplicated between the two files, but moving it to config.md saves no runtime tokens (config.md isn't loaded by ship/review at runtime either) and degrades standalone readability.

Net: no safe, runtime-token-reducing change here. Reopen if a genuine non-cosmetic improvement surfaces.