Agent skill
review-pr
Use this skill when the user wants to review a pull request, check a PR for issues, or get feedback on a GitHub PR. Triggered by "review PR", "check PR", "review pull request", or "/review-pr <number>".
Install this agent skill to your Project
npx add-skill https://github.com/majiayu000/claude-skill-registry/tree/main/skills/other/other/review-pr-imran-ml-claude-skills
SKILL.md
Pull Request Review
Perform a thorough code review of the pull request.
PR Details
!gh pr list --limit 5
Instructions
-
Fetch the PR diff:
bashgh pr diff $ARGUMENTSOr if no number given, use the current branch:
bashgit diff main...HEAD -
Also fetch PR metadata:
bashgh pr view $ARGUMENTS -
Review the following areas:
Code Quality
- Logic errors or edge cases
- Code duplication (DRY violations)
- Overly complex functions (> 50 lines)
- Unused imports, variables, dead code
Security
- Hardcoded secrets or credentials
- SQL injection, XSS, command injection risks
- Improper authentication/authorization
- Insecure dependencies
Performance
- N+1 query problems
- Unnecessary re-renders (React)
- Missing indexes for new queries
- Large synchronous operations
Testing
- Are new features tested?
- Are edge cases covered?
- Are tests actually testing the right thing?
Documentation
- Are public APIs documented?
- Is the PR description clear?
- Are breaking changes noted?
Output Format
Provide a structured review:
## Summary
[1-2 sentence overview]
## ✅ What's Good
- [positive point]
## ⚠️ Issues
### Critical (must fix)
- [file:line] — [description]
### Important (should fix)
- [file:line] — [description]
### Minor (consider fixing)
- [file:line] — [description]
## 💡 Suggestions
- [optional improvements]
## Verdict: APPROVE / REQUEST CHANGES / COMMENT
$ARGUMENTS
Recommended Agent Skills
Expand your agent's capabilities with these related and highly-rated skills.
agent-ops-spec
Manage specification documents in .agent/specs/. Use when user provides requirements, acceptance criteria, or feature descriptions that need to be tracked and validated against implementation.
agent-ops-state
Maintain .agent state files. Use at session start, after meaningful steps, and before concluding: read/update constitution/memory/focus/issues/baseline consistently.
agent-ops-spec
Manage specification documents in .agent/specs/. Use when user provides requirements, acceptance criteria, or feature descriptions that need to be tracked and validated against implementation.
agent-ops-testing
Test strategy, execution, and coverage analysis. Use when designing tests, running test suites, or analyzing test results beyond baseline checks.
agent-ops-testing
Test strategy, execution, and coverage analysis. Use when designing tests, running test suites, or analyzing test results beyond baseline checks.
agent-ops-state
Maintain .agent state files. Use at session start, after meaningful steps, and before concluding: read/update constitution/memory/focus/issues/baseline consistently.
Didn't find tool you were looking for?