@ian-pascoe/pi-todo 0.2.0 → 0.3.0

package/README.md

@@ -35,9 +35,9 @@ A Pi `codemode` script receives the result as an object (the tool declares an `o
 
 Task status is `pending`, `active`, or `completed`; `add` defaults to `pending`. Tasks are flat, duplicate titles are allowed, and multiple Tasks may be active. Set `description` to `null` during `update` to remove it.
 
-Each changed Todo List is projected from its immutable session state entry as a hidden full snapshot at a fixed conversation position. Mutations inside a tool group appear after all sibling results; later requests keep earlier snapshots in place. Clearing the list produces an explicit empty snapshot. The system prompt and tool description do not change.
+Each tool group that changes the Todo List is projected from the immutable session state entries as one hidden full **Todo List Snapshot** of the group's final state, at a fixed conversation position after all of the group's sibling results. A tool group is one assistant turn's tool calls, including several `todo` calls inside one `codemode` script, so ten calls in a script add one Todo List Snapshot, not ten. A group that ends in the state it began with adds none. A change made outside a tool group, such as `/todo clear`, still projects its own Todo List Snapshot right away. Each Todo List Snapshot starts with the header `Todo List state from the pi-todo extension (not a user message):` so it does not read like user input. Later requests keep earlier Todo List Snapshots in place. Clearing the list produces an explicit empty Todo List Snapshot. The system prompt and tool description do not change.
 
-After compaction, a fixed baseline immediately after the summary restores the state from before the retained Tail. Retained and newer mutations follow chronologically. This preserves previously written conversation cache prefixes between checkpoints; it does not guarantee provider cache hits.
+After compaction, a fixed baseline immediately after the summary restores the state from before the retained Tail. Retained and newer tool groups follow chronologically. This preserves previously written conversation cache prefixes between checkpoints; it does not guarantee provider cache hits.
 
 `todo` declares MCP-style `annotations`: not read-only, but non-destructive (it only appends to the session's own journal), not idempotent, and closed-world. Pi reports them through `pi.getAllTools()` for permission extensions and does not send them to model providers.
 
@@ -49,6 +49,6 @@ State is stored only in `pi-todo-state` session entries, without a second messag
 
 A failed journal write leaves acknowledged Todo state unchanged and disables Todo in that loaded session, including across `/reload`. Pi has an existing persistence limitation: subsequent entries can reference the unwritten entry, so reopening may omit earlier history from the selected branch even though it remains in the file. Todo does not repair session history; recovery is outside this cache-prefix change.
 
-Missing or ambiguous conversation anchors abort the request instead of silently relocating snapshots. Extensions that rewrite those anchors are unsupported.
+Missing or ambiguous conversation anchors abort the request instead of silently relocating Todo List Snapshots. Extensions that rewrite those anchors are unsupported.
 
 This is privileged extension code: review it before installing it into an agent that can access local files, tools, or credentials.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ian-pascoe/pi-todo",
-  "version": "0.2.0",
+  "version": "0.3.0",
   "private": false,
   "description": "Minimal session-native Todo List for Pi agents.",
   "keywords": [
@@ -50,7 +50,7 @@
     ]
   },
   "scripts": {
-    "test": "vitest run --config ../../vitest.config.ts --root .",
+    "test": "vitest run",
     "typecheck": "tsc --noEmit -p tsconfig.json"
   }
 }

package/src/todo-context.ts

@@ -14,6 +14,8 @@ import {
 } from "./todo-list.js";
 
 type Message = ContextEvent["messages"][number];
+/** Marks a snapshot as extension state so the model does not read it as user-authored. */
+const SNAPSHOT_HEADER = "Todo List state from the pi-todo extension (not a user message):";
 const ProjectionDetails = Type.Object(
   {
     version: Type.Literal(1),
@@ -50,7 +52,7 @@ function snapshotMessage(
     role: "custom",
     customType: "pi-todo-context",
     display: false,
-    content: `Todo List:\n${formatTodoList(state.tasks)}`,
+    content: `${SNAPSHOT_HEADER}\n${formatTodoList(state.tasks)}`,
     timestamp: Date.parse(entry.timestamp),
     details: { version: 1, stateEntryId: entry.id, checkpointId },
   };
@@ -110,27 +112,26 @@ export function projectTodoContext(
     }
   }
   const outstanding = new Map<string, number>();
-  let pending: Message[] = [];
+  // A tool group projects only its final state, once its last result has landed.
+  let pending: { entry: SessionEntry; state: TodoStateSnapshot } | undefined;
+  const project = (entry: SessionEntry, state: TodoStateSnapshot): void => {
+    const content = formatTodoList(state.tasks);
+    if (content !== previousContent && (previousContent !== undefined || state.tasks.length > 0))
+      insert(snapshotMessage(entry, state, null));
+    previousContent = content;
+  };
   for (const entry of branch.slice(start)) {
     if (entry === checkpoint) continue;
     const state = todoStateFromEntry(entry);
     if (state) {
-      const content = formatTodoList(state.tasks);
-      if (
-        content !== previousContent &&
-        (previousContent !== undefined || state.tasks.length > 0)
-      ) {
-        const snapshot = snapshotMessage(entry, state, null);
-        if (outstanding.size > 0) pending.push(snapshot);
-        else insert(snapshot);
-      }
-      previousContent = content;
+      if (outstanding.size > 0) pending = { entry, state };
+      else project(entry, state);
       continue;
     }
     const message = conversationMessage(entry);
     if (!message) continue;
     if (message.role === "assistant") {
-      if (outstanding.size > 0 && pending.length > 0)
+      if (outstanding.size > 0 && pending)
         throw new Error("Todo mutation has an incomplete tool group");
       outstanding.clear();
       for (const block of message.content)
@@ -142,12 +143,12 @@ export function projectTodoContext(
       else outstanding.delete(message.toolCallId);
     }
     anchor = message;
-    if (outstanding.size === 0 && pending.length > 0) {
-      for (const snapshot of pending) insert(snapshot);
-      pending = [];
+    if (outstanding.size === 0 && pending) {
+      project(pending.entry, pending.state);
+      pending = undefined;
     }
   }
-  if (pending.length > 0) throw new Error("Todo mutation has an incomplete tool group");
+  if (pending) throw new Error("Todo mutation has an incomplete tool group");
   return [
     ...(insertions.get(-1) ?? []),
     ...messages.flatMap((message, index) => [message, ...(insertions.get(index) ?? [])]),