review-roadmap
Review Roadmap
Assess a thread-root roadmap.md as a decomposition handoff, strictly read-only. The one question you answer: could each child brief become an independently executable thread without inventing intent? You read the roadmap as the agent who will open and run each child would, judge it against the axes below, and record any findings for later attention. You never touch the roadmap or any other artifact, and you produce no durable report — a clean roadmap earns a chat judgment and nothing on disk.
This is a handoff-quality review: whether the decomposition, briefs, and dependencies are safe to hand off as written. You do not re-decide how the work should divide, and you do not confirm the roadmap faithfully carries each governing decision — that fidelity check is a separate operation.
Procedure
-
Resolve the thread. Work inside one thread root at
docs/threads/<YYMMDDHHMMSSZ-slug>/. Ifcwdalready sits inside a thread root, that is the thread. Two situations make a findings bundle physically impossible —.pending-reviews/would live inside the very thread that failed to resolve — so in both, refuse in chat, write nothing, and end withOutcome: REFUSED — <reason>: no thread exists yet, or several thread roots exist and which is active is ambiguous (never silently pick the most recent stamp). -
Read the roadmap read-only. The target is the thread-root
roadmap.md. Read it end to end at least once, as the agent who will materialize and execute each child with no memory of the conversation that produced it. If noroadmap.mdexists at the thread root, tell the user there is nothing to review, write nothing, and end withOutcome: REFUSED — no roadmap.md to review. You do not edit it, rewrite it, or propose edits into it — your output is a judgment, never a changed roadmap. -
Judge against the readiness axes. Assess the roadmap on each axis below (
## What you judge). For every real weakness, form a finding: what is wrong, where in the roadmap it shows, why it would force whoever opens a child to invent intent or hit a wall, and a severity —blocker(a child cannot be opened or run safely as briefed),issue(a real gap that will cause rework or a wrong guess), ornit(soft or imprecise, but survivable). Tether every finding to that downstream impact: "this brief is thin" is not a finding; "this brief omits its outcome, so whoever opens the child must guess what result it should produce" is. -
Decide the outcome. If the roadmap passes the bar — every child could be opened and run as an independent thread without inventing intent — the review is clean. Otherwise you hold one or more findings to record.
-
Report. A clean review returns a concise readiness judgment in chat and writes no file, ending
Outcome: DONE — <the readiness judgment>. A review with findings emits exactly one bundle (## Recording findings), reports its path, and endsOutcome: DONE — findings at <bundle path>. No preamble, no closing remark.