fix(mixkit): don't guess JSON-LD clip dimensions from the filename

The JSON-LD fallback reported every "-1080" contentUrl as 1920x1080 and
everything else as 1280x720, so portrait clips surfaced as landscape.
Use the VideoObject's declared width/height (number, string, or
QuantitativeValue) and leave them 0 (unknown) when absent, matching the
card path.

Adds portrait and missing-dimension JSON-LD tests.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Author: RhinoSandwitch

tests/tools/test_stock_source_adapters.py

@@ -251,7 +251,7 @@ def fake_get(url, headers=None, params=None, timeout=None):
     assert (results[0].width, results[0].height) == (1920, 1080)
 
 
-def test_mixkit_falls_back_to_json_ld_video_objects(monkeypatch):
+def _mixkit_json_ld_search(monkeypatch, extra_fields=""):
     # Mixkit's current markup has no result cards, only schema.org JSON-LD.
     pytest.importorskip("bs4")
     html = """<html><head><script type="application/ld+json">
@@ -263,18 +263,39 @@ def test_mixkit_falls_back_to_json_ld_video_objects(monkeypatch):
        "description": "Round candle lit illuminating while it goes out.",
        "thumbnailUrl": "https://assets.mixkit.co/videos/3458/3458-thumb-720-0.jpg",
        "contentUrl": "https://assets.mixkit.co/videos/3458/3458-1080.mp4",
+       %s
        "embedUrl": "https://assets.mixkit.co/videos/3458/3458-360.mp4"}
-    ]}</script></head><body></body></html>"""
+    ]}</script></head><body></body></html>""" % extra_fields
     requests_stub = types.ModuleType("requests")
     requests_stub.get = lambda *_a, **_k: _JsonResponse(text=html)
     monkeypatch.setitem(sys.modules, "requests", requests_stub)
+    return get_source("mixkit").search("candle", SearchFilters(kind="video", per_page=5))
+
 
-    results = get_source("mixkit").search("candle", SearchFilters(kind="video", per_page=5))
+def test_mixkit_falls_back_to_json_ld_video_objects(monkeypatch):
+    results = _mixkit_json_ld_search(monkeypatch)
 
     assert len(results) == 1
     clip = results[0]
     assert clip.source_id == "mixkit_3458"
     assert clip.download_url == "https://assets.mixkit.co/videos/3458/3458-1080.mp4"
     assert clip.extra["detail_url"] == clip.download_url
-    assert (clip.width, clip.height) == (1920, 1080)
     assert "Candle lighting in the dark" in clip.source_tags
+    # No declared size: stay unknown rather than guessing from the "-1080" suffix.
+    assert (clip.width, clip.height) == (0, 0)
+
+
+@pytest.mark.parametrize(
+    "fields",
+    [
+        '"width": 1080, "height": 1920,',
+        '"width": "1080", "height": "1920 px",',
+        '"width": {"@type": "QuantitativeValue", "value": 1080},'
+        ' "height": {"@type": "QuantitativeValue", "value": 1920},',
+    ],
+    ids=["numbers", "strings", "quantitative-value"],
+)
+def test_mixkit_json_ld_keeps_declared_portrait_dimensions(monkeypatch, fields):
+    (clip,) = _mixkit_json_ld_search(monkeypatch, fields)
+
+    assert (clip.width, clip.height) == (1080, 1920)

tools/video/stock_sources/mixkit.py

@@ -29,6 +29,20 @@
 _LICENSE = "Mixkit License (free for commercial and personal use, no attribution required)"
 
 
+def _ld_dimension(value: Any) -> int:
+    """Pixel size from a schema.org width/height (number, "1920", "1920 px",
+    or a QuantitativeValue). Returns 0 when absent or unparseable."""
+    if isinstance(value, dict):
+        value = value.get("value")
+    if isinstance(value, (int, float)):
+        return max(0, int(value))
+    if isinstance(value, str):
+        match = re.match(r"\s*(\d+)", value)
+        if match:
+            return int(match.group(1))
+    return 0
+
+
 class MixkitSource:
     """Mixkit video adapter. Satisfies `StockSource`."""
 
@@ -164,16 +178,17 @@ def _parse_json_ld(self, soup, query: str, filters: SearchFilters) -> list[Candi
                     continue
                 href = (node.get("@id") or "").split("#", 1)[0]
                 clip_id = href.rstrip("/").rsplit("-", 1)[-1] if href else content_url
-                hd = "-1080." in content_url
                 out.append(
                     Candidate(
                         source=self.name,
                         source_id=f"mixkit_{clip_id}",
                         source_url=href or content_url,
                         download_url=content_url,
                         kind="video",
-                        width=1920 if hd else 1280,
-                        height=1080 if hd else 720,
+                        # Only trust declared dimensions; the "-1080" filename suffix
+                        # says nothing about orientation. 0 = unknown until probed.
+                        width=_ld_dimension(node.get("width")),
+                        height=_ld_dimension(node.get("height")),
                         duration=0.0,
                         creator="Mixkit",
                         license=_LICENSE,