fix(worktree): force-remove leftover dir so claims don't loop on 'already exists' #21

Merged

janpeter merged 1 commit from fix/remove-worktree-fs-fallback into main

2026-05-28 15:09:10 +02:00

janpeter commented

2026-05-28 14:14:33 +02:00

Owner

Probleem

SPRINT_IMPLEMENTATION-jobs (en in principe elke job met een worktree) konden permanent vastlopen in een loop: claim → getFullJobContext geeft null → rollback → re-queue → opnieuw. Waargenomen op 2026-05-27: job cmpnyil26000pvi7rjjncv5sz deed 20 mislukte runs tot hij met de hand op CANCELLED ging.

Oorzaak

Na een mislukte run (bv. Anthropic 529 "Overloaded") bleef de worktree-map /home/agent/.scrum4me-agent-worktrees/<jobId> staan. removeWorktreeForJob deed alleen git worktree remove --force. Als git het pad niet (meer) als worktree kent — half-afgebroken git worktree add, handmatige leftover, of een container-recreate die de clone wiste — faalt dat commando, blijft de map staan, en gooit createWorktreeForJob vervolgens Worktree path already exists (src/git/worktree.ts) → getFullJobContext geeft null.

Dit is de defense-in-depth aanvulling op #20, dat rollbackClaim al de sprint-worktree + sprint_task_executions laat opruimen. Die fix dekt de happy-path rollback; deze PR dekt het geval waarin het pad bestaat maar git het niet als worktree herkent.

Fix

removeWorktreeForJob: als git worktree remove --force faalt, fallback naar fs.rm(path, { recursive, force }) + git worktree prune, zodat het pad altijd verdwijnt en een volgende claim niet opnieuw loopt.

Test

Nieuwe vitest-case in __tests__/git/worktree.test.ts: een plain directory op het worktree-pad die git niet als worktree kent, wordt nu force-verwijderd (removed: true, map weg). Faalde vóór de fix met fatal: '…' is not a working tree.

worktree-suite: 11/11 groen
tsc --noEmit: clean

## Probleem `SPRINT_IMPLEMENTATION`-jobs (en in principe elke job met een worktree) konden permanent vastlopen in een loop: claim → `getFullJobContext` geeft `null` → rollback → re-queue → opnieuw. Waargenomen op 2026-05-27: job `cmpnyil26000pvi7rjjncv5sz` deed 20 mislukte runs tot hij met de hand op CANCELLED ging. ## Oorzaak Na een mislukte run (bv. Anthropic 529 "Overloaded") bleef de worktree-map `/home/agent/.scrum4me-agent-worktrees/<jobId>` staan. `removeWorktreeForJob` deed alleen `git worktree remove --force`. Als git het pad **niet (meer) als worktree kent** — half-afgebroken `git worktree add`, handmatige leftover, of een container-recreate die de clone wiste — faalt dat commando, blijft de map staan, en gooit `createWorktreeForJob` vervolgens `Worktree path already exists` (`src/git/worktree.ts`) → `getFullJobContext` geeft `null`. Dit is de defense-in-depth aanvulling op #20, dat `rollbackClaim` al de sprint-worktree + `sprint_task_executions` laat opruimen. Die fix dekt de happy-path rollback; deze PR dekt het geval waarin het pad bestaat maar git het niet als worktree herkent. ## Fix `removeWorktreeForJob`: als `git worktree remove --force` faalt, fallback naar `fs.rm(path, { recursive, force })` + `git worktree prune`, zodat het pad altijd verdwijnt en een volgende claim niet opnieuw loopt. ## Test Nieuwe vitest-case in `__tests__/git/worktree.test.ts`: een plain directory op het worktree-pad die git niet als worktree kent, wordt nu force-verwijderd (`removed: true`, map weg). Faalde vóór de fix met `fatal: '…' is not a working tree`. - worktree-suite: 11/11 groen - `tsc --noEmit`: clean

janpeter added 1 commit

2026-05-28 14:14:33 +02:00

fix(worktree): force-remove leftover dir when git no longer tracks it 7bb62b067b

removeWorktreeForJob threw when `git worktree remove` hit a path git
doesn't register as a worktree (partial `worktree add`, manual leftover,
or a container recreate that wiped the clone). The directory then
survived rollback cleanup, so the next claim looped forever on
"Worktree path already exists" -> getFullJobContext null (notably for
SPRINT_IMPLEMENTATION jobs after a transient Claude failure). Fall back
to fs.rm + `git worktree prune` so cleanup always clears the path.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

janpeter merged commit d9c97fcc44 into main

2026-05-28 15:09:10 +02:00

janpeter referenced this pull request from a commit

2026-05-28 15:09:11 +02:00

Merge pull request 'fix(worktree): force-remove leftover dir so claims don't loop on 'already exists'' (#21) from fix/remove-worktree-fs-fallback into main

janpeter referenced this pull request

2026-06-17 09:34:04 +02:00

feat: JobKindConfig fase 3 — mcp live-resolutie + codex-args overrides #59