davinci-resolve-mcp 2.126.0 → 2.127.1

package/docs/reference/api-limitations.md

@@ -557,7 +557,7 @@ values, or automation-hostile modal prompts.
 ### MediaTimemapBA keyframes are generation-split; 19.x silently ignores the R21 protobuf form
 
 - **Object:** `Sm2TimeMap (per-clip retime blob)`
-- **Behavior:** Resolve 21 encodes a retimed clip's KeyframesBA as protobuf points; Resolve 19.1.3 encodes it as a keyed-dict of keyed-dict keyframes ({interp, YOut, YIn, Y, XOut, XIn, X}). On import, 19 SILENTLY IGNORES the protobuf form — the clip reads back and plays at 100% with no warning (measured: identical timelines, one per form; protobuf → source 0..96 over 96 frames, keyed → source 0..48 over 96 frames and a live 50% render). The map spans the WHOLE source stretched by 1/speed; the clip's <In>/<Duration> window into it in RECORD frames (srcIn converts by /speed). REVERSE is the same envelope with the Y endpoints swapped - kf0=(0,YMax), kf1=(XMax,0) - and In then measures from the source END: (frames - srcIn - dur*speed)/speed (measured: a reversed srcIn-24 dur-48 cut reads back source 71->23). A FLAT map (both keyframes at the same Y - a freeze) is the one shape where readback and render DIVERGE: the item reads back frozen (source 96..96) but renders MOVING (48/48 unique frames measured). Do not author freezes as flat timemaps.
+- **Behavior:** Resolve 21 encodes a retimed clip's KeyframesBA as protobuf points; Resolve 19.1.3 encodes it as a keyed-dict of keyed-dict keyframes ({interp, YOut, YIn, Y, XOut, XIn, X}). On import, 19 SILENTLY IGNORES the protobuf form — the clip reads back and plays at 100% with no warning (measured: identical timelines, one per form; protobuf → source 0..96 over 96 frames, keyed → source 0..48 over 96 frames and a live 50% render). The map spans the WHOLE source stretched by 1/speed; the clip's <In>/<Duration> window into it in RECORD frames (srcIn converts by /speed). REVERSE is the same envelope with the Y endpoints swapped - kf0=(0,YMax), kf1=(XMax,0) - and In then measures from the source END: (frames - srcIn - dur*speed)/speed (measured: a reversed srcIn-24 dur-48 cut reads back source 71->23). A FLAT map (both keyframes at the same Y - a freeze) is the one shape where readback and render DIVERGE: the item reads back frozen (source 96..96) but renders MOVING (48/48 unique frames measured). Do not author freezes as flat timemaps. AUDIO clips ignore the timemap entirely: a 50% keyed map on an imported audio clip reads back retimed (source 0..48 over 96 record frames) but RENDERS at 100% - unchanged pitch and spectrum (highpass/lowpass split identical to the 1x reference).
 - **Workaround / current handling:** Author retimes for pre-21 hosts with the keyed form (drt.assemble cuts[].speed does this; encoder byte-exact against a live 19.1.3.7 harvest). Treat any cross-generation timemap as unverified until a readback shows the retimed source range.
 - **Tags:** retime, import, silent-failure, drt
 

package/install.py

@@ -37,7 +37,7 @@ from src.utils.update_check import (
 
 # ─── Version ──────────────────────────────────────────────────────────────────
 
-VERSION = "2.126.0"
+VERSION = "2.127.1"
 # 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.126.0",
+  "version": "2.127.1",
   "description": "NPM bootstrapper for the DaVinci Resolve MCP Server.",
   "license": "MIT",
   "author": "Samuel Gursky <samgursky@gmail.com>",

package/resolve-advanced/server/aaf.mjs

@@ -133,7 +133,9 @@ export async function parseAafDocument(contentOrPath) {
   const { sequences } = await runProbe(aafPath);
   const events = [];
   for (const seq of sequences || []) for (const ev of seq.events || []) events.push(ev);
-  return { events, sequences: (sequences || []).map(sequenceSummary) };
+  // Summaries PLUS each sequence's own events — the assemble picker needs
+  // per-sequence events without a second probe run.
+  return { events, sequences: (sequences || []).map((sq) => ({ ...sequenceSummary(sq), events: sq.events || [] })) };
 }
 
 /**

package/resolve-advanced/server/author-interchange.mjs

@@ -329,7 +329,7 @@ export function eventsToAssembleSpec(events, opts = {}) {
     const durationFrames = recOut - recIn;
     if (durationFrames <= 0) continue;
     if ((e.speed ?? 100) !== 100 || e.reverse) {
-      audioRetimesSkipped.push({ index: e.index, source: e.source, speed: e.speed, reverse: !!e.reverse, reason: 'audio retime not supported — played at 100%' });
+      audioRetimesSkipped.push({ index: e.index, source: e.source, speed: e.speed, reverse: !!e.reverse, reason: 'audio retime not authorable — the audio engine ignores the clip timemap (measured: reads back retimed, renders 100%); played at 100%' });
     }
     const track = audioTrackNum(e.track);
     const cut = { startFrame: recIn, durationFrames, srcIn: toTl(e.srcIn ?? 0, e.fps), audioOnly: true, track };
@@ -382,11 +382,16 @@ export function eventsToAssembleSpec(events, opts = {}) {
     }
   }
 
-  const media = [...perSource.entries()].map(([reel, cuts]) => ({
-    mediaFilePath: sourceMap[reel].mediaFilePath,
-    spec: sourceMap[reel].spec,
-    cuts,
-  }));
+  // Reel aliasing: multiple reels legitimately map to ONE file (Avid mob
+  // names vs tape names, re-linked dailies). Group by mediaFilePath so the
+  // assembly sees one source per FILE, not per reel.
+  const byFile = new Map();
+  for (const [reel, cuts] of perSource.entries()) {
+    const fp = sourceMap[reel].mediaFilePath;
+    if (!byFile.has(fp)) byFile.set(fp, { mediaFilePath: fp, spec: sourceMap[reel].spec, cuts: [] });
+    byFile.get(fp).cuts.push(...cuts);
+  }
+  const media = [...byFile.values()];
 
   // Author cross-dissolves where the geometry allows it (render-verified on
   // 19.1.3.7: an offline Sm2TiTransition over transplanted cross-source media

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

@@ -41,6 +41,8 @@ const assembleFromInterchangeSchema = z.object({
   outputPath: z.string().describe('Where the importable .drt is written'),
   targetAppVersion: z.union([z.string(), z.number()]).optional()
     .describe("Host Resolve version, e.g. '19.1' for pre-21"),
+  sequenceName: z.string().optional().describe('Multi-sequence AAF/prproj: assemble THIS sequence (see editorial.list_sequences)'),
+  sequenceIndex: z.number().int().optional().describe('Multi-sequence AAF/prproj: assemble the sequence at this 0-based index'),
   preserveStartTimecode: z.boolean().optional()
     .describe('Keep the interchange\'s ABSOLUTE record start: the assembled timeline starts at the turnover\'s real first record frame instead of 01:00:00:00 (start-TC patch render-verified on 19). AAF conforms should pass true.'),
 });
@@ -212,22 +214,42 @@ export const drtTool = {
       const { eventsToAssembleSpec } = await import('../author-interchange.mjs');
       let content = p.content;
       let events;
+      const pickSequence = (sequences) => {
+        // Multi-sequence containers (AAF/prproj): pick ONE sequence by name
+        // or index instead of flattening everything (overlapping record
+        // ranges across sequences would refuse in the overlap check).
+        if (p.sequenceName !== undefined) {
+          const hit = sequences.find((sq) => sq.name === p.sequenceName);
+          if (!hit) throw new Error(`sequenceName ${JSON.stringify(p.sequenceName)} not found — available: ${sequences.map((sq) => sq.name).join(', ')}`);
+          return hit.events;
+        }
+        if (p.sequenceIndex !== undefined) {
+          if (p.sequenceIndex < 0 || p.sequenceIndex >= sequences.length) throw new Error(`sequenceIndex ${p.sequenceIndex} out of range (${sequences.length} sequences)`);
+          return sequences[p.sequenceIndex].events;
+        }
+        if (sequences.length > 1) {
+          const nonEmpty = sequences.filter((sq) => (sq.events || []).length);
+          if (nonEmpty.length > 1) throw new Error(
+            `the file holds ${nonEmpty.length} sequences with events — pass sequenceName or sequenceIndex ` +
+            `(available: ${sequences.map((sq, i) => `${i}:${sq.name}`).join(', ')})`);
+          if (nonEmpty.length === 1) return nonEmpty[0].events;
+        }
+        return sequences.flatMap((sq) => sq.events || []);
+      };
       if (p.format === 'aaf') {
-        // BUG FIX: this branch used to fall through to the sync
-        // parseInterchange, which THROWS for aaf ("parse it via the async
-        // AAF path") — the tool-layer AAF route never worked until now.
+        // BUG FIX (v2.126.0): this branch used to fall through to the sync
+        // parseInterchange, which THROWS for aaf — the tool-layer AAF route
+        // never worked before.
         if (!p.path) return { error: 'aaf input requires path' };
-        const { parseAAF } = await import('../aaf.mjs');
-        const parsed = await parseAAF(p.path);
-        events = Array.isArray(parsed) ? parsed : parsed.events;
+        const { parseAafDocument } = await import('../aaf.mjs');
+        const parsed = await parseAafDocument(p.path);
+        events = pickSequence(parsed.sequences);
       } else if (p.format === 'prproj') {
-        // Premiere: offline gunzip+graph read (no Premiere, no Resolve import
-        // path). parsePrproj flattens EVERY sequence's events; a multi-sequence
-        // .prproj with overlapping record ranges refuses naturally in the
-        // overlap check — pre-split via editorial.list_sequences if needed.
+        // Premiere: offline gunzip+graph read (no Premiere, no Resolve
+        // import path).
         if (!p.path) return { error: 'prproj input requires path' };
-        const { parsePrproj } = await import('../prproj.mjs');
-        events = parsePrproj(p.path);
+        const { parsePrprojDoc } = await import('../prproj.mjs');
+        events = pickSequence(parsePrprojDoc(p.path).sequences);
       } else if (!content) {
         if (!p.path) return { error: 'provide content or path' };
         content = await fs.readFile(p.path, 'utf8');

package/src/granular/common.py

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

package/src/utils/api_truth.py

@@ -2033,7 +2033,12 @@ API_TRUTH: List[Dict[str, Any]] = [
                    "the one shape where readback and render DIVERGE: the "
                    "item reads back frozen (source 96..96) but renders "
                    "MOVING (48/48 unique frames measured). Do not author "
-                   "freezes as flat timemaps.",
+                   "freezes as flat timemaps. AUDIO clips ignore the "
+                   "timemap entirely: a 50% keyed map on an imported audio "
+                   "clip reads back retimed (source 0..48 over 96 record "
+                   "frames) but RENDERS at 100% - unchanged pitch and "
+                   "spectrum (highpass/lowpass split identical to the 1x "
+                   "reference).",
         "recommended": "Author retimes for pre-21 hosts with the keyed "
                        "form (drt.assemble cuts[].speed does this; encoder "
                        "byte-exact against a live 19.1.3.7 harvest). Treat "