davinci-resolve-mcp 4.8.11 → 4.8.13

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/CHANGELOG.md CHANGED
@@ -2,6 +2,92 @@
2
2
 
3
3
  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.
4
4
 
5
+ ## What's New in v4.8.13 — `include_linked="false"` no longer relinks
6
+
7
+ ### Fixed
8
+
9
+ - **`_normalize_include_linked` handled the boolean `False` but not the string
10
+ spellings.** ([#262](https://github.com/samuelgursky/davinci-resolve-mcp/pull/262), @Dev-next-gen)
11
+ A caller sending `include_linked="false"` (or `"no"`, `"0"`, `"off"`, `"none"`) to a
12
+ timeline move got a set containing the literal string instead of an empty set;
13
+ `bool({"false"})` is `True`, so the `relink` flag defaulted to on — the opposite of
14
+ what was asked. The twin of the `ripple`, `overwrite`, permission-flag,
15
+ `allow_non_mcp_name` and `allow_partial_item_delete` fixes, in the one normaliser
16
+ that parses its own strings. The false spellings now return an empty set; `True`,
17
+ `False`, `"all"`, type lists and real lists are unchanged. Guard test: seven
18
+ subtests in `tests/test_append_clip_infos_result_handling.py`, all failing on the
19
+ previous code.
20
+
21
+ ## What's New in v4.8.12 — the offline suite no longer writes traces, reports or update state into `logs/`
22
+
23
+ Test-harness fix only. No tool, action, or Resolve behaviour changed.
24
+
25
+ ### Fixed
26
+
27
+ - **The offline suite wrote synthetic records into the operator's `logs/`.**
28
+ `logs/execution-traces.jsonl`, `logs/execution-reports/` and
29
+ `logs/update-check.json` are the server's defaults, and `tests/offline_guard.py`
30
+ did not move them the way it already moves the audit log, the operation log and
31
+ the media-analysis preferences. Measured with
32
+ `python -m unittest discover -s tests -t .` from a checkout with an empty `logs/`:
33
+ one run appended 1,865 tool-call records (672 KB) to the trace log, wrote an audit
34
+ report for a synthetic execution, and created `update-check.json`. In a main
35
+ checkout the trace records share a file with the real ones, and that file is
36
+ where `list_recent_executions` sends a reviewer.
37
+ - `offline_guard` now points `RESOLVE_MCP_TRACE_FILE`,
38
+ `RESOLVE_MCP_TRACE_REPORT_DIR` and `DAVINCI_RESOLVE_MCP_UPDATE_STATE` at a temp
39
+ directory for the run, unless the caller already set them, and restores them on
40
+ `uninstall`. It sets environment variables instead of swapping functions: the
41
+ server already reads these variables, `src.server` binds its own
42
+ `update_state_path`, and a child process inherits the environment.
43
+ - `tests/test_execution_trace.py` unset two of these variables with a bare
44
+ `os.environ.pop`, which would have removed the redirect for every later test.
45
+ Both classes now scope the change with `mock.patch.dict`.
46
+ - New tripwire `tests/test_repo_logs_isolation.py`. It fails if any of the three
47
+ variables is unset or points into the repo's `logs/`, if the server resolves a
48
+ path other than the one the guard set, or if a traced tool call and its exported
49
+ report reach the real `logs/`. It looks for a unique execution id, so a live
50
+ server appending to the same log cannot make it pass or fail.
51
+
52
+ ### Validation
53
+
54
+ - Full offline suite from an empty `logs/`, before and after the change: 1,865 trace
55
+ lines, one report and `update-check.json` before; only `.gitkeep` after. 3,741
56
+ tests run (3 new), 84 skipped, 12 errors, the same 12 as before the change:
57
+ `numpy` and `requests` absent from the venv, `jszip` not installed, and one launch
58
+ attempt stopped by the verification tripwire described below.
59
+ - The new tripwire against the previous guard: 4 failures. With the previous
60
+ `test_execution_trace.py` run first in the same process, it fails on the two
61
+ variables that file popped.
62
+ - Both runs used a `sitecustomize` tripwire that stood in for `DaVinciResolveScript`
63
+ and blocked Resolve launches and outbound requests. It recorded 1,174
64
+ `scriptapp("Resolve")` calls and one `open DaVinci Resolve.app` from the granular
65
+ server, before and after this change.
66
+ [#255](https://github.com/samuelgursky/davinci-resolve-mcp/pull/255) closes that
67
+ gap; it is unrelated to this one. It also recorded one request to GitHub's
68
+ `releases/latest` endpoint, made by `test_scripting_lib_discovery` running
69
+ `install.py`'s update check. The state that check writes is now redirected. The
70
+ request itself is refused by
71
+ [#259](https://github.com/samuelgursky/davinci-resolve-mcp/pull/259).
72
+ - That tripwire missed one child process. `install.verify_resolve_connection`
73
+ replaces `PYTHONPATH` with Resolve's `Modules` directory, so its probe child never
74
+ loaded the tripwire. In both runs `test_the_live_probe_agrees_with_the_summary`
75
+ passed instead of skipping. That child therefore loaded the real scripting module
76
+ and called `scriptapp("Resolve")` against the Resolve that was open, one
77
+ read-only probe per run. This change does not touch that path;
78
+ [#260](https://github.com/samuelgursky/davinci-resolve-mcp/pull/260) puts the test
79
+ behind `RESOLVE_VERIFY=1`.
80
+ - Re-run for this release behind a tripwire that also refuses any child whose
81
+ command line imports the scripting module: 3,741 tests, 85 skipped, 14 errors.
82
+ The extra skip is that probe test. The two extra errors are the `test_doctor_paths`
83
+ probe children the stricter tripwire refuses. The error set matches v4.8.4 under
84
+ the same tripwire, and `logs/` held only `.gitkeep` afterwards.
85
+ - Not run under pytest, which is not installed here. `tests/conftest.py` calls the
86
+ same idempotent `install()`/`uninstall()`. That lifecycle was checked directly:
87
+ a second install is a no-op, uninstall restores the environment, and a value the
88
+ caller set is left alone.
89
+ - No Resolve behaviour changed; live test not required.
90
+
5
91
  ## What's New in v4.8.11 — the installer's live probe no longer runs in the offline suite
6
92
 
7
93
  Test-only. No tool, action or runtime code changed.
package/README.md CHANGED
@@ -2,7 +2,7 @@
2
2
 
3
3
  English | [简体中文](README.zh-CN.md)
4
4
 
5
- [![Version](https://img.shields.io/badge/version-4.8.11-blue.svg)](https://github.com/samuelgursky/davinci-resolve-mcp/releases)
5
+ [![Version](https://img.shields.io/badge/version-4.8.13-blue.svg)](https://github.com/samuelgursky/davinci-resolve-mcp/releases)
6
6
  [![npm](https://img.shields.io/npm/v/davinci-resolve-mcp.svg?label=npm&color=CB3837)](https://www.npmjs.com/package/davinci-resolve-mcp)
7
7
  [![API Coverage](https://img.shields.io/badge/API%20Coverage-100%25-brightgreen.svg)](docs/reference/api-coverage.md)
8
8
  [![Tools](https://img.shields.io/badge/MCP%20Tools-37%20(389%20full)-blue.svg)](#server-modes)
package/README.zh-CN.md CHANGED
@@ -2,7 +2,7 @@
2
2
 
3
3
  [English](README.md) | 简体中文
4
4
 
5
- [![Version](https://img.shields.io/badge/version-4.8.11-blue.svg)](https://github.com/samuelgursky/davinci-resolve-mcp/releases)
5
+ [![Version](https://img.shields.io/badge/version-4.8.13-blue.svg)](https://github.com/samuelgursky/davinci-resolve-mcp/releases)
6
6
  [![npm](https://img.shields.io/npm/v/davinci-resolve-mcp.svg?label=npm&color=CB3837)](https://www.npmjs.com/package/davinci-resolve-mcp)
7
7
  [![API Coverage](https://img.shields.io/badge/API%20Coverage-100%25-brightgreen.svg)](docs/reference/api-coverage.md)
8
8
  [![Tools](https://img.shields.io/badge/MCP%20Tools-37%20(389%20full)-blue.svg)](#服务器模式)
@@ -12,7 +12,7 @@
12
12
  [![Python](https://img.shields.io/badge/python-3.10+-green.svg)](https://www.python.org/downloads/)
13
13
  [![License](https://img.shields.io/badge/license-MIT-blue.svg)](https://opensource.org/licenses/MIT)
14
14
 
15
- > 本翻译对应 v4.8.11 版 README。如与英文原版有出入,以 [英文原版](README.md) 为准。
15
+ > 本翻译对应 v4.8.13 版 README。如与英文原版有出入,以 [英文原版](README.md) 为准。
16
16
 
17
17
  一个 Model Context Protocol (MCP) 服务器,让 AI 助手通过官方脚本 API 控制 DaVinci Resolve Studio(达芬奇)。它提供完整的 API 覆盖,外加带护栏的工作流助手,涵盖剪辑、媒体池整理、渲染设置、审阅标记、调色、Fusion、Fairlight、项目生命周期任务、扩展开发,以及不碰源媒体的媒体分析。
18
18
 
package/install.py CHANGED
@@ -37,7 +37,7 @@ from src.utils.update_check import (
37
37
 
38
38
  # ─── Version ──────────────────────────────────────────────────────────────────
39
39
 
40
- VERSION = "4.8.11"
40
+ VERSION = "4.8.13"
41
41
  # Only hard floor: mcp[cli] requires Python 3.10+. There is no upper bound —
42
42
  # Resolve's scripting bridge loads into newer interpreters on recent builds
43
43
  # (Python 3.14 verified against Resolve Studio 20.3.2). Older Resolve builds
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "davinci-resolve-mcp",
3
- "version": "4.8.11",
3
+ "version": "4.8.13",
4
4
  "description": "NPM bootstrapper for the DaVinci Resolve MCP Server.",
5
5
  "license": "MIT",
6
6
  "author": "Samuel Gursky <samgursky@gmail.com>",
@@ -93,7 +93,7 @@ if not logging.getLogger().handlers:
93
93
  handlers=[logging.StreamHandler()],
94
94
  )
95
95
 
96
- VERSION = "4.8.11"
96
+ VERSION = "4.8.13"
97
97
  logger = logging.getLogger("davinci-resolve-mcp")
98
98
  logger.info(f"Starting DaVinci Resolve MCP Server v{VERSION}")
99
99
  logger.info(f"Detected platform: {get_platform()}")
package/src/server.py CHANGED
@@ -11,7 +11,7 @@ Usage:
11
11
  python src/server.py --full # Start the 377-tool granular server instead
12
12
  """
13
13
 
14
- VERSION = "4.8.11"
14
+ VERSION = "4.8.13"
15
15
 
16
16
  import base64
17
17
  import os
@@ -4705,6 +4705,8 @@ def _normalize_include_linked(raw):
4705
4705
  lowered = raw.strip().lower()
4706
4706
  if lowered in {"all", "true", "yes"}:
4707
4707
  return {"video", "audio"}
4708
+ if lowered in {"false", "no", "0", "off", "none"}:
4709
+ return set()
4708
4710
  return {part.strip().lower() for part in lowered.split(",") if part.strip()}
4709
4711
  if isinstance(raw, list):
4710
4712
  return {str(part).strip().lower() for part in raw if str(part).strip()}