fix(daemon): skip closed/completing sessions when seeding pr-monitor

saltbo · Copilot · saltbo · commit 511370375450 · 2026-04-15T09:24:41.000-04:00
On daemon restart, prMonitor.start() was seeding tracked tasks from all
on-disk worker sessions including those in 'closed' state. Previously
session files were deleted on completion, but now they are retained for
history lookup. This caused noisy startup logs as the monitor immediately
untracked already-done/cancelled tasks on the first check.

Fix: skip sessions with status 'closed' or 'completing' when seeding,
since those tasks are already terminal and need no PR monitoring.

Co-authored-by: Copilot &lt;223556219+Copilot@users.noreply.github.com&gt;
diff --git a/packages/cli/src/daemon/prMonitor.ts b/packages/cli/src/daemon/prMonitor.ts
@@ -34,7 +34,9 @@ export class PrMonitor {
     // in_progress), we still observe it.
     const sm = getSessionManager();
     for (const s of sm.list({ type: "worker" })) {
-      if (s.taskId) this.trackedTasks.add(s.taskId);
+      if (s.taskId && s.status !== "closed" && s.status !== "completing") {
+        this.trackedTasks.add(s.taskId);
+      }
     }
     saveTrackedTasks(this.trackedTasks);
     this.timer = setInterval(() => this.check(), PR_CHECK_INTERVAL);
