Files
silo-server/internal/metadata/show_status.go
0694787504 fix(overlays): show_status persistence + card overlay layout fixes (#335)
* fix(metadata): persist series show_status from provider metadata

Plugin-reported series status (proto status field 31) was mapped into
MetadataResult.ShowStatus but dropped by both metadataResultToItem and
itemToMetadataResult, so media_items.show_status stayed empty for every
movie and series - only the manga enrichment path ever wrote it. This
left the Show Status card overlay permanently blank for series.

- carry ShowStatus through both converters; series values normalize to
  a canonical lowercase domain (returning/ended/cancelled/in_production/
  upcoming) so TMDB "Returning Series"/"Canceled" and TVDB
  "Continuing"/"Upcoming" converge on one spelling
- pass non-series values through verbatim so the manga status domain
  ("Ongoing", ...) can never be mangled by a generic refresh round-trip
- round-tripping the existing item's status also stops refreshes from
  wiping a previously persisted value via show_status = EXCLUDED.show_status
- extend the web overlay formatter with continuing/upcoming/planned

TMDB/TVDB plugins need follow-up changes to actually emit the status
field; prepped separately in their repos.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* fix(web): overlay badge layout, ordering, and wordmark rendering

Fixes from a full card-overlay audit (every issue verified by DOM
geometry measurement before/after):

- render each card edge as one flex row holding both corner stacks so
  opposing badges share the width (min-w-0 + truncate) instead of
  overlapping on narrow cards; long labels ellipsize instead of
  wrapping over the opposite corner
- honor prefs.order via orderedOverlaysForPosition — the renderer
  previously ignored the stored order entirely
- cap corners at 3 badges so maxed-out configs can't collide with the
  opposite vertical corner
- lift bottom-right badges above the card menu button, which is always
  visible on touch devices and occluded them
- suppress the text label when a wordmark icon (HDR10/ATMOS/AV1/HDR)
  already spells it — pill/vibrant presets rendered "HDR10 HDR10" —
  and widen the wordmark viewBoxes, which clipped their own text;
  drop the never-used iconOnly flag the wordmark rule supersedes
- standalone resolution badge now uses prettyResolution ("4K", not
  "2160P"), matching the combined badge
- manga cards skip generic overlays (their status/count chips own both
  top corners) and the two chips now share a row and truncate instead
  of overlapping each other
- useOverlayPrefs returns null while loading so cards no longer flash
  default badges before the user's config or admin kill switch arrives
- settings rows for Resolution/HDR now say why they're hidden while
  the combined badge is enabled
- add a CardOverlays test suite covering every registered overlay,
  ordering, suppression, wordmarks, the corner cap, and menu clearance

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
2026-07-07 17:16:38 -04:00

36 lines
1.3 KiB
Go

package metadata
import "strings"
// NormalizeShowStatus maps provider-reported series lifecycle statuses onto
// the canonical domain persisted in media_items.show_status for series:
// "returning", "ended", "cancelled", "in_production", "upcoming", or "".
// Providers spell these differently (TMDB "Returning Series" / "Canceled",
// TVDB "Continuing" / "Upcoming"), so persistence converges on one spelling
// clients can rely on. Unrecognized values pass through trimmed and
// lowercased rather than being dropped, so a new provider status still
// reaches clients (which fall back to displaying the raw value).
//
// This is series-domain only. Manga statuses use a different value domain
// ("Ongoing", "Completed", ...) normalized by the manga enrichment pipeline;
// callers must not route manga values through this function.
func NormalizeShowStatus(raw string) string {
cleaned := strings.ToLower(strings.TrimSpace(raw))
switch cleaned {
case "":
return ""
case "returning", "returning series", "continuing":
return "returning"
case "ended":
return "ended"
case "cancelled", "canceled":
return "cancelled"
case "in production", "in_production", "pilot":
return "in_production"
case "upcoming", "planned":
return "upcoming"
default:
return cleaned
}
}