@trevonistrevon/pi-loop 0.5.5 → 0.5.7

package/dist/runtime/session-runtime.js

@@ -1,7 +1,29 @@
+// Wall-clock cadence for the idle heartbeat that pumps the scheduler. Cron is
+// minute-granular, so 30s gives sub-minute wake latency while idle.
+const HEARTBEAT_MS = 30_000;
 export function registerSessionRuntimeHooks(options) {
     const { pi, getLoopScope, getPiLoopEnv, recreateSessionStore, clearAllLoops, getStore, getScheduler, getTriggerSystem, setLatestCtx, setSessionId, widget, notificationRuntime, flushPendingNotifications, cleanupTaskBacklogLoops, hasPendingTasks, cleanDoneTasks, } = options;
     let storeUpgraded = false;
     let persistedShown = false;
+    let heartbeatTimer;
+    // The CronScheduler is pump-driven; without this heartbeat it only advances at
+    // turn boundaries (turn_start/agent_end), so a loop whose fire time elapses
+    // while the agent is idle would never fire and never re-wake the agent. The
+    // timer is unref'd so it never keeps a one-shot (`pi -p`) process alive.
+    function ensureHeartbeat() {
+        if (heartbeatTimer)
+            return;
+        heartbeatTimer = setInterval(() => {
+            void pumpLoops();
+        }, HEARTBEAT_MS);
+        heartbeatTimer.unref?.();
+    }
+    function stopHeartbeat() {
+        if (!heartbeatTimer)
+            return;
+        clearInterval(heartbeatTimer);
+        heartbeatTimer = undefined;
+    }
     function upgradeStoreIfNeeded(ctx) {
         if (storeUpgraded)
             return;
@@ -20,6 +42,7 @@ export function registerSessionRuntimeHooks(options) {
             getStore().clearExpired();
             getStore().expireEventLoops(sessionStartedAt);
             getTriggerSystem().start();
+            ensureHeartbeat();
             widget.update();
         }
     }
@@ -46,6 +69,7 @@ export function registerSessionRuntimeHooks(options) {
         setSessionId(ctx.sessionManager.getSessionId());
         widget.setUICtx(ctx.ui);
         upgradeStoreIfNeeded(ctx);
+        ensureHeartbeat();
         widget.update();
         await pumpLoops();
     });
@@ -53,6 +77,7 @@ export function registerSessionRuntimeHooks(options) {
         setLatestCtx(ctx);
         widget.setUICtx(ctx.ui);
         upgradeStoreIfNeeded(ctx);
+        ensureHeartbeat();
         showPersistedLoops();
         widget.update();
     });
@@ -76,12 +101,14 @@ export function registerSessionRuntimeHooks(options) {
         await pumpLoops();
     });
     pi.on("session_shutdown", async () => {
+        stopHeartbeat();
         notificationRuntime.clear("session_shutdown");
     });
     pi.on("session_switch", async (event, ctx) => {
         setLatestCtx(ctx);
         widget.setUICtx(ctx.ui);
         getTriggerSystem().stop();
+        stopHeartbeat();
         notificationRuntime.clear("session_switch");
         setSessionId(undefined);
         const isResume = event?.reason === "resume";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@trevonistrevon/pi-loop",
-  "version": "0.5.5",
+  "version": "0.5.7",
   "description": "A pi extension for cron/event-based agent re-wake loops and background process monitoring.",
   "author": "trevonistrevon",
   "license": "MIT",

package/src/index.ts

@@ -55,6 +55,9 @@ export default function (pi: ExtensionAPI) {
   let scheduler: CronScheduler;
   let triggerSystem: TriggerSystem;
   const widget = new LoopWidget(store, monitorManager);
+  // Repaint the status bar when a monitor finishes/prunes on its own (no tool
+  // call), so stale monitors don't linger in the count between turns.
+  monitorManager.setOnChange(() => widget.update());
   widget.setTaskSummaryProvider(() => {
     if (!nativeTaskStore) return { count: 0 };
     const tasks = nativeTaskStore.list().filter(t => t.status === "pending" || t.status === "in_progress");

package/src/monitor-manager.ts

@@ -11,9 +11,20 @@ import type { MonitorEntry, MonitorProcess } from "./types.js";
 export class MonitorManager {
   private processes = new Map<string, MonitorProcess>();
   private nextId = 1;
+  private onChange: (() => void) | undefined;
 
   constructor(private pi: ExtensionAPI) {}
 
+  /**
+   * Register a callback fired when a monitor's status changes or it is pruned
+   * (e.g. autonomous completion/error/stop/prune with no tool call). Used to
+   * repaint the status widget, which otherwise only refreshes on turn
+   * boundaries and explicit tool actions. Not fired for output lines.
+   */
+  setOnChange(cb: () => void): void {
+    this.onChange = cb;
+  }
+
   private toReducerState(): MonitorReducerState {
     return {
       nextId: this.nextId,
@@ -34,9 +45,31 @@ export class MonitorManager {
       }
       process.entry = updated;
     }
+    // Repaint on status/set transitions, but not on the frequent output-line
+    // events (which never change the visible count).
+    if (event.type !== "MONITOR_OUTPUT" && event.type !== "MONITOR_ONDONE_REGISTERED") {
+      this.onChange?.();
+    }
     return result;
   }
 
+  // Remove a finished monitor (completed/errored/stopped) after a brief delay so
+  // tool consumers can still read its final state via MonitorList. Shared by
+  // finish() and stop() so every terminal status is pruned consistently — a
+  // stopped monitor that is never pruned lingers in list() and the widget count.
+  private schedulePrune(id: string): void {
+    setTimeout(() => {
+      this.applyReducerEvent({
+        type: "MONITOR_PRUNED",
+        at: Date.now(),
+        source: "system",
+        entityType: "monitor",
+        entityId: id,
+        payload: { id },
+      });
+    }, 30000);
+  }
+
   create(command: string, description?: string, timeout = 300000): MonitorEntry {
     const now = Date.now();
     const result = reduceMonitorState(this.toReducerState(), {
@@ -132,18 +165,7 @@ export class MonitorManager {
       bp.completionCallbacks = [];
       for (const resolve of bp.waiters) resolve();
       bp.waiters = [];
-      // Remove completed/errored monitors after a brief delay so tool
-      // consumers have time to read the final state via MonitorList.
-      setTimeout(() => {
-        this.applyReducerEvent({
-          type: "MONITOR_PRUNED",
-          at: Date.now(),
-          source: "system",
-          entityType: "monitor",
-          entityId: id,
-          payload: { id },
-        });
-      }, 30000);
+      this.schedulePrune(id);
     };
 
     child.on("close", (code) => {
@@ -214,6 +236,7 @@ export class MonitorManager {
         reason: "manual",
       },
     });
+    this.schedulePrune(id);
     bp.proc.kill("SIGTERM");
 
     await new Promise<void>((resolve) => {

package/src/runtime/session-runtime.ts

@@ -7,6 +7,10 @@ export interface SessionSwitchEvent {
   reason?: string;
 }
 
+// Wall-clock cadence for the idle heartbeat that pumps the scheduler. Cron is
+// minute-granular, so 30s gives sub-minute wake latency while idle.
+const HEARTBEAT_MS = 30_000;
+
 export interface SessionRuntimeOptions {
   pi: ExtensionAPI;
   getLoopScope: () => LoopScope;
@@ -48,6 +52,25 @@ export function registerSessionRuntimeHooks(options: SessionRuntimeOptions): voi
 
   let storeUpgraded = false;
   let persistedShown = false;
+  let heartbeatTimer: ReturnType<typeof setInterval> | undefined;
+
+  // The CronScheduler is pump-driven; without this heartbeat it only advances at
+  // turn boundaries (turn_start/agent_end), so a loop whose fire time elapses
+  // while the agent is idle would never fire and never re-wake the agent. The
+  // timer is unref'd so it never keeps a one-shot (`pi -p`) process alive.
+  function ensureHeartbeat(): void {
+    if (heartbeatTimer) return;
+    heartbeatTimer = setInterval(() => {
+      void pumpLoops();
+    }, HEARTBEAT_MS);
+    heartbeatTimer.unref?.();
+  }
+
+  function stopHeartbeat(): void {
+    if (!heartbeatTimer) return;
+    clearInterval(heartbeatTimer);
+    heartbeatTimer = undefined;
+  }
 
   function upgradeStoreIfNeeded(ctx: ExtensionContext) {
     if (storeUpgraded) return;
@@ -66,6 +89,7 @@ export function registerSessionRuntimeHooks(options: SessionRuntimeOptions): voi
       getStore().clearExpired();
       getStore().expireEventLoops(sessionStartedAt);
       getTriggerSystem().start();
+      ensureHeartbeat();
       widget.update();
     }
   }
@@ -89,6 +113,7 @@ export function registerSessionRuntimeHooks(options: SessionRuntimeOptions): voi
     setSessionId(ctx.sessionManager.getSessionId());
     widget.setUICtx(ctx.ui);
     upgradeStoreIfNeeded(ctx);
+    ensureHeartbeat();
     widget.update();
     await pumpLoops();
   });
@@ -97,6 +122,7 @@ export function registerSessionRuntimeHooks(options: SessionRuntimeOptions): voi
     setLatestCtx(ctx);
     widget.setUICtx(ctx.ui);
     upgradeStoreIfNeeded(ctx);
+    ensureHeartbeat();
     showPersistedLoops();
     widget.update();
   });
@@ -123,6 +149,7 @@ export function registerSessionRuntimeHooks(options: SessionRuntimeOptions): voi
   });
 
   pi.on("session_shutdown", async () => {
+    stopHeartbeat();
     notificationRuntime.clear("session_shutdown");
   });
 
@@ -130,6 +157,7 @@ export function registerSessionRuntimeHooks(options: SessionRuntimeOptions): voi
     setLatestCtx(ctx);
     widget.setUICtx(ctx.ui);
     getTriggerSystem().stop();
+    stopHeartbeat();
     notificationRuntime.clear("session_switch");
     setSessionId(undefined);
 

package/tmp/perf-cap-data/loops.json

@@ -1,455 +0,0 @@
-{
-  "nextId": 26,
-  "loops": [
-    {
-      "id": "1",
-      "prompt": "Loop 1",
-      "trigger": {
-        "type": "cron",
-        "schedule": "*/5 * * * *"
-      },
-      "recurring": true,
-      "autoTask": false,
-      "taskBacklog": false,
-      "readOnly": false,
-      "maxFires": 30,
-      "fireCount": 0,
-      "status": "active",
-      "createdAt": 1780943919086,
-      "updatedAt": 1780943919086,
-      "expiresAt": 1781548719086
-    },
-    {
-      "id": "2",
-      "prompt": "Loop 2",
-      "trigger": {
-        "type": "cron",
-        "schedule": "*/5 * * * *"
-      },
-      "recurring": true,
-      "autoTask": false,
-      "taskBacklog": false,
-      "readOnly": false,
-      "maxFires": 30,
-      "fireCount": 0,
-      "status": "active",
-      "createdAt": 1780943919086,
-      "updatedAt": 1780943919086,
-      "expiresAt": 1781548719086
-    },
-    {
-      "id": "3",
-      "prompt": "Loop 3",
-      "trigger": {
-        "type": "cron",
-        "schedule": "*/5 * * * *"
-      },
-      "recurring": true,
-      "autoTask": false,
-      "taskBacklog": false,
-      "readOnly": false,
-      "maxFires": 30,
-      "fireCount": 0,
-      "status": "active",
-      "createdAt": 1780943919086,
-      "updatedAt": 1780943919086,
-      "expiresAt": 1781548719086
-    },
-    {
-      "id": "4",
-      "prompt": "Loop 4",
-      "trigger": {
-        "type": "cron",
-        "schedule": "*/5 * * * *"
-      },
-      "recurring": true,
-      "autoTask": false,
-      "taskBacklog": false,
-      "readOnly": false,
-      "maxFires": 30,
-      "fireCount": 0,
-      "status": "active",
-      "createdAt": 1780943919086,
-      "updatedAt": 1780943919086,
-      "expiresAt": 1781548719086
-    },
-    {
-      "id": "5",
-      "prompt": "Loop 5",
-      "trigger": {
-        "type": "cron",
-        "schedule": "*/5 * * * *"
-      },
-      "recurring": true,
-      "autoTask": false,
-      "taskBacklog": false,
-      "readOnly": false,
-      "maxFires": 30,
-      "fireCount": 0,
-      "status": "active",
-      "createdAt": 1780943919086,
-      "updatedAt": 1780943919086,
-      "expiresAt": 1781548719086
-    },
-    {
-      "id": "6",
-      "prompt": "Loop 6",
-      "trigger": {
-        "type": "cron",
-        "schedule": "*/5 * * * *"
-      },
-      "recurring": true,
-      "autoTask": false,
-      "taskBacklog": false,
-      "readOnly": false,
-      "maxFires": 30,
-      "fireCount": 0,
-      "status": "active",
-      "createdAt": 1780943919086,
-      "updatedAt": 1780943919086,
-      "expiresAt": 1781548719086
-    },
-    {
-      "id": "7",
-      "prompt": "Loop 7",
-      "trigger": {
-        "type": "cron",
-        "schedule": "*/5 * * * *"
-      },
-      "recurring": true,
-      "autoTask": false,
-      "taskBacklog": false,
-      "readOnly": false,
-      "maxFires": 30,
-      "fireCount": 0,
-      "status": "active",
-      "createdAt": 1780943919086,
-      "updatedAt": 1780943919086,
-      "expiresAt": 1781548719086
-    },
-    {
-      "id": "8",
-      "prompt": "Loop 8",
-      "trigger": {
-        "type": "cron",
-        "schedule": "*/5 * * * *"
-      },
-      "recurring": true,
-      "autoTask": false,
-      "taskBacklog": false,
-      "readOnly": false,
-      "maxFires": 30,
-      "fireCount": 0,
-      "status": "active",
-      "createdAt": 1780943919086,
-      "updatedAt": 1780943919086,
-      "expiresAt": 1781548719086
-    },
-    {
-      "id": "9",
-      "prompt": "Loop 9",
-      "trigger": {
-        "type": "cron",
-        "schedule": "*/5 * * * *"
-      },
-      "recurring": true,
-      "autoTask": false,
-      "taskBacklog": false,
-      "readOnly": false,
-      "maxFires": 30,
-      "fireCount": 0,
-      "status": "active",
-      "createdAt": 1780943919086,
-      "updatedAt": 1780943919086,
-      "expiresAt": 1781548719086
-    },
-    {
-      "id": "10",
-      "prompt": "Loop 10",
-      "trigger": {
-        "type": "cron",
-        "schedule": "*/5 * * * *"
-      },
-      "recurring": true,
-      "autoTask": false,
-      "taskBacklog": false,
-      "readOnly": false,
-      "maxFires": 30,
-      "fireCount": 0,
-      "status": "active",
-      "createdAt": 1780943919086,
-      "updatedAt": 1780943919086,
-      "expiresAt": 1781548719086
-    },
-    {
-      "id": "11",
-      "prompt": "Loop 11",
-      "trigger": {
-        "type": "cron",
-        "schedule": "*/5 * * * *"
-      },
-      "recurring": true,
-      "autoTask": false,
-      "taskBacklog": false,
-      "readOnly": false,
-      "maxFires": 30,
-      "fireCount": 0,
-      "status": "active",
-      "createdAt": 1780943919086,
-      "updatedAt": 1780943919086,
-      "expiresAt": 1781548719086
-    },
-    {
-      "id": "12",
-      "prompt": "Loop 12",
-      "trigger": {
-        "type": "cron",
-        "schedule": "*/5 * * * *"
-      },
-      "recurring": true,
-      "autoTask": false,
-      "taskBacklog": false,
-      "readOnly": false,
-      "maxFires": 30,
-      "fireCount": 0,
-      "status": "active",
-      "createdAt": 1780943919086,
-      "updatedAt": 1780943919086,
-      "expiresAt": 1781548719086
-    },
-    {
-      "id": "13",
-      "prompt": "Loop 13",
-      "trigger": {
-        "type": "cron",
-        "schedule": "*/5 * * * *"
-      },
-      "recurring": true,
-      "autoTask": false,
-      "taskBacklog": false,
-      "readOnly": false,
-      "maxFires": 30,
-      "fireCount": 0,
-      "status": "active",
-      "createdAt": 1780943919086,
-      "updatedAt": 1780943919086,
-      "expiresAt": 1781548719086
-    },
-    {
-      "id": "14",
-      "prompt": "Loop 14",
-      "trigger": {
-        "type": "cron",
-        "schedule": "*/5 * * * *"
-      },
-      "recurring": true,
-      "autoTask": false,
-      "taskBacklog": false,
-      "readOnly": false,
-      "maxFires": 30,
-      "fireCount": 0,
-      "status": "active",
-      "createdAt": 1780943919086,
-      "updatedAt": 1780943919086,
-      "expiresAt": 1781548719086
-    },
-    {
-      "id": "15",
-      "prompt": "Loop 15",
-      "trigger": {
-        "type": "cron",
-        "schedule": "*/5 * * * *"
-      },
-      "recurring": true,
-      "autoTask": false,
-      "taskBacklog": false,
-      "readOnly": false,
-      "maxFires": 30,
-      "fireCount": 0,
-      "status": "active",
-      "createdAt": 1780943919086,
-      "updatedAt": 1780943919086,
-      "expiresAt": 1781548719086
-    },
-    {
-      "id": "16",
-      "prompt": "Loop 16",
-      "trigger": {
-        "type": "cron",
-        "schedule": "*/5 * * * *"
-      },
-      "recurring": true,
-      "autoTask": false,
-      "taskBacklog": false,
-      "readOnly": false,
-      "maxFires": 30,
-      "fireCount": 0,
-      "status": "active",
-      "createdAt": 1780943919086,
-      "updatedAt": 1780943919086,
-      "expiresAt": 1781548719086
-    },
-    {
-      "id": "17",
-      "prompt": "Loop 17",
-      "trigger": {
-        "type": "cron",
-        "schedule": "*/5 * * * *"
-      },
-      "recurring": true,
-      "autoTask": false,
-      "taskBacklog": false,
-      "readOnly": false,
-      "maxFires": 30,
-      "fireCount": 0,
-      "status": "active",
-      "createdAt": 1780943919086,
-      "updatedAt": 1780943919086,
-      "expiresAt": 1781548719086
-    },
-    {
-      "id": "18",
-      "prompt": "Loop 18",
-      "trigger": {
-        "type": "cron",
-        "schedule": "*/5 * * * *"
-      },
-      "recurring": true,
-      "autoTask": false,
-      "taskBacklog": false,
-      "readOnly": false,
-      "maxFires": 30,
-      "fireCount": 0,
-      "status": "active",
-      "createdAt": 1780943919086,
-      "updatedAt": 1780943919086,
-      "expiresAt": 1781548719086
-    },
-    {
-      "id": "19",
-      "prompt": "Loop 19",
-      "trigger": {
-        "type": "cron",
-        "schedule": "*/5 * * * *"
-      },
-      "recurring": true,
-      "autoTask": false,
-      "taskBacklog": false,
-      "readOnly": false,
-      "maxFires": 30,
-      "fireCount": 0,
-      "status": "active",
-      "createdAt": 1780943919086,
-      "updatedAt": 1780943919086,
-      "expiresAt": 1781548719086
-    },
-    {
-      "id": "20",
-      "prompt": "Loop 20",
-      "trigger": {
-        "type": "cron",
-        "schedule": "*/5 * * * *"
-      },
-      "recurring": true,
-      "autoTask": false,
-      "taskBacklog": false,
-      "readOnly": false,
-      "maxFires": 30,
-      "fireCount": 0,
-      "status": "active",
-      "createdAt": 1780943919086,
-      "updatedAt": 1780943919086,
-      "expiresAt": 1781548719086
-    },
-    {
-      "id": "21",
-      "prompt": "Loop 21",
-      "trigger": {
-        "type": "cron",
-        "schedule": "*/5 * * * *"
-      },
-      "recurring": true,
-      "autoTask": false,
-      "taskBacklog": false,
-      "readOnly": false,
-      "maxFires": 30,
-      "fireCount": 0,
-      "status": "active",
-      "createdAt": 1780943919086,
-      "updatedAt": 1780943919086,
-      "expiresAt": 1781548719086
-    },
-    {
-      "id": "22",
-      "prompt": "Loop 22",
-      "trigger": {
-        "type": "cron",
-        "schedule": "*/5 * * * *"
-      },
-      "recurring": true,
-      "autoTask": false,
-      "taskBacklog": false,
-      "readOnly": false,
-      "maxFires": 30,
-      "fireCount": 0,
-      "status": "active",
-      "createdAt": 1780943919086,
-      "updatedAt": 1780943919086,
-      "expiresAt": 1781548719086
-    },
-    {
-      "id": "23",
-      "prompt": "Loop 23",
-      "trigger": {
-        "type": "cron",
-        "schedule": "*/5 * * * *"
-      },
-      "recurring": true,
-      "autoTask": false,
-      "taskBacklog": false,
-      "readOnly": false,
-      "maxFires": 30,
-      "fireCount": 0,
-      "status": "active",
-      "createdAt": 1780943919086,
-      "updatedAt": 1780943919086,
-      "expiresAt": 1781548719086
-    },
-    {
-      "id": "24",
-      "prompt": "Loop 24",
-      "trigger": {
-        "type": "cron",
-        "schedule": "*/5 * * * *"
-      },
-      "recurring": true,
-      "autoTask": false,
-      "taskBacklog": false,
-      "readOnly": false,
-      "maxFires": 30,
-      "fireCount": 0,
-      "status": "active",
-      "createdAt": 1780943919086,
-      "updatedAt": 1780943919086,
-      "expiresAt": 1781548719086
-    },
-    {
-      "id": "25",
-      "prompt": "Loop 25",
-      "trigger": {
-        "type": "cron",
-        "schedule": "*/5 * * * *"
-      },
-      "recurring": true,
-      "autoTask": false,
-      "taskBacklog": false,
-      "readOnly": false,
-      "maxFires": 30,
-      "fireCount": 0,
-      "status": "active",
-      "createdAt": 1780943919086,
-      "updatedAt": 1780943919086,
-      "expiresAt": 1781548719086
-    }
-  ]
-}