@gamflaz/opencode-todolist 1.0.12 → 1.1.0

package/dist/todos/todo-tree.d.ts

@@ -1,5 +1,5 @@
 import { type TodoStatus } from "../schemas/status.js";
-import type { TodoChildern, TodoRootItem } from "../schemas/todo";
+import type { TodoChildren, TodoRootItem } from "../schemas/todo";
 export interface TodoProgress {
     completed: number;
     total: number;
@@ -34,7 +34,7 @@ export declare class TodoTree {
     private static readonly BAR_UNFILLED;
     constructor(todos: readonly TodoRootItem[]);
     static isOpenStatus(status: TodoStatus): boolean;
-    static calculateProgress(children?: readonly TodoChildern[]): TodoProgress | null;
+    static calculateProgress(children?: readonly TodoChildren[]): TodoProgress | null;
     static renderProgressBar(percent: number, width?: number): {
         filled: string;
         unfilled: string;
@@ -56,7 +56,20 @@ export declare class TodoTree {
      * parent whose children are all completed.
      */
     private normalize;
-    render(): string;
+    /**
+     * Renders the tree.
+     *
+     * `options.visibleIds` narrows which rows are *printed* without touching the
+     * data, so a root keeps its real `(completed/total)` label even when only one
+     * subtask is listed. A scoped read needs exactly that: the root line shows
+     * `(2/2)` while a single subtask row is displayed.
+     *
+     * A root line is printed only when its own id is visible; a child line only
+     * when the child's id is visible. Omitting `visibleIds` prints everything.
+     */
+    render(options?: {
+        visibleIds?: readonly number[];
+    }): string;
     private static icon;
     /** Content with strike-through when completed, prefixed with its id unless it already starts with it. */
     private static renderLabel;

package/dist/todos/todo-tree.js

@@ -159,15 +159,33 @@ export class TodoTree {
         return { valid: true, data: normalized };
     }
     // Rendering
-    render() {
+    /**
+     * Renders the tree.
+     *
+     * `options.visibleIds` narrows which rows are *printed* without touching the
+     * data, so a root keeps its real `(completed/total)` label even when only one
+     * subtask is listed. A scoped read needs exactly that: the root line shows
+     * `(2/2)` while a single subtask row is displayed.
+     *
+     * A root line is printed only when its own id is visible; a child line only
+     * when the child's id is visible. Omitting `visibleIds` prints everything.
+     */
+    render(options) {
         const lines = [];
+        const visible = options?.visibleIds === undefined
+            ? undefined
+            : new Set(options.visibleIds);
         for (const parent of this.todos) {
             const progress = TodoTree.calculateProgress(parent.children);
             const progressText = progress ? ` (${progress.label})` : "";
-            lines.push(`${TodoTree.icon(parent.status)} ${TodoTree.renderLabel(parent, 0)}${progressText}`);
-            if (parent.note)
-                lines.push(`    ↳ note: ${parent.note}`);
+            if (!visible || visible.has(parent.id)) {
+                lines.push(`${TodoTree.icon(parent.status)} ${TodoTree.renderLabel(parent, 0)}${progressText}`);
+                if (parent.note)
+                    lines.push(`    ↳ note: ${parent.note}`);
+            }
             for (const child of parent.children ?? []) {
+                if (visible && !visible.has(child.id))
+                    continue;
                 lines.push(`    ${TodoTree.icon(child.status)} ${TodoTree.renderLabel(child, 1)}`);
                 if (child.note)
                     lines.push(`        ↳ note: ${child.note}`);

package/dist/todos/update.js

@@ -74,7 +74,7 @@ export function assertSequentialScope(todos, input) {
     const targetRootIndex = todos.indexOf(targetRoot);
     if (targetRootIndex !== firstOpen && !hasError(targetRoot)) {
         const blocking = todos[firstOpen];
-        throw new Error(`[manage_todo_list] root ${targetRoot.id} is out of order: finish root ${blocking.id} "${blocking.content}" first (or cancel it). Only a root in 'error' may be fixed early.`);
+        throw new Error(`root ${targetRoot.id} is out of order: finish root ${blocking.id} "${blocking.content}" first (or cancel it). Only a root in 'error' may be fixed early.`);
     }
     const target = targetRoot.id === input.id
         ? targetRoot
@@ -89,7 +89,7 @@ export function assertSequentialScope(todos, input) {
     if (input.status === "in_progress") {
         const running = activeRoot(todos);
         if (running && running.id !== targetRoot.id) {
-            throw new Error(`[manage_todo_list] root ${running.id} is still in_progress. Finish it (or cancel it) before starting ${targetRoot.id}; an error root may be fixed early only when nothing else is running.`);
+            throw new Error(`root ${running.id} is still in_progress. Finish it (or cancel it) before starting ${targetRoot.id}; an error root may be fixed early only when nothing else is running.`);
         }
         const busyChild = targetRoot.children?.find((child) => child.status === "in_progress" && child.id !== input.id);
         if (busyChild) {

package/dist/tui/todo-panel.js

@@ -11,11 +11,10 @@ import { mergeProps as _$mergeProps } from "@opentui/solid";
 import { createElement as _$createElement } from "@opentui/solid";
 import { mkdirSync, watch } from "node:fs";
 import { createEffect, createMemo, createSignal, For, onCleanup, Show } from "solid-js";
-import { supportsImageRendering } from "../icons/image-capabilities.js";
-import { SvgIcon } from "../icons/svg-icon.js";
-import { arrowDownSvgIcon, arrowRightSvgIcon, continuePromptSvgIcon, removeSvgIcon } from "../icons/svg-icons.js";
+import { Glyph } from "../icons/glyph.js";
+import { ARROW_DOWN, ARROW_RIGHT, CLEAR, CONTINUE } from "../icons/glyphs.js";
 import { StatusLabel } from "../schemas/status.js";
-import { getTodosDirectory, hashSessionID, readTodosSync, writeTodos } from "../storage/todo-store.js";
+import { getDataVersion, getTodosDirectory, readTodosSync, writeTodos } from "../storage/todo-store.js";
 import { getTodoDisplaySummary, hasOpenTodos } from "../todos/display.js";
 import { TodoTree } from "../todos/todo-tree.js";
 import { ERROR_PREFIX, ErrorRowWidth, RowWidth } from "./layout.js";
@@ -31,6 +30,15 @@ const SPINNER_FRAMES = ["⠋", "⠙", "⠹", "⠸", "⠼", "⠴", "⠦", "⠧",
 const SPINNER_INTERVAL_MS = 100;
 const WATCH_DEBOUNCE_MS = 50;
 
+/**
+ * Safety-net interval for cross-process change detection.
+ *
+ * `fs.watch` on the database usually fires first; this poll only exists for
+ * the cases where the filesystem event is missed or arrives late. One PRAGMA
+ * per second is negligible next to rendering the panel.
+ */
+const DATA_VERSION_POLL_MS = 1000;
+
 /**
  * Maximum number of todo rows the panel shows before it starts scrolling.
  * Keeping the panel compact protects the composer; scrolling guarantees no
@@ -337,47 +345,69 @@ function createHover() {
 }
 
 /**
- * Watches the todos directory and calls `onChange` (debounced) when a file
- * belonging to the current session changes. Cleans up with the owner scope.
+ * Notifies when the todo data changes, from either process.
+ *
+ * All sessions now live in one SQLite file, so there is no longer a per-session
+ * folder to watch. Two complementary signals are used:
+ *
+ * 1. `fs.watch` on the database file, debounced. Low latency, but SQLite writes
+ *    to the WAL first, so the event can arrive before the reader sees the new
+ *    rows, and some platforms are inconsistent about reporting it at all.
+ * 2. A poll of `PRAGMA data_version`, which changes exactly when *another*
+ *    connection commits. This is the safety net: if the filesystem event is
+ *    missed, the poll still notices within one interval.
+ *
+ * The poll is skipped entirely while the document is idle and nothing moved, so
+ * the steady-state cost is one cheap PRAGMA per interval.
  */
-function watchSessionTodos(sessionID, onChange) {
+function watchSessionTodos(onChange, readDataVersion) {
   let timer;
+  let interval;
   let watcher;
-  const hashed = createMemo(() => {
-    const sid = sessionID();
-    if (!sid) return "";
-    try {
-      return hashSessionID(sid);
-    } catch {
-      return "";
-    }
-  });
   const notify = () => {
     clearTimeout(timer);
     timer = setTimeout(onChange, WATCH_DEBOUNCE_MS);
   };
-  const belongsToSession = name => {
-    const sid = sessionID();
-    if (!sid) return true;
-    const hash = hashed();
-    return hash !== "" && name.startsWith(hash) || name.startsWith(sid) || name.startsWith(`todos-${sid}-`);
+
+  // Seed with the current value so the first poll compares against reality
+  // instead of firing a spurious refresh.
+  let lastVersion = Number.NaN;
+  const poll = () => {
+    try {
+      const version = readDataVersion();
+      if (Number.isNaN(lastVersion)) {
+        lastVersion = version;
+        return;
+      }
+      if (version !== lastVersion) {
+        lastVersion = version;
+        notify();
+      }
+    } catch {
+      // The database may be mid-migration; the next tick retries.
+    }
   };
   try {
     const baseDir = getTodosDirectory();
     mkdirSync(baseDir, {
       recursive: true
     });
+    // Watching the directory rather than the file itself: SQLite may replace
+    // the file on checkpoint, which would detach a file-level watcher.
     watcher = watch(baseDir, {
-      recursive: true,
       persistent: false
     }, (_, filename) => {
-      if (!filename || belongsToSession(String(filename))) notify();
+      const name = filename ? String(filename) : "";
+      if (!name || name.startsWith("todos.db")) notify();
     });
   } catch (err) {
     console.error("[todos] watcher initialization failed:", err);
   }
+  interval = setInterval(poll, DATA_VERSION_POLL_MS);
+  poll();
   onCleanup(() => {
     clearTimeout(timer);
+    if (interval) clearInterval(interval);
     watcher?.close();
   });
 }
@@ -395,12 +425,6 @@ export function TodoProgress(props) {
   const clearHover = createHover();
   const continueHover = createHover();
 
-  // Image capability
-  const [imageSupported, setImageSupported] = createSignal(supportsImageRendering(context.renderer?.capabilities));
-  const onCapabilities = caps => setImageSupported(supportsImageRendering(caps));
-  context.renderer.on("capabilities", onCapabilities);
-  onCleanup(() => context.renderer.off("capabilities", onCapabilities));
-
   // Running state
   const readRunning = () => {
     const sid = sessionID();
@@ -443,7 +467,7 @@ export function TodoProgress(props) {
   });
 
   // Todo data
-  watchSessionTodos(sessionID, () => setRefresh(v => v + 1));
+  watchSessionTodos(() => setRefresh(v => v + 1), getDataVersion);
   const summary = createMemo(() => {
     refresh();
     const sid = sessionID();
@@ -570,18 +594,13 @@ export function TodoProgress(props) {
       _$setProp(_el$18, "width", 2);
       _$setProp(_el$18, "height", 1);
       _$setProp(_el$18, "marginRight", 1);
+      _$setProp(_el$18, "marginLeft", 1);
       _$setProp(_el$18, "alignItems", "flex-start");
-      _$insert(_el$18, _$createComponent(SvgIcon, {
-        get source() {
-          return isCollapsed() ? arrowRightSvgIcon : arrowDownSvgIcon;
-        },
+      _$insert(_el$18, _$createComponent(Glyph, {
+        glyph: () => isCollapsed() ? ARROW_RIGHT : ARROW_DOWN,
         get color() {
           return theme().text.muted;
         },
-        get fallback() {
-          return isCollapsed() ? "▶" : "▼";
-        },
-        width: 2,
         height: 1
       }));
       _$insertNode(_el$19, _el$20);
@@ -642,14 +661,11 @@ export function TodoProgress(props) {
               void continueTodos();
             }
           });
-          _$insert(_el$24, _$createComponent(SvgIcon, {
-            source: continuePromptSvgIcon,
+          _$insert(_el$24, _$createComponent(Glyph, {
+            glyph: () => CONTINUE,
             get color() {
               return hoverColor(continueHover.hovered(), theme().text.action.primary.base);
-            },
-            fallback: "\u21BB",
-            width: 2,
-            height: 1
+            }
           }), _el$25);
           _$insertNode(_el$25, _$createTextNode(`Continue`));
           _$setProp(_el$25, "wrapMode", "none");
@@ -669,6 +685,7 @@ export function TodoProgress(props) {
           return _el$24;
         }
       }), _el$27);
+      _$setProp(_el$27, "marginRight", 1.5);
       _$setProp(_el$27, "flexDirection", "row");
       _$setProp(_el$27, "onMouseDown", e => {
         if (e.button === 0) {
@@ -676,14 +693,11 @@ export function TodoProgress(props) {
           void clearTodos();
         }
       });
-      _$insert(_el$27, _$createComponent(SvgIcon, {
+      _$insert(_el$27, _$createComponent(Glyph, {
+        glyph: () => CLEAR,
         get color() {
           return hoverColor(clearHover.hovered(), theme().text.feedback.error.base);
-        },
-        source: removeSvgIcon,
-        fallback: "\u2716",
-        height: 1,
-        width: 2
+        }
       }));
       _$insert(_el$15, _$createComponent(Show, {
         get when() {
@@ -709,32 +723,26 @@ export function TodoProgress(props) {
       }), null);
       _$effect(_p$ => {
         var _v$11 = theme().border.base,
-          _v$12 = imageSupported() ? 0 : 1,
-          _v$13 = {
+          _v$12 = {
             fg: theme().text.base
           },
-          _v$14 = {
+          _v$13 = {
             fg: theme().text.muted
           },
-          _v$15 = !imageSupported() ? 1.5 : 0,
-          _v$16 = clearHover.onMouseOver,
-          _v$17 = clearHover.onMouseOut;
+          _v$14 = clearHover.onMouseOver,
+          _v$15 = clearHover.onMouseOut;
         _v$11 !== _p$.e && (_p$.e = _$setProp(_el$15, "borderColor", _v$11, _p$.e));
-        _v$12 !== _p$.t && (_p$.t = _$setProp(_el$18, "marginLeft", _v$12, _p$.t));
-        _v$13 !== _p$.a && (_p$.a = _$setProp(_el$21, "style", _v$13, _p$.a));
-        _v$14 !== _p$.o && (_p$.o = _$setProp(_el$23, "style", _v$14, _p$.o));
-        _v$15 !== _p$.i && (_p$.i = _$setProp(_el$27, "marginRight", _v$15, _p$.i));
-        _v$16 !== _p$.n && (_p$.n = _$setProp(_el$27, "onMouseOver", _v$16, _p$.n));
-        _v$17 !== _p$.s && (_p$.s = _$setProp(_el$27, "onMouseOut", _v$17, _p$.s));
+        _v$12 !== _p$.t && (_p$.t = _$setProp(_el$21, "style", _v$12, _p$.t));
+        _v$13 !== _p$.a && (_p$.a = _$setProp(_el$23, "style", _v$13, _p$.a));
+        _v$14 !== _p$.o && (_p$.o = _$setProp(_el$27, "onMouseOver", _v$14, _p$.o));
+        _v$15 !== _p$.i && (_p$.i = _$setProp(_el$27, "onMouseOut", _v$15, _p$.i));
         return _p$;
       }, {
         e: undefined,
         t: undefined,
         a: undefined,
         o: undefined,
-        i: undefined,
-        n: undefined,
-        s: undefined
+        i: undefined
       });
       return _el$15;
     })()

package/drizzle/20261014050406_eminent_zaran/migration.sql

@@ -0,0 +1,49 @@
+CREATE TABLE `tag` (
+	`id` text PRIMARY KEY,
+	`name` text NOT NULL UNIQUE
+);
+--> statement-breakpoint
+CREATE TABLE `todo_item` (
+	`session_key` text NOT NULL,
+	`id` text NOT NULL,
+	`parent_id` text,
+	`content` text NOT NULL,
+	`status` text NOT NULL,
+	`priority` text,
+	`note` text,
+	`position` integer NOT NULL,
+	CONSTRAINT `todo_item_pk` PRIMARY KEY(`session_key`, `id`),
+	CONSTRAINT `fk_todo_item_session_key_todo_session_session_key_fk` FOREIGN KEY (`session_key`) REFERENCES `todo_session`(`session_key`) ON DELETE CASCADE,
+	CONSTRAINT `fk_todo_item_session_key_parent_id_todo_item_session_key_id_fk` FOREIGN KEY (`session_key`,`parent_id`) REFERENCES `todo_item`(`session_key`,`id`) ON DELETE CASCADE
+);
+--> statement-breakpoint
+CREATE TABLE `todo_item_dependency` (
+	`session_key` text NOT NULL,
+	`todo_id` text NOT NULL,
+	`depends_on_id` text NOT NULL,
+	CONSTRAINT `todo_item_dependency_pk` PRIMARY KEY(`session_key`, `todo_id`, `depends_on_id`),
+	CONSTRAINT `fk_todo_item_dependency_session_key_todo_id_todo_item_session_key_id_fk` FOREIGN KEY (`session_key`,`todo_id`) REFERENCES `todo_item`(`session_key`,`id`) ON DELETE CASCADE,
+	CONSTRAINT `fk_todo_item_dependency_session_key_depends_on_id_todo_item_session_key_id_fk` FOREIGN KEY (`session_key`,`depends_on_id`) REFERENCES `todo_item`(`session_key`,`id`) ON DELETE CASCADE
+);
+--> statement-breakpoint
+CREATE TABLE `todo_item_tag` (
+	`session_key` text NOT NULL,
+	`todo_id` text NOT NULL,
+	`tag_id` text NOT NULL,
+	CONSTRAINT `todo_item_tag_pk` PRIMARY KEY(`session_key`, `todo_id`, `tag_id`),
+	CONSTRAINT `fk_todo_item_tag_session_key_todo_id_todo_item_session_key_id_fk` FOREIGN KEY (`session_key`,`todo_id`) REFERENCES `todo_item`(`session_key`,`id`) ON DELETE CASCADE,
+	CONSTRAINT `fk_todo_item_tag_tag_id_tag_id_fk` FOREIGN KEY (`tag_id`) REFERENCES `tag`(`id`) ON DELETE CASCADE
+);
+--> statement-breakpoint
+CREATE TABLE `todo_session` (
+	`session_key` text PRIMARY KEY,
+	`revision` integer DEFAULT 0 NOT NULL,
+	`hint_revision` integer DEFAULT 0 NOT NULL,
+	`created_at` text NOT NULL,
+	`updated_at` text NOT NULL
+);
+--> statement-breakpoint
+CREATE INDEX `todo_item_session_position_idx` ON `todo_item` (`session_key`,`position`);--> statement-breakpoint
+CREATE UNIQUE INDEX `todo_item_dependency_pair_idx` ON `todo_item_dependency` (`session_key`,`todo_id`,`depends_on_id`);--> statement-breakpoint
+CREATE INDEX `todo_item_dependency_depends_on_idx` ON `todo_item_dependency` (`session_key`,`depends_on_id`);--> statement-breakpoint
+CREATE INDEX `todo_item_tag_tag_idx` ON `todo_item_tag` (`tag_id`);