@xynogen/pix-find 0.1.12 → 0.1.13

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-find",
-	"version": "0.1.12",
+	"version": "0.1.13",
 	"description": "Pi tool — glob file search",
 	"type": "module",
 	"main": "src/index.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,
@@ -69,7 +79,7 @@ export function registerFindTool(
 						return makeTextResult<FindResultDetails>(textContent, {
 							_type: "findResult",
 							text: textContent,
-							pattern: params.pattern,
+							pattern: effectiveParams.pattern,
 							matchCount: trimmed.length,
 						});
 					}
@@ -79,7 +89,7 @@ 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;