@xynogen/pix-find 0.2.1 → 0.2.3

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-find",
-	"version": "0.2.1",
+	"version": "0.2.3",
 	"description": "Pi tool — glob file search",
 	"type": "module",
 	"main": "src/index.ts",
@@ -38,7 +38,7 @@
 		"access": "public"
 	},
 	"dependencies": {
-		"@xynogen/pix-pretty": "^1.11.2",
+		"@xynogen/pix-pretty": "^1.19.0",
 		"@xynogen/pix-runtime": "^0.8.0"
 	},
 	"peerDependencies": {

package/src/find.test.ts

@@ -33,6 +33,10 @@ class MockTextComponent {
 	getText() {
 		return this.text;
 	}
+	render(_width?: number) {
+		return this.text.split("\n");
+	}
+	invalidate() {}
 }
 
 describe("applyFindDefaults", () => {
@@ -233,17 +237,18 @@ describe("registerFindTool", () => {
 				matchCount: 0,
 			},
 		};
-		const render = (state: Record<string, unknown>, expanded = false) =>
-			registered
-				.renderResult?.(result, { isPartial: false }, theme, {
-					expanded,
-					isError: true,
-					invalidate: () => {},
-					state,
-				} as unknown as RenderContextLike)
-				?.getText() ?? "";
+		const render = (state: Record<string, unknown>, expanded = false) => {
+			const component = registered.renderResult?.(result, { isPartial: false }, theme, {
+				expanded,
+				isError: true,
+				invalidate: () => {},
+				state,
+			} as unknown as RenderContextLike);
+			return component?.render(120).join("\n") ?? "";
+		};
 
 		expect(render({ timer: 1 })).toContain(diagnostic);
+		expect(render({ timer: 1 })).toContain("─");
 		expect(render({ collapsed: true })).toContain("✗  find [ in src · failed");
 		expect(render({ collapsed: true }, true)).toContain(diagnostic);
 	});

package/src/find.ts

@@ -17,6 +17,7 @@ import type {
 import {
 	appendNotices,
 	fillToolBackground,
+	frameToolResult,
 	getTextContent,
 	hideCollapsedToolCall,
 	makeTextResult,
@@ -25,6 +26,7 @@ import {
 	renderDimPreview,
 	renderToolError,
 	setResultDetails,
+	unframeToolResult,
 } from "@xynogen/pix-pretty/utils";
 import { type CollapseState, tickCollapse } from "@xynogen/pix-runtime/collapse";
 
@@ -157,7 +159,7 @@ export function registerFindTool(
 				return text;
 			text.setText(
 				fillToolBackground(
-					`${theme.fg("toolTitle", theme.bold("find"))} ${theme.fg("muted", pattern)}${path}`,
+					`${theme.fg("toolTitle", theme.bold("find"))} ${theme.fg("dim", pattern)}${path}`,
 				),
 			);
 			return text;
@@ -170,14 +172,15 @@ export function registerFindTool(
 			renderCtx: RenderContextLike,
 		) {
 			resolveBaseBackground(theme);
-			const text = renderCtx.lastComponent ?? new TextComponent("", 0, 0);
+			const text = unframeToolResult(renderCtx.lastComponent ?? new TextComponent("", 0, 0));
 			const d = result.details;
 			const isPartial = _opt?.isPartial === true;
+			const completed = () => frameToolResult(text, theme, renderCtx.isError);
 			const structuredError = renderCtx.isError && d?._type === "findResult";
 
 			if (renderCtx.isError && (!structuredError || isPartial)) {
 				text.setText(renderToolError(getTextContent(result) || "Error", theme));
-				return text;
+				return isPartial ? text : completed();
 			}
 
 			// Auto-collapse: show summary line after delay
@@ -203,14 +206,14 @@ export function registerFindTool(
 
 			if (renderCtx.isError) {
 				text.setText(renderToolError(getTextContent(result) || "Error", theme));
-				return text;
+				return completed();
 			}
 
 			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,
+					frame: !isPartial,
 					paint: (s: string) => theme.fg(renderCtx.isError ? "error" : "success", s),
 					highlight: d?._type === "findResult" ? globHighlight(d.pattern) : undefined,
 				}),