@nguyenquangthai/pi-todo 0.3.3 → 0.3.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nguyenquangthai/pi-todo",
-  "version": "0.3.3",
+  "version": "0.3.4",
   "description": "OpenCode-like session todo checklist for the pi coding agent — todowrite/todoread with a live TUI overlay",
   "type": "module",
   "author": "QuangThai",

package/src/format.ts

@@ -1,8 +1,8 @@
 import type { Theme } from "@earendil-works/pi-coding-agent";
 import { truncateToWidth } from "@earendil-works/pi-tui";
-import type { TodoItem, TodoStatus } from "./types.js";
-import { MAX_OVERLAY_LINES, MAX_RESULT_LINES } from "./types.js";
-import { countCompletedTodos, countOpenTodos, countRunningTodos, hasOpenTodos } from "./validate.js";
+import type { TodoItem, TodoStatus } from "./types.js";
+import { MAX_OVERLAY_LINES, MAX_RESULT_LINES } from "./types.js";
+import { countCompletedTodos, countOpenTodos, countRunningTodos, hasOpenTodos } from "./validate.js";
 
 
 export function getTodoMarker(status: TodoStatus): string {
@@ -19,22 +19,23 @@ export function getTodoMarker(status: TodoStatus): string {
 }
 
 export function formatPlainTodoLine(todo: TodoItem): string {
-  return `${getTodoMarker(todo.status)} ${todo.content}`;
+  const prefix = todo.id ? `${todo.id} ` : "";
+  return `${getTodoMarker(todo.status)} ${prefix}${todo.content}`;
 }
 
 /** Compact checklist for tool responses; caps lines to keep LLM context small. */
 export function formatTodoListText(todos: readonly TodoItem[], summary: string): string {
   if (todos.length === 0) return summary;
 
-  // The list is a workflow timeline. Never reshuffle completed work after the
-  // next task just because its status changed.
-  const ordered = [...todos];
+  // The list is a workflow timeline. Never reshuffle completed work after the
+  // next task just because its status changed.
+  const ordered = [...todos];
 
   if (todos.length <= MAX_RESULT_LINES) {
-    return [summary, ...ordered.map(formatPlainTodoLine)].join("\n");
+    return [summary, ...ordered.map(formatPlainTodoLine)].join("\n");
   }
-  const shown = ordered.slice(0, MAX_RESULT_LINES);
-  const hidden = ordered.length - MAX_RESULT_LINES;
+  const shown = ordered.slice(0, MAX_RESULT_LINES);
+  const hidden = ordered.length - MAX_RESULT_LINES;
   return [
     summary,
     ...shown.map(formatPlainTodoLine),
@@ -58,58 +59,58 @@ export function shouldShowOverlay(todos: readonly TodoItem[]): boolean {
   return hasOpenTodos(todos);
 }
 
-export interface OverlayLayout {
-  visible: TodoItem[];
-  /** Active item repeated below the timeline when it is outside the visible prefix. */
-  pinnedActive?: TodoItem;
-  hiddenCount: number;
-  /** Retained for consumers of the layout API; terminal items are no longer regrouped. */
-  terminalCount: number;
-}
-
-/**
- * Fit the checklist timeline into the overlay without sorting by status or
- * priority. When the active task falls outside the visible prefix, repeat it
- * as a pinned "Active:" row rather than moving it ahead of earlier work.
- */
+export interface OverlayLayout {
+  visible: TodoItem[];
+  /** Active item repeated below the timeline when it is outside the visible prefix. */
+  pinnedActive?: TodoItem;
+  hiddenCount: number;
+  /** Retained for consumers of the layout API; terminal items are no longer regrouped. */
+  terminalCount: number;
+}
+
+/**
+ * Fit the checklist timeline into the overlay without sorting by status or
+ * priority. When the active task falls outside the visible prefix, repeat it
+ * as a pinned "Active:" row rather than moving it ahead of earlier work.
+ */
 export function selectOverlayLayout(
   todos: readonly TodoItem[],
-  maxLines: number = MAX_OVERLAY_LINES,
-): OverlayLayout {
-  if (!shouldShowOverlay(todos)) {
-    return { visible: [], hiddenCount: 0, terminalCount: 0 };
+  maxLines: number = MAX_OVERLAY_LINES,
+): OverlayLayout {
+  if (!shouldShowOverlay(todos)) {
+    return { visible: [], hiddenCount: 0, terminalCount: 0 };
+  }
+
+  const bodyBudget = Math.max(1, maxLines - 1);
+  if (todos.length <= bodyBudget) {
+    // All fit — show the canonical checklist sequence unchanged.
+    return { visible: [...todos], hiddenCount: 0, terminalCount: 0 };
   }
 
-  const bodyBudget = Math.max(1, maxLines - 1);
-  if (todos.length <= bodyBudget) {
-    // All fit — show the canonical checklist sequence unchanged.
-    return { visible: [...todos], hiddenCount: 0, terminalCount: 0 };
-  }
-
-  const active = todos.find((todo) => todo.status === "in_progress");
-  // Reserve one row for the overflow summary. If the active task lies outside
-  // the timeline prefix, reserve another row to pin it without reordering.
-  let visibleCapacity = Math.max(0, bodyBudget - 1);
-  let visible = todos.slice(0, visibleCapacity);
-  let pinnedActive = active && !visible.includes(active) ? active : undefined;
-
-  if (pinnedActive) {
-    visibleCapacity = Math.max(0, bodyBudget - 2);
-    visible = todos.slice(0, visibleCapacity);
-    pinnedActive = active;
-  }
-
-  const hiddenCount = todos.length - visible.length - (pinnedActive ? 1 : 0);
-  return { visible, pinnedActive, hiddenCount, terminalCount: 0 };
+  const active = todos.find((todo) => todo.status === "in_progress");
+  // Reserve one row for the overflow summary. If the active task lies outside
+  // the timeline prefix, reserve another row to pin it without reordering.
+  let visibleCapacity = Math.max(0, bodyBudget - 1);
+  let visible = todos.slice(0, visibleCapacity);
+  let pinnedActive = active && !visible.includes(active) ? active : undefined;
+
+  if (pinnedActive) {
+    visibleCapacity = Math.max(0, bodyBudget - 2);
+    visible = todos.slice(0, visibleCapacity);
+    pinnedActive = active;
+  }
+
+  const hiddenCount = todos.length - visible.length - (pinnedActive ? 1 : 0);
+  return { visible, pinnedActive, hiddenCount, terminalCount: 0 };
 }
 
 export interface RenderOverlayOptions {
   maxLines?: number;
 }
 
-/**
- * The overlay is hidden when no work remains open.
- */
+/**
+ * The overlay is hidden when no work remains open.
+ */
 export function renderOverlayLines(
   todos: readonly TodoItem[],
   theme: Theme,
@@ -118,33 +119,33 @@ export function renderOverlayLines(
 ): string[] {
   if (!shouldShowOverlay(todos)) return [];
 
-  const maxLines = Math.max(1, options.maxLines ?? MAX_OVERLAY_LINES);
-  const truncate = (line: string) => truncateToWidth(line, width, "…");
-  const open = countOpenTodos(todos);
-  const running = countRunningTodos(todos);
-  const completed = countCompletedTodos(todos);
-  const heading = truncate(
-    theme.fg("accent", theme.bold("# Todos")) +
-      theme.fg("dim", ` (${open} open, ${running} running, ${completed} completed)`),
-  );
-
+  const maxLines = Math.max(1, options.maxLines ?? MAX_OVERLAY_LINES);
+  const truncate = (line: string) => truncateToWidth(line, width, "…");
+  const open = countOpenTodos(todos);
+  const running = countRunningTodos(todos);
+  const completed = countCompletedTodos(todos);
+  const heading = truncate(
+    theme.fg("accent", theme.bold("# Todos")) +
+      theme.fg("dim", ` (${open} open, ${running} running, ${completed} completed)`),
+  );
+
   // Small gap between heading and first row — budget -1 to account for the blank line
   const layout = selectOverlayLayout(todos, Math.max(3, maxLines - 1));
   const lines: string[] = [heading, ""];
 
-  for (const todo of layout.visible) {
-    lines.push(truncate(formatThemedTodoLine(todo, theme)));
-  }
-  if (layout.pinnedActive) {
-    lines.push(
-      truncate(theme.fg("warning", `Active: ${formatPlainTodoLine(layout.pinnedActive)}`)),
-    );
-  }
+  for (const todo of layout.visible) {
+    lines.push(truncate(formatThemedTodoLine(todo, theme)));
+  }
+  if (layout.pinnedActive) {
+    lines.push(
+      truncate(theme.fg("warning", `Active: ${formatPlainTodoLine(layout.pinnedActive)}`)),
+    );
+  }
   if (layout.hiddenCount > 0) {
     lines.push(truncate(theme.fg("dim", `+${layout.hiddenCount} more`)));
   }
-  lines.push("");
-  // Layout reserves content rows, but the heading and trailing spacer are
-  // rendered here. Enforce the public maxLines contract at the final boundary.
-  return lines.slice(0, maxLines);
-}
+  lines.push("");
+  // Layout reserves content rows, but the heading and trailing spacer are
+  // rendered here. Enforce the public maxLines contract at the final boundary.
+  return lines.slice(0, maxLines);
+}

package/src/tools/todoupdate.ts

@@ -1,16 +1,18 @@
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { Text } from "@earendil-works/pi-tui";
+import { formatTodoListText } from "../format.js";
 import { TodoUpdateParams } from "../schema.js";
 import { getTodos, setTodos, withStoreLock } from "../store.js";
 import type { TodoWriteDetails } from "../types.js";
 import { TODO_STATE_ENTRY_TYPE, TOOL_UPDATE } from "../types.js";
-import { ensureTodoIds, validateTodoUpdate } from "../validate.js";
+import { countOpenTodos, ensureTodoIds, validateTodoUpdate } from "../validate.js";
 
 export function registerTodoUpdateTool(pi: ExtensionAPI, options: { onCommit?: () => void }): void {
   pi.registerTool({
     name: TOOL_UPDATE,
     label: "Todo Update",
-    description: "Patch existing todos by stable ID. Use todo_read first to obtain IDs. This never deletes items.",
+    description: "Patch existing todos by stable ID without replacing the full list or changing its order. id is required, must be a non-empty string, and must match a current todo; use todo_read first to obtain it. This tool never deletes items.\n\nNote: todo_write can reassign IDs for items whose content/status/priority changed. Always call todo_read before todo_update to get the current IDs.",
+    promptGuidelines: ["Always call todo_read before todo_update to obtain current IDs; IDs can change after todo_write if the item\u2019s content, status, or priority was modified."],
     promptSnippet: "Patch one or more existing todos by ID without replacing the full list",
     parameters: TodoUpdateParams,
     async execute(_toolCallId, params) {
@@ -30,7 +32,11 @@ export function registerTodoUpdateTool(pi: ExtensionAPI, options: { onCommit?: (
         }
         setTodos(todos);
         options.onCommit?.();
-        return { content: [{ type: "text", text: result.unchanged ? "No change" : `Updated ${params.updates.length} todo(s)` }], details: { todos, ...(result.unchanged ? { unchanged: true } : {}) } as TodoWriteDetails };
+        const open = countOpenTodos(todos);
+        const text = result.unchanged
+          ? "No change"
+          : `Updated ${params.updates.length} todo(s)\n\n${formatTodoListText(todos, `${open} open / ${todos.length} total`)}`;
+        return { content: [{ type: "text", text }], details: { todos, ...(result.unchanged ? { unchanged: true } : {}) } as TodoWriteDetails };
       });
     },
     renderCall(args, theme) { return new Text(theme.fg("toolTitle", theme.bold("todo_update ")) + theme.fg("accent", `${Array.isArray(args.updates) ? args.updates.length : 0} patch(es)`), 0, 0); },