Loading...
Loading...
Use when reviewing pull requests with comprehensive code analysis, incremental or full review options, and constructive feedback - provides thorough code reviews with severity ratings
npx skill4agent add microck/ordinary-claude-skills reviewing-pr# PR Info
gh pr view <number> # View PR details
gh pr view <number> --json number,title,body,files # Get PR metadata
gh pr diff <number> # Get full PR diff
# Comments
gh pr view <number> --comments # View existing comments
gh api repos/{owner}/{repo}/pulls/<number>/comments # Get inline comments
gh api repos/{owner}/{repo}/issues/<number>/comments # Get issue comments
gh pr comment <number> --body "" # Post comment
# Review Actions
gh pr review <number> --approve --body "" # Approve PR
gh pr review <number> --request-changes --body "" # Request changes
gh pr review <number> --comment --body "" # Comment without approval
# Git Commands
git diff HEAD~1..HEAD # Last commit diff
git rev-parse HEAD # Get commit SHA
git log -1 --pretty=%s # Last commit message
git log --oneline -n 5 # Recent commitspr_numberincremental# Get PR info
gh pr view <pr_number> --json number,title,body,files
# Pull ALL comments (always do this first)
gh pr view <pr_number> --comments
gh api repos/{owner}/{repo}/pulls/<pr_number>/comments
# Get appropriate diff
if incremental:
git diff HEAD~1..HEAD # Latest commit only
else:
gh pr diff <pr_number> # Full PR diffgh pr comment <pr_number> --body "$(cat <<'EOF'
## 🔄 Incremental Review - Latest Changes
**Commit**: $(git rev-parse --short HEAD) - $(git log -1 --pretty=%s)
**Scope**: [Files changed in this commit only]
### ✅ What's Good
[Positive aspects of the changes]
### 📝 Review Findings
#### 🚨 Critical Issues
[Security vulnerabilities, data loss risks, breaking changes]
#### ⚠️ Major Issues
[Performance problems, logic errors, architectural concerns]
#### 📝 Minor Issues
[Code style, missing docs, naming conventions]
#### 💡 Suggestions
[Optional improvements, refactoring opportunities]
### Recommendations
[Specific next steps if any issues found]
### Status
✅ Changes approved / ⚠️ Minor suggestions / 🚨 Issues to address
*Reviewed: $(git rev-parse HEAD)*
EOF
)"gh pr comment <pr_number> --body "$(cat <<'EOF'
## 🔍 Code Review: PR #<pr_number>
### 📊 Overview
**Files Changed**: [X files]
**Lines**: +[additions] -[deletions]
[High-level summary of the PR's purpose and approach]
### ✅ Strengths
[What the PR does well]
### 📝 Review Findings
#### 🚨 Critical Issues
[Security vulnerabilities, data loss risks, breaking changes]
#### ⚠️ Major Issues
[Performance problems, logic errors, architectural concerns]
#### 📝 Minor Issues
[Code style, missing docs, naming conventions]
#### 💡 Suggestions
[Optional improvements, refactoring opportunities]
### 📚 Documentation
[Comments on docs, README updates, API changes]
### 🧪 Testing
[Test coverage, test quality, missing test cases]
### Recommendations
1. [Specific actionable feedback]
2. [Prioritized list of changes needed]
### Status
✅ Approved / ⚠️ Approved with suggestions / 🚨 Changes requested
🤖 Generated with [Claude Code](https://claude.ai/code)
EOF
)"# Approve if all good
gh pr review <pr_number> --approve --body "LGTM! [summary]"
# Request changes if critical/major issues
gh pr review <pr_number> --request-changes --body "[summary of required changes]"
# Comment only for suggestions
gh pr review <pr_number> --comment --body "[suggestions without blocking]"
# Add labels
gh pr edit <pr_number> --add-label "needs-review"
gh pr edit <pr_number> --add-label "approved"# Review latest commit only
incremental=true
pr_number=123
# Quick focused review of new changes
gh pr diff HEAD~1..HEAD
# Post incremental review comment# Full PR review
incremental=false
pr_number=123
# Analyze entire diff
gh pr diff 123
# Check test coverage
# Review documentation
# Post comprehensive review# Check what was previously requested
gh pr view 123 --comments
# Review new commits since last review
git log --oneline -n 5
# Verify issues are resolved
# Post follow-up review