hirokimry/pr-review-loop

PRレビュー修正ループ（同期版）

PR の状態を gh pr view でポーリングし、MERGED / CLOSED に到達するか escalation 条件に当たるまで、teammate のターン内で完遂する。CHANGES_REQUESTED を検知したら /vibecorp:pr-review-fix を同期呼び出しして指摘を消化する。

/loop や ScheduleWakeup のような非同期スケジューラには依存しない。teammate（/vibecorp:ship-parallel 配下の Agent）はメッセージ駆動で idle 化すると wakeup が届かないため、そこで PR が放置される構造欠陥を回避する。

使用方法

/vibecorp:pr-review-loop                    # 現在のブランチの PR を対象に開始
/vibecorp:pr-review-loop <PR URL>           # PR URL を直接指定
/vibecorp:pr-review-loop --worktree <path>  # worktree 内で実行

worktree モード

--worktree <path> が指定された場合、全操作を指定パス内で実行する。

• Bash: 全コマンドを cd <path> && command で実行する
• Read/Write/Edit: <path>/ を基準とした絶対パスを使用する
• サブスキル呼び出し: --worktree <path> を引き継いで /vibecorp:pr-review-fix を呼ぶ
• 未指定時は CWD で実行する（後方互換）

ループ制御

| 値 | 設定 | 理由 | |-----|-----|------| | polling 間隔 | 30 秒 | CodeRabbit / CI 遷移頻度に対して充分 | | max iterations | 20 | 30 秒 × 20 = 約 10 分 | | timeout | 60 分 | fail-safe |

ワークフロー

1. PR 情報の特定

PR URL が指定されていればそこから owner/repo/PR 番号を抽出する。未指定なら現在のブランチから自動検出する。

gh pr view --json number,url,headRefName,baseRefName --jq '{number, url, headRefName, baseRefName}'

PR が見つからなければ「PR 未作成のため /vibecorp:pr を先に実行してください」と報告して終了する。

2. CodeRabbit 有効性の確認

awk '/^coderabbit:/{found=1; next} found && /^[^ ]/{exit} found && /enabled:/{print $2}' \
  "$CLAUDE_PROJECT_DIR"/.claude/vibecorp.yml

• false の場合は CodeRabbit レビュー待ちをスキップし、ステップ 4（auto-merge 設定）から実行して終了する
• true または未定義（空）の場合はステップ 3 へ

3. auto-merge 設定（冪等）

autoMergeRequest が未設定なら設定する。既設定ならスキップする。

gh pr view <pr_number> --json autoMergeRequest --jq '.autoMergeRequest'
gh pr merge <pr_number> --squash --auto

4. 同期ポーリングループ

MERGED / CLOSED か escalation 条件に到達するまで、以下を最大 20 反復・60 分以内で繰り返す。

各反復の先頭で PR 状態を一括取得する。

gh pr view <pr_number> \
  --json state,mergeStateStatus,reviewDecision,autoMergeRequest \
  --jq '{state, mergeStateStatus, reviewDecision, autoMergeRequest}'

状態遷移表

| state | reviewDecision | mergeStateStatus | 行動 | |-------|----------------|-------------------|------| | MERGED | – | – | 成功終了 | | CLOSED | – | – | 「PR がクローズされました」と報告して正常終了 | | OPEN | CHANGES_REQUESTED | – | /vibecorp:pr-review-fix を同期呼び出し → sleep 30 → 次反復 | | OPEN | – | CLEAN | auto-merge 発動待ち（sleep 30 → 次反復） | | OPEN | – | BLOCKED / BEHIND / UNSTABLE / UNKNOWN / HAS_HOOKS | CI / approve 待ち（sleep 30 → 次反復） | | OPEN | – | DIRTY | escalate（マージコンフリクト発生） | | OPEN | – | DRAFT | escalate（Draft PR は対象外） |

CHANGES_REQUESTED 時の処理

1. /vibecorp:pr-review-fix（worktree モードでは --worktree <path> を引き継ぐ）を Skill ツールで同期呼び出しする
2. /vibecorp:pr-review-fix が rate limit 停止を返した場合は escalate
3. 完了後 sleep 30 で CodeRabbit 再レビューを待つ
4. 次反復で再度 gh pr view

iterations / timeout 制御

各反復の先頭で経過時間と反復回数を確認する。

• 反復数が 20 を超えた場合 → escalate
• 開始から 60 分（3600 秒）を超えた場合 → escalate

5. escalation

以下のいずれかが満たされたら escalation する。

• max iterations（20）到達
• timeout（60 分）到達
• mergeStateStatus == DIRTY
• mergeStateStatus == DRAFT
• /vibecorp:pr-review-fix が rate limit 停止を返した

teammate 配下の場合（SendMessage が利用可能）

SendMessage ツールが使える場合は teammate と判断し、team-lead に escalation を送る。

to: team-lead
message: "PR #<番号> で /vibecorp:pr-review-loop が <理由> により escalation。状態: <state/mergeStateStatus/reviewDecision> 反復: <n>"

main session の場合（SendMessage が利用不可）

標準出力に escalation 内容を表示して終了する。

6. 結果報告

## /vibecorp:pr-review-loop 完了

- PR: #<pr_number>
- 結果: MERGED / CLOSED / escalated（理由）
- 反復回数: <n>
- 経過時間: <分>
- /vibecorp:pr-review-fix 呼び出し回数: <n>

制約

• --force、--hard、--no-verify は使用しない
• ユーザーの明示的な指示なしに force push しない
• jq では string interpolation \(...) を使わない — 必ず + で結合する（根拠）
• コマンドをそのまま実行する — 2>/dev/null、|| echo、; echo 等のリダイレクトやフォールバックを付加しない（根拠）
• Bash は 1 コマンド 1 呼び出しに分割する — cd ... && cmd | head 2>/dev/null のように cd + パイプ + リダイレクトを含む compound command は Claude Code 本体の built-in security check で止められる（参照: #258）
• /loop や ScheduleWakeup を使わない — このスキルは teammate のターン内で同期完遂することが要件（根拠）