# Pr Review

> Use to run the pre-merge pull request review checklist so required validations are never skipped. Use when reviewing or approving a pull request before merge. Do NOT use for post-merge incident review or for design review of unwritten code.

- Skill: `bonaniibm/pr-review` (Agent Skill)
- Install (CLI): `npx skillmds@latest add bonaniibm/pr-review`
- Raw SKILL.md: https://api.skillmd.com/api/skills/bonaniibm/pr-review/raw
- Safety review: pending
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: Coding & Dev Tools
- License: MIT
- Author: bonaniibm (https://skillmd.com/u/bonaniibm)
- Updated: 2026-09-22
- Page: https://skillmd.com/skills/bonaniibm/pr-review

---


# Pull Request Review Checklist

## When to use
Run before approving any pull request for merge.

## Checklist
- [ ] Tests cover the change — MUST pass.
- [ ] No secrets or credentials in the diff — MUST pass.
- [ ] Public API changes are documented — MUST pass.
- [ ] CI is green — MUST pass.

## Rules
- NEVER approve with failing CI.
- ALWAYS block on a missing test for new behavior.

## Example
A PR adds a new endpoint but no tests. Block on the missing-test item and request coverage before approving.