@xynogen/pix-todo 0.1.23 → 0.1.25

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-todo",
-	"version": "0.1.23",
+	"version": "0.1.25",
 	"description": "Pi tool — durable execution checklist (todo)",
 	"type": "module",
 	"main": "src/index.ts",
@@ -36,7 +36,7 @@
 	"dependencies": {
 		"@xynogen/pix-data": "^0.4.3",
 		"@xynogen/pix-pretty": "^1.11.2",
-		"@xynogen/pix-runtime": "^0.5.3",
+		"@xynogen/pix-runtime": "^0.6.0",
 		"typebox": "^1.1.38"
 	},
 	"peerDependencies": {

package/src/todo.test.ts

@@ -819,7 +819,7 @@ describe("todo card layout", () => {
 
 		expect(rendered).toContain("1. alpha");
 		expect(rendered).toContain("[muted]○[/]");
-		expect(rendered).not.toContain("[success]✓[/] [toolTitle]<b>todo</b>[/]");
+		expect(rendered).not.toContain("[success]✓ [/] [toolTitle]<b>todo</b>[/]");
 	});
 
 	test("failed todo actions render their exact error", async () => {
@@ -860,7 +860,7 @@ describe("todo card layout", () => {
 		expect(result.details).toBeDefined();
 		expect(lines).toHaveLength(1);
 		expect(lines[0]?.trimEnd()).toBe(
-			"[success]✓[/] [toolTitle]<b>todo</b>[/] [muted]#2 todo renderer[/] [dim]·[/] [dim]1/2 done[/]",
+			"[success]✓ [/] [toolTitle]<b>todo</b>[/] [muted]#2 todo renderer[/] [dim]·[/] [dim]1/2 done[/]",
 		);
 	});
 });
@@ -891,7 +891,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>[/] [muted]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[/]",
 		);
 	});
 
@@ -901,7 +908,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>[/] [muted]#2 b[/] [dim]·[/] [dim]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 { formatCollapsedToolRow } from "@xynogen/pix-pretty/utils";
+import { dotJoin, formatCollapsedToolRow } 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";
@@ -64,15 +64,16 @@ export function renderTodoSummaryLine(items: TodoItem[], theme: TodoTheme): stri
 	const done = items.filter((t) => t.status === "done").length;
 	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 meta = dotJoin([`${done}/${items.length} done`, blocked > 0 && `${blocked} blocked`]);
 	const target = active
 		? `#${active.id} ${active.text}`
 		: done === items.length
 			? "complete"
 			: "checklist";
-	return formatCollapsedToolRow(theme, "todo", target, meta, "success");
+	// Blocked work with nothing active is not a success — surface the warning glyph
+	// (text meta already names the block count) so status is not color-only.
+	const status = blocked > 0 && !active && done !== items.length ? "warning" : "success";
+	return formatCollapsedToolRow(theme, "todo", target, meta, status);
 }
 
 /** Colored checklist for the TUI: glyphs tinted by status, active row bold. */