@nguyenquangthai/pi-todo 0.3.2 → 0.3.4

package/CHANGELOG.md

@@ -1,5 +1,27 @@
 # Changelog
 
+## 0.3.3 (2026-07-16)
+
+### Changed
+
+- **Overlay heading**: Shows open, running, and completed counts. Completed
+  counts include only `completed` todos; `cancelled` todos remain excluded.
+- **Simplified status display**: Removed the ANSI progress bar and `done/total`
+  counter from the overlay.
+- **Documentation and screenshot**: Updated the README, screenshot PNG, and
+  screenshot HTML source to match the new heading.
+
+## 0.3.2 (2026-07-16)
+
+### Fixed
+
+- **Todo ID integrity**: Prevented id-less todos from taking IDs explicitly
+  retained by another todo during a full `todo_write` replacement.
+- **Diagnostics**: Added ID-integrity reporting and `repair_needed` status to
+  `todo_diagnose`.
+- **Mutation bounds**: Capped todo mutations and persisted snapshots at 200
+  items.
+
 ## 0.3.1 (2026-07-15)
 
 ### Fixed

package/README.md

@@ -81,12 +81,12 @@ Shown above the editor while any **open** todo remains (`pending` / `in_progress
 
 Hidden when the list is empty or every item is `completed` / `cancelled`.
 
-Heading shows open + running counts + background-color progress bar, e.g. `# Todos (3 open, 1 running) ▓▓▓▓░░░░ 1/4`:
-
-- **open** = `pending` + `in_progress`
-- **running** = `in_progress` only (0 or 1 after a valid write)
-- **progress bar** = ANSI background color via reverse video, using theme `accent` (filled) and `muted` (empty)
-
+Heading shows open, running, and completed counts, e.g. `# Todos (3 open, 1 running, 1 completed)`:
+
+- **open** = `pending` + `in_progress`
+- **running** = `in_progress` only (0 or 1 after a valid write)
+- **completed** = `completed` only; `cancelled` todos are not counted
+
 Items always stay in the array's workflow order; status changes only their marker/color.
 When space is tight, the overlay shows the earliest checklist items and `+N more`. If the active item is outside that prefix, it is repeated as `Active: [•] …` rather than moved ahead of earlier work.
 A blank line separates the heading from the first todo row for visual breathing room.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nguyenquangthai/pi-todo",
-  "version": "0.3.2",
+  "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 { 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,49 +59,49 @@ 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 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 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 };
 }
 
 export interface RenderOverlayOptions {
@@ -108,11 +109,7 @@ export interface RenderOverlayOptions {
 }
 
 /**
- * Heading counts (when overlay is visible):
- * - open    = pending + in_progress
- * - running = in_progress only (0 or 1 after a valid todo_write)
- * Overlay is hidden when open === 0, so "(0 open, 0 running)" is never shown.
- * Includes a background-color progress bar using theme colors.
+ * The overlay is hidden when no work remains open.
  */
 export function renderOverlayLines(
   todos: readonly TodoItem[],
@@ -122,45 +119,33 @@ export function renderOverlayLines(
 ): string[] {
   if (!shouldShowOverlay(todos)) return [];
 
-  const maxLines = Math.max(1, options.maxLines ?? MAX_OVERLAY_LINES);
+  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);
-
-  // Background-color progress bar via reverse video — uses theme accent/dim
-  const total = todos.length;
-  const done = total - open;
-  const barWidth = Math.max(4, Math.min(10, Math.floor(width / 8)));
-  const filled = Math.round((barWidth * done) / total);
-  const REV = "\x1b[7m";
-  const filledBar = REV + theme.fg("accent", " ".repeat(filled));
-  const emptyBar =
-    filled < barWidth ? REV + theme.fg("muted", " ".repeat(barWidth - filled)) : "";
-  const bar = filledBar + emptyBar + "\x1b[0m";
-
+  const completed = countCompletedTodos(todos);
   const heading = truncate(
-    theme.fg("accent", theme.bold(`# Todos`)) +
-      theme.fg("dim", ` (${open} open, ${running} running)`) +
-      ` ${bar}` + theme.fg("dim", ` ${done}/${total}`),
+    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); },

package/src/validate.ts

@@ -228,3 +228,8 @@ export function countOpenTodos(todos: readonly TodoItem[]): number {
 export function countRunningTodos(todos: readonly TodoItem[]): number {
   return todos.filter((t) => t.status === "in_progress").length;
 }
+
+/** Count successfully finished items; cancelled items are intentionally excluded. */
+export function countCompletedTodos(todos: readonly TodoItem[]): number {
+  return todos.filter((todo) => todo.status === "completed").length;
+}