acedergren/agentic-tools/review-all

Review All

When to Use

Load this skill when the user request matches the frontmatter description for Review All.

Comprehensive pre-PR review: run specialized reviewers in parallel, synthesize into a single report. Read-only — no changes.

NEVER

• Never let any reviewer edit files during this pipeline — read-only is non-negotiable.
• Never report duplicate findings separately when two reviewers flag the same line — merge into one finding.
• Never review the whole repository when the user only changed a narrow diff — scope to changed files.
• Never use this as a substitute for lint, typecheck, or tests — it complements them, runs after them.
• Never run this for implementation tasks or auto-remediation requests — wrong tool.

Pipeline

Step 1: Identify changed files

git diff --name-only main...HEAD
# On main: git diff --name-only HEAD~5
# Or: bash scripts/detect-review-range.sh

Step 2: Launch parallel review agents

Spawn all agents simultaneously via Task tool:

Add project-specific reviewers as needed (DB query reviewer, framework reviewer).

Step 3: Synthesize report

## Pre-PR Review Report
 
### Summary
| Reviewer  | Findings | Critical | Warnings |
|-----------|----------|----------|----------|
| Security  | 2        | 0        | 2        |
| API Audit | 3        | 1        | 2        |
| Scope     | 1        | 0        | 1        |
 
### Critical Issues (must fix before merge)
[CRITICAL/HIGH findings with file:line references]
 
### Warnings (consider fixing)
[MEDIUM/LOW findings]
 
### Clean Areas
[What passed with no issues]

Step 4: Verdict

End with one clear statement:

• READY TO MERGE — No critical issues, warnings acceptable
• NEEDS FIXES — Critical issues found; list exactly what must change
• NEEDS DISCUSSION — Architectural concerns or ambiguous scope

Arguments

• (empty): Review changes vs main
• HEAD~3: Review last 3 commits
• --security-only: Only security reviewer