Cache Keepalive: skip + disarm if the idle timer fired >5s late (sleep/throttle guard)

d82870e0eac66ac100238311f8ea2dbdbcfa78b0

permissionBRICK <permissionBRICK@gmail.com>

public/scripts/extensions/keepalive/index.js+24 -1 Viewed

 // user message can pre-empt the keepalive.
 const EXPIRED_GRACE_MS = 3000;
+// If the idle timer fires more than this many ms later than it was scheduled to, assume the
+// machine slept / the tab was throttled — far more wall-clock time passed than the interval,
+// so the cache almost certainly lapsed. Skip the ping and go dormant rather than guess.
+const MAX_TIMER_DRIFT_MS = 5000;
 const defaultSettings = {
     enabled: false,
     timeoutSeconds: 295,

 // history already present) we have no way to know whether anything is still cached
 // server-side — so pinging before then would be pointless. Resets on page reload.
 let armed = false;
+// Wall-clock time (Date.now() + delay) the pending idle timer is expected to fire, or null when
+// none is scheduled. Used to detect a timer that fired far too late (sleep/throttle).
+let scheduledFireTime = null;
 // Per-profile "time since last message", keyed by connection profile id (or NONE_KEY).
 // In-memory only: on reload every profile is treated as freshly active.
 const lastActivity = {};

 function scheduleIdleTimer() {
     clearIdleTimer();
     if (!armed || !isKeepaliveEnabledForActive()) {
+        scheduledFireTime = null;
         return;
+    }
     const timeoutMs = Math.max(60, Number(extension_settings[MODULE].timeoutSeconds) || 295) * 1000;
     const last = lastActivity[getActiveProfileKey()] ?? Date.now();
     const remaining = timeoutMs - (Date.now() - last);
-    idleTimer = setTimeout(fireKeepalive, remaining > 0 ? remaining : EXPIRED_GRACE_MS);
+    const delay = remaining > 0 ? remaining : EXPIRED_GRACE_MS;
+    scheduledFireTime = Date.now() + delay;
+    idleTimer = setTimeout(fireKeepalive, delay);
+}
 /**

         return;
+    }
+    // If the timer fired much later than scheduled (machine sleep, background-tab throttling,
+    // etc.), far more wall-clock time has elapsed than the idle interval, so the cache has
+    // almost certainly expired. Don't ping on a stale assumption — go dormant until the next
+    // real message re-arms keepalive, exactly like a freshly opened tab.
+    if (scheduledFireTime !== null && (Date.now() - scheduledFireTime) > MAX_TIMER_DRIFT_MS) {
+        console.debug('[Keepalive] Idle timer fired', Date.now() - scheduledFireTime, 'ms late; skipping and disarming until next activity.');
+        armed = false;
+        clearIdleTimer();
+        scheduledFireTime = null;
+        return;
+    }
     // Never start while any generation is in progress, or while a previous
     // keepalive is still running. Reschedule and bail.
     if (keepaliveActive || isGenerating() || streamingProcessor) {