Review pr
Review a pull request for what actually matters. Use when asked to review a PR or diff. A structured pass over correctness, boundaries, error handling, data, tests, performance, and security — findings ranked by severity.From its SKILL.md
npx -y skills add vaibhavsaxena022/skills --skill review-prAssembled from the repository path, not quoted from the project. Check it against their README if it does not work.
One thing to look at
- 0 stars0 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.
SKILL.md
1.4 KB, 225 tokens by cl100k_base, as published. Nobody here has run it
Review PR
Give a focused, useful review — not nitpicks.
Pass over
- Correctness — does it do what the PR claims? edge cases, off-by-one, null/empty, concurrency.
- Boundaries — input validation, trust boundaries, handling of failed external calls.
- Errors — right status codes, consistent shape, no internal leakage, nothing silently swallowed.
- Data — transactions/locking, N+1 queries, backward-compatible migrations.
- Tests — cover the new logic + a failure path; assert behavior, not internals.
- Security — authorization checks, injection, secrets, sensitive data in logs.
- Simplicity — dead code, needless abstraction, a clearer alternative.
Output
Findings ranked by severity (blocker → nit), each: file:line → issue → suggested fix. Note what's done well.
Rules
- Prioritize correctness and security over style.
- Be specific and actionable; cite
file:line. - Don't demand changes that don't matter.
What ships with it
Read from the repository
Just SKILL.md. No reference files, no scripts.