review-pr

review-pr is a skill for Claude Code from smartwhale8/claude-playbook. It costs 29 tokens per session (438 once invoked), scanned A, original, MIT.

A pull-request review checklist for examining code changes before they are merged. A pull request is a proposed set of changes submitted for review.

In plain words
What is it for?
It is used to inspect a pull request and its changes, check correctness and security, assess testing, and provide specific review feedback.
Why use it?
It helps catch incorrect behavior, security risks, unhandled edge cases, architectural problems, and missing tests.

Skill for Claude Code

Written for Claude Code: allowed-tools in frontmatter. Also seen: agent in frontmatter.

Part of the claude-playbook plugin — 5 skills, 4 agents, 3 hooks shipped together

Good fit It is used to inspect a pull request and its changes, check correctness and security, assess testing, and provide specific review feedback.

Compare 6 skills from other repositories ↓
Install with agentmods
npx agentmods add skills/smartwhale8/claude-playbook/review-pr
Install

Getting it into your agent

One page per mod, every tool's command on it. A separate URL per tool would split the same page into five that compete with each other.

Any agent
npx skills add smartwhale8/claude-playbook --skill review-pr
Clone the repo
git clone --depth 1 https://github.com/smartwhale8/claude-playbook

Made for: Claude Code.

Or install claude-playbook, the plugin that ships this one along with the rest of its 5 skills, 4 agents, 3 hooks.

Wrote this? Show the measurements

A badge with what this costs and how it scanned, read live from this page, so it follows the numbers instead of freezing them. Markdown for a README, HTML for a documentation site or a project page.

agentmods badge for review-pr

README.md
[![agentmods](https://agentmods.dev/badge/skills/smartwhale8/claude-playbook/review-pr/github.svg)](https://agentmods.dev/skills/smartwhale8/claude-playbook/review-pr)
Your own site
<a href="https://agentmods.dev/skills/smartwhale8/claude-playbook/review-pr"><img src="https://agentmods.dev/badge/skills/smartwhale8/claude-playbook/review-pr/github.svg" alt="Measured on agentmods" height="20"></a>

Or the 80×15 button, for a site that already has a row of RSS and ATOM ones. Only the verdict fits; the numbers stay here.

agentmods 80×15 button for review-pr

Your own site · 80×15
<a href="https://agentmods.dev/skills/smartwhale8/claude-playbook/review-pr"><img src="https://agentmods.dev/badge/skills/smartwhale8/claude-playbook/review-pr.svg" alt="Reviewed on agentmods" width="80" height="20"></a>
Per session 29 Skills are progressive disclosure: only the name and description are preloaded; the body loads when the skill is used.
When invoked 438 The whole file, excluding the scripts and references it only reads on demand.
Security scan A 0 findings. A grade says what 26 rules found in the file — not that it is safe.
Origin original No closer match found in the catalogue.
Token cost

What it costs to keep this loaded

Counted locally with the o200k_base tokenizer, which is exact for GPT models; Claude uses its own tokenizer and its counts differ. Treat this as one consistent yardstick across the catalogue rather than a bill. Prices are per million input tokens.

ModelPer sessionOnce invoked
Fable 5.1 $0.00029 $0.00438
Opus 5.5 $0.00012 $0.00175
Sonnet 5.5 $0.00006 $0.00088
Haiku 4.5 $0.00003 $0.00044

Measured 1mo ago against content hash e48115b1835c, method: parsed. Prices are Anthropic first-party input rates as of 2026-10-07, from the pricing page.

Security

Grade A, and why

review-pr scanned grade A with 0 findings against 26 rules in 11 categories — prompt injection, anti-refusal, data exfiltration, privilege escalation, supply chain, agent snooping, system-prompt leakage, SSRF and excessive agency — measured 1mo ago.

A static scan of the body, not an audit. Every finding is printed with the line that produced it so you can judge whether it matters here. A mod is markdown that instructs an agent; that is exactly why what it instructs is worth reading.

Nothing flagged

None of the 26 patterns this scan looks for appear in this file: no shell pipes, no recursive deletes, no credential paths, no hidden text, no instruction-override or anti-refusal phrasing, no agent-config snooping. That is not a guarantee, it is the absence of the things that are checkable.

.claude/skills/review-pr/SKILL.md · 43 lines

What it actually says

Review pull request $ARGUMENTS

The pull request

!gh pr view $ARGUMENTS 2>&1 || echo "Could not read PR $ARGUMENTS."

The diff

!gh pr diff $ARGUMENTS 2>&1 | head -1500 || true

What to check

  1. Intent. What is this pull request for? Does the diff do that, and only that?

  2. Correctness. Trace the changed paths. Unhandled edge cases, off-by-one errors, null and undefined paths, race conditions, error paths that cannot be reached.

  3. Fit. Does the change follow the patterns already in this codebase? Read the neighbouring files rather than assuming. Name any duplication with the path to the original. Are new dependencies justified?

  4. Security. Input validation at the boundary, parameterized queries, escaped output, authorization checks on new endpoints, no secrets in the diff.

  5. Tests. Is each new behaviour covered? Does a bug fix carry a regression test? Are the edge cases tested, or only the happy path?

  6. CI. Check gh pr checks $ARGUMENTS. Do not approve over a red build.

Output

Open with the verdict: approve, approve with comments, or request changes.

Then the findings, ordered by severity. Each one names the file and line, states the problem, and gives the change that resolves it.

Say plainly what you did not check, for example a subsystem you could not reach or a test you could not run. A review that hides its gaps is worse than a short one.

Changes

What this file has done since we first saw it

Hashed on every crawl. A supply-chain change to an agent config is a question of when, not whether, so the history is kept rather than the latest state alone.

  1. 1mo ago Changed · +14 lines · +15 tokens per session e48115b1835c
  2. 1mo ago First seen · 29 lines · 14 tokens per session scan A b5dff32c643f

Subscribe to this mod's changes

review-pr is a skill published in the GitHub repository smartwhale8/claude-playbook (43 stars, last pushed 1mo ago), licensed MIT. It adds 29 tokens to every session and 438 once invoked, about $0.0001 per session on Opus 5.5. A static security scan graded it A with 0 findings. No closer match exists in the catalogue, so it is treated as the original; first seen 2026-08-30.

Related

Other skills, from other repositories

rust-skills

Comprehensive Rust coding guidelines with 265 rules across 26 categories. Use when writing, reviewing, or refactoring Rust code. Covers ownership, error handling, async patterns, concurrency, unsafe code, API design, memory optimization, performance, numeric safety, conversions, serde, pattern matching, macros…

leonardomso/rust-skills · 84 tokens

ring:migrating-to-lib-observability

Migrating a Lerian Go app off lib-commons observability imports (deprecated shims or removed APIs) to lib-observability via a fixed mapping table, then bumps go.mod and validates the build; ring:backend-go applies the edits. Covers log/zap/runtime/assert, opentelemetry/tracing, HTTP middleware, context helpers, and…

LerianStudio/ring · 104 tokens

ring:using-lib-systemplane

Using lib-systemplane, the hot-reload runtime-config plane (Postgres LISTEN/NOTIFY or MongoDB change streams), in two modes. Sweep Mode detects DIY config reload (SIGHUP, fsnotify, viper, pgx LISTEN), manual tenant-scoping, hand-built admin CRUD, and v4 residue. Reference Mode catalogs client lifecycle and…

LerianStudio/ring · 106 tokens

ring:adopting-lib-commons-huma-wrapper

Adopting the lib-commons/v5 shared Huma (OAS 3.1) OpenAPI wrapper + RFC 9457 problem model (commons/net/http/{openapi,problem}) in a Lerian Go service: wire openapi.New/ServeSpec + problem.Install (central >=500 scrub) on BOTH runtime and spec-gen paths, the per-rail problem.MapError flex seam, and rename-only spec…

LerianStudio/ring · 142 tokens

ring:applying-composition-patterns

React composition patterns that scale. Avoid boolean prop proliferation by using compound components, lifting state, and composing internals. Use when refactoring components with boolean prop proliferation, building flexible component libraries, or during architecture review. Skip for simple components with 1-2 props…

LerianStudio/ring · 68 tokens

ring:using-lib-observability

Using lib-observability v1.1.0, Lerian's OpenTelemetry foundation (lib-commons, lib-systemplane, lib-streaming depend on it), in two modes. Sweep Mode detects DIY zap/slog logging, raw OTel metrics, hand-rolled redaction, and hard-coded attribute strings. Reference Mode catalogs the log, metrics, zap, redaction, and…

LerianStudio/ring · 102 tokens