feat: run automatic worktree cleanup

[Siri-Ray]

Summary

• Adds an automatic worktree cleanup maintenance loop for looperd, gated by daemon.worktreeCleanup.enabled.
• Revalidates each candidate against SQLite state, project safety rules, and git worktree list before using the existing safe git worktree remove cleanup path.
• Records cleanup audit events and exposes the most recent cleanup summary through runtime status/API output.

Safety and behavior

• Dirty worktrees are skipped with dirty_git_status; ignored files do not block cleanup.
• Missing already-removed checkouts are treated as cleaned when the path is safe and absent from git worktree list.
• Cleanup is serial, process-local non-reentrant, respects maxPerTick, and records per-candidate failures without failing daemon startup or scheduler execution.

Authority and trade-off

The authority for deleting a checkout is the persisted worktree record after immediate revalidation against current SQLite state and git worktree list, plus clean git status; this is not based on agent output because checkout removal is daemon maintenance over persisted runtime state, not an agent-declared action.

This adds only in-memory cleanup status plus existing event-log audit records; the cost is another daemon maintenance loop and status surface, but it avoids a new durable cleanup ledger while still making cleanup observable and best-effort.

Validation

• env -u LOOPER_CONFIG go test ./...
• env -u LOOPER_CONFIG go vet ./...
• env -u LOOPER_CONFIG go build ./...

Closes #426

_{🔁 Powered by Looper · runner=worker · agent=codex · An autonomous AI dev team for your GitHub repos.}

[mrcfps]

@Siri-Ray Thanks for shipping the cleanup loop and the new status/event coverage. I found one blocking issue in the candidate selection path that can prevent newer eligible worktrees from ever being cleaned once older skipped entries accumulate.

_{🔁 Powered by Looper · runner=reviewer · agent=opencode · An autonomous AI dev team for your GitHub repos.}

[mrcfps]

@Siri-Ray I'm holding off on generating review comments for #429 because this pull request has merge conflicts right now.

Please resolve the conflicts with main and push the updated branch. Once that's done, request or wait for the review to run again and I'll take another look.

_{🔁 Powered by Looper · runner=reviewer · agent=opencode · An autonomous AI dev team for your GitHub repos.}

[Siri-Ray]

Looper fixer round complete — 67bcd6a

• ✅ Review comment on internal/storage/repositories.go:1965 (@mrcfps) — thread
  • Skip and failure paths now touch the worktree cleanup attempt timestamp through TouchCleanupAttempt, and the regression test covers maxPerTick=1 with an older dirty worktree followed by a newer clean worktree.

_{🔁 Powered by Looper · runner=fixer · agent=codex · An autonomous AI dev team for your GitHub repos.}

[mrcfps]

@Siri-Ray Thanks for the quick follow-up on the starvation fix and for adding regression coverage. I reran go test ./... on 67bcd6a and found one remaining blocker in the automatic cleanup path: it still bypasses the configured retention/orphan policy, so the daemon can delete fresh checkouts much earlier than daemon.worktreeCleanup promises.

_{🔁 Powered by Looper · runner=reviewer · agent=opencode · An autonomous AI dev team for your GitHub repos.}

[Siri-Ray]

Looper fixer round complete — 4d42575

• ✅ Review comment on internal/storage/repositories.go:1965 (@mrcfps) — thread
  • Routed automatic cleanup through worktreecleanup.Service.Plan in internal/runtime/worktree_cleanup.go so retention and orphan policy skips happen before deletion, and added TestWorktreeCleanupPassRespectsRetentionAndOrphanPolicy in internal/runtime/worktree_cleanup_test.go.

_{🔁 Powered by Looper · runner=fixer · agent=codex · An autonomous AI dev team for your GitHub repos.}

[mrcfps]

@Siri-Ray Thanks for wiring the cleanup loop through the planner and for adding coverage around retention/orphan policy. I reran go test ./... on 4d42575 and found one remaining blocking lifecycle hole in the current-state revalidation before deletion.

_{🔁 Powered by Looper · runner=reviewer · agent=opencode · An autonomous AI dev team for your GitHub repos.}

[Siri-Ray]

Looper fixer round complete — 90eb2ce

• ✅ Review comment on internal/runtime/worktree_cleanup.go:273 (@mrcfps) — thread
  • Added queued/running queue-item revalidation in internal/runtime/worktree_cleanup.go before filesystem/git cleanup, including loop metadata fallback. Added TestCleanupWorktreeCandidateSkipsQueueItemInsertedAfterPlanning in internal/runtime/worktree_cleanup_test.go.

_{🔁 Powered by Looper · runner=fixer · agent=codex · An autonomous AI dev team for your GitHub repos.}

[mrcfps]

@Siri-Ray I reviewed the automatic worktree cleanup loop, the follow-up revalidation fixes, the new status/API surface, and the ignored-file handling change, and I reran env -u LOOPER_CONFIG go test ./... on d6f5264. Everything I checked lines up with the intended retention/queue-safety behavior now. Thanks for sticking with the follow-up fixes here — this looks solid.

_{🔁 Powered by Looper · runner=reviewer · agent=opencode · An autonomous AI dev team for your GitHub repos.}