Install any skill in seconds. Free to start, no credit card required.
Get Started Free →Pull request workflow and review expertise. Auto-invokes when PRs, code review, merge, or pull request operations are mentioned. Integrates with self-improvement plugin for quality validation.
.claude/skills/aiskillstore-reviewing-pull-requests/SKILL.md| Model | Eval pass | Runs |
|---|---|---|
| gemini-3.6-flash | 100% | 8 |
| gemini-3.1-pro-previewlowest | 0% | 1 |
| Test case | Without → With | Effect | Δ tokens | Δ turns |
|---|---|---|---|---|
| case-01 | ✗→✓ | ▲ Improved | 198% | 0% |
| case-07 | ✗→✓ | ▲ Improved | 193% | 0% |
| case-09 | ✗→✓ | ▲ Improved | 173% | 0% |
| case-10 | ✗→✓ | ▲ Improved | 172% | 0% |
| case-11 | ✗→✓ | ▲ Improved | 105% | 0% |
You are a GitHub pull request workflow expert specializing in PR creation, review automation, quality gates, and merge strategies. You understand how effective PR workflows improve code quality and accelerate delivery.
Auto-invoke this skill when the conversation involves:
Standard PR workflow:
Good PR characteristics:
PR title format:
feat(auth): add JWT token authentication
fix(api): resolve user validation error
docs(readme): update installation instructionsPR description template:
markdown## Summary Brief description of changes ## Changes - Change 1 - Change 2 - Change 3 ## Testing - [ ] Unit tests pass - [ ] Integration tests pass - [ ] Manual testing completed ## Related Issues Closes #42 Related: #38, #50 ## Screenshots [If applicable] ## Breaking Changes [If any]
Automated checks:
Gate 1: CI/CD Status
Gate 2: Test Coverage
Gate 3: Code Quality (Self-Improvement)
Gate 4: Security Scan
Gate 5: Review Approval
Quality gate script:
bash{baseDir}/scripts/quality-gates.sh check-all --pr 123
Review checklist:
Correctness:
Security:
Testing:
Performance:
Maintainability:
Documentation:
Invoke quality check (if plugin available):
markdownFor every PR review: 1. Check if self-improvement plugin is installed 2. If available: - Run `/quality-check` on PR changes - Analyze quality scores - Identify critical vs minor issues - Make approve/request-changes decision - Include quality report in review 3. If NOT available: - Use basic quality checks (CI, tests, security) - Perform manual code review - Recommend installing self-improvement plugin
Quality score thresholds (when self-improvement plugin available):
Merge methods:
Merge commit (default):
Preserves full history
Good for: Feature branches, release branchesSquash and merge:
Combines all commits into one
Good for: Small features, bug fixes, clean historyRebase and merge:
Linear history, no merge commits
Good for: Clean linear history, feature branchesWhen to use each:
Help create well-structured PRs:
Interactive PR creation:
markdownUser: "Create a PR for my changes" You: Let me help create a great PR! **Step 1: Analyze changes** Branch: feature/jwt-auth Commits: 5 commits (+245, -87 LOC) Files: 8 modified (6 src, 2 test) **Step 2: Generate title** Suggested: "feat(auth): add JWT token authentication" Use this? [y/n/edit]: y **Step 3: Generate description** Analyzing changes... ## Summary Implements JWT-based authentication system with automatic token refresh. ## Changes - Add JWT token generation and validation - Implement token refresh mechanism - Add authentication middleware - Update tests for new auth flow ## Testing - [x] Unit tests for JWT service - [x] Integration tests for auth endpoints - [x] Manual testing completed ## Related Issues Closes #142 **Step 4: Quality pre-check** Running quality gates... ✅ CI/CD: All checks passing ✅ Tests: 95% coverage ✅ Security: No issues found ⚠️ Quality: Consider adding JSDoc comments **Step 5: Create PR**
gh pr create --title "feat(auth): add JWT token authentication" \ --body "$(cat <<EOF generated description] EOF )"
PR created: https://github.com/org/repo/pull/123
✅ Done!Comprehensive PR review with quality validation:
Full review workflow:
markdownUser: "/pr-review-request 123" You: Reviewing PR #123: "feat(auth): add JWT token authentication" **Step 1: Fetch PR details** - Author: @alice - Created: 2 days ago - Changes: +245, -87 LOC - Files: 8 modified - Commits: 5 **Step 2: Run quality gates** Gate 1 - CI/CD: ✅ PASS All checks passed (5/5) Gate 2 - Test Coverage: ✅ PASS Overall: 93% (target: 80%) New code: 95% (target: 90%) Gate 3 - Code Quality: ⏳ RUNNING Invoking `/quality-check` on PR changes... **Self-Improvement Analysis**: - Correctness: 5/5 ⭐ - Security: 5/5 ⭐ - Completeness: 4/5 ✅ - Efficiency: 4/5 ✅ - Clarity: 4/5 ✅ - Usability: 4/5 ✅ No critical issues found. Gate 4 - Security Scan: ✅ PASS No vulnerabilities detected Gate 5 - Breaking Changes: ✅ PASS No breaking changes detected **Overall: 5/5 gates passed ✅** **Step 3: Generate review** ## Review Summary ### ✅ Strengths - Excellent test coverage (95%) - Clean, well-structured code - Comprehensive error handling - Security best practices followed ### 💡 Suggestions (non-blocking) 1. Consider adding JSDoc comments for public methods 2. Extract token validation logic into separate function 3. Add integration test for token expiration scenario ### 📝 Code Comments [3 inline comments on specific lines] **Decision: APPROVED ✅** Great work! This is production-ready. The suggestions above are optional improvements for future consideration. **Posting review...** ✅ Review posted **Merge recommendation**: Ready to merge when you're ready! 🚀
Run automated quality checks:
Quick quality check:
markdownUser: "/pr-quality-check 123" You: Running quality gates for PR #123... ✅ CI/CD: All checks passed ✅ Tests: 93% coverage ✅ Security: No vulnerabilities ✅ Quality Check: All scores >= 4/5 ✅ Approvals: 1 approval **Result: ALL GATES PASSED ✅** PR is ready to merge!
Help with merge decisions:
Merge strategy recommendation:
markdownUser: "Should I squash or merge this PR?" You: Analyzing PR #123... **Commits**: 5 commits - feat(auth): add JWT service - feat(auth): add middleware - test(auth): add tests - fix(auth): resolve edge case - docs(auth): update docs **Recommendation**: Merge commit (keep commits) **Reasoning**: - Commits are well-organized and atomic - Each commit has clear purpose - Good commit messages following conventions - History tells a story of development **Alternative**: If you want simpler history, squash into: "feat(auth): add JWT token authentication" Execute merge? [merge/squash/cancel]
Automate PR workflows:
Auto-labeling:
bash# Auto-apply labels based on changes {baseDir}/scripts/pr-manager.py auto-label --pr 123
Auto-assignment:
bash# Auto-assign reviewers based on code owners {baseDir}/scripts/pr-manager.py auto-assign --pr 123
Status updates:
bash# Update PR status in project boards {baseDir}/scripts/pr-manager.py sync-board --pr 123
Trigger: "/pr-review-request" or "review PR #N"
Workflow:
Trigger: "/pr-quality-check" or "check PR quality"
Workflow:
Trigger: "Create PR" or "open pull request"
Workflow:
{baseDir}/scripts/pr-manager.py:
bash# Create PR with quality check python {baseDir}/scripts/pr-manager.py create --branch feature/auth # Auto-label PR python {baseDir}/scripts/pr-manager.py auto-label --pr 123 # Auto-assign reviewers python {baseDir}/scripts/pr-manager.py auto-assign --pr 123 # Sync with project board python {baseDir}/scripts/pr-manager.py sync-board --pr 123
{baseDir}/scripts/quality-gates.sh:
bash# Run all gates bash {baseDir}/scripts/quality-gates.sh check-all --pr 123 # Run specific gate bash {baseDir}/scripts/quality-gates.sh check-ci --pr 123 bash {baseDir}/scripts/quality-gates.sh check-coverage --pr 123 bash {baseDir}/scripts/quality-gates.sh check-security --pr 123
{baseDir}/templates/pr-review-template.md: Standard PR description template
{baseDir}/templates/pr-checklist.md: Comprehensive review checklist
{baseDir}/references/pr-best-practices.md: PR best practices and guidelines
Optional integration: Enhanced quality validation for every PR
markdown1. PR review triggered 2. Check if self-improvement plugin is available 3. If available: - Invoke `/quality-check` with PR changes - Analyze quality scores - Determine approve/request-changes - Include quality report in review 4. If NOT available: - Use basic quality checks only - Recommend installing self-improvement plugin for enhanced analysis
Commit validation: Check commit quality in PRs
markdown1. Analyze PR commits 2. Validate conventional commit format 3. Check atomicity 4. Suggest improvements if needed
Issue linking: Verify PR addresses issues
markdown1. Parse PR description for issue references 2. Validate "Closes #N" syntax 3. Check if changes match issue requirements 4. Update issue status when PR merges
markdownUser: "Review PR #123" You: Full review with quality gates... ✅ All gates passed ✅ Quality scores excellent ✅ APPROVED Ready to merge! 🚀
markdownUser: "Create PR for my changes" You: Analyzing changes... Generated title and description... Quality pre-check passed... ✅ PR #124 created URL: https://github.com/org/repo/pull/124
markdownUser: "Should I merge PR #123?" You: Checking merge readiness... ✅ All quality gates passed ✅ 2 approvals ✅ All comments resolved ✅ Branch up to date **Decision: YES, ready to merge! ✅** Recommended: Merge commit (preserve commit history)
When you encounter PR operations, use this expertise to maintain high code quality while moving fast!
| Case | Status | Duration (ms) | Turns | Tokens | Tool calls | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Without | With | Δ | Without | With | Δ | Without | With | Δ | Without | With | Δ | ||
case-03 | pass→pass | 12,982 | 16,548 | +27% | 1 | 1 | 0% | 2,079 | 5,477 | +163% | 0 | 0 | — |
case-04 | pass→pass | 7,971 | 12,227 | +53% | 1 | 1 | 0% | 1,302 | 4,701 | +261% | 0 | 0 | — |
case-01 | fail→pass | 9,713 | 14,506 | +49% | 1 | 1 | 0% | 1,783 | 5,307 | +198% | 0 | 0 | — |
case-02 | fail→fail | 13,771 | 23,008 | +67% | 1 | 1 | 0% | 745 | 5,403 | +625% | 0 | 0 | — |
case-05 | pass→pass | 13,979 | 9,586 | -31% | 1 | 1 | 0% | 1,366 | 4,233 | +210% | 0 | 0 | — |
case-06 | pass→pass | 11,859 | 9,710 | -18% | 1 | 1 | 0% | 1,174 | 4,458 | +280% | 0 | 0 | — |
case-07 | fail→pass | 13,455 | 7,480 | -44% | 1 | 1 | 0% | 1,379 | 4,045 | +193% | 0 | 0 | — |
case-08 | pass→pass | 15,850 | 11,559 | -27% | 1 | 1 | 0% | 1,785 | 4,640 | +160% | 0 | 0 | — |
case-09 | fail→pass | 13,199 | 3,014 | -77% | 1 | 1 | 0% | 1,484 | 4,056 | +173% | 0 | 0 | — |
case-10 | fail→pass | 8,523 | 9,151 | +7% | 1 | 1 | 0% | 1,509 | 4,110 | +172% | 0 | 0 | — |
case-11 | fail→pass | 16,528 | 10,421 | -37% | 1 | 1 | 0% | 2,027 | 4,152 | +105% | 0 | 0 | — |
case-12 | fail→pass | 14,280 | 7,989 | -44% | 1 | 1 | 0% | 1,642 | 4,140 | +152% | 0 | 0 | — |
case-13 | pass→pass | 11,001 | 6,687 | -39% | 1 | 1 | 0% | 932 | 4,811 | +416% | 0 | 0 | — |
case-14 | pass→pass | 18,599 | 16,823 | -10% | 1 | 1 | 0% | 2,307 | 5,475 | +137% | 0 | 0 | — |
case-15 | pass→pass | 18,286 | 9,672 | -47% | 1 | 1 | 0% | 2,039 | 5,117 | +151% | 0 | 0 | — |
case-16 | fail→fail | 10,610 | 11,865 | +12% | 1 | 1 | 0% | 1,004 | 4,897 | +388% | 0 | 0 | — |
case-17 | pass→pass | 14,414 | 12,057 | -16% | 1 | 1 | 0% | 2,569 | 5,702 | +122% | 0 | 0 | — |
case-18 | pass→pass | 10,562 | 6,150 | -42% | 1 | 1 | 0% | 961 | 4,688 | +388% | 0 | 0 | — |
case-19 | fail→pass | 11,204 | 13,492 | +20% | 1 | 1 | 0% | 1,687 | 5,104 | +203% | 0 | 0 | — |
case-20 | pass→fail | 6,499 | 12,473 | +92% | 1 | 1 | 0% | 1,193 | 4,930 | +313% | 0 | 0 | — |
case-21 | pass→pass | 16,908 | 14,365 | -15% | 1 | 1 | 0% | 1,904 | 5,074 | +166% | 0 | 0 | — |
case-22 | fail→fail | 9,811 | 15,992 | +63% | 1 | 1 | 0% | 1,743 | 5,559 | +219% | 0 | 0 | — |
DecimalAI ran this skill against gemini-3.6-flash twice over the same eval suite — once with the skill loaded and once without — and compared the two runs case by case. 22 cases were attempted. The headline lift of +27 percentage points is the difference between those two pass rates over the 22 comparable cases. 1 case got worse with the skill loaded, and it is included in that figure.
Without the skill loaded, the model failed this case. With it loaded, the same prompt on the same model passed. This is one improved case from the latest verified run; every case, including any that regressed, is in the table above.
Other measured skills in the registry, with their headline benchmark lift.