davinci-resolve-mcp 4.17.0 → 4.17.1

package/CHANGELOG.md

@@ -2,6 +2,33 @@
 
 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.17.1 — Media analysis decodes each file once
+
+### Changed
+
+- **The readthrough pass of media analysis runs ffmpeg once instead of four
+  times** (#329, thanks @Dev-next-gen). Loudness, scene scoring, black frames
+  and silence each used to be a separate ffmpeg run, and each run decoded the
+  whole file: `-af` alone does not drop the video stream from a null output,
+  so the audio passes decoded every video frame too. The four filters now
+  share one run through `split`/`asplit`. On a 1080p H.264 file the
+  readthrough took 8.1 s instead of 22.1 s, and CPU time fell from 84.8 s to
+  30.5 s; ProRes and HEVC dropped by about the same share. The results are
+  the same as before. The parsers are unchanged.
+- The four measurements now share one exit code and one timeout. The single
+  run gets four times the old per-pass timeout, the budget of the four runs it
+  replaces. If that run times out, all four report `success: false`, where
+  before the other three could still succeed.
+
+### Validation
+
+- Python suite with the PR merged on `main`: 4,291 tests. The only failures
+  are the 4 `source_media_guard` path checks that always fail in a scratch
+  worktree. The new test compares the single run against the four separate
+  runs with the real ffmpeg on an A/V file, a video-only file and an
+  audio-only file.
+- No Resolve behaviour changed; this is ffmpeg only.
+
 ## What's New in v4.17.0 — Power windows for Resolve 21
 
 ### Added

package/README.md

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

package/src/granular/common.py

@@ -92,7 +92,7 @@ if not logging.getLogger().handlers:
         handlers=[logging.StreamHandler()],
     )
 
-VERSION = "4.17.0"
+VERSION = "4.17.1"
 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 377-tool granular server instead
 """
 
-VERSION = "4.17.0"
+VERSION = "4.17.1"
 
 import base64
 import os

package/src/utils/media_analysis.py

@@ -2781,7 +2781,13 @@ def _media_duration_seconds(record: Dict[str, Any], technical: Dict[str, Any]) -
     return None
 
 
-def _ffmpeg_stderr_filter(path: str, video_filter: Optional[str] = None, audio_filter: Optional[str] = None, frames: Optional[int] = None) -> Tuple[int, str]:
+def _ffmpeg_stderr_filter(
+    path: str,
+    video_filter: Optional[str] = None,
+    audio_filter: Optional[str] = None,
+    frames: Optional[int] = None,
+    timeout: int = COMMAND_TIMEOUT_SECONDS,
+) -> Tuple[int, str]:
     args = ["ffmpeg", "-hide_banner", "-nostats", "-i", path]
     if video_filter:
         args.extend(["-vf", video_filter])
@@ -2790,7 +2796,7 @@ def _ffmpeg_stderr_filter(path: str, video_filter: Optional[str] = None, audio_f
     if frames is not None:
         args.extend(["-frames:v", str(frames)])
     args.extend(["-f", "null", "-"])
-    code, _, stderr = _run_command(args)
+    code, _, stderr = _run_command(args, timeout=timeout)
     return code, stderr
 
 
@@ -2934,25 +2940,45 @@ def _parse_idet(stderr: str) -> Dict[str, Any]:
     }
 
 
+# The four whole-file measurements `_readthrough_analysis` makes. They used to be four
+# ffmpeg runs, and every run decoded the whole file: `-af` alone does not drop the video
+# stream from a null output, so the loudness and silence passes decoded every frame just
+# as the scene and black passes did, and each video pass decoded the audio too. One run
+# with `split`/`asplit` decodes each stream once and feeds all four filters. Each parser
+# below matches only its own filter's lines, so sharing one stderr changes no result.
+# A missing stream behaves as it did per pass: ffmpeg skips the -vf or -af it cannot
+# use, exits 0, and that filter's parser finds nothing.
+_SCENE_SCORE_FILTER = "select='gt(scene,0)',metadata=print:key=lavfi.scene_score,showinfo"
+_BLACKDETECT_FILTER = "blackdetect=d=0.5:pix_th=0.10"
+_LOUDNESS_FILTER = "ebur128=peak=true"
+_SILENCEDETECT_FILTER = "silencedetect=noise=-50dB:d=1"
+_READTHROUGH_VIDEO_FILTER = f"split[scene][black];[scene]{_SCENE_SCORE_FILTER},nullsink;[black]{_BLACKDETECT_FILTER}"
+_READTHROUGH_AUDIO_FILTER = f"asplit[loud][silence];[loud]{_LOUDNESS_FILTER},anullsink;[silence]{_SILENCEDETECT_FILTER}"
+
+
 def _readthrough_analysis(path: str) -> Dict[str, Any]:
     result: Dict[str, Any] = {"success": True}
 
-    loud_code, loud_stderr = _ffmpeg_stderr_filter(path, audio_filter="ebur128=peak=true")
+    # The one run carries the timeout budget of the four it replaces, so a file whose
+    # slowest pass (scene scoring) finished in time before still finishes in time.
+    code, stderr = _ffmpeg_stderr_filter(
+        path,
+        video_filter=_READTHROUGH_VIDEO_FILTER,
+        audio_filter=_READTHROUGH_AUDIO_FILTER,
+        timeout=COMMAND_TIMEOUT_SECONDS * 4,
+    )
+
     result["loudness"] = {
-        "success": loud_code == 0,
-        "metrics": _parse_loudness(loud_stderr),
+        "success": code == 0,
+        "metrics": _parse_loudness(stderr),
     }
 
-    # Adaptive scene detection. One ffmpeg pass dumps a (pts_time, scene_score) pair
+    # Adaptive scene detection. The scene branch dumps a (pts_time, scene_score) pair
     # for every frame whose scene score is > 0 (i.e. every non-first frame); we then
     # compute a content-aware threshold from that distribution and keep peaks above it.
     # Replaces the legacy hardcoded ``gt(scene,0.3)``, which was too coarse on
     # high-motion content (missed real cuts) and too sensitive on locked-off content.
-    scene_code, scene_stderr = _ffmpeg_stderr_filter(
-        path,
-        video_filter="select='gt(scene,0)',metadata=print:key=lavfi.scene_score,showinfo",
-    )
-    scene_score_pairs = _parse_scene_score_pairs(scene_stderr)
+    scene_score_pairs = _parse_scene_score_pairs(stderr)
     scene_threshold, scene_threshold_stats = _adaptive_scene_threshold(scene_score_pairs)
     adaptive_scene_items = [
         {"time_seconds": pts, "score": score}
@@ -2960,22 +2986,20 @@ def _readthrough_analysis(path: str) -> Dict[str, Any]:
         if score is not None and score > scene_threshold
     ]
     result["scenes"] = {
-        "success": scene_code == 0,
+        "success": code == 0,
         "items": adaptive_scene_items,
         "threshold": scene_threshold,
         "threshold_stats": scene_threshold_stats,
     }
 
-    black_code, black_stderr = _ffmpeg_stderr_filter(path, video_filter="blackdetect=d=0.5:pix_th=0.10")
     result["black_frames"] = {
-        "success": black_code == 0,
-        "items": _parse_blackdetect(black_stderr),
+        "success": code == 0,
+        "items": _parse_blackdetect(stderr),
     }
 
-    silence_code, silence_stderr = _ffmpeg_stderr_filter(path, audio_filter="silencedetect=noise=-50dB:d=1")
     result["silence"] = {
-        "success": silence_code == 0,
-        "items": _parse_silencedetect(silence_stderr),
+        "success": code == 0,
+        "items": _parse_silencedetect(stderr),
     }
 
     idet_code, idet_stderr = _ffmpeg_stderr_filter(path, video_filter="idet", frames=500)