davinci-resolve-mcp 4.8.19 → 4.8.21

package/src/utils/api_truth.py

@@ -2005,6 +2005,30 @@ API_TRUTH: List[Dict[str, Any]] = [
                        "when mirroring keep-ranges into clipInfos.",
         "tags": ["timeline", "edit", "off-by-one", "readback"],
     },
+    {
+        "symbol": "Timeline.GetEndFrame (exclusive bound)",
+        "object": "Timeline",
+        "signature": "() -> int",
+        "reality": "GetEndFrame() is an EXCLUSIVE bound, one past the last "
+                   "frame: a timeline's frame count is "
+                   "GetEndFrame() - GetStartFrame(), not that plus one. "
+                   "Measured live on Studio 19.1.3.7 (2026-09-26): a "
+                   "timeline whose only full-length item is a 32742-frame "
+                   "clip (media-pool 'Frames' 32742, 'End' 32741) reads "
+                   "GetStartFrame() 0 / GetEndFrame() 32742, and that item's "
+                   "TimelineItem.GetEnd() is also 32742 - the same half-open "
+                   "convention as clipInfo endFrame above. Until v4.8.21 two "
+                   "readers here (granular get_current_timeline and "
+                   "get_project_info's per-timeline duration) added 1 and "
+                   "reported 601 frames for a 600-frame timeline while every "
+                   "other duration path in the server did not (PR #269).",
+        "recommended": "duration_frames = GetEndFrame() - GetStartFrame(). "
+                       "The last frame index is GetEndFrame() - 1. Apply the "
+                       "same rule to TimelineItem.GetEnd(): an item occupies "
+                       "[GetStart(), GetEnd()).",
+        "tags": ["timeline", "off-by-one", "readback", "duration"],
+        "verified_on": "DaVinci Resolve Studio 19.1.3.7",
+    },
     {
         "symbol": "MediaPool.ImportTimelineFromFile (internal sequence name overrides timelineName)",
         "object": "MediaPool",

package/src/utils/dctl_templates.py

@@ -12,6 +12,8 @@ Resolve re-scan its LUT directory. ACES DCTLs are scanned only at startup.
 
 from typing import Any, Dict, List, Optional
 
+from src.utils.bool_params import coerce_bool
+
 
 def header(name: str, kind: str) -> str:
     """Marker comment used by the dctl tool's `list` action."""
@@ -264,7 +266,7 @@ def aces_idt(name: str, options: Optional[Dict[str, Any]] = None) -> str:
     Install requires a Resolve restart. See docs/notes/dctl-notes.md → "DCTL And ACES".
     """
     options = options or {}
-    parametric = bool(options.get("parametric", False))
+    parametric = coerce_bool(options.get("parametric"), False)
     body = options.get("body") or (
         "    return make_float3(p_R, p_G, p_B);"
     )
@@ -290,7 +292,7 @@ def aces_odt(name: str, options: Optional[Dict[str, Any]] = None) -> str:
     Resolve which side of the pipeline this transform belongs to.
     """
     options = options or {}
-    parametric = bool(options.get("parametric", False))
+    parametric = coerce_bool(options.get("parametric"), False)
     body = options.get("body") or (
         "    return make_float3(p_R, p_G, p_B);"
     )

package/src/utils/fuse_templates.py

@@ -10,6 +10,8 @@ Reference: Fusion Fuse SDK (June 2023), Blackmagic Design.
 
 from typing import Any, Dict, Optional
 
+from src.utils.bool_params import coerce_bool
+
 # Maps the public `type` argument to Fusion's internal ClassType enum.
 CLASS_TYPES = {
     "tool": "CT_Tool",
@@ -149,7 +151,7 @@ def per_pixel(name: str, options: Optional[Dict[str, Any]] = None) -> str:
     if n_inputs not in (1, 2):
         raise ValueError("per_pixel: inputs must be 1 or 2")
     expression = options.get("expression", "    return p1")
-    expose_amount = bool(options.get("amount", True))
+    expose_amount = coerce_bool(options.get("amount"), True)
 
     pixel_args = "x, y, p1, p2" if n_inputs == 2 else "x, y, p1"
 
@@ -1021,7 +1023,7 @@ def channel_op(name: str, options: Optional[Dict[str, Any]] = None) -> str:
     if operation not in valid_ops:
         raise ValueError(f"Invalid channel_op operation '{operation}'. "
                          f"Valid: {list(valid_ops)}")
-    rgba_only = bool(options.get("rgba_only", True))
+    rgba_only = coerce_bool(options.get("rgba_only"), True)
 
     aux_channels = ""
     aux_doc = ""

package/src/utils/media_analysis_jobs.py

@@ -18,6 +18,8 @@ import time
 from pathlib import Path
 from typing import Any, Dict, Iterable, List, Optional, Tuple
 
+from src.utils.bool_params import coerce_bool
+
 from src.utils.media_analysis import (
     ANALYSIS_VERSION,
     build_analysis_index,
@@ -641,7 +643,7 @@ def _finish_job_if_complete(
     _event(conn, job_id, "info", "Batch job completed", counts)
     conn.commit()
     summarize_reports(root)
-    if params.get("auto_build_index", True) and not index_already_refreshed:
+    if coerce_bool(params.get("auto_build_index"), True) and not index_already_refreshed:
         _auto_build_index(conn, root, job_id, "Analysis index rebuilt")
 
 
@@ -781,7 +783,7 @@ def run_batch_job_slice(
             conn.commit()
 
         index_refreshed = False
-        if params.get("auto_build_index", True) and any(
+        if coerce_bool(params.get("auto_build_index"), True) and any(
             row.get("status") in {"succeeded", "skipped"} for row in processed
         ):
             _auto_build_index(conn, root, job_id, "Analysis index refreshed after batch slice")

package/src/utils/multicam.py

@@ -10,6 +10,8 @@ from __future__ import annotations
 import re
 from typing import Any, Callable, Dict, List, Optional, Tuple
 
+from src.utils.bool_params import coerce_bool
+
 
 FindClip = Callable[[Any, str], Any]
 
@@ -323,8 +325,8 @@ def build_multicam_setup_plan(root: Any, params: Dict[str, Any], find_clip: Find
     if not sync_mode:
         return None, _err("sync_mode must be stack_start, record_frame, or source_timecode")
 
-    include_video = bool(params.get("include_video", params.get("includeVideo", True)))
-    include_audio = bool(params.get("include_audio", params.get("includeAudio", False)))
+    include_video = coerce_bool(params.get("include_video", params.get("includeVideo")), True)
+    include_audio = coerce_bool(params.get("include_audio", params.get("includeAudio")), False)
     if not include_video and not include_audio:
         return None, _err("At least one of include_video or include_audio must be true")
     audio_mode = _normalize_audio_mode(params.get("audio_track_mode", params.get("audioTrackMode")), include_audio)
@@ -347,7 +349,7 @@ def build_multicam_setup_plan(root: Any, params: Dict[str, Any], find_clip: Find
     angles: List[Dict[str, Any]] = []
     max_video_track = 0
     max_audio_track = 0
-    allow_negative = bool(params.get("allow_negative_record_frame", params.get("allowNegativeRecordFrame", False)))
+    allow_negative = coerce_bool(params.get("allow_negative_record_frame", params.get("allowNegativeRecordFrame")), False)
     record_frame_mode = params.get("record_frame_mode", params.get("recordFrameMode", "relative"))
 
     for index, raw in enumerate(raw_angles):

package/src/utils/project_properties.py

@@ -592,7 +592,7 @@ def get_project_info(project_obj) -> Dict[str, Any]:
                 timeline_info = {
                     "name": timeline.GetName(),
                     "isCurrent": timeline.GetName() == current_timeline_name,
-                    "duration": timeline.GetEndFrame() - timeline.GetStartFrame() + 1
+                    "duration": timeline.GetEndFrame() - timeline.GetStartFrame()
                 }
                 project_info["timelines"].append(timeline_info)