# Pr Checklist

> This skill should be used when creating or reviewing a pull request, when asked if a PR is ready to merge, when preparing a PR for review, or when running a pre-merge checklist. Provides a structured PR readiness evaluation.

- Skill: `haruhikomotokawa/pr-checklist` (Agent Skill, multi-file: 2 files)
- Install (CLI): `npx skillmds@latest add haruhikomotokawa/pr-checklist`
- Raw SKILL.md: https://api.skillmd.com/api/skills/haruhikomotokawa/pr-checklist/raw
- Safety review: pending (external: skill-scanner PASS, skillspector PASS)
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: Coding & Dev Tools
- Author: HaruhikoMotokawa (https://skillmd.com/u/haruhikomotokawa)
- Updated: 2026-09-10
- Page: https://skillmd.com/skills/haruhikomotokawa/pr-checklist

---


# PR Readiness Checklist

When evaluating a pull request, check each item in [checklist.md](checklist.md) and report:

- **PASS** — criterion is met
- **FAIL** — criterion is not met (include file:line reference)
- **N/A** — criterion does not apply to this change

## Evaluation Process

1. Read the PR diff and all changed files
2. Check each item in `checklist.md`
3. Report results in this format:

```
## PR Checklist Results: {pr-title}

| # | Item | Result | Notes |
|---|------|--------|-------|
| 1 | No TODO without issue ref | PASS | |
| 2 | New functionality has tests | FAIL | lib/auth/service.dart — no test for refreshToken() |
| 3 | README updated | N/A | Internal refactor only |

### Summary: X/Y criteria met

### Verdict: READY TO MERGE / NEEDS FIXES
```

## Criteria Count

Always include the total number of applicable criteria and how many passed.
A PR is READY TO MERGE only when all non-N/A items are PASS.