@cohortapp/agent-sdk 2.18.11 → 2.18.12

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cohortapp/agent-sdk",
-  "version": "2.18.11",
+  "version": "2.18.12",
   "description": "Cohort Agent SDK — autonomous AI colleague runtime. Deploy senior AI colleagues on dedicated Mac minis, wired to the Cohort operating surface.",
   "type": "module",
   "bin": {

package/scripts/daemon/agent-daemon.mjs

@@ -160,7 +160,7 @@ import { processOne } from "../../lib/execution/pipeline.mjs";
 // gate is pure (lib/session/frontdoor.mjs); the assurance sweep reopens a
 // session claim that was neither replied nor done within 20 min.
 import { shouldReviveFrontDoor, reviveCommand, sessionJobLabelOnDisk } from "../../lib/session/revive.mjs";
-import { makeFrontDoorGate, sessionLiveFromHeartbeat } from "../../lib/session/frontdoor.mjs";
+import { makeFrontDoorGate, sessionLiveFromHeartbeat, readFrontDoorState } from "../../lib/session/frontdoor.mjs";
 import { CADENCE_REGISTRY } from "./cadence-handlers.mjs";
 import { sweepSessionInboxForDaemon } from "../../lib/session/inbox-claims.mjs";
 import { defaultEffects, scheduleToQueue } from "../../lib/execution/effects.mjs";
@@ -291,6 +291,8 @@ const _frontDoorGate = makeFrontDoorGate({ agentRoot: AGENT_REPO_DIR });
 // State is per-process and deliberately not persisted: a daemon restart is
 // itself a change of circumstances, and a fresh budget after one is the answer
 // we want rather than a stale count carried across it.
+/** How often the daemon asks whether the front door has gone quiet. */
+const REVIVE_CHECK_INTERVAL_MS = 60 * 1000;
 const _revive = { attempts: 0, lastAt: null };
 async function reviveFrontDoorIfShut(state, deps = {}) {
   const now = deps.now ? deps.now() : Date.now();
@@ -325,9 +327,6 @@ async function pollService(svc) {
     // Front door: a live main session owns this service's inbox — leave it.
     const fd = _frontDoorGate.check(svc.name);
     if (fd.changed) console.log(`[daemon] front door for ${svc.name}: ${fd.dispatch ? "daemon dispatches" : "main session owns intake"} (${fd.reason})`);
-    // A door that is shut is not a door that owns anything. Ask on every poll,
-    // cheap and bounded: the common answer is "answering" off a fresh beat.
-    if (svc.name === "cohort") await reviveFrontDoorIfShut(fd.state).catch(() => {});
     if (!fd.dispatch) return 0;
     const result = await svc.fn();
     // Dedup by raw_ref first: the same Slack message arrives via multiple paths
@@ -3150,6 +3149,22 @@ async function main() {
     pollCohortFast().catch((err) => console.error("[daemon] cohort fast-poll error:", err.message));
   }, COHORT_FAST_INTERVAL);
 
+  // ── THE SHUT-DOOR CHECK GETS ITS OWN TICK ─────────────────────────────────
+  //
+  // It lived inside pollService for one release, which was the wrong place for
+  // exactly the reason it exists: `pollCohortFast` guards itself with
+  // `_cohortFastRunning`, so a poll that hangs starves every later call — and a
+  // seat whose front door is wedged is precisely a seat where things hang. The
+  // remedy must not share a lane with the thing it is rescuing.
+  //
+  // Its own interval, reading the front-door state directly, so nothing between
+  // here and the kickstart can be blocked by the wedge.
+  setInterval(() => {
+    let state = null;
+    try { state = readFrontDoorState(AGENT_REPO_DIR, { now: Date.now() }); } catch { state = null; }
+    if (state) reviveFrontDoorIfShut(state).catch(() => {});
+  }, REVIVE_CHECK_INTERVAL_MS);
+
   // Backlog sweep
   setInterval(async () => {
     try {