davinci-resolve-mcp 2.212.1 → 2.212.2

package/CHANGELOG.md

@@ -2,6 +2,32 @@
 
 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.212.2 — failed verification evidence survives the operation envelope
+
+### Fixed
+
+- **A payload that carried its own `verification` block won outright, hiding
+  contradicting evidence beside it.** `extract_verification` returned any
+  pre-shaped `verification` dict untouched, so a top-level `contradiction:
+  true`, a `readback.missing` list, or a failed check inside the block itself
+  could sit next to `status: "passed"`. Evidence now merges: the explicit
+  status, readback misses, post-state readback and property-restore failures
+  are all collected, and precedence runs contradiction > failed > partial >
+  passed > unverified. Contributed in #195 by @denoise.
+- **Bulk command counts no longer count as verification.** `succeeded` /
+  `failed` tallies record what the server sent, not what Resolve honoured; a
+  `succeeded: 3, failed: 0` result used to read `verification.status:
+  "passed"` with no readback at all. The counts still drive the envelope's
+  own `status` (`partial` when both are non-zero), but the verification block
+  stays `unverified` until real evidence — a readback — arrives.
+
+### Validation
+
+- The PR's regression tests plus two added on landing: bulk counts cannot
+  mask a failed readback, and readback evidence is what establishes a pass on
+  a bulk result. Full offline suite, the drift guards and the advanced Node
+  suite are green. No Resolve behavior changed; live test not required.
+
 ## What's New in v2.212.1 — the networked transport's generated bearer token no longer lands in server.log
 
 ### Fixed

package/README.md

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

package/README.zh-CN.md

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

package/src/granular/common.py

@@ -87,7 +87,7 @@ if not logging.getLogger().handlers:
         handlers=[logging.StreamHandler()],
     )
 
-VERSION = "2.212.1"
+VERSION = "2.212.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 353-tool granular server instead
 """
 
-VERSION = "2.212.1"
+VERSION = "2.212.2"
 
 import base64
 import os

package/src/utils/operation_result.py

@@ -166,75 +166,57 @@ def extract_verification(raw: Any) -> Dict[str, Any]:
     if not isinstance(raw, dict):
         return unverified
 
-    # An impl that already speaks this shape wins outright.
     existing = raw.get("verification")
-    if isinstance(existing, dict) and existing:
-        merged = dict(existing)
-        merged.setdefault(
-            "status", "passed" if existing.get("verified") else "unverified")
-        merged.setdefault("checks", [])
-        merged.setdefault("contradiction", False)
-        return merged
-
-    checks: List[Dict[str, Any]] = []
-    status = "unverified"
-    contradiction = False
+    existing = existing if isinstance(existing, dict) else {}
+    checks = list(existing.get("checks") or [])
+    statuses = []
+    existing_status = existing.get("status")
+    if existing_status in {"passed", "failed", "partial", "contradiction"}:
+        statuses.append(existing_status)
+    elif existing.get("verified") is True:
+        statuses.append("passed")
+    elif existing.get("verified") is False:
+        statuses.append("failed")
+    contradiction = existing.get("contradiction") is True or raw.get("contradiction") is True
 
     readback = raw.get("readback")
-    if isinstance(readback, dict):
-        missing = readback.get("missing")
-        if isinstance(missing, list):
-            checks.append({
-                "check": "readback_verification",
-                "passed": not missing,
-                "missing_items": len(missing),
-            })
-            status = "passed" if not missing else "failed"
-
-    # verify_by_readback's own shape: a mutation that reported success while the
-    # post-state disagrees is a contradiction, this repo's single most valuable
-    # reliability signal — it must not be flattened into a plain failure.
-    if "verified" in raw:
-        verified = bool(raw["verified"])
-        contradiction = bool(raw.get("contradiction"))
-        checks.append({
-            "check": "readback_post_state",
-            "passed": verified,
-            "contradiction": contradiction,
-            "observed": raw.get("observed"),
-        })
-        status = "contradiction" if contradiction else ("passed" if verified else "failed")
+    if isinstance(readback, dict) and isinstance(readback.get("missing"), list):
+        missing = readback["missing"]
+        checks.append({"check": "readback_verification", "passed": not missing,
+                       "missing_items": len(missing)})
+        statuses.append("failed" if missing else "passed")
+
+    if type(raw.get("verified")) is bool:
+        checks.append({"check": "readback_post_state", "passed": raw["verified"],
+                       "contradiction": contradiction, "observed": raw.get("observed")})
+        statuses.append("passed" if raw["verified"] else "failed")
 
     if "property_restore_failures" in raw:
         failures = _as_int(raw.get("property_restore_failures"))
-        checks.append({
-            "check": "property_restore",
-            "passed": failures == 0,
-            "restored_items": _as_int(raw.get("properties_restored_items")),
-            "failures": failures,
-        })
-        if failures and status in ("passed", "unverified"):
-            status = "partial"
-
-    succeeded, failed = raw.get("succeeded"), raw.get("failed")
-    if isinstance(succeeded, int) and isinstance(failed, int):
-        checks.append({
-            "check": "bulk_operations",
-            "passed": failed == 0,
-            "succeeded": succeeded,
-            "failed": failed,
-        })
-        if status == "unverified":
-            if failed == 0 and succeeded > 0:
-                status = "passed"
-            elif succeeded > 0:
-                status = "partial"
-            elif failed > 0:
-                status = "failed"
-
-    if not checks:
-        return unverified
-    return {"status": status, "checks": checks, "contradiction": contradiction}
+        checks.append({"check": "property_restore", "passed": failures == 0,
+                       "restored_items": _as_int(raw.get("properties_restored_items")),
+                       "failures": failures})
+        if failures:
+            statuses.append("partial")
+
+    for check in checks:
+        if isinstance(check, dict):
+            contradiction = contradiction or check.get("contradiction") is True
+            if check.get("passed") is False:
+                statuses.append("partial" if check.get("check") == "property_restore" else "failed")
+    # Command counts are not evidence that Resolve honored those commands.
+    if contradiction or "contradiction" in statuses:
+        status = "contradiction"
+        contradiction = True
+    elif "failed" in statuses:
+        status = "failed"
+    elif "partial" in statuses:
+        status = "partial"
+    elif "passed" in statuses:
+        status = "passed"
+    else:
+        status = "unverified"
+    return {**existing, "status": status, "checks": checks, "contradiction": contradiction}
 
 
 # ─── Changes ─────────────────────────────────────────────────────────────────