@trevonistrevon/pi-loop 0.5.1 → 0.5.2

package/dist/goal-store.d.ts

@@ -2,9 +2,11 @@ import type { GoalCriteria, GoalEntry, GoalProgressSnapshot, GoalScope } from ".
 export declare class GoalStore {
     private filePath;
     private lockPath;
+    private lastLoadedSignature;
     private nextId;
     private goals;
     constructor(listIdOrPath?: string);
+    private getFileSignature;
     private load;
     private save;
     private withLock;

package/dist/goal-store.js

@@ -1,4 +1,4 @@
-import { existsSync, mkdirSync, readFileSync, renameSync, unlinkSync, writeFileSync } from "node:fs";
+import { existsSync, mkdirSync, readFileSync, renameSync, statSync, unlinkSync, writeFileSync } from "node:fs";
 import { homedir } from "node:os";
 import { dirname, isAbsolute, join } from "node:path";
 import { reduceGoalState } from "./goal-reducer.js";
@@ -52,6 +52,7 @@ function isProcessRunning(pid) {
 export class GoalStore {
     filePath;
     lockPath;
+    lastLoadedSignature;
     nextId = 1;
     goals = new Map();
     constructor(listIdOrPath) {
@@ -64,10 +65,19 @@ export class GoalStore {
         this.lockPath = filePath + ".lock";
         this.load();
     }
-    load() {
+    getFileSignature() {
+        if (!this.filePath || !existsSync(this.filePath))
+            return undefined;
+        const stat = statSync(this.filePath);
+        return `${stat.mtimeMs}:${stat.size}`;
+    }
+    load(force = false) {
         if (!this.filePath)
             return;
-        if (!existsSync(this.filePath))
+        const signature = this.getFileSignature();
+        if (!signature)
+            return;
+        if (!force && signature === this.lastLoadedSignature)
             return;
         try {
             const data = JSON.parse(readFileSync(this.filePath, "utf-8"));
@@ -76,6 +86,7 @@ export class GoalStore {
             for (const goal of data.goals) {
                 this.goals.set(goal.id, goal);
             }
+            this.lastLoadedSignature = signature;
         }
         catch { /* corrupt file — start fresh */ }
     }
@@ -89,13 +100,14 @@ export class GoalStore {
         const tmpPath = this.filePath + ".tmp";
         writeFileSync(tmpPath, JSON.stringify(data, null, 2));
         renameSync(tmpPath, this.filePath);
+        this.lastLoadedSignature = this.getFileSignature();
     }
     withLock(fn) {
         if (!this.lockPath)
             return fn();
         acquireLock(this.lockPath);
         try {
-            this.load();
+            this.load(true);
             const result = fn();
             this.save();
             return result;

package/dist/store.d.ts

@@ -2,9 +2,11 @@ import type { LoopEntry, Trigger } from "./types.js";
 export declare class LoopStore {
     private filePath;
     private lockPath;
+    private lastLoadedSignature;
     private nextId;
     private loops;
     constructor(listIdOrPath?: string);
+    private getFileSignature;
     private load;
     private save;
     private withLock;

package/dist/store.js

@@ -1,4 +1,4 @@
-import { existsSync, mkdirSync, readFileSync, renameSync, unlinkSync, writeFileSync } from "node:fs";
+import { existsSync, mkdirSync, readFileSync, renameSync, statSync, unlinkSync, writeFileSync } from "node:fs";
 import { homedir } from "node:os";
 import { dirname, isAbsolute, join } from "node:path";
 import { reduceLoopState } from "./loop-reducer.js";
@@ -52,6 +52,7 @@ function isProcessRunning(pid) {
 export class LoopStore {
     filePath;
     lockPath;
+    lastLoadedSignature;
     nextId = 1;
     loops = new Map();
     constructor(listIdOrPath) {
@@ -64,10 +65,19 @@ export class LoopStore {
         this.lockPath = filePath + ".lock";
         this.load();
     }
-    load() {
+    getFileSignature() {
+        if (!this.filePath || !existsSync(this.filePath))
+            return undefined;
+        const stat = statSync(this.filePath);
+        return `${stat.mtimeMs}:${stat.size}`;
+    }
+    load(force = false) {
         if (!this.filePath)
             return;
-        if (!existsSync(this.filePath))
+        const signature = this.getFileSignature();
+        if (!signature)
+            return;
+        if (!force && signature === this.lastLoadedSignature)
             return;
         try {
             const data = JSON.parse(readFileSync(this.filePath, "utf-8"));
@@ -76,6 +86,7 @@ export class LoopStore {
             for (const loop of data.loops) {
                 this.loops.set(loop.id, loop);
             }
+            this.lastLoadedSignature = signature;
         }
         catch { /* corrupt file — start fresh */ }
     }
@@ -89,13 +100,14 @@ export class LoopStore {
         const tmpPath = this.filePath + ".tmp";
         writeFileSync(tmpPath, JSON.stringify(data, null, 2));
         renameSync(tmpPath, this.filePath);
+        this.lastLoadedSignature = this.getFileSignature();
     }
     withLock(fn) {
         if (!this.lockPath)
             return fn();
         acquireLock(this.lockPath);
         try {
-            this.load();
+            this.load(true);
             const result = fn();
             this.save();
             return result;

package/dist/task-store.d.ts

@@ -2,9 +2,11 @@ import type { TaskEntry } from "./task-types.js";
 export declare class TaskStore {
     private filePath;
     private lockPath;
+    private lastLoadedSignature;
     private nextId;
     private tasks;
     constructor(listIdOrPath?: string);
+    private getFileSignature;
     private load;
     private save;
     private withLock;

package/dist/task-store.js

@@ -1,4 +1,4 @@
-import { existsSync, mkdirSync, readFileSync, renameSync, unlinkSync, writeFileSync } from "node:fs";
+import { existsSync, mkdirSync, readFileSync, renameSync, statSync, unlinkSync, writeFileSync } from "node:fs";
 import { homedir } from "node:os";
 import { dirname, isAbsolute, join } from "node:path";
 import { reduceTaskState } from "./task-reducer.js";
@@ -52,6 +52,7 @@ function isProcessRunning(pid) {
 export class TaskStore {
     filePath;
     lockPath;
+    lastLoadedSignature;
     nextId = 1;
     tasks = new Map();
     constructor(listIdOrPath) {
@@ -64,10 +65,19 @@ export class TaskStore {
         this.lockPath = filePath + ".lock";
         this.load();
     }
-    load() {
+    getFileSignature() {
+        if (!this.filePath || !existsSync(this.filePath))
+            return undefined;
+        const stat = statSync(this.filePath);
+        return `${stat.mtimeMs}:${stat.size}`;
+    }
+    load(force = false) {
         if (!this.filePath)
             return;
-        if (!existsSync(this.filePath))
+        const signature = this.getFileSignature();
+        if (!signature)
+            return;
+        if (!force && signature === this.lastLoadedSignature)
             return;
         try {
             const data = JSON.parse(readFileSync(this.filePath, "utf-8"));
@@ -76,6 +86,7 @@ export class TaskStore {
             for (const task of data.tasks) {
                 this.tasks.set(task.id, task);
             }
+            this.lastLoadedSignature = signature;
         }
         catch { /* corrupt file — start fresh */ }
     }
@@ -89,13 +100,14 @@ export class TaskStore {
         const tmpPath = this.filePath + ".tmp";
         writeFileSync(tmpPath, JSON.stringify(data, null, 2));
         renameSync(tmpPath, this.filePath);
+        this.lastLoadedSignature = this.getFileSignature();
     }
     withLock(fn) {
         if (!this.lockPath)
             return fn();
         acquireLock(this.lockPath);
         try {
-            this.load();
+            this.load(true);
             const result = fn();
             this.save();
             return result;

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

@@ -7,14 +7,17 @@ export function registerNativeTaskTools(options) {
     pi.registerTool({
         name: "TaskCreate",
         label: "TaskCreate",
-        description: `Create a task for tracking work across turns. Use when you need to track progress on complex multi-step tasks.
+        description: `Create a task for tracking work across turns. Use when you need to track progress on complex multi-step tasks or turn a broad user goal into a concrete backlog.
 
 Fields:
 - subject: brief actionable title
-- description: detailed requirements
-- metadata: optional tags/metadata`,
+- description: detailed requirements and done condition`,
         promptGuidelines: [
             "Use TaskCreate to track complex multi-step work across turns.",
+            "When the user gives a broad goal, use multiple TaskCreate calls to decompose it into a small backlog of concrete tasks rather than one oversized task.",
+            "Prefer 2-5 tasks that separate investigation, implementation, validation, and reporting or commit-prep when those phases are distinct.",
+            "Make each `subject` a short verb-object action.",
+            "Make each `description` include the expected artifact, outcome, or done condition so another turn can pick the task up cleanly.",
             "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.",
         ],

package/dist/ui/widget.d.ts

@@ -9,7 +9,6 @@ export declare class LoopWidget {
     private store;
     private monitorManager;
     private uiCtx;
-    private interval;
     private taskSummaryProvider;
     constructor(store: LoopStore, monitorManager: MonitorManager);
     setUICtx(ctx: ExtensionUIContext): void;

package/dist/ui/widget.js

@@ -2,7 +2,6 @@ export class LoopWidget {
     store;
     monitorManager;
     uiCtx;
-    interval;
     taskSummaryProvider;
     constructor(store, monitorManager) {
         this.store = store;
@@ -20,15 +19,7 @@ export class LoopWidget {
     update() {
         if (!this.uiCtx)
             return;
-        const status = this.computeStatus();
-        if (status && !this.interval) {
-            this.interval = setInterval(() => this.update(), 5000);
-        }
-        if (!status && this.interval) {
-            clearInterval(this.interval);
-            this.interval = undefined;
-        }
-        this.uiCtx.setStatus("loops", status);
+        this.uiCtx.setStatus("loops", this.computeStatus());
     }
     computeStatus() {
         const loops = this.store.list().filter(l => l.status === "active");
@@ -50,10 +41,6 @@ export class LoopWidget {
         return line;
     }
     dispose() {
-        if (this.interval) {
-            clearInterval(this.interval);
-            this.interval = undefined;
-        }
         this.uiCtx?.setStatus("loops", undefined);
     }
 }

package/package.json

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

package/src/goal-store.ts

@@ -1,4 +1,4 @@
-import { existsSync, mkdirSync, readFileSync, renameSync, unlinkSync, writeFileSync } from "node:fs";
+import { existsSync, mkdirSync, readFileSync, renameSync, statSync, unlinkSync, writeFileSync } from "node:fs";
 import { homedir } from "node:os";
 import { dirname, isAbsolute, join } from "node:path";
 import { type GoalReducerEvent, reduceGoalState } from "./goal-reducer.js";
@@ -51,6 +51,7 @@ function isProcessRunning(pid: number): boolean {
 export class GoalStore {
   private filePath: string | undefined;
   private lockPath: string | undefined;
+  private lastLoadedSignature: string | undefined;
 
   private nextId = 1;
   private goals = new Map<string, GoalEntry>();
@@ -65,9 +66,17 @@ export class GoalStore {
     this.load();
   }
 
-  private load(): void {
+  private getFileSignature(): string | undefined {
+    if (!this.filePath || !existsSync(this.filePath)) return undefined;
+    const stat = statSync(this.filePath);
+    return `${stat.mtimeMs}:${stat.size}`;
+  }
+
+  private load(force = false): void {
     if (!this.filePath) return;
-    if (!existsSync(this.filePath)) return;
+    const signature = this.getFileSignature();
+    if (!signature) return;
+    if (!force && signature === this.lastLoadedSignature) return;
     try {
       const data: GoalStoreData = JSON.parse(readFileSync(this.filePath, "utf-8"));
       this.nextId = data.nextId;
@@ -75,6 +84,7 @@ export class GoalStore {
       for (const goal of data.goals) {
         this.goals.set(goal.id, goal);
       }
+      this.lastLoadedSignature = signature;
     } catch { /* corrupt file — start fresh */ }
   }
 
@@ -87,13 +97,14 @@ export class GoalStore {
     const tmpPath = this.filePath + ".tmp";
     writeFileSync(tmpPath, JSON.stringify(data, null, 2));
     renameSync(tmpPath, this.filePath);
+    this.lastLoadedSignature = this.getFileSignature();
   }
 
   private withLock<T>(fn: () => T): T {
     if (!this.lockPath) return fn();
     acquireLock(this.lockPath);
     try {
-      this.load();
+      this.load(true);
       const result = fn();
       this.save();
       return result;

package/src/store.ts

@@ -1,4 +1,4 @@
-import { existsSync, mkdirSync, readFileSync, renameSync, unlinkSync, writeFileSync } from "node:fs";
+import { existsSync, mkdirSync, readFileSync, renameSync, statSync, unlinkSync, writeFileSync } from "node:fs";
 import { homedir } from "node:os";
 import { dirname, isAbsolute, join } from "node:path";
 import { type LoopReducerEvent, type LoopReducerState, reduceLoopState } from "./loop-reducer.js";
@@ -44,6 +44,7 @@ function isProcessRunning(pid: number): boolean {
 export class LoopStore {
   private filePath: string | undefined;
   private lockPath: string | undefined;
+  private lastLoadedSignature: string | undefined;
 
   private nextId = 1;
   private loops = new Map<string, LoopEntry>();
@@ -58,9 +59,17 @@ export class LoopStore {
     this.load();
   }
 
-  private load(): void {
+  private getFileSignature(): string | undefined {
+    if (!this.filePath || !existsSync(this.filePath)) return undefined;
+    const stat = statSync(this.filePath);
+    return `${stat.mtimeMs}:${stat.size}`;
+  }
+
+  private load(force = false): void {
     if (!this.filePath) return;
-    if (!existsSync(this.filePath)) return;
+    const signature = this.getFileSignature();
+    if (!signature) return;
+    if (!force && signature === this.lastLoadedSignature) return;
     try {
       const data: LoopStoreData = JSON.parse(readFileSync(this.filePath, "utf-8"));
       this.nextId = data.nextId;
@@ -68,6 +77,7 @@ export class LoopStore {
       for (const loop of data.loops) {
         this.loops.set(loop.id, loop);
       }
+      this.lastLoadedSignature = signature;
     } catch { /* corrupt file — start fresh */ }
   }
 
@@ -80,13 +90,14 @@ export class LoopStore {
     const tmpPath = this.filePath + ".tmp";
     writeFileSync(tmpPath, JSON.stringify(data, null, 2));
     renameSync(tmpPath, this.filePath);
+    this.lastLoadedSignature = this.getFileSignature();
   }
 
   private withLock<T>(fn: () => T): T {
     if (!this.lockPath) return fn();
     acquireLock(this.lockPath);
     try {
-      this.load();
+      this.load(true);
       const result = fn();
       this.save();
       return result;

package/src/task-store.ts

@@ -1,4 +1,4 @@
-import { existsSync, mkdirSync, readFileSync, renameSync, unlinkSync, writeFileSync } from "node:fs";
+import { existsSync, mkdirSync, readFileSync, renameSync, statSync, unlinkSync, writeFileSync } from "node:fs";
 import { homedir } from "node:os";
 import { dirname, isAbsolute, join } from "node:path";
 import { reduceTaskState, type TaskReducerEvent, type TaskReducerState } from "./task-reducer.js";
@@ -44,6 +44,7 @@ function isProcessRunning(pid: number): boolean {
 export class TaskStore {
   private filePath: string | undefined;
   private lockPath: string | undefined;
+  private lastLoadedSignature: string | undefined;
 
   private nextId = 1;
   private tasks = new Map<string, TaskEntry>();
@@ -58,9 +59,17 @@ export class TaskStore {
     this.load();
   }
 
-  private load(): void {
+  private getFileSignature(): string | undefined {
+    if (!this.filePath || !existsSync(this.filePath)) return undefined;
+    const stat = statSync(this.filePath);
+    return `${stat.mtimeMs}:${stat.size}`;
+  }
+
+  private load(force = false): void {
     if (!this.filePath) return;
-    if (!existsSync(this.filePath)) return;
+    const signature = this.getFileSignature();
+    if (!signature) return;
+    if (!force && signature === this.lastLoadedSignature) return;
     try {
       const data: TaskStoreData = JSON.parse(readFileSync(this.filePath, "utf-8"));
       this.nextId = data.nextId;
@@ -68,6 +77,7 @@ export class TaskStore {
       for (const task of data.tasks) {
         this.tasks.set(task.id, task);
       }
+      this.lastLoadedSignature = signature;
     } catch { /* corrupt file — start fresh */ }
   }
 
@@ -80,13 +90,14 @@ export class TaskStore {
     const tmpPath = this.filePath + ".tmp";
     writeFileSync(tmpPath, JSON.stringify(data, null, 2));
     renameSync(tmpPath, this.filePath);
+    this.lastLoadedSignature = this.getFileSignature();
   }
 
   private withLock<T>(fn: () => T): T {
     if (!this.lockPath) return fn();
     acquireLock(this.lockPath);
     try {
-      this.load();
+      this.load(true);
       const result = fn();
       this.save();
       return result;

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

@@ -24,14 +24,17 @@ export function registerNativeTaskTools(options: NativeTaskToolsOptions): void {
   pi.registerTool({
     name: "TaskCreate",
     label: "TaskCreate",
-    description: `Create a task for tracking work across turns. Use when you need to track progress on complex multi-step tasks.
+    description: `Create a task for tracking work across turns. Use when you need to track progress on complex multi-step tasks or turn a broad user goal into a concrete backlog.
 
 Fields:
 - subject: brief actionable title
-- description: detailed requirements
-- metadata: optional tags/metadata`,
+- description: detailed requirements and done condition`,
     promptGuidelines: [
       "Use TaskCreate to track complex multi-step work across turns.",
+      "When the user gives a broad goal, use multiple TaskCreate calls to decompose it into a small backlog of concrete tasks rather than one oversized task.",
+      "Prefer 2-5 tasks that separate investigation, implementation, validation, and reporting or commit-prep when those phases are distinct.",
+      "Make each `subject` a short verb-object action.",
+      "Make each `description` include the expected artifact, outcome, or done condition so another turn can pick the task up cleanly.",
       "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.",
     ],

package/src/ui/widget.ts

@@ -9,7 +9,6 @@ interface TaskSummary {
 
 export class LoopWidget {
   private uiCtx: ExtensionUIContext | undefined;
-  private interval: ReturnType<typeof setInterval> | undefined;
   private taskSummaryProvider: (() => TaskSummary) | undefined;
 
   constructor(
@@ -31,17 +30,7 @@ export class LoopWidget {
 
   update() {
     if (!this.uiCtx) return;
-
-    const status = this.computeStatus();
-    if (status && !this.interval) {
-      this.interval = setInterval(() => this.update(), 5000);
-    }
-    if (!status && this.interval) {
-      clearInterval(this.interval);
-      this.interval = undefined;
-    }
-
-    this.uiCtx.setStatus("loops", status);
+    this.uiCtx.setStatus("loops", this.computeStatus());
   }
 
   private computeStatus(): string | undefined {
@@ -64,10 +53,6 @@ export class LoopWidget {
   }
 
   dispose() {
-    if (this.interval) {
-      clearInterval(this.interval);
-      this.interval = undefined;
-    }
     this.uiCtx?.setStatus("loops", undefined);
   }
 }