fleet/fleetd
3
0
Fork 0
Code Issues 73 Pull Requests Actions Packages Projects Releases 1 Wiki Activity

CB-576: release preserves a dirty worktree instead of deleting it #53

Merged
ltms merged 2 commits from worker/cb576-01a04b-17 into main 2026-08-15 08:54:33 +02:00
Conversation 0 Commits 2 Files Changed 7
+139 -1
agent commented 2026-08-15 07:20:31 +02:00
Member
Copy Link
Copy Source

Releasing a COMPLETED session whose worktree holds uncommitted work now preserves the directory and logs a WARN instead of --force-deleting the worker's only copy of its work (the exact shape lost in the incident: a worker that ended a turn without committing).

  • Worktrees gains hasUncommitted (git status --porcelain, incl. untracked-only).
  • SessionManager checks it before any non-preserving remove; dirty => preserve + WARN naming path/session/cause.
  • Worktrees.remove keeps its --force contract; the check lives at the caller layer.

Tests: mvn -f bridged/pom.xml clean install -> BUILD SUCCESS, Tests run: 678, Failures: 0, Errors: 0, Skipped: 0

Releasing a COMPLETED session whose worktree holds uncommitted work now preserves the directory and logs a WARN instead of --force-deleting the worker's only copy of its work (the exact shape lost in the incident: a worker that ended a turn without committing). - Worktrees gains hasUncommitted (git status --porcelain, incl. untracked-only). - SessionManager checks it before any non-preserving remove; dirty => preserve + WARN naming path/session/cause. - Worktrees.remove keeps its --force contract; the check lives at the caller layer. Tests: mvn -f bridged/pom.xml clean install -> BUILD SUCCESS, Tests run: 678, Failures: 0, Errors: 0, Skipped: 0
ltms added 1 commit 2026-08-15 07:59:54 +02:00
CB-576: release preserves a dirty worktree instead of deleting it
CI / contract (pull_request) Successful in 1m2s
Details
CI / build (pull_request) Successful in 1m35s
Details
9118ce2537
ltms force-pushed worker/cb576-01a04b-17 from 0b081deb5e to 9118ce2537 2026-08-15 07:59:54 +02:00 Compare
ltms added 1 commit 2026-08-15 08:48:59 +02:00
CB-576: hasUncommitted tolerates an already-gone worktree
CI / contract (pull_request) Successful in 1m8s
Details
CI / build (pull_request) Successful in 1m37s
Details
b525b0f08f
ltms merged commit 500bfa2c33 into main 2026-08-15 08:54:33 +02:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
blocked
Cannot start until something else lands. The body says what.
 needs-live-proof
Merged and green, but never shown working on the running daemon. Not the same as done.
 ready-to-delegate
Scope, files and acceptance criteria are written. A worker can be briefed from the body alone.
 silent-default
A feature that compiles, passes tests, and ships turned off. Nine recurrences and counting.
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
agent kevin ltms
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: fleet/fleetd#53
Delete Branch "worker/cb576-01a04b-17"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?