kryten-webqueue 0.32.1__tar.gz → 0.32.2__tar.gz

{kryten_webqueue-0.32.1 → kryten_webqueue-0.32.2}/CHANGELOG.md

@@ -5,6 +5,12 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.32.2] — 2026-07-09
+
+### Fixed
+
+- **Promo clips are exempt from recently-played hiding.** Playing a promo-pool clip no longer records any hide state — promos are already excluded from the public catalog and must not be treated like normal mutable/immutable playlist items. `record_play_completion` now skips promo-pool members entirely. A one-time migration (v14) purges any promo hide state recorded before this fix; the same cleanup is also available on-demand via `purge_promo_hide_state()`.
+
 ## [0.32.1] — 2026-07-09
 
 ### Added
@@ -28,6 +34,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 [0.32.0]: https://github.com/grobertson/kryten-webqueue/releases/tag/v0.32.0
 [0.32.1]: https://github.com/grobertson/kryten-webqueue/releases/tag/v0.32.1
+[0.32.2]: https://github.com/grobertson/kryten-webqueue/releases/tag/v0.32.2
 
 ## [0.31.0] — 2026-07-04
 

{kryten_webqueue-0.32.1 → kryten_webqueue-0.32.2}/PKG-INFO

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

{kryten_webqueue-0.32.1 → kryten_webqueue-0.32.2}/kryten_webqueue/catalog/db/_catalog.py

@@ -360,7 +360,19 @@ class _CatalogMixin:
         collection releases together. Everything else (movies, standalone clips,
         long items even inside a mutable playlist) gets a time-boxed
         ``play_completions`` row instead.
+
+        Promo-pool clips are exempt entirely: they are already excluded from the
+        public catalog and must never be treated like normal (mutable/immutable)
+        playlist items, so playing one records nothing.
         """
+        promo = await self._fetch_one(
+            "SELECT 1 FROM saved_playlist_items spi "
+            "JOIN saved_playlists sp ON sp.id = spi.playlist_id "
+            "WHERE spi.media_id = ? AND spi.media_type = ? AND sp.promo_type IS NOT NULL LIMIT 1",
+            [friendly_token, media_type],
+        )
+        if promo:
+            return
         rows = await self._fetch_all(
             """
             SELECT spi.playlist_id AS playlist_id, spi.position AS position,
@@ -435,6 +447,29 @@ class _CatalogMixin:
         )
         return {"completions": pc["c"] if pc else 0, "playlist_pass": pip["c"] if pip else 0}
 
+    async def purge_promo_hide_state(self) -> dict:
+        """Remove any recently-played hide state recorded for promo-pool clips.
+
+        Promos must never be subject to recently-played hiding. New completions
+        are already skipped in ``record_play_completion``; this purges rows
+        written before that exemption existed (also run once by migration v14).
+        Returns the row counts removed.
+        """
+        promo_subquery = (
+            "SELECT spi.media_id FROM saved_playlist_items spi "
+            "JOIN saved_playlists sp ON sp.id = spi.playlist_id "
+            "WHERE sp.promo_type IS NOT NULL AND spi.media_type = 'cm'"
+        )
+        pc = await self._fetch_one(
+            f"SELECT COUNT(*) AS c FROM play_completions WHERE media_id IN ({promo_subquery})"
+        )
+        pip = await self._fetch_one(
+            f"SELECT COUNT(*) AS c FROM playlist_item_played WHERE media_id IN ({promo_subquery})"
+        )
+        await self._execute(f"DELETE FROM play_completions WHERE media_id IN ({promo_subquery})")
+        await self._execute(f"DELETE FROM playlist_item_played WHERE media_id IN ({promo_subquery})")
+        return {"completions": pc["c"] if pc else 0, "playlist_pass": pip["c"] if pip else 0}
+
     async def get_recently_played_debug(self, days: int) -> dict:
         """Snapshot of what the recently-played rules currently hide (admin test).
 

{kryten_webqueue-0.32.1 → kryten_webqueue-0.32.2}/kryten_webqueue/catalog/db/_connection.py

@@ -291,6 +291,24 @@ MIGRATIONS = [
     );
     CREATE INDEX IF NOT EXISTS idx_playlist_item_played_media ON playlist_item_played(media_id);
     """,
+    # v14: One-time cleanup of recently-played hide state wrongly recorded for
+    # promo-pool clips. Promos are excluded from the public catalog and must
+    # never be treated like normal playlist items, so any hide state recorded for
+    # a promo clip (before the exemption in record_play_completion) is purged.
+    """
+    DELETE FROM play_completions
+    WHERE media_id IN (
+        SELECT spi.media_id FROM saved_playlist_items spi
+        JOIN saved_playlists sp ON sp.id = spi.playlist_id
+        WHERE sp.promo_type IS NOT NULL AND spi.media_type = 'cm'
+    );
+    DELETE FROM playlist_item_played
+    WHERE media_id IN (
+        SELECT spi.media_id FROM saved_playlist_items spi
+        JOIN saved_playlists sp ON sp.id = spi.playlist_id
+        WHERE sp.promo_type IS NOT NULL AND spi.media_type = 'cm'
+    );
+    """,
 ]
 
 

{kryten_webqueue-0.32.1 → kryten_webqueue-0.32.2}/pyproject.toml

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

{kryten_webqueue-0.32.1 → kryten_webqueue-0.32.2}/tests/test_recently_played_hide.py

@@ -349,4 +349,52 @@ async def test_recently_played_debug_reports_both_signals(db):
     assert {r["media_id"] for r in debug["by_playlist_pass"]} == {"ep1"}
 
 
+async def test_promo_clips_are_exempt_from_hiding(db):
+    # A clip in a promo pool (promo_type set) must never be recorded as played
+    # nor appear in the recently-played debug list.
+    await _add_catalog(db, "promo1", "Bumper", duration_sec=30)
+    pid = await db.create_saved_playlist(
+        name="Bumpers", description=None, is_immutable=False,
+        created_by="admin", promo_type="general",
+    )
+    await db.replace_playlist_items(pid, [
+        {"media_type": "cm", "media_id": "promo1", "title": "Bumper", "duration_sec": 30},
+    ])
+
+    await db.record_play_completion(friendly_token="promo1", duration_sec=30)
+
+    rows = await db._fetch_all("SELECT media_id FROM play_completions")
+    assert rows == []
+    pip = await db._fetch_all("SELECT media_id FROM playlist_item_played")
+    assert pip == []
+    debug = await db.get_recently_played_debug(21)
+    assert debug["by_completion"] == []
+    assert debug["by_playlist_pass"] == []
+
+
+async def test_purge_promo_hide_state_removes_stale_rows(db):
+    # Rows recorded before the promo exemption (e.g. in production) are cleaned up
+    # at the data layer by purge_promo_hide_state (also run by migration v14).
+    await _add_catalog(db, "promo1", "Bumper", duration_sec=30)
+    pid = await db.create_saved_playlist(
+        name="Bumpers", description=None, is_immutable=False,
+        created_by="admin", promo_type="general",
+    )
+    await db.replace_playlist_items(pid, [
+        {"media_type": "cm", "media_id": "promo1", "title": "Bumper", "duration_sec": 30},
+    ])
+    # Simulate a stale row written before the exemption existed.
+    await db._execute("INSERT INTO play_completions (media_type, media_id) VALUES ('cm', 'promo1')")
+    # Before the purge it shows in the (unfiltered) debug view.
+    assert {r["media_id"] for r in (await db.get_recently_played_debug(21))["by_completion"]} == {"promo1"}
+
+    removed = await db.purge_promo_hide_state()
+    assert removed["completions"] == 1
+
+    assert await db._fetch_all("SELECT media_id FROM play_completions") == []
+    assert (await db.get_recently_played_debug(21))["by_completion"] == []
+
+
+
+