davinci-resolve-mcp 2.32.0 → 2.32.2

package/CHANGELOG.md

@@ -2,6 +2,42 @@
 
 Release history for the DaVinci Resolve MCP Server. The latest release is summarized in the root README; older entries live here to keep the README focused.
 
+## What's New in v2.32.2
+
+Fixes `fusion_comp(action="get_keyframes")` serialization.
+
+The handler iterated `Input.GetKeyFrames()` as if it returned `{time: value}`,
+but Fusion returns `{1-based index: frame_position}`. The result put the
+keyframe **index** in `time` and the **frame position** in `value` — the actual
+keyframed values were never reported.
+
+The handler now treats the dict values as frame positions and reads each
+keyframed value back via `GetInput(input_name, frame)`.
+
+- **Fixed** `get_keyframes` now returns `[{"time": <frame>, "value": <value>}, ...]`
+  in frame order (live-validated on DaVinci Resolve Studio 21.0.0:
+  `Size` keyed `1.0@f0` / `1.4@f75` → `[{0.0: 1.0}, {75.0: 1.4}]`).
+- Follow-up to the `add_keyframe` fix in v2.32.1; flagged by @sandypoli-boop in #56.
+
+## What's New in v2.32.1
+
+Fixes `fusion_comp(action="add_keyframe")` so it actually **animates** the input.
+
+Previously the handler did `inp[time] = value` on the input directly. On an input
+with no animation spline, that only assigns a **static** value (last write wins) —
+no keyframe is created. Symptoms: `get_keyframes` returned `[]`, `get_input` at
+different times returned the same value, and the clip never animated.
+
+The handler now attaches a `BezierSpline` modifier the first time an input is
+animated, then sets the keyframe. A new optional `modifier` param lets callers
+pass e.g. `"Path"` for Point inputs such as `Center`. Behavior is unchanged for
+inputs that are already animated or otherwise connected.
+
+- **Fixed** `add_keyframe` now creates real, editable keyframes (live-validated on
+  DaVinci Resolve Studio 21.0.0).
+- **Added** optional `modifier` param to `add_keyframe`.
+- Thanks to @sandypoli-boop for the diagnosis and fix ([#56](https://github.com/samuelgursky/davinci-resolve-mcp/pull/56)).
+
 ## What's New in v2.32.0
 
 Adds **governance tiers** for the media-creating Resolve 21 AI ops (Phase 3, the

package/README.md

@@ -1,6 +1,6 @@
 # DaVinci Resolve MCP Server
 
-[![Version](https://img.shields.io/badge/version-2.32.0-blue.svg)](https://github.com/samuelgursky/davinci-resolve-mcp/releases)
+[![Version](https://img.shields.io/badge/version-2.32.2-blue.svg)](https://github.com/samuelgursky/davinci-resolve-mcp/releases)
 [![npm](https://img.shields.io/npm/v/davinci-resolve-mcp.svg?label=npm&color=CB3837)](https://www.npmjs.com/package/davinci-resolve-mcp)
 [![API Coverage](https://img.shields.io/badge/API%20Coverage-100%25-brightgreen.svg)](docs/reference/api-coverage.md)
 [![Tools](https://img.shields.io/badge/MCP%20Tools-32%20(341%20full)-blue.svg)](#server-modes)

package/install.py

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

package/src/granular/common.py

@@ -80,7 +80,7 @@ if not logging.getLogger().handlers:
         handlers=[logging.StreamHandler()],
     )
 
-VERSION = "2.32.0"
+VERSION = "2.32.2"
 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 341-tool granular server instead
 """
 
-VERSION = "2.32.0"
+VERSION = "2.32.2"
 
 import base64
 import os
@@ -18663,7 +18663,17 @@ def fusion_comp(action: str, params: Optional[Dict[str, Any]] = None) -> Dict[st
             inp = tool[p["input_name"]]
             if not inp:
                 return _err(f"Input '{p['input_name']}' not found on tool '{p['tool_name']}'")
-            inp[p["time"]] = p["value"]
+            # Attach a BezierSpline modifier the first time this input is animated.
+            # Without a spline, `inp[time] = value` only sets a STATIC value (the last
+            # write wins) and never creates a keyframe. See the Fusion Scripting Guide.
+            # Optional `modifier` lets callers pass e.g. "Path" for Point inputs.
+            try:
+                _already_animated = inp.GetConnectedOutput() is not None
+            except Exception:
+                _already_animated = False
+            if not _already_animated:
+                tool.AddModifier(p["input_name"], p.get("modifier", "BezierSpline"))
+            tool[p["input_name"]][p["time"]] = p["value"]
             return _ok()
         finally:
             comp.Unlock()
@@ -18675,11 +18685,17 @@ def fusion_comp(action: str, params: Optional[Dict[str, Any]] = None) -> Dict[st
         inp = tool[p["input_name"]]
         if not inp:
             return _err(f"Input '{p['input_name']}' not found on tool '{p['tool_name']}'")
+        # GetKeyFrames() returns {1-based index: frame_position}, NOT
+        # {time: value}. Iterating it directly puts the index in `time` and the
+        # frame position in `value`. Read the actual keyframed value back from
+        # the input at each frame position via GetInput.
         keyframes = []
         kfs = inp.GetKeyFrames()
         if kfs:
-            for t in kfs:
-                keyframes.append({"time": t, "value": _ser(kfs[t])})
+            for idx in sorted(kfs):
+                frame = kfs[idx]
+                value = tool.GetInput(p["input_name"], frame)
+                keyframes.append({"time": frame, "value": _ser(value)})
         return {"keyframes": keyframes}
 
     elif action == "delete_keyframe":