plan/implement/revise実行前に、git pullする

[douhashi]

要求

• plan / implement / revise 実行前(worktree作成前) に親リポジトリの最新化を行ってほしい。なぜならば、worktreeの作成元が古いと、コンフリクトが発生したり、前段の実装に依存する実装がうまくできない可能性があるからだ

[douhashi]

Implementation Plan: plan/implement/revise実行前のgit pull機能実装

Requirements Overview

• 目的: worktree作成前に親リポジトリを最新化し、古いコードベースに起因するコンフリクトを回避する
• 背景: 現在、worktreeが古い状態のコードベースから作成されると、最新の変更との不整合やコンフリクトが発生する可能性がある
• 機能要件: plan/implement/reviseの各フェーズ開始前（worktree作成前）に、自動的にベースブランチを最新化する
• 受け入れ基準:
  • 各フェーズ実行前に、親リポジトリのベースブランチが自動的に最新化される
  • リモートとの同期が失敗した場合は適切にエラーハンドリングされる

Design Policy

• 技術選択: 既存のgit fetch実装をgit pullに拡張
• 選択理由:
  • git fetchのみでは参照の更新のみで、ローカルブランチが更新されない
  • git pullにより、ローカルブランチを確実にリモートと同期できる
• アーキテクチャ考慮事項:
  • 既存のUpdateBaseBranchメソッドを改善し、完全な同期を実現
  • worktree作成前の処理フローに統合
  • エラー時の適切なリカバリー戦略

Implementation Steps

1. GitClientインターフェースの拡張

   • 作業内容: UpdateBaseBranchメソッドをgit pull対応に改良
   • 関連ファイル: internal/infra/git/client.go

2. UpdateBaseBranchメソッドの実装改善

   • 作業内容: git fetchからgit pullへの変更実装
   • 関連ファイル: internal/infra/git/client.go
   • 詳細: 現在のブランチを保存し、ベースブランチに切り替えてpullを実行後、元のブランチに戻る

3. PrepareWorkspaceメソッドの改善

   • 作業内容: worktree作成前のベースブランチ更新処理を強化
   • 関連ファイル: internal/service/git_workspace.go
   • 詳細: エラーハンドリングとログ出力の改善

4. 単体テストの更新

   • 作業内容: 新しいgit pull動作をカバーするテストケース追加
   • 関連ファイル:
     • internal/infra/git/client_test.go
     • internal/service/git_workspace_test.go

5. 統合テストの作成

   • 作業内容: エンドツーエンドのワークフロー検証
   • 関連ファイル: internal/service/workflow_executor_test.go

Test Plan

• 単体テスト:
  • UpdateBaseBranchがgit pullを正しく実行することを確認
  • リモートが存在しない場合のエラーハンドリング
  • ブランチの切り替えと復元が正しく行われることを確認
• 統合テスト:
  • plan/implement/revise各フェーズでベースブランチが更新されることを確認
  • worktree作成と連携して正しく動作することを検証

Risks and Countermeasures

• リスク: git pullでコンフリクトが発生する可能性
  • 対策: fast-forwardマージのみを許可（--ff-onlyオプション）、コンフリクト時はエラーとして処理
• リスク: ネットワーク障害によるpull失敗
  • 対策: 適切なタイムアウトとリトライ機構、失敗時も処理を続行できる選択肢を提供
• リスク: 現在のブランチに未コミットの変更がある場合
  • 対策: stashを使用した一時退避、または明確なエラーメッセージ表示

Schedule

• 見積もり: 合計 4時間
• ステップ別:
  • GitClient実装改善: 1.5時間
  • PrepareWorkspace改善: 0.5時間
  • テスト作成・更新: 2時間

[douhashi]

Created PR #150