SamGu-NRX

pr-writing

Writes or revises pull request titles, descriptions and visual evidence. Use when preparing a PR, improving how it reads, or updating it after its scope changes. Writing the text does not publish the PR.

SamGu-NRX 1 Updated 2d ago

Resources

2
GitHub

Install

npx skillscat add samgu-nrx/spurs-hackathon/pr-writing

Install via the SkillsCat registry.

SKILL.md

Write a reviewable PR

Before drafting, read the repository's template and contributor guide, the diff against the PR's real base, and the current PR body. Apply unslop and technical-writing to everything you write. Keep what others added: human edits, evidence links and sections a bot manages. Fetch the body again right before you publish, so you don't overwrite someone else's change.

Explain the change

Open with the problem and what the code does now. Name the screen, operation or interface it touches, so someone who never saw the task can follow. For a large change, say what used to happen, what that cost, and how the change fixes it. A new feature needs no broken predecessor. Describe what it adds.

Order independent changes by how much they matter to users or to the rest of the code. Each heading names a change. Filenames and commit order are not an outline. Explain the mechanism when a reviewer needs it to judge correctness or a trade-off. Link the relevant part of the diff, or the file at the reviewed commit when no diff link exists. Cite earlier research only when a choice rests on it, with the source and why it matters.

Write whole sentences in which someone does something. Leave out clipped status phrases, internal names nobody explained, claims that the work was thorough, and narration about the reviewer or your process. Keep technical detail a reviewer can use, and cut any sentence that changes no decision. Read the short writing examples before drafting. Match how much they explain, not how their sentences are built or how long they are.

Size the body to the change. A small fix gets a short explanation and the one check that proves it. A broad PR can use one section per behavior, with evidence beside each claim. Put long logs, inventories and test matrices in linked files. Never paste a task transcript, a subagent report or a commit-by-commit diary into the description.

Show evidence

If the change is visible, read visual evidence and include matched before-and-after captures from the running app. Add a short recording when the change is motion, timing or interaction. Show only the behavior that changed. A new screen gets an after capture alone, labeled as new. If evidence is missing, say so in the body.

When a reviewer has to picture something the diff doesn't show at a glance, such as a flow, a state change, how parts connect, or before against after in a structure, add one simple diagram in a "How it works" section. Use a Mermaid chart, which GitHub renders, or an image. Draw it with few boxes, plain-word labels anyone could read, one idea per diagram, and a one-line caption saying what to notice. Skip it when the prose already makes the change obvious. A diagram supports the explanation; it doesn't replace it.

Report the checks you ran and what they returned, with the command or a link to the run. Put each limit next to the claim it weakens: fixture data, simulator-only behavior, a check that failed or was skipped, a service nobody tested. Keep steps to reproduce apart from checks you finished. Green CI says nothing about how the screen looks or whether the product owner will accept it.

Use the repository template

The shared template has sections for What changed, Why, How it works, UI changes, Verification and Risk, then a four-item checklist. Keep every section the repository requires. Delete an optional section that would be empty, and don't explain the same thing under two headings. Where a checklist item doesn't apply, replace it with N/A and the reason. Tick only work you finished. A large change that belongs together is fine. Don't call it small to tick a box.

Write the title about the final change, in the repository's title format. In stacked work, name the base branch and any PRs this one needs, and describe only what this PR adds. When the scope changes, update the title, the body and the affected evidence in the same edit. Keep captures that still hold, labeled with the commit they show, so nobody mistakes them for the newer build.

Publish with a body file or a structured tool argument. Then open the saved PR and check that the text, links, side-by-side images and videos render. A local draft or a successful upload doesn't prove the published body looks right. Push and open the PR only when the person you work for asks.