@trevonistrevon/pi-loop 0.5.6 → 0.5.8

package/dist/tools/native-task-tools.js

@@ -1,4 +1,5 @@
 import { Type } from "typebox";
+import { emitNativeTaskEvent } from "../runtime/task-events.js";
 function textResult(msg) {
     return { content: [{ type: "text", text: msg }], details: undefined };
 }
@@ -30,12 +31,7 @@ Fields:
         }),
         async execute(_toolCallId, params) {
             const entry = taskStore.create(params.subject, params.description);
-            pi.events.emit("tasks:created", {
-                taskId: entry.id,
-                subject: entry.subject,
-                description: entry.description,
-                status: entry.status,
-            });
+            emitNativeTaskEvent(pi, "tasks:created", entry);
             const backlog = await evaluateTaskBacklog(taskStore, taskStore.pendingCount());
             updateWidget();
             const autoLoopMsg = backlog.created && backlog.entry
@@ -91,16 +87,28 @@ Parameters: id (required), status, subject, description`,
             let entry = taskStore.get(id);
             if (!entry)
                 return Promise.resolve(textResult(`Task #${id} not found`));
-            if (status === "in_progress")
+            const previousStatus = entry.status;
+            if (status === "in_progress") {
                 entry = taskStore.start(id);
-            else if (status === "completed")
+                if (entry)
+                    emitNativeTaskEvent(pi, "tasks:started", entry, previousStatus);
+            }
+            else if (status === "completed") {
                 entry = taskStore.complete(id);
-            else if (status === "pending")
+                if (entry)
+                    emitNativeTaskEvent(pi, "tasks:completed", entry, previousStatus);
+            }
+            else if (status === "pending") {
                 entry = taskStore.reopen(id);
+                if (entry)
+                    emitNativeTaskEvent(pi, "tasks:reopened", entry, previousStatus);
+            }
             if (!entry)
                 return Promise.resolve(textResult(`Task #${id} not found`));
             if (subject !== undefined || description !== undefined) {
                 entry = taskStore.updateDetails(id, { subject, description });
+                if (entry)
+                    emitNativeTaskEvent(pi, "tasks:updated", entry, previousStatus);
             }
             if (!entry)
                 return Promise.resolve(textResult(`Task #${id} not found`));
@@ -121,9 +129,12 @@ Parameters: id (required), status, subject, description`,
             id: Type.String({ description: "Task ID to delete" }),
         }),
         async execute(_toolCallId, params) {
+            const existing = taskStore.get(params.id);
             const deleted = taskStore.delete(params.id);
             updateWidget();
             if (deleted) {
+                if (existing)
+                    emitNativeTaskEvent(pi, "tasks:deleted", existing, existing.status);
                 await evaluateTaskBacklog(taskStore, taskStore.pendingCount());
                 return Promise.resolve(textResult(`Task #${params.id} deleted`));
             }

package/package.json

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

package/src/commands/tasks-command.ts

@@ -1,4 +1,5 @@
 import type { ExtensionAPI, ExtensionCommandContext, ExtensionUIContext } from "@earendil-works/pi-coding-agent";
+import { emitNativeTaskEvent } from "../runtime/task-events.js";
 import { TaskStore } from "../task-store.js";
 import type { TaskEntry } from "../task-types.js";
 
@@ -18,12 +19,7 @@ export function registerTasksCommand(options: TasksCommandOptions): void {
   const { pi, getNativeTaskStore, evaluateTaskBacklog, updateWidget } = options;
 
   async function emitCreated(entry: TaskEntry) {
-    pi.events.emit("tasks:created", {
-      taskId: entry.id,
-      subject: entry.subject,
-      description: entry.description,
-      status: entry.status,
-    });
+    emitNativeTaskEvent(pi, "tasks:created", entry);
     const taskStore = getNativeTaskStore();
     if (!taskStore) return { created: false } satisfies TaskBacklogResult;
     const backlog = await evaluateTaskBacklog(taskStore, taskStore.pendingCount());
@@ -94,15 +90,19 @@ export function registerTasksCommand(options: TasksCommandOptions): void {
 
     if (action === "x Delete") {
       taskStore.delete(task.id);
+      emitNativeTaskEvent(pi, "tasks:deleted", task, task.status);
       ui.notify(`Task #${task.id} deleted`, "info");
     } else if (action === "> Start") {
-      taskStore.start(task.id);
+      const next = taskStore.start(task.id);
+      if (next) emitNativeTaskEvent(pi, "tasks:started", next, task.status);
       ui.notify(`Task #${task.id} started`, "info");
     } else if (action === "ok Complete") {
-      taskStore.complete(task.id);
+      const next = taskStore.complete(task.id);
+      if (next) emitNativeTaskEvent(pi, "tasks:completed", next, task.status);
       ui.notify(`Task #${task.id} completed`, "info");
     } else if (action === "* Return to pending" || action === "* Reopen") {
-      taskStore.reopen(task.id);
+      const next = taskStore.reopen(task.id);
+      if (next) emitNativeTaskEvent(pi, "tasks:reopened", next, task.status);
       ui.notify(`Task #${task.id} reopened`, "info");
     }
 

package/src/monitor-manager.ts

@@ -58,7 +58,8 @@ export class MonitorManager {
   // 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(() => {
+    // unref so a pending prune never keeps a one-shot (`pi -p`) process alive.
+    const timer = setTimeout(() => {
       this.applyReducerEvent({
         type: "MONITOR_PRUNED",
         at: Date.now(),
@@ -68,6 +69,7 @@ export class MonitorManager {
         payload: { id },
       });
     }, 30000);
+    timer.unref?.();
   }
 
   create(command: string, description?: string, timeout = 300000): MonitorEntry {

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,27 @@ 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(() => {
+      // Swallow pump failures so a transient error never surfaces as an
+      // unhandled rejection; the next tick retries.
+      void pumpLoops().catch(() => {});
+    }, HEARTBEAT_MS);
+    heartbeatTimer.unref?.();
+  }
+
+  function stopHeartbeat(): void {
+    if (!heartbeatTimer) return;
+    clearInterval(heartbeatTimer);
+    heartbeatTimer = undefined;
+  }
 
   function upgradeStoreIfNeeded(ctx: ExtensionContext) {
     if (storeUpgraded) return;
@@ -66,6 +91,7 @@ export function registerSessionRuntimeHooks(options: SessionRuntimeOptions): voi
       getStore().clearExpired();
       getStore().expireEventLoops(sessionStartedAt);
       getTriggerSystem().start();
+      ensureHeartbeat();
       widget.update();
     }
   }
@@ -89,6 +115,7 @@ export function registerSessionRuntimeHooks(options: SessionRuntimeOptions): voi
     setSessionId(ctx.sessionManager.getSessionId());
     widget.setUICtx(ctx.ui);
     upgradeStoreIfNeeded(ctx);
+    ensureHeartbeat();
     widget.update();
     await pumpLoops();
   });
@@ -97,6 +124,7 @@ export function registerSessionRuntimeHooks(options: SessionRuntimeOptions): voi
     setLatestCtx(ctx);
     widget.setUICtx(ctx.ui);
     upgradeStoreIfNeeded(ctx);
+    ensureHeartbeat();
     showPersistedLoops();
     widget.update();
   });
@@ -123,6 +151,7 @@ export function registerSessionRuntimeHooks(options: SessionRuntimeOptions): voi
   });
 
   pi.on("session_shutdown", async () => {
+    stopHeartbeat();
     notificationRuntime.clear("session_shutdown");
   });
 
@@ -130,6 +159,7 @@ export function registerSessionRuntimeHooks(options: SessionRuntimeOptions): voi
     setLatestCtx(ctx);
     widget.setUICtx(ctx.ui);
     getTriggerSystem().stop();
+    stopHeartbeat();
     notificationRuntime.clear("session_switch");
     setSessionId(undefined);
 

package/src/runtime/task-events.ts

@@ -0,0 +1,41 @@
+import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
+import type { TaskEntry, TaskStatus } from "../task-types.js";
+
+export type NativeTaskEventName =
+  | "tasks:created"
+  | "tasks:started"
+  | "tasks:completed"
+  | "tasks:reopened"
+  | "tasks:updated"
+  | "tasks:deleted";
+
+export interface NativeTaskEventPayload {
+  taskId: string;
+  subject: string;
+  description: string;
+  status: TaskStatus;
+  previousStatus?: TaskStatus;
+  createdAt: number;
+  updatedAt: number;
+  completedAt?: number;
+  metadata?: Record<string, unknown>;
+}
+
+export function emitNativeTaskEvent(
+  pi: ExtensionAPI,
+  name: NativeTaskEventName,
+  entry: TaskEntry,
+  previousStatus?: TaskStatus,
+): void {
+  pi.events.emit(name, {
+    taskId: entry.id,
+    subject: entry.subject,
+    description: entry.description,
+    status: entry.status,
+    previousStatus,
+    createdAt: entry.createdAt,
+    updatedAt: entry.updatedAt,
+    completedAt: entry.completedAt,
+    metadata: entry.metadata,
+  } satisfies NativeTaskEventPayload);
+}

package/src/runtime/task-rpc.ts

@@ -2,6 +2,7 @@ import { randomUUID } from "node:crypto";
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import type { TaskStore } from "../task-store.js";
 import type { LoopEntry } from "../types.js";
+import { emitNativeTaskEvent } from "./task-events.js";
 
 export interface TaskRuntimeBridgeOptions {
   pi: ExtensionAPI;
@@ -82,6 +83,7 @@ export function createTaskRuntimeBridge(options: TaskRuntimeBridgeOptions): Task
       `Auto-created from loop #${entry.id}`,
       { loopId: entry.id, trigger: entry.trigger },
     );
+    emitNativeTaskEvent(pi, "tasks:created", task);
     onNativeTaskCreated?.(nativeTaskStore);
     return task.id;
   }

package/src/tools/native-task-tools.ts

@@ -1,5 +1,6 @@
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { Type } from "typebox";
+import { emitNativeTaskEvent } from "../runtime/task-events.js";
 import { TaskStore } from "../task-store.js";
 
 export interface TaskBacklogResult {
@@ -47,12 +48,7 @@ Fields:
     }),
     async execute(_toolCallId, params) {
       const entry = taskStore.create(params.subject, params.description);
-      pi.events.emit("tasks:created", {
-        taskId: entry.id,
-        subject: entry.subject,
-        description: entry.description,
-        status: entry.status,
-      });
+      emitNativeTaskEvent(pi, "tasks:created", entry);
       const backlog = await evaluateTaskBacklog(taskStore, taskStore.pendingCount());
       updateWidget();
 
@@ -111,13 +107,22 @@ Parameters: id (required), status, subject, description`,
       let entry = taskStore.get(id);
       if (!entry) return Promise.resolve(textResult(`Task #${id} not found`));
 
-      if (status === "in_progress") entry = taskStore.start(id);
-      else if (status === "completed") entry = taskStore.complete(id);
-      else if (status === "pending") entry = taskStore.reopen(id);
+      const previousStatus = entry.status;
+      if (status === "in_progress") {
+        entry = taskStore.start(id);
+        if (entry) emitNativeTaskEvent(pi, "tasks:started", entry, previousStatus);
+      } else if (status === "completed") {
+        entry = taskStore.complete(id);
+        if (entry) emitNativeTaskEvent(pi, "tasks:completed", entry, previousStatus);
+      } else if (status === "pending") {
+        entry = taskStore.reopen(id);
+        if (entry) emitNativeTaskEvent(pi, "tasks:reopened", entry, previousStatus);
+      }
 
       if (!entry) return Promise.resolve(textResult(`Task #${id} not found`));
       if (subject !== undefined || description !== undefined) {
         entry = taskStore.updateDetails(id, { subject, description });
+        if (entry) emitNativeTaskEvent(pi, "tasks:updated", entry, previousStatus);
       }
       if (!entry) return Promise.resolve(textResult(`Task #${id} not found`));
       updateWidget();
@@ -138,9 +143,11 @@ Parameters: id (required), status, subject, description`,
       id: Type.String({ description: "Task ID to delete" }),
     }),
     async execute(_toolCallId, params) {
+      const existing = taskStore.get(params.id);
       const deleted = taskStore.delete(params.id);
       updateWidget();
       if (deleted) {
+        if (existing) emitNativeTaskEvent(pi, "tasks:deleted", existing, existing.status);
         await evaluateTaskBacklog(taskStore, taskStore.pendingCount());
         return Promise.resolve(textResult(`Task #${params.id} deleted`));
       }

package/vitest.config.ts

@@ -11,14 +11,14 @@ export default defineConfig({
       include: ["src/**/*.ts"],
       // Type-only modules and test helpers carry no executable logic worth gating.
       exclude: ["src/**/*-types.ts", "src/types.ts"],
-      // Floors set just below current actuals (stmts 80%, branches 69.5%,
-      // funcs 92.5%, lines 82.9%) to catch regressions. Ratchet up as the
-      // runtime/ and tools/ suites land in Phase 4.
+      // Floors set just below current actuals (stmts 84%, branches 75%,
+      // funcs 95%, lines 86%) to catch regressions. Raised in Phase 4 after the
+      // runtime/ + tools/ suites landed.
       thresholds: {
-        statements: 78,
-        branches: 67,
-        functions: 90,
-        lines: 80,
+        statements: 82,
+        branches: 73,
+        functions: 94,
+        lines: 84,
       },
     },
   },