---
name: mohitagw15856/pr-description
source: https://app.decimal.ai/s/mohitagw15856-pr-description@1/SKILL.md
source_sha256: 62aff359d7da
---

# PR Description Skill

A good PR description is a gift to the reviewer: it explains *intent* before they read the diff, so review is
fast and confident. This skill turns a change into a structured PR write-up — what and why, how it was tested,
the risk, and where to focus — the difference between a one-pass approval and three rounds of confused
back-and-forth.

## Required Inputs

Ask for these only if they aren't already provided:

- **The change** — what was done (the diff summary, commits, or a description).
- **The why** — the problem/issue it solves (link the ticket).
- **Testing** — how it was verified (tests added, manual steps, edge cases checked).
- **Risk & rollout** — blast radius, migrations, flags, backward compatibility, how to roll back.

## Output Format

### [Concise, imperative PR title] (e.g. "Add rate limiting to the login endpoint")

**What & why** — 2–4 sentences: the problem and what this change does about it. Link the issue (`Closes #123`).

**Changes** — the key changes as bullets (the substantive ones, not every file). Group if large.

**How it was tested** — tests added/updated, and the manual verification + edge cases checked. Be specific enough that the reviewer trusts it works.

**Risk & rollout** — blast radius, any migration/flag/config, backward-compatibility notes, and how to roll back if it goes wrong. Say "low risk, no migration" if so.

**Reviewer guide** — where to start, what to scrutinize, anything intentionally out of scope or deferred (with a follow-up note). Call out anything you're unsure about and want eyes on.

**Screenshots / output** (if UI or user-facing) — before/after.

Keep it proportional — a one-line fix gets a short description; a big change earns the full structure.

## Quality Checks

- [ ] Title is concise and imperative; the why and linked issue are clear up front
- [ ] Changes summarize intent, not a file-by-file dump
- [ ] Testing is specific (what was run, which edge cases) — not "tested locally"
- [ ] Risk, rollout, and rollback are addressed (even if "low risk, none")
- [ ] A reviewer guide points to where to focus and flags anything uncertain
- [ ] Length is proportional to the size of the change

## Anti-Patterns

- [ ] Do not just paste the commit list — explain intent the diff can't convey
- [ ] Do not say "tested" without saying how — give the reviewer something to trust
- [ ] Do not hide risk or migrations — surface them so they're reviewed deliberately
- [ ] Do not write a novel for a one-line change — match effort to size
- [ ] Do not omit the "what to focus on" — undirected review is slow review

## Based On

Code-review and PR best practices (explain intent, make review easy, surface risk) — modern engineering norms.