@encoredigitalgroup/opencode 0.2.6 → 0.2.7

package/README.md

@@ -67,11 +67,17 @@ Each todo requires:
 - `status`: `pending`, `in_progress`, `completed`, or `cancelled`.
 - `priority`: `high`, `medium`, or `low`.
 
+Each tracker call replaces the complete ordered list. Include discovered work and current statuses in every replacement. Mark a todo completed only after
+verifying its result. Mark a todo cancelled only when the user explicitly cancels that work. Never clear a list or change a status to avoid tracker updates,
+tool restrictions, or difficult work.
+
 The checklist renders only in the OpenCode terminal TUI sidebar. After a restart, it appears only when the completed `todotracker` result is in the host's loaded
 message cache. That cache starts with the newest 20 messages, so older results are not shown until the host has loaded them.
 
-After a session's first successful `todotracker` call, the plugin requires another update at least once per minute. When an update is due, it blocks every other
-tool until the tracker is updated. A tracked subagent completing also requires its parent session to update the tracker before continuing. Enforcement is disabled
+Automatic refresh enforcement runs only while the latest valid list contains nonterminal work. After a session's first successful `todotracker` call, the plugin
+requires another update at least once per minute while that work remains. When an update is due, it blocks every other tool until the tracker is updated. A valid
+empty list or a list containing only `completed` and `cancelled` todos ends automatic refresh enforcement. A tracked subagent completing also requires its parent
+session to update the tracker before continuing. Malformed tracker results remain enforcement-active because completion was not verified. Enforcement is disabled
 for sessions that have never used `todotracker`, and can be disabled for all sessions with `"enforceTodoTracker": false`.
 
 When a root session is idle, the sidebar renders the final in-progress todo as completed only when every earlier todo is completed. It restores the original status

package/dist/lib/TodoTracker.d.ts

@@ -50,5 +50,7 @@ export declare function todosFromMessages(messages: readonly SessionMessageInfo[
 export declare class TodoTracker {
     static marker(status: TodoStatus): string;
     static sidebar(messages: readonly SessionMessageInfo[], completed?: boolean): readonly Todo[];
-    static register(context: ServerPlugin.Context, config: PluginConfig): Promise<import("@opencode/plugin/promise/registration").Registration>;
+    static register(context: ServerPlugin.Context, config: PluginConfig): Promise<{
+        dispose: () => Promise<void>;
+    }>;
 }

package/dist/server/index.js

@@ -192,6 +192,10 @@ var todoTrackerInput = {
 var todoTrackerName = "todotracker";
 var todoTrackerDescription = "Replace the current session's complete ordered todo list. " + "Pass every remaining todo on each call; pass an empty array to clear the list.";
 var trackerRefreshMessage = "Update todotracker with the complete current todo list, including newly discovered work and current statuses, before continuing tool work.";
+var todoTrackerContract = "Todotracker rules: Each call replaces the complete ordered todo list. Include discovered work and current statuses in every replacement. Mark a todo completed only after verifying its result. Mark a todo cancelled only when the user explicitly cancels that work. Never clear a list or change a status to avoid tracker updates, tool restrictions, or difficult work.";
+var compactionTrackerContract = `Retain this TodoTracker contract verbatim in the continuation summary:
+
+${todoTrackerContract}`;
 var todoStatuses = ["pending", "in_progress", "completed", "cancelled"];
 var todoPriorities = ["high", "medium", "low"];
 function isRecord(value) {
@@ -222,22 +226,34 @@ function isTodoTrackerInput(value) {
 function isTodoResultMetadata(value) {
   return isTodoTrackerInput(value);
 }
-function todosFromToolResult(metadata, content) {
+function parseTodoList(metadata, content) {
   if (isTodoResultMetadata(metadata)) {
-    return metadata.todos;
+    return { valid: true, todos: metadata.todos };
   }
-  for (const item of content) {
+  const contentItems = typeof content === "string" ? [{ type: "text", text: content }] : content;
+  for (const item of contentItems) {
     if (item.type !== "text" || item.text === undefined) {
       continue;
     }
     try {
       const todos = JSON.parse(item.text);
       if (isTodoList(todos)) {
-        return todos;
+        return { valid: true, todos };
       }
     } catch {}
   }
-  return [];
+  return { valid: false };
+}
+function todosFromToolResult(metadata, content) {
+  const parsed = parseTodoList(metadata, content);
+  return parsed.valid ? parsed.todos : [];
+}
+function isTerminalTodoList(todos) {
+  return todos.every((todo) => todo.status === "completed" || todo.status === "cancelled");
+}
+function stateFromTodoResult(metadata, content, refreshedAt = Date.now()) {
+  const parsed = parseTodoList(metadata, content);
+  return parsed.valid && isTerminalTodoList(parsed.todos) ? { kind: "untracked" } : refreshedTrackerState(refreshedAt);
 }
 function todosFromMessages(messages) {
   for (const message of messages.toReversed()) {
@@ -272,7 +288,7 @@ function trackerStateFromMessages(messages) {
         continue;
       }
       if (part.name === todoTrackerName) {
-        state = refreshedTrackerState(part.time?.completed ?? part.time?.ran ?? part.time?.created ?? message.time.completed ?? message.time.created);
+        state = stateFromTodoResult(part.state.metadata, part.state.content, part.time?.completed ?? part.time?.ran ?? part.time?.created ?? message.time.completed ?? message.time.created);
       }
     }
   }
@@ -282,6 +298,7 @@ function trackerStateFromMessages(messages) {
 class TodoTrackerEnforcement {
   context;
   states = new Map;
+  instructedSessionIDs = new Set;
   constructor(context) {
     this.context = context;
   }
@@ -300,49 +317,61 @@ class TodoTrackerEnforcement {
       state.refreshRequired = true;
     }
   }
-  async register() {
+  async register(enforceRefresh) {
     let before;
     let after;
     let context;
+    let compaction;
+    let observer;
+    const controller = new AbortController;
     try {
-      before = await this.context.tool.hook("execute.before", async (event) => {
-        const state = await this.stateFor(event.sessionID);
-        if (requiresTrackerRefresh(state, event.tool)) {
-          throw new Error(trackerRefreshMessage);
+      context = await this.context.session.hook("context", async (event) => {
+        if (!this.instructedSessionIDs.has(event.sessionID)) {
+          this.instructedSessionIDs.add(event.sessionID);
+          event.system.push({ type: "text", text: todoTrackerContract });
         }
-      });
-      after = await this.context.tool.hook("execute.after", async (event) => {
-        if (event.status !== "completed") {
+        if (!enforceRefresh) {
           return;
         }
-        if (event.tool === todoTrackerName) {
-          this.states.set(event.sessionID, refreshedTrackerState());
-        }
-      });
-      context = await this.context.session.hook("context", async (event) => {
-        const state = await this.stateFor(event.sessionID);
-        if (trackerRefreshDue(state)) {
+        if (trackerRefreshDue(await this.stateFor(event.sessionID))) {
           event.system.push({ type: "text", text: trackerRefreshMessage });
         }
       });
+      compaction = await this.context.session.hook("compaction", async (event) => {
+        event.system.push({ type: "text", text: compactionTrackerContract });
+      });
+      if (enforceRefresh) {
+        before = await this.context.tool.hook("execute.before", async (event) => {
+          if (requiresTrackerRefresh(await this.stateFor(event.sessionID), event.tool)) {
+            throw new Error(trackerRefreshMessage);
+          }
+        });
+        after = await this.context.tool.hook("execute.after", async (event) => {
+          if (event.status === "completed" && event.tool === todoTrackerName) {
+            this.states.set(event.sessionID, stateFromTodoResult(event.result.metadata, event.result.content ?? []));
+          }
+        });
+        observer = this.observeSubagentCompletion(controller.signal);
+      }
       const beforeRegistration = before;
       const afterRegistration = after;
       const contextRegistration = context;
-      const controller = new AbortController;
-      const observer = this.observeSubagentCompletion(controller.signal);
+      const compactionRegistration = compaction;
       return {
         dispose: async () => {
           controller.abort();
           await Promise.all([
-            beforeRegistration.dispose(),
-            afterRegistration.dispose(),
+            beforeRegistration?.dispose(),
+            afterRegistration?.dispose(),
             contextRegistration.dispose(),
+            compactionRegistration.dispose(),
             observer
           ]);
         }
       };
     } catch (error) {
-      await Promise.all([before?.dispose(), after?.dispose(), context?.dispose()]);
+      controller.abort();
+      await Promise.all([before?.dispose(), after?.dispose(), context?.dispose(), compaction?.dispose(), observer]);
       throw error;
     }
   }
@@ -401,11 +430,8 @@ class TodoTracker {
         }
       });
     });
-    if (!config.enforceTodoTracker) {
-      return tool;
-    }
     try {
-      const enforcement = await new TodoTrackerEnforcement(context).register();
+      const enforcement = await new TodoTrackerEnforcement(context).register(config.enforceTodoTracker);
       return {
         dispose: async () => {
           try {

package/dist/tui/index.js

@@ -977,6 +977,10 @@ var todoTrackerInput = {
 var todoTrackerName = "todotracker";
 var todoTrackerDescription = "Replace the current session's complete ordered todo list. " + "Pass every remaining todo on each call; pass an empty array to clear the list.";
 var trackerRefreshMessage = "Update todotracker with the complete current todo list, including newly discovered work and current statuses, before continuing tool work.";
+var todoTrackerContract = "Todotracker rules: Each call replaces the complete ordered todo list. Include discovered work and current statuses in every replacement. Mark a todo completed only after verifying its result. Mark a todo cancelled only when the user explicitly cancels that work. Never clear a list or change a status to avoid tracker updates, tool restrictions, or difficult work.";
+var compactionTrackerContract = `Retain this TodoTracker contract verbatim in the continuation summary:
+
+${todoTrackerContract}`;
 var todoStatuses = ["pending", "in_progress", "completed", "cancelled"];
 var todoPriorities = ["high", "medium", "low"];
 function isRecord(value) {
@@ -1007,22 +1011,34 @@ function isTodoTrackerInput(value) {
 function isTodoResultMetadata(value) {
   return isTodoTrackerInput(value);
 }
-function todosFromToolResult(metadata, content) {
+function parseTodoList(metadata, content) {
   if (isTodoResultMetadata(metadata)) {
-    return metadata.todos;
+    return { valid: true, todos: metadata.todos };
   }
-  for (const item of content) {
+  const contentItems = typeof content === "string" ? [{ type: "text", text: content }] : content;
+  for (const item of contentItems) {
     if (item.type !== "text" || item.text === undefined) {
       continue;
     }
     try {
       const todos = JSON.parse(item.text);
       if (isTodoList(todos)) {
-        return todos;
+        return { valid: true, todos };
       }
     } catch {}
   }
-  return [];
+  return { valid: false };
+}
+function todosFromToolResult(metadata, content) {
+  const parsed = parseTodoList(metadata, content);
+  return parsed.valid ? parsed.todos : [];
+}
+function isTerminalTodoList(todos) {
+  return todos.every((todo) => todo.status === "completed" || todo.status === "cancelled");
+}
+function stateFromTodoResult(metadata, content, refreshedAt = Date.now()) {
+  const parsed = parseTodoList(metadata, content);
+  return parsed.valid && isTerminalTodoList(parsed.todos) ? { kind: "untracked" } : refreshedTrackerState(refreshedAt);
 }
 function todosFromMessages(messages) {
   for (const message of messages.toReversed()) {
@@ -1057,7 +1073,7 @@ function trackerStateFromMessages(messages) {
         continue;
       }
       if (part.name === todoTrackerName) {
-        state = refreshedTrackerState(part.time?.completed ?? part.time?.ran ?? part.time?.created ?? message.time.completed ?? message.time.created);
+        state = stateFromTodoResult(part.state.metadata, part.state.content, part.time?.completed ?? part.time?.ran ?? part.time?.created ?? message.time.completed ?? message.time.created);
       }
     }
   }
@@ -1067,6 +1083,7 @@ function trackerStateFromMessages(messages) {
 class TodoTrackerEnforcement {
   context;
   states = new Map;
+  instructedSessionIDs = new Set;
   constructor(context) {
     this.context = context;
   }
@@ -1085,49 +1102,61 @@ class TodoTrackerEnforcement {
       state.refreshRequired = true;
     }
   }
-  async register() {
+  async register(enforceRefresh) {
     let before;
     let after;
     let context;
+    let compaction;
+    let observer;
+    const controller = new AbortController;
     try {
-      before = await this.context.tool.hook("execute.before", async (event) => {
-        const state = await this.stateFor(event.sessionID);
-        if (requiresTrackerRefresh(state, event.tool)) {
-          throw new Error(trackerRefreshMessage);
+      context = await this.context.session.hook("context", async (event) => {
+        if (!this.instructedSessionIDs.has(event.sessionID)) {
+          this.instructedSessionIDs.add(event.sessionID);
+          event.system.push({ type: "text", text: todoTrackerContract });
         }
-      });
-      after = await this.context.tool.hook("execute.after", async (event) => {
-        if (event.status !== "completed") {
+        if (!enforceRefresh) {
           return;
         }
-        if (event.tool === todoTrackerName) {
-          this.states.set(event.sessionID, refreshedTrackerState());
-        }
-      });
-      context = await this.context.session.hook("context", async (event) => {
-        const state = await this.stateFor(event.sessionID);
-        if (trackerRefreshDue(state)) {
+        if (trackerRefreshDue(await this.stateFor(event.sessionID))) {
           event.system.push({ type: "text", text: trackerRefreshMessage });
         }
       });
+      compaction = await this.context.session.hook("compaction", async (event) => {
+        event.system.push({ type: "text", text: compactionTrackerContract });
+      });
+      if (enforceRefresh) {
+        before = await this.context.tool.hook("execute.before", async (event) => {
+          if (requiresTrackerRefresh(await this.stateFor(event.sessionID), event.tool)) {
+            throw new Error(trackerRefreshMessage);
+          }
+        });
+        after = await this.context.tool.hook("execute.after", async (event) => {
+          if (event.status === "completed" && event.tool === todoTrackerName) {
+            this.states.set(event.sessionID, stateFromTodoResult(event.result.metadata, event.result.content ?? []));
+          }
+        });
+        observer = this.observeSubagentCompletion(controller.signal);
+      }
       const beforeRegistration = before;
       const afterRegistration = after;
       const contextRegistration = context;
-      const controller = new AbortController;
-      const observer = this.observeSubagentCompletion(controller.signal);
+      const compactionRegistration = compaction;
       return {
         dispose: async () => {
           controller.abort();
           await Promise.all([
-            beforeRegistration.dispose(),
-            afterRegistration.dispose(),
+            beforeRegistration?.dispose(),
+            afterRegistration?.dispose(),
             contextRegistration.dispose(),
+            compactionRegistration.dispose(),
             observer
           ]);
         }
       };
     } catch (error) {
-      await Promise.all([before?.dispose(), after?.dispose(), context?.dispose()]);
+      controller.abort();
+      await Promise.all([before?.dispose(), after?.dispose(), context?.dispose(), compaction?.dispose(), observer]);
       throw error;
     }
   }
@@ -1186,11 +1215,8 @@ class TodoTracker {
         }
       });
     });
-    if (!config.enforceTodoTracker) {
-      return tool;
-    }
     try {
-      const enforcement = await new TodoTrackerEnforcement(context).register();
+      const enforcement = await new TodoTrackerEnforcement(context).register(config.enforceTodoTracker);
       return {
         dispose: async () => {
           try {

package/package.json

@@ -1,7 +1,7 @@
 {
     "name": "@encoredigitalgroup/opencode",
     "type": "module",
-    "version": "0.2.6",
+    "version": "0.2.7",
     "description": "OpenCode file-picker commands for planning skills.",
     "publishConfig": {
         "access": "public"