Skip to content

[Refactor] Create-prにPhase 6 CI結果確認を追加#198

Merged
BlueEventHorizon merged 1 commit into
developfrom
refactor/194-ci-check-phase-create-pr
Jul 12, 2026
Merged

[Refactor] Create-prにPhase 6 CI結果確認を追加#198
BlueEventHorizon merged 1 commit into
developfrom
refactor/194-ci-check-phase-create-pr

Conversation

@BlueEventHorizon

Copy link
Copy Markdown
Owner

概要

/anvil:create-pr に Phase 6「CI 結果確認」を追加し、PR 作成後に CI 結果を自動確認するステップを実装した。

背景

PR 作成後に CI 結果を確認せずに次の作業を進めてしまい、CI が pending のまま気づかない・マージ後の追加コミットが未反映のまま気づかないという問題が実際に発生していた(Issue #194)。

やったこと

  • .github/workflows/ の存在確認 → 存在する場合のみ gh pr checks <PR番号> --watch で CI 完了までブロッキング待機するステップ(Phase 6)を追加
  • CI 全件成功時は完了報告に結果を追記
  • CI 失敗時は失敗内容を要約し、AskUserQuestion で対応(修正する / このまま報告して終了)を確認
  • レビュー過程で追加検出した既存ギャップ(Phase 4.1 が既存リモートブランチの stale 状態を検出しない)も、ユーザー判断によりスコープに含めて修正
    • git ls-remote --heads origin <branch> で existence-check と stale 判定の比較方法を統一(remote-tracking ref の fetch 依存を回避)

やらないこと(このプルリクエストのスコープ外とすること)

  • create-pr の入力契約(Issue 番号 / artifacts の明示的な受け渡し)拡張は対象外(既存の意図的な暫定設計、impl-issue/SKILL.md 13-2 に記載済み)
  • gh write 権限の事前確認追加は対象外(別途 needs_review として記録)
  • ポーリング間隔・タイムアウトの設定可能化は対象外(gh pr checks --watch の既定動作に委譲)

レビュー観点

  • Phase 6 の CI 未設定時のスキップ判定
  • Phase 4.1 の stale branch 検出ロジック(git ls-remote --heads ベースの比較)

レビューレベル

  • Lv0: まったく見ないでAcceptする
  • Lv1: ぱっとみて違和感がないかチェックしてAcceptする
  • Lv2: 仕様レベルまで理解して、仕様通りに動くかある程度検証してAcceptする
  • Lv3: 実際に環境で動作確認したうえでAcceptする

備考

Closes #194

@BlueEventHorizon BlueEventHorizon self-assigned this Jul 12, 2026
@BlueEventHorizon
BlueEventHorizon marked this pull request as ready for review July 12, 2026 22:14
@BlueEventHorizon
BlueEventHorizon merged commit b17be5b into develop Jul 12, 2026
1 check passed
@BlueEventHorizon
BlueEventHorizon deleted the refactor/194-ci-check-phase-create-pr branch July 12, 2026 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant