davinci-resolve-mcp 2.34.1 → 2.35.1

package/CHANGELOG.md

@@ -2,6 +2,28 @@
 
 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.35.1
+
+- **Added** `media_pool_item(action="extract_frames", clip_id, timestamps, output_dir?)`
+  extracts still JPEGs from a clip's source media at the given timestamps (seconds)
+  via ffmpeg. Source-safe: it reads the source and writes only to a scratch output
+  directory — it never modifies, transcodes, or proxies the source. (Closes a
+  read/write-symmetry gap: the analysis sampler existed internally but had no
+  standalone frame-extraction tool.)
+
+## What's New in v2.35.0
+
+The Cut-IR executor — transcript-driven editing now closes the loop onto the
+timeline.
+
+- **Added** `timeline(action="apply_cuts", cuts, dry_run?, confirm_token?)` applies
+  a CutList (from `propose_cuts`) to the timeline as lift / ripple deletes. It is
+  **DRY-RUN by default**; applying is destructive and fully governed: confirm-token
+  gated, and a timeline version is archived first (so it is reversible), through
+  the existing destructive hook. Cuts apply latest-first so ripple deletes do not
+  invalidate earlier spans. Live-validated end-to-end (propose -> token -> apply ->
+  version archived).
+
 ## What's New in v2.34.1
 
 Page-switch serialization — the concurrency primitive for safe multi-agent use.

package/README.md

@@ -1,6 +1,6 @@
 # DaVinci Resolve MCP Server
 
-[![Version](https://img.shields.io/badge/version-2.34.1-blue.svg)](https://github.com/samuelgursky/davinci-resolve-mcp/releases)
+[![Version](https://img.shields.io/badge/version-2.35.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-32%20(341%20full)-blue.svg)](#server-modes)

package/docs/SKILL.md

@@ -731,6 +731,9 @@ Key actions:
   text `{text, cue_count, has_subtitles, cues}`
 - `propose_cuts(cues?, long_pause_frames?)` — DRY-RUN: mechanically detect
   candidate cuts (fillers, long pauses, repeats) from the transcript; proposes only
+- `apply_cuts(cuts, dry_run?, confirm_token?)` — apply a CutList as lift/ripple
+  deletes. DRY-RUN by default; applying is destructive (confirm-token gated, a
+  timeline version is archived first). Cuts apply latest-first
 - `add_track(track_type, sub_type?)` / `delete_track(track_type, index)`
 - `get_items(track_type, index)` — items on a track
 - `clip_where(track_type?, track_index?, name_contains?, duration_lt?, duration_gt?)` —

package/install.py

@@ -35,7 +35,7 @@ from src.utils.update_check import (
 
 # ─── Version ──────────────────────────────────────────────────────────────────
 
-VERSION = "2.34.1"
+VERSION = "2.35.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": "2.34.1",
+  "version": "2.35.1",
   "description": "NPM bootstrapper for the DaVinci Resolve MCP Server.",
   "license": "MIT",
   "author": "Samuel Gursky <samgursky@gmail.com>",

package/src/granular/common.py

@@ -80,7 +80,7 @@ if not logging.getLogger().handlers:
         handlers=[logging.StreamHandler()],
     )
 
-VERSION = "2.34.1"
+VERSION = "2.35.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 341-tool granular server instead
 """
 
-VERSION = "2.34.1"
+VERSION = "2.35.1"
 
 import base64
 import os
@@ -729,6 +729,7 @@ _destructive_hook.register_preference_provider(_destructive_preference_provider)
 # a version. Keep in sync with the _issue_confirm_token call sites.
 _TOKEN_GATED_DESTRUCTIVE_ACTIONS = frozenset({
     ("timeline", "delete_track"),
+    ("timeline", "apply_cuts"),
     ("graph", "apply_grade_from_drx"),
     ("graph", "reset_all_grades"),
     # 21.0 AI ops that render/generate NEW media files (additive, but expensive
@@ -5509,6 +5510,55 @@ def _audio_mapping_report(mp, tl, p: Dict[str, Any]):
     return {"timeline_items": timeline_items, "media_pool_items": clip_rows}
 
 
+def _extract_clip_frames(clip, p: Dict[str, Any]) -> Dict[str, Any]:
+    """Extract still frames from a clip's source media at given timestamps.
+
+    Source-safe: reads the source file with ffmpeg and writes JPEGs to a scratch
+    output directory only — it never modifies, transcodes, or proxies the source.
+    `timestamps` is a list of seconds. Returns the written frame paths.
+    """
+    try:
+        src = clip.GetClipProperty("File Path")
+    except Exception:
+        src = None
+    if not src or not os.path.isfile(src):
+        return _err("Clip has no readable source file (File Path)")
+    timestamps = p.get("timestamps")
+    if not isinstance(timestamps, list) or not timestamps:
+        return _err("extract_frames requires 'timestamps' (a non-empty list of seconds)")
+    if not shutil.which("ffmpeg"):
+        return _err("ffmpeg not found on PATH")
+    out_dir = p.get("output_dir") or tempfile.mkdtemp(prefix="mcp_frames_")
+    try:
+        os.makedirs(out_dir, exist_ok=True)
+    except OSError as exc:
+        return _err(f"Could not create output_dir: {exc}")
+    paths, errors = [], []
+    for i, ts in enumerate(timestamps):
+        out_path = os.path.join(out_dir, f"frame_{i:04d}.jpg")
+        try:
+            safe_run(
+                ["ffmpeg", "-y", "-ss", str(float(ts)), "-i", src,
+                 "-frames:v", "1", "-q:v", "2", out_path],
+                capture_output=True, timeout=60,
+            )
+        except (OSError, subprocess.SubprocessError, ValueError) as exc:
+            errors.append({"timestamp": ts, "error": str(exc)})
+            continue
+        if os.path.isfile(out_path):
+            paths.append(out_path)
+        else:
+            errors.append({"timestamp": ts, "error": "no frame written (timestamp out of range?)"})
+    return {
+        "success": bool(paths),
+        "source": src,
+        "output_dir": out_dir,
+        "frame_paths": paths,
+        "count": len(paths),
+        "errors": errors,
+    }
+
+
 def _clip_name(clip):
     try:
         return clip.GetName()
@@ -13796,6 +13846,9 @@ def media_pool_item(action: str, params: Optional[Dict[str, Any]] = None) -> Dic
       get_transcription(clip_id) -> {text, truncated, status, has_transcription}
         Read a clip's transcription. `truncated` flags when Resolve's preview
         property cut the text off (the full transcript is longer).
+      extract_frames(clip_id, timestamps, output_dir?) -> {frame_paths, output_dir, count, errors}
+        Extract still JPEGs from the clip's source at the given timestamps (seconds)
+        via ffmpeg. Source-safe: reads source, writes only to a scratch dir.
       perform_audio_classification(clip_id) -> {success}  — Resolve 21+
       clear_audio_classification(clip_id) -> {success}  — Resolve 21+
       analyze_for_intellisearch(clip_id, identify_faces?, is_better_mode?) -> {success}  — Resolve 21+, AI IntelliSearch Extra
@@ -14011,6 +14064,8 @@ def media_pool_item(action: str, params: Optional[Dict[str, Any]] = None) -> Dic
             "status": status or None,
             "has_transcription": bool(text.strip()),
         }
+    elif action == "extract_frames":
+        return _extract_clip_frames(clip, p)
     elif action == "perform_audio_classification":
         missing = _requires_method(clip, "PerformAudioClassification", "21.0")
         if missing:
@@ -14095,7 +14150,7 @@ def media_pool_item(action: str, params: Optional[Dict[str, Any]] = None) -> Dic
         return {"success": bool(clip.SetMarkInOut(clean["mark_in"], clean["mark_out"], p.get("type", "all")))}
     elif action == "clear_mark_in_out":
         return {"success": bool(clip.ClearMarkInOut(p.get("type", "all")))}
-    return _unknown(action, ["get_name","get_metadata","set_metadata","get_third_party_metadata","set_third_party_metadata","get_media_id","get_clip_property","set_clip_property","get_clip_color","set_clip_color","clear_clip_color","link_proxy","unlink_proxy","replace_clip","set_name","link_full_resolution_media","monitor_growing_file","replace_clip_preserve_sub_clip","get_unique_id","transcribe_audio","clear_transcription","get_transcription","perform_audio_classification","clear_audio_classification","analyze_for_intellisearch","analyze_for_slate","remove_motion_blur","get_audio_mapping","get_mark_in_out","set_mark_in_out","clear_mark_in_out","open_in_viewer"])
+    return _unknown(action, ["get_name","get_metadata","set_metadata","get_third_party_metadata","set_third_party_metadata","get_media_id","get_clip_property","set_clip_property","get_clip_color","set_clip_color","clear_clip_color","link_proxy","unlink_proxy","replace_clip","set_name","link_full_resolution_media","monitor_growing_file","replace_clip_preserve_sub_clip","get_unique_id","transcribe_audio","clear_transcription","get_transcription","extract_frames","perform_audio_classification","clear_audio_classification","analyze_for_intellisearch","analyze_for_slate","remove_motion_blur","get_audio_mapping","get_mark_in_out","set_mark_in_out","clear_mark_in_out","open_in_viewer"])
 
 
 # ═══════════════════════════════════════════════════════════════════════════════
@@ -15209,6 +15264,10 @@ def timeline(action: str, params: Optional[Dict[str, Any]] = None) -> Dict[str,
       propose_cuts(cues?, long_pause_frames?) -> {cuts, cut_count, basis_cue_count, pass, note}
         DRY-RUN. Mechanically detect candidate cuts (filler words, long pauses,
         repeated lines) from the subtitle transcript. Proposes only; applies nothing.
+      apply_cuts(cuts, dry_run?, confirm_token?, allow_partial_item_delete?) -> {applied, total, results}
+        Apply a CutList (from propose_cuts) as lift/ripple deletes. DRY-RUN by
+        default; applying is DESTRUCTIVE — confirm-token gated and a timeline
+        version is archived first. Cuts apply latest-first.
       get_mark_in_out() -> {mark}
       set_mark_in_out(mark_in, mark_out, type?) -> {success}
       clear_mark_in_out(type?) -> {success}
@@ -15503,6 +15562,61 @@ def timeline(action: str, params: Optional[Dict[str, Any]] = None) -> Dict[str,
         if cues is None:
             cues = _timeline_transcript(tl, with_timecodes=True)["cues"]
         return _build_cut_list(cues, long_pause_frames=int(p.get("long_pause_frames", 48)))
+    elif action == "apply_cuts":
+        # Apply a CutList (from propose_cuts) to the timeline. DRY-RUN by default;
+        # applying is destructive (confirm-token gated, a version is archived
+        # first by the destructive hook). Cuts are applied latest-first so ripple
+        # deletes don't invalidate earlier spans.
+        cuts = p.get("cuts")
+        if not isinstance(cuts, list):
+            return _err("apply_cuts requires 'cuts' (a list, e.g. from propose_cuts)")
+        applicable = [
+            c for c in cuts
+            if isinstance(c, dict) and c.get("action") in ("lift", "ripple_delete")
+            and isinstance(c.get("span"), dict)
+            and c["span"].get("start") is not None and c["span"].get("end") is not None
+        ]
+        applicable.sort(key=lambda c: c["span"]["start"], reverse=True)
+        plan = [{"action": c["action"], "span": c["span"], "kind": c.get("kind")} for c in applicable]
+
+        if p.get("dry_run", True):
+            return {
+                "dry_run": True,
+                "would_apply": len(applicable),
+                "plan": plan,
+                "note": "No edits made. Re-run with dry_run=false (and a confirm_token) to apply.",
+            }
+
+        if "confirm_token" not in p and "confirmToken" not in p and _confirm_token_required():
+            return _issue_confirm_token(
+                action="timeline.apply_cuts", params=p,
+                preview={
+                    "operation": "timeline.apply_cuts",
+                    "warning": "Applies lift/ripple deletes to the timeline; ripple closes "
+                               "gaps and cannot be selectively undone. A timeline version is "
+                               "archived first.",
+                    "would_apply": len(applicable),
+                    "plan": plan,
+                },
+            )
+        blocked = _consume_confirm_token(action="timeline.apply_cuts", params=p)
+        if blocked:
+            return blocked
+
+        allow_partial = bool(p.get("allow_partial_item_delete", True))
+        results = []
+        for c in applicable:
+            sp = c["span"]
+            res = _timeline_lift_range_impl(tl, {
+                "start_frame": sp["start"],
+                "end_frame": sp["end"],
+                "ripple": c["action"] == "ripple_delete",
+                "allow_partial_item_delete": allow_partial,
+            })
+            results.append({"action": c["action"], "span": sp, "result": res})
+        applied = sum(1 for r in results
+                      if isinstance(r["result"], dict) and r["result"].get("success"))
+        return {"success": True, "applied": applied, "total": len(applicable), "results": results}
     elif action == "get_mark_in_out":
         return _ser(tl.GetMarkInOut())
     elif action == "set_mark_in_out":
@@ -15712,7 +15826,7 @@ def timeline(action: str, params: Optional[Dict[str, Any]] = None) -> Dict[str,
         if mp_err:
             return mp_err
         return _fairlight_boundary_report(proj, mp, tl, p)
-    return _unknown(action, ["list","get_current","set_current","get_name","set_name","get_start_frame","get_end_frame","get_start_timecode","set_start_timecode","get_track_count","add_track","delete_track","get_track_sub_type","set_track_enable","get_track_enabled","set_track_lock","get_track_locked","get_track_name","set_track_name","get_items","delete_clips","set_clips_linked","duplicate","duplicate_clips","copy_clips","move_clips","copy_range","duplicate_range","overwrite_range","lift_range","story_spine_report","create_variant_from_ranges","bulk_set_item_properties","apply_look_to_items","thumbnail_contact_sheet","marker_thumbnail_review","edit_kernel_capabilities","probe_edit_kernel_item","title_property_scan","set_title_text","bulk_set_title_text","create_compound_clip","create_fusion_clip","import_into_timeline","export","get_setting","set_setting","insert_generator","insert_fusion_generator","insert_fusion_composition","insert_ofx_generator","insert_title","insert_fusion_title","get_unique_id","get_node_graph","get_media_pool_item","get_transcript","propose_cuts","get_mark_in_out","set_mark_in_out","clear_mark_in_out","convert_to_stereo","get_items_in_track","get_voice_isolation_state","set_voice_isolation_state","extract_source_frame_ranges","audio_mix_capability_report",*_TIMELINE_CONFORM_KERNEL_ACTIONS,*_TIMELINE_AUDIO_KERNEL_ACTIONS])
+    return _unknown(action, ["list","get_current","set_current","get_name","set_name","get_start_frame","get_end_frame","get_start_timecode","set_start_timecode","get_track_count","add_track","delete_track","get_track_sub_type","set_track_enable","get_track_enabled","set_track_lock","get_track_locked","get_track_name","set_track_name","get_items","delete_clips","set_clips_linked","duplicate","duplicate_clips","copy_clips","move_clips","copy_range","duplicate_range","overwrite_range","lift_range","story_spine_report","create_variant_from_ranges","bulk_set_item_properties","apply_look_to_items","thumbnail_contact_sheet","marker_thumbnail_review","edit_kernel_capabilities","probe_edit_kernel_item","title_property_scan","set_title_text","bulk_set_title_text","create_compound_clip","create_fusion_clip","import_into_timeline","export","get_setting","set_setting","insert_generator","insert_fusion_generator","insert_fusion_composition","insert_ofx_generator","insert_title","insert_fusion_title","get_unique_id","get_node_graph","get_media_pool_item","get_transcript","propose_cuts","apply_cuts","get_mark_in_out","set_mark_in_out","clear_mark_in_out","convert_to_stereo","get_items_in_track","get_voice_isolation_state","set_voice_isolation_state","extract_source_frame_ranges","audio_mix_capability_report",*_TIMELINE_CONFORM_KERNEL_ACTIONS,*_TIMELINE_AUDIO_KERNEL_ACTIONS])
 
 
 # ═══════════════════════════════════════════════════════════════════════════════

package/src/utils/destructive_hook.py

@@ -70,6 +70,7 @@ DESTRUCTIVE_ACTIONS_BY_TOOL: Dict[str, FrozenSet[str]] = {
         "duplicate_range",
         "overwrite_range",
         "lift_range",
+        "apply_cuts",
         "create_stereo_clip",
         "auto_sync_audio",
         "create_compound_clip",