GH PR
Overview
Create or update GitHub Pull Requests with the gh CLI using a detailed body template and strict same-branch rules.
Decision rules (must follow)
- Do not create or switch branches. Always use the current branch as the PR head.
- Check local working tree state before push/PR operations.
git status --porcelain- If output is non-empty (tracked or untracked changes), pause and ask the user what to do.
- Present 3 options: continue as-is, abort, or manual cleanup then rerun.
- Do not run
git stash, git commit, or git clean automatically unless explicitly requested.
- Check for an existing PR for the current head branch.
gh pr list --head <head> --state all --json number,state,mergedAt,updatedAt,url,title,mergeCommit
- If no PR exists → create a new PR.
- If any PR exists and is NOT merged (
mergedAt is null) → push only and finish (do not create a new PR).
- This applies to OPEN or CLOSED (unmerged) PRs.
- Only update title/body/labels if the user explicitly requests changes.
- If all PRs for the head are merged → check for post-merge commits (see below).
- If multiple PRs exist for the head → use the most recently updated PR for reporting, but the create vs push decision is based on
mergedAt.
Post-merge commit check (critical)
When all PRs for the head branch are merged, you must check whether there are new commits after the merge:
- Get the merge commit SHA of the most recent merged PR.
- Count commits after the merge:
git rev-list --count <merge_commit>..HEAD
- Decision:
- If new commits exist → create a new PR (these changes are not in the base branch)
- If no new commits → report "No changes since last merge" and finish (do not create an empty PR)
Why this matters
- Scenario A: PR merged → user makes local changes → pushes → changes are NOT in the merged PR
- Without this check, the changes would be lost or require manual intervention
- Scenario B: PR merged → user says "create PR" without new changes → would create empty/duplicate PR
- This check prevents unnecessary PR creation
PR title rules (must follow)
- Format:
<type>(<scope>): <subject> — Conventional Commits 形式に準拠する。
- type:
feat / fix / docs / chore / refactor / test / ci / perf のいずれか。
- scope: 省略可。変更の影響範囲を端的に示す（例:
gui, core, pty）。
- subject: 70文字以内。命令形（imperative mood）で書く（例: "add …" / "fix …"）。先頭大文字禁止、末尾ピリオド禁止。
- ブランチ名にプレフィックス（
feat/, fix/ など）がある場合、タイトルの type と一致させる。
PR body rules (must follow)
Section classification
| Section |
Required |
Notes |
| Summary |
YES |
1-3 bullet points。"what" と "why" を両方含める |
| Changes |
YES |
ファイル/モジュール単位で変更内容を列挙 |
| Testing |
YES |
実行したコマンドまたは手動テスト手順を具体的に記載 |
| Related Issues / Links |
YES |
Issue番号、spec、または "None" を明記 |
| Checklist |
YES |
全項目を確認してチェック/N-A を付ける |
| Context |
Conditional |
3ファイル以上の変更、または非自明な変更理由がある場合は必須 |
| Risk / Impact |
Conditional |
破壊的変更・パフォーマンス影響・ロールバック手順がある場合は必須 |
| Screenshots |
Conditional |
UI 変更がある場合のみ必須 |
| Deployment |
Optional |
デプロイ手順がある場合のみ記載 |
| Notes |
Optional |
レビュアーへの補足がある場合のみ記載 |
Validation (agent must check before creating PR)
- Required セクションに
TODO が残っていたら PR を作成してはならない。
- Conditional セクションが該当しない場合は、セクション自体を削除する（空の TODO を残さない）。
- Summary の各 bullet は 1文で完結 させる。曖昧な表現（"いくつかの変更", "various fixes"）を禁止する。
- Changes は 変更ファイルまたはモジュール名を含む 具体的な記述にする。
- Testing は 再現可能な手順 を書く（"テスト済み" のような曖昧な記述を禁止）。
- Checklist の未チェック項目には理由コメントを付ける（例:
- [ ] Docs updated — N/A: no user-facing change）。
- Related Issues は
#123 形式または URL で記載する。該当なしの場合は "None" と明記する。
Issue/PR Comment Formatting (must follow)
- Final comment text must not contain escaped newline literals such as
\n.
- Use real line breaks in comment bodies. Do not rely on escaped sequences for formatting.
- Before posting, verify the final body does not accidentally include escaped control sequences (
\n, \t).
- If a raw escape sequence must be shown for explanation, include it only inside a fenced code block and clarify it is intentional.
Issue Progress Comment Template (required for issue-based work)
When work is tracked in GitHub Issues, progress updates must use this template:
Progress
- ...
Done
- ...
Next
- ...
- Post updates at least when starting work, after meaningful progress, and when blocked/unblocked.
- In
Next, explicitly state blockers or the immediate next action.
Workflow (recommended)
Confirm repo + branches
- Repo root:
git rev-parse --show-toplevel
- Current branch (head):
git rev-parse --abbrev-ref HEAD
- Base branch defaults to
develop unless user specifies.
Check local working tree state (preflight)
- Run
git status --porcelain.
- If empty, continue.
- If non-empty, show detected files and ask the user to choose:
- Continue as-is
- Abort
- Manual cleanup first (
git commit / git stash / git clean) and rerun
- Proceed only when the user explicitly chooses continue.
Fetch latest remote state
git fetch origin to ensure accurate comparison
Check existing PR for head branch
- Use decision rules above to pick action.
- Treat
mergedAt as the source of truth for "merged".
If all PRs are merged, perform post-merge commit check
- Get merge commit:
gh pr list --head <head> --state merged --json mergeCommit -q '.[0].mergeCommit.oid'
- Count new commits:
git rev-list --count <merge_commit>..HEAD
- If 0 → finish with message "No new changes since merge"
- If >0 → proceed to create new PR
Ensure the head branch is pushed
- If no upstream:
git push -u origin <head>
- Otherwise:
git push
Collect PR inputs (for new PR or explicit update)
- Title, Summary, Context, Changes, Testing, Risk/Impact, Deployment, Screenshots, Related Links, Notes
- Optional: labels, reviewers, assignees, draft
Build PR body from template
- Read the template from the gh-pr skill path (not the current project path):
GH_PR_SKILL_DIR="${GH_PR_SKILL_DIR:-$HOME/.codex/skills/gh-pr}"PR_BODY_TEMPLATE="${GH_PR_SKILL_DIR}/references/pr-body-template.md"
- Read
${PR_BODY_TEMPLATE} and fill all required placeholders.
- Conditional セクションが該当しない場合はセクションごと削除する。
- テンプレート内の
<!-- GUIDE: ... --> コメントは最終出力から削除する。
- Required セクションに TODO が残っている場合は PR を作成せず、ユーザーに不足情報を確認する。
Create or update the PR
- Create:
gh pr create -B <base> -H <head> --title "<title>" --body-file <file>
- Update (only if user asked):
gh pr edit <number> --title "<title>" --body-file <file>
Return PR URL
gh pr view <number> --json url -q .url
- Post-PR CI/merge check (automatic).
- After PR creation or push, load
github/skills/gh-fix-pr/SKILL.md and
follow its workflow to inspect CI status, merge state, and review feedback.
- If all CI checks are still pending, poll (30s interval) until complete.
- If conflicts, review issues, or CI failures are detected, proceed with
the gh-fix-pr workflow to diagnose and fix.
Command snippets (bash)
head=$(git rev-parse --abbrev-ref HEAD)
base=develop
GH_PR_SKILL_DIR="${GH_PR_SKILL_DIR:-$HOME/.codex/skills/gh-pr}"
PR_BODY_TEMPLATE="${GH_PR_SKILL_DIR}/references/pr-body-template.md"
if [ ! -f "$PR_BODY_TEMPLATE" ]; then
echo "PR template not found: $PR_BODY_TEMPLATE" >&2
exit 1
fi
# Preflight: local working tree state
status_lines=$(git status --porcelain)
if [ -n "$status_lines" ] && [ "${ALLOW_DIRTY_WORKTREE:-0}" != "1" ]; then
echo "Detected local uncommitted/untracked changes:" >&2
echo "$status_lines" >&2
echo "Choose one before continuing: continue as-is, abort, or manual cleanup then rerun." >&2
echo "Set ALLOW_DIRTY_WORKTREE=1 only after explicit user confirmation to continue." >&2
exit 1
fi
# Fetch latest remote state
git fetch origin
# Check existing PRs for the head branch
pr_json=$(gh pr list --head "$head" --state all --json number,state,mergedAt,mergeCommit)
pr_count=$(echo "$pr_json" | jq 'length')
unmerged_count=$(echo "$pr_json" | jq 'map(select(.mergedAt == null)) | length')
if [ "$pr_count" -eq 0 ]; then
action=create
elif [ "$unmerged_count" -gt 0 ]; then
action=push_only
else
# All PRs are merged - check for post-merge commits
merge_commit=$(echo "$pr_json" | jq -r 'sort_by(.mergedAt) | last | .mergeCommit.oid')
if [ -n "$merge_commit" ] && [ "$merge_commit" != "null" ]; then
new_commits=$(git rev-list --count "$merge_commit"..HEAD 2>/dev/null || echo "0")
if [ "$new_commits" -gt 0 ]; then
echo "Found $new_commits commit(s) after merge - creating new PR"
action=create
else
echo "No new commits since merge - nothing to do"
action=none
fi
else
# Fallback: check against base branch
new_commits=$(git rev-list --count "origin/$base"..HEAD 2>/dev/null || echo "0")
if [ "$new_commits" -gt 0 ]; then
action=create
else
action=none
fi
fi
fi
# Execute action
case "$action" in
create)
cp "$PR_BODY_TEMPLATE" /tmp/pr-body.md
git push -u origin "$head"
gh pr create -B "$base" -H "$head" --title "..." --body-file /tmp/pr-body.md
;;
push_only)
echo "Existing unmerged PR found - pushing changes only"
git push
gh pr list --head "$head" --state open --json url -q '.[0].url'
;;
none)
echo "No action needed - no new changes since last merge"
;;
esac
References
${GH_PR_SKILL_DIR}/references/pr-body-template.md: PR body template
Converted and distributed by TomeVault — claim your Tome and manage your conversions.
1---2name: akiojin-skills-gh-pr3description: GH PR4---56# GH PR78## Overview910Create or update GitHub Pull Requests with the gh CLI using a detailed body template and strict same-branch rules.1112## Decision rules (must follow)13141. **Do not create or switch branches.** Always use the current branch as the PR head.152. **Check local working tree state before push/PR operations.**16 - `git status --porcelain`17 - If output is non-empty (tracked or untracked changes), pause and ask the user what to do.18 - Present 3 options: continue as-is, abort, or manual cleanup then rerun.19 - **Do not** run `git stash`, `git commit`, or `git clean` automatically unless explicitly requested.203. **Check for an existing PR for the current head branch.**21 - `gh pr list --head <head> --state all --json number,state,mergedAt,updatedAt,url,title,mergeCommit`224. **If no PR exists** → create a new PR.235. **If any PR exists and is NOT merged** (`mergedAt` is null) → push only and finish (do **not** create a new PR).24 - This applies to OPEN or CLOSED (unmerged) PRs.25 - Only update title/body/labels if the user explicitly requests changes.266. **If all PRs for the head are merged** → check for post-merge commits (see below).277. **If multiple PRs exist for the head** → use the most recently updated PR for reporting, but the create vs push decision is based on `mergedAt`.2829## Post-merge commit check (critical)3031When all PRs for the head branch are merged, you **must** check whether there are new commits after the merge:32331. **Get the merge commit SHA** of the most recent merged PR.342. **Count commits after the merge**: `git rev-list --count <merge_commit>..HEAD`353. **Decision**:36 - If new commits exist → create a new PR (these changes are not in the base branch)37 - If no new commits → report "No changes since last merge" and finish (do **not** create an empty PR)3839### Why this matters4041- **Scenario A**: PR merged → user makes local changes → pushes → changes are NOT in the merged PR42 - Without this check, the changes would be lost or require manual intervention43- **Scenario B**: PR merged → user says "create PR" without new changes → would create empty/duplicate PR44 - This check prevents unnecessary PR creation4546## PR title rules (must follow)47481. **Format**: `<type>(<scope>): <subject>` — Conventional Commits 形式に準拠する。492. **type**: `feat` / `fix` / `docs` / `chore` / `refactor` / `test` / `ci` / `perf` のいずれか。503. **scope**: 省略可。変更の影響範囲を端的に示す（例: `gui`, `core`, `pty`）。514. **subject**: 70文字以内。命令形（imperative mood）で書く（例: "add …" / "fix …"）。先頭大文字禁止、末尾ピリオド禁止。525. ブランチ名にプレフィックス（`feat/`, `fix/` など）がある場合、**タイトルの type と一致させる**。5354## PR body rules (must follow)5556### Section classification5758| Section | Required | Notes |59|---------|----------|-------|60| Summary | **YES** | 1-3 bullet points。"what" と "why" を両方含める |61| Changes | **YES** | ファイル/モジュール単位で変更内容を列挙 |62| Testing | **YES** | 実行したコマンドまたは手動テスト手順を具体的に記載 |63| Related Issues / Links | **YES** | Issue番号、spec、または "None" を明記 |64| Checklist | **YES** | 全項目を確認してチェック/N-A を付ける |65| Context | Conditional | 3ファイル以上の変更、または非自明な変更理由がある場合は必須 |66| Risk / Impact | Conditional | 破壊的変更・パフォーマンス影響・ロールバック手順がある場合は必須 |67| Screenshots | Conditional | UI 変更がある場合のみ必須 |68| Deployment | Optional | デプロイ手順がある場合のみ記載 |69| Notes | Optional | レビュアーへの補足がある場合のみ記載 |7071### Validation (agent must check before creating PR)72731. **Required セクションに `TODO` が残っていたら PR を作成してはならない。**742. Conditional セクションが該当しない場合は、セクション自体を削除する（空の TODO を残さない）。753. Summary の各 bullet は **1文で完結** させる。曖昧な表現（"いくつかの変更", "various fixes"）を禁止する。764. Changes は **変更ファイルまたはモジュール名を含む** 具体的な記述にする。775. Testing は **再現可能な手順** を書く（"テスト済み" のような曖昧な記述を禁止）。786. Checklist の未チェック項目には理由コメントを付ける（例: `- [ ] Docs updated — N/A: no user-facing change`）。797. Related Issues は `#123` 形式または URL で記載する。該当なしの場合は "None" と明記する。8081## Issue/PR Comment Formatting (must follow)8283- Final comment text must not contain escaped newline literals such as `\n`.84- Use real line breaks in comment bodies. Do not rely on escaped sequences for formatting.85- Before posting, verify the final body does not accidentally include escaped control sequences (`\n`, `\t`).86- If a raw escape sequence must be shown for explanation, include it only inside a fenced code block and clarify it is intentional.8788## Issue Progress Comment Template (required for issue-based work)8990When work is tracked in GitHub Issues, progress updates must use this template:9192```markdown93Progress94- ...9596Done97- ...9899Next100- ...101```102103- Post updates at least when starting work, after meaningful progress, and when blocked/unblocked.104- In `Next`, explicitly state blockers or the immediate next action.105106## Workflow (recommended)1071081. **Confirm repo + branches**109 - Repo root: `git rev-parse --show-toplevel`110 - Current branch (head): `git rev-parse --abbrev-ref HEAD`111 - Base branch defaults to `develop` unless user specifies.1121132. **Check local working tree state (preflight)**114 - Run `git status --porcelain`.115 - If empty, continue.116 - If non-empty, show detected files and ask the user to choose:117 - Continue as-is118 - Abort119 - Manual cleanup first (`git commit` / `git stash` / `git clean`) and rerun120 - Proceed only when the user explicitly chooses continue.1211223. **Fetch latest remote state**123 - `git fetch origin` to ensure accurate comparison1241254. **Check existing PR for head branch**126 - Use decision rules above to pick action.127 - Treat `mergedAt` as the source of truth for "merged".1281295. **If all PRs are merged, perform post-merge commit check**130 - Get merge commit: `gh pr list --head <head> --state merged --json mergeCommit -q '.[0].mergeCommit.oid'`131 - Count new commits: `git rev-list --count <merge_commit>..HEAD`132 - If 0 → finish with message "No new changes since merge"133 - If >0 → proceed to create new PR1341356. **Ensure the head branch is pushed**136 - If no upstream: `git push -u origin <head>`137 - Otherwise: `git push`1381397. **Collect PR inputs (for new PR or explicit update)**140 - Title, Summary, Context, Changes, Testing, Risk/Impact, Deployment, Screenshots, Related Links, Notes141 - Optional: labels, reviewers, assignees, draft1421438. **Build PR body from template**144 - Read the template from the gh-pr skill path (not the current project path):145 - `GH_PR_SKILL_DIR="${GH_PR_SKILL_DIR:-$HOME/.codex/skills/gh-pr}"`146 - `PR_BODY_TEMPLATE="${GH_PR_SKILL_DIR}/references/pr-body-template.md"`147 - Read `${PR_BODY_TEMPLATE}` and fill all required placeholders.148 - **Conditional セクションが該当しない場合はセクションごと削除する。**149 - **テンプレート内の `<!-- GUIDE: ... -->` コメントは最終出力から削除する。**150 - **Required セクションに TODO が残っている場合は PR を作成せず、ユーザーに不足情報を確認する。**1511529. **Create or update the PR**153 - Create: `gh pr create -B <base> -H <head> --title "<title>" --body-file <file>`154 - Update (only if user asked): `gh pr edit <number> --title "<title>" --body-file <file>`15515610. **Return PR URL**157 - `gh pr view <number> --json url -q .url`15815911. **Post-PR CI/merge check (automatic).**160 - After PR creation or push, load `github/skills/gh-fix-pr/SKILL.md` and161 follow its workflow to inspect CI status, merge state, and review feedback.162 - If all CI checks are still pending, poll (30s interval) until complete.163 - If conflicts, review issues, or CI failures are detected, proceed with164 the gh-fix-pr workflow to diagnose and fix.165166## Command snippets (bash)167168```bash169head=$(git rev-parse --abbrev-ref HEAD)170base=develop171GH_PR_SKILL_DIR="${GH_PR_SKILL_DIR:-$HOME/.codex/skills/gh-pr}"172PR_BODY_TEMPLATE="${GH_PR_SKILL_DIR}/references/pr-body-template.md"173174if [ ! -f "$PR_BODY_TEMPLATE" ]; then175 echo "PR template not found: $PR_BODY_TEMPLATE" >&2176 exit 1177fi178179# Preflight: local working tree state180status_lines=$(git status --porcelain)181if [ -n "$status_lines" ] && [ "${ALLOW_DIRTY_WORKTREE:-0}" != "1" ]; then182 echo "Detected local uncommitted/untracked changes:" >&2183 echo "$status_lines" >&2184 echo "Choose one before continuing: continue as-is, abort, or manual cleanup then rerun." >&2185 echo "Set ALLOW_DIRTY_WORKTREE=1 only after explicit user confirmation to continue." >&2186 exit 1187fi188189# Fetch latest remote state190git fetch origin191192# Check existing PRs for the head branch193pr_json=$(gh pr list --head "$head" --state all --json number,state,mergedAt,mergeCommit)194pr_count=$(echo "$pr_json" | jq 'length')195unmerged_count=$(echo "$pr_json" | jq 'map(select(.mergedAt == null)) | length')196197if [ "$pr_count" -eq 0 ]; then198 action=create199elif [ "$unmerged_count" -gt 0 ]; then200 action=push_only201else202 # All PRs are merged - check for post-merge commits203 merge_commit=$(echo "$pr_json" | jq -r 'sort_by(.mergedAt) | last | .mergeCommit.oid')204205 if [ -n "$merge_commit" ] && [ "$merge_commit" != "null" ]; then206 new_commits=$(git rev-list --count "$merge_commit"..HEAD 2>/dev/null || echo "0")207208 if [ "$new_commits" -gt 0 ]; then209 echo "Found $new_commits commit(s) after merge - creating new PR"210 action=create211 else212 echo "No new commits since merge - nothing to do"213 action=none214 fi215 else216 # Fallback: check against base branch217 new_commits=$(git rev-list --count "origin/$base"..HEAD 2>/dev/null || echo "0")218219 if [ "$new_commits" -gt 0 ]; then220 action=create221 else222 action=none223 fi224 fi225fi226227# Execute action228case "$action" in229 create)230 cp "$PR_BODY_TEMPLATE" /tmp/pr-body.md231232 git push -u origin "$head"233 gh pr create -B "$base" -H "$head" --title "..." --body-file /tmp/pr-body.md234 ;;235 push_only)236 echo "Existing unmerged PR found - pushing changes only"237 git push238 gh pr list --head "$head" --state open --json url -q '.[0].url'239 ;;240 none)241 echo "No action needed - no new changes since last merge"242 ;;243esac244```245246## References247248- `${GH_PR_SKILL_DIR}/references/pr-body-template.md`: PR body template249250---251> Converted and distributed by [TomeVault](https://tomevault.io/claim/akiojin) — claim your Tome and manage your conversions.252<!-- tomevault:4.0:skill_md:2026-04-13 -->
tomevault-io