@getpipher/armory-todo 0.1.0 → 0.2.0

package/src/todo-store.ts

@@ -1,8 +1,10 @@
 // Pure, pi-independent TODO store for armory-todo.
 //
-// A global, cross-session TODO list backed by a single JSON file on disk
-// (default ~/.pi/agent/todo.json; override with TODO_STORE_PATH for tests).
-// Deliberately NOT pi session-entries — this survives across all sessions.
+// A global, cross-session TODO list backed by a JSON file under TODO_DIR
+// (default ~/.pi/agent/todo/todo.json). The live store holds open, in_progress,
+// and parked todos; done/cancelled are moved to todo-archive.json by `prune`
+// (see archive.ts). v1 single-file stores at ~/.pi/agent/todo.json are
+// migrated into the folder on first load (see migrate.ts).
 //
 // Kept free of any pi/typebox imports so it can be unit-tested standalone.
 
@@ -14,18 +16,16 @@ import {
   renameSync,
   writeFileSync,
 } from "node:fs";
-import { dirname, join } from "node:path";
-import { homedir } from "node:os";
-
-const DEFAULT_PATH = join(homedir(), ".pi", "agent", "todo.json");
-const STORE_PATH = process.env.TODO_STORE_PATH || DEFAULT_PATH;
+import { dirname } from "node:path";
+import { getLivePath, getTodoDir, getLegacyPath } from "./paths.ts";
+import { migrateIfNeeded } from "./migrate.ts";
 
 export type Priority = "low" | "med" | "high" | "critical";
-export type Status = "open" | "in_progress" | "done" | "cancelled";
+export type Status = "open" | "in_progress" | "parked" | "done" | "cancelled";
 
 const PRIO_ORDER: Record<Priority, number> = { critical: 0, high: 1, med: 2, low: 3 };
 const PRIORITIES: Priority[] = ["low", "med", "high", "critical"];
-const STATUSES: Status[] = ["open", "in_progress", "done", "cancelled"];
+const STATUSES: Status[] = ["open", "in_progress", "parked", "done", "cancelled"];
 const PRIO_SET = new Set(PRIORITIES);
 const STATUS_SET = new Set(STATUSES);
 
@@ -43,7 +43,7 @@ export interface Todo {
 }
 
 export interface Store {
-  version: 1;
+  version: 2;
   updatedAt: string;
   todos: Todo[];
 }
@@ -68,6 +68,11 @@ export interface ListFilter {
   status?: Status | "all";
   project?: string;
   tag?: string;
+  text?: string;       // substring match on todo.text (case-insensitive)
+  since?: string;      // ISO date; filter createdAt >= since
+  before?: string;     // ISO date; filter createdAt < before
+  limit?: number;      // default 20
+  page?: number;       // default 1 (1-indexed)
 }
 
 export class TodoError extends Error {}
@@ -82,30 +87,37 @@ function genId(): string {
 }
 
 function emptyStore(): Store {
-  return { version: 1, updatedAt: now(), todos: [] };
+  return { version: 2, updatedAt: now(), todos: [] };
 }
 
 export function getStorePath(): string {
-  return STORE_PATH;
+  return getLivePath();
 }
 
-/** Load the store from disk. On corruption, back up the bad file and start fresh. */
+/** Load the live store from disk. Runs v1→v2 migration first — but ONLY when
+ *  using the default TODO_DIR (not overridden by TODO_DIR env, e.g. tests).
+ *  On corruption, backs up the bad file and starts fresh. */
 export function loadStore(): Store {
-  if (!existsSync(STORE_PATH)) return emptyStore();
+  if (!process.env.TODO_DIR) {
+    migrateIfNeeded({ todoDir: getTodoDir(), legacyPath: getLegacyPath() });
+  }
+  const path = getLivePath();
+  if (!existsSync(path)) return emptyStore();
   try {
-    const raw = readFileSync(STORE_PATH, "utf8");
+    const raw = readFileSync(path, "utf8");
     const parsed = JSON.parse(raw) as Store;
     if (!parsed || typeof parsed !== "object" || !Array.isArray(parsed.todos)) {
       throw new Error("invalid store shape");
     }
-    if (parsed.version !== 1) {
-      // Future: migrate. v1 only — reset on unknown version with backup.
-      throw new Error("unsupported store version: " + String(parsed.version));
+    if (parsed.version !== 2) {
+      // v1 → v2: accept it (the migration moved the file), just bump the version in memory.
+      // The data shape is otherwise identical; parked status is new but old todos won't have it.
+      parsed.version = 2;
     }
     return parsed;
   } catch {
     try {
-      renameSync(STORE_PATH, `${STORE_PATH}.bad-${Date.now()}`);
+      renameSync(path, `${path}.bad-${Date.now()}`);
     } catch {
       // best-effort backup; swallow
     }
@@ -116,16 +128,17 @@ export function loadStore(): Store {
 /** Atomic, 0600 write. */
 export function saveStore(store: Store): void {
   store.updatedAt = now();
-  const dir = dirname(STORE_PATH);
+  const path = getLivePath();
+  const dir = dirname(path);
   mkdirSync(dir, { recursive: true });
-  const tmp = `${STORE_PATH}.tmp`;
+  const tmp = `${path}.tmp`;
   writeFileSync(tmp, JSON.stringify(store, null, 2) + "\n", { encoding: "utf8", mode: 0o600 });
   try {
     chmodSync(tmp, 0o600);
   } catch {
     // some filesystems ignore mode bits; not fatal
   }
-  renameSync(tmp, STORE_PATH);
+  renameSync(tmp, path);
 }
 
 function assertPriority(p: unknown): asserts p is Priority {
@@ -180,7 +193,13 @@ export function listTodos(filter: ListFilter = {}): Todo[] {
   }
   if (filter.project) out = out.filter((t) => t.project === filter.project);
   if (filter.tag) out = out.filter((t) => t.tags.includes(filter.tag as string));
-  return out.slice().sort((a, b) => {
+  if (filter.text) {
+    const q = filter.text.toLowerCase();
+    out = out.filter((t) => t.text.toLowerCase().includes(q));
+  }
+  if (filter.since) out = out.filter((t) => t.createdAt >= (filter.since as string));
+  if (filter.before) out = out.filter((t) => t.createdAt < (filter.before as string));
+  const sorted = out.slice().sort((a, b) => {
     if (a.status !== b.status) {
       // in_progress before open (actionable ordering)
       return a.status === "in_progress" ? -1 : b.status === "in_progress" ? 1 : 0;
@@ -190,6 +209,10 @@ export function listTodos(filter: ListFilter = {}): Todo[] {
     }
     return a.createdAt.localeCompare(b.createdAt);
   });
+  const limit = filter.limit ?? 20;
+  const page = filter.page ?? 1;
+  const start = (page - 1) * limit;
+  return sorted.slice(start, start + limit);
 }
 
 export function updateTodo(id: string, patch: UpdateInput): Todo {
@@ -223,6 +246,10 @@ export function completeTodo(id: string): Todo {
   return updateTodo(id, { status: "done" });
 }
 
+export function parkTodo(id: string): Todo {
+  return updateTodo(id, { status: "parked" });
+}
+
 export function deleteTodo(id: string): Todo {
   return updateTodo(id, { status: "cancelled" });
 }