Pr review
Open specification for packaging procedural knowledge for AI agents. Framework-agnostic SKILL.md format.
npx -y skills add choutos/agent-skills-spec --skill pr-reviewAssembled from the repository path, not quoted from the project. Check it against their README if it does not work.
One thing to look at
- 1 stars1 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.
What its author says it does
Copied from the file, not written here
Review pull requests for code quality, correctness, and maintainability. Use when asked to review a PR, check a diff, or provide code feedback. Triggers on: review, PR, pull request, code review, diff, merge request.
SKILL.md
1.4 KB, as published. Nobody here has run it
PR Review
Process
- Read the PR description and linked issue (if any)
- Review the diff file by file
- Provide feedback privately first (never post to GitHub/GitLab without explicit approval)
What to Check
Correctness
- Does the code do what the PR claims?
- Are edge cases handled?
- Could this break existing functionality?
Quality
- Clear naming and structure?
- Appropriate error handling?
- Tests added or updated?
- No hardcoded values that should be config?
Security
- No secrets in code?
- Input validation present?
- SQL injection / XSS risks?
Operations
- Will this affect deployment?
- Database migrations needed?
- Config changes required?
Feedback Format
For each issue found:
**[severity]** file:line — description
Suggestion: ...
Severity levels: critical (blocks merge), important (should fix), nit (optional improvement).
Rules
- Never post reviews publicly without explicit approval
- Acknowledge what's done well, not just problems
- Be specific: "line 42 has X" beats "the code has issues"
- Suggest fixes, don't just point out problems