@trevonistrevon/pi-loop 0.5.8 → 0.5.9

package/coverage/src/trigger-system.ts.html

@@ -224,15 +224,15 @@
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
-<span class="cline-any cline-yes">121x</span>
-<span class="cline-any cline-yes">121x</span>
-<span class="cline-any cline-yes">121x</span>
+<span class="cline-any cline-yes">123x</span>
+<span class="cline-any cline-yes">123x</span>
+<span class="cline-any cline-yes">123x</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
-<span class="cline-any cline-yes">121x</span>
-<span class="cline-any cline-yes">121x</span>
-<span class="cline-any cline-yes">121x</span>
-<span class="cline-any cline-yes">121x</span>
+<span class="cline-any cline-yes">123x</span>
+<span class="cline-any cline-yes">123x</span>
+<span class="cline-any cline-yes">123x</span>
+<span class="cline-any cline-yes">123x</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
@@ -532,7 +532,7 @@ export class TriggerSystem {
             <div class='footer quiet pad2 space-top1 center small'>
                 Code coverage generated by
                 <a href="https://istanbul.js.org/" target="_blank" rel="noopener noreferrer">istanbul</a>
-                at 2026-06-19T00:56:38.236Z
+                at 2026-06-23T21:49:18.784Z
             </div>
         <script src="../prettify.js"></script>
         <script>

package/coverage/src/ui/index.html

@@ -101,7 +101,7 @@
             <div class='footer quiet pad2 space-top1 center small'>
                 Code coverage generated by
                 <a href="https://istanbul.js.org/" target="_blank" rel="noopener noreferrer">istanbul</a>
-                at 2026-06-19T00:56:38.236Z
+                at 2026-06-23T21:49:18.784Z
             </div>
         <script src="../../prettify.js"></script>
         <script>

package/coverage/src/ui/widget.ts.html

@@ -147,8 +147,8 @@
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
-<span class="cline-any cline-yes">68x</span>
-<span class="cline-any cline-yes">68x</span>
+<span class="cline-any cline-yes">70x</span>
+<span class="cline-any cline-yes">70x</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
@@ -160,11 +160,11 @@
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
-<span class="cline-any cline-yes">63x</span>
+<span class="cline-any cline-yes">65x</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
-<span class="cline-any cline-yes">192x</span>
+<span class="cline-any cline-yes">189x</span>
 <span class="cline-any cline-yes">164x</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
@@ -277,7 +277,7 @@ function isStatusVisibleLoop(loop: LoopEntry): boolean {
             <div class='footer quiet pad2 space-top1 center small'>
                 Code coverage generated by
                 <a href="https://istanbul.js.org/" target="_blank" rel="noopener noreferrer">istanbul</a>
-                at 2026-06-19T00:56:38.236Z
+                at 2026-06-23T21:49:18.784Z
             </div>
         <script src="../../prettify.js"></script>
         <script>

package/dist/index.js

@@ -36,6 +36,9 @@ function debug(...args) {
     if (DEBUG)
         console.error("[pi-loop]", ...args);
 }
+function isStaleExtensionContextError(error) {
+    return error instanceof Error && error.message.includes("extension ctx is stale");
+}
 export default function (pi) {
     const piLoopEnv = process.env.PI_LOOP;
     const piLoopScope = process.env.PI_LOOP_SCOPE;
@@ -263,28 +266,40 @@ export default function (pi) {
         },
     });
     // ── Native task tools (only when pi-tasks is absent) ──
-    setTimeout(async () => {
+    const nativeTaskFallbackTimer = setTimeout(() => {
         if (tasksAvailable || nativeTasksRegistered)
             return;
-        nativeTaskStore = new TaskStore(resolveTaskStorePath(getScopeOptions(), _sessionId));
+        const taskStore = new TaskStore(resolveTaskStorePath(getScopeOptions(), _sessionId));
+        try {
+            registerTasksCommand({
+                pi,
+                getNativeTaskStore: () => nativeTaskStore,
+                evaluateTaskBacklog,
+                updateWidget: () => {
+                    widget.update();
+                },
+            });
+            registerNativeTaskTools({
+                pi,
+                taskStore,
+                evaluateTaskBacklog,
+                updateWidget: () => {
+                    widget.update();
+                },
+            });
+        }
+        catch (error) {
+            if (isStaleExtensionContextError(error)) {
+                debug("native task fallback skipped: extension context went stale");
+                return;
+            }
+            throw error;
+        }
+        nativeTaskStore = taskStore;
         nativeTasksRegistered = true;
-        const taskStore = nativeTaskStore;
-        registerTasksCommand({
-            pi,
-            getNativeTaskStore: () => nativeTaskStore,
-            evaluateTaskBacklog,
-            updateWidget: () => {
-                widget.update();
-            },
-        });
-        registerNativeTaskTools({
-            pi,
-            taskStore,
-            evaluateTaskBacklog,
-            updateWidget: () => {
-                widget.update();
-            },
-        });
         debug("native task tools registered (pi-tasks not detected)");
     }, 6000);
+    pi.on("session_shutdown", () => {
+        clearTimeout(nativeTaskFallbackTimer);
+    });
 }

package/dist/monitor-manager.d.ts

@@ -1,11 +1,14 @@
+import { type ChildProcess, type SpawnOptions } from "node:child_process";
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import type { MonitorEntry, MonitorProcess } from "./types.js";
+export type SpawnFn = (command: string, args: string[], options: SpawnOptions) => ChildProcess;
 export declare class MonitorManager {
     private pi;
     private processes;
     private nextId;
     private onChange;
-    constructor(pi: ExtensionAPI);
+    private spawnFn;
+    constructor(pi: ExtensionAPI, spawnFn?: SpawnFn);
     /**
      * 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

package/dist/monitor-manager.js

@@ -1,12 +1,14 @@
-import { spawn } from "node:child_process";
+import { spawn as nodeSpawn } from "node:child_process";
 import { reduceMonitorState, } from "./monitor-reducer.js";
 export class MonitorManager {
     pi;
     processes = new Map();
     nextId = 1;
     onChange;
-    constructor(pi) {
+    spawnFn;
+    constructor(pi, spawnFn) {
         this.pi = pi;
+        this.spawnFn = spawnFn ?? ((cmd, args, opts) => nodeSpawn(cmd, args, opts));
     }
     /**
      * Register a callback fired when a monitor's status changes or it is pruned
@@ -76,7 +78,7 @@ export class MonitorManager {
         const id = String(this.nextId - 1);
         const entry = result.state.monitorsById[id];
         const abortController = new AbortController();
-        const child = spawn("sh", ["-c", command], {
+        const child = this.spawnFn("sh", ["-c", command], {
             stdio: ["ignore", "pipe", "pipe"],
             signal: abortController.signal,
             env: { ...process.env },

package/docker-compose.yml

@@ -0,0 +1,12 @@
+services:
+  ci-node20:
+    build:
+      context: .
+      target: node20
+    command: sh -c "npm run typecheck && npm run lint && npm run test:coverage && npm run build"
+
+  ci-node22:
+    build:
+      context: .
+      target: node22
+    command: sh -c "npm run typecheck && npm run lint && npm run test:coverage && npm run build"

package/package.json

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

package/release-please-config.json

@@ -0,0 +1,9 @@
+{
+  "packages": {
+    ".": {
+      "release-type": "node",
+      "bump-minor-pre-major": true,
+      "bump-patch-for-minor-pre-major": true
+    }
+  }
+}

package/src/index.ts

@@ -43,6 +43,10 @@ function debug(...args: unknown[]) {
   if (DEBUG) console.error("[pi-loop]", ...args);
 }
 
+function isStaleExtensionContextError(error: unknown): boolean {
+  return error instanceof Error && error.message.includes("extension ctx is stale");
+}
+
 export default function (pi: ExtensionAPI) {
   const piLoopEnv = process.env.PI_LOOP;
   const piLoopScope = process.env.PI_LOOP_SCOPE as "memory" | "session" | "project" | undefined;
@@ -297,30 +301,42 @@ export default function (pi: ExtensionAPI) {
 
   // ── Native task tools (only when pi-tasks is absent) ──
 
-  setTimeout(async () => {
+  const nativeTaskFallbackTimer = setTimeout(() => {
     if (tasksAvailable || nativeTasksRegistered) return;
-    nativeTaskStore = new TaskStore(resolveTaskStorePath(getScopeOptions(), _sessionId));
-    nativeTasksRegistered = true;
-    const taskStore = nativeTaskStore;
-
-    registerTasksCommand({
-      pi,
-      getNativeTaskStore: () => nativeTaskStore,
-      evaluateTaskBacklog,
-      updateWidget: () => {
-        widget.update();
-      },
-    });
+    const taskStore = new TaskStore(resolveTaskStorePath(getScopeOptions(), _sessionId));
+
+    try {
+      registerTasksCommand({
+        pi,
+        getNativeTaskStore: () => nativeTaskStore,
+        evaluateTaskBacklog,
+        updateWidget: () => {
+          widget.update();
+        },
+      });
 
-    registerNativeTaskTools({
-      pi,
-      taskStore,
-      evaluateTaskBacklog,
-      updateWidget: () => {
-        widget.update();
-      },
-    });
+      registerNativeTaskTools({
+        pi,
+        taskStore,
+        evaluateTaskBacklog,
+        updateWidget: () => {
+          widget.update();
+        },
+      });
+    } catch (error) {
+      if (isStaleExtensionContextError(error)) {
+        debug("native task fallback skipped: extension context went stale");
+        return;
+      }
+      throw error;
+    }
 
+    nativeTaskStore = taskStore;
+    nativeTasksRegistered = true;
     debug("native task tools registered (pi-tasks not detected)");
   }, 6000);
+
+  pi.on("session_shutdown", () => {
+    clearTimeout(nativeTaskFallbackTimer);
+  });
 }

package/src/monitor-manager.ts

@@ -1,5 +1,5 @@
 
-import { spawn } from "node:child_process";
+import { type ChildProcess, spawn as nodeSpawn, type SpawnOptions } from "node:child_process";
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import {
   type MonitorReducerEvent,
@@ -8,12 +8,20 @@ import {
 } from "./monitor-reducer.js";
 import type { MonitorEntry, MonitorProcess } from "./types.js";
 
+export type SpawnFn = (command: string, args: string[], options: SpawnOptions) => ChildProcess;
+
 export class MonitorManager {
   private processes = new Map<string, MonitorProcess>();
   private nextId = 1;
   private onChange: (() => void) | undefined;
+  private spawnFn: SpawnFn;
 
-  constructor(private pi: ExtensionAPI) {}
+  constructor(
+    private pi: ExtensionAPI,
+    spawnFn?: SpawnFn,
+  ) {
+    this.spawnFn = spawnFn ?? ((cmd, args, opts) => nodeSpawn(cmd, args, opts));
+  }
 
   /**
    * Register a callback fired when a monitor's status changes or it is pruned
@@ -90,7 +98,7 @@ export class MonitorManager {
     const entry = result.state.monitorsById[id]!;
 
     const abortController = new AbortController();
-    const child = spawn("sh", ["-c", command], {
+    const child = this.spawnFn("sh", ["-c", command], {
       stdio: ["ignore", "pipe", "pipe"],
       signal: abortController.signal,
       env: { ...process.env },

package/vitest.config.ts

@@ -5,6 +5,10 @@ export default defineConfig({
     environment: "node",
     include: ["test/**/*.test.ts"],
     exclude: ["node_modules", "dist"],
+    // CI runners are slower than local; 15s gives real-child-process
+    // tests (monitor stop, lifecycle) enough headroom.
+    testTimeout: 15000,
+    hookTimeout: 15000,
     coverage: {
       provider: "v8",
       reporter: ["text-summary", "html", "json"],