@trevonistrevon/pi-loop 0.4.5 → 0.4.7

package/README.md

@@ -107,6 +107,8 @@ Only task counts and the single active/next task are shown in the widget so atte
 | `PI_LOOP_SCOPE` | `memory` (ephemeral), `session` (per-session file), `project` (shared) | `session` |
 | `PI_LOOP_DEBUG` | Debug logging to stderr | unset |
 
+In `session` scope (default), loop and task files are saved per session ID (e.g. `.pi/tasks/tasks-<sessionId>.json`) so concurrent sessions and worktree agents do not share state. In `memory` scope nothing persists to disk.
+
 ## Limits
 
 25 active loops, 25 running monitors. Recurring loops expire after 7 days.

package/dist/index.js

@@ -53,9 +53,14 @@ export default function (pi) {
             return undefined;
         return join(process.cwd(), ".pi", "loops", "loops.json");
     }
-    function resolveTaskStorePath() {
+    function resolveTaskStorePath(sessionId) {
         if (loopScope === "memory")
             return undefined;
+        if (loopScope === "session" && sessionId) {
+            return join(process.cwd(), ".pi", "tasks", `tasks-${sessionId}.json`);
+        }
+        if (loopScope === "session")
+            return undefined;
         return join(process.cwd(), ".pi", "tasks", "tasks.json");
     }
     let store = new LoopStore(resolveStorePath());
@@ -319,6 +324,7 @@ export default function (pi) {
     let storeUpgraded = false;
     let persistedShown = false;
     let _latestCtx;
+    let _sessionId;
     function upgradeStoreIfNeeded(ctx) {
         if (storeUpgraded)
             return;
@@ -347,6 +353,7 @@ export default function (pi) {
     }
     pi.on("turn_start", async (_event, ctx) => {
         _latestCtx = ctx;
+        _sessionId = ctx.sessionManager.getSessionId();
         widget.setUICtx(ctx.ui);
         upgradeStoreIfNeeded(ctx);
         widget.update();
@@ -381,6 +388,7 @@ export default function (pi) {
         triggerSystem.stop();
         agentRunning = false;
         pendingNotifications.clear();
+        _sessionId = undefined;
         const isResume = event?.reason === "resume";
         storeUpgraded = false;
         persistedShown = false;
@@ -918,6 +926,34 @@ Use MonitorList to find the monitor ID, then stop it with this tool.`,
         const active = loops.filter(l => l.status === "active").length;
         ui.notify(`${active}/${loops.length} active loops (max 25)`, "info");
     }
+    const AUTO_TASK_WORKER_THRESHOLD = 5;
+    const AUTO_TASK_WORKER_PROMPT = "Run TaskList, pick next pending task, mark it in_progress, implement it, run validation, complete it. If no pending tasks remain, call LoopDelete on your own loop ID.";
+    function findAutoTaskWorkerLoop() {
+        return store.list().find(entry => entry.status === "active"
+            && entry.prompt === AUTO_TASK_WORKER_PROMPT
+            && triggerHasEventSource(entry.trigger, "tasks:created"));
+    }
+    async function ensureAutoTaskWorkerLoop(taskStore) {
+        if (taskStore.pendingCount() < AUTO_TASK_WORKER_THRESHOLD)
+            return { created: false };
+        const existing = findAutoTaskWorkerLoop();
+        if (existing)
+            return { entry: existing, created: false };
+        const trigger = {
+            type: "hybrid",
+            cron: "*/5 * * * *",
+            event: { source: "tasks:created" },
+            debounceMs: 30000,
+        };
+        const entry = store.create(trigger, AUTO_TASK_WORKER_PROMPT, {
+            recurring: true,
+            maxFires: 30,
+        });
+        triggerSystem.add(entry);
+        await maybeBootstrapTaskLoop(entry);
+        widget.update();
+        return { entry, created: true };
+    }
     async function createNativeTaskInteractively(ui) {
         if (!nativeTaskStore) {
             ui.notify("Native tasks are unavailable while pi-tasks is active", "warning");
@@ -928,8 +964,18 @@ Use MonitorList to find the monitor ID, then stop it with this tool.`,
             return;
         const description = await ui.input("Task description") || subject;
         const entry = nativeTaskStore.create(subject, description);
+        pi.events.emit("tasks:created", {
+            taskId: entry.id,
+            subject: entry.subject,
+            description: entry.description,
+            status: entry.status,
+        });
+        const worker = await ensureAutoTaskWorkerLoop(nativeTaskStore);
         widget.update();
         ui.notify(`Task #${entry.id} created`, "info");
+        if (worker.created && worker.entry) {
+            ui.notify(`Worker loop #${worker.entry.id} auto-created`, "info");
+        }
     }
     async function viewNativeTasks(ui) {
         if (!nativeTaskStore) {
@@ -995,7 +1041,7 @@ Use MonitorList to find the monitor ID, then stop it with this tool.`,
     setTimeout(async () => {
         if (tasksAvailable || nativeTasksRegistered)
             return;
-        nativeTaskStore = new TaskStore(resolveTaskStorePath());
+        nativeTaskStore = new TaskStore(resolveTaskStorePath(_sessionId));
         nativeTasksRegistered = true;
         const taskStore = nativeTaskStore;
         pi.registerCommand("tasks", {
@@ -1014,8 +1060,12 @@ Use MonitorList to find the monitor ID, then stop it with this tool.`,
                         description: entry.description,
                         status: entry.status,
                     });
+                    const worker = await ensureAutoTaskWorkerLoop(nativeTaskStore);
                     widget.update();
                     ctx.ui.notify(`Task #${entry.id} created`, "info");
+                    if (worker.created && worker.entry) {
+                        ctx.ui.notify(`Worker loop #${worker.entry.id} auto-created`, "info");
+                    }
                     return;
                 }
                 await viewNativeTasks(ctx.ui);
@@ -1032,13 +1082,14 @@ Fields:
 - metadata: optional tags/metadata`,
             promptGuidelines: [
                 "Use TaskCreate to track complex multi-step work across turns.",
+                "Break work into small, independently completable tasks. A task should be finishable in one focused session — if a task would take multiple turns, split it further.",
                 "TaskCreate accepts `subject` and `description` parameters only — do not invent extra fields unless the schema explicitly adds them.",
             ],
             parameters: Type.Object({
                 subject: Type.String({ description: "Brief actionable title for the task" }),
                 description: Type.String({ description: "Detailed description of what needs to be done" }),
             }),
-            execute(_toolCallId, params, _signal, _onUpdate, _ctx) {
+            async execute(_toolCallId, params, _signal, _onUpdate, _ctx) {
                 const entry = taskStore.create(params.subject, params.description);
                 pi.events.emit("tasks:created", {
                     taskId: entry.id,
@@ -1046,12 +1097,12 @@ Fields:
                     description: entry.description,
                     status: entry.status,
                 });
+                const worker = await ensureAutoTaskWorkerLoop(taskStore);
                 widget.update();
-                const pending = taskStore.pendingCount();
-                const hint = pending >= 5
-                    ? `\n(${pending} pending tasks — consider creating a worker loop: LoopCreate trigger='tasks:created' recurring: true maxFires: 30 prompt='Run TaskList, pick next pending task, mark it in_progress, implement it, run validation, complete it. If no pending tasks remain, call LoopDelete on your own loop ID.')`
+                const autoLoopMsg = worker.created && worker.entry
+                    ? `\nWorker loop #${worker.entry.id} auto-created`
                     : "";
-                return Promise.resolve(textResult(`Task #${entry.id} created: ${entry.subject}${hint}`));
+                return Promise.resolve(textResult(`Task #${entry.id} created: ${entry.subject}${autoLoopMsg}`));
             },
         });
         pi.registerTool({

package/package.json

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

package/src/index.ts

@@ -73,8 +73,12 @@ export default function (pi: ExtensionAPI) {
     return join(process.cwd(), ".pi", "loops", "loops.json");
   }
 
-  function resolveTaskStorePath(): string | undefined {
+  function resolveTaskStorePath(sessionId?: string): string | undefined {
     if (loopScope === "memory") return undefined;
+    if (loopScope === "session" && sessionId) {
+      return join(process.cwd(), ".pi", "tasks", `tasks-${sessionId}.json`);
+    }
+    if (loopScope === "session") return undefined;
     return join(process.cwd(), ".pi", "tasks", "tasks.json");
   }
 
@@ -349,6 +353,7 @@ export default function (pi: ExtensionAPI) {
   let storeUpgraded = false;
   let persistedShown = false;
   let _latestCtx: ExtensionContext | undefined;
+  let _sessionId: string | undefined;
 
   function upgradeStoreIfNeeded(ctx: ExtensionContext) {
     if (storeUpgraded) return;
@@ -378,6 +383,7 @@ export default function (pi: ExtensionAPI) {
 
   pi.on("turn_start", async (_event, ctx) => {
     _latestCtx = ctx;
+    _sessionId = ctx.sessionManager.getSessionId();
     widget.setUICtx(ctx.ui);
     upgradeStoreIfNeeded(ctx);
     widget.update();
@@ -417,6 +423,7 @@ export default function (pi: ExtensionAPI) {
     triggerSystem.stop();
     agentRunning = false;
     pendingNotifications.clear();
+    _sessionId = undefined;
 
     const isResume = event?.reason === "resume";
     storeUpgraded = false;
@@ -998,6 +1005,39 @@ Use MonitorList to find the monitor ID, then stop it with this tool.`,
     ui.notify(`${active}/${loops.length} active loops (max 25)`, "info");
   }
 
+  const AUTO_TASK_WORKER_THRESHOLD = 5;
+  const AUTO_TASK_WORKER_PROMPT = "Run TaskList, pick next pending task, mark it in_progress, implement it, run validation, complete it. If no pending tasks remain, call LoopDelete on your own loop ID.";
+
+  function findAutoTaskWorkerLoop(): LoopEntry | undefined {
+    return store.list().find(entry =>
+      entry.status === "active"
+      && entry.prompt === AUTO_TASK_WORKER_PROMPT
+      && triggerHasEventSource(entry.trigger, "tasks:created")
+    );
+  }
+
+  async function ensureAutoTaskWorkerLoop(taskStore: TaskStore): Promise<{ entry?: LoopEntry; created: boolean }> {
+    if (taskStore.pendingCount() < AUTO_TASK_WORKER_THRESHOLD) return { created: false };
+
+    const existing = findAutoTaskWorkerLoop();
+    if (existing) return { entry: existing, created: false };
+
+    const trigger: Trigger = {
+      type: "hybrid",
+      cron: "*/5 * * * *",
+      event: { source: "tasks:created" },
+      debounceMs: 30000,
+    };
+    const entry = store.create(trigger, AUTO_TASK_WORKER_PROMPT, {
+      recurring: true,
+      maxFires: 30,
+    });
+    triggerSystem.add(entry);
+    await maybeBootstrapTaskLoop(entry);
+    widget.update();
+    return { entry, created: true };
+  }
+
   async function createNativeTaskInteractively(ui: ExtensionUIContext) {
     if (!nativeTaskStore) {
       ui.notify("Native tasks are unavailable while pi-tasks is active", "warning");
@@ -1008,8 +1048,18 @@ Use MonitorList to find the monitor ID, then stop it with this tool.`,
     if (!subject) return;
     const description = await ui.input("Task description") || subject;
     const entry = nativeTaskStore.create(subject, description);
+    pi.events.emit("tasks:created", {
+      taskId: entry.id,
+      subject: entry.subject,
+      description: entry.description,
+      status: entry.status,
+    });
+    const worker = await ensureAutoTaskWorkerLoop(nativeTaskStore);
     widget.update();
     ui.notify(`Task #${entry.id} created`, "info");
+    if (worker.created && worker.entry) {
+      ui.notify(`Worker loop #${worker.entry.id} auto-created`, "info");
+    }
   }
 
   async function viewNativeTasks(ui: ExtensionUIContext): Promise<void> {
@@ -1076,7 +1126,7 @@ Use MonitorList to find the monitor ID, then stop it with this tool.`,
 
   setTimeout(async () => {
     if (tasksAvailable || nativeTasksRegistered) return;
-    nativeTaskStore = new TaskStore(resolveTaskStorePath());
+    nativeTaskStore = new TaskStore(resolveTaskStorePath(_sessionId));
     nativeTasksRegistered = true;
     const taskStore = nativeTaskStore;
 
@@ -1096,8 +1146,12 @@ Use MonitorList to find the monitor ID, then stop it with this tool.`,
             description: entry.description,
             status: entry.status,
           });
+          const worker = await ensureAutoTaskWorkerLoop(nativeTaskStore);
           widget.update();
           ctx.ui.notify(`Task #${entry.id} created`, "info");
+          if (worker.created && worker.entry) {
+            ctx.ui.notify(`Worker loop #${worker.entry.id} auto-created`, "info");
+          }
           return;
         }
         await viewNativeTasks(ctx.ui);
@@ -1115,13 +1169,14 @@ Fields:
 - metadata: optional tags/metadata`,
       promptGuidelines: [
         "Use TaskCreate to track complex multi-step work across turns.",
+        "Break work into small, independently completable tasks. A task should be finishable in one focused session — if a task would take multiple turns, split it further.",
         "TaskCreate accepts `subject` and `description` parameters only — do not invent extra fields unless the schema explicitly adds them.",
       ],
       parameters: Type.Object({
         subject: Type.String({ description: "Brief actionable title for the task" }),
         description: Type.String({ description: "Detailed description of what needs to be done" }),
       }),
-      execute(_toolCallId, params, _signal, _onUpdate, _ctx) {
+      async execute(_toolCallId, params, _signal, _onUpdate, _ctx) {
         const entry = taskStore.create(params.subject, params.description);
         pi.events.emit("tasks:created", {
           taskId: entry.id,
@@ -1129,13 +1184,13 @@ Fields:
           description: entry.description,
           status: entry.status,
         });
+        const worker = await ensureAutoTaskWorkerLoop(taskStore);
         widget.update();
 
-        const pending = taskStore.pendingCount();
-        const hint = pending >= 5
-          ? `\n(${pending} pending tasks — consider creating a worker loop: LoopCreate trigger='tasks:created' recurring: true maxFires: 30 prompt='Run TaskList, pick next pending task, mark it in_progress, implement it, run validation, complete it. If no pending tasks remain, call LoopDelete on your own loop ID.')`
+        const autoLoopMsg = worker.created && worker.entry
+          ? `\nWorker loop #${worker.entry.id} auto-created`
           : "";
-        return Promise.resolve(textResult(`Task #${entry.id} created: ${entry.subject}${hint}`));
+        return Promise.resolve(textResult(`Task #${entry.id} created: ${entry.subject}${autoLoopMsg}`));
       },
     });