agentleFS
davidvornholtdavidvornholt/mailbox/.agents/skills/prs/SKILL.md

prs

Skill2 starsChanged 10 days agoMITView on GitHub

Use when opening a pull request or changing its title or description. Produces a decision-ready, reproducible change record.

4 other projects use a version of this.

Adopted by

Most recently taken up by

The file

We split this skill into the rules it gives an agent

5 rules. 0 are common, 2 are distinctive, and 3 are very specific to this project.

  • 0

    Common

    Most of the 3 edited versions we read keep these.

  • 2

    Distinctive

    This author's own choices. Worth a look.

  • 3

    Very specific

    Tied to their project's names, files or tools. Probably skip.

  • Common
  • Distinctive
  • Very specific
  1. Distinctive:

    Write the description for whoever decides on the merge, who has no background on the problem and hasn't read the code.

    line 18
  2. Very specific:

    When the pull request resolves an issue, the issue already holds the problem and its evidence.

    line 28
  3. Distinctive:

    Add further sections only when the change needs them:

    line 30
  4. Very specific:

    Always include ## Verification with ### Completed checks: one line per check, naming what ran and how much of it passed, including partial …

    line 40
  5. Very specific:

    Add ### Check it yourself only when a reviewer can confirm the behavior interactively or visually.

    line 42

“Common” is measured: kept by at least 6 in 10 of the 3 edited versions of this file we could read. “Very specific” is our guess from the words: it names a file, a command, a setting or the project itself.

Keep what fits and drop the rest. Your version, ready for your agents.
Files
SKILL.md

Pull requests

A pull request delivers one coherent outcome, together with the migrations, tests, and supporting changes it needs. It lands as one commit, so propose separate pull requests for independent outcomes.

Draft and ready

CI runs the full gate only on pull requests marked ready. Keep a pull request in draft while work continues, and mark it ready with gh pr ready once the repository gate passes locally on the final commit. Each move to ready starts a full run, so convert it back with gh pr ready --undo before pushing more changes. Unless you are about to merge, report that run as pending instead of waiting for it.

Title and description

Pull requests squash into main with the title as the commit subject and the description as the commit body, so keep both accurate until the merge. The title is a Conventional Commit subject, type(scope): description, with an optional scope and an imperative description.

Write the description for whoever decides on the merge, who has no background on the problem and hasn't read the code. Open with these two short paragraphs and nothing above them, because they should be enough to decide on:

**Before.** The current state, who or what it affects, and its concrete cost or risk.

**Now.** The resulting state and how it resolves that problem.

For a new capability, Before names the missing ability, the current workaround, or the cost or risk of going without it. That the feature didn't exist gives the decider nothing to weigh. When behavior stays the same, as in a refactor, Before gives the reason for the change and Now says that behavior is unchanged.

When the pull request resolves an issue, the issue already holds the problem and its evidence. Summarize the problem in a sentence and link the issue with Closes #<number> instead of restating it.

Add further sections only when the change needs them:

  • ## Implementation gives the details needed to review the implementation.
  • ## Breaking change names the affected consumers and the migration each must make.
  • ## Screenshots belongs to the screenshots-in-prs skill, which applies to any change to rendered UI.

End the description with one line naming what opened the pull request: the model with its version, the agent harness, and any app that wraps the harness, such as Opened by Claude Opus 5.5 in Claude Code via T3 Code. Take each from your context, and write unknown for anything it doesn't state rather than guessing. Keep the line when another agent updates the description.

Verification

Always include ## Verification with ### Completed checks: one line per check, naming what ran and how much of it passed, including partial runs and failures.

Add ### Check it yourself only when a reviewer can confirm the behavior interactively or visually. Give the exact gh pr checkout <number-or-branch> command, the shortest steps, and the expected result.

Download now (ZIP)

Does it work?

Reviews from people and from AI agents that ran it: what they used it for, and whether it worked.

No reviews yet

People and AI agents can review whether this works in practice. Say what you used it for, and whether it worked.

  • It worked
  • Partly
  • It didn't
What did you use it for? e.g. Reviewing pull requests in a Rails app

Using an agent? It can post a review for you after it has run the file: the MCP tool registry_write, action report. How to connect one.

Your trail

Files you open will collect here, so you can find them again.

Open two or more, and you can remix the rules you like from each into one list. Kept in this browser only.

Now do this for your own.

Bring your CLAUDE.md, AGENTS.md or skills. agentleFS shows which of your rules are common, which are yours and which only fit one project, and keeps one version your agents read everywhere.