goatedit-mcp 1.1.22 → 1.1.23

package/bundle/goatedit-mcp.js

@@ -20413,7 +20413,7 @@ var BASE_TOOLS = [
   },
   {
     name: "update_shape_clip",
-    description: "Change a shape after it was made \u2014 fill, size, corners, outline, and a dashed or rounded outline \u2014 without deleting it and losing its id, keyframes and effects.",
+    description: "Change a shape after it was made \u2014 fill, size, position, length, corners, outline, and a dashed or rounded outline \u2014 without deleting it and losing its id, keyframes and effects.",
     inputSchema: {
       type: "object",
       properties: {
@@ -20433,7 +20433,10 @@ var BASE_TOOLS = [
         closed: bool("Path shapes: join the last anchor back to the first."),
         trimStart: num("Path shapes: where along the path drawing starts, 0..1. Keyframe it as pathTrimStart."),
         trimEnd: num("Path shapes: where drawing stops, 0..1. 0 hides the path; keyframing pathTrimEnd 0 to 1 with set_keyframes draws it on."),
-        trimOffset: num("Path shapes: slides the drawn stretch along the path, degrees.")
+        trimOffset: num("Path shapes: slides the drawn stretch along the path, degrees."),
+        x: num("Centre across the frame, 0..1 (sets posX; keyframes on posX still win)."),
+        y: num("Centre down the frame, 0..1 (sets posY)."),
+        duration: num("New length in seconds, from the same start. Refused if it would run into the next clip on the track.")
       },
       required: ["clipId"]
     },
@@ -21434,7 +21437,8 @@ var BASE_TOOLS = [
         height: num("Render height in pixels. Defaults to the preset's, or the project height."),
         posX: num("Horizontal centre as a fraction of the frame, 0..1. 0.5 is centre."),
         posY: num("Vertical centre as a fraction of the frame, 0..1. 0.5 is centre."),
-        scale: num("Uniform scale, 1 being unchanged."),
+        scale: num('Uniform scale, 1 being unchanged. With size "native", a multiplier on the native size.'),
+        size: str(`"fit" (default): like any picture, the width x height box is fitted inside the frame, so a 600x200 card fills the frame width. "native": one authored px is one frame px, so a 600x200 card is 600x200 on screen. Use native for UI pieces authored at their real size. The result's onScreen gives the box in frame px either way.`),
         opacity: num("Opacity, 0..1. Defaults to 1."),
         name: str("Display name for the clip and its bin entry.")
       },
@@ -21624,10 +21628,11 @@ var BASE_TOOLS = [
       type: "object",
       properties: {
         clipId: CLIP_ID,
-        newStartTime: num("New timeline start time in seconds."),
+        startTime: num("New timeline start time in seconds."),
+        newStartTime: num("Same as startTime (older name)."),
         newTrackId: str('Track to move onto. Defaults to the clip\u2019s current track. Accepts "__new_video__" to pick a video track free across the clip\u2019s span, or "__new_track__" for a new video track on top.')
       },
-      required: ["clipId", "newStartTime"]
+      required: ["clipId"]
     },
     method: "moveClip"
   },
@@ -25719,6 +25724,42 @@ async function importLocalFile(args, target) {
     );
   }
 }
+var RUN_STEP_COMMON_ARGS = /* @__PURE__ */ new Set(["projectId", "expectedSequenceId", "opId"]);
+function preflightSteps(calls) {
+  const errors = [], warnings = [];
+  calls.forEach((raw, i) => {
+    const step = raw;
+    const n = i + 1;
+    const tool = typeof step?.tool === "string" ? step.tool : "";
+    if (!tool) {
+      errors.push(`step ${n}: no tool name.`);
+      return;
+    }
+    if (tool === "run_tools") {
+      errors.push(`step ${n}: run_tools cannot be nested.`);
+      return;
+    }
+    const def = BROWSER_TOOLS_BY_NAME.get(tool);
+    if (!def) return;
+    const args = step.args && typeof step.args === "object" ? step.args : {};
+    const props = def.inputSchema?.properties ?? {};
+    for (const key of def.inputSchema?.required ?? []) {
+      if (args[key] === void 0) errors.push(`step ${n} (${tool}): missing required "${key}".`);
+    }
+    for (const [key, value] of Object.entries(args)) {
+      const spec = props[key];
+      if (!spec) {
+        if (!RUN_STEP_COMMON_ARGS.has(key)) warnings.push(`step ${n} (${tool}): "${key}" is not an argument of ${tool} and is ignored. It takes: ${Object.keys(props).join(", ")}.`);
+        continue;
+      }
+      if (typeof value === "string" && value.startsWith("$")) continue;
+      const want = spec.type;
+      const ok = !want || value === null || (want === "number" || want === "integer" ? typeof value === "number" : want === "string" ? typeof value === "string" : want === "boolean" ? typeof value === "boolean" : want === "array" ? Array.isArray(value) : want === "object" ? typeof value === "object" && !Array.isArray(value) : true);
+      if (!ok) errors.push(`step ${n} (${tool}): "${key}" should be ${want}, got ${Array.isArray(value) ? "array" : typeof value}.`);
+    }
+  });
+  return { errors, warnings };
+}
 async function runTools(args) {
   const calls = args.calls;
   if (!Array.isArray(calls) || !calls.length) {
@@ -25727,6 +25768,16 @@ async function runTools(args) {
   if (args.rollback && args.continueOnError) {
     throw new McpError(ErrorCode.InvalidParams, "rollback and continueOnError cannot be combined: a run that keeps going after a failure has nothing to roll back to.");
   }
+  const check = preflightSteps(calls);
+  if (check.errors.length) {
+    return {
+      ok: false,
+      ran: 0,
+      errors: check.errors,
+      note: "Nothing ran: fix these and send the run again.",
+      ...check.warnings.length ? { warnings: check.warnings } : {}
+    };
+  }
   const results = [];
   const images = [];
   const refs = /* @__PURE__ */ new Map();
@@ -25781,6 +25832,8 @@ async function runTools(args) {
   return {
     ok: failed === 0,
     results,
+    ...check.warnings.length ? { warnings: check.warnings } : {},
+    ...failed && !args.continueOnError && !args.rollback ? { hint: "One failing step skips the rest. Pass continueOnError: true to run the other steps anyway." } : {},
     ...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.22",
+  "version": "1.1.23",
   "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",