@zhushanwen/pi-todo 0.8.7 → 0.8.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zhushanwen/pi-todo",
-  "version": "0.8.7",
+  "version": "0.8.8",
   "description": "AI-driven todo list for Pi — stateful task management with session persistence and /todos command.",
   "type": "module",
   "main": "src/index.ts",
@@ -29,8 +29,8 @@
     "vitest": "^4.1.8"
   },
   "dependencies": {
-    "@xyz-agent/extension-protocol": "0.8.0",
-    "@zhushanwen/pi-extension-logger": "0.4.0"
+    "@xyz-agent/extension-protocol": "0.8.1",
+    "@zhushanwen/pi-extension-logger": "0.4.1"
   },
   "peerDependencies": {
     "@earendil-works/pi-coding-agent": "^0.84.4",

package/src/__tests__/render.test.ts

@@ -0,0 +1,102 @@
+import type { Theme } from "@earendil-works/pi-coding-agent";
+import { describe, expect, it } from "vitest";
+
+import type { Todo, TodoDetails } from "../model";
+import { renderTodoResult } from "../render";
+
+// ── mock theme（fg 直通，便于断言纯文本）────────────
+
+const mockTheme = {
+	fg: (_color: string, text: string) => text,
+	bg: (_color: string, text: string) => text,
+	bold: (text: string) => text,
+	italic: (text: string) => text,
+	underline: (text: string) => text,
+	inverse: (text: string) => text,
+	strikethrough: (text: string) => text,
+	getFgAnsi: (_color: string) => "",
+	getBgAnsi: (_color: string) => "",
+	getColorMode: () => "truecolor" as const,
+	getThinkingBorderColor: () => (text: string) => text,
+	getBashModeBorderColor: () => (text: string) => text,
+} as unknown as Theme;
+
+// ── fixture ─────────────────────────────────────────
+
+const todos: Todo[] = [
+	{ id: 1, text: "Write docs", status: "completed" },
+	{ id: 2, text: "Ship it", status: "in_progress" },
+];
+
+const listDetails: TodoDetails = { action: "list", todos, nextId: 3 };
+
+/** Text 实例的渲染行（width 足够宽避免 wrap；每行被 pad，断言用 toContain） */
+function renderedText(result: unknown, expanded = false): string {
+	return renderTodoResult(result, { expanded }, mockTheme).render(400).join("\n");
+}
+
+// ── renderTodoResult ────────────────────────────────
+
+describe("renderTodoResult", () => {
+	it("无 details → 渲染 content[0] 的 text", () => {
+		const result = { content: [{ type: "text", text: "Unknown payload" }] };
+		expect(renderedText(result)).toContain("Unknown payload");
+	});
+
+	it("无 details 且 content[0] 非 text 类型 → 空文本（渲染为空）", () => {
+		const result = { content: [{ type: "image", text: "ignored" }] };
+		expect(renderTodoResult(result, { expanded: false }, mockTheme).render(400)).toEqual([]);
+	});
+
+	it("无 details 且 content[0].text 缺失 → 空文本", () => {
+		const result = { content: [{ type: "text" }] };
+		expect(renderTodoResult(result, { expanded: false }, mockTheme).render(400)).toEqual([]);
+	});
+
+	it("action=list → 渲染全量列表（标题 + 每条 mark/#id/文本）", () => {
+		const result = { content: [{ type: "text", text: "ignored" }], details: listDetails };
+		const out = renderedText(result);
+		expect(out).toContain("2 todos:");
+		expect(out).toContain("✓ #1 Write docs");
+		expect(out).toContain("● #2 Ship it");
+	});
+
+	it.each(["add", "update", "delete"] as const)(
+		"action=%s → 确认消息 + 空行 + 全量列表",
+		(action) => {
+			const result = {
+				content: [{ type: "text", text: "Done thing" }],
+				details: { action, todos, nextId: 3 },
+			};
+			const out = renderedText(result);
+			expect(out).toContain("✓ Done thing");
+			expect(out).toContain("2 todos:");
+			expect(out).toContain("#1 Write docs");
+		},
+	);
+
+	it("action=delete 时 details 有 text 缺失 → 确认段为空串，列表仍在", () => {
+		const result = {
+			content: [{ type: "text" }],
+			details: { action: "delete", todos, nextId: 3 },
+		};
+		const out = renderedText(result);
+		expect(out).toContain("2 todos:");
+	});
+
+	it("未知 action + 有消息 → dim 渲染消息原文", () => {
+		const result = {
+			content: [{ type: "text", text: "Archived" }],
+			details: { action: "archive", todos: [], nextId: 1 } as unknown as TodoDetails,
+		};
+		expect(renderedText(result)).toContain("Archived");
+	});
+
+	it("未知 action + 无消息 → 兜底 Done", () => {
+		const result = {
+			content: [{ type: "text" }],
+			details: { action: "archive", todos: [], nextId: 1 } as unknown as TodoDetails,
+		};
+		expect(renderedText(result)).toContain("Done");
+	});
+});

package/src/render.ts

@@ -161,14 +161,28 @@ function buildTodoListText(todoList: Todo[], options: { expanded: boolean }, the
 
 import { Text } from "@earendil-works/pi-tui";
 
+/** content[0] 提取 text（缺失/非 text 类型 → 空串） */
+function firstContentText(r: { content: Array<{ type: string; text?: string }> }): string {
+	const text = r.content[0];
+	return text?.type === "text" ? (text.text ?? "") : "";
+}
+
 export function renderTodoResult(result: unknown, options: { expanded: boolean }, theme: Theme): Text {
 	const r = result as { content: Array<{ type: string; text?: string }>; details?: unknown };
 	const details = r.details as TodoDetails | undefined;
 	if (!details) {
-		const text = r.content[0];
-		return new Text(text?.type === "text" ? (text.text ?? "") : "", 0, 0);
+		return new Text(firstContentText(r), 0, 0);
 	}
+	return renderDetailedTodoResult(r, options, theme, details);
+}
 
+/** 有 details 时按 action 分发渲染（switch 每个 case 体独立成函数） */
+function renderDetailedTodoResult(
+	r: { content: Array<{ type: string; text?: string }> },
+	options: { expanded: boolean },
+	theme: Theme,
+	details: TodoDetails,
+): Text {
 	const todoList = details.todos;
 
 	switch (details.action) {
@@ -179,20 +193,36 @@ export function renderTodoResult(result: unknown, options: { expanded: boolean }
 		case "add":
 		case "update":
 		case "delete": {
-			const text = r.content[0];
-			const msg = text?.type === "text" ? (text.text ?? "") : "";
-			const listText = buildTodoListText(todoList, options, theme);
-			return new Text(
-				theme.fg("success", "\u2713 ") + theme.fg("muted", msg) + "\n\n" + listText,
-				0,
-				0,
-			);
+			return renderMutationTodoResult(r, options, theme, todoList);
 		}
 
 		default: {
-			const text = r.content[0];
-			const msg = text?.type === "text" ? (text.text ?? "") : "";
-			return new Text(theme.fg("dim", msg || "Done"), 0, 0);
+			return renderFallbackTodoResult(r, theme);
 		}
 	}
 }
+
+/** add/update/delete：确认消息 + 空行 + 全量列表 */
+function renderMutationTodoResult(
+	r: { content: Array<{ type: string; text?: string }> },
+	options: { expanded: boolean },
+	theme: Theme,
+	todoList: Todo[],
+): Text {
+	const msg = firstContentText(r);
+	const listText = buildTodoListText(todoList, options, theme);
+	return new Text(
+		theme.fg("success", "\u2713 ") + theme.fg("muted", msg) + "\n\n" + listText,
+		0,
+		0,
+	);
+}
+
+/** 其余/未知 action：dim 文案，空文案兜底 "Done" */
+function renderFallbackTodoResult(
+	r: { content: Array<{ type: string; text?: string }> },
+	theme: Theme,
+): Text {
+	const msg = firstContentText(r);
+	return new Text(theme.fg("dim", msg || "Done"), 0, 0);
+}