[build] Finalize release test workflow

Author: Breakthrough

.github/workflows/release-test.yml

@@ -21,7 +21,9 @@ jobs:
           pip install build twine pip-audit
       - name: Version consistency check
         run: |
-          VERSION=$(python -c "import scenedetect; print(scenedetect.__version__)")
+          # Parse __version__ directly so we don't have to install scenedetect
+          # (importing it triggers a cv2-availability guard).
+          VERSION=$(python -c "import ast,pathlib; print(next(n.value.value for n in ast.parse(pathlib.Path('scenedetect/__init__.py').read_text()).body if isinstance(n, ast.Assign) and any(getattr(t,'id',None)=='__version__' for t in n.targets)))")
           echo "scenedetect.__version__ = $VERSION"
           if [[ "${{ github.ref }}" == refs/tags/* ]]; then
             TAG_VERSION=${GITHUB_REF#refs/tags/v}
@@ -37,10 +39,16 @@ jobs:
           fi
       - name: Build and Check
         run: |
-          python -m build
-          twine check dist/*
+          # Build to a clean output dir - `dist/` already holds tracked installer
+          # scripts and config (e.g. dist/generate_assets.py, dist/installer/),
+          # so `twine check dist/*` would try to validate non-distribution files.
+          python -m build --outdir build-dist
+          twine check build-dist/*
       - name: pip-audit
-        run: pip-audit
+        # CVE-2026-3219 in pip 26.0.1 has no fix version available upstream
+        # and pip ships pre-installed on the runner (not controlled by this
+        # project). Re-evaluate when pip publishes a fix.
+        run: pip-audit --ignore-vuln CVE-2026-3219
 
   release-tests:
     needs: static
@@ -68,7 +76,7 @@ jobs:
           pip install .[opencv,pyav,moviepy]
           pip install opentimelineio pillow psutil pytest
       - name: Run release tests
-        run: pytest -m release -vv --ignore=tests/release/test_long_video_stress.py --ignore=tests/release/test_install_matrix.py
+        run: pytest -m release -vv --ignore=tests/release/test_long_video_stress.py
 
   long-stress:
     needs: static
@@ -93,52 +101,3 @@ jobs:
           pip install psutil pytest
       - name: Run long stress test
         run: pytest -m release -k long_video -vv
-
-  install-matrix:
-    needs: static
-    strategy:
-      matrix:
-        os: [ubuntu-latest, windows-latest]
-    runs-on: ${{ matrix.os }}
-    steps:
-      - uses: actions/checkout@v4
-      - name: Checkout resources branch
-        run: |
-          git fetch --depth=1 origin refs/heads/resources:refs/remotes/origin/resources
-          git checkout refs/remotes/origin/resources -- tests/resources/
-          git reset
-      - name: Set up Python
-        uses: actions/setup-python@v5
-        with:
-          python-version: '3.10'
-      - name: Build wheel
-        run: |
-          pip install build
-          python -m build --wheel
-      - name: Test Bare Install
-        shell: bash
-        run: |
-          WHEEL=$(ls dist/*.whl)
-          PY=${{ matrix.os == 'windows-latest' && 'venv_bare/Scripts/python' || 'venv_bare/bin/python' }}
-          PYTEST=${{ matrix.os == 'windows-latest' && 'venv_bare/Scripts/pytest' || 'venv_bare/bin/pytest' }}
-          python -m venv venv_bare
-          $PY -m pip install pytest "$WHEEL"
-          $PYTEST -m release -k test_install_bare
-      - name: Test OpenCV Install
-        shell: bash
-        run: |
-          WHEEL=$(ls dist/*.whl)
-          PY=${{ matrix.os == 'windows-latest' && 'venv_opencv/Scripts/python' || 'venv_opencv/bin/python' }}
-          PYTEST=${{ matrix.os == 'windows-latest' && 'venv_opencv/Scripts/pytest' || 'venv_opencv/bin/pytest' }}
-          python -m venv venv_opencv
-          $PY -m pip install pytest "${WHEEL}[opencv]"
-          $PYTEST -m release -k test_opencv_only
-      - name: Test PyAV Install
-        shell: bash
-        run: |
-          WHEEL=$(ls dist/*.whl)
-          PY=${{ matrix.os == 'windows-latest' && 'venv_pyav/Scripts/python' || 'venv_pyav/bin/python' }}
-          PYTEST=${{ matrix.os == 'windows-latest' && 'venv_pyav/Scripts/pytest' || 'venv_pyav/bin/pytest' }}
-          python -m venv venv_pyav
-          $PY -m pip install pytest "${WHEEL}[pyav]"
-          $PYTEST -m release -k test_pyav_only

RELEASE-PLAN.md

@@ -1,7 +1,7 @@
 # PySceneDetect Release Checklist
 
 Use one copy per release (e.g. tick the boxes in a tracking issue or draft PR).
-Version referenced below as `X.Y[.Z]` — replace with the real version throughout.
+Version referenced below as `X.Y[.Z]` - replace with the real version throughout.
 
 ## 0. Branch setup
 
@@ -11,7 +11,7 @@ Version referenced below as `X.Y[.Z]` — replace with the real version througho
 ## 1. Code & version
 
 - [ ] Bump `__version__` in `scenedetect/__init__.py`.
-- [ ] Bump `ProductVersion` in `dist/installer/PySceneDetect.aip` (must match `__version__` — `dist/pre_release.py --release` asserts this).
+- [ ] Bump `ProductVersion` in `dist/installer/PySceneDetect.aip` (must match `__version__` - `dist/pre_release.py --release` asserts this).
 - [ ] No `-dev` / pre-release suffix on the version string for a final release.
 
 > **Note:** `setup.cfg` reads the package version dynamically via `version = attr: scenedetect.__version__`, and `pyproject.toml` does not declare a `version` field. The single source of truth is `scenedetect/__init__.py`; the `.aip` is the only other place to keep in sync.
@@ -33,22 +33,22 @@ Version referenced below as `X.Y[.Z]` — replace with the real version througho
 
 - [ ] Unit tests green locally and in CI: `pytest -vv` (should collect `-m 'not release'` by default).
 - [ ] `ruff check scenedetect/ tests/` and `ruff format --check scenedetect/ tests/` pass.
-- [ ] Release test suite green: tag a disposable `vX.Y.Z-release-rc` or use `workflow_dispatch` on `.github/workflows/release-test.yml` — all 4 jobs (`static`, `release-tests`, `install-matrix`, `long-stress`) green across the 3-OS × 2-Python matrix. See `RELEASE-TEST-PLAN.md` for what the suite covers.
+- [ ] Release test suite green: tag a disposable `vX.Y.Z-release-rc` or use `workflow_dispatch` on `.github/workflows/release-test.yml` - all 4 jobs (`static`, `release-tests`, `install-matrix`, `long-stress`) green across the 3-OS × 2-Python matrix. See `RELEASE-TEST-PLAN.md` for what the suite covers.
 - [ ] `resources` branch has the artifacts the release tests need (goldens under `tests/resources/goldens/`, `tests/resources/stress_15min.mp4`). Re-push if any golden was regenerated.
 - [ ] Manual smoke: fresh venv, `pip install .` then `pip install .[opencv]` then `pip install .[pyav]`; run `scenedetect -i <video> detect-content list-scenes save-images` and eyeball the output.
 - [ ] `pip-audit` clean (or exceptions documented in the changelog).
 
 ## 5. Windows installer
 
 - [ ] `python dist/pre_release.py --release` passes (enforces `.aip` ↔ `__version__` parity, writes `dist/.version_info`).
-- [ ] `pyinstaller dist/scenedetect.spec` produces a working `scenedetect.exe` — run it against a sample video.
+- [ ] `pyinstaller dist/scenedetect.spec` produces a working `scenedetect.exe` - run it against a sample video.
 - [ ] Build the MSI via Advanced Installer (`dist/installer/PySceneDetect.aip`); install into a clean Windows VM and run the CLI.
 
 ## 6. Cut the release
 
 - [ ] Final commit on `releases/X.Y`: "Release vX.Y[.Z]".
-- [ ] Tag `vX.Y[.Z]-release` on that commit and push — this fires `release-test.yml`. Wait for all jobs green.
-- [ ] Merge `releases/X.Y` into `main` (fast-forward or merge commit — keep history clean).
+- [ ] Tag `vX.Y[.Z]-release` on that commit and push - this fires `release-test.yml`. Wait for all jobs green.
+- [ ] Merge `releases/X.Y` into `main` (fast-forward or merge commit - keep history clean).
 - [ ] Tag the final release `vX.Y[.Z]` on the merged commit and push.
 
 ## 7. Publish
@@ -72,4 +72,4 @@ Version referenced below as `X.Y[.Z]` — replace with the real version througho
 
 - **Branching model**: work spans multiple commits on `releases/X.Y`; the final one gets the `vX.Y[.Z]-release` tag which gates the release-test workflow. A passing release-test is a hard prerequisite for publishing.
 - **Version consistency** is enforced in two places (`__init__.py`, `PySceneDetect.aip`). The `static` job of `release-test.yml` checks `__init__.py` against the tag and verifies the changelog has a matching `## PySceneDetect X.Y` heading; the installer parity is checked by `dist/pre_release.py --release`.
-- **Changelog convention**: the in-development section lives at the *bottom* of `website/pages/changelog.md` under the "Development" heading — don't move it to the top.
+- **Changelog convention**: the in-development section lives at the *bottom* of `website/pages/changelog.md` under the "Development" heading - don't move it to the top.

docs/api/migration_guide.rst

@@ -169,7 +169,7 @@ The ``StatsManager`` methods ``get_metrics()``, ``set_metrics()``, and ``metrics
 ``SceneDetector`` Annotation Fixes
 =======================================================================
 
-``SceneDetector.post_process()`` now declares its parameter as ``timecode: FrameTimecode`` (previously typed as ``int``). The method already received a ``FrameTimecode`` at runtime and concrete detectors (e.g. ``ThresholdDetector``, ``ContentDetector``) already used the ``FrameTimecode`` type — only the abstract-base-class annotation was inconsistent. No call-site changes are needed; this just brings the signature into agreement with the documented and actual behavior.
+``SceneDetector.post_process()`` now declares its parameter as ``timecode: FrameTimecode`` (previously typed as ``int``). The method already received a ``FrameTimecode`` at runtime and concrete detectors (e.g. ``ThresholdDetector``, ``ContentDetector``) already used the ``FrameTimecode`` type - only the abstract-base-class annotation was inconsistent. No call-site changes are needed; this just brings the signature into agreement with the documented and actual behavior.
 
 
 =======================================================================

scenedetect/detectors/threshold_detector.py

@@ -146,7 +146,7 @@ def process_frame(
                 if (timecode - self.last_scene_cut) >= self.min_scene_len:
                     # Just faded into a new scene, compute timecode for the scene
                     # split based on the fade bias. Use frame-number arithmetic so the
-                    # result is identical across backends — float seconds + framerate
+                    # result is identical across backends - float seconds + framerate
                     # multiplication can land on a .5 rounding boundary and tip the
                     # frame number by 1 between PyAV (sub-microsecond PTS) and OpenCV
                     # (millisecond-truncated CAP_PROP_POS_MSEC).

scenedetect/output/__init__.py

@@ -456,7 +456,7 @@ def write_scene_list_fcp7(
         video_name: Display name used for project and sequence. Defaults to the stem of
             `video_path`.
         source_duration: Total duration of the source media. Required on ``<file>`` so NLEs
-            (DaVinci Resolve, Premiere) can seek into the source — without it the clip plays
+            (DaVinci Resolve, Premiere) can seek into the source - without it the clip plays
             frozen. If None, falls back to the last scene's end time.
     """
     assert scene_list

scripts/update_copyright.py

@@ -47,7 +47,7 @@
 }
 
 # Foreign-licensed files we never modify. The _thirdparty package's own
-# __init__.py is excluded from this list — it carries the project header.
+# __init__.py is excluded from this list - it carries the project header.
 SKIP_PATHS = {
     "scenedetect/_thirdparty/simpletable.py",  # vendored from third party
 }

tests/helpers.py

@@ -23,8 +23,8 @@ def invoke_cli(args: list[str], catch_exceptions: bool = False) -> tuple[int, st
 
     Replicates the two-step execution of ``__main__.py``:
 
-    1. ``scenedetect.main(obj=context)`` — parse args and register callbacks on ``CliContext``
-    2. ``run_scenedetect(context)`` — execute detection and output commands
+    1. ``scenedetect.main(obj=context)`` - parse args and register callbacks on ``CliContext``
+    2. ``run_scenedetect(context)`` - execute detection and output commands
 
     Returns ``(exit_code, output_text)``.
     """

tests/release/synthetic.py

@@ -11,8 +11,7 @@
 #
 """Synthetic Video Generation
 
-This module provides functions to generate pathological or specific video files
-using ffmpeg for testing purposes.
+Functions to generate synthetic video files using ffmpeg for testing purposes.
 """
 
 import subprocess

tests/release/test_cross_backend.py tests/release/test_backends.pytests/release/test_cross_backend.py renamed to tests/release/test_backends.py

@@ -9,10 +9,9 @@
 # PySceneDetect is licensed under the BSD 3-Clause License; see the
 # included LICENSE file, or visit one of the above pages for details.
 #
-"""Category 2: Cross-Backend Consistency
+"""Backend Consistency
 
-Verifies that all available backends produce consistent cut lists for both CFR
-and VFR videos.
+Verifies that all available backends produce consistent cut lists for both CFR and VFR videos.
 """
 
 import importlib.util
@@ -50,7 +49,7 @@ def test_cross_backend_consistency(rel_path, is_vfr):
 
     backends = _installed_backends()
     if is_vfr and "moviepy" in backends:
-        # MoviePy does not honor per-frame PTS on VFR video — tracked separately
+        # MoviePy does not honor per-frame PTS on VFR video - tracked separately
         # from the OpenCV/PyAV VFR path that this test gates.
         backends = [b for b in backends if b != "moviepy"]
     if len(backends) < 2:

tests/release/test_cli_permutations.py

@@ -9,14 +9,9 @@
 # PySceneDetect is licensed under the BSD 3-Clause License; see the
 # included LICENSE file, or visit one of the above pages for details.
 #
-"""Category 7: CLI Permutation Smoke Tests.
-
-Exercises CLI command chains via subprocess. We use subprocess (not Click's
-CliRunner) because the CLI's argument-parsing pass and its controller run are
-separate calls in ``scenedetect.__main__`` — ``CliRunner`` only drives the
-first pass and does not execute the controller. The TODO at
-``tests/test_cli.py:19`` tracks a future refactor that would make CliRunner
-viable.
+"""CLI Permutation Smoke Tests
+
+Exercises CLI command chains via subprocess.
 """
 
 import os