@xynogen/pix-todo 0.1.15 → 0.1.17

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-todo",
-	"version": "0.1.15",
+	"version": "0.1.17",
 	"description": "Pi tool — durable execution checklist (todo)",
 	"type": "module",
 	"main": "src/index.ts",
@@ -36,6 +36,7 @@
 	"dependencies": {
 		"@xynogen/pix-data": "^0.3.2",
 		"@xynogen/pix-pretty": "^1.7.20",
+		"@xynogen/pix-runtime": "^0.1.1",
 		"typebox": "^1.1.38"
 	},
 	"peerDependencies": {

package/src/todo.test.ts

@@ -27,6 +27,7 @@ function makeHost(
 	}> = [],
 ) {
 	let capturedParameters: unknown;
+	let capturedRenderShell: unknown;
 	let capturedExecute:
 		| ((
 				id: string,
@@ -57,10 +58,12 @@ function makeHost(
 			name: string;
 			parameters: unknown;
 			execute: typeof capturedExecute;
+			renderShell?: unknown;
 			renderCall?: typeof capturedRenderCall;
 			renderResult?: typeof capturedRender;
 		}) {
 			capturedParameters = def.parameters;
+			capturedRenderShell = def.renderShell;
 			capturedExecute = def.execute;
 			if (def.renderCall) capturedRenderCall = def.renderCall;
 			if (def.renderResult) capturedRender = def.renderResult;
@@ -99,6 +102,9 @@ function makeHost(
 			if (!capturedExecute) throw new Error("execute not captured");
 			return capturedExecute;
 		},
+		get renderShell() {
+			return capturedRenderShell;
+		},
 		get renderCall() {
 			if (!capturedRenderCall) throw new Error("renderCall not captured");
 			return capturedRenderCall;
@@ -777,13 +783,20 @@ describe("renderTodoLines (colored TUI render)", () => {
 		expect(out).toContain("[text]3. charlie[/]"); // pending body text
 	});
 
-	test("shows the done/total count header", () => {
+	test("shows the done/total count header in blue", () => {
 		const out = renderTodoLines(items, tagTheme);
-		expect(out).toContain("[muted]Todos 1/4 done:[/]");
+		expect(out).toContain("[accent]Todos 1/4 done:[/]");
+		expect(out).not.toContain("[muted]Todos 1/4 done:[/]");
 	});
 });
 
 describe("todo card layout", () => {
+	test("uses the self-rendered shell so the compact checkmark has no leading space", () => {
+		const host = makeHost();
+		registerTodo(host.pi);
+		expect(host.renderShell).toBe("self");
+	});
+
 	test("keeps the call row empty so the collapsed card is one line", () => {
 		const host = makeHost();
 		registerTodo(host.pi);

package/src/todo.ts

@@ -13,10 +13,9 @@ import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { Text } from "@earendil-works/pi-tui";
 import { type CollapseState, tickCollapse } from "@xynogen/pix-data/collapse";
 import { formatCollapsedToolRow } from "@xynogen/pix-pretty/utils";
+import { once } from "@xynogen/pix-runtime/once";
 import { Type } from "typebox";
 
-import { once } from "./once.ts";
-
 export type TodoStatus = "pending" | "in_progress" | "done" | "blocked";
 
 export interface TodoItem {
@@ -54,7 +53,7 @@ export type TodoTheme = {
 	bold: (text: string) => string;
 };
 
-/** One-line dim summary used once a card has collapsed. */
+/** One-line shared summary used once a card has collapsed. */
 export function renderTodoSummaryLine(items: TodoItem[], theme: TodoTheme): string {
 	if (!items.length) return formatCollapsedToolRow(theme, "todo", "empty");
 	const done = items.filter((t) => t.status === "done").length;
@@ -75,7 +74,7 @@ export function renderTodoSummaryLine(items: TodoItem[], theme: TodoTheme): stri
 export function renderTodoLines(items: TodoItem[], theme: TodoTheme): string {
 	if (!items.length) return theme.fg("muted", "(no todos)");
 	const done = items.filter((t) => t.status === "done").length;
-	const head = theme.fg("muted", `Todos ${done}/${items.length} done:`);
+	const head = theme.fg("accent", `Todos ${done}/${items.length} done:`);
 	const lines = items.map((t) => {
 		const color = TODO_COLOR[t.status];
 		const glyph = theme.fg(color, TODO_GLYPH[t.status]);
@@ -134,6 +133,9 @@ export default function registerTodo(pi: ExtensionAPI): void {
 		pi.registerTool({
 			name: "todo",
 			label: "Todo",
+			// Avoid the default Box shell's one-cell x padding: Todo owns its compact
+			// result row and should align its status glyph with other compact tools.
+			renderShell: "self",
 			description:
 				"Track BUILD-phase execution progress. Durable across context compaction. Actions: list, set (replace all items from newline/numbered text), add, update (change one item's status), clear.",
 			promptSnippet:

package/src/once.ts

@@ -1,27 +0,0 @@
-/**
- * Per-instance idempotency guard for extension activation.
- *
- * pix-core (the meta-package) invokes this package's factory, and a standalone
- * install makes Pi invoke it again — sometimes against the SAME `pi`. We must
- * dedupe that. But Pi rebuilds the extension runtime on /new, /resume, /fork,
- * and /reload, handing the factory a BRAND-NEW `pi`; that must re-register.
- *
- * Keying the registry on the `pi` instance satisfies both: same instance =>
- * skip, new instance => run. The registry lives on globalThis because jiti
- * (`moduleCache: false`) re-evaluates this module on every load pass, so a
- * module-scoped WeakMap would not be shared between the aggregator pass and the
- * standalone pass within a single session.
- */
-export function once(pi: object, key: string, fn: () => void): void {
-	const g = globalThis as { __pixOnce?: WeakMap<object, Set<string>> };
-	if (!g.__pixOnce) g.__pixOnce = new WeakMap<object, Set<string>>();
-	const registry = g.__pixOnce;
-	let loaded = registry.get(pi);
-	if (!loaded) {
-		loaded = new Set<string>();
-		registry.set(pi, loaded);
-	}
-	if (loaded.has(key)) return;
-	loaded.add(key);
-	fn();
-}