pi-task-manager 0.1.0 → 0.1.1

package/package.json

@@ -1,28 +1,53 @@
 {
-	"name": "pi-task-manager",
-	"version": "0.1.0",
-	"description": "Task/todo management extension for pi",
-	"keywords": ["pi-package"],
-	"repository": {
-		"type": "git",
-		"url": "git+https://github.com/gilgil/pi-task-manager.git"
-	},
-	"type": "module",
-	"pi": {
-		"extensions": ["./index.ts"],
-		"skills": ["./skills"]
-	},
-	"scripts": {
-		"test": "node --test tests/*.test.ts",
-		"typecheck": "tsc --noEmit"
-	},
-	"peerDependencies": {
-		"@earendil-works/pi-ai": "*",
-		"@earendil-works/pi-coding-agent": "*",
-		"typebox": "*"
-	},
-	"devDependencies": {
-		"@types/node": "^22",
-		"typescript": "^5"
-	}
+  "name": "pi-task-manager",
+  "version": "0.1.1",
+  "description": "Hierarchical task/todo management extension for Pi. Tasks live in a TODO.md file as an indented tree.",
+  "keywords": [
+    "pi-package",
+    "pi",
+    "extension",
+    "task",
+    "todo",
+    "task-manager",
+    "hierarchical"
+  ],
+  "author": "Gil <gil.assayag@gmail.com>",
+  "license": "MIT",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/gilgil/pi-task-manager.git"
+  },
+  "homepage": "https://github.com/gilgil/pi-task-manager#readme",
+  "bugs": {
+    "url": "https://github.com/gilgil/pi-task-manager/issues"
+  },
+  "type": "module",
+  "files": [
+    "index.ts",
+    "lib",
+    "skills",
+    "README.md",
+    "LICENSE"
+  ],
+  "pi": {
+    "extensions": [
+      "./index.ts"
+    ],
+    "skills": [
+      "./skills"
+    ]
+  },
+  "scripts": {
+    "test": "node --test tests/*.test.ts",
+    "typecheck": "tsc --noEmit"
+  },
+  "peerDependencies": {
+    "@earendil-works/pi-ai": "*",
+    "@earendil-works/pi-coding-agent": "*",
+    "typebox": "*"
+  },
+  "devDependencies": {
+    "@types/node": "^22",
+    "typescript": "^5"
+  }
 }

package/PLAN.md

@@ -1,106 +0,0 @@
-# pi-task-manager — Plan
-
-A pi extension that provides task/todo management via custom tools, with the task logic ported from Python to TypeScript.
-
-## Architecture
-
-**TypeScript extension** that registers tools with TypeBox schemas. The task management logic (parsing, editing, saving) is ported from `task_manager.py` to TypeScript — no Python subprocess needed.
-
-### Why migrate to TypeScript instead of wrapping Python?
-
-- **No subprocess overhead** — no spawn, no JSON serialization, no process lifecycle management
-- **Native pi integration** — runs in the same process, shares `ctx`, `signal`, etc.
-- **Simpler debugging** — one language, one runtime, one stack trace
-- **Better state management** — the singleton `_manager` pattern works naturally in TypeScript (no per-call process restart)
-- **No Python dependency** — users don't need Python installed
-
-## Project Structure
-
-```
-pi-task-manager/
-├── package.json            # pi package manifest + dependencies
-├── README.md               # User-facing docs
-├── PLAN.md                 # This file
-├── extension/
-│   ├── index.ts            # Extension entry: register tools, commands
-│   └── lib/
-│       ├── task-manager.ts # Core TaskManager class (ported from Python)
-│       ├── task.ts         # Task class / dataclass equivalent
-│       ├── parser.ts       # Markdown line parser (ported from _parse_task_line)
-│       └── tools.ts        # Tool definitions with TypeBox schemas
-└── skills/
-    └── task-manager/
-        └── SKILL.md        # Usage instructions for the LLM
-```
-
-## Porting from Python
-
-### Source files to reference
-
-- `~/agent/tools/task_manager.py` — Main implementation (TaskManager class, Task dataclass, parsing)
-- `~/agent/tools/task_manager_design.md` — Design rationale
-- `~/agent/tools/tasks_emojis_format.md` — Emoji conventions for tasks
-
-### Key Python constructs to port
-
-1. **Task dataclass** → TypeScript class/interface with fields:
-   - `id`, `text`, `completed`, `emoji`, `category`, `priority`, `project`, `context`, `tags`, `due`, `created`, `modified`, `indent`
-
-2. **TaskManager class** → TypeScript class with methods:
-   - `openFile(path)`, `addTask()`, `editTask()`, `moveTask()`, `getTask()`, `listTasks()`, `save()`, `closeFile()`
-
-3. **_parse_task_line** → TypeScript function returning parsed Task fields from a markdown line
-
-4. **Emoji handling** → Same emoji map, just TypeScript objects
-
-## Tool Definitions
-
-Each Python action maps to a pi tool:
-
-| Tool Name | Action | Description |
-|-----------|--------|-------------|
-| `task_open` | `open_file` | Open a TODO.md file for editing |
-| `task_add` | `add_task` | Add a new task |
-| `task_edit` | `edit_task` | Edit an existing task |
-| `task_move` | `move_task` | Move a task to a different category |
-| `task_get` | `get_task` | Get details of a single task |
-| `task_list` | `list_tasks` | List tasks (with filters) |
-| `task_save` | `save` | Save current state to disk |
-| `task_close` | `close_file` | Close the current file |
-
-## Implementation Steps
-
-### Phase 1: Core (Day 1)
-
-1. Create project structure with `package.json`
-2. Port `Task` dataclass → TypeScript interface
-3. Port `_parse_task_line` → TypeScript parser function
-4. Port `TaskManager` class → TypeScript (core methods only)
-5. Write basic tests for parser
-
-### Phase 2: Extension (Day 2)
-
-6. Create `extension/index.ts` with tool registrations
-7. Define TypeBox schemas for each tool
-8. Wire tool execute handlers to TaskManager methods
-9. Test extension in pi (load via `pi -e`)
-
-### Phase 3: Polish (Day 3)
-
-10. Create `skills/task-manager/SKILL.md` with usage instructions
-11. Add `resources_discover` handler for skill auto-discovery
-12. Write `README.md`
-13. Test full workflow end-to-end
-
-### Phase 4: Optional
-
-14. Add custom TUI rendering (status bar showing task counts)
-15. Add `/tasks` command for quick status
-16. Add `session_start` hook to auto-open TODO.md if present in cwd
-
-## Design Decisions
-
-- **State management**: The TaskManager singleton persists across tool calls within a session. `open_file` sets the active file, subsequent actions operate on it, `close_file` clears it. This matches the Python behavior.
-- **No subprocess**: Pure TypeScript, no Python dependency.
-- **TypeBox schemas**: Use `Type.Object()` for parameters, `StringEnum` for fixed choices (category, priority).
-- **Skill + Extension**: The extension provides the tools; the skill provides the LLM with usage instructions and conventions.

package/tests/parser.test.ts

@@ -1,120 +0,0 @@
-import { test } from "node:test";
-import assert from "node:assert/strict";
-import {
-	parseTaskLine,
-	parseTodoFile,
-	buildTaskLine,
-	tasksToMarkdown,
-} from "../lib/parser.ts";
-import { depthOf, type Task } from "../lib/task.ts";
-
-/** Flatten a tree to DFS order (for assertions). */
-function flatten(roots: Task[]): Task[] {
-	const out: Task[] = [];
-	const walk = (tasks: Task[]): void => {
-		for (const t of tasks) {
-			out.push(t);
-			walk(t.children);
-		}
-	};
-	walk(roots);
-	return out;
-}
-
-test("parse simple task line", () => {
-	const t = parseTaskLine("- [ ] Buy milk (ID: `abc123`)")!;
-	assert.equal(t.id, "abc123");
-	assert.equal(t.description, "Buy milk");
-	assert.equal(t.status, " ");
-	assert.equal(t.parent, null);
-	assert.deepEqual(t.children, []);
-});
-
-test("parse indented task line", () => {
-	const t = parseTaskLine("  - [x] Done thing (ID: `def456`)")!;
-	assert.equal(t.id, "def456");
-	assert.equal(t.status, "x");
-});
-
-test("parse all annotations", () => {
-	const line =
-		"- [>] Roof 🔺 ⏳ 2026-01-01 🛫 2026-01-02 📅 2026-01-03 ✅ 2026-01-04 ❌ 2026-01-05 🔁 weekly 🗑️ ⛔ abc123,def456 📎 [spec](task-ghi789.md) ➕ 2026-01-01 🖊️ 2026-01-02 (ID: `ghi789`)";
-	const t = parseTaskLine(line)!;
-	assert.equal(t.description, "Roof");
-	assert.equal(t.priority, "highest");
-	assert.equal(t.dateScheduled, "2026-01-01");
-	assert.equal(t.dateStart, "2026-01-02");
-	assert.equal(t.dateDue, "2026-01-03");
-	assert.equal(t.dateDone, "2026-01-04");
-	assert.equal(t.dateCancelled, "2026-01-05");
-	assert.equal(t.recurrence, "weekly");
-	assert.equal(t.onCompletion, "delete");
-	assert.deepEqual(t.dependsOn, ["abc123", "def456"]);
-	assert.equal(t.hasSpec, true);
-	assert.equal(t.dateCreated, "2026-01-01");
-	assert.equal(t.dateModified, "2026-01-02");
-});
-
-test("recurrence rule extends to next emoji", () => {
-	const t = parseTaskLine("- [ ] Meet 🔁 every 2 weeks on Monday 📅 2026-02-01 (ID: `abc123`)")!;
-	assert.equal(t.recurrence, "every 2 weeks on Monday");
-	assert.equal(t.dateDue, "2026-02-01");
-});
-
-test("invalid lines return null", () => {
-	assert.equal(parseTaskLine("- [ ] No id here"), null);
-	assert.equal(parseTaskLine("- [q] Bad status (ID: `abc123`)"), null);
-	assert.equal(parseTaskLine("# comment"), null);
-	assert.equal(parseTaskLine(""), null);
-});
-
-test("hierarchy: depth, parent, children", () => {
-	const content = [
-		"# TODO",
-		"",
-		"- [ ] A (ID: `aaaaaa`)",
-		"  - [ ] B (ID: `bbbbbb`)",
-		"    - [ ] C (ID: `cccccc`)",
-		"  - [ ] D (ID: `dddddd`)",
-		"- [ ] E (ID: `eeeeee`)",
-		"",
-	].join("\n");
-	const roots = parseTodoFile(content);
-	assert.deepEqual(roots.map((t) => t.id), ["aaaaaa", "eeeeee"]);
-	const byId = Object.fromEntries(flatten(roots).map((t) => [t.id, t]));
-	assert.equal(byId["bbbbbb"].parent?.id, "aaaaaa");
-	assert.equal(depthOf(byId["bbbbbb"]), 1);
-	assert.equal(byId["cccccc"].parent?.id, "bbbbbb");
-	assert.equal(depthOf(byId["cccccc"]), 2);
-	assert.equal(byId["dddddd"].parent?.id, "aaaaaa");
-	assert.equal(byId["aaaaaa"].children.indexOf(byId["dddddd"]), 1);
-	assert.deepEqual(byId["aaaaaa"].children.map((c) => c.id), [
-		"bbbbbb",
-		"dddddd",
-	]);
-	assert.deepEqual(byId["bbbbbb"].children.map((c) => c.id), ["cccccc"]);
-});
-
-test("round-trip: parse -> build -> parse is stable", () => {
-	const content = [
-		"# TODO",
-		"",
-		"- [ ] A ⏫ ⏳ 2026-09-01 📅 2026-12-01 ➕ 2026-08-14 🖊️ 2026-08-14 (ID: `aaaaaa`)",
-		"  - [>] B 🔺 🔁 monthly 🗑️ ⛔ aaaaaa ➕ 2026-08-14 🖊️ 2026-08-14 (ID: `bbbbbb`)",
-		"",
-	].join("\n");
-	const once = parseTodoFile(content);
-	const rebuilt = tasksToMarkdown(once);
-	assert.equal(rebuilt, content);
-	const twice = parseTodoFile(rebuilt);
-	assert.deepEqual(
-		flatten(twice).map((t) => t.id),
-		flatten(once).map((t) => t.id),
-	);
-	assert.equal(tasksToMarkdown(twice), content);
-});
-
-test("buildTaskLine: no annotations", () => {
-	const t = parseTaskLine("- [ ] Plain (ID: `aaaaaa`)")!;
-	assert.equal(buildTaskLine(t), "- [ ] Plain (ID: `aaaaaa`)");
-});

package/tests/robustness.test.ts

@@ -1,154 +0,0 @@
-import { test } from "node:test";
-import assert from "node:assert/strict";
-import fs from "node:fs";
-import os from "node:os";
-import path from "node:path";
-import { TaskManager } from "../lib/task-manager.ts";
-
-function setup() {
-	const dir = fs.mkdtempSync(path.join(os.tmpdir(), "tm-rob-"));
-	const tm = new TaskManager();
-	tm.openFile(dir);
-	return { tm, dir };
-}
-
-const id = (r: Record<string, unknown>) => r.task_id as string;
-
-// ── annotation emoji in descriptions (CdqV8G) ─────────────────────────
-
-test("addTask: rejects description containing priority emoji", () => {
-	const { tm } = setup();
-	const r = tm.addTask("Fix 🔺 icon");
-	assert.equal(r.status, "error");
-	assert.match(r.error as string, /emoji/i);
-});
-
-test("addTask: rejects description containing date emoji", () => {
-	const { tm } = setup();
-	const r = tm.addTask("done ✅ today");
-	assert.equal(r.status, "error");
-	assert.match(r.error as string, /emoji/i);
-});
-
-test("addTask: rejects description containing recurrence emoji", () => {
-	const { tm } = setup();
-	const r = tm.addTask("repeats 🔁 forever");
-	assert.equal(r.status, "error");
-	assert.match(r.error as string, /emoji/i);
-});
-
-test("editTask: rejects description containing annotation emoji", () => {
-	const { tm } = setup();
-	const a = id(tm.addTask("A"));
-	const r = tm.editTask(a, "Fix 🔺 icon");
-	assert.equal(r.status, "error");
-	assert.match(r.error as string, /emoji/i);
-});
-
-test("addTask: accepts description with non-annotation emoji (round-trips)", () => {
-	const { tm, dir } = setup();
-	const a = id(tm.addTask("Fix the 🐛 bug"));
-	tm.closeFile();
-	const tm2 = new TaskManager();
-	tm2.openFile(dir);
-	const t = tm2.getTask(a) as any;
-	assert.equal(t.task.description, "Fix the 🐛 bug");
-});
-
-// ── status revert (hjKhGg) ───────────────────────────────────────────────────
-
-test("editTask: reverting status from x clears date_done", () => {
-	const { tm } = setup();
-	const a = id(tm.addTask("A"));
-	tm.editTask(a, undefined, "x");
-	assert.ok((tm.getTask(a) as any).task.date_done, "date_done stamped on x");
-	tm.editTask(a, undefined, " ");
-	assert.equal((tm.getTask(a) as any).task.date_done, null);
-});
-
-test("editTask: reverting status from - clears date_cancelled", () => {
-	const { tm } = setup();
-	const a = id(tm.addTask("A"));
-	tm.editTask(a, undefined, "-");
-	assert.ok((tm.getTask(a) as any).task.date_cancelled, "date_cancelled stamped on -");
-	tm.editTask(a, undefined, " ");
-	assert.equal((tm.getTask(a) as any).task.date_cancelled, null);
-});
-
-test("editTask: x to - clears date_done and stamps date_cancelled", () => {
-	const { tm } = setup();
-	const a = id(tm.addTask("A"));
-	tm.editTask(a, undefined, "x");
-	tm.editTask(a, undefined, "-");
-	const t = (tm.getTask(a) as any).task;
-	assert.equal(t.date_done, null);
-	assert.ok(t.date_cancelled);
-});
-
-// ── openFile error handling (3Dqwgr) ──────────────────────────────────
-
-test("openFile: bad path returns error Result instead of throwing", () => {
-	const tm = new TaskManager();
-	const r = tm.openFile("/nonexistent/definitely/missing");
-	assert.equal(r.status, "error");
-	assert.match(r.error as string, /open/i);
-	assert.equal(tm.isOpen, false);
-});
-
-test("openFile: path is a file, not a directory → error Result", () => {
-	const dir = fs.mkdtempSync(path.join(os.tmpdir(), "tm-rob-"));
-	const file = path.join(dir, "notadir");
-	fs.writeFileSync(file, "hi");
-	const tm = new TaskManager();
-	const r = tm.openFile(file);
-	assert.equal(r.status, "error");
-	assert.equal(tm.isOpen, false);
-});
-
-// ── save failure surfacing (gtoVfN) ───────────────────────────────────
-
-test("save: returns error when directory is read-only", () => {
-	const dir = fs.mkdtempSync(path.join(os.tmpdir(), "tm-rob-"));
-	const tm = new TaskManager();
-	tm.openFile(dir);
-	const a = id(tm.addTask("A"));
-	fs.chmodSync(dir, 0o555);
-	try {
-		const r = tm.save();
-		assert.equal(r.status, "error");
-		assert.match(r.error as string, /save/i);
-	} finally {
-		fs.chmodSync(dir, 0o755);
-	}
-});
-
-test("addTask: reports warning when auto-save fails", () => {
-	const dir = fs.mkdtempSync(path.join(os.tmpdir(), "tm-rob-"));
-	const tm = new TaskManager();
-	tm.openFile(dir);
-	id(tm.addTask("A"));
-	fs.chmodSync(dir, 0o555);
-	try {
-		const r = tm.addTask("B");
-		assert.equal(r.status, "ok");
-		assert.match(r.warning as string, /save failed/i);
-	} finally {
-		fs.chmodSync(dir, 0o755);
-	}
-});
-
-test("closeFile: save failure returns error and keeps file open", () => {
-	const dir = fs.mkdtempSync(path.join(os.tmpdir(), "tm-rob-"));
-	const tm = new TaskManager();
-	tm.openFile(dir);
-	fs.chmodSync(dir, 0o555);
-	try {
-		tm.addTask("A"); // auto-save fails, stays dirty
-		const r = tm.closeFile();
-		assert.equal(r.status, "error");
-		assert.match(r.error as string, /save/i);
-		assert.equal(tm.isOpen, true);
-	} finally {
-		fs.chmodSync(dir, 0o755);
-	}
-});

package/tests/task-manager.test.ts

@@ -1,158 +0,0 @@
-import { test } from "node:test";
-import assert from "node:assert/strict";
-import fs from "node:fs";
-import os from "node:os";
-import path from "node:path";
-import { TaskManager } from "../lib/task-manager.ts";
-
-function setup() {
-	const dir = fs.mkdtempSync(path.join(os.tmpdir(), "tm-test-"));
-	const tm = new TaskManager();
-	tm.openFile(dir);
-	return { tm, dir };
-}
-
-const id = (r: Record<string, unknown>) => r.task_id as string;
-
-test("open creates TODO.md", () => {
-	const { tm, dir } = setup();
-	assert.ok(fs.existsSync(path.join(dir, "TODO.md")));
-	assert.equal(tm.isOpen, true);
-	tm.closeFile();
-});
-
-test("add: hierarchy and positions", () => {
-	const { tm } = setup();
-	const a = id(tm.addTask("A"));
-	const b = id(tm.addTask("B", a));
-	const c = id(tm.addTask("C", a, b));
-	const d = id(tm.addTask("D", a));
-	const e = id(tm.addTask("E"));
-	const list = tm.listTasks() as any;
-	assert.deepEqual(
-		list.tasks.map((t: any) => t.id),
-		[a, c, b, d, e],
-	);
-	const byId = Object.fromEntries(list.tasks.map((t: any) => [t.id, t]));
-	assert.equal(byId[b].parent_id, a);
-	assert.equal(byId[b].depth, 1);
-	assert.equal(byId[c].position, 0);
-	assert.equal(byId[b].position, 1);
-});
-
-test("add: invalid parent", () => {
-	const { tm } = setup();
-	const r = tm.addTask("X", "nope12");
-	assert.equal(r.status, "error");
-});
-
-test("edit: no fields is an error", () => {
-	const { tm } = setup();
-	const a = id(tm.addTask("A"));
-	const r = tm.editTask(a);
-	assert.equal(r.status, "error");
-	assert.equal(r.error, "No fields to edit.");
-});
-
-test("edit: invalid status and priority", () => {
-	const { tm } = setup();
-	const a = id(tm.addTask("A"));
-	assert.equal((tm.editTask(a, undefined, "z") as any).status, "error");
-	assert.equal((tm.editTask(a, undefined, undefined, "urgent") as any).status, "error");
-	const ok = tm.editTask(a, "A2", "x", "null") as any;
-	assert.equal(ok.status, "ok");
-	assert.equal(ok.task.description, "A2");
-	assert.equal(ok.task.status, "x");
-	assert.equal(ok.task.priority, null);
-	assert.ok(ok.task.date_done);
-});
-
-test("edit: circular dependency rejected", () => {
-	const { tm } = setup();
-	const a = id(tm.addTask("A"));
-	const b = id(tm.addTask("B"));
-	tm.editTask(b, undefined, undefined, undefined, undefined, undefined, undefined, undefined, undefined, [a]);
-	const r = tm.editTask(a, undefined, undefined, undefined, undefined, undefined, undefined, undefined, undefined, [b]);
-	assert.equal(r.status, "error");
-	assert.match(r.error as string, /Circular/);
-});
-
-test("move: under own descendant rejected", () => {
-	const { tm } = setup();
-	const a = id(tm.addTask("A"));
-	const b = id(tm.addTask("B", a));
-	const r = tm.moveTask(a, b);
-	assert.equal(r.status, "error");
-});
-
-test("move: before later sibling keeps order", () => {
-	const { tm } = setup();
-	const a = id(tm.addTask("A"));
-	const b = id(tm.addTask("B", a));
-	const c = id(tm.addTask("C", a));
-	// b already sits before c; re-inserting "before c" must not swap them
-	// (the index must be computed after detaching b).
-	const r = tm.moveTask(b, a, c);
-	assert.equal(r.status, "ok");
-	const ids = (tm.listTasks().tasks as any[]).map((t) => t.id);
-	assert.deepEqual(ids, [a, b, c]);
-});
-
-test("move: with subtree, positions recomputed", () => {
-	const { tm } = setup();
-	const a = id(tm.addTask("A"));
-	const b = id(tm.addTask("B"));
-	const c = id(tm.addTask("C", b));
-	const d = id(tm.addTask("D"));
-	tm.moveTask(b, a);
-	const list = tm.listTasks() as any;
-	assert.deepEqual(
-		list.tasks.map((t: any) => t.id),
-		[a, b, c, d],
-	);
-	const byId = Object.fromEntries(list.tasks.map((t: any) => [t.id, t]));
-	assert.equal(byId[b].parent_id, a);
-	assert.equal(byId[c].parent_id, b);
-	assert.equal(byId[c].depth, 2);
-});
-
-test("move: no destination deletes task and subtree", () => {
-	const { tm } = setup();
-	const a = id(tm.addTask("A"));
-	const b = id(tm.addTask("B", a));
-	const c = id(tm.addTask("C", b));
-	const r = tm.moveTask(a);
-	assert.equal(r.status, "ok");
-	assert.match(r.message as string, /2 sub-task/);
-	const list = tm.listTasks() as any;
-	assert.equal(list.count, 0);
-});
-
-test("list: filters", () => {
-	const { tm } = setup();
-	const a = id(tm.addTask("A", undefined, undefined, undefined, "high"));
-	tm.addTask("B");
-	tm.editTask(a, undefined, "x");
-	const byStatus = tm.listTasks(undefined, "x") as any;
-	assert.equal(byStatus.count, 1);
-	assert.equal(byStatus.tasks[0].id, a);
-	const byPriority = tm.listTasks(undefined, undefined, "high") as any;
-	assert.equal(byPriority.count, 1);
-	const subtree = tm.listTasks(a, undefined, undefined, true) as any;
-	assert.equal(subtree.count, 1);
-	assert.equal(subtree.tasks[0].id, a);
-});
-
-test("save and close", () => {
-	const { tm, dir } = setup();
-	tm.addTask("A");
-	const s = tm.save() as any;
-	assert.equal(s.status, "ok");
-	assert.equal(s.task_count, 1);
-	const c = tm.closeFile() as any;
-	assert.equal(c.status, "ok");
-	assert.equal(tm.isOpen, false);
-	const r = tm.save();
-	assert.equal(r.status, "error");
-	assert.ok(fs.existsSync(path.join(dir, "TODO.md.bak")));
-});

package/tests/validation.test.ts

@@ -1,194 +0,0 @@
-import { test } from "node:test";
-import assert from "node:assert/strict";
-import fs from "node:fs";
-import os from "node:os";
-import path from "node:path";
-import { TaskManager } from "../lib/task-manager.ts";
-
-function setup() {
-	const dir = fs.mkdtempSync(path.join(os.tmpdir(), "tm-val-"));
-	const tm = new TaskManager();
-	tm.openFile(dir);
-	return { tm, dir };
-}
-
-const id = (r: Record<string, unknown>) => r.task_id as string;
-
-// ── addTask validation ────────────────────────────────────────────────
-
-test("addTask: rejects multiline description", () => {
-	const { tm } = setup();
-	const r = tm.addTask("line1\nline2");
-	assert.equal(r.status, "error");
-	assert.match(r.error as string, /newline/i);
-});
-
-test("addTask: rejects invalid scheduled date", () => {
-	const { tm } = setup();
-	const r = tm.addTask("A", undefined, undefined, undefined, undefined, "tomorrow");
-	assert.equal(r.status, "error");
-	assert.match(r.error as string, /scheduled/i);
-});
-
-test("addTask: rejects invalid start date", () => {
-	const { tm } = setup();
-	const r = tm.addTask("A", undefined, undefined, undefined, undefined, undefined, "2026-1-1");
-	assert.equal(r.status, "error");
-	assert.match(r.error as string, /start/i);
-});
-
-test("addTask: rejects invalid due date", () => {
-	const { tm } = setup();
-	const r = tm.addTask("A", undefined, undefined, undefined, undefined, undefined, undefined, "15/08/2026");
-	assert.equal(r.status, "error");
-	assert.match(r.error as string, /due/i);
-});
-
-test("addTask: rejects invalid priority", () => {
-	const { tm } = setup();
-	const r = tm.addTask("A", undefined, undefined, undefined, "urgent");
-	assert.equal(r.status, "error");
-	assert.match(r.error as string, /priority/i);
-});
-
-test("addTask: accepts valid dates and priority", () => {
-	const { tm } = setup();
-	const r = tm.addTask(
-		"A",
-		undefined,
-		undefined,
-		undefined,
-		"high",
-		"2026-09-01",
-		"2026-09-02",
-		"2026-09-03",
-	);
-	assert.equal(r.status, "ok");
-	const t = tm.getTask(id(r)) as any;
-	assert.equal(t.task.priority, "high");
-	assert.equal(t.task.date_scheduled, "2026-09-01");
-	assert.equal(t.task.date_start, "2026-09-02");
-	assert.equal(t.task.date_due, "2026-09-03");
-});
-
-// ── editTask validation ───────────────────────────────────────────────
-
-test("editTask: rejects multiline description", () => {
-	const { tm } = setup();
-	const a = id(tm.addTask("A"));
-	const r = tm.editTask(a, "line1\nline2");
-	assert.equal(r.status, "error");
-	assert.match(r.error as string, /newline/i);
-});
-
-test("editTask: rejects invalid dates", () => {
-	const { tm } = setup();
-	const a = id(tm.addTask("A"));
-	assert.equal((tm.editTask(a, undefined, undefined, undefined, "tomorrow") as any).status, "error");
-	assert.equal((tm.editTask(a, undefined, undefined, undefined, undefined, "tomorrow") as any).status, "error");
-	assert.equal((tm.editTask(a, undefined, undefined, undefined, undefined, undefined, "tomorrow") as any).status, "error");
-});
-
-// ── openFile validation: orphaned indented lines ─────────────────────
-
-test("openFile: rejects orphaned indented lines (missing ancestor)", () => {
-	const dir = fs.mkdtempSync(path.join(os.tmpdir(), "tm-val-"));
-	const orphan = `# TODO
-
-- [ ] Parent (ID: \`a1b2c3\`)
-    - [ ] Orphan at depth 2 (ID: \`x1y2z3\`)
-`;
-	fs.writeFileSync(path.join(dir, "TODO.md"), orphan);
-	const tm = new TaskManager();
-	const r = tm.openFile(dir);
-	assert.equal(r.status, "error");
-	assert.match(r.error as string, /orphan/i);
-	assert.equal(tm.isOpen, false);
-});
-
-test("openFile: rejects indented line with no root at all", () => {
-	const dir = fs.mkdtempSync(path.join(os.tmpdir(), "tm-val-"));
-	const orphan = `# TODO
-
-  - [ ] Indented but no ancestor (ID: \`q1w2e3\`)
-`;
-	fs.writeFileSync(path.join(dir, "TODO.md"), orphan);
-	const tm = new TaskManager();
-	const r = tm.openFile(dir);
-	assert.equal(r.status, "error");
-	assert.match(r.error as string, /orphan/i);
-	assert.equal(tm.isOpen, false);
-});
-
-test("openFile: rejects tab indentation", () => {
-	const dir = fs.mkdtempSync(path.join(os.tmpdir(), "tm-val-"));
-	const tabbed = `# TODO
-
-- [ ] Root (ID: \`a1b2c3\`)
-\t- [ ] Tab child (ID: \`x1y2z3\`)
-`;
-	fs.writeFileSync(path.join(dir, "TODO.md"), tabbed);
-	const tm = new TaskManager();
-	const r = tm.openFile(dir);
-	assert.equal(r.status, "error");
-	assert.match(r.error as string, /tab/i);
-	assert.equal(tm.isOpen, false);
-});
-
-test("openFile: rejects duplicate task IDs", () => {
-	const dir = fs.mkdtempSync(path.join(os.tmpdir(), "tm-val-"));
-	const dups = `# TODO
-
-- [ ] A (ID: \`a1b2c3\`)
-- [ ] B (ID: \`a1b2c3\`)
-`;
-	fs.writeFileSync(path.join(dir, "TODO.md"), dups);
-	const tm = new TaskManager();
-	const r = tm.openFile(dir);
-	assert.equal(r.status, "error");
-	assert.match(r.error as string, /duplicate/i);
-	assert.equal(tm.isOpen, false);
-});
-
-test("openFile: accepts properly nested file", () => {
-	const dir = fs.mkdtempSync(path.join(os.tmpdir(), "tm-val-"));
-	const valid = `# TODO
-
-- [ ] A (ID: \`a1b2c3\`)
-  - [ ] B (ID: \`d4e5f6\`)
-    - [ ] C (ID: \`g7h8i9\`)
-- [ ] D (ID: \`j1k2l3\`)
-`;
-	fs.writeFileSync(path.join(dir, "TODO.md"), valid);
-	const tm = new TaskManager();
-	const r = tm.openFile(dir);
-	assert.equal(r.status, "ok");
-	assert.equal((r as any).task_count, 4);
-	tm.closeFile();
-});
-
-// ── round-trip guard: valid data survives save/reopen ─────────────────
-
-test("round-trip: valid dates + priority survive save/reopen", () => {
-	const { tm, dir } = setup();
-	const a = id(
-		tm.addTask(
-			"A",
-			undefined,
-			undefined,
-			undefined,
-			"high",
-			"2026-09-01",
-			"2026-09-02",
-			"2026-09-03",
-		),
-	);
-	tm.closeFile();
-	const tm2 = new TaskManager();
-	tm2.openFile(dir);
-	const t = tm2.getTask(a) as any;
-	assert.equal(t.task.priority, "high");
-	assert.equal(t.task.date_scheduled, "2026-09-01");
-	assert.equal(t.task.date_start, "2026-09-02");
-	assert.equal(t.task.date_due, "2026-09-03");
-});

package/tsconfig.json

@@ -1,21 +0,0 @@
-{
-	"compilerOptions": {
-		"target": "ES2022",
-		"module": "NodeNext",
-		"moduleResolution": "NodeNext",
-		"strict": true,
-		"noEmit": true,
-		"skipLibCheck": true,
-		"allowImportingTsExtensions": true,
-		"verbatimModuleSyntax": true,
-		"paths": {
-			"@earendil-works/pi-ai": [
-				"/home/gil/.local/share/pi-node/node-v22.23.2-linux-x64/lib/node_modules/@earendil-works/pi-coding-agent/node_modules/@earendil-works/pi-ai/dist/index.d.ts"
-			],
-			"@earendil-works/pi-coding-agent": [
-				"/home/gil/.local/share/pi-node/node-v22.23.2-linux-x64/lib/node_modules/@earendil-works/pi-coding-agent/dist/index.d.ts"
-			]
-		}
-	},
-	"include": ["index.ts", "lib", "tests"]
-}