fix(ce-plan): reinforce mandatory document-review after auto deepening (#450)

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Trevin Chow
2026-03-30 00:22:43 -07:00
committed by GitHub
parent a01a8aa0d2
commit 42fa8c3e08

View File

@@ -900,7 +900,7 @@ When presenting findings from multiple agents targeting the same section, presen
After all agents have been reviewed, carry only the accepted findings forward to 5.3.7.
If the user accepted no findings, report "No findings accepted — plan unchanged." If artifact-backed mode was used, clean up the scratch directory before continuing. Then proceed directly to Phase 5.4 (skip document-review and synthesis — the plan was not modified).
If the user accepted no findings, report "No findings accepted — plan unchanged." If artifact-backed mode was used, clean up the scratch directory before continuing. Then proceed directly to Phase 5.4 (skip document-review and synthesis — the plan was not modified). This interactive-mode-only skip does not apply in auto mode; auto mode always proceeds through 5.3.7 and 5.3.8.
If findings were accepted and the plan was modified, proceed through 5.3.7 and 5.3.8 as normal — document-review acts as a quality gate on the changes.
@@ -935,7 +935,7 @@ If research reveals a product-level ambiguity that should change behavior or sco
##### 5.3.8 Document Review
After the confidence check (and any deepening), run the `document-review` skill on the plan file. Pass the plan path as the argument.
After the confidence check (and any deepening), run the `document-review` skill on the plan file. Pass the plan path as the argument. When this step is reached, it is mandatory — do not skip it because the confidence check already ran. The two tools catch different classes of issues.
The confidence check and document-review are complementary:
- The confidence check strengthens rationale, sequencing, risk treatment, and grounding