@xynogen/pix-todo 0.3.0 → 0.3.2

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-todo",
-	"version": "0.3.0",
+	"version": "0.3.2",
 	"description": "Pi tool — durable execution checklist (todo)",
 	"type": "module",
 	"main": "src/index.ts",
@@ -34,7 +34,7 @@
 		"access": "public"
 	},
 	"dependencies": {
-		"@xynogen/pix-pretty": "^1.11.2",
+		"@xynogen/pix-pretty": "^1.19.0",
 		"@xynogen/pix-runtime": "^0.8.0",
 		"typebox": "^1.1.38"
 	},

package/src/todo.test.ts

@@ -886,6 +886,7 @@ describe("todo card layout", () => {
 		expect(rendered).toContain("alpha");
 		expect(rendered).toContain("[text]○ alpha"); // pending card, no id number
 		expect(rendered).not.toContain("[success]✓ [/] [toolTitle]<b>todo</b>[/]");
+		expect(rendered.split("\n")[0]).toBe(`[success]${"─".repeat(80)}[/]`);
 	});
 
 	test("opening a new todo card collapses the previously open one", async () => {
@@ -931,7 +932,8 @@ describe("todo card layout", () => {
 			.join("\n")
 			.trimEnd();
 
-		expect(rendered).toBe("No todo with id 99.");
+		expect(rendered.split("\n")[0]).toBe(`[error]${"─".repeat(80)}[/]`);
+		expect(rendered).toContain("No todo with id 99.");
 	});
 
 	test("a collapsed result is exactly one shared-style line", async () => {
@@ -954,8 +956,9 @@ describe("todo card layout", () => {
 
 		expect(result.details).toBeDefined();
 		expect(lines).toHaveLength(1);
+		expect(lines[0]).not.toContain("────");
 		expect(lines[0]?.trimEnd()).toBe(
-			"[success]✓ [/] [toolTitle]<b>todo</b>[/] [muted]#2 todo renderer[/] [dim]·[/] [dim]1/2 done[/]",
+			"[success]✓ [/] [toolTitle]<b>todo</b>[/] [dim]#2 todo renderer[/] [muted]·[/] [muted]1/2 done[/]",
 		);
 	});
 });
@@ -986,14 +989,14 @@ describe("renderResult snapshot isolation", () => {
 describe("renderTodoSummaryLine (collapsed one-liner)", () => {
 	test("empty list renders a compact tool row", () => {
 		expect(renderTodoSummaryLine([], tagTheme)).toBe(
-			"[success]✓ [/] [toolTitle]<b>todo</b>[/] [muted]empty[/]",
+			"[success]✓ [/] [toolTitle]<b>todo</b>[/] [dim]empty[/]",
 		);
 	});
 
 	test("uses warning status when work is blocked", () => {
 		const items: TodoItem[] = [{ id: 1, text: "Need approval", status: "blocked" }];
 		expect(renderTodoSummaryLine(items, tagTheme)).toBe(
-			"[warning]⚠ [/] [toolTitle]<b>todo</b>[/] [muted]checklist[/] [dim]·[/] [dim]0/1 done · 1 blocked[/]",
+			"[warning]⚠ [/] [toolTitle]<b>todo</b>[/] [dim]checklist[/] [muted]·[/] [muted]0/1 done · 1 blocked[/]",
 		);
 	});
 
@@ -1003,7 +1006,7 @@ describe("renderTodoSummaryLine (collapsed one-liner)", () => {
 			{ id: 2, text: "b", status: "in_progress" },
 		];
 		expect(renderTodoSummaryLine(items, tagTheme)).toBe(
-			"[success]✓ [/] [toolTitle]<b>todo</b>[/] [muted]#2 b[/] [dim]·[/] [dim]1/2 done[/]",
+			"[success]✓ [/] [toolTitle]<b>todo</b>[/] [dim]#2 b[/] [muted]·[/] [muted]1/2 done[/]",
 		);
 	});
 });

package/src/todo.ts

@@ -12,7 +12,7 @@
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { Text } from "@earendil-works/pi-tui";
 import { icon } from "@xynogen/pix-pretty/icon-catalog";
-import { dotJoin, formatCollapsedToolRow, termW } from "@xynogen/pix-pretty/utils";
+import { dotJoin, formatCollapsedToolRow, frameToolResult, termW } from "@xynogen/pix-pretty/utils";
 import { type CollapseState, tickCollapse } from "@xynogen/pix-runtime/collapse";
 import { once } from "@xynogen/pix-runtime/once";
 import { Type } from "typebox";
@@ -271,7 +271,7 @@ export default function registerTodo(pi: ExtensionAPI): void {
 				const t = theme as TodoTheme;
 				const action = (args as { action?: string })?.action ?? "";
 				const title = t.fg("toolTitle", t.bold("todo"));
-				return new Text(action ? `${title} ${t.fg("muted", action)}` : title, 0, 0);
+				return new Text(action ? `${title} ${t.fg("dim", action)}` : title, 0, 0);
 			},
 			renderResult(result, options, theme, context) {
 				const details = result.details as TodoResultDetails | undefined;
@@ -279,8 +279,10 @@ export default function registerTodo(pi: ExtensionAPI): void {
 					.filter((part) => part.type === "text")
 					.map((part) => part.text)
 					.join("\n");
+				const isPartial = options.isPartial === true;
 				if (context.isError || details?.outcome === "error" || !details) {
-					return new Text(resultText, 0, 0);
+					const component = new Text(resultText, 0, 0);
+					return isPartial ? component : frameToolResult(component, theme, true);
 				}
 
 				const collapsed = tickCollapse(
@@ -292,7 +294,8 @@ export default function registerTodo(pi: ExtensionAPI): void {
 				// An open card claims focus, collapsing any earlier open board.
 				if (!collapsed) focusCard(context.state as CollapseState, context.invalidate);
 				const render = collapsed ? renderTodoSummaryLine : renderTodoLines;
-				return new Text(render(details.snapshot, theme as TodoTheme), 0, 0);
+				const component = new Text(render(details.snapshot, theme as TodoTheme), 0, 0);
+				return collapsed || isPartial ? component : frameToolResult(component, theme, false);
 			},
 
 			async execute(_id, params) {