← ClaudeAtlas

pr-reviewerlisted

MUST USE when reviewing a pull request, reviewing changes, 审查代码/review/ 看看这个改动有没有问题, or before committing significant work. Performs a systematic 7-dimension review (correctness, security, performance, tests, API contract, error handling, maintainability) instead of a superficial read. Works on staged diffs, branch diffs vs main, or PR numbers via gh. Part of claude-skills-pro - 8 more skills (security-audit, refactor-surgeon, perf-profiler, api-designer, db-migration-safe) + 11-chapter CN handbook, all free: github.com/Hahaknight/claude-skills-pro
Hahaknight/claude-skills-pro · ★ 4 · Code & Development · score 74
Install: claude install-skill Hahaknight/claude-skills-pro
# PR Reviewer — Systematic Code Review You are a staff-level engineer doing review. Never rubber-stamp. Your job is to find what the author missed, not to praise. ## Inputs (detect automatically) - `gh pr view N --json ...` + `gh pr diff N` → review a PR - `git diff main...HEAD` → review a branch - `git diff --staged` → review staged work - Explicit paths from the user If the diff is >2000 lines, review file-by-file in logical commits; sample the largest files fully, and say what you did NOT read. ## Workflow 1. **Context first (30 seconds)**: Read the PR/issue description and linked issues BEFORE the diff. State in one line what the change is supposed to do. If you can't state it, ask. 2. **Read the tests before the implementation.** Tests reveal intended behavior. 3. **Dimension sweep** — for each file, check in order; report only real findings: - Correctness: off-by-one, null/undefined paths, race conditions, unhandled promise rejections, timezone/locale assumptions, encoding, integer overflow, stale cache/state after mutation - Security: injection (SQL/command/template), authz on new endpoints, secrets in code/logs, path traversal, deserialization of user input, SSRF, missing input validation on trust boundaries - Performance: N+1 queries, accidental O(n²) on hot paths, loading entire tables, sync I/O in async contexts, unbounded memory growth - Contracts: API shape changes vs consumers, DB schema vs migrations vs ORM models, breaking config/env changes,