Skip to content

Commit 710b88c

Browse files
chaliyclaude
andauthored
chore(ship): prompt for PR evidence in the ship skill (#175)
## What changed The `/ship` command's Push-and-PR phase now prompts for **evidence in the PR body** — a Before / After with proof: CLI/API output or logs (fetch results, MCP responses). The Body line is also updated to reference `.github/pull_request_template.md` with the current sections (What changed / Why / Before / After / Risk / Checklist) and to center the description on functional change and impact. ## Why The ship flow already gathers validation, but the command didn't tell the agent to attach that proof to the PR, and its body reference still listed the old What/Why/How sections. Reviewers benefit from seeing the change work, not reading that it does. ## Before / After Docs/skill-guidance only — no runtime behavior changes. - **Before:** Body step said "use What / Why / How / Risk / Checklist." - **After:** references the PR template (What changed / Why / Before / After / Risk / Checklist) + functional framing, and adds an Evidence line (CLI/API output or logs), noting when there's no observable change. ## Risk - Low - Skill/docs only; no source, config, or CI logic touched. ## Checklist - [x] Unit tests are passed — N/A (skill/docs only) - [x] Smoke tests are passed — N/A - [x] Documentation is updated - [x] Specs are up to date and not in conflict --- _Generated by [Claude Code](https://claude.ai/code/session_013LSB82Db25Rf3tVPqWb1JQ)_ Co-authored-by: Claude <noreply@anthropic.com>
1 parent ff4e757 commit 710b88c

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

.claude/commands/ship.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,8 @@ gh pr view --json url 2>/dev/null
114114
If no PR exists, create one:
115115

116116
- Title: conventional commit style from the branch changes
117-
- Body: use What / Why / How / Risk / Checklist
117+
- Body: use `.github/pull_request_template.md` (What changed / Why / Before / After / Risk / Checklist); center it on functional change and impact, not a code-location walkthrough
118+
- Evidence: in the Before / After, attach proof — CLI/API output or logs (fetch results, MCP responses). Say so when there is no observable behavior change
118119
- Never add AI attribution or session links
119120

120121
If a PR exists, update it if needed and report its URL.

0 commit comments

Comments
 (0)