@trevonistrevon/pi-loop 0.1.8 → 0.2.0

package/CONTRIBUTING.md

@@ -5,7 +5,7 @@ Thanks for contributing! This document covers local dev setup, conventions, and
 ## Getting Started
 
 ```bash
-git clone https://github.com/trevonistrevon/pi-loop.git
+git clone https://github.com/trvon/pi-loop.git
 cd pi-loop
 npm install
 ```

package/dist/index.js

@@ -30,8 +30,6 @@ function debug(...args) {
 function textResult(msg) {
     return { content: [{ type: "text", text: msg }], details: undefined };
 }
-const LOOP_TOOL_NAMES = new Set(["LoopCreate", "LoopList", "LoopDelete", "MonitorCreate", "MonitorList", "MonitorStop"]);
-const REMINDER_INTERVAL = 3;
 const SYSTEM_REMINDER_TEMPLATE = `<system-reminder>
 Scheduled loop "%propmpt%" fired. Trigger: %trigger_info%.
 [loop:%loop_id%]
@@ -183,9 +181,7 @@ export default function (pi) {
         showPersistedLoops(isResume);
     });
     // ── System-reminder injection for loop fires ──
-    let currentTurn = 0;
-    let lastLoopToolUseTurn = 0;
-    let reminderInjectedThisCycle = false;
+    let canInjectReminder = true;
     const pendingReminders = [];
     pi.on("loop:fire", (data) => {
         const triggerInfo = typeof data.trigger === "string"
@@ -201,22 +197,15 @@ export default function (pi) {
             .replace("%loop_id%", data.loopId || "unknown");
         pendingReminders.push(reminder);
     });
+    // Allow one reminder injection per agent turn (at the first tool call),
+    // so the reminder arrives between batches of work, not mid-execution.
     pi.on("turn_start", async () => {
-        currentTurn++;
+        canInjectReminder = true;
     });
     pi.on("tool_result", async (event) => {
-        if (LOOP_TOOL_NAMES.has(event.toolName)) {
-            lastLoopToolUseTurn = currentTurn;
-            reminderInjectedThisCycle = false;
+        if (!canInjectReminder || pendingReminders.length === 0)
             return {};
-        }
-        if (currentTurn - lastLoopToolUseTurn < REMINDER_INTERVAL || reminderInjectedThisCycle) {
-            return {};
-        }
-        if (pendingReminders.length === 0)
-            return {};
-        reminderInjectedThisCycle = true;
-        lastLoopToolUseTurn = currentTurn;
+        canInjectReminder = false;
         const reminder = pendingReminders.shift();
         return {
             content: [...event.content, { type: "text", text: reminder }],
@@ -237,6 +226,7 @@ Use this tool whenever the user asks to:
 - "watch for..." or "when X happens, do Y"
 - "schedule a recurring check"
 - set up a periodic monitor or poller
+- has a task list with open items — create a loop to work through tasks automatically
 
 DO NOT use raw Bash loops (for/sleep/while). Use LoopCreate instead — it integrates with the session lifecycle, survives across turns, and the scheduler handles timing.
 
@@ -261,6 +251,7 @@ Skip this tool when the task is a one-off check (just do it directly) or when th
             "When the user asks for a loop, repeating task, periodic check, or scheduled reminder, use LoopCreate — not raw Bash for/sleep/while.",
             "Use LoopCreate for any 'every X seconds/minutes/hours' requests.",
             "After creating a loop, tell the user the loop ID so they can cancel it with LoopDelete.",
+            "After creating tasks with TaskCreate, create a loop to keep working through them: LoopCreate trigger='30s' prompt='Run TaskList, pick the next available pending task, work on it. Continue until all tasks are completed.' This ensures the agent keeps making progress across turns instead of stopping after one task.",
         ],
         parameters: Type.Object({
             trigger: Type.String({ description: "Cron expression (e.g., '5m', '1h', '0 9 * * 1-5'), event source (e.g., 'tool_execution_start'), or JSON hybrid spec" }),

package/package.json

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

package/src/index.ts

@@ -35,9 +35,6 @@ function textResult(msg: string) {
   return { content: [{ type: "text" as const, text: msg }], details: undefined as any };
 }
 
-const LOOP_TOOL_NAMES = new Set(["LoopCreate", "LoopList", "LoopDelete", "MonitorCreate", "MonitorList", "MonitorStop"]);
-const REMINDER_INTERVAL = 3;
-
 const SYSTEM_REMINDER_TEMPLATE = `<system-reminder>
 Scheduled loop "%propmpt%" fired. Trigger: %trigger_info%.
 [loop:%loop_id%]
@@ -197,9 +194,7 @@ export default function (pi: ExtensionAPI) {
 
   // ── System-reminder injection for loop fires ──
 
-  let currentTurn = 0;
-  let lastLoopToolUseTurn = 0;
-  let reminderInjectedThisCycle = false;
+  let canInjectReminder = true;
   const pendingReminders: string[] = [];
 
   pi.on("loop:fire" as any, (data: any) => {
@@ -219,25 +214,16 @@ export default function (pi: ExtensionAPI) {
     pendingReminders.push(reminder);
   });
 
+  // Allow one reminder injection per agent turn (at the first tool call),
+  // so the reminder arrives between batches of work, not mid-execution.
   pi.on("turn_start", async () => {
-    currentTurn++;
+    canInjectReminder = true;
   });
 
   pi.on("tool_result", async (event) => {
-    if (LOOP_TOOL_NAMES.has(event.toolName)) {
-      lastLoopToolUseTurn = currentTurn;
-      reminderInjectedThisCycle = false;
-      return {};
-    }
-
-    if (currentTurn - lastLoopToolUseTurn < REMINDER_INTERVAL || reminderInjectedThisCycle) {
-      return {};
-    }
-
-    if (pendingReminders.length === 0) return {};
+    if (!canInjectReminder || pendingReminders.length === 0) return {};
 
-    reminderInjectedThisCycle = true;
-    lastLoopToolUseTurn = currentTurn;
+    canInjectReminder = false;
     const reminder = pendingReminders.shift()!;
     return {
       content: [...event.content, { type: "text" as const, text: reminder }],
@@ -260,6 +246,7 @@ Use this tool whenever the user asks to:
 - "watch for..." or "when X happens, do Y"
 - "schedule a recurring check"
 - set up a periodic monitor or poller
+- has a task list with open items — create a loop to work through tasks automatically
 
 DO NOT use raw Bash loops (for/sleep/while). Use LoopCreate instead — it integrates with the session lifecycle, survives across turns, and the scheduler handles timing.
 
@@ -284,6 +271,7 @@ Skip this tool when the task is a one-off check (just do it directly) or when th
       "When the user asks for a loop, repeating task, periodic check, or scheduled reminder, use LoopCreate — not raw Bash for/sleep/while.",
       "Use LoopCreate for any 'every X seconds/minutes/hours' requests.",
       "After creating a loop, tell the user the loop ID so they can cancel it with LoopDelete.",
+      "After creating tasks with TaskCreate, create a loop to keep working through them: LoopCreate trigger='30s' prompt='Run TaskList, pick the next available pending task, work on it. Continue until all tasks are completed.' This ensures the agent keeps making progress across turns instead of stopping after one task.",
     ],
     parameters: Type.Object({
       trigger: Type.String({ description: "Cron expression (e.g., '5m', '1h', '0 9 * * 1-5'), event source (e.g., 'tool_execution_start'), or JSON hybrid spec" }),