feat: add architecture and file size checks to review loops

Spec reviewer now checks for unit decomposition with clear boundaries.
Plan reviewer now checks file structure and whether files will grow
too large to reason about.
This commit is contained in:
Jesse Vincent
2026-02-08 12:30:27 -08:00
parent 6ca430a0d3
commit b0519329df
2 changed files with 5 additions and 0 deletions

View File

@@ -23,6 +23,7 @@ Task tool (general-purpose):
| Consistency | Internal contradictions, conflicting requirements |
| Clarity | Ambiguous requirements |
| YAGNI | Unrequested features, over-engineering |
| Architecture | Units with clear boundaries, well-defined interfaces, independently understandable and testable |
## CRITICAL
@@ -30,6 +31,7 @@ Task tool (general-purpose):
- Any TODO markers or placeholder text
- Sections saying "to be defined later" or "will spec when X is done"
- Sections noticeably less detailed than others
- Units that lack clear boundaries or interfaces — can you understand what each unit does without reading its internals?
## Output Format