Skill Tracer
Skills › Superpowers › subagent-driven-development › final-review.md

final-review.md

47 lines · skills/subagent-driven-development/references/final-review.md at 2ce8aa1

Open on GitHub

Most marks come from matching words in the file, so treat them as pointers.

1# Final Review and Finish
2
3Part of superpowers:subagent-driven-development. Every task in the plan is
4complete.
5
6The final whole-branch review gets a package too: run
7`bash scripts/review-package PLAN_FILE MERGE_BASE HEAD` (MERGE_BASE = the commit the
8branch started from, e.g. `git merge-base main HEAD`) and include the
9printed path in the final review dispatch, so the final reviewer reads
10one file instead of re-deriving the branch diff with git commands. Dispatch
11on the most capable available model (see SKILL.md's Model Selection), using
13[code-reviewer.md](../../requesting-code-review/code-reviewer.md). Point it at
14the ledger's deferred-minor and parked lines so it can triage which must be
15fixed before merge.
17If the final whole-branch review returns findings, dispatch ONE fix subagent
18with the complete findings list — not one fixer per finding.
19Per-finding fixers each rebuild context and re-run suites; a real
20session's final-review fix wave cost more than all its tasks combined.
21Then run exactly one scoped re-review of the fix wave
22(`bash scripts/review-package PLAN_FILE FIX_BASE HEAD` over the fix range,
23[re-review-prompt.md](../re-review-prompt.md)).
24Adjudicate any residual findings as in the fix loop's breaker
25(references/fix-loop.md): park with rulings, or rule on the load-bearing
26ones and ledger what you decided. Only the four things that stop a running
27plan (top of SKILL.md) stop you here. There is no second fix wave —
28residual load-bearing findings surface to your human partner when
29finishing-a-development-branch presents the options.

Final Review and Finish

Part of superpowers:subagent-driven-development. Every task in the plan is complete.

The final whole-branch review gets a package too: run bash scripts/review-package PLAN_FILE MERGE_BASE HEAD (MERGE_BASE = the commit the branch started from, e.g. git merge-base main HEAD) and include the printed path in the final review dispatch, so the final reviewer reads one file instead of re-deriving the branch diff with git commands. Dispatch on the most capable available model (see SKILL.md's Model Selection), using superpowers:requesting-code-review's code-reviewer.md. Point it at the ledger's deferred-minor and parked lines so it can triage which must be fixed before merge.

If the final whole-branch review returns findings, dispatch ONE fix subagent with the complete findings list — not one fixer per finding. Per-finding fixers each rebuild context and re-run suites; a real session's final-review fix wave cost more than all its tasks combined. Then run exactly one scoped re-review of the fix wave (bash scripts/review-package PLAN_FILE FIX_BASE HEAD over the fix range, re-review-prompt.md). Adjudicate any residual findings as in the fix loop's breaker (references/fix-loop.md): park with rulings, or rule on the load-bearing ones and ledger what you decided. Only the four things that stop a running plan (top of SKILL.md) stop you here. There is no second fix wave — residual load-bearing findings surface to your human partner when finishing-a-development-branch presents the options.

Raw lines 1 to 30
Starts an agent subagent :3
Runs code scripts/review-package :7
Starts an agent dispatch :9
Starts an agent Dispatch :10
Names a skill requesting-code-review :12
31### Finish
33Before you delete anything, collect every ledger line containing `Ruling:` —
34preflight rulings, parked findings, breaker adjudications, all of them — into
35your final message under "Rulings I made", in the order you made them, each
36with what it costs if wrong. The list is exhaustive: if the ledger holds a
37ruling, the list holds it. That list is the only place the decisions you
38took on your human partner's behalf reach them — they read it and rework
39whatever you got wrong. A ruling that dies with the workspace was a decision
40made in secret.
42When the final whole-branch review is clean and its fixes are merged,
43delete this plan's workspace (`rm -rf <workspace>`) — the git history is
44the record now. Sibling directories belong to other plans; leave them
45alone.

Finish

Before you delete anything, collect every ledger line containing Ruling: — preflight rulings, parked findings, breaker adjudications, all of them — into your final message under "Rulings I made", in the order you made them, each with what it costs if wrong. The list is exhaustive: if the ledger holds a ruling, the list holds it. That list is the only place the decisions you took on your human partner's behalf reach them — they read it and rework whatever you got wrong. A ruling that dies with the workspace was a decision made in secret.

When the final whole-branch review is clean and its fixes are merged, delete this plan's workspace (rm -rf <workspace>) — the git history is the record now. Sibling directories belong to other plans; leave them alone.

Use superpowers:finishing-a-development-branch.

Raw lines 31 to 47