davinci-resolve-mcp 4.8.24 → 4.8.25

package/CHANGELOG.md

@@ -2,6 +2,47 @@
 
 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.25 — open_settings and open_app_preferences say what Resolve cannot do
+
+### Fixed
+
+- **The granular `open_settings` and `open_app_preferences` tools could never
+  work, and reported that as an ordinary failure.** Both went through
+  `Resolve.GetUIManager()`, which does not exist. Measured on Studio 19.1.3.7:
+  `dir(resolve)` lists 23 methods and `GetUIManager` is not one of them;
+  `Fusion().UIManager` is real but has neither `OpenProjectSettings` nor
+  `OpenPreferences`; and none of the three names appears in the 21.1 typed API.
+  The call raised `'NoneType' object is not callable`, a broad `except`
+  swallowed it, an ERROR was logged, and the tool answered
+  `Failed to open Project Settings dialog` with no reason.
+  Both tools now answer `Not supported:` and name the call that is missing;
+  `open_settings` also names the tools that read and write project settings.
+  Nothing is logged as an error, because nothing went wrong.
+- The route is now probed with `has_method` rather than `hasattr`, which is
+  true for every name on a Resolve object. If a future build does provide these
+  calls they are used, and their result is reported: the old code discarded the
+  return and answered success regardless, so a refusal would have read as a
+  dialog that opened.
+
+### Documentation
+
+- `api_truth` records the absence, with what `Fusion().UIManager` does expose.
+  Whether `UIManager.DoAction` or `QueueAction` can open these dialogs was not
+  tried: both dialogs are modal, and a modal dialog blocks the scripting API
+  until a person closes it.
+- `scripts/audit_api_parity.py` no longer describes `GetUIManager` as a
+  documented API.
+
+### Tests
+
+- `tests/test_app_control_dialogs.py`: a fake that fabricates attributes the way
+  a Resolve object does (every `hasattr` true, a missing `getattr` is `None`).
+  Covers the measured build, a manager without the method, a build that has
+  the call, a refusal, an exception, and both granular tools. Eight of its
+  twelve tests fail against v4.8.24.
+- `tests/test_discarded_resolve_returns.py` now covers every `Open*` call, not
+  only `OpenPage`.
+
 ## What's New in v4.8.24 — a frame capture leaves the render output folder and file name alone
 
 ### Fixed

package/README.md

@@ -2,7 +2,7 @@
 
 English | [简体中文](README.zh-CN.md)
 
-[![Version](https://img.shields.io/badge/version-4.8.24-blue.svg)](https://github.com/samuelgursky/davinci-resolve-mcp/releases)
+[![Version](https://img.shields.io/badge/version-4.8.25-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.24-blue.svg)](https://github.com/samuelgursky/davinci-resolve-mcp/releases)
+[![Version](https://img.shields.io/badge/version-4.8.25-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.24 版 README。如与英文原版有出入，以 [英文原版](README.md) 为准。
+> 本翻译对应 v4.8.25 版 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.24"
+VERSION = "4.8.25"
 # 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.24",
+  "version": "4.8.25",
   "description": "NPM bootstrapper for the DaVinci Resolve MCP Server.",
   "license": "MIT",
   "author": "Samuel Gursky <samgursky@gmail.com>",

package/scripts/audit_api_parity.py

@@ -145,10 +145,13 @@ ALLOWLIST_UNDOCUMENTED: Set[str] = {
     "AddKeyframe", "DeleteKeyframe", "ModifyKeyframe", "RemoveKeyFrame",
     "GetKeyframeAtIndex", "GetKeyframeCount", "GetPropertyAtKeyframeIndex",
     "SetKeyframeInterpolation", "Render", "StartUndo",
-    # UIManager / Resolve app-control API (documented under UIManager, not
-    # the main Resolve scripting README)
-    "GetUIManager", "OpenPreferences", "SetHighPriority",
-    "OpenProjectSettings", "LoadUILayout", "SaveUILayout",
+    # Resolve app-control method, not in the scripting README this audit parses
+    "SetHighPriority",
+    # NOT a documented API: Resolve has no GetUIManager on any build measured
+    # (Studio 19.1.3.7; api_truth 'Resolve.GetUIManager ...'). It is called
+    # only behind has_method in src/utils/app_control.py, and from the unused
+    # helpers in src/utils/layout_presets.py along with the other two.
+    "GetUIManager", "LoadUILayout", "SaveUILayout",
     # Lua-table iteration helper used as a fallback in object_inspection.py
     "GetKeyList",
     # Project metadata accessor used defensively (hasattr-guarded)

package/src/granular/common.py

@@ -93,7 +93,7 @@ if not logging.getLogger().handlers:
         handlers=[logging.StreamHandler()],
     )
 
-VERSION = "4.8.24"
+VERSION = "4.8.25"
 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/resolve_control.py

@@ -371,32 +371,45 @@ def restart_app(wait_seconds: int = 5) -> str:
 
 @mcp.tool()
 def open_settings() -> str:
-    """Open the Project Settings dialog in DaVinci Resolve."""
+    """Open the Project Settings dialog in DaVinci Resolve.
+
+    Not available on any Resolve build measured so far: the scripting API has
+    no call that opens this dialog (Studio 19.1.3.7 measured; absent from the
+    21.1 typed API). The reply says so by name instead of a bare failure. To
+    read or change project settings use get_project_settings,
+    get_project_setting and set_project_setting.
+    """
     resolve = get_resolve()
     if resolve is None:
         return "Error: Not connected to DaVinci Resolve"
-    
+
     result = open_project_settings(resolve)
-    
-    if result:
+    if result["success"]:
         return "Project Settings dialog opened successfully"
-    else:
-        return "Failed to open Project Settings dialog"
+    if not result["supported"]:
+        return (
+            result["message"]
+            + " Use get_project_settings, get_project_setting and set_project_setting instead."
+        )
+    return result["message"]
 
 
 @mcp.tool()
 def open_app_preferences() -> str:
-    """Open the Preferences dialog in DaVinci Resolve."""
+    """Open the Preferences dialog in DaVinci Resolve.
+
+    Not available on any Resolve build measured so far: the scripting API has
+    no call that opens this dialog (Studio 19.1.3.7 measured; absent from the
+    21.1 typed API). The reply says so by name instead of a bare failure.
+    """
     resolve = get_resolve()
     if resolve is None:
         return "Error: Not connected to DaVinci Resolve"
-    
+
     result = open_preferences(resolve)
-    
-    if result:
+    if result["success"]:
         return "Preferences dialog opened successfully"
-    else:
-        return "Failed to open Preferences dialog"
+    return result["message"]
 
 
 @mcp.tool()

package/src/server.py

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

package/src/utils/api_truth.py

@@ -2749,6 +2749,34 @@ API_TRUTH: List[Dict[str, Any]] = [
         "verified_on": "DaVinci Resolve Studio 19.1.3.7",
         "mitigation": ["_render_target_dir", "_playhead_frame_render"],
     },
+    {
+        "symbol": "Resolve.GetUIManager / UIManager.OpenProjectSettings / OpenPreferences (do not exist)",
+        "object": "Resolve",
+        "reality": "No scripting call opens the Project Settings or Preferences "
+                   "dialog. Measured 2026-09-30 on Studio 19.1.3.7, direct "
+                   "connection: dir(resolve) lists 23 methods and GetUIManager "
+                   "is not among them (getattr returns None; hasattr says True, "
+                   "as it does for every name). Fusion().UIManager is a real "
+                   "object with 15 names — AddNotify, Comp, Composition, "
+                   "DoAction, FindWindow, FindWindows, GetData, GetEvent, GetID, "
+                   "GetReg, QueueAction, QueueEvent, RemoveNotify, SetData, "
+                   "TriggerEvent — and none of OpenProjectSettings, "
+                   "OpenPreferences, SaveUILayout or LoadUILayout. None of those "
+                   "names, nor GetUIManager, appears in the 21.1 typed API "
+                   "either. Code written against them calls None and raises "
+                   "\"'NoneType' object is not callable\"; wrapped in a broad "
+                   "except, that reads as an ordinary failure. Whether "
+                   "UIManager.DoAction or QueueAction can open these dialogs was "
+                   "not tried: both dialogs are modal, and a modal dialog blocks "
+                   "the scripting API until a person closes it.",
+        "recommended": "Do not offer to open these dialogs. Read and write "
+                       "project settings through Project.GetSetting/SetSetting. "
+                       "UI layouts go through Resolve.SaveLayoutPreset / "
+                       "LoadLayoutPreset, which do exist. Probe with "
+                       "resolve_probe.has_method, never hasattr.",
+        "tags": ["ui", "unsupported", "dialog"],
+        "verified_on": "DaVinci Resolve Studio 19.1.3.7",
+    },
     {
         "symbol": "ProjectManager.SaveProject",
         "object": "ProjectManager",

package/src/utils/app_control.py

@@ -16,6 +16,8 @@ import platform
 import subprocess
 from typing import Dict, Any, Optional, Union, List
 
+from src.utils.resolve_probe import has_method
+
 # Configure logging
 logger = logging.getLogger("davinci-resolve-mcp.app_control")
 APP_CONTROL_TIMEOUT_SECONDS = 10
@@ -267,65 +269,68 @@ def restart_resolve_app(resolve_obj, wait_seconds: int = 5) -> bool:
         logger.error(f"Error restarting DaVinci Resolve: {str(e)}")
         return False
 
-def open_project_settings(resolve_obj) -> bool:
-    """
-    Open the Project Settings dialog in DaVinci Resolve.
-    
-    Args:
-        resolve_obj: DaVinci Resolve API object
-        
-    Returns:
-        True if successful, False otherwise
-    """
-    try:
-        # Check if UI Manager is available
-        ui_manager = resolve_obj.GetUIManager()
-        if not ui_manager:
-            logger.error("Failed to get UI Manager")
-            return False
-        
-        # Open Project Settings dialog
-        if hasattr(ui_manager, 'OpenProjectSettings') and callable(getattr(ui_manager, 'OpenProjectSettings')):
-            ui_manager.OpenProjectSettings()
-            return True
-        
-        # Alternative method - send keyboard shortcut based on platform
-        current_page = resolve_obj.GetCurrentPage()
-        
-        # Ensure we're on a page that supports project settings
-        if current_page not in ['media', 'cut', 'edit', 'fusion', 'color', 'fairlight', 'deliver']:
-            logger.error(f"Can't open settings from page: {current_page}")
-            return False
-        
-        return False  # Keyboard shortcuts not implemented yet
-    except Exception as e:
-        logger.error(f"Error opening project settings: {str(e)}")
-        return False
+def _open_dialog(resolve_obj, method_name: str, label: str) -> Dict[str, Any]:
+    """Ask Resolve to open a dialog, and say what actually happened.
 
-def open_preferences(resolve_obj) -> bool:
-    """
-    Open the Preferences dialog in DaVinci Resolve.
-    
-    Args:
-        resolve_obj: DaVinci Resolve API object
-        
-    Returns:
-        True if successful, False otherwise
+    Returns {"success", "supported", "message"}.
+
+    The route this module has always used is Resolve.GetUIManager() and then a
+    method on the manager. Neither half exists on any build measured so far.
+    On Studio 19.1.3.7 (2026-09-30): dir(resolve) lists 23 methods and
+    GetUIManager is not one of them; Fusion().UIManager is real but offers
+    neither OpenProjectSettings nor OpenPreferences; and none of the three
+    names appears in the 21.1 typed API. So the old code raised "'NoneType'
+    object is not callable" on its first line, caught it, logged an error and
+    returned a bare False — a tool that could never work, reporting that as an
+    ordinary failure with no reason.
+
+    `hasattr` cannot make this distinction: it is True for every name on a
+    Resolve object (see src/utils/resolve_probe.py). `has_method` can, so the
+    route is probed with it, and a build that does grow these calls is used
+    and its answer reported instead of assumed.
     """
+    if not has_method(resolve_obj, "GetUIManager"):
+        return {
+            "success": False,
+            "supported": False,
+            "message": (
+                f"Not supported: DaVinci Resolve's scripting API has no call that opens the "
+                f"{label} dialog. Resolve.GetUIManager does not exist on this build."
+            ),
+        }
+    ui_manager = resolve_obj.GetUIManager()
+    if not has_method(ui_manager, method_name):
+        return {
+            "success": False,
+            "supported": False,
+            "message": (
+                f"Not supported: DaVinci Resolve's scripting API has no call that opens the "
+                f"{label} dialog. UIManager.{method_name} does not exist on this build."
+            ),
+        }
     try:
-        # Check if UI Manager is available
-        ui_manager = resolve_obj.GetUIManager()
-        if not ui_manager:
-            logger.error("Failed to get UI Manager")
-            return False
-        
-        # Open Preferences dialog
-        if hasattr(ui_manager, 'OpenPreferences') and callable(getattr(ui_manager, 'OpenPreferences')):
-            ui_manager.OpenPreferences()
-            return True
-        
-        # Alternative method - send keyboard shortcut based on platform
-        return False  # Keyboard shortcuts not implemented yet
-    except Exception as e:
-        logger.error(f"Error opening preferences: {str(e)}")
-        return False
+        opened = getattr(ui_manager, method_name)()
+    except Exception as exc:
+        logger.error("UIManager.%s raised: %s", method_name, exc)
+        return {
+            "success": False,
+            "supported": True,
+            "message": f"Failed to open the {label} dialog: UIManager.{method_name} raised {exc}",
+        }
+    if opened is False:
+        return {
+            "success": False,
+            "supported": True,
+            "message": f"Failed to open the {label} dialog: UIManager.{method_name} returned False",
+        }
+    return {"success": True, "supported": True, "message": f"{label} dialog opened"}
+
+
+def open_project_settings(resolve_obj) -> Dict[str, Any]:
+    """Open the Project Settings dialog. See `_open_dialog` for the result shape."""
+    return _open_dialog(resolve_obj, "OpenProjectSettings", "Project Settings")
+
+
+def open_preferences(resolve_obj) -> Dict[str, Any]:
+    """Open the Preferences dialog. See `_open_dialog` for the result shape."""
+    return _open_dialog(resolve_obj, "OpenPreferences", "Preferences")