pr-format
Skill0 starsChanged 23 days agoMITView on GitHub
The required commit-message, PR-title, and PR-body format for everything the pipeline commits and opens. Bound by the committer. Consistency over creativity.
In this repository9 other files it gives its agents.
- engageSkill
- gate-runnerSkill
- preflightSkill
- provision-repoSkill
- reportingSkill
- review-rubricSkill
- state-protocolSkill
- tdd-procedureSkill
- ticket-updatesSkill
The file
We split this skill into the rules it gives an agent
3 rules. About 0 are common, 3 are distinctive, and 0 are very specific to this project.
≈0
Common · estimate
Many files like this have a rule of this kind.
≈3
Distinctive · estimate
This author's own choices. Worth a look.
0
Very specific
Tied to their project's names, files or tools. Probably skip.
- Common
- Distinctive
- Very specific
- Distinctive: line 8
Every PR the pipeline opens MUST follow this exactly.
- Distinctive: line 36
Use these short sections. Omit any that don't apply, and don't pad:
- Distinctive: line 50
Do not add a per-acceptance-criterion coverage section or an exhaustive change list — that detail lives in the ticket and the tests, not …
≈ “Common” is an estimate: at least 1 in 5 of the 350,495 skill files we read have a rule of the same kind, not the same words. “Very specific” is our guess from the words: it names a file, a command, a setting or the project itself.
Files
PR format
Every PR the pipeline opens MUST follow this exactly.
Title
<type>(<KEY>): <summary> — where type comes from state.json.issue_type (same source as the branch prefix; a Sub-task uses its parent's type):
- Task / Story →
feat(<KEY>): <summary> - Bug / Development bug →
fix(<KEY>): <summary> - Epic / any other type → no type:
<KEY>: <summary>
<summary> rules: imperative mood, lowercase, ≤ ~72 chars, no trailing period, no code symbols/identifiers (say "the JSON naming policy", not PropertyNamingPolicy = CamelCase).
Examples:
feat(PROJ-101): finish removing Newtonsoft.Json from the auth servicefix(PROJ-404): stop null reference when importing an empty payloadPROJ-500: split billing into its own module(epic/other)
Commit message
- Subject = the PR title, verbatim (
feat(<KEY>): …/fix(<KEY>): …/<KEY>: …). Same rules (imperative, lowercase, ≤ ~72 chars, no trailing period). - Body (optional): 1–3 short lines only if the subject doesn't fully convey the change. Not a diff re-narration.
- Trailer (required): the
Co-Authored-By:trailer for the model actually running, as the session'sattributionsetting gives it. Don't copy a version out of this document — a pinned model name is wrong the next time the model changes. Plus any trailer the repo mandates (checkgit log). - One commit per repo, on that repo's branch. Don't
git add -A— stage only the intended files.
Body — short TL;DR
Keep the body a scannable TL;DR that complements the diff — the reviewer reads the code; the body just orients them. Short is the point: roughly the length of the diff summary, not longer. Rules: inline links only (never reference-style [x]: footnotes); no pipeline-internal language — never "gate", "write-tests stage", "sandbox", "PAT", "--expect-fail", "the orchestrator"; if a build/CI caveat genuinely matters, phrase it as a plain reviewer/CI note.
Use these short sections. Omit any that don't apply, and don't pad:
**What** — 1–2 sentences: what the PR does and the risk it addresses. Inline the ticket link [<KEY>](https://<jira_site>/browse/<KEY>) here, taking the host from `config.jira_site`; if it's empty, resolve it with `getAccessibleAtlassianResources` rather than emitting a link with no host.
**How** — the key mechanism, 1–3 lines. NOT a file-by-file re-narration (the diff carries that).
**Verify** — how it was checked, one line (e.g. "Tests 57/57 pass, build clean", or a plain CI note).
**Risks / notes** — behaviour-preservation notes, footguns, out-of-scope items worth flagging. Omit if none.
**Related** — for a multi-repo change, inline links to the sibling-repo PRs.Do not add a per-acceptance-criterion coverage section or an exhaustive change list — that detail lives in the ticket and the tests, not the PR body.
End the body with the Claude Code trailer. For a multi-repo change, each repo's PR uses the same title (its own <KEY>), and Related cross-links the sibling PRs so the coordinated change reviews together.
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
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.

