@empiricalrun/test-gen 0.74.2 → 0.76.0

package/dist/tools/definitions/extract-frames-from-video.d.ts

@@ -0,0 +1,39 @@
+import { ToolDefinition } from "@empiricalrun/shared-types";
+import z from "zod";
+export declare const extractFramesFromVideoSchema: z.ZodObject<{
+    videoUrl: z.ZodString;
+    params: z.ZodOptional<z.ZodObject<{
+        fps: z.ZodOptional<z.ZodDefault<z.ZodNumber>>;
+        threshold: z.ZodOptional<z.ZodDefault<z.ZodNumber>>;
+        startTime: z.ZodOptional<z.ZodString>;
+        duration: z.ZodOptional<z.ZodString>;
+    }, "strip", z.ZodTypeAny, {
+        startTime?: string | undefined;
+        threshold?: number | undefined;
+        fps?: number | undefined;
+        duration?: string | undefined;
+    }, {
+        startTime?: string | undefined;
+        threshold?: number | undefined;
+        fps?: number | undefined;
+        duration?: string | undefined;
+    }>>;
+}, "strip", z.ZodTypeAny, {
+    videoUrl: string;
+    params?: {
+        startTime?: string | undefined;
+        threshold?: number | undefined;
+        fps?: number | undefined;
+        duration?: string | undefined;
+    } | undefined;
+}, {
+    videoUrl: string;
+    params?: {
+        startTime?: string | undefined;
+        threshold?: number | undefined;
+        fps?: number | undefined;
+        duration?: string | undefined;
+    } | undefined;
+}>;
+export declare const extractFramesFromVideo: ToolDefinition<z.infer<typeof extractFramesFromVideoSchema>>;
+//# sourceMappingURL=extract-frames-from-video.d.ts.map

package/dist/tools/definitions/extract-frames-from-video.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"extract-frames-from-video.d.ts","sourceRoot":"","sources":["../../../src/tools/definitions/extract-frames-from-video.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,cAAc,EAAE,MAAM,4BAA4B,CAAC;AAC5D,OAAO,CAAC,MAAM,KAAK,CAAC;AAEpB,eAAO,MAAM,4BAA4B;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;EAoCvC,CAAC;AAEH,eAAO,MAAM,sBAAsB,EAAE,cAAc,CACjD,CAAC,CAAC,KAAK,CAAC,OAAO,4BAA4B,CAAC,CAoB7C,CAAC"}

package/dist/tools/definitions/extract-frames-from-video.js

@@ -0,0 +1,60 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.extractFramesFromVideo = exports.extractFramesFromVideoSchema = void 0;
+const zod_1 = __importDefault(require("zod"));
+exports.extractFramesFromVideoSchema = zod_1.default.object({
+    videoUrl: zod_1.default
+        .string()
+        .url("Must be a valid URL")
+        .describe("The URL of the video to extract frames from."),
+    params: zod_1.default
+        .object({
+        fps: zod_1.default
+            .number()
+            .int()
+            .min(1)
+            .max(120)
+            .default(30)
+            .optional()
+            .describe("Frames per second to extract from the video (default: 30)"),
+        threshold: zod_1.default
+            .number()
+            .min(0)
+            .max(0.5)
+            .default(0.001)
+            .optional()
+            .describe("Deduplication threshold (fraction of pixels that may differ to consider frames identical). Lower = stricter. Default: 0.001"),
+        startTime: zod_1.default
+            .string()
+            .optional()
+            .describe("Start time in format MM:SS, example 1 min 32 sec -> 01:32"),
+        duration: zod_1.default
+            .string()
+            .optional()
+            .describe("Duration time in format MM:SS, example 1 min 32 sec -> 01:32"),
+    })
+        .optional(),
+});
+exports.extractFramesFromVideo = {
+    schema: {
+        name: "extractFramesFromVideo",
+        description: `
+Extracts frames from a video with precise timing control.
+
+**Input:** Video URL (required), optional parameters for timing, frame rate, and output format
+**Output:** PNG formate Frame URLs with metadata.
+**Use when:** You need specific frames from a video segment, want to analyze particular time ranges, or need distinct frames fromthe video, or just need all frames from the video for further processing
+
+**Key Features:**
+- Time-based frame extraction (start time + duration)
+- Configurable frame rate and deduplication threshold
+- Frame metadata including timestamps and indices
+- Memory-efficient processing for large videos`,
+        parameters: exports.extractFramesFromVideoSchema,
+    },
+    needsBrowser: false,
+    isInlineTool: false,
+};

package/dist/tools/definitions/fetch-video-analysis.d.ts

@@ -2,10 +2,38 @@ import { ToolDefinition } from "@empiricalrun/shared-types";
 import z from "zod";
 export declare const videoAnalysisSchema: z.ZodObject<{
     videoUrl: z.ZodString;
+    params: z.ZodOptional<z.ZodObject<{
+        fps: z.ZodOptional<z.ZodDefault<z.ZodNumber>>;
+        threshold: z.ZodOptional<z.ZodDefault<z.ZodNumber>>;
+        model: z.ZodOptional<z.ZodDefault<z.ZodString>>;
+        featureFlag: z.ZodOptional<z.ZodDefault<z.ZodEnum<["none", "send-all-frames", "separate-threshold", "ai-based-frames"]>>>;
+    }, "strip", z.ZodTypeAny, {
+        model?: string | undefined;
+        threshold?: number | undefined;
+        fps?: number | undefined;
+        featureFlag?: "none" | "send-all-frames" | "separate-threshold" | "ai-based-frames" | undefined;
+    }, {
+        model?: string | undefined;
+        threshold?: number | undefined;
+        fps?: number | undefined;
+        featureFlag?: "none" | "send-all-frames" | "separate-threshold" | "ai-based-frames" | undefined;
+    }>>;
 }, "strip", z.ZodTypeAny, {
     videoUrl: string;
+    params?: {
+        model?: string | undefined;
+        threshold?: number | undefined;
+        fps?: number | undefined;
+        featureFlag?: "none" | "send-all-frames" | "separate-threshold" | "ai-based-frames" | undefined;
+    } | undefined;
 }, {
     videoUrl: string;
+    params?: {
+        model?: string | undefined;
+        threshold?: number | undefined;
+        fps?: number | undefined;
+        featureFlag?: "none" | "send-all-frames" | "separate-threshold" | "ai-based-frames" | undefined;
+    } | undefined;
 }>;
 export declare const fetchVideoAnalysis: ToolDefinition<z.infer<typeof videoAnalysisSchema>>;
 //# sourceMappingURL=fetch-video-analysis.d.ts.map

package/dist/tools/definitions/fetch-video-analysis.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"fetch-video-analysis.d.ts","sourceRoot":"","sources":["../../../src/tools/definitions/fetch-video-analysis.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,cAAc,EAAE,MAAM,4BAA4B,CAAC;AAC5D,OAAO,CAAC,MAAM,KAAK,CAAC;AAEpB,eAAO,MAAM,mBAAmB;;;;;;EAK9B,CAAC;AAEH,eAAO,MAAM,kBAAkB,EAAE,cAAc,CAC7C,CAAC,CAAC,KAAK,CAAC,OAAO,mBAAmB,CAAC,CAapC,CAAC"}
+{"version":3,"file":"fetch-video-analysis.d.ts","sourceRoot":"","sources":["../../../src/tools/definitions/fetch-video-analysis.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,cAAc,EAAE,MAAM,4BAA4B,CAAC;AAC5D,OAAO,CAAC,MAAM,KAAK,CAAC;AAEpB,eAAO,MAAM,mBAAmB;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;EA2C9B,CAAC;AAEH,eAAO,MAAM,kBAAkB,EAAE,cAAc,CAC7C,CAAC,CAAC,KAAK,CAAC,OAAO,mBAAmB,CAAC,CAcpC,CAAC"}

package/dist/tools/definitions/fetch-video-analysis.js

@@ -10,15 +10,50 @@ exports.videoAnalysisSchema = zod_1.default.object({
         .string()
         .url("Must be a valid URL")
         .describe("The URL of the video to analyze."),
+    params: zod_1.default
+        .object({
+        fps: zod_1.default
+            .number()
+            .int()
+            .min(1)
+            .max(120)
+            .default(30)
+            .optional()
+            .describe("Frames per second to extract from the video (default: 30)"),
+        threshold: zod_1.default
+            .number()
+            .min(0)
+            .max(0.5)
+            .default(0.001)
+            .optional()
+            .describe("Deduplication threshold (fraction of pixels that may differ to consider frames identical). Lower = stricter. Default: 0.001"),
+        model: zod_1.default
+            .string()
+            .default("gemini-2.5-pro")
+            .optional()
+            .describe("Gemini LLM model identifier to use for analysis defaults to gemini-2.5-pro"),
+        featureFlag: zod_1.default
+            .enum([
+            "none",
+            "send-all-frames",
+            "separate-threshold",
+            "ai-based-frames",
+        ])
+            .default("none")
+            .optional()
+            .describe("Feature flag to use for analysis defaults to none"),
+    })
+        .optional(),
 });
 exports.fetchVideoAnalysis = {
     schema: {
         name: "fetchVideoAnalysis",
-        description: `Analyzes Playwright test execution videos to identify test failures and UI issues.
+        description: `
+Analyzes Playwright test execution videos to identify test failures and UI issues.
   
-  **Input:** Video URL of a test execution recording, optional fps (frames per second, default: 30)
-  **Output:** analysis summary of what happened during the test execution
-  **Use when:** You have a failing test with a video recording and need to understand what went wrong`,
+**Input:** The video URL is mandatory (supports webm, mp4). DO NOT specify params UNLESS the user has explicitly asked for a specific param value to be tested.
+**Output:** analysis summary of what happened during the test execution.
+**Use when:** You have a failing test with a video recording and need to understand what went wrong`,
         parameters: exports.videoAnalysisSchema,
     },
     needsBrowser: false,

package/dist/tools/definitions/rename-file.d.ts

@@ -0,0 +1,3 @@
+import type { ToolDefinition } from "@empiricalrun/shared-types";
+export declare const renameFileTool: ToolDefinition;
+//# sourceMappingURL=rename-file.d.ts.map

package/dist/tools/definitions/rename-file.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"rename-file.d.ts","sourceRoot":"","sources":["../../../src/tools/definitions/rename-file.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,4BAA4B,CAAC;AAgBjE,eAAO,MAAM,cAAc,EAAE,cAU5B,CAAC"}

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

@@ -0,0 +1,23 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.renameFileTool = void 0;
+const zod_1 = require("zod");
+const RenameFileInputSchema = zod_1.z.object({
+    oldPath: zod_1.z
+        .string()
+        .describe("The current path to the file to rename (relative to the repository root). For example, tests/foo.spec.ts"),
+    newPath: zod_1.z
+        .string()
+        .describe("The new path for the file (relative to the repository root). For example, tests/new-dir/bar.spec.ts"),
+});
+exports.renameFileTool = {
+    schema: {
+        name: "renameFile",
+        description: `Rename or move a file in the filesystem.
+This tool will move the specified file from oldPath to newPath.
+The operation can rename the file, move it to a different directory, or both.`,
+        parameters: RenameFileInputSchema,
+    },
+    needsBrowser: false,
+    isInlineTool: false,
+};

package/dist/tools/delete-file/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/tools/delete-file/index.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAEV,IAAI,EAEL,MAAM,4BAA4B,CAAC;AAOpC,eAAO,MAAM,cAAc,EAAE,IAiE5B,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/tools/delete-file/index.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAEV,IAAI,EAEL,MAAM,4BAA4B,CAAC;AASpC,eAAO,MAAM,cAAc,EAAE,IA4E5B,CAAC"}

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

@@ -4,8 +4,10 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.deleteFileTool = void 0;
+const child_process_1 = require("child_process");
 const promises_1 = require("fs/promises");
 const path_1 = __importDefault(require("path"));
+const web_1 = require("../../bin/utils/platform/web");
 const delete_file_1 = require("../definitions/delete-file");
 const git_1 = require("../executor/utils/git");
 exports.deleteFileTool = {
@@ -38,10 +40,12 @@ exports.deleteFileTool = {
                 };
             }
             await (0, promises_1.unlink)(filePath);
+            // Stage the deletion
+            (0, child_process_1.execSync)(`git add "${input.path}"`, { cwd: repoPath });
             // Collect git patch artifact after deletion
             if (collectArtifacts) {
                 try {
-                    const gitPatch = (0, git_1.getGitDiff)(input.path, repoPath);
+                    const gitPatch = (0, git_1.getGitDiffStaged)(input.path, repoPath);
                     if (gitPatch.trim()) {
                         const patchArtifact = {
                             name: `${path_1.default.basename(input.path, path_1.default.extname(input.path))}_delete.patch`,
@@ -55,6 +59,14 @@ exports.deleteFileTool = {
                     console.warn("Failed to create git patch artifact:", error);
                 }
             }
+            // Run TypeScript compilation check
+            const tscResult = await (0, web_1.runTypescriptCompiler)(repoPath);
+            if (!tscResult.success) {
+                return {
+                    result: `File ${input.path} has been deleted. However, type checks are failing with errors:\n\n${tscResult.errors.join("\n")}`,
+                    isError: true,
+                };
+            }
             return {
                 isError: false,
                 result: `Successfully deleted file: ${input.path}`,

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

@@ -1,6 +1,6 @@
 import { TraceClient } from "@empiricalrun/llm";
 import { PendingToolCall, ToolResult } from "@empiricalrun/llm/chat";
-import { ChatSessionInfo, IDashboardAPIClient, ToolExecute } from "@empiricalrun/shared-types";
+import type { ChatSessionInfo, IDashboardAPIClient, ToolExecute } from "@empiricalrun/shared-types";
 export declare class ToolExecutor {
     toolExecutors: Record<string, ToolExecute>;
     chatSession: ChatSessionInfo | null;

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

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/tools/executor/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,WAAW,EAAE,MAAM,mBAAmB,CAAC;AAChD,OAAO,EAAE,eAAe,EAAE,UAAU,EAAE,MAAM,wBAAwB,CAAC;AACrE,OAAO,EAEL,eAAe,EACf,mBAAmB,EACnB,WAAW,EACZ,MAAM,4BAA4B,CAAC;AAyBpC,qBAAa,YAAY;IACvB,aAAa,EAAE,MAAM,CAAC,MAAM,EAAE,WAAW,CAAC,CAAM;IAChD,WAAW,EAAE,eAAe,GAAG,IAAI,CAAC;IACpC,UAAU,EAAE,MAAM,CAAC;IACnB,QAAQ,EAAE,MAAM,CAAC;IACjB,SAAS,EAAE,mBAAmB,CAAC;IAC/B,KAAK,CAAC,EAAE,WAAW,CAAC;IACpB,YAAY,EAAE,MAAM,EAAE,CAAC;IACvB,oBAAoB,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;gBAEjC,EACV,WAAW,EACX,UAAU,EACV,QAAQ,EACR,SAAS,EACT,KAAK,EACL,YAAY,EACZ,oBAAyB,GAC1B,EAAE;QACD,WAAW,EAAE,eAAe,GAAG,IAAI,CAAC;QACpC,UAAU,EAAE,MAAM,CAAC;QACnB,QAAQ,EAAE,MAAM,CAAC;QACjB,SAAS,EAAE,mBAAmB,CAAC;QAC/B,KAAK,CAAC,EAAE,WAAW,CAAC;QACpB,YAAY,EAAE,MAAM,EAAE,CAAC;QACvB,oBAAoB,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;KAC/C;IA6CK,OAAO,CAAC,SAAS,EAAE,eAAe,EAAE,GAAG,OAAO,CAAC,UAAU,EAAE,CAAC;CAqEnE"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/tools/executor/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,WAAW,EAAE,MAAM,mBAAmB,CAAC;AAChD,OAAO,EAAE,eAAe,EAAE,UAAU,EAAE,MAAM,wBAAwB,CAAC;AACrE,OAAO,KAAK,EAEV,eAAe,EACf,mBAAmB,EACnB,WAAW,EACZ,MAAM,4BAA4B,CAAC;AA8BpC,qBAAa,YAAY;IACvB,aAAa,EAAE,MAAM,CAAC,MAAM,EAAE,WAAW,CAAC,CAAM;IAChD,WAAW,EAAE,eAAe,GAAG,IAAI,CAAC;IACpC,UAAU,EAAE,MAAM,CAAC;IACnB,QAAQ,EAAE,MAAM,CAAC;IACjB,SAAS,EAAE,mBAAmB,CAAC;IAC/B,KAAK,CAAC,EAAE,WAAW,CAAC;IACpB,YAAY,EAAE,MAAM,EAAE,CAAC;IACvB,oBAAoB,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;gBAEjC,EACV,WAAW,EACX,UAAU,EACV,QAAQ,EACR,SAAS,EACT,KAAK,EACL,YAAY,EACZ,oBAAyB,GAC1B,EAAE;QACD,WAAW,EAAE,eAAe,GAAG,IAAI,CAAC;QACpC,UAAU,EAAE,MAAM,CAAC;QACnB,QAAQ,EAAE,MAAM,CAAC;QACjB,SAAS,EAAE,mBAAmB,CAAC;QAC/B,KAAK,CAAC,EAAE,WAAW,CAAC;QACpB,YAAY,EAAE,MAAM,EAAE,CAAC;QACvB,oBAAoB,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;KAC/C;IAiDK,OAAO,CAAC,SAAS,EAAE,eAAe,EAAE,GAAG,OAAO,CAAC,UAAU,EAAE,CAAC;CA8EnE"}

package/dist/tools/executor/index.js

@@ -2,12 +2,15 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ToolExecutor = void 0;
 const artifacts_1 = require("../../artifacts");
-const commit_and_create_pr_1 = require("../commit-and-create-pr");
+const file_1 = require("../../utils/file");
+const create_pull_request_1 = require("../create-pull-request");
 const delete_file_1 = require("../delete-file");
 const diagnosis_fetcher_1 = require("../diagnosis-fetcher");
 const download_build_1 = require("../download-build");
-const fetch_image_1 = require("../fetch-image");
+const extract_frames_from_video_1 = require("../extract-frames-from-video");
+const fetch_file_1 = require("../fetch-file");
 const fetch_last_successful_test_run_1 = require("../fetch-last-successful-test-run");
+const fetch_session_diff_1 = require("../fetch-session-diff");
 const fetch_video_analysis_1 = require("../fetch-video-analysis");
 const file_operations_1 = require("../file-operations");
 const grep_1 = require("../grep");
@@ -15,6 +18,8 @@ const issues_1 = require("../issues");
 const list_environments_1 = require("../list-environments");
 const list_tests_and_projects_1 = require("../list-tests-and-projects");
 const merge_conflicts_1 = require("../merge-conflicts");
+const rename_file_1 = require("../rename-file");
+const review_pull_request_1 = require("../review-pull-request");
 const run_test_1 = require("../run-test");
 const test_gen_browser_1 = require("../test-gen-browser");
 const test_run_fetcher_1 = require("../test-run-fetcher");
@@ -47,22 +52,26 @@ class ToolExecutor {
             test_run_fetcher_1.fetchTestRunReportTool,
             diagnosis_fetcher_1.fetchDiagnosisReportTool,
             test_gen_browser_1.generateTestWithBrowserAgent,
-            commit_and_create_pr_1.createPullRequestTool,
+            create_pull_request_1.createPullRequestTool,
             merge_conflicts_1.mergeConflictsTool,
             list_environments_1.listEnvironmentsTool,
             issues_1.listIssuesTool,
             issues_1.createIssueTool,
             download_build_1.downloadBuildTool,
             upgrade_packages_1.upgradePackagesTool,
-            fetch_image_1.fetchImageTool,
+            fetch_file_1.fetchFileTool,
             fetch_video_analysis_1.fetchVideoAnalysis,
             delete_file_1.deleteFileTool,
+            rename_file_1.renameFileTool,
             trace_dot_zip_1.traceDotZipTool,
             view_failed_test_run_report_1.viewFailedTestRunReportTool,
             issues_1.updateIssueTool,
             fetch_last_successful_test_run_1.fetchLastSuccessfulTestRunTool,
             triage_summary_1.sendTriageSummaryTool,
             list_tests_and_projects_1.listProjectsTool,
+            extract_frames_from_video_1.extractFramesFromVideo,
+            fetch_session_diff_1.fetchSessionDiffTool,
+            review_pull_request_1.reviewPullRequestTool,
             ...file_operations_1.textEditorTools,
         ];
         tools.forEach((tool) => {
@@ -87,8 +96,15 @@ class ToolExecutor {
                 input: toolCall.input,
             });
             const uploadArtifactsQueue = new artifacts_1.UploadArtifactsQueue(this.repoPath, toolCall.id);
-            const collectArtifactsFn = (artifactsInput) => {
-                uploadArtifactsQueue.addTask(artifactsInput).catch((error) => {
+            const collectArtifactsFn = (artifactsInput, cleanUpDir) => {
+                uploadArtifactsQueue
+                    .addTask(artifactsInput)
+                    .then(() => {
+                    if (cleanUpDir) {
+                        void (0, file_1.safeCleanupDirectory)(cleanUpDir);
+                    }
+                })
+                    .catch((error) => {
                     console.error("Error collecting artifacts:", error);
                 });
             };
@@ -129,7 +145,6 @@ class ToolExecutor {
             }
         }
         (0, checkpoint_1.createCommitCheckpoint)({
-            toolCalls,
             branchName: this.branchName,
             repoPath: this.repoPath,
         });

package/dist/tools/executor/utils/checkpoint.d.ts

@@ -1,6 +1,4 @@
-import { PendingToolCall } from "@empiricalrun/llm/chat";
-export declare function createCommitCheckpoint({ toolCalls, branchName, repoPath, }: {
-    toolCalls: PendingToolCall[];
+export declare function createCommitCheckpoint({ branchName, repoPath, }: {
     branchName: string;
     repoPath: string;
 }): void;

package/dist/tools/executor/utils/checkpoint.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"checkpoint.d.ts","sourceRoot":"","sources":["../../../../src/tools/executor/utils/checkpoint.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,eAAe,EAAE,MAAM,wBAAwB,CAAC;AAKzD,wBAAgB,sBAAsB,CAAC,EACrC,SAAS,EACT,UAAU,EACV,QAAQ,GACT,EAAE;IACD,SAAS,EAAE,eAAe,EAAE,CAAC;IAC7B,UAAU,EAAE,MAAM,CAAC;IACnB,QAAQ,EAAE,MAAM,CAAC;CAClB,QA0BA"}
+{"version":3,"file":"checkpoint.d.ts","sourceRoot":"","sources":["../../../../src/tools/executor/utils/checkpoint.ts"],"names":[],"mappings":"AAIA,wBAAgB,sBAAsB,CAAC,EACrC,UAAU,EACV,QAAQ,GACT,EAAE;IACD,UAAU,EAAE,MAAM,CAAC;IACnB,QAAQ,EAAE,MAAM,CAAC;CAClB,QAsBA"}

package/dist/tools/executor/utils/checkpoint.js

@@ -3,24 +3,24 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.createCommitCheckpoint = createCommitCheckpoint;
 const child_process_1 = require("child_process");
 const git_1 = require("./git");
-function createCommitCheckpoint({ toolCalls, branchName, repoPath, }) {
-    const filesChanged = (0, git_1.getFilesChanged)(repoPath);
-    const toolsWithUpdatedFiles = toolCalls
-        .filter((tc) => "path" in tc.input &&
-        tc.input.path &&
-        filesChanged.includes(tc.input.path))
-        .map((toolCall) => ({
-        name: toolCall.name,
-        path: toolCall.input.path,
-    }));
-    const filesToCommit = toolsWithUpdatedFiles.map((tool) => tool.path);
-    if (toolsWithUpdatedFiles.length > 0) {
-        let commitMessage = `${toolsWithUpdatedFiles.map((tool) => `${tool.name} on ${tool.path}`).join(", ")}`;
-        (0, git_1.commitFilesAsBotUser)({
-            commitMessage,
-            files: filesToCommit,
-            repoPath,
+function createCommitCheckpoint({ branchName, repoPath, }) {
+    const filesChanged = (0, git_1.getStagedFiles)(repoPath);
+    if (filesChanged.length > 0) {
+        const commitMessages = filesChanged.map((change) => {
+            switch (change.type) {
+                case "added":
+                    return `added ${change.path}`;
+                case "modified":
+                    return `modified ${change.path}`;
+                case "removed":
+                    return `removed ${change.path}`;
+                case "renamed":
+                    return `renamed ${change.oldPath} → ${change.path}`;
+            }
         });
+        const commitMessage = commitMessages.join(", ");
+        // Files are already staged by individual tools, just commit
+        (0, git_1.commitAsBotUser)(commitMessage, repoPath);
         (0, child_process_1.execSync)(`git push origin ${branchName}`, { cwd: repoPath });
     }
 }

package/dist/tools/executor/utils/git.d.ts

@@ -1,15 +1,23 @@
 import { IDashboardAPIClient } from "@empiricalrun/shared-types";
 export declare function getGitDiff(filepath: string, cwd: string): string;
+export declare function getGitDiffStaged(filepath: string, cwd: string): string;
+export declare function getGitDiffForStagedOrUnstaged(filepath: string, cwd: string): string;
 export declare function getGitDiffForNewFile(filepath: string, cwd: string): string;
 export declare function checkoutBranch(branchName: string, cwd: string): Promise<void>;
 export declare function commitAsBotUser(commitMessage: string, cwd: string): boolean;
 export declare function getCurrentBranchName(repoPath: string): Promise<string>;
 export declare function pullBranch(branchName: string, cwd: string): Promise<void>;
-export declare function commitFilesAsBotUser({ commitMessage, files, repoPath, }: {
+export declare function stageAndCommitFilesAsBotUser({ commitMessage, files, repoPath, }: {
     commitMessage: string;
     files: string[];
     repoPath: string;
 }): void;
+export type FileChange = {
+    type: "added" | "modified" | "removed" | "renamed";
+    path: string;
+    oldPath?: string;
+};
+export declare function getStagedFiles(cwd: string): FileChange[];
 export declare function getFilesChanged(cwd: string): string[];
 export declare function mergePullRequest({ repoName, prNumber, apiClient, }: {
     repoName: string;

package/dist/tools/executor/utils/git.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"git.d.ts","sourceRoot":"","sources":["../../../../src/tools/executor/utils/git.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,mBAAmB,EAAE,MAAM,4BAA4B,CAAC;AAMjE,wBAAgB,UAAU,CAAC,QAAQ,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,GAAG,MAAM,CAMhE;AAED,wBAAgB,oBAAoB,CAAC,QAAQ,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,GAAG,MAAM,CAe1E;AAED,wBAAsB,cAAc,CAAC,UAAU,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,iBAMnE;AAED,wBAAgB,eAAe,CAAC,aAAa,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,WAcjE;AAED,wBAAsB,oBAAoB,CAAC,QAAQ,EAAE,MAAM,mBAO1D;AAED,wBAAsB,UAAU,CAAC,UAAU,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,iBAE/D;AAED,wBAAgB,oBAAoB,CAAC,EACnC,aAAa,EACb,KAAK,EACL,QAAQ,GACT,EAAE;IACD,aAAa,EAAE,MAAM,CAAC;IACtB,KAAK,EAAE,MAAM,EAAE,CAAC;IAChB,QAAQ,EAAE,MAAM,CAAC;CAClB,QAGA;AAED,wBAAgB,eAAe,CAAC,GAAG,EAAE,MAAM,YAU1C;AAED,wBAAsB,gBAAgB,CAAC,EACrC,QAAQ,EACR,QAAQ,EACR,SAAS,GACV,EAAE;IACD,QAAQ,EAAE,MAAM,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;IACjB,SAAS,EAAE,mBAAmB,CAAC;CAChC,oBA8BA"}
+{"version":3,"file":"git.d.ts","sourceRoot":"","sources":["../../../../src/tools/executor/utils/git.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,mBAAmB,EAAE,MAAM,4BAA4B,CAAC;AAMjE,wBAAgB,UAAU,CAAC,QAAQ,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,GAAG,MAAM,CAMhE;AAED,wBAAgB,gBAAgB,CAAC,QAAQ,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,GAAG,MAAM,CAMtE;AAED,wBAAgB,6BAA6B,CAC3C,QAAQ,EAAE,MAAM,EAChB,GAAG,EAAE,MAAM,GACV,MAAM,CAaR;AAED,wBAAgB,oBAAoB,CAAC,QAAQ,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,GAAG,MAAM,CAe1E;AAED,wBAAsB,cAAc,CAAC,UAAU,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,iBAMnE;AAED,wBAAgB,eAAe,CAAC,aAAa,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,WAcjE;AAED,wBAAsB,oBAAoB,CAAC,QAAQ,EAAE,MAAM,mBAO1D;AAED,wBAAsB,UAAU,CAAC,UAAU,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,iBAE/D;AAED,wBAAgB,4BAA4B,CAAC,EAC3C,aAAa,EACb,KAAK,EACL,QAAQ,GACT,EAAE;IACD,aAAa,EAAE,MAAM,CAAC;IACtB,KAAK,EAAE,MAAM,EAAE,CAAC;IAChB,QAAQ,EAAE,MAAM,CAAC;CAClB,QAGA;AAED,MAAM,MAAM,UAAU,GAAG;IACvB,IAAI,EAAE,OAAO,GAAG,UAAU,GAAG,SAAS,GAAG,SAAS,CAAC;IACnD,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,CAAC,EAAE,MAAM,CAAC;CAClB,CAAC;AAEF,wBAAgB,cAAc,CAAC,GAAG,EAAE,MAAM,GAAG,UAAU,EAAE,CA6CxD;AAED,wBAAgB,eAAe,CAAC,GAAG,EAAE,MAAM,GAAG,MAAM,EAAE,CAUrD;AAED,wBAAsB,gBAAgB,CAAC,EACrC,QAAQ,EACR,QAAQ,EACR,SAAS,GACV,EAAE;IACD,QAAQ,EAAE,MAAM,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;IACjB,SAAS,EAAE,mBAAmB,CAAC;CAChC,oBA8BA"}

package/dist/tools/executor/utils/git.js

@@ -1,12 +1,15 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.getGitDiff = getGitDiff;
+exports.getGitDiffStaged = getGitDiffStaged;
+exports.getGitDiffForStagedOrUnstaged = getGitDiffForStagedOrUnstaged;
 exports.getGitDiffForNewFile = getGitDiffForNewFile;
 exports.checkoutBranch = checkoutBranch;
 exports.commitAsBotUser = commitAsBotUser;
 exports.getCurrentBranchName = getCurrentBranchName;
 exports.pullBranch = pullBranch;
-exports.commitFilesAsBotUser = commitFilesAsBotUser;
+exports.stageAndCommitFilesAsBotUser = stageAndCommitFilesAsBotUser;
+exports.getStagedFiles = getStagedFiles;
 exports.getFilesChanged = getFilesChanged;
 exports.mergePullRequest = mergePullRequest;
 const child_process_1 = require("child_process");
@@ -19,6 +22,27 @@ function getGitDiff(filepath, cwd) {
     });
     return diff;
 }
+function getGitDiffStaged(filepath, cwd) {
+    const diff = (0, child_process_1.execSync)(`git diff --cached ${filepath}`, {
+        encoding: "utf-8",
+        cwd,
+    });
+    return diff;
+}
+function getGitDiffForStagedOrUnstaged(filepath, cwd) {
+    // First try to get staged changes (index vs HEAD)
+    try {
+        const stagedDiff = getGitDiffStaged(filepath, cwd);
+        if (stagedDiff.trim()) {
+            return stagedDiff;
+        }
+    }
+    catch {
+        // Fall through to try unstaged diff
+    }
+    // If no staged changes, try unstaged changes (working dir vs index)
+    return getGitDiff(filepath, cwd);
+}
 function getGitDiffForNewFile(filepath, cwd) {
     try {
         const diff = (0, child_process_1.execSync)(`git diff --no-index /dev/null ${filepath}`, {
@@ -67,10 +91,56 @@ async function getCurrentBranchName(repoPath) {
 async function pullBranch(branchName, cwd) {
     (0, child_process_1.execSync)(`git pull origin ${branchName}`, { cwd });
 }
-function commitFilesAsBotUser({ commitMessage, files, repoPath, }) {
+function stageAndCommitFilesAsBotUser({ commitMessage, files, repoPath, }) {
     (0, child_process_1.execSync)(`git add ${files.join(" ")}`, { cwd: repoPath });
     commitAsBotUser(commitMessage, repoPath);
 }
+function getStagedFiles(cwd) {
+    const output = (0, child_process_1.execSync)("git diff --cached --name-status", {
+        cwd,
+    }).toString();
+    const filesChanged = [];
+    output.split("\n").forEach((line) => {
+        const trimmed = line.trim();
+        if (!trimmed)
+            return;
+        const parts = trimmed.split("\t");
+        if (parts.length < 2)
+            return;
+        const status = parts[0];
+        const filePath = parts[1];
+        if (status.startsWith("R")) {
+            // Handle rename: "R100	old_file	new_file"
+            const newFilePath = parts[2];
+            if (newFilePath) {
+                filesChanged.push({
+                    type: "renamed",
+                    path: newFilePath,
+                    oldPath: filePath,
+                });
+            }
+        }
+        else if (status === "A") {
+            filesChanged.push({
+                type: "added",
+                path: filePath,
+            });
+        }
+        else if (status === "M") {
+            filesChanged.push({
+                type: "modified",
+                path: filePath,
+            });
+        }
+        else if (status === "D") {
+            filesChanged.push({
+                type: "removed",
+                path: filePath,
+            });
+        }
+    });
+    return filesChanged;
+}
 function getFilesChanged(cwd) {
     const output = (0, child_process_1.execSync)("git status --porcelain --untracked-files", {
         cwd,

package/dist/tools/extract-frames-from-video/index.d.ts

@@ -0,0 +1,7 @@
+import type { Tool } from "@empiricalrun/shared-types";
+import type { z } from "zod";
+import { type extractFramesFromVideoSchema } from "../definitions/extract-frames-from-video";
+type ExtractFramesInput = z.infer<typeof extractFramesFromVideoSchema>;
+export declare const extractFramesFromVideo: Tool<ExtractFramesInput>;
+export {};
+//# sourceMappingURL=index.d.ts.map

package/dist/tools/extract-frames-from-video/index.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/tools/extract-frames-from-video/index.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAEV,IAAI,EAEL,MAAM,4BAA4B,CAAC;AAEpC,OAAO,KAAK,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAI7B,OAAO,EAEL,KAAK,4BAA4B,EAClC,MAAM,0CAA0C,CAAC;AAElD,KAAK,kBAAkB,GAAG,CAAC,CAAC,KAAK,CAAC,OAAO,4BAA4B,CAAC,CAAC;AA0EvE,eAAO,MAAM,sBAAsB,EAAE,IAAI,CAAC,kBAAkB,CAkH3D,CAAC"}