Decide reviewer strategy based on diff size and complexity
mainUse the following rubric to determine how many reviewers to spawn:
- Skip Review: If the change is trivial (comments, formatting, typos, or single-line fixes with no logic effect). Tell the user: "Change is trivial, skipping review."
- Single Reviewer: For changes under ~50 lines. Use the Single Reviewer template (covers both Bug Hunter and Rules Auditor checklists).
- Parallel Specialists (2 Reviewers): For changes with 50+ lines or spanning 2+ modules. Spawn a Bug Hunter (correctness + security) and a Rules Auditor (project rules + tests) in parallel.
- Architect (Optional 3rd Reviewer): Add an Architect if the change is design-significant, such as:
- Spanning 2+ modules/components.
- Introducing a new abstraction or API contract.
- Unfinished refactors/migrations or file moves.
- Evidence of scope creep.