@xynogen/pix-todo 0.1.6 → 0.1.7

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-todo",
-	"version": "0.1.6",
+	"version": "0.1.7",
 	"description": "Pi tool — durable execution checklist (todo)",
 	"type": "module",
 	"main": "src/index.ts",

package/src/todo.test.ts

@@ -357,6 +357,50 @@ describe("todo actions", () => {
 		expect(out).toContain("Todos 1/3 done");
 	});
 
+	test("opening a later item cascade-closes skipped pending items", async () => {
+		const host = makeHost();
+		registerTodo(host.pi);
+		await host.emit(
+			"session_start",
+			{},
+			{ sessionManager: host.sessionManager },
+		);
+		await run(host.execute, { action: "set", items: "a\nb\nc\nd" });
+		// Jump straight to id 4 without opening 1-3; they should all auto-close.
+		const result = await run(host.execute, {
+			action: "update",
+			id: 4,
+			status: "in_progress",
+		});
+		const out = text(result);
+		expect(out).toContain("● 1. a");
+		expect(out).toContain("● 2. b");
+		expect(out).toContain("● 3. c");
+		expect(out).toContain("◐ 4. d");
+		expect(out).toContain("Todos 3/4 done");
+	});
+
+	test("cascade-close leaves a blocked earlier item untouched", async () => {
+		const host = makeHost();
+		registerTodo(host.pi);
+		await host.emit(
+			"session_start",
+			{},
+			{ sessionManager: host.sessionManager },
+		);
+		await run(host.execute, { action: "set", items: "a\nb\nc" });
+		await run(host.execute, { action: "update", id: 1, status: "blocked" });
+		const result = await run(host.execute, {
+			action: "update",
+			id: 3,
+			status: "in_progress",
+		});
+		const out = text(result);
+		expect(out).toContain("⊘ 1. a"); // still blocked, not force-closed
+		expect(out).toContain("● 2. b"); // pending -> done
+		expect(out).toContain("◐ 3. c");
+	});
+
 	test("update unknown id returns error", async () => {
 		const host = makeHost();
 		registerTodo(host.pi);
@@ -439,8 +483,10 @@ describe("todo actions", () => {
 			{ sessionManager: host.sessionManager },
 		);
 		await run(host.execute, { action: "set", items: "a\nb\nc\nd" });
-		await run(host.execute, { action: "update", id: 1, status: "pending" });
+		// Open id 2 first (cascade-closes nothing earlier we assert on), then set
+		// the others directly so each glyph is exercised without cascade interfering.
 		await run(host.execute, { action: "update", id: 2, status: "in_progress" });
+		await run(host.execute, { action: "update", id: 1, status: "pending" });
 		await run(host.execute, { action: "update", id: 3, status: "done" });
 		await run(host.execute, { action: "update", id: 4, status: "blocked" });
 		const out = text(await run(host.execute, { action: "list" }));

package/src/todo.ts

@@ -111,7 +111,7 @@ export default function registerTodo(pi: ExtensionAPI): void {
 				"todo(action, items?, id?, status?, text?) — action: list|set|add|update|clear. Use to track implementation progress, especially when executing a plan.",
 			promptGuidelines: [
 				"When you start executing a multi-step plan in BUILD mode, seed the todo list with `todo(action:'set', items: <plan Implementation Phases>)`.",
-				"Mark each item in_progress before working it via `todo(action:'update', id, status)`; opening one auto-closes the previous in_progress item, so just open the next.",
+				"Mark each item in_progress before working it via `todo(action:'update', id, status)`; opening one auto-closes every earlier item, so just open the next and skipped steps mark done themselves.",
 				"Call `todo(action:'list')` to recover your place after long runs or context compaction.",
 			],
 			parameters: Type.Object({
@@ -215,11 +215,17 @@ export default function registerTodo(pi: ExtensionAPI): void {
 						const t = todos.find((x) => x.id === params.id);
 						if (!t) return fail(`No todo with id ${params.id}.`);
 						if (params.status) {
-							// Single-active invariant: opening a new task closes any other
-							// in_progress one, so the checklist always shows one focus.
+							// Sequential-progress invariant: opening a task means everything
+							// before it is finished. Cascade-close every earlier pending or
+							// in_progress item (ids are sequential) so the model never has to
+							// mark skipped steps done by hand. `blocked` is left untouched.
 							if (params.status === "in_progress")
 								for (const other of todos)
-									if (other.id !== t.id && other.status === "in_progress")
+									if (
+										other.id < t.id &&
+										(other.status === "pending" ||
+											other.status === "in_progress")
+									)
 										other.status = "done";
 							t.status = params.status;
 						}