goatedit-mcp 1.1.20 → 1.1.21

package/bundle/goatedit-mcp.js

@@ -20043,6 +20043,16 @@ var BASE_TOOLS = [
     },
     method: "getEditLog"
   },
+  {
+    name: "get_operation",
+    description: 'Where an edit stands after its call timed out or the connection dropped: queued, running, done (with its reply) or failed, by the opId the error gave. "unknown" means the tab never got it, so sending it again is safe. To retry, repeat the call with that same opId: the tab runs an opId once and answers a repeat with the first reply, so a retry never makes a clip twice.',
+    inputSchema: {
+      type: "object",
+      properties: { opId: { type: "string", description: "The id from the timeout or disconnect message." } },
+      required: ["opId"]
+    },
+    method: "getOperation"
+  },
   {
     name: "clean_unused_sequences",
     description: "Find nested sequences that no clip uses any more \u2014 left behind when a nest clip is removed or a shot is rebuilt \u2014 and optionally delete them with their bin items. Without apply it only lists them. A scene (a sequence once placed with add_sequence_clip) counts as a nest. Other top-level sequences, and nests reachable from one or open in a tab, are never touched; top-level ones nothing plays are listed under otherUnplayed for delete_sequence.",
@@ -22035,6 +22045,7 @@ var TOOLS_BY_NAME = new Map([
 var READ_ONLY_TOOLS = /* @__PURE__ */ new Set([
   "get_session_status",
   "get_design_guide",
+  "get_operation",
   "get_project_state",
   "get_transcript",
   "render_preview_frame",
@@ -22423,6 +22434,10 @@ function withRunSequence(stepArgs, sequenceId, tool) {
   if (!tool || !("expectedSequenceId" in (tool.inputSchema.properties ?? {}))) return stepArgs;
   return { ...stepArgs, expectedSequenceId: sequenceId };
 }
+function withRunOp(stepArgs, runOpId, index2) {
+  if (!runOpId || typeof stepArgs.opId === "string") return stepArgs;
+  return { ...stepArgs, opId: `${runOpId}.${index2 + 1}` };
+}
 
 // src/tab-registry.ts
 function indexTabs(reports, isOpen) {
@@ -25250,13 +25265,16 @@ function resolveTab(projectId) {
   if (!id) throw new Error(notConnectedMessage());
   return tabs.get(id);
 }
+function unsettled(message) {
+  return Object.assign(new Error(message), { unsettled: true });
+}
 var pendingRequests = /* @__PURE__ */ new Map();
 function rejectPendingOn(socket, why) {
   for (const [id, pending] of pendingRequests) {
     if (pending.socket !== socket) continue;
     clearTimeout(pending.timeoutId);
     pendingRequests.delete(id);
-    pending.reject(new Error(`${why} before ${pending.method} answered. Check the editor tab, then call get_timeline to see whether the edit landed.`));
+    pending.reject(unsettled(`${why} before ${pending.method} answered. Check the editor tab.`));
   }
 }
 function explainTabError(error2) {
@@ -25397,7 +25415,7 @@ function failUpstream(why) {
   for (const [id, p] of upstreamPending) {
     clearTimeout(p.timeoutId);
     upstreamPending.delete(id);
-    p.reject(new Error(`${why} before ${p.method} answered.`));
+    p.reject(unsettled(`${why} before ${p.method} answered.`));
   }
 }
 function connectUpstream() {
@@ -25455,7 +25473,7 @@ function sendUpstream(method, params, projectId) {
     const id = crypto2.randomUUID();
     const timeoutId = setTimeout(() => {
       upstreamPending.delete(id);
-      reject(new Error(`Request timed out: ${method}`));
+      reject(unsettled(`Request timed out: ${method}`));
     }, REQUEST_TIMEOUT_MS);
     upstreamPending.set(id, { resolve, reject, timeoutId, method });
     ws.send(JSON.stringify({ id, method, params, ...projectId ? { projectId } : {} }));
@@ -25469,7 +25487,7 @@ async function sendToBrowser(method, params = {}, projectId) {
     const id = crypto2.randomUUID();
     const timeoutId = setTimeout(() => {
       pendingRequests.delete(id);
-      reject(new Error(`Request timed out after ${REQUEST_TIMEOUT_MS / 1e3}s: ${method}. Call get_timeline to see whether it landed.`));
+      reject(unsettled(`Request timed out after ${REQUEST_TIMEOUT_MS / 1e3}s: ${method}.`));
     }, REQUEST_TIMEOUT_MS);
     pendingRequests.set(id, { resolve, reject, timeoutId, socket: tab.ws, method });
     tab.ws.send(JSON.stringify({ id, method, params, projectId: tab.projectId }));
@@ -25576,7 +25594,8 @@ var LOCAL_TOOLS = [
         continueOnError: { type: "boolean", description: "Run the remaining steps after one fails." },
         rollback: { type: "boolean", description: "Take a checkpoint first and restore it if any step fails, so a run either lands whole or not at all. Cannot be combined with continueOnError." },
         projectId: { type: "string", description: "Run every step on this project (from list_connected_projects). Use it when several agents edit different tabs at once; a step's own projectId wins." },
-        expectedSequenceId: { type: "string", description: "Run every step that takes expectedSequenceId on this sequence (from list_sequences), open or not. Use it when several agents edit different sequences at once; a step's own expectedSequenceId wins." }
+        expectedSequenceId: { type: "string", description: "Run every step that takes expectedSequenceId on this sequence (from list_sequences), open or not. Use it when several agents edit different sequences at once; a step's own expectedSequenceId wins." },
+        opId: { type: "string", description: "Only to retry a run that failed or timed out: the opId its reply or error gave, with the same calls in the same order. Steps that already finished are answered without running again." }
       },
       required: ["calls"]
     }
@@ -25590,6 +25609,7 @@ for (const [alias, name] of Object.entries(TOOL_ALIASES)) {
   if (tool) BROWSER_TOOLS_BY_NAME.set(alias, tool);
 }
 var PROJECT_FREE_TOOLS = /* @__PURE__ */ new Set(["create_project", "begin_request", "record_choice", "set_style_brand"]);
+var VIEW_TOOLS = /* @__PURE__ */ new Set(["switch_project", "open_sequence", "playback_control", "set_playhead", "set_tool"]);
 var SAVES_FRAMES = new Set(
   BROWSER_TOOLS.filter((t) => "saveToDisk" in (t.inputSchema.properties ?? {})).map((t) => t.name)
 );
@@ -25601,7 +25621,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => ({
   ]
 }));
 async function callTool(name, args) {
-  const { projectId: rawProjectId, ...forwarded } = args;
+  const { projectId: rawProjectId, opId: rawOpId, ...forwarded } = args;
   const projectId = typeof rawProjectId === "string" && rawProjectId ? rawProjectId : void 0;
   switch (name) {
     case "get_design_guide":
@@ -25631,6 +25651,7 @@ async function callTool(name, args) {
   }
   const saveToDisk = SAVES_FRAMES.has(name) && forwarded.saveToDisk !== false;
   if (SAVES_FRAMES.has(name)) forwarded.saveToDisk = false;
+  if (name === "get_operation") forwarded.opId = rawOpId;
   const invalid = validateArgs(tool, forwarded);
   if (invalid) throw new McpError(ErrorCode.InvalidParams, invalid);
   const target = projectId ?? chosenProjectId;
@@ -25643,11 +25664,25 @@ async function callTool(name, args) {
       );
     }
   }
-  if (name === "import_local_file") return importLocalFile(forwarded, target);
-  const result = await sendToBrowser(tool.method, forwarded, target);
+  if (!READ_ONLY_TOOLS.has(tool.name) && !VIEW_TOOLS.has(tool.name)) {
+    forwarded.opId = typeof rawOpId === "string" && rawOpId ? rawOpId : `op-${crypto2.randomUUID().slice(0, 8)}`;
+  }
+  if (name === "import_local_file") return withOpIdHint(importLocalFile(forwarded, target), forwarded.opId);
+  const result = await withOpIdHint(sendToBrowser(tool.method, forwarded, target), forwarded.opId);
   const withFile = saveFileToDisk(result);
   return saveToDisk ? saveFramesToDisk(withFile, name) : withFile;
 }
+async function withOpIdHint(call, opId) {
+  try {
+    return await call;
+  } catch (err) {
+    const message = err?.message ?? String(err);
+    if (typeof opId !== "string" || !err?.unsettled) throw err;
+    throw new Error(
+      `${message} The edit may still finish. It is operation ${opId}: call get_operation with opId "${opId}" to see whether it did, or repeat this exact call with opId "${opId}" \u2014 the tab runs an opId once, so the retry cannot make anything twice.`
+    );
+  }
+}
 async function importLocalFile(args, target) {
   const local = await callLocal("find_local_file", { filename: args.filename }, target).catch((err) => {
     if (/^Several files/.test(err?.message ?? "")) throw err;
@@ -25676,6 +25711,7 @@ async function runTools(args) {
   const images = [];
   const refs = /* @__PURE__ */ new Map();
   let failed = 0;
+  const runOpId = args.rollback ? null : typeof args.opId === "string" && args.opId ? args.opId : `op-${crypto2.randomUUID().slice(0, 8)}`;
   let checkpointId;
   if (args.rollback) {
     const made = await callTool("create_checkpoint", withRunProject({ name: "run_tools rollback" }, args.projectId));
@@ -25686,7 +25722,7 @@ async function runTools(args) {
     const tool = typeof step?.tool === "string" ? step.tool : "";
     try {
       const stepArgs = resolveRefs(step.args && typeof step.args === "object" ? step.args : {}, refs);
-      const raw = await callTool(tool, withRunSequence(withRunProject(stepArgs, args.projectId), args.expectedSequenceId, BROWSER_TOOLS_BY_NAME.get(tool)));
+      const raw = await callTool(tool, withRunOp(withRunSequence(withRunProject(stepArgs, args.projectId), args.expectedSequenceId, BROWSER_TOOLS_BY_NAME.get(tool)), runOpId, i));
       if (typeof step.as === "string" && step.as) {
         if (raw && typeof raw === "object" && !Array.isArray(raw)) {
           const { _image, _images, ...kept } = raw;
@@ -25725,6 +25761,7 @@ async function runTools(args) {
   return {
     ok: failed === 0,
     results,
+    ...failed && runOpId ? { opId: runOpId, retry: `Repeat this run_tools call with opId "${runOpId}": finished steps answer from the tab without running again.` } : {},
     ...rolledBack !== void 0 ? { rolledBack } : {},
     ...images.length ? { _images: images } : {}
   };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "goatedit-mcp",
-  "version": "1.1.20",
+  "version": "1.1.21",
   "description": "Local MCP server for the GoatEdit video editor: drive your open editor tab from Claude Code or any MCP client",
   "license": "UNLICENSED",
   "type": "module",