9700da9dc6
refactor(code-review): remove input_file_patterns from workflow.md to prevent context leak
2026-01-05 01:14:37 -08:00
ae9b83388c
refactor(code-review): reorder phases - adversarial first, context-aware second
...
- Swap step-03 and step-04: adversarial review now runs before context-aware
- Move discover_inputs from step-01 to step-04 (JIT loading)
- Add input_file_patterns to workflow.md frontmatter
- Adversarial runs lean (just diff + code), context-aware loads planning docs
2026-01-05 01:09:38 -08:00
64c32d8c8c
refactor(code-review): add web_bundle: false, use "Read and follow" wording
...
- Add web_bundle: false to frontmatter (workflow needs file access)
- Change "Load and execute" to "Read and follow" (clearer for LLMs)
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
2026-01-05 00:39:17 -08:00
eae4ad46a1
refactor(code-review): remove unused validation path and checklist
2026-01-04 21:38:04 -08:00
a8758b0393
refactor(code-review): remove CRITICAL DIRECTIVES, add communication_language
2026-01-04 21:32:10 -08:00
ac081a27e8
docs(code-review): clarify step file loading in workflow architecture
2026-01-04 21:15:04 -08:00
7c914ae8b2
refactor(code-review): inline single-use adversarial task path
2026-01-04 21:05:48 -08:00
25f93a3b64
refactor(code-review): simplify workflow.md
2026-01-04 20:59:58 -08:00
460c27e29a
refactor(code-review): convert to sharded format with dual-phase review
...
Convert monolithic code-review workflow to step-file architecture:
- workflow.md: Overview and initialization
- step-01: Load story and discover git changes
- step-02: Build review attack plan
- step-03: Context-aware review (validates ACs, audits tasks)
- step-04: Adversarial review (information-asymmetric diff review)
- step-05: Consolidate findings (merge + deduplicate)
- step-06: Resolve findings and update status
Key features:
- Dual-phase review: context-aware + context-independent adversarial
- Information asymmetry: adversarial reviewer sees only diff, no story
- Uses review-adversarial-general.xml via subagent (with fallbacks)
- Findings consolidation with severity (CRITICAL/HIGH/MEDIUM/LOW)
- State variables for cross-step persistence
🤖 Generated with [Claude Code](https://claude.com/claude-code )
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-04 01:07:58 -08:00
Alex Verkhovsky