kryten-webqueue 0.36.4__tar.gz → 0.36.5__tar.gz

{kryten_webqueue-0.36.4 → kryten_webqueue-0.36.5}/CHANGELOG.md

@@ -2,6 +2,17 @@
 
 ## [Unreleased]
 
+## [0.36.5] - 2026-08-14
+
+### Changed
+
+- **Tags step no longer reads every item.** With `mediacms_manage_all_media` set,
+  the tags step now issues a single idempotent `add_tags` POST per item (the
+  server skips tags already present) instead of a per-item `GET` to read current
+  tags/owner. The redundant per-item reverse-sync `GET` was also removed — catalog
+  sync already mirrors CMS tags in bulk via `/api/v1/media_facets`. Stock-CMS
+  (owner-scoped) deployments still do the ownership `GET` on the fallback path.
+
 ## [0.36.4] - 2026-08-14
 
 ### Changed

{kryten_webqueue-0.36.4 → kryten_webqueue-0.36.5}/PKG-INFO

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

{kryten_webqueue-0.36.4 → kryten_webqueue-0.36.5}/kryten_webqueue/catalog/enrichment/steps/tags.py

@@ -117,8 +117,6 @@ class TagsStep:
                             desired,
                             api_user=api_user,
                         )
-                        # Reverse sync: pull any CMS-only tags back to local DB
-                        await self._reverse_sync(client, cls.friendly_token)
                         if changed:
                             result.changed += 1
                         else:
@@ -148,50 +146,37 @@ class TagsStep:
 
         The ``PUT /media/{token}`` endpoint only accepts title/description/media_file
         and silently ignores a ``tags`` field, so tag changes must go through
-        ``POST /media/user/bulk_actions`` with ``action: add_tags``. On stock
-        MediaCMS that endpoint is owner-scoped, so unless ``manage_all`` is set we
-        skip the push for media the API user does not own rather than flooding
-        CMS with 400s.
+        ``POST /media/user/bulk_actions`` with ``action: add_tags``.
+
+        When ``manage_all`` is set the server-side ``add_tags`` is idempotent
+        (it skips tags already present), so we POST directly with no per-item GET.
+        On stock MediaCMS the endpoint is owner-scoped, so we must first GET the
+        item to check ownership and skip the push for media we do not own.
         """
-        detail_url = f"{self._base}/api/v1/media/{token}"
         try:
-            resp = await client.get(detail_url)
-            if resp.status_code != 200:
-                return False
-            data = resp.json()
-            owner = data.get("user")
-            if not self._manage_all and (api_user is None or owner != api_user):
-                return False
-            current = {
-                (t.get("title") if isinstance(t, dict) else t)
-                for t in (data.get("tags_info") or [])
-            }
-            to_add = [t for t in new_tags if t not in current]
-            if not to_add:
-                return False
+            if not self._manage_all:
+                resp = await client.get(f"{self._base}/api/v1/media/{token}")
+                if resp.status_code != 200:
+                    return False
+                data = resp.json()
+                owner = data.get("user")
+                if api_user is None or owner != api_user:
+                    return False
+                current = {
+                    (t.get("title") if isinstance(t, dict) else t)
+                    for t in (data.get("tags_info") or [])
+                }
+                new_tags = [t for t in new_tags if t not in current]
+                if not new_tags:
+                    return False
             put = await client.post(
                 f"{self._base}/api/v1/media/user/bulk_actions",
                 json={
                     "action": "add_tags",
                     "media_ids": [token],
-                    "tag_titles": to_add,
+                    "tag_titles": new_tags,
                 },
             )
             return put.status_code in (200, 201)
         except (httpx.HTTPError, json.JSONDecodeError, ValueError):
             return False
-
-    async def _reverse_sync(self, client: httpx.AsyncClient, token: str) -> None:
-        """Insert any CMS-only tags into the local catalog_tags table."""
-        url = f"{self._base}/api/v1/media/{token}"
-        try:
-            resp = await client.get(url)
-            if resp.status_code != 200:
-                return
-            data = resp.json()
-            for t in data.get("tags_info") or []:
-                name = t.get("title") if isinstance(t, dict) else t
-                if name:
-                    await self._db.add_catalog_tag(token, str(name))
-        except (httpx.HTTPError, json.JSONDecodeError, ValueError) as exc:
-            logger.warning("[tags] Failed reverse-sync for %s: %s", token, exc)

{kryten_webqueue-0.36.4 → kryten_webqueue-0.36.5}/pyproject.toml

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