davinci-resolve-mcp 2.134.1 → 2.135.0

package/install.py

@@ -37,7 +37,7 @@ from src.utils.update_check import (
 
 # ─── Version ──────────────────────────────────────────────────────────────────
 
-VERSION = "2.134.1"
+VERSION = "2.135.0"
 # Only hard floor: mcp[cli] requires Python 3.10+. There is no upper bound —
 # Resolve's scripting bridge loads into newer interpreters on recent builds
 # (Python 3.14 verified against Resolve Studio 20.3.2). Older Resolve builds

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "davinci-resolve-mcp",
-  "version": "2.134.1",
+  "version": "2.135.0",
   "description": "NPM bootstrapper for the DaVinci Resolve MCP Server.",
   "license": "MIT",
   "author": "Samuel Gursky <samgursky@gmail.com>",

package/resolve-advanced/server/tools/audio.mjs

@@ -22,9 +22,19 @@ function load(mod) {
   return require(`../../vendor/audio/${mod}.js`);
 }
 
-const splitSchema = z.object({ input: z.string(), output: z.string().optional(), mode: z.string().optional(), opts: z.object({}).passthrough().optional() });
-const trimSchema = z.object({ input: z.string(), output: z.string(), durationFrames: z.number().optional(), opts: z.object({}).passthrough().optional() });
-const convertSchema = z.object({ input: z.string(), output: z.string(), opts: z.object({}).passthrough().optional() });
+// .strict(): a mistyped window key (start/duration/…) must refuse, not be
+// silently stripped — a stripped window turned `trim` into a whole-file copy
+// reported as success (measured through the MCP layer, E59 sweep). Extra
+// ffmpeg knobs belong in `opts`, which stays passthrough.
+const splitSchema = z.object({ input: z.string(), output: z.string().optional(), mode: z.string().optional(), opts: z.object({}).passthrough().optional() }).strict();
+// durationFrames is REQUIRED: trim without a window is a no-op copy wearing
+// a trim's name. Use `convert` for windowless transcodes. fps converts it to
+// the vendored module's seconds vocabulary (default 24) — the module knows
+// {startTime, endTime, duration} in SECONDS only, so an untranslated
+// durationFrames was silently discarded and trim NEVER trimmed (measured
+// through the MCP layer, E59: durationFrames 24 returned the whole file).
+const trimSchema = z.object({ input: z.string(), output: z.string(), durationFrames: z.number().positive(), fps: z.number().positive().optional(), opts: z.object({}).passthrough().optional() }).strict();
+const convertSchema = z.object({ input: z.string(), output: z.string(), opts: z.object({}).passthrough().optional() }).strict();
 
 export const audioTool = {
   name: 'audio',
@@ -41,7 +51,7 @@ export const audioTool = {
       const p = trimSchema.parse(args);
       const m = load('trim');
       const fn = m.trimAudio || m.trim || m.default;
-      return fn(p.input, p.output, { durationFrames: p.durationFrames, ...(p.opts || {}) });
+      return fn(p.input, p.output, { duration: p.durationFrames / (p.fps ?? 24), ...(p.opts || {}) });
     }
     if (action === 'convert') {
       const p = convertSchema.parse(args);

package/src/granular/common.py

@@ -87,7 +87,7 @@ if not logging.getLogger().handlers:
         handlers=[logging.StreamHandler()],
     )
 
-VERSION = "2.134.1"
+VERSION = "2.135.0"
 logger = logging.getLogger("davinci-resolve-mcp")
 logger.info(f"Starting DaVinci Resolve MCP Server v{VERSION}")
 logger.info(f"Detected platform: {get_platform()}")

package/src/server.py

@@ -11,7 +11,7 @@ Usage:
     python src/server.py --full       # Start the 353-tool granular server instead
 """
 
-VERSION = "2.134.1"
+VERSION = "2.135.0"
 
 import base64
 import os