davinci-resolve-mcp 4.8.25 → 4.8.27

package/CHANGELOG.md

@@ -2,6 +2,52 @@
 
 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.27 — Linux exports no longer redirected away from /tmp-named folders
+
+### Fixed
+
+- **On Linux, an output folder whose name merely began with `/tmp` or
+  `/var/tmp` was silently replaced.** `_resolve_safe_dir` tested
+  `path.startswith("/tmp")`, which is also true of `/tmpdata`, `/tmpfiles`,
+  `/tmp-scratch` and `/var/tmpdata`. Each was treated as a temp directory and
+  swapped for `~/Documents/resolve-stills`, and
+  `gallery_stills(grab_and_export)` then reported that folder as if it were the
+  one asked for. The same helper picks the output folder for `encrypt_dctl` and
+  the granular `save_project` export fallback. The Linux branch now compares by
+  path segment, as the macOS branch already did; `/tmp`, `/tmp/…`, `/var/tmp`
+  and `/var/tmp/…` still redirect. Both copies of the helper (`src/server.py`
+  and `src/granular/common.py`) are fixed. Thanks to @Dev-next-gen (#271).
+
+### Tests
+
+- `tests/test_granular_safe_dir.py` gains a Linux class covering both copies:
+  real temp paths still redirect, sibling names are left alone.
+
+### Validation
+
+- Path classification only, decided before any Resolve call; no Resolve
+  behavior changed. Live test not required.
+
+## What's New in v4.8.26 — two dead layout-preset helpers removed
+
+### Removed
+
+- `src/utils/layout_presets.py` no longer carries `save_layout_preset` and
+  `load_layout_preset`. Both went through `Resolve.GetUIManager()` and then
+  `SaveUILayout` / `LoadUILayout`, none of which exist on any build measured
+  (Studio 19.1.3.7; see the `api_truth` entry added in v4.8.25). Nothing called
+  them: the granular `save_layout_preset_tool` and `load_layout_preset_tool`
+  use `Resolve.SaveLayoutPreset` / `LoadLayoutPreset` directly, and still do.
+  No tool, action or count changes. The module docstring now says what the
+  file actually does (preset files on disk) and where live save/load happens.
+- `scripts/audit_api_parity.py` drops `LoadUILayout` and `SaveUILayout` from
+  its allowlist, since no source calls them any more.
+
+### Validation
+
+- No Resolve behavior changed; the removed functions had no callers. Live test
+  not required.
+
 ## What's New in v4.8.25 — open_settings and open_app_preferences say what Resolve cannot do
 
 ### Fixed

package/README.md

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

package/scripts/audit_api_parity.py

@@ -148,10 +148,9 @@ ALLOWLIST_UNDOCUMENTED: Set[str] = {
     # 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",
+    # (Studio 19.1.3.7; api_truth 'Resolve.GetUIManager ...'). Called only
+    # behind has_method in src/utils/app_control.py.
+    "GetUIManager",
     # 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

@@ -45,8 +45,6 @@ from src.utils.layout_presets import (
     export_layout_preset,
     import_layout_preset,
     list_layout_presets,
-    load_layout_preset,
-    save_layout_preset,
 )
 from src.utils.object_inspection import inspect_object, print_object_help
 from src.utils.platform import get_platform, get_resolve_paths
@@ -93,7 +91,7 @@ if not logging.getLogger().handlers:
         handlers=[logging.StreamHandler()],
     )
 
-VERSION = "4.8.25"
+VERSION = "4.8.27"
 logger = logging.getLogger("davinci-resolve-mcp")
 logger.info(f"Starting DaVinci Resolve MCP Server v{VERSION}")
 logger.info(f"Detected platform: {get_platform()}")
@@ -379,7 +377,10 @@ def _resolve_safe_dir(path):
         # silently into both, same as /var/folders (matches src/server.py).
         _is_sandbox = path.startswith(("/var/", "/private/var/", "/tmp/", "/private/tmp/")) or path in ("/tmp", "/private/tmp")
     elif platform.system() == "Linux":
-        _is_sandbox = path.startswith("/tmp") or path.startswith("/var/tmp")
+        # By segment, not by character prefix (matches src/server.py): a sibling
+        # of /tmp whose name merely begins with it — /tmpfiles, /tmp-scratch,
+        # /var/tmpdata — cleared `startswith("/tmp")` and was redirected.
+        _is_sandbox = path.startswith(("/tmp/", "/var/tmp/")) or path in ("/tmp", "/var/tmp")
     elif platform.system() == "Windows":
         try:
             _is_sandbox = os.path.commonpath([os.path.abspath(path), os.path.abspath(system_temp)]) == os.path.abspath(system_temp)

package/src/server.py

@@ -11,7 +11,7 @@ Usage:
     python src/server.py --full       # Start the 377-tool granular server instead
 """
 
-VERSION = "4.8.25"
+VERSION = "4.8.27"
 
 import base64
 import os
@@ -1421,7 +1421,13 @@ def _resolve_safe_dir(path):
         # silently fails into both (live-verified 2026-07-03), same as /var/folders.
         _is_sandbox = path.startswith(("/var/", "/private/var/", "/tmp/", "/private/tmp/")) or path in ("/tmp", "/private/tmp")
     elif platform.system() == "Linux":
-        _is_sandbox = path.startswith("/tmp") or path.startswith("/var/tmp")
+        # By segment, not by character prefix: `startswith("/tmp")` also matched a
+        # sibling of /tmp whose name merely begins with it — /tmpfiles,
+        # /tmp-scratch, /var/tmpdata — so an export the caller aimed at one of
+        # those was redirected away from it. The Darwin branch already compares
+        # this way, and `/tmpfiles/out` is one of the paths
+        # tests/test_granular_safe_dir.py asserts is left alone.
+        _is_sandbox = path.startswith(("/tmp/", "/var/tmp/")) or path in ("/tmp", "/var/tmp")
     elif platform.system() == "Windows":
         # Check if path is under the system temp directory (e.g. AppData\Local\Temp)
         try:

package/src/utils/layout_presets.py

@@ -2,11 +2,17 @@
 """
 DaVinci Resolve MCP Server - Layout Presets Utilities
 
-This module provides functions for working with DaVinci Resolve UI layout presets:
-- Saving layout presets
-- Loading layout presets
+This module works with DaVinci Resolve UI layout preset FILES on disk:
+- Listing the presets in Resolve's preset folder
 - Exporting/importing preset files
-- Managing layout configurations
+- Deleting a preset file
+
+Saving and loading the live layout are not done here. They go through
+Resolve.SaveLayoutPreset / LoadLayoutPreset, which the granular tools in
+src/granular/resolve_control.py call directly. Two earlier helpers here tried
+Resolve.GetUIManager().SaveUILayout / LoadUILayout instead; neither method
+exists on any build measured (api_truth 'Resolve.GetUIManager ...'), nothing
+called them, and they were removed in v4.8.26.
 """
 
 import os
@@ -137,71 +143,6 @@ def list_layout_presets(layout_type: str = "ui") -> List[Dict[str, Any]]:
     
     return presets
 
-def save_layout_preset(resolve_obj, preset_name: str, layout_type: str = "ui") -> bool:
-    """
-    Save the current layout as a preset.
-    
-    Args:
-        resolve_obj: DaVinci Resolve API object
-        preset_name: Name for the saved preset
-        layout_type: Type of layout to save ('ui', 'window', 'workspace')
-        
-    Returns:
-        True if successful, False otherwise
-    """
-    try:
-        # Ensure preset name has no spaces or special characters
-        safe_name = preset_name.replace(" ", "_").replace("/", "_").replace("\\", "_")
-        
-        # Different layout types have different save methods
-        if layout_type.lower() == "ui":
-            # For UI layouts, use the UI Manager
-            ui_manager = resolve_obj.GetUIManager()
-            if not ui_manager:
-                logger.error("Failed to get UI Manager")
-                return False
-            
-            # Save the current UI layout
-            return ui_manager.SaveUILayout(safe_name)
-        else:
-            # Other layout types would be handled here
-            logger.error(f"Unsupported layout type: {layout_type}")
-            return False
-    except Exception as e:
-        logger.error(f"Error saving layout preset: {str(e)}")
-        return False
-
-def load_layout_preset(resolve_obj, preset_name: str, layout_type: str = "ui") -> bool:
-    """
-    Load a layout preset.
-    
-    Args:
-        resolve_obj: DaVinci Resolve API object
-        preset_name: Name of the preset to load
-        layout_type: Type of layout to load ('ui', 'window', 'workspace')
-        
-    Returns:
-        True if successful, False otherwise
-    """
-    try:
-        # Different layout types have different load methods
-        if layout_type.lower() == "ui":
-            # For UI layouts, use the UI Manager
-            ui_manager = resolve_obj.GetUIManager()
-            if not ui_manager:
-                logger.error("Failed to get UI Manager")
-                return False
-            
-            # Load the specified UI layout
-            return ui_manager.LoadUILayout(preset_name)
-        else:
-            # Other layout types would be handled here
-            logger.error(f"Unsupported layout type: {layout_type}")
-            return False
-    except Exception as e:
-        logger.error(f"Error loading layout preset: {str(e)}")
-        return False
-
 def export_layout_preset(preset_name: str, export_path: str, layout_type: str = "ui") -> bool:
     """
     Export a layout preset to a file.