@ian-pascoe/pi-todo 0.1.2 → 0.1.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ian-pascoe/pi-todo",
-  "version": "0.1.2",
+  "version": "0.1.4",
   "private": false,
   "description": "Minimal session-native Todo List for Pi agents.",
   "keywords": [

package/skills/pi-todo/SKILL.md

@@ -2,6 +2,7 @@
 name: pi-todo
 description: Diagnose Pi Todo when its tool, restored Todo List, model context, clear command, transcript rendering, or widget is missing or incorrect.
 license: MIT
+disable-model-invocation: true
 ---
 
 # Pi Todo

package/src/pi-todo-extension.ts

@@ -13,8 +13,10 @@ import {
   type TodoStateSnapshot,
   type TodoStatus,
   type TodoTask,
+  TodoOperationError,
   type TodoToolDetails,
 } from "./todo-list.js";
+import { TROUBLESHOOTING_HINT } from "./troubleshooting-skill.js";
 
 const TODO_STATE_ENTRY_TYPE = "pi-todo-state";
 const JOURNAL_FAULT = Symbol.for("@ian-pascoe/pi-todo/journal-fault");
@@ -161,7 +163,15 @@ export default function piTodoExtension(pi: ExtensionAPI): void {
     executionMode: "sequential",
     async execute(_toolCallId, params, signal, _onUpdate, context) {
       signal?.throwIfAborted();
-      const result = runTodoAction(params, context);
+      let result: ReturnType<typeof runTodoAction>;
+      try {
+        result = runTodoAction(params, context);
+      } catch (cause) {
+        // Task-operation errors are model-fixable input errors; anything else is a journal failure.
+        if (cause instanceof TodoOperationError) throw cause;
+        const message = cause instanceof Error ? cause.message : String(cause);
+        throw new Error(`${message}\n\n${TROUBLESHOOTING_HINT}`, { cause });
+      }
       return {
         content: [{ type: "text", text: result.message }],
         details: result.details,

package/src/todo-context.ts

@@ -23,6 +23,13 @@ const ProjectionDetails = Type.Object(
   { additionalProperties: false },
 );
 
+/** Context handlers never receive system messages, so anchors must skip Pi's system state. */
+function conversationMessage(entry: SessionEntry): Message | undefined {
+  return buildSessionContext([entry], entry.id).messages.find(
+    (message) => message.role !== "system",
+  );
+}
+
 /** Validates both the serialized shape and Todo List invariants at the journal boundary. */
 export function todoStateFromEntry(entry: SessionEntry): TodoStateSnapshot | undefined {
   if (
@@ -93,7 +100,7 @@ export function projectTodoContext(
     const checkpointIndex = branch.indexOf(checkpoint);
     if (start < 0 || start > checkpointIndex)
       throw new Error("Todo checkpoint cutoff is unavailable");
-    anchor = buildSessionContext([checkpoint], checkpoint.id).messages[0];
+    anchor = conversationMessage(checkpoint);
     for (const entry of branch.slice(0, start).toReversed()) {
       const state = todoStateFromEntry(entry);
       if (!state) continue;
@@ -120,7 +127,7 @@ export function projectTodoContext(
       previousContent = content;
       continue;
     }
-    const message = buildSessionContext([entry], entry.id).messages[0];
+    const message = conversationMessage(entry);
     if (!message) continue;
     if (message.role === "assistant") {
       if (outstanding.size > 0 && pending.length > 0)

package/src/troubleshooting-skill.ts

@@ -0,0 +1,9 @@
+import { fileURLToPath } from "node:url";
+
+/** Absolute path of this package's user-invoked troubleshooting Skill. */
+export const TROUBLESHOOTING_SKILL_PATH = fileURLToPath(
+  new URL("../skills/pi-todo/SKILL.md", import.meta.url),
+);
+
+/** Model-facing pointer appended to failures this package's Skill diagnoses. */
+export const TROUBLESHOOTING_HINT = `For diagnosis, read the pi-todo troubleshooting Skill at ${TROUBLESHOOTING_SKILL_PATH}.`;