Skip to content

Fix: give Plex guide titles a poster when Plex returns another entry - #162

Merged
SandObserver merged 2 commits into
mainfrom
fix/plex-guide-name-clash
Oct 14, 2026
Merged

SandObserver merged 2 commits into
mainfrom
fix/plex-guide-name-clash

Conversation

@SandObserver

Copy link
Copy Markdown
Owner

Summary

  • Replace guide_is with guide_target, which returns the key that reaches the guide entry with the show's guid, or nothing.
  • When the name key returns another entry, such as "News" returning "News at Ten", find the entry's numeric id: read ids in batches of 300 until the batch holds the guid, then halve the range down to one id.
  • Check the entry's guid again before writing, so a wrong id leads to a skip, never to another entry's poster.
  • Keep the record under the name key, because numeric ids change at every guide refresh. Only entries the name key reaches are probed for deleted uploads.
  • Jellyfin returns the key unchanged.

Testing

Format, lint, type checks and the full suite pass. New tests cover the id search across batches with gaps, a guid that no id has, and uploads and re-selection by id without touching the other entry.

Checklist

  • Format, lint, type checks and tests pass.
  • DESIGN_VERSION bumped and tests/golden written again if rendered images change.
  • CHANGELOG.md updated under Unreleased for user-visible changes.

Plex finds a guide entry by its name key loosely, so "News" can return
"News at Ten". When the entry it returns has another guid, find the
entry's numeric id from batched id reads and write the poster there. The
record stays under the name key, and only entries the name key reaches
are probed for deleted uploads.
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

Deploying posteryard with  Cloudflare Pages  Cloudflare Pages

Latest commit: 3377c2c
Status: ✅  Deploy successful!
Preview URL: https://fda16b88.posteryard.pages.dev
Branch Preview URL: https://fix-plex-guide-name-clash.posteryard.pages.dev

View logs

@SandObserver
SandObserver merged commit 2918c43 into main Oct 14, 2026
6 checks passed
@SandObserver
SandObserver deleted the fix/plex-guide-name-clash branch October 14, 2026 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant