@xynogen/pix-todo 0.1.12 → 0.1.13

package/package.json

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

package/src/todo.test.ts

@@ -26,6 +26,7 @@ function makeHost(
 		data?: unknown;
 	}> = [],
 ) {
+	let capturedParameters: unknown;
 	let capturedExecute:
 		| ((
 				id: string,
@@ -50,9 +51,11 @@ function makeHost(
 	const pi = {
 		registerTool(def: {
 			name: string;
+			parameters: unknown;
 			execute: typeof capturedExecute;
 			renderResult?: typeof capturedRender;
 		}) {
+			capturedParameters = def.parameters;
 			capturedExecute = def.execute;
 			if (def.renderResult) capturedRender = def.renderResult;
 		},
@@ -82,6 +85,10 @@ function makeHost(
 	return {
 		pi,
 		sessionManager,
+		get parameters() {
+			if (!capturedParameters) throw new Error("parameters not captured");
+			return capturedParameters;
+		},
 		get execute() {
 			if (!capturedExecute) throw new Error("execute not captured");
 			return capturedExecute;
@@ -119,6 +126,30 @@ function text(result: { content: Array<{ type: string; text: string }> }) {
 	return result.content.map((c) => c.text).join("\n");
 }
 
+// ─── Tool schema ────────────────────────────────────────────────────────────
+
+test("todo exposes action and status as guided string enums", () => {
+	const host = makeHost();
+	registerTodo(host.pi);
+	const schema = host.parameters as {
+		properties: {
+			action: { type?: string; enum?: string[]; description?: string };
+			status: { type?: string; enum?: string[]; description?: string };
+		};
+	};
+	const action = schema.properties.action;
+	const status = schema.properties.status;
+
+	expect(action.type).toBe("string");
+	expect(action.enum).toEqual(["list", "set", "add", "update", "clear"]);
+	expect(action.description).toContain('"list" shows items');
+	expect(action.description).toContain('"update" changes one item by id');
+	expect(status?.type).toBe("string");
+	expect(status?.enum).toEqual(["pending", "in_progress", "done", "blocked"]);
+	expect(status?.description).toContain('"pending" = not started');
+	expect(status?.description).toContain('"blocked" = cannot proceed');
+});
+
 // ─── parseItems (via set/add) ───────────────────────────────────────────────
 
 describe("todo actions", () => {

package/src/todo.ts

@@ -178,16 +178,11 @@ export default function registerTodo(pi: ExtensionAPI): void {
 				"Call `todo(action:'list')` to recover your place after long runs or context compaction.",
 			],
 			parameters: Type.Object({
-				action: Type.Union(
-					[
-						Type.Literal("list"),
-						Type.Literal("set"),
-						Type.Literal("add"),
-						Type.Literal("update"),
-						Type.Literal("clear"),
-					],
-					{ description: "Operation to perform" },
-				),
+				action: Type.Enum(["list", "set", "add", "update", "clear"] as const, {
+					type: "string",
+					description:
+						'Required operation: "list" shows items; "set" replaces all from items; "add" appends items; "update" changes one item by id; "clear" removes all.',
+				}),
 				items: Type.Optional(
 					Type.String({
 						description: "For set/add: newline-separated or numbered list of todo texts.",
@@ -195,15 +190,11 @@ export default function registerTodo(pi: ExtensionAPI): void {
 				),
 				id: Type.Optional(Type.Number({ description: "For update: target todo id." })),
 				status: Type.Optional(
-					Type.Union(
-						[
-							Type.Literal("pending"),
-							Type.Literal("in_progress"),
-							Type.Literal("done"),
-							Type.Literal("blocked"),
-						],
-						{ description: "For update: new status." },
-					),
+					Type.Enum(["pending", "in_progress", "done", "blocked"] as const, {
+						type: "string",
+						description:
+							'For update: "pending" = not started; "in_progress" = active; "done" = finished; "blocked" = cannot proceed.',
+					}),
 				),
 				text: Type.Optional(
 					Type.String({