spacehub

surshar / claude-code

Claude Code is an agentic coding tool that lives in your terminal, understands your codebase, and helps you code faster by executing routine tasks, explaining complex code, and handling git workflows - all through natural language commands.

Built by Anthropic · github.com/anthropics

claude-code / plugins / pr-review-toolkit / commands / review-pr.md
189 lines · 4.9 KBPreviewCodeRaw

description: "Comprehensive PR review using specialized agents" argument-hint: "[review-aspects]"

allowed-tools: ["Bash", "Glob", "Grep", "Read", "Task"]

Comprehensive PR Review

Run a comprehensive pull request review using multiple specialized agents, each focusing on a different aspect of code quality.

Review Aspects (optional): "$ARGUMENTS"

Review Workflow:

  1. Determine Review Scope
  2. Check git status to identify changed files
  3. Parse arguments to see if user requested specific review aspects
  4. Default: Run all applicable reviews
  1. Available Review Aspects:
  • comments - Analyze code comment accuracy and maintainability
  • tests - Review test coverage quality and completeness
  • errors - Check error handling for silent failures
  • types - Analyze type design and invariants (if new types added)
  • code - General code review for project guidelines
  • simplify - Simplify code for clarity and maintainability
  • all - Run all applicable reviews (default)
  1. Identify Changed Files
  2. Run git diff --name-only to see modified files
  3. Check if PR already exists: gh pr view
  4. Identify file types and what reviews apply
  1. Determine Applicable Reviews

Based on changes:

  • Always applicable: code-reviewer (general quality)
  • If test files changed: pr-test-analyzer
  • If comments/docs added: comment-analyzer
  • If error handling changed: silent-failure-hunter
  • If types added/modified: type-design-analyzer
  • After passing review: code-simplifier (polish and refine)
  1. Launch Review Agents

Sequential approach (one at a time):

  • Easier to understand and act on
  • Each report is complete before next
  • Good for interactive review

Parallel approach (user can request):

  • Launch all agents simultaneously
  • Faster for comprehensive review
  • Results come back together
  1. Aggregate Results

After agents complete, summarize:

  • Critical Issues (must fix before merge)
  • Important Issues (should fix)
  • Suggestions (nice to have)
  • Positive Observations (what's good)
  1. Provide Action Plan

Organize findings:

   # PR Review Summary

   ## Critical Issues (X found)
   - [agent-name]: Issue description [file:line]

   ## Important Issues (X found)
   - [agent-name]: Issue description [file:line]

   ## Suggestions (X found)
   - [agent-name]: Suggestion [file:line]

   ## Strengths
   - What's well-done in this PR

   ## Recommended Action
   1. Fix critical issues first
   2. Address important issues
   3. Consider suggestions
   4. Re-run review after fixes

Usage Examples:

Full review (default):

/pr-review-toolkit:review-pr

Specific aspects:

/pr-review-toolkit:review-pr tests errors
# Reviews only test coverage and error handling

/pr-review-toolkit:review-pr comments
# Reviews only code comments

/pr-review-toolkit:review-pr simplify
# Simplifies code after passing review

Parallel review:

/pr-review-toolkit:review-pr all parallel
# Launches all agents in parallel

Agent Descriptions:

comment-analyzer:

  • Verifies comment accuracy vs code
  • Identifies comment rot
  • Checks documentation completeness

pr-test-analyzer:

  • Reviews behavioral test coverage
  • Identifies critical gaps
  • Evaluates test quality

silent-failure-hunter:

  • Finds silent failures
  • Reviews catch blocks
  • Checks error logging

type-design-analyzer:

  • Analyzes type encapsulation
  • Reviews invariant expression
  • Rates type design quality

code-reviewer:

  • Checks CLAUDE.md compliance
  • Detects bugs and issues
  • Reviews general code quality

code-simplifier:

  • Simplifies complex code
  • Improves clarity and readability
  • Applies project standards
  • Preserves functionality

Tips:

  • Run early: Before creating PR, not after
  • Focus on changes: Agents analyze git diff by default
  • Address critical first: Fix high-priority issues before lower priority
  • Re-run after fixes: Verify issues are resolved
  • Use specific reviews: Target specific aspects when you know the concern

Workflow Integration:

Before committing:

1. Write code
2. Run: /pr-review-toolkit:review-pr code errors
3. Fix any critical issues
4. Commit

Before creating PR:

1. Stage all changes
2. Run: /pr-review-toolkit:review-pr all
3. Address all critical and important issues
4. Run specific reviews again to verify
5. Create PR

After PR feedback:

1. Make requested changes
2. Run targeted reviews based on feedback
3. Verify issues are resolved
4. Push updates

Notes:

  • Agents run autonomously and return detailed reports
  • Each agent focuses on its specialty for deep analysis
  • Results are actionable with specific file:line references
  • Agents use appropriate models for their complexity
  • All agents available in /agents list