Agent Skills
› NeverSight/learn-skills.dev
› review-all
review-all
GitHubPR提交前的综合代码审查工具，并行运行安全、API审计和范围检查代理，生成汇总报告。只读操作，用于评估代码质量并给出合并建议。
Trigger Scenarios
创建Pull Request前
完成工作阶段后
需要全面代码质量检查时
Install
npx skills add NeverSight/learn-skills.dev --skill review-all -g -y
SKILL.md
Frontmatter
{
"name": "review-all",
"description": "Pre-PR review pipeline — runs security, API audit, and scope check agents in parallel. Read-only, no changes. Use before creating PRs or after completing a phase of work."
}
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
- Example:
Key Rules
- Read-only — do not modify any files
- Parallel execution — all agents run simultaneously for speed
- De-duplicate — if two agents flag the same line, merge into one finding
- No false positives — only report genuine issues with file:line references
Version History
- e0220ca Current 2026-07-05 22:58
