review-all

📁 acedergren/agentic-tools 📅 10 days ago
4
总安装量
4
周安装量
#51487
全站排名
安装命令
npx skills add https://github.com/acedergren/agentic-tools --skill review-all

Agent 安装分布

amp 4
gemini-cli 4
claude-code 4
github-copilot 4
codex 4
kimi-cli 4

Skill 文档

Review All

Comprehensive pre-PR review that runs specialized reviewers in parallel and synthesizes findings into a single report. Read-only — no changes.

When to Use

  • Before creating a PR
  • After completing a phase of work
  • When you want a full-spectrum code quality check beyond what linting covers

Pipeline

Step 1: Identify Changed Files

git diff --name-only main...HEAD

If on main, use git diff --name-only HEAD~5 (last 5 commits) or ask the user for the commit range.

Step 2: Launch Parallel Review Agents

Spawn agents simultaneously using the Task tool:

Agent Type Scope What it checks
Security Reviewer security-reviewer (custom) Changed files only OWASP Top 10, IDOR, injection, auth gaps
API Route Auditor Explore agent Routes + types dirs Schema coverage, type drift, auth hooks
Scope Auditor Explore agent git diff output Files modified outside task scope, formatting-only changes

Add project-specific reviewers as needed (e.g., database query reviewer, framework-specific reviewer).

Step 3: Synthesize Report

Combine all agent outputs into a single 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)
[List any CRITICAL/HIGH findings]

### Warnings (consider fixing)
[List MEDIUM/LOW findings]

### Clean Areas
[List what passed review with no issues]

Step 4: Verdict

End with a clear go/no-go:

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

Arguments

  • $ARGUMENTS: Optional scope or commit range
    • Example: /review-all — review changes vs main
    • Example: /review-all HEAD~3 — review last 3 commits
    • Example: /review-all --security-only — only security reviewer

Key Rules

  1. Read-only — do not modify any files
  2. Parallel execution — all agents run simultaneously for speed
  3. De-duplicate — if two agents flag the same line, merge into one finding
  4. No false positives — only report genuine issues with file:line references