Loading source
Pulling the file list, source metadata, and syntax-aware rendering for this listing.
Source from repo
Anthropic-origin PR review toolkit with bundled agents and commands for portable installs across Codex, OpenClaw, and Claude workflows.
Files
Skill
Size
Entrypoint
Format
Open file
Syntax-highlighted preview of this file as included in the skill package.
commands/review-pr.md
1---2description: "Comprehensive PR review using specialized agents"3argument-hint: "[review-aspects]"4allowed-tools: ["Bash", "Glob", "Grep", "Read", "Task"]5---67# Comprehensive PR Review89Run a comprehensive pull request review using multiple specialized agents, each focusing on a different aspect of code quality.1011**Review Aspects (optional):** "$ARGUMENTS"1213## Review Workflow:14151. **Determine Review Scope**16- Check git status to identify changed files17- Parse arguments to see if user requested specific review aspects18- Default: Run all applicable reviews19202. **Available Review Aspects:**2122- **comments** - Analyze code comment accuracy and maintainability23- **tests** - Review test coverage quality and completeness24- **errors** - Check error handling for silent failures25- **types** - Analyze type design and invariants (if new types added)26- **code** - General code review for project guidelines27- **simplify** - Simplify code for clarity and maintainability28- **all** - Run all applicable reviews (default)29303. **Identify Changed Files**31- Run `git diff --name-only` to see modified files32- Check if PR already exists: `gh pr view`33- Identify file types and what reviews apply34354. **Determine Applicable Reviews**3637Based on changes:38- **Always applicable**: code-reviewer (general quality)39- **If test files changed**: pr-test-analyzer40- **If comments/docs added**: comment-analyzer41- **If error handling changed**: silent-failure-hunter42- **If types added/modified**: type-design-analyzer43- **After passing review**: code-simplifier (polish and refine)44455. **Launch Review Agents**4647**Sequential approach** (one at a time):48- Easier to understand and act on49- Each report is complete before next50- Good for interactive review5152**Parallel approach** (user can request):53- Launch all agents simultaneously54- Faster for comprehensive review55- Results come back together56576. **Aggregate Results**5859After agents complete, summarize:60- **Critical Issues** (must fix before merge)61- **Important Issues** (should fix)62- **Suggestions** (nice to have)63- **Positive Observations** (what's good)64657. **Provide Action Plan**6667Organize findings:68```markdown69# PR Review Summary7071## Critical Issues (X found)72- [agent-name]: Issue description [file:line]7374## Important Issues (X found)75- [agent-name]: Issue description [file:line]7677## Suggestions (X found)78- [agent-name]: Suggestion [file:line]7980## Strengths81- What's well-done in this PR8283## Recommended Action841. Fix critical issues first852. Address important issues863. Consider suggestions874. Re-run review after fixes88```8990## Usage Examples:9192**Full review (default):**93```94/pr-review-toolkit:review-pr95```9697**Specific aspects:**98```99/pr-review-toolkit:review-pr tests errors100# Reviews only test coverage and error handling101102/pr-review-toolkit:review-pr comments103# Reviews only code comments104105/pr-review-toolkit:review-pr simplify106# Simplifies code after passing review107```108109**Parallel review:**110```111/pr-review-toolkit:review-pr all parallel112# Launches all agents in parallel113```114115## Agent Descriptions:116117**comment-analyzer**:118- Verifies comment accuracy vs code119- Identifies comment rot120- Checks documentation completeness121122**pr-test-analyzer**:123- Reviews behavioral test coverage124- Identifies critical gaps125- Evaluates test quality126127**silent-failure-hunter**:128- Finds silent failures129- Reviews catch blocks130- Checks error logging131132**type-design-analyzer**:133- Analyzes type encapsulation134- Reviews invariant expression135- Rates type design quality136137**code-reviewer**:138- Checks CLAUDE.md compliance139- Detects bugs and issues140- Reviews general code quality141142**code-simplifier**:143- Simplifies complex code144- Improves clarity and readability145- Applies project standards146- Preserves functionality147148## Tips:149150- **Run early**: Before creating PR, not after151- **Focus on changes**: Agents analyze git diff by default152- **Address critical first**: Fix high-priority issues before lower priority153- **Re-run after fixes**: Verify issues are resolved154- **Use specific reviews**: Target specific aspects when you know the concern155156## Workflow Integration:157158**Before committing:**159```1601. Write code1612. Run: /pr-review-toolkit:review-pr code errors1623. Fix any critical issues1634. Commit164```165166**Before creating PR:**167```1681. Stage all changes1692. Run: /pr-review-toolkit:review-pr all1703. Address all critical and important issues1714. Run specific reviews again to verify1725. Create PR173```174175**After PR feedback:**176```1771. Make requested changes1782. Run targeted reviews based on feedback1793. Verify issues are resolved1804. Push updates181```182183## Notes:184185- Agents run autonomously and return detailed reports186- Each agent focuses on its specialty for deep analysis187- Results are actionable with specific file:line references188- Agents use appropriate models for their complexity189- All agents available in `/agents` list190