@xynogen/pix-find 0.1.23 → 0.1.24

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-find",
-	"version": "0.1.23",
+	"version": "0.1.24",
 	"description": "Pi tool — glob file search",
 	"type": "module",
 	"main": "src/index.ts",

package/src/find.test.ts

@@ -110,8 +110,73 @@ describe("registerFindTool", () => {
 
 		expect(result?.getText()).toContain("src/one.ts");
 		expect(result?.getText()).toContain("src/two.ts");
-		// Framed view shows the same semantic count as the collapsed row (no drift).
-		expect(result?.getText()).toContain("2 files");
+		// No floating count header in the framed view — the collapsed row carries it.
+		expect(result?.getText()).not.toContain("2 files");
+	});
+
+	it("frames single-file output like multi-file (no inline row)", () => {
+		const registered: { renderResult?: (...args: unknown[]) => MockTextComponent } = {};
+		const mockPi: PiPrettyApi = {
+			registerTool(tool: unknown) {
+				Object.assign(registered, tool);
+			},
+			registerCommand() {},
+			on() {},
+		};
+		registerFindTool(
+			mockPi,
+			() => ({ execute: async () => ({ content: [], details: undefined }) }),
+			{
+				cwd: process.cwd(),
+				sp: (p: string) => p,
+				TextComponent: MockTextComponent as unknown as TextComponentCtor,
+				fffState: {
+					module: null,
+					finder: null,
+					partialIndex: false,
+					dbDir: null,
+				} satisfies FffState,
+				cursorStore: { store: () => "", get: () => undefined } as unknown as CursorStore,
+			},
+		);
+		const theme: ThemeLike = { fg: (_k: string, v: string) => v, bold: (v: string) => v };
+		const strip = (s: string) => s.replace(/\u001b\[[0-9;]*m/g, "");
+		const result = {
+			content: [{ type: "text", text: "src/a.ts" }],
+			details: { _type: "findResult", text: "src/a.ts", pattern: "*.ts", matchCount: 1 } as never,
+		};
+		const out =
+			registered
+				.renderResult?.(result, { isPartial: false }, theme, {
+					expanded: false,
+					isError: false,
+					invalidate: () => {},
+					state: {},
+				} as unknown as RenderContextLike)
+				?.getText() ?? "";
+		// Single result is framed just like multi — one shape, no inline row.
+		expect(out).toContain("─");
+		expect(out).toContain("src/a.ts");
+		expect(strip(out)).not.toContain("file");
+		const multi = {
+			content: [{ type: "text", text: "a.ts\nb.ts\nc.ts" }],
+			details: {
+				_type: "findResult",
+				text: "a.ts\nb.ts\nc.ts",
+				pattern: "*.ts",
+				matchCount: 3,
+			} as never,
+		};
+		const multiOut =
+			registered
+				.renderResult?.(multi, { isPartial: false }, theme, {
+					expanded: false,
+					isError: false,
+					invalidate: () => {},
+					state: {},
+				} as unknown as RenderContextLike)
+				?.getText() ?? "";
+		expect(multiOut).toContain("─");
 	});
 
 	it("collapses structured errors and restores the exact diagnostic on expansion", () => {
@@ -160,7 +225,7 @@ describe("registerFindTool", () => {
 				?.getText() ?? "";
 
 		expect(render({ timer: 1 })).toContain(diagnostic);
-		expect(render({ collapsed: true })).toContain("✗ find [ in src · failed");
+		expect(render({ collapsed: true })).toContain("✗  find [ in src · failed");
 		expect(render({ collapsed: true }, true)).toContain(diagnostic);
 	});
 });

package/src/find.ts

@@ -94,19 +94,35 @@ export function registerFindTool(
 			}
 
 			// SDK fallback
-			const result = await origFind.execute(tid, effectiveParams, sig, upd as never, toolCtx);
-			const textContent = getTextContent(result);
-			const matchCount = textContent ? textContent.trim().split("\n").filter(Boolean).length : 0;
-
-			setResultDetails<FindResultDetails>(result, {
-				_type: "findResult",
-				text: textContent,
-				pattern: params.pattern,
-				path: params.path,
-				matchCount,
-			});
-
-			return result;
+			try {
+				const result = await origFind.execute(tid, effectiveParams, sig, upd as never, toolCtx);
+				const textContent = getTextContent(result);
+				const matchCount = textContent ? textContent.trim().split("\n").filter(Boolean).length : 0;
+
+				setResultDetails<FindResultDetails>(result, {
+					_type: "findResult",
+					text: textContent,
+					pattern: params.pattern,
+					path: params.path,
+					matchCount,
+				});
+
+				return result;
+			} catch (error) {
+				const text = error instanceof Error ? error.message : String(error);
+				if (sig?.aborted || /aborted/i.test(text)) throw error;
+				return {
+					content: [{ type: "text" as const, text }],
+					details: {
+						_type: "findResult" as const,
+						text,
+						pattern: params.pattern,
+						path: params.path,
+						matchCount: 0,
+					},
+					isError: true,
+				};
+			}
 		},
 
 		renderCall(args: FindParams, theme: ThemeLike, renderCtx: RenderContextLike) {
@@ -172,14 +188,11 @@ export function registerFindTool(
 			}
 
 			const output = getTextContent(result) || "found";
+			// One framed shape; rules follow status color. Count lives in the collapsed row.
 			text.setText(
 				renderDimPreview(output, theme, {
 					frame: true,
-					// Same semantic count the collapsed row shows — one source, no drift.
-					header:
-						d?._type === "findResult" && d.matchCount != null
-							? pluralize(d.matchCount, "file")
-							: undefined,
+					paint: (s: string) => theme.fg(renderCtx.isError ? "error" : "success", s),
 				}),
 			);
 			return text;