davinci-resolve-mcp 2.125.0 → 2.127.0

package/install.py

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

package/resolve-advanced/server/aaf.mjs

@@ -18,7 +18,13 @@ const PROBE = fileURLToPath(new URL('./aaf_probe.py', import.meta.url));
 
 /** python interpreter — overridable for environments/tests (must have `aaf2` importable). */
 function pythonCmd() {
-  return process.env.AAF_PROBE_PYTHON || process.env.PYTHON || 'python3';
+  if (process.env.AAF_PROBE_PYTHON) return process.env.AAF_PROBE_PYTHON;
+  if (process.env.PYTHON) return process.env.PYTHON;
+  // The repo venv carries pyaaf2; a bare python3 usually does not. Prefer it
+  // when present so the tool path works without env plumbing.
+  const venvPy = path.resolve(path.dirname(fileURLToPath(import.meta.url)), '..', '..', 'venv', 'bin', 'python');
+  if (existsSync(venvPy)) return venvPy;
+  return 'python3';
 }
 
 const REMEDIATION =
@@ -127,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

@@ -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,17 +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 (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' };
-        content = p.path;
+        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.125.0"
+VERSION = "2.127.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.125.0"
+VERSION = "2.127.0"
 
 import base64
 import os