davinci-resolve-mcp 4.8.28 → 4.8.30

package/CHANGELOG.md

@@ -2,6 +2,67 @@
 
 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 v4.8.30 — conform_lint reports every reuse of one source
+
+### Fixed
+
+- **`editorial(action="conform_lint")` reported only the first reuse when
+  several pulls came from inside one long pull.** `check_duplicate_usage`
+  sorted each source's pulls by in-point and compared each one only with the
+  pull just before it. A long take followed by short callbacks lifted from
+  inside it therefore compared the second callback against the first, which
+  it does not overlap, and the reuse of the long take went unreported from
+  then on. An interview used at 0–5000 with callbacks at 100–200 and 300–400
+  reported one reuse instead of two. Each pull is now compared with the
+  earlier pull that reaches furthest, so every pull that overlaps any earlier
+  pull is flagged. Where out-points already increase that is the previous
+  pull, so ordinary material is reported exactly as before; adjacent pulls
+  are still two shots, not a duplicate. Contributed by @Dev-next-gen (#276).
+
+### Tests
+
+- `tests/test_conform_lint.py` gains `DuplicateUsageTests`, the check's first
+  tests: adjacent pulls stay unreported, and every callback inside one long
+  pull is reported. The second fails on v4.8.29 with the
+  `INTERVIEW`/`CALLBACK 2` pair missing.
+
+### Validation
+
+- `lint_timeline` is a pure function over a timeline snapshot; no Resolve
+  scripting call changed, so no live run was required.
+
+## What's New in v4.8.29 — optional granular arguments accept an explicit null
+
+### Fixed
+
+- **Sixteen optional arguments on nine granular tools rejected `null`.**
+  They were annotated `str` / `float` / `bool` / `int` with a `None`
+  default, so the generated schema advertised `"default": null` on a field
+  whose type excluded null. Clients that fill every optional argument with
+  its advertised default sent `null`, and the call failed argument
+  validation before the tool ran. Affected: `create_project`
+  (`media_location_path`), `set_color_space_tool` (`gamma`),
+  `import_layout_preset_tool` and `import_user_preferences_preset`
+  (`preset_name`), `set_timeline_item_composite` (`composite_mode`,
+  `opacity`), `set_timeline_item_retime` (`speed`, `process`),
+  `set_timeline_item_stabilization` (`enabled`, `method`, `strength`),
+  `set_timeline_item_audio` (`volume`, `pan`, `eq_enabled`) and
+  `modify_keyframe` (`new_value`, `new_frame`). They are now `Optional[T]`;
+  callers that omit them see no change. Only the granular server (`--full`)
+  was affected. Contributed by @vishalhabib99 (#274).
+
+### Tests
+
+- `tests/test_tool_schema_null_defaults.py` (new) lists every tool on the
+  compound and granular servers offline and fails on any argument whose
+  default is null but whose schema rejects null. On v4.8.28 it names all
+  sixteen.
+
+### Validation
+
+- Schema-only change; no Resolve scripting call changed, so no live run was
+  required.
+
 ## What's New in v4.8.28 — ffmpeg can no longer hang on the protocol stream; a cancelled call no longer kills the server
 
 ### Fixed

package/README.md

@@ -2,7 +2,7 @@
 
 English | [简体中文](README.zh-CN.md)
 
-[![Version](https://img.shields.io/badge/version-4.8.28-blue.svg)](https://github.com/samuelgursky/davinci-resolve-mcp/releases)
+[![Version](https://img.shields.io/badge/version-4.8.30-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-37%20(389%20full)-blue.svg)](#server-modes)

package/README.zh-CN.md

@@ -2,7 +2,7 @@
 
 [English](README.md) | 简体中文
 
-[![Version](https://img.shields.io/badge/version-4.8.28-blue.svg)](https://github.com/samuelgursky/davinci-resolve-mcp/releases)
+[![Version](https://img.shields.io/badge/version-4.8.30-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-37%20(389%20full)-blue.svg)](#服务器模式)
@@ -12,7 +12,7 @@
 [![Python](https://img.shields.io/badge/python-3.10+-green.svg)](https://www.python.org/downloads/)
 [![License](https://img.shields.io/badge/license-MIT-blue.svg)](https://opensource.org/licenses/MIT)
 
-> 本翻译对应 v4.8.28 版 README。如与英文原版有出入，以 [英文原版](README.md) 为准。
+> 本翻译对应 v4.8.30 版 README。如与英文原版有出入，以 [英文原版](README.md) 为准。
 
 一个 Model Context Protocol (MCP) 服务器，让 AI 助手通过官方脚本 API 控制 DaVinci Resolve Studio（达芬奇）。它提供完整的 API 覆盖，外加带护栏的工作流助手，涵盖剪辑、媒体池整理、渲染设置、审阅标记、调色、Fusion、Fairlight、项目生命周期任务、扩展开发，以及不碰源媒体的媒体分析。
 

package/install.py

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

package/src/granular/common.py

@@ -91,7 +91,7 @@ if not logging.getLogger().handlers:
         handlers=[logging.StreamHandler()],
     )
 
-VERSION = "4.8.28"
+VERSION = "4.8.30"
 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/granular/project.py

@@ -140,7 +140,7 @@ def open_project(name: str) -> str:
 
 
 @mcp.tool()
-def create_project(name: str, media_location_path: str = None) -> str:
+def create_project(name: str, media_location_path: Optional[str] = None) -> str:
     """Create a new project with the given name.
     
     Args:
@@ -728,7 +728,7 @@ def set_color_science_mode_tool(mode: str) -> str:
 
 @mcp.tool()
 @granular_destructive_op()
-def set_color_space_tool(color_space: str, gamma: str = None) -> str:
+def set_color_space_tool(color_space: str, gamma: Optional[str] = None) -> str:
     """Set timeline color space and gamma.
     
     Args:

package/src/granular/resolve_control.py

@@ -262,7 +262,7 @@ def export_layout_preset_tool(preset_name: str, export_path: str) -> Dict[str, A
 
 
 @mcp.tool()
-def import_layout_preset_tool(import_path: str, preset_name: str = None) -> Dict[str, Any]:
+def import_layout_preset_tool(import_path: str, preset_name: Optional[str] = None) -> Dict[str, Any]:
     """Import a layout preset from a file.
 
     Calls Resolve.ImportLayoutPreset() to import a preset from disk.
@@ -701,7 +701,7 @@ def delete_user_preferences_preset(preset_name: str) -> Dict[str, Any]:
 
 
 @mcp.tool()
-def import_user_preferences_preset(import_path: str, preset_name: str = None) -> Dict[str, Any]:
+def import_user_preferences_preset(import_path: str, preset_name: Optional[str] = None) -> Dict[str, Any]:
     """Import a user-preferences preset from a file (Resolve 21.0.4+).
 
     The imported preset is NOT auto-loaded; it takes its name from the file

package/src/granular/timeline_item.py

@@ -358,8 +358,8 @@ def set_timeline_item_crop(timeline_item_id: str,
 @mcp.tool()
 @granular_destructive_op()
 def set_timeline_item_composite(timeline_item_id: str, 
-                               composite_mode: str = None, 
-                               opacity: float = None) -> str:
+                               composite_mode: Optional[str] = None, 
+                               opacity: Optional[float] = None) -> str:
     """Set composite properties for a timeline item.
     
     Args:
@@ -446,8 +446,8 @@ def set_timeline_item_composite(timeline_item_id: str,
 @mcp.tool()
 @granular_destructive_op()
 def set_timeline_item_retime(timeline_item_id: str, 
-                            speed: float = None, 
-                            process: str = None) -> str:
+                            speed: Optional[float] = None, 
+                            process: Optional[str] = None) -> str:
     """Set retiming properties for a timeline item.
     
     Args:
@@ -525,9 +525,9 @@ def set_timeline_item_retime(timeline_item_id: str,
 @mcp.tool()
 @granular_destructive_op()
 def set_timeline_item_stabilization(timeline_item_id: str, 
-                                   enabled: bool = None, 
-                                   method: str = None,
-                                   strength: float = None) -> str:
+                                   enabled: Optional[bool] = None, 
+                                   method: Optional[str] = None,
+                                   strength: Optional[float] = None) -> str:
     """Set stabilization properties for a timeline item.
     
     Args:
@@ -617,9 +617,9 @@ def set_timeline_item_stabilization(timeline_item_id: str,
 @mcp.tool()
 @granular_destructive_op()
 def set_timeline_item_audio(timeline_item_id: str, 
-                           volume: float = None, 
-                           pan: float = None,
-                           eq_enabled: bool = None) -> str:
+                           volume: Optional[float] = None, 
+                           pan: Optional[float] = None,
+                           eq_enabled: Optional[bool] = None) -> str:
     """Set audio properties for a timeline item.
     
     Args:
@@ -947,7 +947,7 @@ def add_keyframe(timeline_item_id: str, property_name: str, frame: int, value: f
 
 
 @mcp.tool()
-def modify_keyframe(timeline_item_id: str, property_name: str, frame: int, new_value: float = None, new_frame: int = None) -> str:
+def modify_keyframe(timeline_item_id: str, property_name: str, frame: int, new_value: Optional[float] = None, new_frame: Optional[int] = None) -> str:
     """Modify an existing keyframe by changing its value or frame position.
     
     Args:

package/src/server.py

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

package/src/utils/conform_lint.py

@@ -337,7 +337,17 @@ def check_duplicate_usage(items: Sequence[Mapping[str, Any]]) -> List[Dict[str,
                 length = int(entry["timeline_end_frame"]) - int(entry["timeline_start_frame"])
             spans.append((start, start + max(0, length), _name(entry)))
         spans.sort()
-        for (a_start, a_end, a_name), (b_start, b_end, b_name) in zip(spans, spans[1:]):
+        # Each pull is compared against the furthest-reaching EARLIER pull, not
+        # against its immediate predecessor. Sorted by source in-point, one long
+        # pull is followed by every short callback lifted from inside it, so
+        # `spans[i - 1]` is the previous CALLBACK — which the next callback need
+        # not overlap — and the reuse of the long pull went unreported from the
+        # second callback onward. Where the out-points already increase the
+        # furthest pull *is* the predecessor, so ordinary material is unchanged.
+        furthest = spans[0]
+        for current in spans[1:]:
+            b_start, b_end, b_name = current
+            a_start, a_end, a_name = furthest
             if b_start < a_end:
                 dupes.append({
                     "source": str(ref),
@@ -345,6 +355,8 @@ def check_duplicate_usage(items: Sequence[Mapping[str, Any]]) -> List[Dict[str,
                     "second": b_name,
                     "overlap_frames": min(a_end, b_end) - b_start,
                 })
+            if b_end > a_end:
+                furthest = current
     if not dupes:
         return []
     return [_finding(