davinci-resolve-mcp 2.134.0 → 2.135.0

package/docs/guides/native-drt-authoring.md

@@ -55,7 +55,7 @@ window. `render.verify_output` covers the container-level checks.
 | Subtitles | `spec.subtitles` / `spec.subtitlesSrt` (raw SRT) | v2.128 |
 | Compound clips | survive `extract_from_drp` → `.drt` (inner containers kept, recursive) | v2.130 |
 | Compound authoring | `spec.compounds` (multiple compose; nested edits render) | v2.131–2.132 |
-| Nested compounds | `compounds[].compounds` (depth-2 playback render-verified) | v2.134 |
+| Nested compounds | `compounds[].compounds` (depth-2 and depth-3 playback render-verified) | v2.134 |
 | Fusion titles | `elements: [{type:'title', text}]` — **21-gen hosts only** | v2.108 |
 
 `assemble_from_interchange` drives the same engine from an EDL / OTIO /

package/install.py

@@ -37,7 +37,7 @@ from src.utils.update_check import (
 
 # ─── Version ──────────────────────────────────────────────────────────────────
 
-VERSION = "2.134.0"
+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.0",
+  "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/resolve-advanced/server/tools/drt.mjs

@@ -51,7 +51,7 @@ const assembleSchema = z.object({
   spec: z
     .object({})
     .passthrough()
-    .describe("assembleTimeline spec: { timelineName?, startFrame? (timeline start frame @24, default 86400=01:00:00:00 — sets the start TIMECODE, render-verified on 19), media?: {mediaFilePath, spec:{width,height,frameCount,fps}, cuts:[{startFrame,durationFrames,srcIn?,track? (1-based video track; >1 = video-only, render-verified stacking),speed?/reverse? (constant retime, e.g. 0.5, forward or backwards; video-only; readback+render-verified on 19),freeze? (true = hold source frame srcIn for the whole cut; video-only; render-proven frozen on 19 via freezedetect),audioOnly?+track? (explicit AUDIO placement on audio track 1-8; presence suppresses the A1 mirror; render-verified on 19)}]} | [same, ...] (multi-source needs media_pool.capture_media_template run once per file), transitions?: [{track, atFrame, durationFrames?, trackType? ('video' dissolve | 'audio' cross-fade, both render-verified on 19)}], markers?: [{frame (timeline-absolute), color? (16 names), name?, note?, duration?, customData?}] (readback-verified on 19), compounds?: [{name, startFrame (parent, absolute), durationFrames, track?, cuts:[{mediaFilePath, startFrame (INNER, 0-based), durationFrames, srcIn?}], compounds?: [same, nested — frames inner-relative]}] (multiple PARALLEL compounds compose AND compounds NEST — depth-2 playback render-verified on 19 (the old depth-2 black was a missing SequenceSetup key, fixed); deeper nesting composes but is unverified; inner cuts need captured templates w/ native clips), subtitles?: [{startFrame (timeline-absolute), durationFrames, text}] + subtitlesSrt? (raw SRT, cues anchor at the origin; readback-verified on 19; angle-bracket runs read as SRT markup), elements?: [{type:'title'|'generator', track, startFrame, durationFrames?, text?, generatorName? ('Solid Color'|'SMPTE Color Bar'|'Grey Scale' render-verified on 19), ...}] }. startFrame is timeline-absolute (origin 86400)."),
+    .describe("assembleTimeline spec: { timelineName?, startFrame? (timeline start frame @24, default 86400=01:00:00:00 — sets the start TIMECODE, render-verified on 19), media?: {mediaFilePath, spec:{width,height,frameCount,fps}, cuts:[{startFrame,durationFrames,srcIn?,track? (1-based video track; >1 = video-only, render-verified stacking),speed?/reverse? (constant retime, e.g. 0.5, forward or backwards; video-only; readback+render-verified on 19),freeze? (true = hold source frame srcIn for the whole cut; video-only; render-proven frozen on 19 via freezedetect),audioOnly?+track? (explicit AUDIO placement on audio track 1-8; presence suppresses the A1 mirror; render-verified on 19)}]} | [same, ...] (multi-source needs media_pool.capture_media_template run once per file), transitions?: [{track, atFrame, durationFrames?, trackType? ('video' dissolve | 'audio' cross-fade, both render-verified on 19)}], markers?: [{frame (timeline-absolute), color? (16 names), name?, note?, duration?, customData?}] (readback-verified on 19), compounds?: [{name, startFrame (parent, absolute), durationFrames, track?, cuts:[{mediaFilePath, startFrame (INNER, 0-based), durationFrames, srcIn?}], compounds?: [same, nested — frames inner-relative]}] (multiple PARALLEL compounds compose AND compounds NEST recursively — depth-2 AND depth-3 playback render-verified on 19 (the old depth-2 black was a missing SequenceSetup key, fixed); inner cuts need captured templates w/ native clips), subtitles?: [{startFrame (timeline-absolute), durationFrames, text}] + subtitlesSrt? (raw SRT, cues anchor at the origin; readback-verified on 19; angle-bracket runs read as SRT markup), elements?: [{type:'title'|'generator', track, startFrame, durationFrames?, text?, generatorName? ('Solid Color'|'SMPTE Color Bar'|'Grey Scale' render-verified on 19), ...}] }. startFrame is timeline-absolute (origin 86400)."),
   outputPath: z.string().describe('Absolute path where the importable .drt will be written'),
   targetAppVersion: z
     .union([z.string(), z.number()])

package/resolve-advanced/vendor/drp-format/assemble-timeline.js

@@ -248,8 +248,8 @@ async function assembleTimeline(spec = {}) {
     // Recursive: compounds nest (comp.compounds, frames inner-relative like
     // cuts). Depth-2 playback is RENDER-VERIFIED on 19.1.3.7 (E56, white 234
     // through two levels) — the historical depth-2 black was one missing key,
-    // SequenceSetup, in the donor template's embedded-sequence blob. Depths
-    // beyond 2 compose structurally but are not render-verified.
+    // SequenceSetup, in the donor template's embedded-sequence blob. Depth-3 is
+    // render-verified too (E58, white 234 through three levels).
     const placeTree = async (comp, parentUuid, label) => {
       if (!comp || typeof comp !== 'object') throw new TypeError(`assembleTimeline: ${label} must be an object`);
       const res = await placeCompound(buffer, {

package/src/granular/common.py

@@ -87,7 +87,7 @@ if not logging.getLogger().handlers:
         handlers=[logging.StreamHandler()],
     )
 
-VERSION = "2.134.0"
+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.0"
+VERSION = "2.135.0"
 
 import base64
 import os