Skip to content

fix(emby): bound Episodes-Latest ancestor fanout - #5

Open
Pukabyte wants to merge 1 commit into
darthShadow:mainfrom
Pukabyte:pr/emby-large-ancestor-fanout-public
Open

fix(emby): bound Episodes-Latest ancestor fanout#5
Pukabyte wants to merge 1 commit into
darthShadow:mainfrom
Pukabyte:pr/emby-large-ancestor-fanout-public

Conversation

@Pukabyte

Copy link
Copy Markdown

Summary

  • fail open Episodes-Latest dashboard rewrites when the ancestor list has 30 or more entries
  • retain the optimized rewrite for smaller ancestor lists
  • add a 30-ancestor contract test that verifies source-SQL fallback and result parity

Validation

  • linked emby_fts_rewrite_smoke suite passes

This is deliberately scoped to the large Episodes-Latest ancestor-list shape; other rewrite modes are unchanged.

@Pukabyte
Pukabyte requested a review from darthShadow as a code owner July 24, 2026 10:24
@Pukabyte

Pukabyte commented Aug 1, 2026

Copy link
Copy Markdown
Author

@darthShadow This is what I've put in place to fix the homepage loading/spinning bug

@darthShadow

Copy link
Copy Markdown
Owner

Yeah, sorry, this is on my list to take a look at but it won't be in the 2 weeks or so. In the meanwhile, you can just fork this repo and spin up your images with the fix, it should be straightforward.

@Pukabyte

Pukabyte commented Aug 1, 2026

Copy link
Copy Markdown
Author

@darthShadow this is pr'd from my forked repo which I use in production.

No worries man, just making sure you saw this, but haven't had an incident since deploying these changes.

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.

3 participants