fix(daemon): exclude closed sessions from stale cleanup and audit

saltbo · Copilot · saltbo · commit 04f46caa820a · 2026-04-15T10:54:42.000-04:00
cleanupStaleSessions was treating 'closed' worker sessions as stale,
calling closeSession on an already-closed server session and deleting
the local file. auditOrphanedTasks was also making unnecessary API
calls for closed sessions.

Exclude closed sessions from both paths — they are intentionally
retained on disk for tunnel history lookups.

Co-authored-by: Copilot &lt;223556219+Copilot@users.noreply.github.com&gt;
diff --git a/packages/cli/src/daemon/cleanup.ts b/packages/cli/src/daemon/cleanup.ts
@@ -50,7 +50,9 @@ export async function cleanupStaleSessions(client: MachineClient, machineId: str
         // orphaned at boot time (no live handle can exist yet).
         // Leader sessions: pid liveness.
         if (local?.type === "worker") {
-          if (local.status === "in_review") continue;
+          // in_review survives restarts (reject-resume entry point).
+          // closed sessions are intentionally retained for history lookup — skip.
+          if (local.status === "in_review" || local.status === "closed") continue;
         } else if (local?.type === "leader") {
           if (isPidAlive(local.pid)) continue;
         }
@@ -79,7 +81,7 @@ export async function cleanupStaleSessions(client: MachineClient, machineId: str
 // the right action depends on context (release vs cancel vs manual merge).
 export async function auditOrphanedTasks(client: MachineClient, _machineId: string): Promise<void> {
   try {
-    const workers = listSessions({ type: "worker" });
+    const workers = listSessions({ type: "worker" }).filter((s) => s.status !== "closed");
     let resumeQueued = 0;
     let diverged = 0;
 
