ffmpeg-skill 2.10.0 → 2.10.1

package/README.md

@@ -248,7 +248,7 @@ These are the rules the skill file gives the agent and the code enforces.
 
 | Tool | What it does |
 |---|---|
-| `export.py` | Presets `youtube`, `youtube4k`, `reels`, `tiktok`, `shorts`, `linkedin`, `facebook`, `x`, `youtube-hdr` (HEVC Main10, source HDR tags kept), `youtube-av1`, `prores`, `h265`, `gif`, `copy`, all tagged BT.709 unless they carry HDR; `--normalize` meets the platform's loudness in the same call (`render.py` turns it on by default for platform presets) |
+| `export.py` | Presets `youtube`, `youtube4k`, `reels`, `tiktok`, `shorts`, `linkedin`, `facebook`, `x`, `youtube-hdr` (HEVC Main10, source HDR tags kept), `youtube-av1`, `prores`, `h265`, `gif`, `copy`, all tagged BT.709 unless they carry HDR (a BT.601-matrix source is converted to BT.709, not relabelled); `--normalize` meets the platform's loudness in the same call (`render.py` turns it on by default for platform presets) |
 | `proxy.py` | Small, low-bitrate proxy for downstream AI analysis/preview/editing decisions — resize by `--width`/`--scale`, proxy-grade `--quality` (default 30), `--fps`, `--no-audio`; not a delivery preset |
 | `check.py` | PASS / WARN / FAIL against YouTube, Shorts, Reels, TikTok, X, LinkedIn, Facebook, broadcast and podcast specs, from the same delivery table the export presets and templates read (podcast also reports chapter markers and channel count), with the fix for each failure and a `format` / `judgement` kind per row |
 | `report.py` | Single-file HTML delivery report: before/after sheets, media facts, loudness, compliance, the commands run; `--pack` renders a social pack table |

package/docs/contract.md

@@ -21,7 +21,7 @@ The contract is derived from the code that runs, not maintained beside it:
 | Field | Meaning | Changes when |
 |---|---|---|
 | `contract_version` | shape of this document (`1.0`) | a key is renamed, removed or changes meaning |
-| `skill.version` | the npm / package.json version (`2.10.0`) | any release |
+| `skill.version` | the npm / package.json version (`2.10.1`) | any release |
 
 A release that adds a tool or a flag keeps `contract_version`; a breaking change to the
 ToolSpec shape bumps it. Consumers pin on `contract_version` and read `skill.version`
@@ -119,7 +119,7 @@ JSON would have been rewritten for that. The flat keys, typed per tool in `outpu
   "contract_version": "1.0",
   "deprecated": [],
   "removed": [{"what": "...", "since": "1.10.0", "replacement": "...", "removed_in": "2.0.0", "where": "cli | json | mcp | behaviour"}],
-  "skill": {"id": "ffmpeg-skill", "version": "2.10.0", "execution_mode": "local", "kind": "execution",
+  "skill": {"id": "ffmpeg-skill", "version": "2.10.1", "execution_mode": "local", "kind": "execution",
             "entrypoints": {"cli": "...", "mcp": "...", "contract": "...", "doctor": "..."},
             "not_provided": ["AI reasoning", "decisions", "production plans", "project IR", "approvals", "network access", "transcription engine"]},
   "requirements": {"python": ">=3.9 (standard library only)", "ffmpeg": ">=5.0", "ffprobe": ">=5.0"},
@@ -217,6 +217,10 @@ BT.709 for SDR; av1 is SVT-AV1 with libaom as the fallback; prores is 422 HQ and
 its default is the tool's own (18, `proxy.py` 30). 1.x also accepted `--crf` as an alias; 2.0
 removed it (`export.py` keeps its own `--crf`). Without `--codec` the encoder is what it always was (x264 for SDR, x265
 Main10 for HDR), so the flags add no behaviour to a caller that does not pass them.
+Every SDR line except prores is tagged BT.709; a source whose matrix is BT.601 (`smpte170m`,
+`bt470bg`, or `fcc`/`smpte240m`) is converted to it in the filter graph (`colorspace`, matrix
+only; `_common.bt709_matrix_filter()`), so the label and the pixels agree. An untagged or BT.709
+source adds no filter, and the tags themselves never convert (`bt709_tag_args()`).
 `cut.py --keep-hevc` (opt-in) re-encodes an SDR HEVC source as x265 8-bit BT.709 instead of x264; an HDR or
 BT.2020 source keeps the Main10 line with its own colour tags, with or without the flag. The
 encoder each value needs is listed under the tool's optional capabilities (`--codec hevc` and

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ffmpeg-skill",
-  "version": "2.10.0",
+  "version": "2.10.1",
   "description": "Agent Skill that gives coding agents (Claude Code, Cursor, Codex) a local video editor: 42 FFmpeg tools with a machine-readable contract, contract-derived MCP server, FFmpeg capability detection, probe-first / verify-last workflow. Cut, join, silence removal, fit, captions and karaoke, overlays, motion graphics, HDR to SDR, LUTs, audio clean-up and typed dynamics, sync with drift correction, multicam, loudness, delivery checks, project rendering, batch. No API keys, no cloud, no dependencies.",
   "keywords": [
     "ffmpeg",

package/references/gotchas.md

@@ -8,7 +8,10 @@ detail, not an aside.
 ## Colour, HDR and Log
 
 ### HDR and colour
-SDR outputs are H.264 tagged BT.709 `yuv420p`. When `probe.py` reports
+SDR outputs are H.264 tagged BT.709 `yuv420p`. A source tagged with a BT.601 matrix
+(`smpte170m`, `bt470bg`; also `fcc`, `smpte240m`) has its pixels converted to that BT.709 matrix
+by every re-encode (through 2.10.0 only the label changed); `color.py --retag bt709` is a
+stream copy and still only relabels. Untagged sources are taken as BT.709 and left alone. When `probe.py` reports
 `bt2020_or_hdr: true` (HDR10/PQ, HLG, Dolby Vision, or BT.2020 SDR), every editing script keeps
 the output on the 10-bit path (HEVC Main10, source colour tags) so nothing is silently
 flattened. Decide with the user: keep HDR (fine for YouTube/phones) or run

package/references/scripts.md

@@ -1615,7 +1615,7 @@ export.py INPUT --preset youtube|youtube4k|reels|tiktok|shorts|linkedin|facebook
                 [--fit pad|crop] [--no-scale] [--allow-long] [--crf N] [--normalize] [-o OUT]
 export.py --list
 ```
-Scales into the preset frame (pad by default), tags BT.709, sets `+faststart`,
+Scales into the preset frame (pad by default), tags BT.709 (a BT.601-matrix source is converted to it, as by every re-encode), sets `+faststart`,
 trims to platform maximums (Reels 90 s, X 140 s) unless `--allow-long`. It
 does not touch levels: for youtube / youtube4k / reels / x the written file is
 measured and the result's `loudness` (and a `notes` line) says when it is

package/scripts/_common/__init__.py

@@ -83,7 +83,7 @@ from _common.decision import (
     _onset_strength, _pick_onsets, _autocorrelation_peak, _grid_score,
     brand_states_font, cfr_args, concat_list_line, db_to_linear, default_output, encoder_args, escape_filter_path,
     fmt_secs, fmt_smpte_time, fmt_srt_time, frame_size, is_audio_output, load_brand, MissingFpsError, pad_filters, parse_time,
-    signed_time_arg, source_codec_video_args, SVT_PRESET, time_arg, video_args, x264_args, _x264_raw
+    signed_time_arg, source_codec_video_args, matrix_chain, SVT_PRESET, time_arg, video_args, x264_args, _x264_raw
 )
 from _common.asr import (
     ASR_ENGINES, ASR_INSTALL_HINT, _asr_run, die_no_engine, parse_srt, transcribe, _transcribe_in,
@@ -91,7 +91,8 @@ from _common.asr import (
     whisper_word_timings, write_srt
 )
 from _common.color import (
-    bt709_tag_args, color_hex, _COLOR_TOKEN_RE, _sdr_bt709, validate_color
+    bt709_matrix_args, bt709_matrix_filter, bt709_tag_args, color_hex, _COLOR_TOKEN_RE, fold_bt709_matrix,
+    NON_BT709_MATRICES, _sdr_bt709, validate_color
 )
 from _common.text import (
     ADVANCE_EM, BIDI_SCRIPTS, _char_em, char_script, default_font_file, detect_script, drawtext_boxborderw, drawtext_center_align,
@@ -178,7 +179,7 @@ __all__ = [
     "platform", "re", "shutil", "subprocess", "sys", "unicodedata",
     "aac_args", "add_common", "add_pad_fill_args", "ADVANCE_EM", "analyze_levels", "apply_common", "detect_scenes", "detect_silences", "SCORE_RE", "SIL_RE", "_aspect_string",
     "audio_codec_for", "AUDIO_CODECS", "BIDI_SCRIPTS", "_bit_depth", "brand_caption_style", "BRAND_DEFAULTS",
-    "brand_states_font", "_brief", "_BRIEF_DROP", "_brief_summary", "bt709_tag_args", "cfr_args", "_char_em",
+    "brand_states_font", "_brief", "_BRIEF_DROP", "_brief_summary", "bt709_matrix_args", "bt709_matrix_filter", "bt709_tag_args", "cfr_args", "_char_em",
     "char_script", "_check_existing_output", "refuse_existing_outputs", "_check_no_overwrite_input", "_check_output_path", "child_args",
     "child_limit", "_CHILDREN", "_cleanup_partial_output", "_cmdline", "CODECS", "color_hex", "_COLOR_TOKEN_RE",
     "concat_list_line", "Context", "_CURRENT_CTX", "db_to_linear", "decode_pcm_mono", "description_block", "_evidence_rank", "fmt_chapter_time", "propose_chapters",
@@ -196,12 +197,12 @@ __all__ = [
     "_EMOJI_TAIL", "encoder_args", "_ENCODERS", "_env_timeout", "ERROR_CODE", "ERROR_RETRYABLE", "escape_drawtext",
     "escape_filter_path", "EVEN_SCALE", "_execute", "_fail", "_family_rank", "FC_LANG", "_fc_list_fonts",
     "FC_UNKNOWN", "ffmpeg_base", "ffmpeg_encoders", "ffmpeg_exe", "_FFMPEG_VERSION", "ffmpeg_version", "fingerprint",
-    "flush_drawtext_textfiles", "fmt_secs", "fmt_smpte_time", "fmt_srt_time", "font_covers_script",
+    "flush_drawtext_textfiles", "fold_bt709_matrix", "fmt_secs", "fmt_smpte_time", "fmt_srt_time", "font_covers_script",
     "font_family_for_script", "font_family_of_file", "FONT_FLAG_HINT", "font_for_script", "FONT_INSTALL_HINT",
     "fonts_dir_covers_script", "_fraction", "has_emoji", "info", "INSTALL_HINTS", "install_signal_handlers",
     "is_audio_output", "_is_emoji_base", "_is_emoji_char", "_is_ffmpeg", "_is_mark", "_KEYCAP", "_KEYCAP_BASES",
     "keyframes_near", "classify_frame_timing", "measure_frame_timing", "LANGUAGE_NAMES", "LATIN_EM", "LEADING_VOWELS", "_libass_color_probe", "_limit_for",
-    "load_brand", "measured_level_dbfs", "AUDIO_MEDIA_EXT", "MEDIA_EXT", "MissingFpsError", "needs_shaping", "NO_SPACE_SCRIPTS",
+    "load_brand", "measured_level_dbfs", "AUDIO_MEDIA_EXT", "MEDIA_EXT", "MissingFpsError", "needs_shaping", "NO_SPACE_SCRIPTS", "NON_BT709_MATRICES",
     "_odd_dimension_retry", "_on_signal", "_output_failed", "_OutputLock", "pad_filters", "parse_time", "_pid_dead",
     "place_output", "_plan_at_exit", "_plan_inputs", "_PLAN_STRIP", "PLAN_VERSION", "PREFERRED_FAMILIES",
     "print_json", "probe", "PROBE_TIMEOUT", "_progress_line", "read_text_or_die", "refuse_output_is_input",
@@ -212,7 +213,7 @@ __all__ = [
     "SCRIPTS", "_sdr_bt709", "_set_current_ctx", "_SHAPING_BUILD_CACHE", "SHAPING_SCRIPTS", "shell_quote", "sibling_temp",
     "_SIGNALS_INSTALLED", "signed_time_arg", "_stage_existing_output", "STATE", "SVT_PRESET", "text_width_em",
     "time_arg", "_timed_out", "_to_float", "_to_int", "_unwatch", "validate_color", "verify_output",
-    "source_codec_video_args", "video_args", "_VS15", "_VS16", "_watch", "WINDOWS_FONTS", "write_plan", "x264_args", "X264_PRESETS",
+    "source_codec_video_args", "matrix_chain", "video_args", "_VS15", "_VS16", "_watch", "WINDOWS_FONTS", "write_plan", "x264_args", "X264_PRESETS",
     "_x264_raw", "_ZWJ",
     "ASR_ENGINES", "ASR_INSTALL_HINT", "_asr_run", "die_no_engine", "parse_srt", "transcribe",
     "_transcribe_in", "transcribe_words", "_words_from_openai_whisper_json",

package/scripts/_common/color.py

@@ -4,7 +4,7 @@ validation for the tools that take a `--color`.
 from __future__ import annotations
 
 import re
-from typing import List, Tuple
+from typing import Any, Dict, List, Optional, Sequence, Tuple
 from _common.emit import die
 from _common.runner import ffmpeg_version
 
@@ -18,9 +18,13 @@ def bt709_tag_args(encoder: str = "libx264") -> List[str]:
     screen recordings, many cameras) the CLI now auto-inserts a *real* matrix conversion (its
     guess for "unknown" is bt601) into every SDR re-encode: a --lut-strength 0 no-op grade
     came back ~24 dB PSNR from its source on 7.1. From 7.1 on, the tags therefore go through
-    the encoder's own VUI parameters instead, which libavfilter never sees; a source that is
-    genuinely tagged bt601/bt2020 is left alone either way (it keeps its own tags on the old
-    path, and the encoder VUI is a label, not a conversion, on the new one).
+    the encoder's own VUI parameters instead, which libavfilter never sees.
+
+    These tags are a label on every FFmpeg version: they never convert the pixels. A source
+    whose matrix is not BT.709 (a BT.601 `smpte170m`/`bt470bg` file) gets its pixels converted
+    separately, by bt709_matrix_filter() in the filter graph; through 2.10.0 nothing did, and
+    such a source came out labelled BT.709 over BT.601 pixels (measured on 6.1: 48.5 dB YUV
+    PSNR against the source, i.e. untouched, tagged bt709/bt709/bt709).
     """
     if ffmpeg_version() < (7, 1):
         return ["-colorspace", "bt709", "-color_primaries", "bt709", "-color_trc", "bt709"]
@@ -42,6 +46,101 @@ def _sdr_bt709(encoder: str) -> "Tuple[str, List[str]]":
     return "colorprim=bt709:transfer=bt709:colormatrix=bt709", []
 
 
+# The SDR matrices (ffprobe's names) whose pixels an output tagged BT.709 has to convert:
+# BT.601 (`smpte170m` 525-line, `bt470bg` 625-line: the same coefficients) and the two rare
+# pre-BT.709 ones the colorspace filter also reads. bt2020nc/c sources take the HDR/BT.2020 path.
+NON_BT709_MATRICES = ("smpte170m", "bt470bg", "fcc", "smpte240m")
+
+_MATRIX_FILTER_RE = re.compile(r"^colorspace=ispace=(?:%s):iprimaries=bt709:itrc=bt709:space=bt709:primaries=bt709:trc=bt709$"
+                               % "|".join(NON_BT709_MATRICES))
+_INPUT_VIDEO_REF = re.compile(r"\[(\d+):v(?::0)?\]")
+
+
+def bt709_matrix_filter(meta: Optional[Dict[str, Any]]) -> Optional[str]:
+    """The filter that turns the probed source's pixels into BT.709 ones before an output is
+    tagged BT.709, or None when nothing needs converting: an SDR source tagged with a non-BT.709
+    matrix (NON_BT709_MATRICES) only. An untagged or BT.709 source, an HDR/BT.2020 one (it keeps
+    its own tags, or color.py --to-sdr converts it) and a missing probe get None, so their
+    commands are what they were.
+
+    The matrix is converted and nothing else: primaries and transfer are passed through
+    (iprimaries/itrc = the output's), so the picture a player that decodes with the source's
+    matrix showed is the picture the BT.709 output shows. The `colorspace` filter is in every
+    FFmpeg build (no zscale needed) and takes explicit input properties, so the conversion does
+    not depend on how a build reads or negotiates the frames' own tags. The range is left as it is."""
+    v = (meta or {}).get("video") or {}
+    matrix = v.get("color_space")
+    if v.get("bt2020_or_hdr") or matrix not in NON_BT709_MATRICES:
+        return None
+    return f"colorspace=ispace={matrix}:iprimaries=bt709:itrc=bt709:space=bt709:primaries=bt709:trc=bt709"
+
+
+def bt709_matrix_args(meta: Optional[Dict[str, Any]]) -> List[str]:
+    """["-vf", bt709_matrix_filter(meta)] for an encoder line that tags its output BT.709, or [].
+
+    It rides on the encoder line (decision.video_args() and friends) so every tool that
+    re-encodes through that line converts without building it into its own graph. That -vf is
+    valid as it stands for a command with no other filter; run() folds it into the tool's own
+    -vf or -filter_complex (fold_bt709_matrix()) before the command is recorded or run."""
+    f = bt709_matrix_filter(meta)
+    return ["-vf", f] if f else []
+
+
+def fold_bt709_matrix(cmd: Sequence[str]) -> List[str]:
+    """`cmd` with the conversion bt709_matrix_args() appended to its encoder line moved into the
+    tool's own filter graph, where it runs first, on the decoded source:
+
+    - no other filter: unchanged (the -vf is the graph);
+    - a -vf: the conversion is prepended to that chain;
+    - a -filter_complex: every `[N:v]` / `[N:v:0]` reference to an input that reads the same file
+      as input 0 (the file the encoder line was built for; cut.py's segment join opens it once
+      per segment) reads that input through the conversion instead. Other inputs are untouched.
+
+    A -filter_complex with no such reference is refused rather than run with two graphs (ffmpeg
+    would refuse it anyway, less clearly). Every other command is returned as it is."""
+    out: List[str] = []
+    conv, at = None, -1
+    i = 0
+    while i < len(cmd):
+        if cmd[i] == "-vf" and i + 1 < len(cmd) and _MATRIX_FILTER_RE.match(str(cmd[i + 1])):
+            conv, at = conv or str(cmd[i + 1]), (at if at >= 0 else len(out))
+            i += 2
+            continue
+        out.append(cmd[i])
+        i += 1
+    if conv is None:
+        return list(cmd)
+    graph_at = next((k for k, a in enumerate(out[:-1]) if a in ("-filter_complex", "-lavfi")), None)
+    if graph_at is not None:
+        inputs = [out[k + 1] for k in range(len(out) - 1) if out[k] == "-i"]
+        source = {n for n, p in enumerate(inputs) if p == inputs[0]} if inputs else set()
+        refs: Dict[int, List[str]] = {}  # input index -> [its first reference, the labels standing in for it]
+
+        def through(m: "re.Match[str]") -> str:
+            n = int(m.group(1))
+            if n not in source:
+                return m.group(0)
+            label = f"[bt709src{sum(len(v) - 1 for v in refs.values())}]"
+            refs.setdefault(n, [m.group(0)]).append(label)
+            return label
+
+        graph = _INPUT_VIDEO_REF.sub(through, str(out[graph_at + 1]))
+        if not refs:
+            die("internal: a BT.709 matrix conversion was requested for a -filter_complex that never reads "
+                "input 0's video as [0:v]", kind="ffmpeg")
+        # one conversion per input, split when the graph reads that input more than once
+        # (speedramp.py trims [0:v] once per segment)
+        chains = [f"{ref}{conv}" + (f",split={len(labels)}" if len(labels) > 1 else "") + "".join(labels)
+                  for ref, *labels in refs.values()]
+        out[graph_at + 1] = ";".join(chains + [graph])
+        return out
+    vf_at = next((k for k, a in enumerate(out[:-1]) if a == "-vf"), None)
+    if vf_at is not None:
+        out[vf_at + 1] = f"{conv},{out[vf_at + 1]}"
+        return out
+    return out[:at] + ["-vf", conv] + out[at:]
+
+
 def color_hex(value: str) -> str:
     """Normalise '#ffd200' / 'ffd200' / '0xFFD200' to 'FFD200'."""
     v = str(value).strip().lstrip("#")

package/scripts/_common/decision.py

@@ -14,7 +14,7 @@ import argparse
 from fractions import Fraction
 from pathlib import Path
 from typing import Any, Dict, List, Optional, Sequence, Tuple
-from _common.color import bt709_tag_args, _sdr_bt709
+from _common.color import bt709_matrix_args, bt709_matrix_filter, bt709_tag_args, _sdr_bt709
 from _common.emit import die
 from _common.runner import CODECS, STATE, ffmpeg_encoders, ffmpeg_version, hw_platform_reason
 
@@ -497,30 +497,55 @@ def _x264_raw(crf: int, preset: str, keep_bt709: bool = True) -> List[str]:
     return args
 
 
-def x264_args(crf: int = 18, preset: str = "medium", keep_bt709: bool = True) -> List[str]:
+def x264_args(crf: int = 18, preset: str = "medium", keep_bt709: bool = True,
+              source: Optional[Dict[str, Any]] = None) -> List[str]:
     """SDR H.264 encoder args -- or, when --codec named another encoder, that encoder's SDR args
-    (color.py's --to-sdr path builds its own H.264 line; the flag still has to reach it)."""
+    (color.py's --to-sdr path builds its own H.264 line; the flag still has to reach it).
+
+    `source`: the probe of the file being re-encoded, when the pixels reaching the encoder are
+    still that file's. A BT.709-tagged line then carries the conversion of a non-BT.709 matrix
+    (bt709_matrix_args); a caller whose graph already outputs BT.709 (--to-sdr) leaves it out."""
     if STATE.codec and STATE.codec != "h264":
-        return encoder_args(STATE.codec, crf, preset, None, keep_bt709)
-    return _maybe_hw("h264", crf, None, keep_bt709, _x264_raw(crf, preset, keep_bt709))
+        args = encoder_args(STATE.codec, crf, preset, None, keep_bt709)
+    else:
+        args = _maybe_hw("h264", crf, None, keep_bt709, _x264_raw(crf, preset, keep_bt709))
+    return args + (bt709_matrix_args(source) if keep_bt709 and STATE.codec != "prores" else [])
 
 
-def video_args(meta: Optional[Dict[str, Any]], crf: int = 18, preset: str = "medium") -> List[str]:
+def video_args(meta: Optional[Dict[str, Any]], crf: int = 18, preset: str = "medium", convert_matrix: bool = True) -> List[str]:
     """Encoder args that preserve what the source is.
 
     SDR sources -> H.264 8-bit tagged BT.709 (x264_args). HDR sources (HDR10/PQ, HLG,
     Dolby Vision base layer, BT.2020) -> HEVC Main10 with the source's own colour tags,
     so cutting/captioning/fitting an iPhone HDR clip stays HDR instead of becoming a
     washed-out file mislabelled as BT.709. Use color.py --to-sdr when SDR is wanted.
+
+    An SDR source tagged with a BT.601 (or another non-BT.709) matrix is converted to BT.709
+    before it is tagged so: the line ends with the `-vf` of bt709_matrix_args(), which run()
+    folds into the tool's own graph (color.fold_bt709_matrix). `convert_matrix=False` is for a
+    tool that mixes several sources and converts each one in its own graph (join, multicam,
+    broll, overlay --video). --codec prores keeps the source's tags, so it converts nothing.
     """
-    if STATE.codec:
-        return encoder_args(STATE.codec, crf, preset, meta)
     v = (meta or {}).get("video") or {}
+    fix = bt709_matrix_args(meta) if convert_matrix and STATE.codec != "prores" else []
+    if STATE.codec:
+        return encoder_args(STATE.codec, crf, preset, meta) + fix
     if not v.get("bt2020_or_hdr"):
-        return x264_args(crf, preset)
+        return x264_args(crf, preset) + fix
     return _maybe_hw("hevc", crf, meta, True, _x265_hdr_default(v, crf, preset))
 
 
+def matrix_chain(meta: Optional[Dict[str, Any]], line_meta: Optional[Dict[str, Any]]) -> str:
+    """For a tool that mixes several sources in one graph and encodes it through
+    video_args(line_meta, convert_matrix=False): the conversion one source's own chain starts
+    with ("<filter>," -- bt709_matrix_filter), or "" when that source needs none or the line does
+    not tag BT.709 (an HDR/BT.2020 line keeps its tags, --codec prores keeps the source's)."""
+    if STATE.codec == "prores" or ((line_meta or {}).get("video") or {}).get("bt2020_or_hdr"):
+        return ""
+    f = bt709_matrix_filter(meta)
+    return f + "," if f else ""
+
+
 def source_codec_video_args(meta: Optional[Dict[str, Any]], crf: int = 18, preset: str = "medium",
                             keep_hevc: bool = False) -> List[str]:
     """video_args(), except that with `keep_hevc` (cut.py --keep-hevc) an SDR HEVC source re-encodes
@@ -530,7 +555,7 @@ def source_codec_video_args(meta: Optional[Dict[str, Any]], crf: int = 18, prese
     source always takes video_args()'s Main10 line, and --codec always wins."""
     v = (meta or {}).get("video") or {}
     if keep_hevc and not STATE.codec and v.get("codec") == "hevc" and not v.get("bt2020_or_hdr"):
-        return encoder_args("hevc", crf, preset, meta)
+        return encoder_args("hevc", crf, preset, meta) + bt709_matrix_args(meta)
     return video_args(meta, crf, preset)
 
 

package/scripts/_common/runner.py

@@ -878,6 +878,11 @@ def run(cmd: Sequence[str], *, quiet: bool = False, check: bool = True, ctx: "Op
     ctx = ctx or STATE
     is_ffmpeg = _is_ffmpeg(cmd)
     if is_ffmpeg:
+        # an encoder line that converts a BT.601 source to the BT.709 it is tagged with carries
+        # that conversion as a -vf; it joins the tool's own graph here, before anything records
+        # or runs the command (color.py imports this module, hence the late import)
+        from _common.color import fold_bt709_matrix
+        cmd = fold_bt709_matrix(cmd)
         _check_no_overwrite_input(cmd)
         _check_output_path(cmd)
         _check_existing_output(cmd)

package/scripts/broll.py

@@ -21,7 +21,7 @@ import argparse
 import sys
 from typing import Any, Dict, List
 
-from _common import STATE, add_common, aac_args, apply_common, cfr_args, default_output, die, emit, ffmpeg_base, info, time_arg, probe, run, validate_color, video_args, X264_PRESETS, fmt_secs
+from _common import STATE, add_common, aac_args, apply_common, cfr_args, default_output, die, emit, ffmpeg_base, info, time_arg, matrix_chain, probe, run, validate_color, video_args, X264_PRESETS, fmt_secs
 
 
 def main() -> int:
@@ -87,7 +87,7 @@ def main() -> int:
             die(f"cutaway {i + 1}: {path} has only {dur_b - start_b:.3f}s from {start_b:g}s, {length:g}s asked for")
         if cutaways and at < cutaways[-1]["at"] + cutaways[-1]["length"]:
             die(f"cutaway {i + 1} at {at:g}s overlaps the previous one (ends {cutaways[-1]['at'] + cutaways[-1]['length']:g}s)")
-        cutaways.append({"path": path, "at": at, "length": length, "from": start_b, "has_audio": bool(meta_b.get("audio"))})
+        cutaways.append({"path": path, "at": at, "length": length, "from": start_b, "has_audio": bool(meta_b.get("audio")), "meta": meta_b})
     if args.audio != "a" and not all(c["has_audio"] for c in cutaways):
         die("--audio b/mix needs audio on every B-roll clip")
 
@@ -101,10 +101,14 @@ def main() -> int:
     # A shows through unchanged, so A's timeline is never touched.
     parts: List[str] = []
     cur = "[0:v]"
+    # every clip is converted by its own colour tags to the BT.709 the output is tagged with
+    if matrix_chain(meta_a, meta_a):
+        parts.append(f"[0:v]{matrix_chain(meta_a, meta_a)}null[a709]")
+        cur = "[a709]"
     for i, c in enumerate(cutaways):
         geo = f"scale={w}:{h}:force_original_aspect_ratio=decrease,pad={w}:{h}:(ow-iw)/2:(oh-ih)/2:color={args.pad_color}"
         parts.append(f"[{i + 1}:v]trim=start={c['from']:.3f}:duration={c['length']:.3f},setpts=PTS-STARTPTS+{c['at']:.3f}/TB,"
-                     f"{geo},setsar=1,fps={fps:g},format={'yuv420p10le' if (meta_a.get('video') or {}).get('bt2020_or_hdr') else 'yuv420p'}[b{i}]")
+                     f"{matrix_chain(c['meta'], meta_a)}{geo},setsar=1,fps={fps:g},format={'yuv420p10le' if (meta_a.get('video') or {}).get('bt2020_or_hdr') else 'yuv420p'}[b{i}]")
         parts.append(f"{cur}[b{i}]overlay=0:0:eof_action=pass:enable='between(t,{c['at']:.3f},{c['at'] + c['length']:.3f})'[v{i}]")
         cur = f"[v{i}]"
     vout = cur
@@ -131,7 +135,7 @@ def main() -> int:
     cmd += ["-filter_complex", ";".join(parts), "-map", vout]
     if aout:
         cmd += ["-map", aout]
-    cmd += video_args(meta_a, args.crf, args.preset) + cfr_args(meta_a, None)
+    cmd += video_args(meta_a, args.crf, args.preset, convert_matrix=False) + cfr_args(meta_a, None)
     if aout == "0:a:0":
         cmd += ["-c:a", "copy"]
     elif aout:

package/scripts/check.py

@@ -175,10 +175,10 @@ def main() -> int:
                 # untagged 8-bit video is treated as BT.709 by every player and platform; nothing to fix
                 row("colour", "PASS", shown + " (players assume bt709)" if matrix else "untagged (players assume bt709)", "bt709/bt709 tagged (or HDR)")
             elif matrix in ("smpte170m", "bt470bg"):
-                # The pixels are BT.601-encoded. Nothing here converts the matrix: --retag bt709 and
-                # every SDR re-encode (bt709_tag_args) only relabel it, which shifts the colours.
+                # The pixels are BT.601-encoded. A re-encode here converts them to the BT.709 it tags
+                # (bt709_matrix_filter); --retag bt709 is a stream copy and only relabels the matrix.
                 row("colour", "WARN", shown, "bt709/bt709 tagged (or HDR)",
-                    "re-export from the master as BT.709; color.py --retag bt709 (like any re-encode here) only relabels the matrix: right only if the pixels really are BT.709",
+                    "export.py (any re-encode here converts the BT.601 matrix to BT.709); color.py --retag bt709 only relabels it: right only if the pixels really are BT.709",
                     reason="a player that honours the BT.601 tag and one that ignores it on HD video (decoding it as BT.709) show different colours")
             else:
                 row("colour", "WARN", shown, "bt709/bt709 tagged (or HDR)", "color.py --retag bt709 when the picture really is 709; color.py --to-sdr when it is HDR")

package/scripts/color.py

@@ -353,7 +353,9 @@ def main() -> int:
         tag = "lut"
 
     cmd = ffmpeg_base() + ["-i", args.input, "-vf", vf, "-map", "0:v:0", "-map", f"0:a:{args.audio_stream}?"]
-    cmd += x264_args(args.crf, args.preset) + cfr_args(meta) + (aac_args() if has_audio else [])
+    # --lut/--correct grade the source's own pixels, so a BT.601 source is converted to the BT.709
+    # the line is tagged with; --to-sdr's zscale chain already writes BT.709 and converts nothing more
+    cmd += x264_args(args.crf, args.preset, source=None if args.to_sdr else meta) + cfr_args(meta) + (aac_args() if has_audio else [])
     dropped_streams = run_keeping_subtitles(cmd, output)
     r = probe(output, role="output")
     info(f"wrote {output} ({fmt_secs(r['duration'])}, {r['video']['width']}x{r['video']['height']}, "

package/scripts/export.py

@@ -36,7 +36,7 @@ import sys
 from pathlib import Path
 from typing import Dict, List
 
-from _common import STATE, add_common, apply_common, bt709_tag_args, child_args, emit, cfr_args, default_output, die, encoder_args, ffmpeg_base, info, probe, run, run_tool, validate_color, pad_filters, add_pad_fill_args, fmt_secs
+from _common import STATE, add_common, apply_common, bt709_matrix_args, bt709_tag_args, child_args, emit, cfr_args, default_output, die, encoder_args, ffmpeg_base, info, probe, run, run_tool, validate_color, pad_filters, add_pad_fill_args, fmt_secs
 from _common.decision import hw_preset_video, restate_last_swap
 from check import SPECS as PLATFORMS, measure_loudness
 from _platforms import ALIASES as _ALIASES, PLATFORMS as PLATFORM_TABLE, resolve as resolve_platform
@@ -216,6 +216,11 @@ def main() -> int:
             cmd += cfr_args(meta)
         if tag_bt709 and not video[video.index("-c:v") + 1].endswith("_videotoolbox"):
             cmd += bt709_tag_args(video[video.index("-c:v") + 1])
+        if args.preset != "prores":
+            # every line but the prores master's says BT.709 (the fixed presets' tags above, an
+            # SDR hevc/av1 line's own): a BT.601 source's pixels are converted to match, in front
+            # of the -vf above (run() folds the two); an untagged, BT.709 or HDR source adds nothing
+            cmd += bt709_matrix_args(meta)
     if out_ext == "mp4":
         cmd += ["-movflags", "+faststart"]
     cmd += (p["audio"] if has_audio else ["-an"])

package/scripts/join.py

@@ -39,7 +39,7 @@ import os
 import sys
 from typing import Any, Dict, List, Optional, Tuple
 
-from _common import STATE, AUDIO_MEDIA_EXT, video_args, aac_args, add_common, apply_common, audio_codec_for, default_output, die, dry_run_input_pending, emit, ffmpeg_base, info, measured_level_dbfs, is_audio_output, probe, require_tool, run, validate_color, X264_PRESETS, fmt_secs
+from _common import STATE, AUDIO_MEDIA_EXT, matrix_chain, video_args, aac_args, add_common, apply_common, audio_codec_for, default_output, die, dry_run_input_pending, emit, ffmpeg_base, info, measured_level_dbfs, is_audio_output, probe, require_tool, run, validate_color, X264_PRESETS, fmt_secs
 
 TRANSITIONS = ["fade", "dissolve", "wipeleft", "wiperight", "wipeup", "wipedown", "slideleft", "slideright",
                "circleopen", "circleclose", "fadeblack", "fadewhite", "smoothleft", "smoothright", "radial", "none"]
@@ -497,7 +497,8 @@ def main() -> int:
     if len(set(chans)) > 1:
         info(f"channel counts differ ({', '.join(str(c) for c in chans)}); every clip becomes {layout}")
     for i in range(n):
-        parts.append(f"[{i}:v]{geo},setsar=1,fps={fps:g},format={pixfmt},settb=AVTB[v{i}]")
+        # a BT.601 clip is converted to the BT.709 the join is tagged with, each clip by its own tags
+        parts.append(f"[{i}:v]{matrix_chain(metas[i], hdr_meta or metas[0])}{geo},setsar=1,fps={fps:g},format={pixfmt},settb=AVTB[v{i}]")
         parts.append(f"[{audio_src[i]}]aformat=sample_rates=48000:channel_layouts={layout},asetpts=PTS-STARTPTS[a{i}]")
 
     # pad the shorter stream to the clip's length: the audio with silence, the picture by holding
@@ -541,7 +542,7 @@ def main() -> int:
     # to meet the AAC tail (a 51-frame dissolve came out 52), which newer builds no longer do -- pass
     # the frames through as built, for a plain cut as much as for a transition
     cmd += ["-fps_mode", "passthrough"]
-    cmd += video_args(hdr_meta or metas[0], args.crf, args.preset) + aac_args() + [output]
+    cmd += video_args(hdr_meta or metas[0], args.crf, args.preset, convert_matrix=False) + aac_args() + [output]
     run(cmd)
     expected = unpending_length(lens, d)
     r = probe(output, role="output")

package/scripts/multicam.py

@@ -31,7 +31,7 @@ import math
 import sys
 from typing import Dict, List, Tuple
 
-from _common import video_args, aac_args, add_common, apply_common, default_output, die, emit, ffmpeg_base, info, time_arg, probe, run, x264_args, X264_PRESETS, fmt_secs, decode_pcm_mono, rms_envelope, STATE
+from _common import matrix_chain, video_args, aac_args, add_common, apply_common, default_output, die, emit, ffmpeg_base, info, time_arg, probe, run, x264_args, X264_PRESETS, fmt_secs, decode_pcm_mono, rms_envelope, STATE
 from sync import measure_offset
 
 ENERGY_WINDOW_S = 0.25  # --switch energy: loudness measured over this window on the reference timeline
@@ -276,7 +276,8 @@ def main() -> int:
         if src_s < 0:
             info(f"warning: camera {c} has not started at reference {s:.2f}s; using camera 0 for that range")
             c, src_s, src_e = 0, s, e
-        parts.append(f"[{c}:v]trim=start={src_s:.4f}:end={src_e:.4f},setpts=PTS-STARTPTS,{geo}[v{i}]")
+        # each camera converted by its own colour tags to the BT.709 the cut is tagged with
+        parts.append(f"[{c}:v]trim=start={src_s:.4f}:end={src_e:.4f},setpts=PTS-STARTPTS,{matrix_chain(metas[c], metas[0])}{geo}[v{i}]")
         labels.append(f"[v{i}]")
         project_clips.append({"src": args.inputs[c], "in": round(src_s, 3), "out": round(src_e, 3)})
     parts.append("".join(labels) + f"concat=n={len(filled)}:v=1:a=0[vout]")
@@ -297,7 +298,7 @@ def main() -> int:
 
     output = args.output or default_output(args.inputs[0], "multicam", "mp4")
     cmd += ["-filter_complex", ";".join(parts), "-map", "[vout]", "-map", "[aout]"]
-    cmd += video_args(metas[0], args.crf, args.preset) + aac_args() + ["-t", f"{ref_dur:.3f}", output]
+    cmd += video_args(metas[0], args.crf, args.preset, convert_matrix=False) + aac_args() + ["-t", f"{ref_dur:.3f}", output]
 
     if args.write_project:
         # Same cut decision as the combined render above (project_clips was built alongside the

package/scripts/overlay.py

@@ -25,7 +25,7 @@ import sys
 from typing import List, Optional
 
 from _platforms import PLATFORMS, PLATFORM_CHOICES, safe_margins_px, resolve as resolve_platform
-from _common import STATE, script_font_for_text, drawtext_text_opts, needs_shaping, LANGUAGE_NAMES, has_emoji, detect_script, emoji_clusters, load_brand, video_args, add_common, apply_common, default_font_file, emit, aac_args, cfr_args, default_output, die, escape_drawtext, escape_filter_path, ffmpeg_base, info, parse_time, probe, run, run_keeping_subtitles, validate_color, x264_args, X264_PRESETS, time_arg, fmt_secs
+from _common import STATE, script_font_for_text, drawtext_text_opts, needs_shaping, LANGUAGE_NAMES, has_emoji, detect_script, emoji_clusters, load_brand, matrix_chain, video_args, add_common, apply_common, default_font_file, emit, aac_args, cfr_args, default_output, die, escape_drawtext, escape_filter_path, ffmpeg_base, info, parse_time, probe, run, run_keeping_subtitles, validate_color, x264_args, X264_PRESETS, time_arg, fmt_secs
 
 # Per-edge margins: a platform's UI does not cover the same fraction of every edge (TikTok's
 # like column is 14 % of the width, its description block 22 % of the height), so a position
@@ -299,7 +299,10 @@ def main() -> int:
         if enable:
             ov += f":enable='{enable}'"
         cmd = ffmpeg_base() + ["-i", args.input, "-i", args.video]
-        fc = f"[1:v]{','.join(chain)}[ov];[0:v][ov]{ov}[out]"
+        # both pictures converted by their own colour tags to the BT.709 the output is tagged with
+        base = matrix_chain(meta, meta)
+        fc = (f"[1:v]{matrix_chain(pip_meta, meta)}{','.join(chain)}[ov];"
+              + (f"[0:v]{base}null[base];[base]" if base else "[0:v]") + f"[ov]{ov}[out]")
         cmd += ["-filter_complex", fc, "-map", "[out]", "-map", f"0:a:{args.audio_stream}?"]
         if meta.get("duration"):
             # See the --image branch above: -t (exact, bounds only the main input) instead of
@@ -329,7 +332,7 @@ def main() -> int:
         if meta.get("audio"):
             cmd += ["-map", f"0:a:{args.audio_stream}"]
 
-    cmd += video_args(meta, args.crf, args.preset) + cfr_args(meta)
+    cmd += video_args(meta, args.crf, args.preset, convert_matrix=not args.video) + cfr_args(meta)
     cmd += aac_args() if meta.get("audio") else ["-an"]
     dropped_streams = run_keeping_subtitles(cmd, output)
     if not STATE.dry_run: