From 5db595da84245d8f0910231d1def765b33687504 Mon Sep 17 00:00:00 2001 From: fullstackjam Date: Tue, 2 Jun 2026 23:50:45 +0800 Subject: [PATCH] chore: add cross-repo checklist to PR template and teach ship-pr to use it --- .claude/skills/ship-pr/SKILL.md | 14 ++++++++++++++ .github/pull_request_template.md | 5 +++++ 2 files changed, 19 insertions(+) diff --git a/.claude/skills/ship-pr/SKILL.md b/.claude/skills/ship-pr/SKILL.md index 9b3deca..8d2e739 100644 --- a/.claude/skills/ship-pr/SKILL.md +++ b/.claude/skills/ship-pr/SKILL.md @@ -51,6 +51,20 @@ is not installed, that's the user's choice; CI will still gate. ### Step 3 — Open the PR +Before writing the body, check for a PR template: + +```bash +cat .github/pull_request_template.md 2>/dev/null +``` + +If the file exists, use it as the body structure — fill in its sections +rather than inventing a different format. Do **not** discard the template +by passing a completely custom `--body` that ignores it. If the template +has a cross-repo checklist item (e.g. "does this need a docs update in +openboot.dev?"), answer it honestly. + +If no template exists, fall back to the default structure: + ```bash gh pr create --title "" --body "$(cat <<'EOF' ## Summary diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index 63a3f3b..d9c7a14 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -14,6 +14,11 @@ - [ ] Relevant tests added or updated - [ ] Tested locally (`./openboot install --dry-run` or similar) +## Cross-repo checklist + +- [ ] Does this need a docs/content update in `openboot.dev`? +- [ ] Does this change the CLI ↔ server API contract? (update `openboot-contract` first) + ## Notes for reviewer