@trevonistrevon/pi-loop 0.5.9 → 0.6.0

package/src/tools/monitor-tools.ts

@@ -1,6 +1,7 @@
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { Type } from "typebox";
 import type { LoopEntry, MonitorEntry, Trigger } from "../types.js";
+import { textResult } from "./tool-result.js";
 
 interface MonitorManagerLike {
   list(): MonitorEntry[];
@@ -26,10 +27,6 @@ export interface MonitorToolsOptions {
   handleMonitorDoneLoop: (doneLoop: LoopEntry, monitorId: string) => void;
 }
 
-function textResult(msg: string) {
-  return { content: [{ type: "text" as const, text: msg }], details: undefined as any };
-}
-
 function formatRemaining(ms: number): string {
   if (ms < 60000) return `${Math.round(ms / 1000)}s`;
   if (ms < 3600000) return `${Math.round(ms / 60000)}m`;

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

@@ -1,12 +1,17 @@
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { Type } from "typebox";
-import { emitNativeTaskEvent } from "../runtime/task-events.js";
+import {
+  createTask,
+  deleteTask,
+  type TaskBacklogResult,
+  type TaskMutationContext,
+  updateTask,
+} from "../runtime/task-mutations.js";
 import { TaskStore } from "../task-store.js";
+import type { TaskStatus } from "../task-types.js";
+import { textResult } from "./tool-result.js";
 
-export interface TaskBacklogResult {
-  created: boolean;
-  entry?: { id: string };
-}
+export type { TaskBacklogResult };
 
 export interface NativeTaskToolsOptions {
   pi: ExtensionAPI;
@@ -15,12 +20,15 @@ export interface NativeTaskToolsOptions {
   updateWidget: () => void;
 }
 
-function textResult(msg: string) {
-  return { content: [{ type: "text" as const, text: msg }], details: undefined as any };
+function backlogSuffix(backlog: TaskBacklogResult): string {
+  return backlog.created && backlog.entry
+    ? `\nBacklog worker loop #${backlog.entry.id} created`
+    : "";
 }
 
 export function registerNativeTaskTools(options: NativeTaskToolsOptions): void {
   const { pi, taskStore, evaluateTaskBacklog, updateWidget } = options;
+  const mutationCtx: TaskMutationContext = { pi, taskStore, evaluateTaskBacklog, updateWidget };
 
   pi.registerTool({
     name: "TaskCreate",
@@ -47,15 +55,11 @@ Fields:
       description: Type.String({ description: "Detailed description of what needs to be done" }),
     }),
     async execute(_toolCallId, params) {
-      const entry = taskStore.create(params.subject, params.description);
-      emitNativeTaskEvent(pi, "tasks:created", entry);
-      const backlog = await evaluateTaskBacklog(taskStore, taskStore.pendingCount());
-      updateWidget();
-
-      const autoLoopMsg = backlog.created && backlog.entry
-        ? `\nBacklog worker loop #${backlog.entry.id} created`
-        : "";
-      return Promise.resolve(textResult(`Task #${entry.id} created: ${entry.subject}${autoLoopMsg}`));
+      const { entry, backlog } = await createTask(mutationCtx, {
+        subject: params.subject,
+        description: params.description,
+      });
+      return textResult(`Task #${entry.id} created: ${entry.subject}${backlogSuffix(backlog)}`);
     },
   });
 
@@ -104,34 +108,15 @@ Parameters: id (required), status, subject, description`,
     }),
     async execute(_toolCallId, params) {
       const { id, status, subject, description } = params;
-      let entry = taskStore.get(id);
-      if (!entry) return Promise.resolve(textResult(`Task #${id} not found`));
-
-      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();
-      const backlog = await evaluateTaskBacklog(taskStore, taskStore.pendingCount());
+      const result = await updateTask(mutationCtx, {
+        id,
+        status: status as TaskStatus | undefined,
+        subject,
+        description,
+      });
+      if (!result) return textResult(`Task #${id} not found`);
       const statusMsg = status ? ` → ${status}` : "";
-      const autoLoopMsg = backlog.created && backlog.entry
-        ? `\nBacklog worker loop #${backlog.entry.id} created`
-        : "";
-      return Promise.resolve(textResult(`Task #${id} updated${statusMsg}${autoLoopMsg}`));
+      return textResult(`Task #${id} updated${statusMsg}${backlogSuffix(result.backlog)}`);
     },
   });
 
@@ -143,15 +128,9 @@ 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`));
-      }
-      return Promise.resolve(textResult(`Task #${params.id} not found`));
+      const result = await deleteTask(mutationCtx, params.id);
+      if (!result) return textResult(`Task #${params.id} not found`);
+      return textResult(`Task #${params.id} deleted`);
     },
   });
 }

package/src/tools/tool-result.ts

@@ -0,0 +1,4 @@
+/** Plain-text tool result in the shape pi's registerTool expects. */
+export function textResult(msg: string) {
+  return { content: [{ type: "text" as const, text: msg }], details: undefined };
+}

package/src/trigger-system.ts

@@ -58,7 +58,7 @@ export class TriggerSystem {
     }
     const subs = this.eventSubscriptions.get(source)!;
 
-    const unsub = this.pi.events.on(source as any, (data: any) => {
+    const unsub = this.pi.events.on(source, (data: unknown) => {
       if (entry.trigger.type === "hybrid") {
         this.handleHybridFire(entry, data);
       } else {
@@ -71,7 +71,7 @@ export class TriggerSystem {
     subs.set(entry.id, unsub);
   }
 
-  private handleHybridFire(entry: LoopEntry, _data: any): void {
+  private handleHybridFire(entry: LoopEntry, _data: unknown): void {
     const now = Date.now();
     const last = this.lastFireTime.get(entry.id) ?? 0;
     const debounceMs = entry.trigger.type === "hybrid" ? entry.trigger.debounceMs : 0;
@@ -123,7 +123,7 @@ export class TriggerSystem {
     }
   }
 
-  private matchesFilter(data: any, filter?: string): boolean {
+  private matchesFilter(data: unknown, filter?: string): boolean {
     if (!filter) return true;
 
     if (filter.startsWith("regex:")) {
@@ -138,7 +138,7 @@ export class TriggerSystem {
     try {
       const parsed = JSON.parse(filter);
       for (const [key, value] of Object.entries(parsed)) {
-        const dataValue = data?.[key];
+        const dataValue = (data as Record<string, unknown> | undefined)?.[key];
         if (dataValue === undefined) return false;
         if (typeof value === "object" && typeof dataValue === "object") {
           if (JSON.stringify(value) !== JSON.stringify(dataValue)) return false;

package/vitest.config.ts

@@ -15,14 +15,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 84%, branches 75%,
-      // funcs 95%, lines 86%) to catch regressions. Raised in Phase 4 after the
-      // runtime/ + tools/ suites landed.
+      // Floors set just below current actuals (stmts 91%, branches 83%,
+      // funcs 98%, lines 94%) to catch regressions. Re-anchored after the
+      // goal-subsystem removal and the rpc/native-task-rpc/command suites landed.
       thresholds: {
-        statements: 82,
-        branches: 73,
-        functions: 94,
-        lines: 84,
+        statements: 89,
+        branches: 80,
+        functions: 96,
+        lines: 91,
       },
     },
   },