@trevonistrevon/pi-loop 0.4.2 → 0.4.3

package/dist/index.js

@@ -209,6 +209,35 @@ export default function (pi) {
             message: buildLoopFireMessage(data),
         };
     }
+    function triggerHasEventSource(trigger, source) {
+        if (typeof trigger === "string")
+            return false;
+        return trigger.type === "event"
+            ? trigger.source === source
+            : trigger.type === "hybrid"
+                ? trigger.event.source === source
+                : false;
+    }
+    async function maybeBootstrapTaskLoop(entry) {
+        if (!entry.recurring)
+            return false;
+        if (!triggerHasEventSource(entry.trigger, "tasks:created"))
+            return false;
+        const pending = await hasPendingTasks();
+        if (pending <= 0)
+            return false;
+        debug(`loop #${entry.id} — bootstrapping existing pending tasks (${pending})`);
+        await queueOrDeliverNotification({
+            loopId: entry.id,
+            prompt: entry.prompt,
+            trigger: entry.trigger,
+            timestamp: Date.now(),
+            readOnly: entry.readOnly,
+            recurring: false,
+            autoTask: true,
+        });
+        return true;
+    }
     async function deliverNotification(notification) {
         if (notification.autoTask) {
             const pending = await hasPendingTasks();
@@ -425,8 +454,9 @@ Skip this tool when the task is a one-off check (just do it directly) or when th
             "## readOnly mode",
             "Set readOnly: true for loops that only observe and report (checks, status polls). This prevents unintended changes.",
             "## Task-driven workflows",
-            "After creating tasks with TaskCreate, use an event loop with autoTask: true so the system checks for pending tasks before firing: LoopCreate trigger='tasks:created' triggerType='event' autoTask: true maxFires: 30 prompt='Run TaskList, pick the next available pending task, work on it.'",
-            "When no tasks are pending, the loop skips the wake entirely — no tokens burned on empty polls.",
+            "Do not rely on a past 'tasks:created' event to replay. If tasks already exist, bootstrap the first pass in the current turn or use a hybrid/event loop that can catch future task creation and a cron safety-net.",
+            "Use autoTask only when you want the loop itself to create a task on each fire. For processing an existing task backlog, leave autoTask off and have the loop run TaskList to pick the next pending task.",
+            "When no tasks are pending, the loop should stop itself or skip the wake entirely — no tokens burned on empty polls.",
             "After creating a loop, tell the user the loop ID so they can cancel it with LoopDelete.",
         ],
         parameters: Type.Object({
@@ -439,7 +469,7 @@ Skip this tool when the task is a one-off check (just do it directly) or when th
             readOnly: Type.Optional(Type.Boolean({ description: "Restrict the agent to read-only tools when this loop fires (default: false)", default: false })),
             maxFires: Type.Optional(Type.Number({ description: "Auto-stop after N fires. Prevents infinite token burn on polling loops." })),
         }),
-        execute(_toolCallId, params, _signal, _onUpdate, _ctx) {
+        async execute(_toolCallId, params, _signal, _onUpdate, _ctx) {
             const { trigger: triggerInput, prompt, recurring, autoTask, triggerType, debounceMs, readOnly, maxFires } = params;
             let trigger;
             const inferred = triggerType ?? inferTriggerType(triggerInput);
@@ -487,6 +517,7 @@ Skip this tool when the task is a one-off check (just do it directly) or when th
                 }
                 catch { /* filter parse failure, ignore */ }
             }
+            const bootstrapped = await maybeBootstrapTaskLoop(entry);
             widget.update();
             const triggerDesc = trigger.type === "cron"
                 ? `schedule: ${trigger.schedule}`
@@ -497,6 +528,7 @@ Skip this tool when the task is a one-off check (just do it directly) or when th
                 `Trigger: ${triggerDesc}\n` +
                 `Recurring: ${entry.recurring}\n` +
                 (entry.autoTask ? `Auto-task: enabled\n` : "") +
+                (bootstrapped ? "Bootstrap: queued initial wake for existing pending tasks\n" : "") +
                 ((tasksAvailable || nativeTasksRegistered) ? "" : "(task system not ready yet — autoTask may not fire until native fallback or pi-tasks becomes available)\n") +
                 `ID: ${entry.id} (use LoopDelete to cancel)`));
         },
@@ -956,6 +988,12 @@ Use MonitorList to find the monitor ID, then stop it with this tool.`,
                 }
                 if (trimmed) {
                     const entry = nativeTaskStore.create(trimmed.slice(0, 80), trimmed);
+                    pi.events.emit("tasks:created", {
+                        taskId: entry.id,
+                        subject: entry.subject,
+                        description: entry.description,
+                        status: entry.status,
+                    });
                     widget.update();
                     ctx.ui.notify(`Task #${entry.id} created`, "info");
                     return;
@@ -982,6 +1020,12 @@ Fields:
             }),
             execute(_toolCallId, params, _signal, _onUpdate, _ctx) {
                 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,
+                });
                 widget.update();
                 return Promise.resolve(textResult(`Task #${entry.id} created: ${entry.subject}`));
             },

package/package.json

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

package/src/index.ts

@@ -229,6 +229,35 @@ export default function (pi: ExtensionAPI) {
     };
   }
 
+  function triggerHasEventSource(trigger: Trigger | string, source: string): boolean {
+    if (typeof trigger === "string") return false;
+    return trigger.type === "event"
+      ? trigger.source === source
+      : trigger.type === "hybrid"
+        ? trigger.event.source === source
+        : false;
+  }
+
+  async function maybeBootstrapTaskLoop(entry: LoopEntry): Promise<boolean> {
+    if (!entry.recurring) return false;
+    if (!triggerHasEventSource(entry.trigger, "tasks:created")) return false;
+
+    const pending = await hasPendingTasks();
+    if (pending <= 0) return false;
+
+    debug(`loop #${entry.id} — bootstrapping existing pending tasks (${pending})`);
+    await queueOrDeliverNotification({
+      loopId: entry.id,
+      prompt: entry.prompt,
+      trigger: entry.trigger,
+      timestamp: Date.now(),
+      readOnly: entry.readOnly,
+      recurring: false,
+      autoTask: true,
+    });
+    return true;
+  }
+
   async function deliverNotification(notification: PendingNotification): Promise<boolean> {
     if (notification.autoTask) {
       const pending = await hasPendingTasks();
@@ -470,8 +499,9 @@ Skip this tool when the task is a one-off check (just do it directly) or when th
       "## readOnly mode",
       "Set readOnly: true for loops that only observe and report (checks, status polls). This prevents unintended changes.",
       "## Task-driven workflows",
-      "After creating tasks with TaskCreate, use an event loop with autoTask: true so the system checks for pending tasks before firing: LoopCreate trigger='tasks:created' triggerType='event' autoTask: true maxFires: 30 prompt='Run TaskList, pick the next available pending task, work on it.'",
-      "When no tasks are pending, the loop skips the wake entirely — no tokens burned on empty polls.",
+      "Do not rely on a past 'tasks:created' event to replay. If tasks already exist, bootstrap the first pass in the current turn or use a hybrid/event loop that can catch future task creation and a cron safety-net.",
+      "Use autoTask only when you want the loop itself to create a task on each fire. For processing an existing task backlog, leave autoTask off and have the loop run TaskList to pick the next pending task.",
+      "When no tasks are pending, the loop should stop itself or skip the wake entirely — no tokens burned on empty polls.",
       "After creating a loop, tell the user the loop ID so they can cancel it with LoopDelete.",
     ],
     parameters: Type.Object({
@@ -485,7 +515,7 @@ Skip this tool when the task is a one-off check (just do it directly) or when th
       maxFires: Type.Optional(Type.Number({ description: "Auto-stop after N fires. Prevents infinite token burn on polling loops." })),
     }),
 
-    execute(_toolCallId, params, _signal, _onUpdate, _ctx) {
+    async execute(_toolCallId, params, _signal, _onUpdate, _ctx) {
       const { trigger: triggerInput, prompt, recurring, autoTask, triggerType, debounceMs, readOnly, maxFires } = params;
 
       let trigger: Trigger;
@@ -536,6 +566,8 @@ Skip this tool when the task is a one-off check (just do it directly) or when th
         } catch { /* filter parse failure, ignore */ }
       }
 
+      const bootstrapped = await maybeBootstrapTaskLoop(entry);
+
       widget.update();
 
       const triggerDesc = trigger.type === "cron"
@@ -549,6 +581,7 @@ Skip this tool when the task is a one-off check (just do it directly) or when th
         `Trigger: ${triggerDesc}\n` +
         `Recurring: ${entry.recurring}\n` +
         (entry.autoTask ? `Auto-task: enabled\n` : "") +
+        (bootstrapped ? "Bootstrap: queued initial wake for existing pending tasks\n" : "") +
         ((tasksAvailable || nativeTasksRegistered) ? "" : "(task system not ready yet — autoTask may not fire until native fallback or pi-tasks becomes available)\n") +
         `ID: ${entry.id} (use LoopDelete to cancel)`
       ));
@@ -1040,6 +1073,12 @@ Use MonitorList to find the monitor ID, then stop it with this tool.`,
         }
         if (trimmed) {
           const entry = nativeTaskStore.create(trimmed.slice(0, 80), trimmed);
+          pi.events.emit("tasks:created", {
+            taskId: entry.id,
+            subject: entry.subject,
+            description: entry.description,
+            status: entry.status,
+          });
           widget.update();
           ctx.ui.notify(`Task #${entry.id} created`, "info");
           return;
@@ -1067,6 +1106,12 @@ Fields:
       }),
       execute(_toolCallId, params, _signal, _onUpdate, _ctx) {
         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,
+        });
         widget.update();
         return Promise.resolve(textResult(`Task #${entry.id} created: ${entry.subject}`));
       },