kryten-webqueue 0.35.9__tar.gz → 0.36.1__tar.gz

{kryten_webqueue-0.35.9 → kryten_webqueue-0.36.1}/CHANGELOG.md

@@ -2,6 +2,46 @@
 
 ## [Unreleased]
 
+## [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
+
+- **New hosted movie shows.** Added support for "13 Nights of Elvira", "JBB" (shorthand
+  for Joe Bob Briggs), and "TLDI" (shorthand for The Last Drive-In).
+- **Hosted movie title formatting.** Hosted movies now display as "Movie Title (YYYY) - Show Name"
+  (e.g., "Friday the 13th (1980) - Svengoolie"), ensuring year is always in parentheses
+  and show name is appended as a suffix.
+- **Collapsible cast & crew in descriptions.** When person/role/studio data exists, the
+  "Cast & Crew" section in item descriptions is now collapsed by default to avoid
+  duplication. Users can expand it with a single click.
+- **Improved YouTube title cleanup.** Enhanced detection of YouTube imports with pipe
+  characters and "full movie" patterns (e.g., "Title | Full Movie HD | Description"),
+  extracting the clean title before the first pipe.
+- **Distinct tag styling.** Tags now display with orange/accent color borders to
+  differentiate them from categories (purple) and persons/studios (green/gold).
+
+### Fixed
+
+- **Better Monstervision title extraction.** Monstervision titles like "Monstervision 01
+  Friday the 13th Part 1" now extract to clean movie titles with proper database lookups.
+
 ## [0.35.9] - 2026-08-14
 
 ### Fixed

{kryten_webqueue-0.35.9 → kryten_webqueue-0.36.1}/PKG-INFO

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

{kryten_webqueue-0.35.9 → kryten_webqueue-0.36.1}/kryten_webqueue/catalog/enrichment/classify.py

@@ -35,6 +35,15 @@ HOSTED_SHOW_REGISTRY: list[HostedShowEntry] = [
         None,
         "joebobbriggs",
     ),
+    # "JBB" shorthand for Joe Bob Briggs
+    HostedShowEntry(
+        re.compile(r"\bJBB\b", re.I),
+        "Joe Bob Briggs",
+        "hosted_movie",
+        "keep",
+        None,
+        "joebobbriggs",
+    ),
     HostedShowEntry(
         re.compile(r"JBBTLDI|Joe\s*Bob\s+TLDI", re.I),
         "The Last Drive-In with Joe Bob Briggs",
@@ -43,6 +52,15 @@ HOSTED_SHOW_REGISTRY: list[HostedShowEntry] = [
         None,
         "lastdrivein",
     ),
+    # "TLDI" shorthand for The Last Drive-In
+    HostedShowEntry(
+        re.compile(r"\bTLDI\b", re.I),
+        "The Last Drive-In with Joe Bob Briggs",
+        "hosted_movie",
+        "keep",
+        None,
+        "lastdrivein",
+    ),
     HostedShowEntry(
         re.compile(r"(?:The\s+)?Last\s+Drive[\s\-]*In", re.I),
         "The Last Drive-In with Joe Bob Briggs",
@@ -67,6 +85,14 @@ HOSTED_SHOW_REGISTRY: list[HostedShowEntry] = [
         None,
         "svengoolie",
     ),
+    HostedShowEntry(
+        re.compile(r"13\s+Nights?\s+of\s+Elvira", re.I),
+        "13 Nights of Elvira",
+        "hosted_movie",
+        "keep",
+        None,
+        "elvira",
+    ),
     HostedShowEntry(
         re.compile(r"Riff\s*Trax(?:\s+Live)?", re.I),
         "RiffTrax",
@@ -225,8 +251,9 @@ class ItemClassification:
 
 
 # YouTube "Title | English Full Movie | Genre1 Genre2" pattern
+# Also handles variants like "Title | Full Movie HD | Description"
 _YT_FULL_MOVIE_RE = re.compile(
-    r"^(?P<title>.+?)\s*\|\s*(?:\w+\s+)?Full\s+Movie\s*(?:\|\s*(?P<genres>.+))?$", re.I
+    r"^(?P<title>.+?)\s*\|.*(Full\s+(?:Movie|Film|Length|Feature)).*$", re.I
 )
 
 
@@ -299,18 +326,20 @@ def classify_item(
 
     # 4. Movie (>= 30 min)
     if duration_sec >= 1800:
+        # YouTube title cleanup: extract title before first pipe if "full movie" appears
         yt = _YT_FULL_MOVIE_RE.match(raw_title.strip())
         if yt:
             clean_title = yt.group("title").strip()
-            genres = (yt.group("genres") or "").split() if yt.group("genres") else []
+            # Extract year from the clean title part
+            clean_title, year = normalize_and_clean(clean_title)
             return ItemClassification(
                 friendly_token=friendly_token,
                 raw_title=raw_title,
                 content_type="movie",
                 hosted=None,
                 lookup_title=clean_title,
-                lookup_year=None,
-                genre_hints=genres,
+                lookup_year=year,
+                genre_hints=[],
                 duration_sec=duration_sec,
                 description_score=description_score,
                 has_real_art=has_real_art,

{kryten_webqueue-0.35.9 → kryten_webqueue-0.36.1}/kryten_webqueue/catalog/enrichment/steps/tags.py

@@ -83,10 +83,14 @@ 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
                         )
-                        # 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
@@ -108,13 +112,18 @@ class TagsStep:
     async def _push_tags(
         self, client: httpx.AsyncClient, token: str, new_tags: list[str]
     ) -> 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``.
+        """
+        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")
             current = {
                 (t.get("title") if isinstance(t, dict) else t)
                 for t in (data.get("tags_info") or [])
@@ -122,13 +131,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.35.9 → kryten_webqueue-0.36.1}/kryten_webqueue/catalog/enrichment/steps/title.py

@@ -89,6 +89,16 @@ class TitleStep:
             else:
                 candidate = clean
             return candidate if candidate != cls.raw_title else None
+        if ct == "hosted_movie" and cls.hosted:
+            # Format as "Movie Title (YYYY) - Show Name"
+            t = cls.hosted.movie_title
+            y = cls.hosted.movie_year
+            show = cls.hosted.show_name
+            if y:
+                candidate = f"{t} ({y}) - {show}"
+            else:
+                candidate = f"{t} - {show}"
+            return candidate if candidate != cls.raw_title else None
         if ct == "riffed_movie" and cls.hosted:
             entry_template = cls.hosted  # HostedInfo has show_name
             if "{title}" in (cls.hosted.show_name or ""):
@@ -100,7 +110,7 @@ class TitleStep:
                 return (
                     f"RiffTrax Presents: {t} ({y})" if y else f"RiffTrax Presents: {t}"
                 )
-        # hosted_movie / archive / unknown / tv_episode: no change
+        # archive / unknown / tv_episode: no change
         return None
 
     async def _push_title(

{kryten_webqueue-0.35.9 → kryten_webqueue-0.36.1}/kryten_webqueue/static/css/main.css

@@ -740,9 +740,9 @@ a.np-chip {
     border: 1px solid var(--accent);
 }
 .np-chip-tag {
-    background: var(--bg-elevated, #2a2a38);
-    color: var(--text-secondary);
-    border: 1px solid var(--border);
+    background: rgba(232, 150, 16, 0.15);
+    color: var(--accent);
+    border: 1px solid var(--accent);
 }
 .np-chip-person {
     background: rgba(0, 168, 120, 0.15);
@@ -768,6 +768,40 @@ a.np-chip {
     font-weight: 600;
 }
 
+/* Cast & Crew collapsible section in item detail */
+.item-detail-cast-crew-collapse {
+    margin-top: 1rem;
+    padding: 0.5rem 0;
+}
+.item-detail-cast-crew-collapse summary {
+    cursor: pointer;
+    padding: 0.5rem;
+    background: var(--bg-secondary);
+    border-radius: var(--radius);
+    color: var(--text-secondary);
+    font-size: 0.9rem;
+    user-select: none;
+    list-style: none;
+    display: flex;
+    align-items: center;
+}
+.item-detail-cast-crew-collapse summary::-webkit-details-marker {
+    display: none;
+}
+.item-detail-cast-crew-collapse summary::before {
+    content: "▸";
+    display: inline-block;
+    margin-right: 0.5rem;
+    transition: transform 0.2s;
+}
+.item-detail-cast-crew-collapse[open] summary::before {
+    transform: rotate(90deg);
+}
+.item-detail-cast-crew-collapse summary:hover {
+    background: var(--bg-hover);
+    color: var(--text-primary);
+}
+
 /* WebSocket status */
 .ws-connected {
     color: var(--success);

{kryten_webqueue-0.35.9 → kryten_webqueue-0.36.1}/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.35.9 → kryten_webqueue-0.36.1}/kryten_webqueue/templates/catalog/item_detail.html

@@ -41,7 +41,22 @@
             </div>
 
             {% if item.description %}
-                <p class="item-detail-description">{{ item.description }}</p>
+                {% set has_people_data = people.cast or people.director or people.producer or people.writer or studios %}
+                {% if has_people_data %}
+                    {# Split description at "Cast & Crew" and make that section collapsible #}
+                    {% set description_parts = item.description.split('Cast & Crew') %}
+                    {% if description_parts|length > 1 %}
+                        <div class="item-detail-description">{{ description_parts[0] }}</div>
+                        <details class="item-detail-cast-crew-collapse">
+                            <summary>Cast & Crew (from description)</summary>
+                            <div class="item-detail-description">{{ description_parts[1] }}</div>
+                        </details>
+                    {% else %}
+                        <p class="item-detail-description">{{ item.description }}</p>
+                    {% endif %}
+                {% else %}
+                    <p class="item-detail-description">{{ item.description }}</p>
+                {% endif %}
             {% else %}
                 <p class="item-detail-description empty-state">No description available.</p>
             {% endif %}

{kryten_webqueue-0.35.9 → kryten_webqueue-0.36.1}/pyproject.toml

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