| name | pr-review |
| description | Review the current Pull Request that has been checked out locally with structured feedback on code quality, issues, testing, and suggestions. Use when you need a comprehensive code review of a PR branch. |
| allowed-tools | Bash(git:*) Read Grep Glob |
| disable-model-invocation | true |
PR Review
Review the current Pull Request that has been checked out locally.
Instructions
-
Prerequisite
- Run
git fetch origin master to ensure we have the latest main branch
-
Get the PR changes
- Run
git diff --name-only $(git merge-base HEAD origin/master)..HEAD for a quick overview of changed files
- Run
git diff $(git merge-base HEAD origin/master)..HEAD to see all changes
- Run
git log --oneline $(git merge-base HEAD origin/master)..HEAD to see commit messages
-
Understand the context
- Examine the changed files to understand the PR's purpose
- Look for any related documentation or comments
-
Provide a structured code review
Summary
- Brief overview of what this PR accomplishes
- Number of files changed and scope
Code Quality
- Code organization and structure
- Readability and maintainability
- Adherence to best practices
- Error handling
Potential Issues
- Bugs or logic errors
- Unhandled edge cases
- Performance concerns
- Security vulnerabilities
- Concurrency issues
Testing
- Adequacy of tests
- Whether existing tests are updated
- Suggested additional test cases
Documentation
- Clarity of code comments
- Updated documentation (README, API docs)
- Explanation of complex logic
Suggestions
- Specific improvements with code examples
- Refactoring opportunities
- Alternative approaches
Verdict
- State one of: Approve, Request Changes, or Comment
- Briefly justify the verdict
-
Output Format
- Use markdown formatting
- Make it ready to paste as a PR comment
- Be constructive and educational
- Praise good practices while noting issues