sentinelayer-cli 0.30.8 → 0.30.9

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sentinelayer-cli",
-  "version": "0.30.8",
+  "version": "0.30.9",
   "description": "Scaffold Sentinelayer spec/prompt/guide artifacts with secure browser auth and token bootstrap.",
   "type": "module",
   "scripts": {

package/src/commands/session.js

@@ -4324,8 +4324,9 @@ export function registerSessionCommand(program) {
       const durablePresenceEnabled = options.presence !== false;
       const publishPresence = durablePresenceEnabled && canPublishListenerPresence(agentId);
       const presenceIntervalMs = Math.max(1, presenceIntervalSeconds) * 1000;
-      const presenceKeepaliveMs =
-        Math.max(presenceIntervalSeconds, presenceKeepaliveSeconds, 1) * 1000;
+      const effectivePresenceKeepaliveSeconds =
+        Math.max(presenceIntervalSeconds, presenceKeepaliveSeconds, intervalSeconds, 1);
+      const presenceKeepaliveMs = effectivePresenceKeepaliveSeconds * 1000;
       let lastPresenceHeartbeatMs = 0;
       let lastPresenceHeartbeatFingerprint = "";
 
@@ -4512,7 +4513,7 @@ export function registerSessionCommand(program) {
             const publishLifecycle = {
               ...lifecycle,
               presenceIntervalSeconds,
-              presenceKeepaliveSeconds: Math.max(presenceIntervalSeconds, presenceKeepaliveSeconds),
+              presenceKeepaliveSeconds: effectivePresenceKeepaliveSeconds,
             };
             if (lifecycleType === "heartbeat") {
               const nowMs = Date.now();

package/src/session/listeners.js

@@ -9,6 +9,7 @@ const LISTENER_EVENT_TYPES = new Set([
 // A heartbeat older than this (and not explicitly stopped) means the
 // listener likely died without a clean stop — show it as stale, not live.
 const DEFAULT_STALE_AFTER_MS = 180_000;
+const MAX_STALE_GRACE_MS = 60_000;
 
 function normalizeString(value) {
   return String(value || "").trim();
@@ -29,6 +30,38 @@ function positiveInt(value) {
   return Number.isFinite(n) && n > 0 ? Math.floor(n) : null;
 }
 
+function listenerStaleAfterMs({
+  staleAfterMs,
+  presenceKeepaliveSeconds,
+  presenceIntervalSeconds,
+  cadenceSeconds,
+  idleIntervalSeconds,
+  activeIntervalSeconds,
+} = {}) {
+  const fallbackMs = Math.max(1, Number(staleAfterMs) || DEFAULT_STALE_AFTER_MS);
+  const keepaliveMs = presenceKeepaliveSeconds ? presenceKeepaliveSeconds * 1000 : null;
+  const expectedIntervalSeconds =
+    cadenceSeconds ||
+    presenceIntervalSeconds ||
+    idleIntervalSeconds ||
+    activeIntervalSeconds ||
+    null;
+  const expectedIntervalMs = expectedIntervalSeconds ? expectedIntervalSeconds * 1000 : 0;
+
+  if (keepaliveMs) {
+    // The listener can only publish on a poll tick. Allow one bounded poll
+    // interval after the advertised keepalive, not the old 2.5x keepalive
+    // window that made dead listeners look live for several extra minutes.
+    return keepaliveMs + Math.min(expectedIntervalMs || 0, MAX_STALE_GRACE_MS);
+  }
+
+  if (expectedIntervalMs) {
+    return Math.max(fallbackMs, Math.round(expectedIntervalMs * 2.5));
+  }
+
+  return fallbackMs;
+}
+
 /**
  * Reduce a stream of session_listener_* events into one row per agent: who is
  * listening, at what cadence, and whether they're currently active or idle.
@@ -66,17 +99,14 @@ export function summarizeListeners(events = [], { nowMs = Date.now(), staleAfter
     const cadenceSeconds = active
       ? activeIntervalSeconds || nextPollSeconds
       : idleIntervalSeconds || nextPollSeconds;
-    const expectedPresenceSeconds =
-      presenceKeepaliveSeconds ||
-      presenceIntervalSeconds ||
-      cadenceSeconds ||
-      idleIntervalSeconds ||
-      activeIntervalSeconds ||
-      null;
-    const staleAfterForRowMs = Math.max(
+    const staleAfterForRowMs = listenerStaleAfterMs({
       staleAfterMs,
-      expectedPresenceSeconds ? expectedPresenceSeconds * 2_500 : 0,
-    );
+      presenceKeepaliveSeconds,
+      presenceIntervalSeconds,
+      cadenceSeconds,
+      idleIntervalSeconds,
+      activeIntervalSeconds,
+    });
     let status;
     if (stopped) status = "stopped";
     else if (ageMs !== null && ageMs > staleAfterForRowMs) status = "stale";
@@ -94,6 +124,7 @@ export function summarizeListeners(events = [], { nowMs = Date.now(), staleAfter
       presenceIntervalSeconds,
       presenceKeepaliveSeconds,
       nextPollSeconds,
+      staleAfterSeconds: Math.round(staleAfterForRowMs / 1000),
       lastSeenAt: epoch ? new Date(epoch).toISOString() : null,
       lastSeenAgoSeconds: ageMs !== null ? Math.round(ageMs / 1000) : null,
       lastHumanActivityAt: normalizeString(payload.lastHumanActivityAt) || null,