| name | github-pr-flow |
| description | 現在のローカル差分を検証し、Codex または Claude Code の review-quality サブエージェントによる独立レビューを PR 作成前に完了してから、commit、push、GitHub PR 作成、CI と既存 review thread の対応まで進める。トリガー: 'github-pr-flow'、'PR作成からレビュー対応まで'、'ローカルレビューしてPR作成'、'/github-pr-flow' |
GitHub PR Flow
現在の変更をローカルで独立レビューしてから PR を作成し、作成後の CI と既存 review thread を ../../github-pr-resolve/SKILL.md で処理する。GitHub 上の AI reviewer にはレビューを依頼しない。
前提
gh auth status が成功する- push 権限がある
- リポジトリと変更対象領域の
AGENTS.md / .claude/rules/ を確認済みである
- Codex または Claude Code のサブエージェント機能を利用できる
- 非対話コマンドを使う
ワークフロー
1. PR に含める差分を確定する
git fetch origin main を実行し、必要なら origin/main を取り込むgit rev-parse origin/main の出力を review base OID として記録する../../workspace-check/SKILL.md で変更面を分類し、必要なローカル検証を選択・実行して、実行済み項目と残るギャップを記録する- PR に含めるファイルだけを stage する
git diff --cached --check と git status --short で、意図しない差分や未追跡ファイルがないことを確認するgit diff --cached --binary origin/main | shasum -a 256 の出力を patch hash として記録し、<review-base-oid>:<patch-hash> を review fingerprint とする
差分が空なら PR を作成しない。未追跡ファイルは stage されるまでレビュー対象に含まれないため、必ず先に対象を確定する。
2. ローカルの独立サブエージェントにレビューを依頼する
実行中のハーネスに応じて、Codex または Claude Code のサブエージェント機能で review-quality エージェントを起動する。親エージェント自身の自己レビューだけで代替しない。
サブエージェントには次を渡す。
- base:
origin/main
- review base OID
- review fingerprint
- 対象:
git diff --cached origin/main と変更ファイルの関連実装・テスト
- 実行済み検証と結果
- 読み取り専用でレビューし、ファイルを変更しないこと
- correctness、回帰、検証不足、source of truth の不整合を優先し、finding は file / line と根拠を付けること
実装方針の正当化や期待する結論を先に教えない。サブエージェントが独立に差分と周辺コードを確認できる情報だけを渡す。
3. finding を解消し、同じ差分を再確認する
- actionable finding がなければ、review fingerprint が変わっていないことを確認して Step 4 へ進む
- actionable finding があれば親エージェントが修正し、対象検証、stage、fingerprint 採取をやり直す
- fingerprint が変わった場合、以前のレビュー結果を再利用せず、新しいサブエージェントで Step 2 をやり直す
- 仕様判断が必要で解消できない finding は、PR を作成せずユーザーに確認する
レビュー結果が空、失敗、権限不足、タイムアウトなどで有効な判定を得られない場合も fail closed とし、PR を作成しない。
4. PR を作成する
PR 作成そのものは次の優先順位で既存 skill に従う。
- リポジトリ内の
github-pr-create
- user-global の
my-github-pr-create
- どちらも使えない場合は、次の manual fallback を使う
manual fallback では、現在 branch が main または detached HEAD でないことを確認する。該当する場合は <type>/<purpose-kebab> 形式の topic branch を作成してから commit する。push 先は現在の GitHub アカウントが書き込める明示的な remote とし、base repository の保護された main へ直接 push しない。
git commit -m '<type>: <summary>'
git push -u <writable-remote> HEAD:refs/heads/<topic-branch>
gh pr create \
--repo tombi-toml/tombi \
--base main \
--head '<owner>:<topic-branch>' \
--title '<title>' \
--body-file <body-file> \
--label '<label>'
<writable-remote> の URL と <owner> が同じ fork を指すこと、push 後の remote OID が local HEAD と一致することを確認する。引数なしの対話的な gh pr create は使わない。
commit 直前に git fetch origin main を再実行し、origin/main の OID と staged patch hash が承認済み review fingerprint と一致することを確認する。commit 後は git diff --binary origin/main HEAD | shasum -a 256 で同じ tree 差分であることに加え、git status --porcelain が空であることを確認する。commit hook、formatter、base drift のいずれかで fingerprint が変わるか未commit差分が残った場合は、Step 1 から検証とレビューをやり直す。
gh pr create の直前にも git ls-remote origin refs/heads/main で remote base OID を直接確認する。承認済み review base OID と異なる場合は PR を作成せず、origin/main を更新して Step 1 からやり直す。
PR 作成時は branch、検証結果、commit、PR URL、必要な label を記録する。GitHub 上の AI reviewer 追加コマンドは実行しない。
5. 作成後の状態を確認する
PR URL、承認済み review base OID、review fingerprint、commit 後の exact head OID を入力として ../../github-pr-resolve/SKILL.md を使う。委譲先は現在の remote base、local / remote / PR head、full PR diff が渡された証拠と一致する場合だけ、そのレビュー結果を引き継ぐ。
- CI の失敗を調査・修正する
- 既に投稿された AI / 人間の review thread があれば通常どおり対応する
- review request の追加や AI reviewer の応答待ちは行わない
- PR はマージしない
PR 作成後の main 取り込み、CI 修正、review 対応などで head tree を変える場合は、push 前に新しい full PR diff を Step 1〜3 と同じ検証・fingerprint・独立サブエージェントレビューへ通す。以前の review fingerprint は再利用しない。
最終 handoff 前に remote base OID を再取得する。base または head tree が最後に承認された fingerprint から変わっていれば、最新の組み合わせで検証と独立レビューをやり直す。その後、現在の head、CI、reviews、comments、全ページの review threads、mergeability を同じ状態で取り直す。
出力
- 利用したローカル review subagent
- 承認済み review base OID、review fingerprint、finding の有無
- 実行した検証
- branch、commit、PR URL、label
- 対応した thread 数と最終未解決 thread 数
- PR を未マージで引き渡したこと
