@empiricalrun/test-gen 0.56.3 → 0.56.4

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # @empiricalrun/test-gen
 
+## 0.56.4
+
+### Patch Changes
+
+- 0d53865: fix: browser agent tool call has different exit code on linux
+- 8e5dcd2: feat: move to ripgrep for platform independent grep tool calls
+
 ## 0.56.3
 
 ### Patch Changes

package/dist/agent/chat/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/agent/chat/index.ts"],"names":[],"mappings":"AAaA,OAAO,EAAoB,mBAAmB,EAAE,MAAM,SAAS,CAAC;AAiBhE,wBAAsB,kBAAkB,CAAC,EACvC,mBAAmB,EACnB,aAAa,EACb,oBAAoB,GACrB,EAAE;IACD,aAAa,EAAE,mBAAmB,CAAC;IACnC,mBAAmB,EAAE,OAAO,CAAC;IAC7B,oBAAoB,EAAE,MAAM,GAAG,SAAS,CAAC;CAC1C,iBAsFA;AAuBD,wBAAsB,wBAAwB,CAAC,EAC7C,aAAa,EACb,aAAa,GACd,EAAE;IACD,aAAa,EAAE,mBAAmB,CAAC;IACnC,aAAa,EAAE,MAAM,CAAC;CACvB,iBA2CA"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/agent/chat/index.ts"],"names":[],"mappings":"AAaA,OAAO,EAAoB,mBAAmB,EAAE,MAAM,SAAS,CAAC;AAiBhE,wBAAsB,kBAAkB,CAAC,EACvC,mBAAmB,EACnB,aAAa,EACb,oBAAoB,GACrB,EAAE;IACD,aAAa,EAAE,mBAAmB,CAAC;IACnC,mBAAmB,EAAE,OAAO,CAAC;IAC7B,oBAAoB,EAAE,MAAM,GAAG,SAAS,CAAC;CAC1C,iBAsFA;AA0BD,wBAAsB,wBAAwB,CAAC,EAC7C,aAAa,EACb,aAAa,GACd,EAAE;IACD,aAAa,EAAE,mBAAmB,CAAC;IACnC,aAAa,EAAE,MAAM,CAAC;CACvB,iBA2CA"}

package/dist/agent/chat/index.js

@@ -112,6 +112,9 @@ async function getChatSessionFromDashboard(chatSessionId) {
             Authorization: `Bearer ${process.env.EMPIRICALRUN_API_KEY}`,
         },
     });
+    if (!response.ok) {
+        throw new Error(`Failed to get chat session: ${response.statusText}`);
+    }
     const data = await response.json();
     return data.data.chat_session;
 }

package/dist/tools/{grep.d.ts → grep/index.d.ts}

@@ -1,3 +1,3 @@
 import type { Tool } from "@empiricalrun/llm/chat";
 export declare const grepTool: Tool;
-//# sourceMappingURL=grep.d.ts.map
+//# sourceMappingURL=index.d.ts.map

package/dist/tools/grep/index.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/tools/grep/index.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAc,MAAM,wBAAwB,CAAC;AAoH/D,eAAO,MAAM,QAAQ,EAAE,IAkBtB,CAAC"}

package/dist/tools/grep/index.js

@@ -0,0 +1,128 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.grepTool = void 0;
+const child_process_1 = require("child_process");
+const path_1 = __importDefault(require("path"));
+const util_1 = require("util");
+const zod_1 = require("zod");
+const repo_tree_1 = require("../../utils/repo-tree");
+const ripgrep_1 = require("./ripgrep");
+const GrepInputSchema = zod_1.z.object({
+    pattern: zod_1.z.string().describe("The pattern to search for"),
+    directory: zod_1.z
+        .string()
+        .optional()
+        .describe("The directory to search in (defaults to current directory)"),
+    filePattern: zod_1.z
+        .string()
+        .optional()
+        .describe("File pattern to search in (e.g., '*.ts' for TypeScript files)"),
+});
+async function usingSystemGrep(input) {
+    try {
+        const dir = input.directory || process.cwd();
+        // Create exclude pattern for grep
+        const excludePatterns = repo_tree_1.DEFAULT_EXCLUDE.map((pattern) => typeof pattern === "string" ? pattern : pattern.source)
+            .map((pattern) => `--exclude-dir="${pattern}"`)
+            .join(" ");
+        // Using -n to show line numbers in output
+        let cmd = `grep -rin ${excludePatterns} "${input.pattern}" ${dir}`;
+        if (input.filePattern) {
+            // For file pattern searches, we'll use find with exclusions
+            const excludeFind = repo_tree_1.DEFAULT_EXCLUDE.map((pattern) => typeof pattern === "string" ? pattern : pattern.source)
+                .map((pattern) => `-not -path "*/${pattern}/*"`)
+                .join(" ");
+            // Modified command to ensure filepath is included in the output
+            // Using grep with -H to always show filename and the filepath as part of the output
+            cmd = `find ${dir} ${excludeFind} -name "${input.filePattern}" | xargs grep -Hn "${input.pattern}"`;
+        }
+        const execAsync = (0, util_1.promisify)(child_process_1.exec);
+        const { stdout, stderr } = await execAsync(cmd);
+        if (stdout) {
+            return {
+                isError: false,
+                result: stdout,
+            };
+        }
+        else if (!stdout && stderr) {
+            return {
+                isError: true,
+                result: stderr,
+            };
+        }
+        else {
+            // Both are empty, which means no results were found
+            return {
+                isError: false,
+                result: "No results found.",
+            };
+        }
+    }
+    catch (error) {
+        console.error("Error executing grep", error);
+        return {
+            isError: true,
+            result: error instanceof Error ? error.message : String(error),
+        };
+    }
+}
+async function usingRipgrep(input) {
+    try {
+        const dir = path_1.default.join(process.cwd(), input.directory || "");
+        const results = await (0, ripgrep_1.ripgrep)(dir, {
+            string: input.pattern,
+            globs: input.filePattern ? [input.filePattern] : undefined,
+        });
+        const resultsSummary = results
+            .map((result) => {
+            // Can add submatches and offset info to the summary if needed
+            return {
+                lines: result.lines.text,
+                path: path_1.default.relative(process.cwd(), result.path.text),
+                // line number is 1-indexed
+                line_number: result.line_number,
+            };
+        })
+            .map((result) => {
+            return `
+${result.path}:${result.line_number}
+\`\`\`
+${result.lines}\`\`\`
+`;
+        });
+        const relDir = path_1.default.relative(process.cwd(), dir);
+        const header = `Found ${resultsSummary.length} results for "${input.pattern}" in "${relDir}".
+All paths are relative to the current working directory.`;
+        return {
+            isError: false,
+            result: `${header}\n${resultsSummary.join("\n")}`,
+        };
+    }
+    catch (error) {
+        console.error("Error executing ripgrep", error);
+        return {
+            isError: true,
+            result: error instanceof Error ? error.message : String(error),
+        };
+    }
+}
+exports.grepTool = {
+    schema: {
+        name: "grep",
+        description: "Search for a pattern in files using grep (case insensitive)",
+        parameters: GrepInputSchema,
+    },
+    execute: async (input) => {
+        const isAvailable = await (0, ripgrep_1.isRgAvailable)();
+        if (isAvailable) {
+            return usingRipgrep(input);
+        }
+        else {
+            console.warn("ripgrep is not available, falling back to system grep.");
+            return usingSystemGrep(input);
+        }
+    },
+};

package/dist/tools/grep/ripgrep/index.d.ts

@@ -0,0 +1,5 @@
+import { Match, Options } from "./types";
+export * from "./types";
+export declare function isRgAvailable(): Promise<unknown>;
+export declare function ripgrep(cwd: string, options: Options): Promise<Array<Match>>;
+//# sourceMappingURL=index.d.ts.map

package/dist/tools/grep/ripgrep/index.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../src/tools/grep/ripgrep/index.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,KAAK,EAAE,OAAO,EAAgB,MAAM,SAAS,CAAC;AAEvD,cAAc,SAAS,CAAC;AAcxB,wBAAgB,aAAa,qBAY5B;AAED,wBAAgB,OAAO,CAAC,GAAG,EAAE,MAAM,EAAE,OAAO,EAAE,OAAO,GAAG,OAAO,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,CA8C5E"}

package/dist/tools/grep/ripgrep/index.js

@@ -0,0 +1,90 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __exportStar = (this && this.__exportStar) || function(m, exports) {
+    for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.ripgrep = exports.isRgAvailable = void 0;
+const child_process_1 = require("child_process");
+const types_1 = require("./types");
+__exportStar(require("./types"), exports);
+function formatResults(stdout) {
+    stdout = stdout.trim();
+    if (!stdout) {
+        return [];
+    }
+    return stdout
+        .split("\n")
+        .map((line) => JSON.parse(line))
+        .filter((jsonLine) => jsonLine.type === "match")
+        .map((jsonLine) => jsonLine.data);
+}
+function isRgAvailable() {
+    return new Promise((resolve) => {
+        // Use 'where' on Windows and 'which' on Unix-like systems
+        const command = process.platform === "win32" ? "where rg" : "which rg";
+        (0, child_process_1.exec)(command, (error) => {
+            if (error) {
+                resolve(false);
+            }
+            else {
+                resolve(true);
+            }
+        });
+    });
+}
+exports.isRgAvailable = isRgAvailable;
+function ripgrep(cwd, options) {
+    if (!cwd) {
+        return Promise.reject(new Error("No `cwd` provided"));
+    }
+    if (arguments.length === 1) {
+        return Promise.reject(new Error("No search term provided"));
+    }
+    let execString = "rg --json";
+    if ("regex" in options) {
+        execString = `${execString} -e ${options.regex}`;
+    }
+    else if ("string" in options) {
+        execString = `${execString} -F ${options.string}`;
+    }
+    if (options.fileType) {
+        if (!Array.isArray(options.fileType)) {
+            options.fileType = [options.fileType];
+        }
+        for (const fileType of options.fileType) {
+            execString = `${execString} -t ${fileType}`;
+        }
+    }
+    if (options.globs) {
+        execString = options.globs.reduce((command, glob) => {
+            return `${command} -g '${glob}'`;
+        }, execString);
+    }
+    if (options.multiline) {
+        execString = `${execString} --multiline`;
+    }
+    // https://github.com/alexlafroscia/ripgrep-js/pull/175/files
+    execString += ` -- ${cwd}`;
+    return new Promise(function (resolve, reject) {
+        (0, child_process_1.exec)(execString, { cwd }, (error, stdout, stderr) => {
+            if (!error || (error && stderr === "")) {
+                resolve(formatResults(stdout));
+            }
+            else {
+                reject(new types_1.RipGrepError(error, stderr));
+            }
+        });
+    });
+}
+exports.ripgrep = ripgrep;

package/dist/tools/grep/ripgrep/types.d.ts

@@ -0,0 +1,45 @@
+/// <reference types=".pnpm/@types+node@20.14.11/node_modules/@types/node/child_process" />
+/// <reference types=".pnpm/@types+node@20.16.10/node_modules/@types/node/child_process" />
+import { ExecException } from "child_process";
+type StringSearchOptions = {
+    string: string;
+};
+type RegexSearchOptions = {
+    regex: string;
+};
+type LocatorOptions = StringSearchOptions | RegexSearchOptions;
+export type Options = LocatorOptions & {
+    globs?: Array<string>;
+    fileType?: string | Array<string>;
+    multiline?: boolean;
+};
+export type RipgrepJsonSubmatch = {
+    match: {
+        text: string;
+    };
+    start: number;
+    end: number;
+};
+export type RipGrepJsonMatch = {
+    type: "match";
+    data: {
+        path: {
+            text: string;
+        };
+        lines: {
+            text: string;
+        };
+        line_number: number;
+        absolute_offset: number;
+        submatches: Array<RipgrepJsonSubmatch>;
+    };
+};
+export type Match = RipGrepJsonMatch["data"];
+export declare class RipGrepError {
+    private error;
+    stderr: string;
+    constructor(error: ExecException, stderr: string);
+    get message(): string;
+}
+export {};
+//# sourceMappingURL=types.d.ts.map

package/dist/tools/grep/ripgrep/types.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"types.d.ts","sourceRoot":"","sources":["../../../../src/tools/grep/ripgrep/types.ts"],"names":[],"mappings":";;AAAA,OAAO,EAAE,aAAa,EAAE,MAAM,eAAe,CAAC;AAE9C,KAAK,mBAAmB,GAAG;IACzB,MAAM,EAAE,MAAM,CAAC;CAChB,CAAC;AAEF,KAAK,kBAAkB,GAAG;IACxB,KAAK,EAAE,MAAM,CAAC;CACf,CAAC;AAEF,KAAK,cAAc,GAAG,mBAAmB,GAAG,kBAAkB,CAAC;AAE/D,MAAM,MAAM,OAAO,GAAG,cAAc,GAAG;IACrC,KAAK,CAAC,EAAE,KAAK,CAAC,MAAM,CAAC,CAAC;IACtB,QAAQ,CAAC,EAAE,MAAM,GAAG,KAAK,CAAC,MAAM,CAAC,CAAC;IAClC,SAAS,CAAC,EAAE,OAAO,CAAC;CACrB,CAAC;AAEF,MAAM,MAAM,mBAAmB,GAAG;IAChC,KAAK,EAAE;QAAE,IAAI,EAAE,MAAM,CAAA;KAAE,CAAC;IACxB,KAAK,EAAE,MAAM,CAAC;IACd,GAAG,EAAE,MAAM,CAAC;CACb,CAAC;AAEF,MAAM,MAAM,gBAAgB,GAAG;IAC7B,IAAI,EAAE,OAAO,CAAC;IACd,IAAI,EAAE;QACJ,IAAI,EAAE;YACJ,IAAI,EAAE,MAAM,CAAC;SACd,CAAC;QACF,KAAK,EAAE;YACL,IAAI,EAAE,MAAM,CAAC;SACd,CAAC;QACF,WAAW,EAAE,MAAM,CAAC;QACpB,eAAe,EAAE,MAAM,CAAC;QACxB,UAAU,EAAE,KAAK,CAAC,mBAAmB,CAAC,CAAC;KACxC,CAAC;CACH,CAAC;AAEF,MAAM,MAAM,KAAK,GAAG,gBAAgB,CAAC,MAAM,CAAC,CAAC;AAE7C,qBAAa,YAAY;IACvB,OAAO,CAAC,KAAK,CAAgB;IAE7B,MAAM,EAAE,MAAM,CAAC;gBAEH,KAAK,EAAE,aAAa,EAAE,MAAM,EAAE,MAAM;IAKhD,IAAI,OAAO,IAAI,MAAM,CAEpB;CACF"}

package/dist/tools/grep/ripgrep/types.js

@@ -0,0 +1,15 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.RipGrepError = void 0;
+class RipGrepError {
+    error;
+    stderr;
+    constructor(error, stderr) {
+        this.error = error;
+        this.stderr = stderr;
+    }
+    get message() {
+        return this.error.message;
+    }
+}
+exports.RipGrepError = RipGrepError;

package/dist/utils/exec.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"exec.d.ts","sourceRoot":"","sources":["../../src/utils/exec.ts"],"names":[],"mappings":"AAQA,qBAAa,cAAc;IACzB,OAAO,CAAC,YAAY,CAA6B;IAE3C,OAAO,CACX,OAAO,EAAE,MAAM,EAAE,EACjB,OAAO,EAAE;QAAE,GAAG,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAA;KAAE,GACxC,OAAO,CAAC,MAAM,CAAC;IAkDlB,SAAS,IAAI,IAAI;IASjB,SAAS,IAAI,OAAO;CAGrB;AAED,wBAAsB,GAAG,CACvB,OAAO,EAAE,MAAM,EAAE,EACjB,OAAO,EAAE;IAAE,GAAG,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAA;CAAE,GACxC,OAAO,CAAC,MAAM,CAAC,CAGjB"}
+{"version":3,"file":"exec.d.ts","sourceRoot":"","sources":["../../src/utils/exec.ts"],"names":[],"mappings":"AAmBA,qBAAa,cAAc;IACzB,OAAO,CAAC,YAAY,CAA6B;IAE3C,OAAO,CACX,OAAO,EAAE,MAAM,EAAE,EACjB,OAAO,EAAE;QAAE,GAAG,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAA;KAAE,GACxC,OAAO,CAAC,MAAM,CAAC;IAkDlB,SAAS,IAAI,IAAI;IASjB,SAAS,IAAI,OAAO;CAGrB;AAED,wBAAsB,GAAG,CACvB,OAAO,EAAE,MAAM,EAAE,EACjB,OAAO,EAAE;IAAE,GAAG,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAA;CAAE,GACxC,OAAO,CAAC,MAAM,CAAC,CAGjB"}

package/dist/utils/exec.js

@@ -6,10 +6,18 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.cmd = exports.ProcessManager = void 0;
 const child_process_1 = require("child_process");
 const process_1 = __importDefault(require("process"));
-const acceptableExitCodes = [
-    0, // Implies successful execution (no errors)
-    130, // Implies user interrupted the process (for SIGINT)
-];
+function isAcceptableExit(exitCode, signal) {
+    // On Linux, code is null and signal is SIGINT
+    if (signal === "SIGINT") {
+        return true;
+    }
+    // On macOS, signal is null and exit code is 130
+    const acceptableExitCodes = [
+        0, // Implies successful execution (no errors)
+        130, // Implies user interrupted the process (for SIGINT)
+    ];
+    return acceptableExitCodes.includes(exitCode ?? 1);
+}
 class ProcessManager {
     childProcess = null;
     async execute(command, options) {
@@ -43,14 +51,14 @@ class ProcessManager {
                 this.childProcess = null;
                 rejectFunc(err);
             });
-            p.on("exit", async (code) => {
+            p.on("exit", async (code, signal) => {
                 this.childProcess = null;
-                if (!acceptableExitCodes.includes(code ?? 1)) {
+                if (!isAcceptableExit(code, signal)) {
                     const errorMessage = errorLogs.slice(-3).join("\n");
                     rejectFunc(new Error(errorMessage));
                 }
                 else {
-                    resolveFunc(code ?? 1);
+                    resolveFunc(0);
                 }
             });
         });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@empiricalrun/test-gen",
-  "version": "0.56.3",
+  "version": "0.56.4",
   "publishConfig": {
     "registry": "https://registry.npmjs.org/",
     "access": "public"

package/dist/tools/grep.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"grep.d.ts","sourceRoot":"","sources":["../../src/tools/grep.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAc,MAAM,wBAAwB,CAAC;AAmB/D,eAAO,MAAM,QAAQ,EAAE,IA0DtB,CAAC"}

package/dist/tools/grep.js

@@ -1,73 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.grepTool = void 0;
-const child_process_1 = require("child_process");
-const util_1 = require("util");
-const zod_1 = require("zod");
-const repo_tree_1 = require("../utils/repo-tree");
-const GrepInputSchema = zod_1.z.object({
-    pattern: zod_1.z.string().describe("The pattern to search for"),
-    directory: zod_1.z
-        .string()
-        .optional()
-        .describe("The directory to search in (defaults to current directory)"),
-    filePattern: zod_1.z
-        .string()
-        .optional()
-        .describe("File pattern to search in (e.g., '*.ts' for TypeScript files)"),
-});
-exports.grepTool = {
-    schema: {
-        name: "grep",
-        description: "Search for a pattern in files using grep (case insensitive)",
-        parameters: GrepInputSchema,
-    },
-    execute: async (input) => {
-        try {
-            const dir = input.directory || process.cwd();
-            // Create exclude pattern for grep
-            const excludePatterns = repo_tree_1.DEFAULT_EXCLUDE.map((pattern) => typeof pattern === "string" ? pattern : pattern.source)
-                .map((pattern) => `--exclude-dir="${pattern}"`)
-                .join(" ");
-            // Using -n to show line numbers in output
-            let cmd = `grep -rin ${excludePatterns} "${input.pattern}" ${dir}`;
-            if (input.filePattern) {
-                // For file pattern searches, we'll use find with exclusions
-                const excludeFind = repo_tree_1.DEFAULT_EXCLUDE.map((pattern) => typeof pattern === "string" ? pattern : pattern.source)
-                    .map((pattern) => `-not -path "*/${pattern}/*"`)
-                    .join(" ");
-                // Modified command to ensure filepath is included in the output
-                // Using grep with -H to always show filename and the filepath as part of the output
-                cmd = `find ${dir} ${excludeFind} -name "${input.filePattern}" | xargs grep -Hn "${input.pattern}"`;
-            }
-            const execAsync = (0, util_1.promisify)(child_process_1.exec);
-            const { stdout, stderr } = await execAsync(cmd);
-            if (stdout) {
-                return {
-                    isError: false,
-                    result: stdout,
-                };
-            }
-            else if (!stdout && stderr) {
-                return {
-                    isError: true,
-                    result: stderr,
-                };
-            }
-            else {
-                // Both are empty, which means no results were found
-                return {
-                    isError: false,
-                    result: "No results found.",
-                };
-            }
-        }
-        catch (error) {
-            console.error("Error executing grep", error);
-            return {
-                isError: true,
-                result: error instanceof Error ? error.message : String(error),
-            };
-        }
-    },
-};