Review pr
Review a pull request for correctness, convention compliance, and test coverage. Use when you want to review a PR, check code quality, or get feedback on changes before merging.From its SKILL.md
npx -y skills add justnau1020/claude-os --skill review-prAssembled from the repository path, not quoted from the project. Check it against their README if it does not work.
2 things to look at
- 3 stars3 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.
- runs commandsInstructs the agent to run 3 commands, including `gh pr diff $ARGUMENTS` and 2 more.
SKILL.md
1.7 KB, 315 tokens by cl100k_base, as published. Nobody here has run it
Review Pull Request
Review PR #$ARGUMENTS for correctness and convention compliance.
PR Context
- PR diff: !
gh pr diff $ARGUMENTS 2>/dev/null || echo "Could not fetch PR diff" - PR description: !
gh pr view $ARGUMENTS 2>/dev/null || echo "Could not fetch PR" - Changed files: !
gh pr diff $ARGUMENTS --name-only 2>/dev/null || echo "Could not list files"
Review Checklist
For each changed file, verify:
Correctness
- Does the code do what the PR description claims?
- Are there any logic errors or edge cases missed?
- Are error paths handled properly?
Conventions
- Are datetimes handled consistently (UTC internally)?
- Are error messages specific and actionable?
- Is async used properly (no sync I/O in async paths)?
- Are type hints present on all new functions?
- Are typed models used for data structures (not raw dicts)?
Tests
- Are new features covered by tests?
- Are edge cases tested?
- Are external APIs mocked (no real network calls)?
Security
- Are authorization checks enforced on new endpoints?
- Are there any secrets or API keys in the diff?
- Is input validation present?
Output
Provide a structured review:
- Summary: One-line assessment (approve/request changes)
- Issues: Specific problems with file paths and line numbers
- Suggestions: Optional improvements (clearly marked as non-blocking)
What ships with it
Read from the repository
Just SKILL.md. No reference files, no scripts.