@xynogen/pix-todo 0.1.13 → 0.1.14

package/package.json

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

package/src/todo.test.ts

@@ -47,16 +47,21 @@ function makeHost(
 				context: unknown,
 		  ) => { render(width: number): string[] })
 		| null = null;
+	let capturedRenderCall:
+		| ((args: unknown, theme: unknown, context: unknown) => { render(width: number): string[] })
+		| null = null;
 
 	const pi = {
 		registerTool(def: {
 			name: string;
 			parameters: unknown;
 			execute: typeof capturedExecute;
+			renderCall?: typeof capturedRenderCall;
 			renderResult?: typeof capturedRender;
 		}) {
 			capturedParameters = def.parameters;
 			capturedExecute = def.execute;
+			if (def.renderCall) capturedRenderCall = def.renderCall;
 			if (def.renderResult) capturedRender = def.renderResult;
 		},
 		appendEntry(type: string, data: unknown) {
@@ -93,6 +98,10 @@ function makeHost(
 			if (!capturedExecute) throw new Error("execute not captured");
 			return capturedExecute;
 		},
+		get renderCall() {
+			if (!capturedRenderCall) throw new Error("renderCall not captured");
+			return capturedRenderCall;
+		},
 		get render() {
 			if (!capturedRender) throw new Error("render not captured");
 			return capturedRender;
@@ -772,6 +781,15 @@ describe("renderTodoLines (colored TUI render)", () => {
 	});
 });
 
+describe("todo card layout", () => {
+	test("keeps the call row empty so the collapsed card is one line", () => {
+		const host = makeHost();
+		registerTodo(host.pi);
+		const call = host.renderCall({ action: "list" }, tagTheme, {});
+		expect(call.render(80).join("\n")).toBe("");
+	});
+});
+
 describe("renderResult snapshot isolation", () => {
 	// The card snapshots `todos` on first render; later execute() mutations must
 	// NOT bleed into an already-rendered card. Guards the invariant the inline
@@ -801,15 +819,19 @@ describe("renderResult snapshot isolation", () => {
 });
 
 describe("renderTodoSummaryLine (collapsed one-liner)", () => {
-	test("empty list renders muted placeholder", () => {
-		expect(renderTodoSummaryLine([], tagTheme)).toBe("[muted](no todos)[/]");
+	test("empty list renders a compact tool row", () => {
+		expect(renderTodoSummaryLine([], tagTheme)).toBe(
+			"[success]✓[/] [toolTitle]<b>todo</b>[/] [muted]empty[/]",
+		);
 	});
 
-	test("renders single dim done/total line with check", () => {
+	test("renders active work and progress in one row", () => {
 		const items: TodoItem[] = [
 			{ id: 1, text: "a", status: "done" },
-			{ id: 2, text: "b", status: "pending" },
+			{ id: 2, text: "b", status: "in_progress" },
 		];
-		expect(renderTodoSummaryLine(items, tagTheme)).toBe("[muted]Todos 1/2 done ✓[/]");
+		expect(renderTodoSummaryLine(items, tagTheme)).toBe(
+			"[success]✓[/] [toolTitle]<b>todo</b>[/] [muted]#2 b[/] [dim]·[/] [dim]1/2 done[/]",
+		);
 	});
 });

package/src/todo.ts

@@ -13,6 +13,7 @@ import { existsSync, readFileSync } from "node:fs";
 import { join } from "node:path";
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { Text } from "@earendil-works/pi-tui";
+import { formatCollapsedToolRow } from "@xynogen/pix-pretty/utils";
 import { Type } from "typebox";
 
 import { once } from "./once.ts";
@@ -99,9 +100,19 @@ export type TodoTheme = {
 
 /** One-line dim summary used once a card has collapsed. */
 export function renderTodoSummaryLine(items: TodoItem[], theme: TodoTheme): string {
-	if (!items.length) return theme.fg("muted", "(no todos)");
+	if (!items.length) return formatCollapsedToolRow(theme, "todo", "empty");
 	const done = items.filter((t) => t.status === "done").length;
-	return theme.fg("muted", `Todos ${done}/${items.length} done ✓`);
+	const active = items.find((t) => t.status === "in_progress");
+	const blocked = items.filter((t) => t.status === "blocked").length;
+	const meta = [`${done}/${items.length} done`, blocked > 0 ? `${blocked} blocked` : ""]
+		.filter(Boolean)
+		.join(" · ");
+	const target = active
+		? `#${active.id} ${active.text}`
+		: done === items.length
+			? "complete"
+			: "checklist";
+	return formatCollapsedToolRow(theme, "todo", target, meta);
 }
 
 /** Colored checklist for the TUI: glyphs tinted by status, active row bold. */
@@ -202,6 +213,11 @@ export default function registerTodo(pi: ExtensionAPI): void {
 					}),
 				),
 			}),
+			// The result already owns the checklist and its collapsed `✓ todo …` row.
+			// Keeping the call renderer empty prevents a duplicate standalone header.
+			renderCall() {
+				return new Text("", 0, 0);
+			},
 			renderResult(_result, _options, theme, context) {
 				// Snapshot this row's todos once (live `todos` mutate across calls;
 				// a card should keep the state it was created with).