Skills pr-reviewer
Automated GitHub PR code review with diff analysis, lint integration, and structured reports. Use when reviewing pull requests, checking for security issues, error handling gaps, test coverage, or code style problems. Supports Go, Python, and JavaScript/TypeScript. Requires `gh` CLI authenticated with repo access.
git clone https://github.com/openclaw/skills
T=$(mktemp -d) && git clone --depth=1 https://github.com/openclaw/skills "$T" && mkdir -p ~/.claude/skills && cp -r "$T/skills/briancolinger/pr-reviewer" ~/.claude/skills/clawdbot-skills-pr-reviewer && rm -rf "$T"
skills/briancolinger/pr-reviewer/SKILL.mdPR Reviewer
Automated code review for GitHub pull requests. Analyzes diffs for security issues, error handling gaps, style problems, and test coverage.
Prerequisites
CLI installed and authenticated (gh
)gh auth status- Repository access (read at minimum, write for posting comments)
- Optional:
for Go linting,golangci-lint
for Python lintingruff
Quick Start
# Review all open PRs in current repo scripts/github/pr-reviewer.sh check # Review a specific PR scripts/github/pr-reviewer.sh review 42 # Post review as GitHub comment scripts/github/pr-reviewer.sh post 42 # Check status of all open PRs scripts/github/pr-reviewer.sh status # List unreviewed PRs (useful for heartbeat/cron integration) scripts/github/pr-reviewer.sh list-unreviewed
Configuration
Set these environment variables or the script auto-detects from the current git repo:
— GitHub repo inPR_REVIEW_REPO
format (default: detected fromowner/repo
)gh repo view
— Local checkout path for lint (default: git root of cwd)PR_REVIEW_DIR
— State file path (default:PR_REVIEW_STATE
)./data/pr-reviews.json
— Report output directory (default:PR_REVIEW_OUTDIR
)./data/pr-reviews/
Directories Written
(default:PR_REVIEW_STATE
) — Tracks reviewed PRs and their HEAD SHAs./data/pr-reviews.json
(default:PR_REVIEW_OUTDIR
) — Markdown review reports./data/pr-reviews/
What It Checks
| Category | Icon | Examples |
|---|---|---|
| Security | 🔴 | Hardcoded credentials, AWS keys, secrets in code |
| Error Handling | 🟡 | Discarded errors (Go ), bare (Python), unchecked |
| Risk | 🟠 | calls, |
| Style | 🔵 | // in prod, very long lines |
| TODOs | 📝 | TODO, FIXME, HACK, XXX markers |
| Test Coverage | 📊 | Source files changed without corresponding test changes |
Smart Re-Review
Tracks HEAD SHA per PR. Only re-reviews when new commits are pushed. Use
review <PR#> to force re-review.
Report Format
Reports are saved as markdown files in the output directory. Each report includes:
- PR metadata (author, branch, changes)
- Commit list
- Changed file categorization by language/type
- Automated diff findings with file, line, category, and context
- Test coverage analysis
- Local lint results (when repo is checked out locally)
- Summary verdict: 🔴 SECURITY / 🟡 NEEDS ATTENTION / 🔵 MINOR NOTES / ✅ LOOKS GOOD
Heartbeat/Cron Integration
Add to a periodic check (heartbeat, cron job, or CI):
UNREVIEWED=$(scripts/github/pr-reviewer.sh list-unreviewed) if [ -n "$UNREVIEWED" ]; then scripts/github/pr-reviewer.sh check fi
Extending
The analysis patterns in the script are organized by language. Add new patterns by appending to the relevant pattern list in the
analyze_diff() function:
# Add a new Go pattern go_patterns.append((r'^\+.*os\.Exit\(', 'RISK', 'Direct os.Exit() — consider returning error'))