kryten-webqueue 0.45.0__tar.gz → 0.45.1__tar.gz

{kryten_webqueue-0.45.0 → kryten_webqueue-0.45.1}/CHANGELOG.md

@@ -1,5 +1,18 @@
 # Changelog
 
+## [0.45.1] - 2026-09-10
+
+### Fixed
+
+- **MOTD mystery boxes now use the branded placeholder art** the catalog browse
+  view already draws from (`placeholder_dir`, served under `/images`), picking a
+  different tape per slot the way browse varies it per tile. The 0.45.0 default
+  pointed at a `motd_boxes/mystery-box.jpg` that doesn't exist, so every
+  unresolved slot would have rendered a broken image. New
+  `motd.mystery_box_base_url` absolutizes those URLs for CyTube, which renders
+  the MOTD off-site; `motd.mystery_box_url` is now an empty-by-default fallback
+  used only when no branded placeholders are installed.
+
 ## [0.45.0] - 2026-09-10
 
 ### Added

{kryten_webqueue-0.45.0 → kryten_webqueue-0.45.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: kryten-webqueue
-Version: 0.45.0
+Version: 0.45.1
 Summary: Netflix/Tubi-style catalog browser and pay-to-play queue management for CyTube
 Author: grobertson
 License-Expression: MIT

{kryten_webqueue-0.45.0 → kryten_webqueue-0.45.1}/config.example.json

@@ -95,7 +95,8 @@
     "banner_url": "https://i.postimg.cc/jdr2mR8Y/1562479186-8-channel-Ztitlenew.png",
     "headline": "CHANNEL Z WEEKEND MOVIE PLAYLIST ({dates})",
     "showtime": "6pm ET",
-    "mystery_box_url": "https://www.dropsugar.co/static/motd_boxes/mystery-box.jpg",
+    "mystery_box_base_url": "https://queue.dropsugar.co",
+    "mystery_box_url": "",
     "mystery_box_href": "https://queue.dropsugar.co/",
     "upload_max_bytes": 5242880,
     "links": [

{kryten_webqueue-0.45.0 → kryten_webqueue-0.45.1}/kryten_webqueue/config.py

@@ -47,7 +47,12 @@ class MOTDConfig(BaseModel):
     # {dates} is substituted with the weekend showtimes (e.g. "8/14 @ 6pm ET & ...").
     headline: str = "CHANNEL Z WEEKEND MOVIE PLAYLIST ({dates})"
     showtime: str = "6pm ET"
-    mystery_box_url: str = "https://www.dropsugar.co/static/motd_boxes/mystery-box.jpg"
+    # Mystery boxes are drawn from the same branded placeholder pool the browse
+    # view uses (served from the webqueue /images mount); this absolutizes them
+    # for CyTube, which renders the MOTD off-site.
+    mystery_box_base_url: str = "https://queue.dropsugar.co"
+    # Single fallback used only when no branded placeholders are installed.
+    mystery_box_url: str = ""
     mystery_box_href: str = "https://queue.dropsugar.co/"
     links: list[MOTDLink] = Field(
         default_factory=lambda: [

{kryten_webqueue-0.45.0 → kryten_webqueue-0.45.1}/kryten_webqueue/jobs/motd_publish.py

@@ -17,7 +17,7 @@ import datetime
 import logging
 from pathlib import Path
 
-from ..motd.builder import build_slots
+from ..motd.builder import build_slots, mystery_pool
 from ..motd.render import render_motd
 from .manager import JobError
 
@@ -140,6 +140,7 @@ async def motd_publish_job(params: dict, ctx) -> dict:
             refresh_art=refresh_art,
             dry_run=dry_run,
             week_offset=week_offset,
+            mystery_urls=mystery_pool(ctx.config, getattr(ctx, "cover_art", None)),
             emit=_emit,
         )
     except RuntimeError as exc:

{kryten_webqueue-0.45.0 → kryten_webqueue-0.45.1}/kryten_webqueue/motd/builder.py

@@ -13,6 +13,7 @@ from __future__ import annotations
 
 import datetime
 import logging
+import random
 import re
 from dataclasses import dataclass, field
 from pathlib import Path
@@ -149,9 +150,28 @@ def _titles_by_night(movies_by_section: dict[str, list[str]]) -> dict[int, list[
     return by_night
 
 
-def _apply_mystery(slot: MOTDSlot, motd_cfg) -> None:
+def mystery_pool(config, cover_art) -> list[str]:
+    """Absolute URLs for the branded placeholder art the browse view uses.
+
+    ``list_placeholder_urls`` returns site-relative ``/images/...`` paths; CyTube
+    renders the MOTD off-site, so they have to be absolutized.
+    """
+    if cover_art is None:
+        return []
+    base = str(getattr(config.motd, "mystery_box_base_url", "") or "").rstrip("/")
+    if not base:
+        return []
+    return [base + url for url in cover_art.list_placeholder_urls()]
+
+
+def _apply_mystery(slot: MOTDSlot, motd_cfg, pool: list[str]) -> None:
     slot.source = "mystery"
-    slot.poster_url = getattr(motd_cfg, "mystery_box_url", "") or ""
+    # Vary the art across the grid the way browse varies it across tiles.
+    slot.poster_url = (
+        random.choice(pool)
+        if pool
+        else (getattr(motd_cfg, "mystery_box_url", "") or "")
+    )
     slot.href = getattr(motd_cfg, "mystery_box_href", "") or ""
 
 
@@ -164,6 +184,7 @@ def build_slots(
     dry_run: bool = False,
     today: datetime.date | None = None,
     week_offset: int = 0,
+    mystery_urls: list[str] | None = None,
     emit=None,
 ) -> MOTDWeek:
     """Resolve the upcoming weekend into a complete grid of poster slots.
@@ -178,6 +199,7 @@ def build_slots(
             emit(detail)
 
     overrides = overrides or {}
+    pool = mystery_urls or []
     motd_cfg = getattr(config, "motd", None)
     slot_count = int(getattr(motd_cfg, "slots", 12) or 12)
     poster_dir = Path(
@@ -234,13 +256,13 @@ def build_slots(
             continue
 
         if not slot.title:
-            _apply_mystery(slot, motd_cfg)
+            _apply_mystery(slot, motd_cfg, pool)
             slot.note = "no title on the schedule yet"
             continue
 
         info = _resolve_omdb(slot.title, api_key=omdb_key) if omdb_key else None
         if not info:
-            _apply_mystery(slot, motd_cfg)
+            _apply_mystery(slot, motd_cfg, pool)
             slot.note = "title could not be verified"
             logger.info("motd: no OMDB match for %r (%s)", slot.title, slot.slot_key)
             if override.get("href"):
@@ -266,7 +288,7 @@ def build_slots(
                 slot.poster_url = f"{poster_base_url}/{filename}"
                 slot.source = "omdb"
             else:
-                _apply_mystery(slot, motd_cfg)
+                _apply_mystery(slot, motd_cfg, pool)
                 slot.note = "poster download failed"
 
         slot.href = override.get("href") or _imdb_href(slot.imdb_id)

{kryten_webqueue-0.45.0 → kryten_webqueue-0.45.1}/kryten_webqueue/routes/admin_motd.py

@@ -18,7 +18,7 @@ from fastapi import APIRouter, Depends, File, Form, HTTPException, Request, Uplo
 from pydantic import BaseModel
 
 from ..auth.session import require_admin
-from ..motd.builder import SLOT_KEY_RE, build_slots, week_context
+from ..motd.builder import SLOT_KEY_RE, build_slots, mystery_pool, week_context
 from ..motd.render import render_motd
 
 router = APIRouter(prefix="/admin/motd", tags=["admin"])
@@ -72,6 +72,9 @@ async def _build_current(request: Request, week: str = "current"):
         overrides=overrides,
         dry_run=True,
         week_offset=_week_offset(week),
+        mystery_urls=mystery_pool(
+            config, getattr(request.app.state, "cover_art", None)
+        ),
     )
 
 

{kryten_webqueue-0.45.0 → kryten_webqueue-0.45.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "kryten-webqueue"
-version = "0.45.0"
+version = "0.45.1"
 description = "Netflix/Tubi-style catalog browser and pay-to-play queue management for CyTube"
 readme = "README.md"
 license = "MIT"

{kryten_webqueue-0.45.0 → kryten_webqueue-0.45.1}/tests/test_motd_publish.py

@@ -30,6 +30,7 @@ def _config(tmp_path, **motd_kw):
         poster_dir=str(tmp_path / "boxes"),
         poster_base_url="https://cdn.example/boxes",
         output_dir=str(tmp_path / "out"),
+        mystery_box_base_url="https://queue.example",
         mystery_box_url="https://cdn.example/boxes/mystery.jpg",
         mystery_box_href="https://queue.example/",
         **motd_kw,
@@ -126,6 +127,43 @@ def test_unverifiable_title_falls_back_to_mystery(tmp_path, monkeypatch, stub_lo
     assert "could not be verified" in slot.note
 
 
+# --- mystery art comes from the browse view's branded placeholder pool ---
+
+
+def test_mystery_pool_absolutizes_browse_placeholders(tmp_path):
+    config = _config(tmp_path)
+    cover_art = SimpleNamespace(
+        list_placeholder_urls=lambda: ["/images/placeholders/a.webp"]
+    )
+    assert builder.mystery_pool(config, cover_art) == [
+        "https://queue.example/images/placeholders/a.webp"
+    ]
+    assert builder.mystery_pool(config, None) == []
+
+
+def test_mystery_slots_use_the_placeholder_pool(tmp_path, monkeypatch, stub_lookup):
+    _stub_workbook(monkeypatch, {})
+    week = builder.build_slots(
+        _config(tmp_path),
+        today=datetime.date(2026, 3, 4),
+        mystery_urls=["https://queue.example/images/placeholders/a.webp"],
+    )
+    assert all(
+        s.poster_url == "https://queue.example/images/placeholders/a.webp"
+        for s in week.slots
+    )
+
+
+def test_mystery_falls_back_when_no_placeholders_installed(
+    tmp_path, monkeypatch, stub_lookup
+):
+    _stub_workbook(monkeypatch, {})
+    week = builder.build_slots(
+        _config(tmp_path), today=datetime.date(2026, 3, 4), mystery_urls=[]
+    )
+    assert all(s.poster_url.endswith("mystery.jpg") for s in week.slots)
+
+
 def test_dry_run_writes_nothing(tmp_path, monkeypatch, stub_lookup):
     _stub_workbook(monkeypatch, {1: ["The Big Bus (1976)"]})
     builder.build_slots(