turnstile-review

Review

Process

1. Look up <id> (from $ARGUMENTS) in the current sprint -- turnstile/sprint.yml, or (legacy) the root turnstile/sprint-<N>.yml with sprint.status: active. Not found, or status not in_progress/review: refuse and say what to do instead (/turnstile:work <id> first; /turnstile:sprint-plan if no active sprint). status: review means a prior review session was interrupted -- say so and resume from step 3. Not a git repository, or no commits yet: tell the user review needs git history to scope the diff; suggest git init and an initial commit -- never review an unscoped file tree.
2. Set the item's status to review.
3. Read the ticket's criteria, first match wins (<n> from <id> = C-<n>): the spec turnstile/specs/SP-<n>.md; the solo-profile plan turnstile/plans/PL-<n>.md (its "## Acceptance criteria" section); legacy spec names (turnstile/specs/<id>-*-spec.md, turnstile/specs/<id>.md); the item note's "## Acceptance criteria" section (quick-lane items). This per-ticket fallback chain is what makes switching profile mid-project safe. Criteria in none of these: refuse -- the item cannot be reviewed.
4. Get the ticket's diff (git diff / git status against the last commit). Work enforces one in_progress item, so this diff should be exactly this ticket's changes; on unrelated uncommitted changes, stop and ask the user.
5. Dispatch the turnstile-reviewer agent, passing only the acceptance criteria and the diff -- no implementation-session narrative or reasoning.
6. FAIL verdict: set status back to in_progress, append the agent's per-criterion reasons to the item's notes, tell the user what needs to change. Do not commit.
7. PASS verdict:
   • Set the item's status to done.
   • If the item has a parent, check every child of that parent (across turnstile/backlog.yml, turnstile/sprint.yml, turnstile/sprints/*.yml; match parent: <parent-id>). All done (ignoring dropped): set the parent to done in turnstile/backlog.yml and tell the user; cascade the check one level up if that parent has a parent. This rollup is the one exception to "only the reviewer marks done" -- each child was individually reviewer-certified.
   • Capture (per the capture config): in gates mode (default), dispatch brain-curator once with bounded input -- the ticket (id, title), its acceptance criteria, and the diff -- to write any decision notes this ticket produced and create or update turnstile/code/ notes for the files the diff touched. Include the estimate-mismatch observation below in the same dispatch when it applies. In opportunistic mode, dispatch only for the mismatch case.
   • Estimate mismatch: compare points against the coarse actual-effort signals on hand -- carryovers and the count of "work pass" notes entries. carryovers > 0, or 3+ work passes against 3 points or less (or an equivalent clear mismatch), is a candidate process learning for brain-curator. Never claim wall-clock timing -- only these coarse counts.
   • Stage the changed files together with the updated sprint file (so the done status lands in the same commit as the implementation), plus turnstile/backlog.yml on a parent rollup, and commit: git commit -m "<verb>: <title> (<id>)" per the brain skill's convention (no Anthropic/Claude co-author tag, no --no-verify).
   • Tell the user the ticket is done and committed.