mcpField
P
pr-review

pr-review

github.com/tossTesting & quality11.4k193 installsUpdated today
Repository

About pr-review

A skill for reviewing recent open PRs in toss/es-toolkit, from triage through source-level assessment. It examines changes, tests, scope, and impact, then summarizes findings and verdicts across the PR set.

Key features

  1. 01Fetches and classifies open PRs
  2. 02Reads current source and tests
  3. 03Assesses changes and breaking impact
  4. 04Checks label-specific requirements
  5. 05Groups duplicates and summarizes verdicts

Use cases

  1. 01Reviewing a batch of recent pull requests
  2. 02Assessing a PR’s implementation and test coverage
  3. 03Finding overlapping changes across PRs

Install

Copy the command and run it in a terminal.

$npx skills add https://github.com/toss/es-toolkit --skill pr-review

After installing, type/pr-reviewin Claude Code to confirm it loaded.

Details

From the project README

PR Review

Fetch recent PRs, classify, read source code, and produce contextual reviews.

Input

$ARGUMENTS — Number of PRs (default: 10)

Examples:

  • /pr-review — 10 most recent open PRs
  • /pr-review 20 — 20 PRs

Workflow

1. Fetch recent PRs
gh pr list --repo toss/es-toolkit --state open --limit {count} --json number,title,author,labels,createdAt
2. Triage each PR

Run /pr-triage {number} for each PR to classify.

3. Deep review per PR

For each PR, go beyond classification — read the actual source code and provide context:

a. Read current code

Read the files being modified on main to understand the current behavior:

  • Function source: what it does, how it works
  • Existing tests: what's already covered
  • JSDoc: documented behavior and examples
b. Understand the PR change

Read the diff to understand what's changing:

  • What's the before vs after?
  • Why does the contributor say this change is needed?
  • Does the change match what the code actually needs?
c. Assess impact
  • Breaking changes: Does the function signature change? Will existing users be affected?
  • JS spec alignment: Does current behavior follow JS/lodash conventions? Does the PR break that?
  • Scope: Are all affected files updated? (e.g., docs in all 4 languages, related functions)
  • Design principles: Does this align with es-toolkit's 85% use case / simplicity philosophy?
d. Label-specific checks
  • compat-fix: Run /compat-review {number}
  • new-function: Check completeness (impl + spec + re-export + 4 lang docs)
  • core-change: yarn vitest run src/{category}/{fn}.spec.ts + review breaking impact
  • docs: Check all 4 language files updated consistently
  • types: tsc --noEmit
4. Detect duplicates

Group PRs by same function name or same files modified.

5. Report per PR

For each PR, output:

### PR #{number} — {title}

**Classification**: {labels}
**Changed files**: {summary}

**Current code**: {what the function does now, with code snippet}
**PR change**: {what's changing and why, with before/after}
**Context**: {JS spec, lodash behavior, design principle considerations}

**Checkpoints**:
- {specific things to verify}

**Verdict**: {merge / merge with changes / hold for discussion / reject}
6. Summary
## PR Review — {date}

| # | Title | Label | Verdict |
|---|-------|-------|---------|

- {N} ready to merge
- {N} need changes
- {N} need discussion
- {N} potential duplicates

Comments

Markdown supported