Pr description
skillet — a package manager for AI agent skills (SKILL.md). Find, install, version & share skills from a Git-backed registry. Zero infra, MCP-native, reproducible.
npx -y skills add jnMetaCode/skillet --skill pr-descriptionAssembled from the repository path, not quoted from the project. Check it against their README if it does not work.
One thing to look at
- 1 stars1 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.
What its author says it does
Copied from the file, not written here
Write a reviewer-first pull-request description from the actual branch diff. Use when opening a PR or asked to write/improve a PR description.
The file declares its own license as MIT. That is the author’s claim about this one file, and it is not the same thing as the license GitHub reports for the repository, which is listed with the other numbers below.
SKILL.md
2.0 KB, as published. Nobody here has run it
pr-description
The reader is a reviewer deciding how to review, not a historian. Write from the diff, not from memory of the conversation.
Procedure
- Establish the real change set:
git log main..HEAD --onelineandgit diff main...HEAD --stat. Read the diff of every file a reviewer will question. - Open with one sentence of why — the problem or goal, not the activity ("Checkout fails for SSO users" beats "Update auth flow").
- What changed, grouped by intent rather than by file: behavior changes first, then refactors, then mechanical churn (renames, formatting) called out explicitly so reviewers can skim past it.
- How to verify: the exact commands/steps you ran and their result. If CI covers it, say which jobs; if manual, give the click-path.
- Risk and rollout: what could break, feature flags, migrations, revert plan. One line each. Omit the section only if genuinely none.
- Call out anything you want focused review on ("the locking in store.js is the risky part") — reviewers allocate attention where the author points.
Anti-patterns
- Restating the commit list — the PR already shows it.
- "See ticket" as the why — summarize the ticket in one line, then link it.
- Screenshots without a caption saying what to look at.
- Describing the diff file-by-file — group by intent.
Skeleton
Fixes checkout failing for SSO users (#412).
**What**
- Treat IdP-initiated sessions as authenticated in the cart guard (behavior)
- Extract session checks into `session.ts` (refactor, no behavior change)
**Verify**: `npm test` (all green) + manual SSO login → add to cart → checkout.
**Risk**: session guard touches every authed route; revert is a single commit.
Focus review on `session.ts:40-75`.