各フェーズ実行時、必ずworktreeに移動するように

[douhashi]

• 現在は implement / revise の開始時にworktreeを作成/移動している
• planやreview時にもworktreeを作成/移動するようにしてほしい

[douhashi]

Implementation Plan: 各フェーズ実行時、必ずworktreeに移動するように

Requirements Overview

• 目的: plan/review時にもworktreeを作成・移動するようにし、全フェーズで一貫した実行環境を提供
• 背景: 現在はimplement/reviseのみでworktreeが作成されるため、plan/review時にissue固有のディレクトリで作業できない
• 受入基準:
  • plan/reviewフェーズ実行時にworktreeが自動的に作成される
  • tmuxコマンド実行時に適切なworktreeディレクトリへ移動している

Design Policy

• 技術選択: 既存のGitWorkspaceManager/workflow_executorの仕組みを拡張
• 設計方針:
  • フェーズ定義（PhaseDefinition）のRequiresWorktreeフラグを活用
  • 既存のworktree作成/移動ロジックを再利用
  • 後方互換性は考慮せず、シンプルで保守しやすい実装を優先

Implementation Steps

1. フェーズ定義の更新

• 作業内容: plan/reviewフェーズのRequiresWorktreeをtrueに設定
• 関連ファイル: internal/domain/phase.go
• 詳細:
  • planフェーズ: RequiresWorktree: true（既に設定済み）
  • reviewフェーズ: RequiresWorktree: falseをtrueに変更

2. workflow_executor.goの修正

• 作業内容: reviewフェーズでもworktreeディレクトリへ移動するようコマンド実行ロジックを調整
• 関連ファイル: internal/service/workflow_executor.go
• 詳細:
  • executeCommand関数内のrequiresWorktree判定ロジックを確認
  • planとreviewでもworktreeディレクトリへのcd処理が実行されることを確認

3. テストの追加・更新

• 作業内容: plan/reviewフェーズのworktree作成・移動をカバーするテストを追加
• 関連ファイル:
  • internal/service/workflow_executor_test.go
  • internal/domain/phase_definition_test.go
• 詳細:
  • TestExecutePhaseにplan/reviewのケースを追加
  • PrepareWorkspaceが呼ばれることを検証
  • cdコマンドが適切に生成されることを検証

4. 統合テストの実施

• 作業内容: 実際のワークフローでplan/reviewフェーズが正しく動作することを確認
• 関連ファイル: internal/service/integration_test.go
• 詳細:
  • 各フェーズでworktreeが作成されることを確認
  • tmuxペインで正しいディレクトリに移動することを確認

Test Plan

• 単体テスト:

  • PhaseDefinitionのRequiresWorktreeフラグが正しく設定されているか
  • executeCommandでplan/reviewフェーズでもcdコマンドが生成されるか
  • PrepareWorkspaceが全フェーズで呼ばれるか

• 統合テスト:

  • sobaワークフロー全体を通じてworktreeが適切に管理されるか
  • tmuxセッション内で正しいディレクトリにいるか

Risks and Countermeasures

• リスク: 既存のplan/reviewフェーズを実行中のIssueで予期しない動作

  • 対策: worktreeが既に存在する場合は再利用する仕組みがあるため影響は最小限

• リスク: worktree作成によるディスク容量の増加

  • 対策: 既存のCleanupWorkspace機能を活用し、完了後の削除を徹底

Schedule

• 見積もり: 合計 3時間
• 内訳:
  • フェーズ定義更新: 0.5時間
  • workflow_executor修正: 1時間
  • テスト追加・更新: 1時間
  • 統合テスト実施: 0.5時間

[douhashi]

Created PR #157