goatedit-mcp 1.1.22 → 1.1.24

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"]
     },
@@ -21174,6 +21177,7 @@ var BASE_TOOLS = [
           items: { type: "object" }
         },
         clear: bool("Remove this property's keyframes instead of setting them, leaving the clip's flat value in charge again."),
+        offset: { description: 'With prop "position" or "scale" and no keyframes: move the animation as it is. Every existing key of both halves is shifted ([dx, dy] added to position) or scaled (a number or [sx, sy] multiplying scale), times and eases kept; a track with no keys has its resting value moved instead. The way to nudge an animated layer or a whole pile without rewriting its keys.' },
         relative: bool("Values are relative to where the layer rests: multiples of its scale/opacity (1.1 = 10% bigger than it is), offsets for everything else (posX +0.05, rotation +15). A punch-in on a card scaled 0.3 stays a punch-in instead of jumping to full size."),
         expression: {
           type: "string",
@@ -21434,7 +21438,9 @@ 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.'),
+        rotation: num("Rotation in degrees."),
+        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. The clip's stored scale stays relative to the fitted box (onScreen.nativeScale is native size), so set_clip_transform and scale keys later take that number, not 1; set_clip_transform on an html clip replies with onScreen too.`),
         opacity: num("Opacity, 0..1. Defaults to 1."),
         name: str("Display name for the clip and its bin entry.")
       },
@@ -21624,10 +21630,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 +25726,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 +25770,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 +25834,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.24",
   "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",