From 12c34b053b8755a375f0694a7aa78ca129fbc2bc Mon Sep 17 00:00:00 2001 From: Michael Hargiss Date: Sun, 15 Mar 2026 10:00:36 -0700 Subject: [PATCH] Improve /pull-request skill with structured PR body template Co-Authored-By: Claude Sonnet 4.6 --- .claude/skills/pull-request/SKILL.md | 5 +---- .claude/skills/pull-request/template.md | 26 +++++++++++++++++++++++++ 2 files changed, 27 insertions(+), 4 deletions(-) create mode 100644 .claude/skills/pull-request/template.md diff --git a/.claude/skills/pull-request/SKILL.md b/.claude/skills/pull-request/SKILL.md index 0e9a6ac..b97942a 100644 --- a/.claude/skills/pull-request/SKILL.md +++ b/.claude/skills/pull-request/SKILL.md @@ -35,10 +35,7 @@ are no new commits to open a PR for. - If there is only one commit, use its subject line directly - If there are multiple commits, synthesize a single summary -**Body:** -- Explain what changed and why in plain prose -- Do not just restate the commit messages — add context -- Keep it concise; one short paragraph is usually enough +**Body:** Follow the template and rules in [template.md](template.md). ### Step 3: Show the title and body to the user and ask for confirmation diff --git a/.claude/skills/pull-request/template.md b/.claude/skills/pull-request/template.md new file mode 100644 index 0000000..3923aeb --- /dev/null +++ b/.claude/skills/pull-request/template.md @@ -0,0 +1,26 @@ +# Pull Request Body Template + +``` +## Problem + + +## Changes +- +- + +## Decisions +- +← omit this section if no new or relevant ADRs + +--- +Closes # ← omit if no related issue +⚠️ Breaking change: ← omit if no breaking change +``` + +## Rules + +- Omit the `Decisions` section if no ADRs are new or relevant to this PR +- Omit the `Closes` line if no related issue; do not guess an issue number +- Include the breaking change line only if a public API is removed, renamed, or + its behavior changes in a way that requires callers to update their code +- Do not add sections for testing — CI is the test record