@trevonistrevon/pi-loop 0.4.9 → 0.4.11

package/README.md

@@ -6,7 +6,7 @@
 ## Install
 
 ```bash
-pi install @trevonistrevon/pi-loop
+pi install npm:@trevonistrevon/pi-loop
 ```
 
 ## Quick start

package/dist/index.js

@@ -403,6 +403,18 @@ export default function (pi) {
         showPersistedLoops(isResume);
         widget.update();
     });
+    pi.on("tool_execution_end", async (event, ctx) => {
+        _latestCtx = ctx;
+        widget.setUICtx(ctx.ui);
+        if (event?.toolName !== "bash" || event?.isError)
+            return;
+        const command = event?.args?.command ?? event?.input?.command;
+        if (typeof command !== "string")
+            return;
+        if (!/^\s*git\s+commit\b/i.test(command))
+            return;
+        await cleanDoneTasks();
+    });
     // ── Dynamic loop pump — fires cron/hybrid loops on idle instead of wall-clock timers ──
     async function pumpLoops() {
         const pendingTasks = new Map();
@@ -571,11 +583,24 @@ Skip this tool when the task is a one-off check (just do it directly) or when th
         },
     });
     function handleMonitorDoneLoop(doneLoop, monitorId) {
-        triggerSystem.add(doneLoop);
+        const deliver = () => {
+            const current = store.get(doneLoop.id);
+            if (!current)
+                return;
+            debug(`onDone loop #${doneLoop.id} — monitor #${monitorId} completed, delivering directly`);
+            onLoopFire(current);
+            store.delete(doneLoop.id);
+        };
+        const registered = monitorManager.onComplete(monitorId, deliver);
+        if (registered)
+            return;
         const monitor = monitorManager.get(monitorId);
         if (monitor && monitor.status !== "running") {
+            if (monitor.status === "completed") {
+                deliver();
+                return;
+            }
             debug(`onDone loop #${doneLoop.id} — monitor #${monitorId} already ${monitor.status}, expiring`);
-            triggerSystem.remove(doneLoop.id);
             store.delete(doneLoop.id);
         }
     }

package/dist/monitor-manager.d.ts

@@ -9,5 +9,6 @@ export declare class MonitorManager {
     get(id: string): MonitorEntry | undefined;
     list(): MonitorEntry[];
     stop(id: string): Promise<boolean>;
+    onComplete(id: string, callback: () => void): boolean;
     getProcess(id: string): MonitorProcess | undefined;
 }

package/dist/monitor-manager.js

@@ -30,6 +30,7 @@ export class MonitorManager {
             proc: child,
             abortController,
             waiters: [],
+            completionCallbacks: [],
         };
         child.stdout?.on("data", (data) => {
             const lines = data.toString().split("\n");
@@ -70,6 +71,11 @@ export class MonitorManager {
                 exitCode: code,
                 outputLines: entry.outputLines,
             });
+            if (status === "completed") {
+                for (const callback of bp.completionCallbacks)
+                    callback();
+            }
+            bp.completionCallbacks = [];
             for (const resolve of bp.waiters)
                 resolve();
             bp.waiters = [];
@@ -90,6 +96,7 @@ export class MonitorManager {
                     monitorId: id,
                     error: err.message,
                 });
+                bp.completionCallbacks = [];
                 for (const resolve of bp.waiters)
                     resolve();
                 bp.waiters = [];
@@ -134,11 +141,25 @@ export class MonitorManager {
             });
         });
         bp.entry.completedAt = Date.now();
+        bp.completionCallbacks = [];
         for (const resolve of bp.waiters)
             resolve();
         bp.waiters = [];
         return true;
     }
+    onComplete(id, callback) {
+        const bp = this.processes.get(id);
+        if (!bp)
+            return false;
+        if (bp.entry.status === "completed") {
+            callback();
+            return true;
+        }
+        if (bp.entry.status !== "running")
+            return false;
+        bp.completionCallbacks.push(callback);
+        return true;
+    }
     getProcess(id) {
         return this.processes.get(id);
     }

package/dist/types.d.ts

@@ -55,4 +55,5 @@ export interface MonitorProcess {
     proc: import("node:child_process").ChildProcess;
     abortController: AbortController;
     waiters: Array<() => void>;
+    completionCallbacks: Array<() => void>;
 }

package/package.json

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

package/src/index.ts

@@ -441,6 +441,19 @@ export default function (pi: ExtensionAPI) {
     widget.update();
   });
 
+  pi.on("tool_execution_end", async (event: any, ctx: ExtensionContext) => {
+    _latestCtx = ctx;
+    widget.setUICtx(ctx.ui);
+
+    if (event?.toolName !== "bash" || event?.isError) return;
+
+    const command = event?.args?.command ?? event?.input?.command;
+    if (typeof command !== "string") return;
+    if (!/^\s*git\s+commit\b/i.test(command)) return;
+
+    await cleanDoneTasks();
+  });
+
   // ── Dynamic loop pump — fires cron/hybrid loops on idle instead of wall-clock timers ──
 
   async function pumpLoops(): Promise<void> {
@@ -621,11 +634,24 @@ Skip this tool when the task is a one-off check (just do it directly) or when th
   });
 
   function handleMonitorDoneLoop(doneLoop: LoopEntry, monitorId: string): void {
-    triggerSystem.add(doneLoop);
+    const deliver = () => {
+      const current = store.get(doneLoop.id);
+      if (!current) return;
+      debug(`onDone loop #${doneLoop.id} — monitor #${monitorId} completed, delivering directly`);
+      onLoopFire(current);
+      store.delete(doneLoop.id);
+    };
+
+    const registered = monitorManager.onComplete(monitorId, deliver);
+    if (registered) return;
+
     const monitor = monitorManager.get(monitorId);
     if (monitor && monitor.status !== "running") {
+      if (monitor.status === "completed") {
+        deliver();
+        return;
+      }
       debug(`onDone loop #${doneLoop.id} — monitor #${monitorId} already ${monitor.status}, expiring`);
-      triggerSystem.remove(doneLoop.id);
       store.delete(doneLoop.id);
     }
   }

package/src/monitor-manager.ts

@@ -35,6 +35,7 @@ export class MonitorManager {
       proc: child,
       abortController,
       waiters: [],
+      completionCallbacks: [],
     };
 
     child.stdout?.on("data", (data: Buffer) => {
@@ -74,6 +75,10 @@ export class MonitorManager {
         exitCode: code,
         outputLines: entry.outputLines,
       });
+      if (status === "completed") {
+        for (const callback of bp.completionCallbacks) callback();
+      }
+      bp.completionCallbacks = [];
       for (const resolve of bp.waiters) resolve();
       bp.waiters = [];
       // Remove completed/errored monitors after a brief delay so tool
@@ -95,6 +100,7 @@ export class MonitorManager {
           monitorId: id,
           error: err.message,
         });
+        bp.completionCallbacks = [];
         for (const resolve of bp.waiters) resolve();
         bp.waiters = [];
       }
@@ -143,11 +149,24 @@ export class MonitorManager {
     });
 
     bp.entry.completedAt = Date.now();
+    bp.completionCallbacks = [];
     for (const resolve of bp.waiters) resolve();
     bp.waiters = [];
     return true;
   }
 
+  onComplete(id: string, callback: () => void): boolean {
+    const bp = this.processes.get(id);
+    if (!bp) return false;
+    if (bp.entry.status === "completed") {
+      callback();
+      return true;
+    }
+    if (bp.entry.status !== "running") return false;
+    bp.completionCallbacks.push(callback);
+    return true;
+  }
+
   getProcess(id: string): MonitorProcess | undefined {
     return this.processes.get(id);
   }

package/src/types.ts

@@ -60,4 +60,5 @@ export interface MonitorProcess {
   proc: import("node:child_process").ChildProcess;
   abortController: AbortController;
   waiters: Array<() => void>;
+  completionCallbacks: Array<() => void>;
 }