kryten-webqueue 0.36.0__tar.gz → 0.36.2__tar.gz

{kryten_webqueue-0.36.0 → kryten_webqueue-0.36.2}/CHANGELOG.md

@@ -2,6 +2,36 @@
 
 ## [Unreleased]
 
+## [0.36.2] - 2026-08-14
+
+### Fixed
+
+- **Stop flooding MediaCMS with futile tag-push 400s.** The tags step attempted
+  `POST /media/user/bulk_actions` (`add_tags`) for every enriched item, but that
+  endpoint filters `Media.objects.filter(user=request.user, ...)` with no manager
+  or superuser bypass, so pushes against media owned by other users always return
+  400 "No matching media found". The step now looks up the API token's username
+  once via `/whoami` and only attempts the CMS push for media that user actually
+  owns. Local `catalog_tags` remains the source of truth for derived
+  genre/MPAA/hosted tags, so tags still display regardless.
+
+## [0.36.1] - 2026-08-14
+
+### Fixed
+
+- **Tags now appear on item detail pages.** The tags enrichment step pushed tags to
+  MediaCMS via `PUT /media/{token}`, but that endpoint only accepts title/description
+  and silently ignores a `tags` field — so tags never landed in CMS and thus never
+  reverse-synced into the local catalog. Tag pushes now use the correct
+  `POST /media/user/bulk_actions` (`action: add_tags`) endpoint, and computed
+  genre/MPAA/hosted-show tags are also written directly to the local `catalog_tags`
+  table so they display regardless of CMS write success.
+- **Person and Studio facet links now work.** Clicking a person or studio chip on an
+  item detail page dropped the filter and landed on the unfiltered browse page whenever
+  the user had a saved sort or hide-short preference: the `applyFacets()` redirect
+  rebuilt the URL from the dropdowns only and discarded the `person`/`studio` query
+  params. It now preserves them.
+
 ## [0.36.0] - 2026-08-14
 
 ### Added

{kryten_webqueue-0.36.0 → kryten_webqueue-0.36.2}/PKG-INFO

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

{kryten_webqueue-0.36.0 → kryten_webqueue-0.36.2}/kryten_webqueue/catalog/enrichment/steps/tags.py

@@ -42,6 +42,22 @@ class TagsStep:
         )
         self._headers = {"Authorization": f"Token {config.mediacms_token}"}
 
+    async def _get_api_username(self, client: httpx.AsyncClient) -> str | None:
+        """Return the API token's username, or None.
+
+        MediaCMS ``/media/user/bulk_actions`` filters ``Media.objects.filter(
+        user=request.user, ...)`` with no manager/superuser bypass, so tag
+        pushes only succeed on media this user owns. We use this to skip futile
+        pushes against media owned by others instead of flooding CMS with 400s.
+        """
+        try:
+            resp = await client.get(f"{self._base}/api/v1/whoami")
+            if resp.status_code != 200:
+                return None
+            return resp.json().get("username")
+        except (httpx.HTTPError, json.JSONDecodeError, ValueError):
+            return None
+
     async def run(
         self,
         *,
@@ -54,6 +70,7 @@ class TagsStep:
         now = datetime.now(UTC).isoformat()
 
         async with httpx.AsyncClient(headers=self._headers, timeout=_TIMEOUT) as client:
+            api_user = None if dry_run else await self._get_api_username(client)
             for cls in classifications:
                 result.processed += 1
                 try:
@@ -83,10 +100,17 @@ class TagsStep:
                         continue
 
                     if not dry_run:
+                        # Persist computed tags locally first so item detail shows
+                        # them even if the CMS write below fails (owner/permission).
+                        for tag in desired:
+                            await self._db.add_catalog_tag(cls.friendly_token, tag)
                         changed = await self._push_tags(
-                            client, cls.friendly_token, desired
+                            client,
+                            cls.friendly_token,
+                            desired,
+                            api_user=api_user,
                         )
-                        # Reverse sync: pull CMS tags back to local DB
+                        # Reverse sync: pull any CMS-only tags back to local DB
                         await self._reverse_sync(client, cls.friendly_token)
                         if changed:
                             result.changed += 1
@@ -106,15 +130,31 @@ class TagsStep:
         return result
 
     async def _push_tags(
-        self, client: httpx.AsyncClient, token: str, new_tags: list[str]
+        self,
+        client: httpx.AsyncClient,
+        token: str,
+        new_tags: list[str],
+        *,
+        api_user: str | None = None,
     ) -> bool:
-        url = f"{self._base}/api/v1/media/{token}"
+        """Add tags to a CMS media item via the bulk_actions ``add_tags`` action.
+
+        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``. That endpoint
+        filters ``Media.objects.filter(user=request.user, ...)`` with no manager or
+        superuser bypass, so we skip the push unless the API user owns the media
+        rather than flooding CMS with 400s.
+        """
+        detail_url = f"{self._base}/api/v1/media/{token}"
         try:
-            resp = await client.get(url)
+            resp = await client.get(detail_url)
             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 [])
@@ -122,13 +162,14 @@ class TagsStep:
             to_add = [t for t in new_tags if t not in current]
             if not to_add:
                 return False
-            merged = list(current | set(to_add))
-            put = await client.put(url, data={"tags": ",".join(merged)})
-            if put.status_code in (200, 201) and owner:
-                await client.post(
-                    f"{self._base}/api/v1/media/user/bulk_actions",
-                    json={"action": "change_owner", "media_ids": [token], "owner": owner},
-                )
+            put = await client.post(
+                f"{self._base}/api/v1/media/user/bulk_actions",
+                json={
+                    "action": "add_tags",
+                    "media_ids": [token],
+                    "tag_titles": to_add,
+                },
+            )
             return put.status_code in (200, 201)
         except (httpx.HTTPError, json.JSONDecodeError, ValueError):
             return False

{kryten_webqueue-0.36.0 → kryten_webqueue-0.36.2}/kryten_webqueue/templates/catalog/browse.html

@@ -203,6 +203,13 @@ function applyFacets() {
     }
     if (cat) params.set('category', cat);
     if (tag) params.set('tag', tag);
+    // Preserve person/studio facets — these come from the URL, not the dropdowns,
+    // and must survive a sort/hide_short change or the filter is silently dropped.
+    const current = new URLSearchParams(window.location.search);
+    const person = current.get('person');
+    const studio = current.get('studio');
+    if (person) params.set('person', person);
+    if (studio) params.set('studio', studio);
     if (sort && sort !== 'default') params.set('sort', sort);
     // Only include hide_short in URL when non-default (default = 1, hide).
     if (!hideShort) params.set('hide_short', '0');

{kryten_webqueue-0.36.0 → kryten_webqueue-0.36.2}/pyproject.toml

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