kryten-webqueue 0.46.0__tar.gz → 0.46.1__tar.gz

{kryten_webqueue-0.46.0 → kryten_webqueue-0.46.1}/CHANGELOG.md

@@ -1,5 +1,18 @@
 # Changelog
 
+## [0.46.1] - 2026-09-10
+
+### Changed
+
+- **The grid is an even split again** (6/6 for a 12-slot Friday+Saturday
+  weekend). The 5/7 shape 0.46.0 inferred from the live MOTD was an artifact of
+  a markup bug: with the `<a>` tags left unclosed, the blank line after the
+  fifth poster looks like a night break but isn't, and most screen widths hide
+  the resulting reflow. The generated snippet closes its tags, so it renders the
+  intended 6/6.
+- A night with more scheduled titles than grid positions now records a warning
+  (job result, admin panel, and log) instead of silently dropping the extras.
+
 ## [0.46.0] - 2026-09-10
 
 ### Changed
@@ -15,6 +28,7 @@
   night is never truncated, so an over-full weekend shows every scheduled movie;
   `motd.slots` is a target the grid is only padded *up* to when the weekend is
   still thin.
+  (Reverted in 0.46.1 — the 5/7 reading was a markup artifact.)
 - **Sunday is excluded.** New `motd.nights` (default `[1, 2]`) controls which
   nights the grid covers; Sunday has no schedule yet, so its workbook titles are
   parsed but ignored. Add `3` once the scheduled-playlists feature fills that

{kryten_webqueue-0.46.0 → kryten_webqueue-0.46.1}/PKG-INFO

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

{kryten_webqueue-0.46.0 → kryten_webqueue-0.46.1}/kryten_webqueue/motd/builder.py

@@ -127,22 +127,13 @@ def _grid_nights(motd_cfg) -> tuple[int, ...]:
     return nights or _NIGHTS
 
 
-def _slot_counts(
-    titles_by_night: dict[int, list[str]], nights: tuple[int, ...], slots: int
-) -> dict[int, int]:
-    """Decide how many positions each night gets.
-
-    The schedule drives the shape — a 5/7 weekend renders 5/7, not an even 6/6 —
-    and a night is never truncated, so a scheduled movie can't be hidden. Only
-    when the weekend is still thin is the grid padded up to ``slots``, handing
-    each extra to whichever night is furthest below its even share.
-    """
-    counts = {n: len(titles_by_night.get(n, [])) for n in nights}
-    share = slots / len(nights)
-    while sum(counts.values()) < slots:
-        night = min(nights, key=lambda n: (counts[n] - share, nights.index(n)))
-        counts[night] += 1
-    return counts
+def _slot_counts(nights: tuple[int, ...], slots: int) -> dict[int, int]:
+    """Split ``slots`` evenly across ``nights``; extras go to the earlier nights."""
+    per_night, remainder = divmod(slots, len(nights))
+    return {
+        night: per_night + (1 if index <= remainder else 0)
+        for index, night in enumerate(nights, 1)
+    }
 
 
 def _blank_grid(
@@ -258,13 +249,26 @@ def build_slots(
         logger.warning("motd: workbook unavailable for %s: %s", week_key, exc)
 
     nights = _grid_nights(motd_cfg)
-    week.slots = _blank_grid(
-        friday,
-        saturday,
-        sunday,
-        _slot_counts(titles_by_night, nights, slot_count),
-        nights,
-    )
+    counts = _slot_counts(nights, slot_count)
+    week.slots = _blank_grid(friday, saturday, sunday, counts, nights)
+
+    # The grid is a fixed size, so a night with more titles than positions would
+    # silently drop the extras. Surface it instead.
+    for night in nights:
+        extra = len(titles_by_night.get(night, [])) - counts[night]
+        if extra > 0:
+            label = _NIGHT_LABELS.get(night, f"Night {night}")
+            week.warnings.append(
+                f"{label}: {extra} scheduled title(s) beyond the "
+                f"{counts[night]}-slot grid are not shown"
+            )
+            logger.warning(
+                "motd: %s has %d title(s) beyond its %d slots on %s",
+                label,
+                extra,
+                counts[night],
+                week_key,
+            )
 
     if not dry_run:
         poster_dir.mkdir(parents=True, exist_ok=True)

{kryten_webqueue-0.46.0 → kryten_webqueue-0.46.1}/pyproject.toml

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

{kryten_webqueue-0.46.0 → kryten_webqueue-0.46.1}/tests/test_motd_publish.py

@@ -85,32 +85,35 @@ def test_grid_pads_a_thin_weekend_to_the_target_size(
     assert len([s for s in week.slots if s.night == 2]) == 6
 
 
-def test_grid_shape_follows_the_schedule(tmp_path, monkeypatch, stub_lookup):
-    # A real 5/7 weekend renders 5/7, not an even 6/6.
+def test_grid_shape_is_an_even_split(tmp_path, monkeypatch, stub_lookup):
     _stub_workbook(
         monkeypatch,
         {
-            1: [f"Friday {n} (1980)" for n in range(5)],
-            2: [f"Saturday {n} (1980)" for n in range(7)],
+            1: [f"Friday {n} (1980)" for n in range(6)],
+            2: [f"Saturday {n} (1980)" for n in range(6)],
         },
     )
     week = builder.build_slots(_config(tmp_path), today=datetime.date(2026, 3, 4))
-    assert len([s for s in week.slots if s.night == 1]) == 5
-    assert len([s for s in week.slots if s.night == 2]) == 7
+    assert len([s for s in week.slots if s.night == 1]) == 6
+    assert len([s for s in week.slots if s.night == 2]) == 6
     assert all(s.resolved for s in week.slots)
+    assert not week.warnings
 
 
-def test_grid_never_truncates_an_overfull_weekend(tmp_path, monkeypatch, stub_lookup):
+def test_overfull_night_warns_instead_of_silently_dropping(
+    tmp_path, monkeypatch, stub_lookup
+):
     _stub_workbook(
         monkeypatch,
         {
             1: [f"Friday {n} (1980)" for n in range(8)],
-            2: [f"Saturday {n} (1980)" for n in range(7)],
+            2: [f"Saturday {n} (1980)" for n in range(6)],
         },
     )
     week = builder.build_slots(_config(tmp_path), today=datetime.date(2026, 3, 4))
-    assert len(week.slots) == 15
-    assert "Friday 7 (1980)" in [s.title for s in week.slots]
+    assert len(week.slots) == 12
+    assert any("2 scheduled title(s)" in w for w in week.warnings)
+    assert "Friday 6 (1980)" not in [s.title for s in week.slots]
 
 
 def test_sunday_titles_are_ignored(tmp_path, monkeypatch, stub_lookup):