Review Plan

Reading workflow files (TOC protocol)

When you Read any file under .claude/workflow/ or .claude/skills/, follow the protocol in conventions.md §1.8:

1. Read the TOC region: from <!--Document index start--> to <!--Document index end--> (read to the closing delimiter, not a fixed line count). If the file has no TOC region (a file whose only ## heading is this bootstrap block carries none, per §1.8(d)), read the file in full.
2. Match TOC rows where Roles contains any of your roles (or your role is any, or the row's Roles is any) AND Phases contains any of your phases (or your phase is any, or the row's Phases is any).
3. Use Read(offset, limit) to read only matched sections; if no row matches your role/phase, the file holds nothing for you — do not read further.

Your role: orchestrator or reviewer-plan (whichever invoked this skill). Your phase: 2 (Plan Review).

Inline refs you find inside workflow files carry the same name:roles:phases suffix; apply file-level filtering before opening: a ref matches when any of your roles is in its roles and any of your phases is in its phases, your own any on either axis matches every ref on that axis, and a ref whose own roles or phases is any matches you. Backtick-wrapped refs carry no suffix; open or skip them at your discretion.

<!--Document index start-->

<!--Document index end-->

Read and follow the workflow for Phase 2 (Implementation Review).

House style for chat-scale prose. User-facing prose produced from this file (status updates, escalation prompts, replanning summaries, review-mode loop turns, handoff notes, whichever apply) follows the AI-tell subset of house-style.md: ## Banned sentence patterns, ## Banned analysis patterns, ## Orientation, and ## Plain language. Structural rules (§ BLUF lead, § Structural rules for the ≤200-word section cap, § Document-shape rules (design / ADR-specific)) do not apply to chat-scale prose. See conventions.md:orchestrator,reviewer-plan:2 §1.5 for the workflow-level anchor and tier mapping.

Manual override. Phase 2 normally runs autonomously as the first phase of /execute-tracks when the startup protocol detects State 0 (the phase ledger has no phase boundary yet — D3; there is no plan ## Plan Review checkbox under the thinned plan). This skill is a manual entry point for re-running the same review — useful after inline replanning has produced a revised plan, or when you want to explicitly re-validate the plan against current code without going through /execute-tracks.

Read these workflow documents in order before starting:

1. conventions.md:orchestrator,reviewer-plan:2 — shared formats, glossary, plan file structure, scope indicators, review iteration protocol
2. implementation-review.md:orchestrator,reviewer-plan:2 — Phase 2 orchestration: autonomous classifier flow (mechanical findings auto-fixed, design-decision findings escalated to user), audit trail, mutation discipline for design.md fixes

You are the Implementation Review Orchestrator. Your job is to validate the plan's consistency with the codebase and design document, then validate its structural quality — applying mechanical fixes autonomously and escalating only design-level decisions to the user. The full orchestration loop, classifier rules, and audit-trail format all live in implementation-review.md:orchestrator,reviewer-plan:2; this skill exists only to provide a manual entry point.

Plan directory name: if "$ARGUMENTS" is non-empty, use it as the directory name. Otherwise, default to the current git branch name (git branch --show-current).

Plan file: docs/adr/<dir-name>/_workflow/implementation-plan.md Track files directory: docs/adr/<dir-name>/_workflow/plan/ Design document: docs/adr/<dir-name>/_workflow/design.md

Each plan/track-N.md track file carries the pending track's detail across four plan-at-start sections: ## Purpose / Big Picture (BLUF + intro paragraph), ## Context and Orientation (codebase state at the start of the track plus any track-level Mermaid diagram), ## Plan of Work (prose sequence of edits, ordering constraints, invariants), and ## Interfaces and Dependencies (in-scope/out-of-scope files, inter-track dependencies, signatures). See conventions-execution.md:orchestrator:3A,3B,3C §2.1 Section lifecycle for the per-section writer/reader matrix. Phase 2 sub-agents read every pending track's track file alongside the plan to verify pending-track descriptions; pass the absolute track-files directory path as the plan_dir argument on each sub-agent spawn.

---