PJR-DT3Q git の一時的な失敗の再試行が worktree の作成で失敗する
1. 課題内容
src/exec/worktree.ts は dubious ownership などの一時的な git の失敗を再試行する。git worktree add -b は最初の試みで worktree と branch を作ったうえで失敗を返すことがあり、再試行が branch がすでにあるで失敗して run が waiting になる。2026-10-17 の PJR-MX4F で起きた
2. 影響範囲
| 観点 | 影響 |
|---|---|
| スコープ | src/exec/worktree.ts の一時的な git の失敗の再試行（runGitWithTransientRetry）と、それを使う worktree の作成 |
| スケジュール | 急がないが、exec run を無駄に止めるため早めに直す |
| コスト | 再試行が失敗すると run が waiting になり、残った worktree と branch を人が片付けてから実行し直す必要がある |
| 品質 | 一時的な失敗のための再試行が、かえって失敗の原因になる |
| 関係者 | exec の利用者 |
3. 対応方針
| 項目 | 内容 |
|---|---|
| 原因 | git worktree add <path> -b <branch> は、dubious ownership の一時的なエラーを返したときも worktree と branch を作り終えていることがある。再試行は同じ引数で起動するため、a branch named ... already exists で失敗する |
| 対応策 | 一時的な失敗のあとに再試行する前に、目的の状態がすでにできているかを確かめる。worktree の作成では、目的の path に目的の branch の worktree があり、その先端が分岐元と同じなら、作成済みとして扱う。できかけの状態（branch だけある、など）なら片付けてから再試行する |
| 依存事項 | なし |
| 完了条件 | 下記の完了条件を満たす |
3.1. 完了条件
- worktree の作成が一時的なエラーを返し、実際には worktree と branch ができていた場合に、再試行で失敗せず、そのまま run を続ける。
- branch だけができていて worktree がない、などのできかけの状態から、再試行で正しく作り直す。分岐元と違う commit を指す既存の branch は消さずに失敗させ、理由を出す。
- ほかの git の操作の再試行で、同じように「やり直すと失敗する」ものがないかを調べ、result に書く。
- 単体テストか統合テストで、作成済みの場合とできかけの場合を確かめる。
- 作業を終える前に、
npm run typecheck・npm run lint:ts・npm run test:unitを実行して通す。 - 親の検証（lint-ts・lint-fm・lint-md・typecheck・test-unit・test-integration・validate-schema）が通る。
- runner のコードを変えるため、統合の後に orchestrator が
exec run --registerを 1 回流し、統合まで終わることを確かめる（本件の close の条件）。
4. 対応結果
runGitWithTransientRetryに、一時的な失敗の後で非冪等なコマンドを再実行する前に目的状態を回復・照合するフックを追加した。git worktree addでは、目的の path・branch・分岐元 commit が作成済みなら成功として続行する。branch だけが同じ commit に作られた場合は、その branch と未登録 path を除去してから再試行する。- 既存 branch が分岐元と異なる commit を指す場合、または別 worktree で使用中の場合は削除せず、理由を示して失敗させる。
- 作成完了後に一時エラーが返る場合、branch だけが残る場合、異なる commit の branch がある場合のテストを追加した。共通の単体・統合テスト、型検査、lint は executor 終了後の親 runner が実行する。
- 共通再試行を使うほかの変更系 Git 操作も確認した。
commit、merge、worktree remove、branch 削除、stash dropなどは一般には再実行が非冪等になり得るが、今回のように複数段階の処理後に ownership エラーを返した既知事例はworktree addだけである。今回追加した回復フックは、同様の事例が確認されたコマンドへ個別の完了状態照合を追加できる形にした。