@empiricalrun/test-gen 0.76.0 → 0.78.0

package/dist/tools/grep/index.js

@@ -8,7 +8,7 @@ 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 constants_1 = require("../../constants");
 const grep_1 = require("../definitions/grep");
 const ripgrep_1 = require("./ripgrep");
 const GrepInputSchema = zod_1.z.object({
@@ -24,18 +24,18 @@ const GrepInputSchema = zod_1.z.object({
         .optional()
         .describe("File pattern to search in (e.g., '*.ts' for TypeScript files)"),
 });
-async function usingSystemGrep(input, repoPath) {
+async function usingSystemGrep(input, repoPath, logger) {
     try {
         const dir = path_1.default.join(repoPath, input.directory || "");
         // Create exclude pattern for grep
-        const excludePatterns = repo_tree_1.DEFAULT_EXCLUDE.map((pattern) => typeof pattern === "string" ? pattern : pattern.source)
+        const excludePatterns = constants_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)
+            const excludeFind = constants_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
@@ -65,14 +65,15 @@ async function usingSystemGrep(input, repoPath) {
         }
     }
     catch (error) {
-        console.error("Error executing grep", error);
+        logger?.error("Error executing grep", error) ||
+            console.error("Error executing grep", error);
         return {
             isError: true,
             result: error instanceof Error ? error.message : String(error),
         };
     }
 }
-async function usingRipgrep(input, repoPath) {
+async function usingRipgrep(input, repoPath, logger) {
     try {
         const dir = path_1.default.join(repoPath, input.directory || "");
         const escapedPattern = input.pattern
@@ -109,7 +110,8 @@ All paths are relative to the current working directory.`;
         };
     }
     catch (error) {
-        console.error("Error executing ripgrep", error);
+        logger?.error("Error executing ripgrep", error) ||
+            console.error("Error executing ripgrep", error);
         return {
             isError: true,
             result: error instanceof ripgrep_1.RipGrepError ? error.message : String(error),
@@ -118,13 +120,18 @@ All paths are relative to the current working directory.`;
 }
 exports.grepTool = {
     ...grep_1.grepTool,
-    execute: async ({ input, repoPath }) => {
+    execute: async ({ input, repoPath, logger }) => {
         if ((0, ripgrep_1.isRgAvailable)()) {
-            return usingRipgrep(input, repoPath);
+            return usingRipgrep(input, repoPath, logger);
         }
         else {
-            console.warn("ripgrep is not available, falling back to system grep.");
-            return usingSystemGrep(input, repoPath);
+            if (logger) {
+                logger.warn("ripgrep is not available, falling back to system grep.");
+            }
+            else {
+                console.warn("ripgrep is not available, falling back to system grep.");
+            }
+            return usingSystemGrep(input, repoPath, logger);
         }
     },
 };

package/dist/tools/index.d.ts

@@ -1,13 +1,12 @@
 import { PendingToolCall, ServicePayload, SupportedChatModels, Tool, ToolDefinition } from "@empiricalrun/shared-types";
 import { createPullRequestTool } from "./create-pull-request";
+import { analyseVideo } from "./definitions/analyse-video";
 import { deleteFileTool } from "./definitions/delete-file";
 import { downloadBuildTool } from "./definitions/download-build";
-import { extractFramesFromVideo } from "./definitions/extract-frames-from-video";
-import { fetchVideoAnalysis } from "./definitions/fetch-video-analysis";
 import { grepTool } from "./definitions/grep";
-import { listProjectsTool, listTestsForProjectTool } from "./definitions/list-tests-and-projects";
 import { mergeConflictsTool } from "./definitions/merge-conflicts";
 import { renameFileTool } from "./definitions/rename-file";
+import { reviewPullRequestTool } from "./definitions/review-pull-request";
 import { runTestTool } from "./definitions/run-test";
 import { generateTestWithBrowserAgent } from "./definitions/test-gen-browser";
 import { upgradePackagesTool } from "./definitions/upgrade-packages";
@@ -17,7 +16,6 @@ import { fetchLastSuccessfulTestRunTool } from "./fetch-last-successful-test-run
 import { fetchSessionDiffTool } from "./fetch-session-diff";
 import { createIssueTool, listIssuesTool, updateIssueTool } from "./issues";
 import { listEnvironmentsTool } from "./list-environments";
-import { reviewPullRequestTool } from "./review-pull-request";
 import { fetchTestRunReportTool } from "./test-run-fetcher";
 import { traceDotZipTool } from "./trace-dot-zip";
 import { sendTriageSummaryTool } from "./triage-summary";
@@ -26,9 +24,11 @@ type ToolOrToolDefinition = Tool | ToolDefinition;
 export declare const commonTools: ToolOrToolDefinition[];
 export declare const testGenerationTools: ToolOrToolDefinition[];
 export declare const triageOnlyTools: ToolOrToolDefinition[];
-export { createIssueTool, createPullRequestTool, deleteFileTool, downloadBuildTool, extractFramesFromVideo, fetchDiagnosisReportTool, fetchFileTool, fetchLastSuccessfulTestRunTool, fetchSessionDiffTool, fetchTestRunReportTool, fetchVideoAnalysis, generateTestWithBrowserAgent, grepTool, listEnvironmentsTool, listIssuesTool, listProjectsTool, listTestsForProjectTool, mergeConflictsTool, renameFileTool, reviewPullRequestTool, runTestTool, sendTriageSummaryTool, traceDotZipTool, updateIssueTool, upgradePackagesTool, viewFailedTestRunReportTool, };
+export { createIssueTool, createPullRequestTool, deleteFileTool, downloadBuildTool, fetchDiagnosisReportTool, fetchFileTool, fetchLastSuccessfulTestRunTool, fetchSessionDiffTool, fetchTestRunReportTool, analyseVideo, generateTestWithBrowserAgent, grepTool, listEnvironmentsTool, listIssuesTool, mergeConflictsTool, renameFileTool, reviewPullRequestTool, runTestTool, sendTriageSummaryTool, traceDotZipTool, updateIssueTool, upgradePackagesTool, viewFailedTestRunReportTool, };
 export declare const allToolsDefinitions: ToolOrToolDefinition[];
+export declare function hasBuiltInTextEditor(model: SupportedChatModels): boolean;
 export declare function textEditorToolsForModel(model: SupportedChatModels): ToolOrToolDefinition[];
+export declare function textViewToolsForModel(model: SupportedChatModels): ToolOrToolDefinition[];
 export declare function toolsNeedBrowser(toolCalls: PendingToolCall[]): boolean;
 export declare function nameForTelemetry(toolCall: PendingToolCall): string;
 export declare function sendToolRequestToRemoteQueue(payload: ServicePayload): Promise<void>;

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

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/tools/index.ts"],"names":[],"mappings":"AACA,OAAO,EACL,eAAe,EACf,cAAc,EACd,mBAAmB,EACnB,IAAI,EACJ,cAAc,EACf,MAAM,4BAA4B,CAAC;AAGpC,OAAO,EAAE,qBAAqB,EAAE,MAAM,uBAAuB,CAAC;AAC9D,OAAO,EAAE,cAAc,EAAE,MAAM,2BAA2B,CAAC;AAC3D,OAAO,EAAE,iBAAiB,EAAE,MAAM,8BAA8B,CAAC;AACjE,OAAO,EAAE,sBAAsB,EAAE,MAAM,yCAAyC,CAAC;AACjF,OAAO,EAAE,kBAAkB,EAAE,MAAM,oCAAoC,CAAC;AACxE,OAAO,EAAE,QAAQ,EAAE,MAAM,oBAAoB,CAAC;AAC9C,OAAO,EACL,gBAAgB,EAChB,uBAAuB,EACxB,MAAM,uCAAuC,CAAC;AAC/C,OAAO,EAAE,kBAAkB,EAAE,MAAM,+BAA+B,CAAC;AACnE,OAAO,EAAE,cAAc,EAAE,MAAM,2BAA2B,CAAC;AAC3D,OAAO,EAAE,WAAW,EAAE,MAAM,wBAAwB,CAAC;AAErD,OAAO,EAAE,4BAA4B,EAAE,MAAM,gCAAgC,CAAC;AAC9E,OAAO,EAAE,mBAAmB,EAAE,MAAM,gCAAgC,CAAC;AACrE,OAAO,EAAE,wBAAwB,EAAE,MAAM,qBAAqB,CAAC;AAC/D,OAAO,EAAE,aAAa,EAAE,MAAM,cAAc,CAAC;AAC7C,OAAO,EAAE,8BAA8B,EAAE,MAAM,kCAAkC,CAAC;AAClF,OAAO,EAAE,oBAAoB,EAAE,MAAM,sBAAsB,CAAC;AAC5D,OAAO,EAAE,eAAe,EAAE,cAAc,EAAE,eAAe,EAAE,MAAM,UAAU,CAAC;AAC5E,OAAO,EAAE,oBAAoB,EAAE,MAAM,qBAAqB,CAAC;AAC3D,OAAO,EAAE,qBAAqB,EAAE,MAAM,uBAAuB,CAAC;AAC9D,OAAO,EAAE,sBAAsB,EAAE,MAAM,oBAAoB,CAAC;AAC5D,OAAO,EAAE,eAAe,EAAE,MAAM,iBAAiB,CAAC;AAClD,OAAO,EAAE,qBAAqB,EAAE,MAAM,kBAAkB,CAAC;AAEzD,OAAO,EAAE,2BAA2B,EAAE,MAAM,+BAA+B,CAAC;AAE5E,KAAK,oBAAoB,GAAG,IAAI,GAAG,cAAc,CAAC;AAIlD,eAAO,MAAM,WAAW,EAAE,oBAAoB,EAQ7C,CAAC;AAEF,eAAO,MAAM,mBAAmB,EAAE,oBAAoB,EAQrD,CAAC;AAEF,eAAO,MAAM,eAAe,EAAE,oBAAoB,EASjD,CAAC;AAGF,OAAO,EACL,eAAe,EACf,qBAAqB,EACrB,cAAc,EACd,iBAAiB,EAEjB,sBAAsB,EACtB,wBAAwB,EACxB,aAAa,EACb,8BAA8B,EAE9B,oBAAoB,EACpB,sBAAsB,EACtB,kBAAkB,EAElB,4BAA4B,EAC5B,QAAQ,EACR,oBAAoB,EAEpB,cAAc,EAEd,gBAAgB,EAChB,uBAAuB,EACvB,kBAAkB,EAClB,cAAc,EACd,qBAAqB,EAErB,WAAW,EACX,qBAAqB,EACrB,eAAe,EACf,eAAe,EACf,mBAAmB,EACnB,2BAA2B,GAC5B,CAAC;AAEF,eAAO,MAAM,mBAAmB,EAAE,oBAAoB,EASrD,CAAC;AAEF,wBAAgB,uBAAuB,CACrC,KAAK,EAAE,mBAAmB,GACzB,oBAAoB,EAAE,CAKxB;AAED,wBAAgB,gBAAgB,CAAC,SAAS,EAAE,eAAe,EAAE,WAM5D;AAED,wBAAgB,gBAAgB,CAAC,QAAQ,EAAE,eAAe,GAAG,MAAM,CAQlE;AAED,wBAAsB,4BAA4B,CAAC,OAAO,EAAE,cAAc,iBAoBzE"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/tools/index.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,eAAe,EACf,cAAc,EACd,mBAAmB,EACnB,IAAI,EACJ,cAAc,EACf,MAAM,4BAA4B,CAAC;AAGpC,OAAO,EAAE,qBAAqB,EAAE,MAAM,uBAAuB,CAAC;AAC9D,OAAO,EAAE,YAAY,EAAE,MAAM,6BAA6B,CAAC;AAC3D,OAAO,EAAE,cAAc,EAAE,MAAM,2BAA2B,CAAC;AAC3D,OAAO,EAAE,iBAAiB,EAAE,MAAM,8BAA8B,CAAC;AACjE,OAAO,EAAE,QAAQ,EAAE,MAAM,oBAAoB,CAAC;AAK9C,OAAO,EAAE,kBAAkB,EAAE,MAAM,+BAA+B,CAAC;AACnE,OAAO,EAAE,cAAc,EAAE,MAAM,2BAA2B,CAAC;AAC3D,OAAO,EAAE,qBAAqB,EAAE,MAAM,mCAAmC,CAAC;AAC1E,OAAO,EAAE,WAAW,EAAE,MAAM,wBAAwB,CAAC;AAKrD,OAAO,EAAE,4BAA4B,EAAE,MAAM,gCAAgC,CAAC;AAC9E,OAAO,EAAE,mBAAmB,EAAE,MAAM,gCAAgC,CAAC;AACrE,OAAO,EAAE,wBAAwB,EAAE,MAAM,qBAAqB,CAAC;AAC/D,OAAO,EAAE,aAAa,EAAE,MAAM,cAAc,CAAC;AAC7C,OAAO,EAAE,8BAA8B,EAAE,MAAM,kCAAkC,CAAC;AAClF,OAAO,EAAE,oBAAoB,EAAE,MAAM,sBAAsB,CAAC;AAC5D,OAAO,EAAE,eAAe,EAAE,cAAc,EAAE,eAAe,EAAE,MAAM,UAAU,CAAC;AAC5E,OAAO,EAAE,oBAAoB,EAAE,MAAM,qBAAqB,CAAC;AAC3D,OAAO,EAAE,sBAAsB,EAAE,MAAM,oBAAoB,CAAC;AAC5D,OAAO,EAAE,eAAe,EAAE,MAAM,iBAAiB,CAAC;AAClD,OAAO,EAAE,qBAAqB,EAAE,MAAM,kBAAkB,CAAC;AAEzD,OAAO,EAAE,2BAA2B,EAAE,MAAM,+BAA+B,CAAC;AAE5E,KAAK,oBAAoB,GAAG,IAAI,GAAG,cAAc,CAAC;AAIlD,eAAO,MAAM,WAAW,EAAE,oBAAoB,EAU7C,CAAC;AAEF,eAAO,MAAM,mBAAmB,EAAE,oBAAoB,EAQrD,CAAC;AAEF,eAAO,MAAM,eAAe,EAAE,oBAAoB,EASjD,CAAC;AAGF,OAAO,EACL,eAAe,EACf,qBAAqB,EACrB,cAAc,EACd,iBAAiB,EAEjB,wBAAwB,EACxB,aAAa,EACb,8BAA8B,EAE9B,oBAAoB,EACpB,sBAAsB,EACtB,YAAY,EAEZ,4BAA4B,EAC5B,QAAQ,EACR,oBAAoB,EAEpB,cAAc,EAEd,kBAAkB,EAClB,cAAc,EACd,qBAAqB,EAErB,WAAW,EACX,qBAAqB,EACrB,eAAe,EACf,eAAe,EACf,mBAAmB,EACnB,2BAA2B,GAC5B,CAAC;AAEF,eAAO,MAAM,mBAAmB,EAAE,oBAAoB,EAQrD,CAAC;AAEF,wBAAgB,oBAAoB,CAAC,KAAK,EAAE,mBAAmB,WAE9D;AAED,wBAAgB,uBAAuB,CACrC,KAAK,EAAE,mBAAmB,GACzB,oBAAoB,EAAE,CAExB;AAED,wBAAgB,qBAAqB,CACnC,KAAK,EAAE,mBAAmB,GACzB,oBAAoB,EAAE,CAExB;AAED,wBAAgB,gBAAgB,CAAC,SAAS,EAAE,eAAe,EAAE,WAM5D;AAED,wBAAgB,gBAAgB,CAAC,QAAQ,EAAE,eAAe,GAAG,MAAM,CAQlE;AAED,wBAAsB,4BAA4B,CAAC,OAAO,EAAE,cAAc,iBAoBzE"}

package/dist/tools/index.js

@@ -1,31 +1,30 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.allToolsDefinitions = exports.viewFailedTestRunReportTool = exports.upgradePackagesTool = exports.updateIssueTool = exports.traceDotZipTool = exports.sendTriageSummaryTool = exports.runTestTool = exports.reviewPullRequestTool = exports.renameFileTool = exports.mergeConflictsTool = exports.listTestsForProjectTool = exports.listProjectsTool = exports.listIssuesTool = exports.listEnvironmentsTool = exports.grepTool = exports.generateTestWithBrowserAgent = exports.fetchVideoAnalysis = exports.fetchTestRunReportTool = exports.fetchSessionDiffTool = exports.fetchLastSuccessfulTestRunTool = exports.fetchFileTool = exports.fetchDiagnosisReportTool = exports.extractFramesFromVideo = exports.downloadBuildTool = exports.deleteFileTool = exports.createPullRequestTool = exports.createIssueTool = exports.triageOnlyTools = exports.testGenerationTools = exports.commonTools = void 0;
+exports.allToolsDefinitions = exports.viewFailedTestRunReportTool = exports.upgradePackagesTool = exports.updateIssueTool = exports.traceDotZipTool = exports.sendTriageSummaryTool = exports.runTestTool = exports.reviewPullRequestTool = exports.renameFileTool = exports.mergeConflictsTool = exports.listIssuesTool = exports.listEnvironmentsTool = exports.grepTool = exports.generateTestWithBrowserAgent = exports.analyseVideo = exports.fetchTestRunReportTool = exports.fetchSessionDiffTool = exports.fetchLastSuccessfulTestRunTool = exports.fetchFileTool = exports.fetchDiagnosisReportTool = exports.downloadBuildTool = exports.deleteFileTool = exports.createPullRequestTool = exports.createIssueTool = exports.triageOnlyTools = exports.testGenerationTools = exports.commonTools = void 0;
+exports.hasBuiltInTextEditor = hasBuiltInTextEditor;
 exports.textEditorToolsForModel = textEditorToolsForModel;
+exports.textViewToolsForModel = textViewToolsForModel;
 exports.toolsNeedBrowser = toolsNeedBrowser;
 exports.nameForTelemetry = nameForTelemetry;
 exports.sendToolRequestToRemoteQueue = sendToolRequestToRemoteQueue;
-const chat_1 = require("@empiricalrun/llm/chat");
 const SQSClient_1 = require("../utils/SQSClient");
 const create_pull_request_1 = require("./create-pull-request");
 Object.defineProperty(exports, "createPullRequestTool", { enumerable: true, get: function () { return create_pull_request_1.createPullRequestTool; } });
+const analyse_video_1 = require("./definitions/analyse-video");
+Object.defineProperty(exports, "analyseVideo", { enumerable: true, get: function () { return analyse_video_1.analyseVideo; } });
 const delete_file_1 = require("./definitions/delete-file");
 Object.defineProperty(exports, "deleteFileTool", { enumerable: true, get: function () { return delete_file_1.deleteFileTool; } });
 const download_build_1 = require("./definitions/download-build");
 Object.defineProperty(exports, "downloadBuildTool", { enumerable: true, get: function () { return download_build_1.downloadBuildTool; } });
-const extract_frames_from_video_1 = require("./definitions/extract-frames-from-video");
-Object.defineProperty(exports, "extractFramesFromVideo", { enumerable: true, get: function () { return extract_frames_from_video_1.extractFramesFromVideo; } });
-const fetch_video_analysis_1 = require("./definitions/fetch-video-analysis");
-Object.defineProperty(exports, "fetchVideoAnalysis", { enumerable: true, get: function () { return fetch_video_analysis_1.fetchVideoAnalysis; } });
 const grep_1 = require("./definitions/grep");
 Object.defineProperty(exports, "grepTool", { enumerable: true, get: function () { return grep_1.grepTool; } });
 const list_tests_and_projects_1 = require("./definitions/list-tests-and-projects");
-Object.defineProperty(exports, "listProjectsTool", { enumerable: true, get: function () { return list_tests_and_projects_1.listProjectsTool; } });
-Object.defineProperty(exports, "listTestsForProjectTool", { enumerable: true, get: function () { return list_tests_and_projects_1.listTestsForProjectTool; } });
 const merge_conflicts_1 = require("./definitions/merge-conflicts");
 Object.defineProperty(exports, "mergeConflictsTool", { enumerable: true, get: function () { return merge_conflicts_1.mergeConflictsTool; } });
 const rename_file_1 = require("./definitions/rename-file");
 Object.defineProperty(exports, "renameFileTool", { enumerable: true, get: function () { return rename_file_1.renameFileTool; } });
+const review_pull_request_1 = require("./definitions/review-pull-request");
+Object.defineProperty(exports, "reviewPullRequestTool", { enumerable: true, get: function () { return review_pull_request_1.reviewPullRequestTool; } });
 const run_test_1 = require("./definitions/run-test");
 Object.defineProperty(exports, "runTestTool", { enumerable: true, get: function () { return run_test_1.runTestTool; } });
 const str_replace_editor_1 = require("./definitions/str_replace_editor");
@@ -47,8 +46,6 @@ Object.defineProperty(exports, "listIssuesTool", { enumerable: true, get: functi
 Object.defineProperty(exports, "updateIssueTool", { enumerable: true, get: function () { return issues_1.updateIssueTool; } });
 const list_environments_1 = require("./list-environments");
 Object.defineProperty(exports, "listEnvironmentsTool", { enumerable: true, get: function () { return list_environments_1.listEnvironmentsTool; } });
-const review_pull_request_1 = require("./review-pull-request");
-Object.defineProperty(exports, "reviewPullRequestTool", { enumerable: true, get: function () { return review_pull_request_1.reviewPullRequestTool; } });
 const test_run_fetcher_1 = require("./test-run-fetcher");
 Object.defineProperty(exports, "fetchTestRunReportTool", { enumerable: true, get: function () { return test_run_fetcher_1.fetchTestRunReportTool; } });
 const trace_dot_zip_1 = require("./trace-dot-zip");
@@ -68,6 +65,8 @@ exports.commonTools = [
     download_build_1.downloadBuildTool,
     fetch_file_1.fetchFileTool,
     trace_dot_zip_1.traceDotZipTool,
+    list_tests_and_projects_1.listProjectsTool,
+    list_tests_and_projects_1.listTestsForProjectTool,
 ];
 exports.testGenerationTools = [
     test_gen_browser_1.generateTestWithBrowserAgent,
@@ -83,7 +82,7 @@ exports.triageOnlyTools = [
     issues_1.createIssueTool,
     issues_1.updateIssueTool,
     view_failed_test_run_report_1.viewFailedTestRunReportTool,
-    fetch_video_analysis_1.fetchVideoAnalysis,
+    analyse_video_1.analyseVideo,
     fetch_last_successful_test_run_1.fetchLastSuccessfulTestRunTool,
     triage_summary_1.sendTriageSummaryTool,
     review_pull_request_1.reviewPullRequestTool,
@@ -93,16 +92,18 @@ exports.allToolsDefinitions = [
     ...exports.testGenerationTools,
     ...exports.triageOnlyTools,
     // Individual tools that were in removed arrays
-    extract_frames_from_video_1.extractFramesFromVideo,
     fetch_session_diff_1.fetchSessionDiffTool,
     list_tests_and_projects_1.listProjectsTool,
     list_tests_and_projects_1.listTestsForProjectTool,
 ];
+function hasBuiltInTextEditor(model) {
+    return model.includes("claude") && !model.includes("haiku");
+}
 function textEditorToolsForModel(model) {
-    if (model && (0, chat_1.getProviderForModel)(model) !== "claude") {
-        return Object.values(str_replace_editor_1.textEditorTools);
-    }
-    return [];
+    return !hasBuiltInTextEditor(model) ? Object.values(str_replace_editor_1.textEditorTools) : [];
+}
+function textViewToolsForModel(model) {
+    return !hasBuiltInTextEditor(model) ? Object.values(str_replace_editor_1.textViewTools) : [];
 }
 function toolsNeedBrowser(toolCalls) {
     return toolCalls.some((toolCall) => exports.allToolsDefinitions.find((t) => t.schema.name === toolCall.name)

package/dist/tools/merge-conflicts/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/tools/merge-conflicts/index.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,4BAA4B,CAAC;AA2CvD,eAAO,MAAM,kBAAkB,EAAE,IAiDhC,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/tools/merge-conflicts/index.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,4BAA4B,CAAC;AA8CvD,eAAO,MAAM,kBAAkB,EAAE,IAiDhC,CAAC"}

package/dist/tools/merge-conflicts/index.js

@@ -11,7 +11,7 @@ const merge_conflicts_1 = require("../definitions/merge-conflicts");
 const git_1 = require("../executor/utils/git");
 function runMergeMain(repoPath) {
     try {
-        (0, child_process_1.execSync)("git merge --no-ff origin/main", { stdio: "pipe", cwd: repoPath });
+        (0, child_process_1.execSync)("git merge --no-ff origin/main -m 'Merge remote-tracking branch origin/main [skip ci]'", { stdio: "pipe", cwd: repoPath });
         return "CLEAN";
     }
     catch (error) {

package/dist/tools/rename-file/index.js

@@ -69,7 +69,7 @@ exports.renameFileTool = {
             if (!tscResult.success) {
                 return {
                     result: `File renamed from '${input.oldPath}' to '${input.newPath}'. However, type checks are failing with errors:\n\n${tscResult.errors.join("\n")}`,
-                    isError: true,
+                    isError: false,
                 };
             }
             return {

package/dist/tools/review-pull-request/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/tools/review-pull-request/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAmB,IAAI,EAAc,MAAM,4BAA4B,CAAC;AAc/E,eAAO,MAAM,qBAAqB,EAAE,IAgGnC,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/tools/review-pull-request/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,IAAI,EAAc,MAAM,4BAA4B,CAAC;AAS9D,eAAO,MAAM,qBAAqB,EAAE,IA2FnC,CAAC"}

package/dist/tools/review-pull-request/index.js

@@ -1,24 +1,13 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.reviewPullRequestTool = void 0;
-const zod_1 = require("zod");
-const exports_1 = require("../../agent/chat/exports");
-const state_1 = require("../../agent/chat/state");
 const code_review_1 = require("../../agent/code-review");
-const fetch_session_diff_1 = require("../fetch-session-diff");
+const executor_1 = require("../../agent/code-review/executor");
+const review_pull_request_1 = require("../definitions/review-pull-request");
 const CODE_REVIEW_MODEL = "claude-sonnet-4-20250514";
-const ReviewPullRequestInputSchema = zod_1.z.object({
-    sessionUrl: zod_1.z.string().describe("The URL of the chat session to review"),
-});
 exports.reviewPullRequestTool = {
-    schema: {
-        name: "reviewPullRequest",
-        description: `Reviews a pull request by using CodeReviewAgent to fetch session diff and analyze it. Returns a boolean result with an explanation.`,
-        parameters: ReviewPullRequestInputSchema,
-    },
-    needsBrowser: false,
-    isInlineTool: true,
-    execute: async ({ input, apiClient }) => {
+    ...review_pull_request_1.reviewPullRequestTool,
+    execute: async ({ input, apiClient, chatSession, environmentOverrides, trace, featureFlags, repoPath, }) => {
         try {
             const { sessionUrl } = input;
             if (!sessionUrl) {
@@ -27,61 +16,51 @@ exports.reviewPullRequestTool = {
                     result: "No chat session URL provided",
                 };
             }
-            const agent = new code_review_1.CodeReviewAgent({
+            if (!apiClient)
+                return {
+                    isError: true,
+                    result: "API Client not available",
+                };
+            const span = trace?.span({
+                name: "code-review-agent",
+                input: {
+                    sessionUrl,
+                    model: CODE_REVIEW_MODEL,
+                },
+            });
+            const codeReviewToolExecutor = new executor_1.CodeReviewToolExecutor({
+                chatSession: chatSession || null,
+                repoPath,
+                apiClient,
+                trace: span,
+                featureFlags,
+                environmentOverrides: environmentOverrides || {},
+            });
+            const agentParams = {
                 featureFlags: [],
                 selectedModel: CODE_REVIEW_MODEL,
-            });
+                chatState: undefined,
+                toolExecutor: codeReviewToolExecutor,
+                trace: span,
+            };
+            const agent = new code_review_1.CodeReviewAgent({ ...agentParams });
+            // Push the review prompt to the agent
             const userPrompt = `Review this session: ${sessionUrl}`;
-            const chatState = (0, state_1.createChatState)({
-                userPrompt,
-                attachments: [],
-                existingState: undefined,
-                selectedModel: CODE_REVIEW_MODEL,
-                error: null,
-            });
-            agent.initializeWithState(chatState);
-            let chatModel = (0, exports_1.createChatModel)(chatState.messages, CODE_REVIEW_MODEL);
-            while (!chatModel.askUserForInput) {
+            agent.pushUserMessage([{ text: userPrompt }]);
+            while (!agent.askUserForInput) {
                 await agent.runLoop({
-                    messages: chatModel.messages,
-                    reporter: async (data) => {
-                        console.log("Review progress:", data);
-                    },
-                    onPendingToolCall: async (toolCalls) => {
-                        const [toolCall] = toolCalls;
-                        // TODO: This will only work for 1 tool - will break if
-                        // code review agent has more tools: need first class primitive
-                        // for inline tool execution
-                        if (toolCall?.name === "fetchSessionDiff") {
-                            const tool = fetch_session_diff_1.fetchSessionDiffTool;
-                            const result = await tool.execute({
-                                input: toolCall.input,
-                                apiClient,
-                                repoPath: ``,
-                                featureFlags: [],
-                            });
-                            chatModel.pushToolResultsMessage([toolCall], [result]);
-                        }
-                    },
-                    onLLMResponse: async (response, model) => {
-                        console.log(`Agent responded using ${model}:`, response);
-                    },
+                    reporter: async () => { },
+                    trace: span,
                 });
-                // Update the chatModel with the agent's final state for next iteration
-                if (agent.messages) {
-                    chatModel = (0, exports_1.createChatModel)(agent.messages, CODE_REVIEW_MODEL);
-                }
             }
-            const messages = agent.messages || [];
-            const lastMessage = messages.length
-                ? messages[messages.length - 1]
-                : undefined;
-            const lastMessageTextPart = lastMessage
-                ? lastMessage.parts
-                    .filter((p) => "text" in p)
-                    .find((p) => "text" in p && !!p.text)
-                : undefined;
-            if (!lastMessageTextPart) {
+            const agentResult = agent.getResult();
+            span?.end({
+                output: {
+                    result: agentResult,
+                    success: !!agentResult,
+                },
+            });
+            if (!agentResult) {
                 return {
                     isError: true,
                     result: "Could not extract text response from agent",
@@ -89,7 +68,7 @@ exports.reviewPullRequestTool = {
             }
             return {
                 isError: false,
-                result: lastMessageTextPart.text,
+                result: JSON.stringify(agentResult),
             };
         }
         catch (error) {

package/dist/tools/run-test.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"run-test.d.ts","sourceRoot":"","sources":["../../src/tools/run-test.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAc,MAAM,4BAA4B,CAAC;AA0CnE,eAAO,MAAM,WAAW,EAAE,IAiFzB,CAAC"}
+{"version":3,"file":"run-test.d.ts","sourceRoot":"","sources":["../../src/tools/run-test.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAU,IAAI,EAAc,MAAM,4BAA4B,CAAC;AA0C3E,eAAO,MAAM,WAAW,EAAE,IA4GzB,CAAC"}

package/dist/tools/run-test.js

@@ -29,7 +29,7 @@ ${JSON.stringify(truncatedSummaryJson)}
 }
 exports.runTestTool = {
     ...run_test_1.runTestTool,
-    execute: async ({ input, repoPath, collectArtifacts, environmentOverrides = {}, }) => {
+    execute: async ({ input, repoPath, collectArtifacts, environmentOverrides = {}, logger, }) => {
         let reportUrl = undefined;
         let envOverrides = { ...environmentOverrides };
         if ((0, artifacts_1.isArtifactCollectionEnabled)()) {
@@ -43,14 +43,18 @@ exports.runTestTool = {
             };
         }
         else {
-            console.warn("R2 credentials not found: report artifacts will not be uploaded");
+            const warnMsg = "R2 credentials not found: report artifacts will not be uploaded";
+            logger?.warn(warnMsg) || console.warn(warnMsg);
         }
         const { filePath } = input;
         const absoluteFilePath = path_1.default.join(repoPath, filePath);
         if (!fs_1.default.existsSync(absoluteFilePath)) {
+            const errorMsg = `File ${filePath} does not exist.`;
+            logger?.error("Test file not found", { filePath, absoluteFilePath }) ||
+                console.error("Test file not found:", filePath);
             return {
                 isError: true,
-                result: `File ${filePath} does not exist.`,
+                result: errorMsg,
             };
         }
         if (input.buildUrl) {
@@ -60,6 +64,12 @@ exports.runTestTool = {
             };
         }
         try {
+            logger?.info("Starting test execution", {
+                testName: input.testName,
+                filePath,
+                project: input.project,
+                suites: input.suites,
+            }) || console.log("Starting test execution:", input.testName);
             const result = await (0, test_run_1.runSingleTest)({
                 testName: input.testName,
                 suites: input.suites,
@@ -67,7 +77,14 @@ exports.runTestTool = {
                 repoDir: repoPath,
                 projects: [input.project],
                 envOverrides,
+                stdout: logger?.createPrefixedStream?.("stdout"),
+                stderr: logger?.createPrefixedStream?.("stderr"),
             });
+            logger?.info("Test execution completed", {
+                passed: result.hasTestPassed,
+                testCount: result.summaryJson?.suites?.length || 0,
+            }) ||
+                console.log("Test execution completed. Passed:", result.hasTestPassed);
             const artifacts = (0, utils_1.extractAttachmentsFromPlaywrightJSONReport)(result.summaryJson);
             void collectArtifacts?.(artifacts);
             return {
@@ -82,6 +99,11 @@ exports.runTestTool = {
         catch (error) {
             // Ensure we capture the full error message regardless of error type
             const errorMessage = error instanceof Error ? error.message : String(error);
+            logger?.error("Test execution failed", {
+                error: errorMessage,
+                testName: input.testName,
+                filePath,
+            }) || console.error("Test execution failed:", errorMessage);
             return {
                 artifacts: null,
                 result: JSON.stringify({ error: errorMessage }),

package/dist/tools/test-gen-browser.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"test-gen-browser.d.ts","sourceRoot":"","sources":["../../src/tools/test-gen-browser.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EACV,IAAI,EAGL,MAAM,4BAA4B,CAAC;AAoBpC,eAAO,MAAM,4BAA4B,EAAE,IA0L1C,CAAC"}
+{"version":3,"file":"test-gen-browser.d.ts","sourceRoot":"","sources":["../../src/tools/test-gen-browser.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAiB,IAAI,EAAE,MAAM,4BAA4B,CAAC;AAqBtE,eAAO,MAAM,4BAA4B,EAAE,IA+L1C,CAAC"}

package/dist/tools/test-gen-browser.js

@@ -10,6 +10,7 @@ const path_1 = __importDefault(require("path"));
 const run_1 = require("../agent/browsing/run");
 const utils_1 = require("../agent/browsing/utils");
 const pw_pause_1 = require("../agent/cua/pw-codegen/pw-pause");
+const utils_2 = require("../artifacts/utils");
 const web_1 = require("../bin/utils/platform/web");
 const scenarios_1 = require("../bin/utils/scenarios");
 const test_gen_browser_1 = require("./definitions/test-gen-browser");
@@ -53,8 +54,9 @@ exports.generateTestWithBrowserAgent = {
         }
         // Prepare the file for the browser agent
         const fileBackup = fs_1.default.readFileSync(absoluteFilePath, "utf-8");
+        let todoContent;
         try {
-            await (0, utils_1.replaceTodoWithCreateTest)(filePath, repoPath);
+            todoContent = (0, utils_1.replaceTodoWithCreateTest)(filePath, repoPath);
         }
         catch (error) {
             // Undo the TODO -> createTest and test.only changes
@@ -83,7 +85,7 @@ exports.generateTestWithBrowserAgent = {
                 BUILD_URL: input.buildUrl,
             };
         }
-        const testGenToken = (0, scenarios_1.buildTokenFromOptions)({ name: testName, file: filePath, prompt: input.changeToMake }, { useComputerUseAgent: true });
+        const testGenToken = (0, scenarios_1.buildTokenFromOptions)({ name: testName, file: filePath }, { useComputerUseAgent: true });
         console.log("[generateTestWithBrowserAgent] Validations passed, starting agent");
         const agentResult = await (0, run_1.runBrowsingAgent)({
             testCaseName: testName,
@@ -100,10 +102,7 @@ exports.generateTestWithBrowserAgent = {
         // Cleanup: Undo the TODO -> createTest changes
         await (0, pw_pause_1.revertToOriginalPwCode)(repoPath);
         fs_1.default.writeFileSync(absoluteFilePath, fileBackup, "utf-8");
-        const { isError, error, result: toolResult, artifacts, usage, } = agentResult;
-        if (artifacts) {
-            void collectArtifacts?.(artifacts);
-        }
+        const { isError, error, result: toolResult, usage } = agentResult;
         if (isError || !toolResult) {
             return {
                 isError,
@@ -112,53 +111,58 @@ ${error}
 `,
             };
         }
-        let result = [
-            {
-                type: "text",
-                text: "Browser agent has finished running. Here is the summary of actions it took, and the generated Playwright code:",
-            },
+        const screenshotArtifacts = toolResult
+            .filter((item) => item.type === "screenshot")
+            .map((item) => item.screenshot)
+            .map((artifact) => ({
+            ...artifact,
+            data: Buffer.isBuffer(artifact.data)
+                ? artifact.data
+                : Buffer.from(artifact.data),
+        }));
+        const artifactsInput = [
+            ...((0, utils_2.findPlaywrightArtifacts)(repoPath) || []),
+            ...screenshotArtifacts,
+        ];
+        const summaryResult = `Browser agent has finished running.
+TODO comment which it worked on: ${todoContent}
+Summary of actions it took, and the generated Playwright code:`;
+        if (!collectArtifacts || artifactsInput.length === 0) {
+            const result = [
+                { type: "text", text: summaryResult },
+                ...toolResult.map((item) => ({
+                    type: "text",
+                    text: item.type === "text" ? item.text : "[Screenshot]",
+                })),
+            ];
+            return { isError, result, usage };
+        }
+        const uploadedArtifacts = await collectArtifacts(artifactsInput);
+        const artifactUrlMap = new Map(uploadedArtifacts.map((a) => [a.name, a.url]));
+        let screenshotIndex = 0;
+        const result = [
+            { type: "text", text: summaryResult },
             ...toolResult.map((item) => {
                 if (item.type === "text") {
-                    return {
-                        type: "text",
-                        text: item.text,
-                    };
+                    return { type: "text", text: item.text };
                 }
                 else if (item.type === "screenshot") {
-                    const bufferData = Buffer.isBuffer(item.screenshot.data)
-                        ? item.screenshot.data
-                        : Buffer.from(item.screenshot.data);
-                    return {
-                        type: "image/png",
-                        base64Data: `${bufferData.toString("base64")}`,
-                    };
-                }
-                else {
-                    return {
-                        type: "text",
-                        text: `Unknown item type: ${item}`,
-                    };
+                    const screenshotName = screenshotArtifacts[screenshotIndex]?.name;
+                    const url = screenshotName
+                        ? artifactUrlMap.get(screenshotName)
+                        : undefined;
+                    screenshotIndex++;
+                    return url
+                        ? { type: "image/png", url }
+                        : { type: "text", text: "[Screenshot upload failed]" };
                 }
+                return { type: "text", text: "Unknown item type" };
             }),
         ];
-        const isMultiModalEnabled = featureFlags.includes("multi_modal_tool_result_from_browser_agent");
-        if (isMultiModalEnabled) {
-            return {
-                isError,
-                result,
-                usage,
-            };
-        }
-        else {
-            const textOnlyResult = result
-                .filter((item) => item.type === "text")
-                .map((item) => item.text)
-                .join("\n");
-            return {
-                isError,
-                result: textOnlyResult,
-                usage,
-            };
-        }
+        return {
+            isError,
+            result,
+            usage,
+        };
     },
 };

package/dist/tools/upgrade-packages/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/tools/upgrade-packages/index.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,4BAA4B,CAAC;AAoBvD,eAAO,MAAM,mBAAmB,EAAE,IAyHjC,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/tools/upgrade-packages/index.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,4BAA4B,CAAC;AAoBvD,eAAO,MAAM,mBAAmB,EAAE,IA8HjC,CAAC"}

package/dist/tools/upgrade-packages/index.js

@@ -62,6 +62,9 @@ exports.upgradePackagesTool = {
             let shouldMerge = false;
             let prNumber = null;
             let merged = false;
+            if (!chatSession) {
+                throw new Error("Chat session is not available.");
+            }
             if (!apiClient) {
                 throw new Error("Dashboard API client is not available.");
             }
@@ -71,6 +74,7 @@ exports.upgradePackagesTool = {
                     repo: repoName,
                     apiClient,
                     branchName,
+                    base: chatSession?.baseBranchName,
                     title: prTitle,
                     body: (0, pr_description_1.addMetadataToPRDescription)(prBody, chatSession),
                     labels: ["automated"],

package/dist/tools/upgrade-packages/utils.d.ts

@@ -1,4 +1,5 @@
 import { IDashboardAPIClient } from "@empiricalrun/shared-types";
+export declare function isPatchUpdate(currentVersion: string | undefined, newVersion: string | undefined): boolean;
 export declare function getLatestVersion(packageName: string): Promise<any>;
 export declare function parsePackageJsonDiff({ diff, }: {
     diff: string;