@xynogen/pix-find 0.1.11 → 0.1.13

package/package.json

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

package/src/extension.ts

@@ -4,11 +4,7 @@ import {
 	type FindToolInput,
 } from "@earendil-works/pi-coding-agent";
 import { CursorStore, fffState } from "@xynogen/pix-pretty/fff";
-import type {
-	PiPrettyApi,
-	TextComponentCtor,
-	ToolFactory,
-} from "@xynogen/pix-pretty/types";
+import type { PiPrettyApi, TextComponentCtor, ToolFactory } from "@xynogen/pix-pretty/types";
 import { shortPath } from "@xynogen/pix-pretty/utils";
 import { registerFindTool } from "./find.js";
 import { once } from "./once.ts";

package/src/find.test.ts

@@ -1,7 +1,7 @@
 import { describe, expect, it } from "bun:test";
 import type { CursorStore, FffState } from "@xynogen/pix-pretty/fff";
 import type { PiPrettyApi, TextComponentCtor } from "@xynogen/pix-pretty/types";
-import { registerFindTool } from "./find";
+import { applyFindDefaults, DEFAULT_FIND_LIMIT, registerFindTool } from "./find";
 
 class MockTextComponent {
 	private text = "";
@@ -13,6 +13,19 @@ class MockTextComponent {
 	}
 }
 
+describe("applyFindDefaults", () => {
+	it("applies a conservative default without overriding an explicit limit", () => {
+		expect(applyFindDefaults({ pattern: "**/*.ts" })).toEqual({
+			pattern: "**/*.ts",
+			limit: DEFAULT_FIND_LIMIT,
+		});
+		expect(applyFindDefaults({ pattern: "**/*.ts", limit: 8 })).toEqual({
+			pattern: "**/*.ts",
+			limit: 8,
+		});
+	});
+});
+
 describe("registerFindTool", () => {
 	it("registers a tool named 'find'", () => {
 		const tools: string[] = [];

package/src/find.ts

@@ -24,6 +24,12 @@ import {
 	setResultDetails,
 } from "@xynogen/pix-pretty/utils";
 
+export const DEFAULT_FIND_LIMIT = 200;
+
+export function applyFindDefaults(params: FindParams): FindParams {
+	return params.limit === undefined ? { ...params, limit: DEFAULT_FIND_LIMIT } : params;
+}
+
 export function registerFindTool(
 	pi: PiPrettyApi,
 	createFindTool: ToolFactory<FindToolInput>,
@@ -35,6 +41,8 @@ export function registerFindTool(
 	pi.registerTool({
 		...origFind,
 		name: "find",
+		description:
+			"Find files by glob pattern. Defaults to 200 paths; use limit to request more. Respects .gitignore and remains capped by Pi's 50KB hard limit.",
 		renderShell: "self",
 
 		async execute(
@@ -44,12 +52,14 @@ export function registerFindTool(
 			upd: unknown,
 			toolCtx: ExtensionContext,
 		) {
+			const effectiveParams = applyFindDefaults(params);
+
 			// Try FFF first (frecency-ranked, SIMD-accelerated)
 			if (fffState.finder && !fffState.finder.isDestroyed) {
 				try {
-					const effectiveLimit = Math.max(1, params.limit ?? 200);
-					let query = params.pattern;
-					if (params.path) query = `${params.path} ${query}`;
+					const effectiveLimit = Math.max(1, effectiveParams.limit ?? DEFAULT_FIND_LIMIT);
+					let query = effectiveParams.pattern;
+					if (effectiveParams.path) query = `${effectiveParams.path} ${query}`;
 
 					const searchResult = fffState.finder.fileSearch(query, {
 						pageSize: effectiveLimit,
@@ -58,12 +68,9 @@ export function registerFindTool(
 						const { items, totalMatched } = searchResult.value;
 						const trimmed = items.slice(0, effectiveLimit);
 						const notices: string[] = [];
-						if (fffState.partialIndex)
-							notices.push("Warning: partial file index");
-						if (trimmed.length >= effectiveLimit)
-							notices.push(`${effectiveLimit} limit reached`);
-						if (totalMatched > trimmed.length)
-							notices.push(`${totalMatched} total matches`);
+						if (fffState.partialIndex) notices.push("Warning: partial file index");
+						if (trimmed.length >= effectiveLimit) notices.push(`${effectiveLimit} limit reached`);
+						if (totalMatched > trimmed.length) notices.push(`${totalMatched} total matches`);
 
 						const textContent = appendNotices(
 							trimmed.map((item) => item.relativePath).join("\n"),
@@ -72,7 +79,7 @@ export function registerFindTool(
 						return makeTextResult<FindResultDetails>(textContent, {
 							_type: "findResult",
 							text: textContent,
-							pattern: params.pattern,
+							pattern: effectiveParams.pattern,
 							matchCount: trimmed.length,
 						});
 					}
@@ -82,17 +89,9 @@ export function registerFindTool(
 			}
 
 			// SDK fallback
-			const result = await origFind.execute(
-				tid,
-				params,
-				sig,
-				upd as never,
-				toolCtx,
-			);
+			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;
+			const matchCount = textContent ? textContent.trim().split("\n").filter(Boolean).length : 0;
 
 			setResultDetails<FindResultDetails>(result, {
 				_type: "findResult",
@@ -104,15 +103,9 @@ export function registerFindTool(
 			return result;
 		},
 
-		renderCall(
-			args: FindParams,
-			theme: ThemeLike,
-			renderCtx: RenderContextLike,
-		) {
+		renderCall(args: FindParams, theme: ThemeLike, renderCtx: RenderContextLike) {
 			const pattern = args.pattern ?? "";
-			const path = args.path
-				? ` ${theme.fg("muted", `in ${sp(args.path)}`)}`
-				: "";
+			const path = args.path ? ` ${theme.fg("muted", `in ${sp(args.path)}`)}` : "";
 			const text = renderCtx.lastComponent ?? new TextComponent("", 0, 0);
 			text.setText(
 				fillToolBackground(
@@ -140,9 +133,7 @@ export function registerFindTool(
 			if (tickCollapse("find", cs, renderCtx.invalidate)) {
 				const d = result.details;
 				const summary =
-					d?._type === "findResult" && d.matchCount != null
-						? `${d.matchCount} files`
-						: "found";
+					d?._type === "findResult" && d.matchCount != null ? `${d.matchCount} files` : "found";
 				text.setText(fillToolBackground(`  ${theme.fg("muted", summary)}`));
 				return text;
 			}