tombi-toml 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 / Cursor のサブエージェント機能で review-quality エージェントを起動する。親エージェント自身の自己レビューだけで代替しない。利用制限で起動できない場合の扱いは Step 3 を参照する。
サブエージェントには次を渡す。
- 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 を作成しない。
例外: review-quality サブエージェント（Codex / Claude Code / Cursor のいずれでも）が利用制限・quota 超過・out of usage などで起動できない、または途中で打ち切られた場合は、独立レビュー結果がなくてもゲートを通過してよい。その旨を記録し、Step 4 へ進む。ローカル検証（Step 1）は省略しない。
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 を未マージで引き渡したこと