sdd: persist re-review verdicts into the task review artifact

This commit is contained in:
Drew Ritter
2026-07-20 02:44:13 -07:00
parent 3f28d9c943
commit 76c656a8b6

View File

@@ -347,10 +347,12 @@ capability bump in one move.
**Every round, either way:** the implementer fixes, re-runs the tests **Every round, either way:** the implementer fixes, re-runs the tests
covering the amended code, appends its fix report to the same report file, covering the amended code, appends its fix report to the same report file,
and returns the short contract. Before re-dispatching the reviewer, confirm and returns the short contract. Before re-dispatching the reviewer, confirm
the fix report contains the covering tests, the command run, and the the fix report contains the covering tests, the command run, and the output;
output; dispatch the re-review once all three are present. Name the dispatch the re-review once all three are present. Name the covering test
covering test files in the fix message — a one-line fix does not need the files in the fix message — a one-line fix does not need the whole suite.
whole suite. Append the re-review's returned text to `<workspace>/task-<N>-review.md` as
well — the artifact accumulates every verdict, and the file's last entry is
the one completion relies on.
**The re-review is scoped.** Run `scripts/review-package PLAN_FILE FIX_BASE HEAD` **The re-review is scoped.** Run `scripts/review-package PLAN_FILE FIX_BASE HEAD`
where FIX_BASE is the head the previous review saw, and dispatch where FIX_BASE is the head the previous review saw, and dispatch