pull-request-review

$npx mdskill add Significant-Gravitas/skills-catalog/pull-request-review

Use this for one pull request or diff at a time.

SKILL.md

.github/skills/pull-request-reviewView on GitHub ↗
---
name: "pull-request-review"
description: "Review a change for bugs and risk, with file, line, trigger, and confidence for each finding."
triggers: ["review this PR", "code review", "pull request review", "check my diff", "review my code"]
version: "1"
---

# Pull request review

Use this for one pull request or diff at a time.

## Read the change in context

Read the description and linked issue first, then the diff, then the code the
change calls and the code that calls it. Check the tests that cover it and
whether CI has run. Note what could not be read, such as a truncated diff or a
missing file.

## Findings by severity

Rank each finding as **blocker**, **should fix**, **question**, or **style**.
For each give:

- file and line;
- what goes wrong, in one sentence;
- the input or state that triggers it;
- confidence, and whether it comes from reading or from running the code;
- a suggested fix, if one is clear.

Keep style notes apart and short. Do not pad the list; say so when the change
looks sound.

## Questions and untested paths

List what the review could not settle: behaviour that depends on data or
config you cannot see, paths with no test, and assumptions the author should
confirm.

Never say code is safe or a test passed without seeing the evidence, and do
not invent a failure you cannot trace to a line. Do not approve, merge, or push
to the branch; the reviewer of record decides.

More from Significant-Gravitas/skills-catalog

SkillDescription
account-health-and-qbrsUse when a support account wobbles or a quarterly business review looms: read the health signals, run the success plan, and prep the review from evidence.
accounts-receivable-follow-upReview open receivables and draft factual, staged payment follow-ups without inventing status or contacting a customer.
ad-copy-variantsWrite ad variants that each test one idea, within platform limits and supported claims.
alex-getting-startedUse on the first conversation with Alex, or whenever their memory has no product preferences yet: learn what the user is building and who for, where specs, roadmap and numbers live, who decides dates and scope, and get them to a first real product deliverable.
alliance-co-commercializationUse when a strategic alliance needs joint selling governance: operating model, joint targeting, steering prep, and milestone accountability.
anika-getting-startedUse on the first conversation with Anika, or whenever their memory has no partnership preferences yet: learn which partners and alliances the user owns, what motion they run, and get one real partner read on screen in the same session.
assure-partner-led-deliveryUse when partners deliver client work in your name: own the in-flight book, run the weekly delivery review, and rescue engagements before clients feel it.
automate-finance-reportingUse to connect a number source, map an export into the finance ledger, or QA a sheet: the column mapping, the dedupe key, the load summary, and the checks that must pass before a read ships.
billing-refunds-and-exceptionsUse when money is on the table: verify the charge, check the policy, and stage a refund or exception draft that stops at the owner's yes.
board-and-investor-metrics-briefPrepare a concise board or investor metrics brief with definitions, sources, comparisons, drivers, risks, and decisions needed.