feat(cli): filter lines with more than 1000 characters (#8714)

* filter out lines greater than 1000 characters

* add tests

Author: uinstinct

extensions/cli/src/tools/searchCode.test.ts

@@ -1,3 +1,18 @@
+import * as nodeUtil from "util";
+
+import { vi } from "vitest";
+
+const execMock: any = vi.fn();
+(execMock as any)[(nodeUtil as any).promisify.custom] = (cmd: any) =>
+  new Promise((resolve, reject) => {
+    execMock(cmd, (err: any, stdout: any, stderr: any) => {
+      if (err) reject(err);
+      else resolve({ stdout, stderr });
+    });
+  });
+
+vi.mock("child_process", () => ({ exec: execMock }));
+
 // Since we want to test just the interface and not the internals,
 // let's create a simplified version of the run function to test the truncation logic
 describe("searchCodeTool", () => {
@@ -51,4 +66,30 @@ describe("searchCodeTool", () => {
     // Check that we have the expected number of lines (header + 50 content lines)
     expect(nonEmptyLines.length).toBe(51);
   });
+
+  describe("searchCodeTool line-length filtering", () => {
+    afterEach(() => {
+      vi.clearAllMocks();
+      vi.resetModules();
+    });
+
+    it("filters out lines longer than 1000 characters", async () => {
+      const childProc = await import("child_process");
+      const long = "a".repeat(1001);
+
+      vi.mocked(childProc.exec as any).mockImplementation((...args: any[]) => {
+        // exec callback signature: (error, stdout, stderr)
+        const cb = args[args.length - 1];
+        cb(null, `path/file.ts:1:${long}\npath/file.ts:2:match`, "");
+        return {} as any;
+      });
+
+      const { searchCodeTool } = await import("./searchCode.js");
+      const result = await searchCodeTool.run({ pattern: "match", path: "." });
+
+      expect(result).toContain("path/file.ts:2:match");
+      expect(result).not.toContain(long);
+      expect(result).not.toContain("[Results truncated:");
+    });
+  });
 });

extensions/cli/src/tools/searchCode.ts

@@ -10,6 +10,7 @@ const execPromise = util.promisify(child_process.exec);
 
 // Default maximum number of results to display
 const DEFAULT_MAX_RESULTS = 100;
+const MAX_LINE_LENGTH = 1000;
 
 export const searchCodeTool: Tool = {
   name: "Search",
@@ -84,7 +85,13 @@ export const searchCodeTool: Tool = {
       }
 
       // Split the results into lines and limit the number of results
-      const lines = stdout.split("\n");
+      const splitLines = stdout.split("\n");
+      const lines = splitLines.filter((line) => line.length <= MAX_LINE_LENGTH);
+      if (lines.length === 0) {
+        return `No matches found for pattern "${args.pattern}"${
+          args.file_pattern ? ` in files matching "${args.file_pattern}"` : ""
+        }.`;
+      }
       const truncated = lines.length > DEFAULT_MAX_RESULTS;
       const limitedLines = lines.slice(0, DEFAULT_MAX_RESULTS);
       const resultText = limitedLines.join("\n");