Skip to content

feat: worktreeを作成するディレクトリを変更 (#154)#155

Merged
douhashi merged 1 commit into
mainfrom
soba/154
Nov 3, 2025
Merged

feat: worktreeを作成するディレクトリを変更 (#154)#155
douhashi merged 1 commit into
mainfrom
soba/154

Conversation

@douhashi
Copy link
Copy Markdown
Owner

@douhashi douhashi commented Nov 3, 2025

Implementation Complete

fixes #154

変更内容

  • worktreeのデフォルト作成先を .git/soba/worktrees から /tmp/soba/worktrees に変更
  • DefaultWorktreeBasePath 定数を更新
  • config_template.ymlのデフォルト値を更新
  • 全てのテストファイルのパスを新しいデフォルト値に更新

背景

  • 現在のworktree作成先(.git/soba/worktrees)は、プロジェクトディレクトリ内に作成されるため、Claudeが混乱することがある
  • /tmp配下に作成することで、他のプロジェクトとの干渉を防ぎ、より安定した動作を実現

テスト結果

  • 単体テスト: ✅ Pass
    • config関連テスト
    • service関連テスト
    • cli関連テスト(一部タイムアウトするテストあり、ただし今回の変更とは無関係)
  • 統合テスト: ✅ Pass

注意事項

  • 後方互換性の維持は不要(要求仕様に基づく)
  • 既存のworktreeは手動での移行が必要
  • /tmp ディレクトリはOS再起動時にクリアされる可能性があるが、worktree作成時に親ディレクトリを自動作成する仕組みは既に実装済み

既知の問題

  • TestInitCommandがタイムアウトする問題を発見したが、今回の変更とは無関係(変更前でも再現)
  • ghコマンドがインタラクティブモードになっている可能性がある

チェックリスト

  • 実装計画に従った実装
  • テストカバレッジの確保
  • 既存機能への影響なし

- デフォルトのworktree作成先を.git/soba/worktreesから/tmp/soba/worktreesに変更
- config_template.ymlのデフォルト値を更新
- 全てのテストファイルのパスを新しいデフォルト値に更新
- 後方互換性は不要のため、既存のworktreeについては手動での移行が必要
@douhashi
Copy link
Copy Markdown
Owner Author

douhashi commented Nov 3, 2025

Review Results

✅ Decision

  • Approve (LGTM)
  • Request changes

🔄 Merge Status

  • No conflicts
  • Conflicts exist (rebase required)

👍 Good Points

  • 実装計画に従った適切な実装がされている
  • デフォルト値の変更が一貫して全ファイルに反映されている
  • テストコードも適切に更新されており、テストカバレッジが維持されている
  • 設定テンプレートのコメントが分かりやすく更新されている
  • CI/CDが全て通過している

🔧 Improvement Suggestions

  • 特になし(要求仕様を満たした適切な実装となっている)

📝 Additional Notes

  • /tmp ディレクトリはOS再起動時にクリアされる点は、ドキュメントへの記載があれば尚良いが、必須ではない
  • 既存のworktreeは手動移行が必要だが、後方互換性不要という要件に従っているため問題なし

@douhashi douhashi added the soba:lgtm PR approved for auto-merge label Nov 3, 2025
@douhashi douhashi merged commit b4dafdf into main Nov 3, 2025
3 checks passed
@douhashi douhashi deleted the soba/154 branch November 3, 2025 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

soba:lgtm PR approved for auto-merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

worktreeを作成するディレクトリを変更

1 participant