(feat/fix) Server side retro artwork enhancements - #251
Draft
WizardOfYendor1 wants to merge 14 commits into
Draft
Conversation
…nails and delivery limits
Cold RDB downloads caused valid artwork to be permanently marked missing; we now retry when metadata arrives and reopen prior false misses. Also tweaks the retry scheduling. If it becomes available during, it will wake immediately. Otherwise, the fallback It'll wait for 5 minutes for the RDB then every 3 hours for 24H then stop...but not be permanently flagged as missing (perhaps overkill). Once everything HAS been tried it is marked as missing - which given the very,very infrequent artwork updates/changes/additions should be fine.
… extension-less filename candidate
…me titles. Bump the artwork provider to v3 so prior false Missing records can reopen. Watch configured ROM roots recursively for file changes and watcher overflow. Wait five seconds after the last filesystem event before reconciling.
Build SuccessfulThe plugin compiled successfully against .NET 8 / Jellyfin 10.10.0.
|
RadicalMuffinMan
force-pushed
the
master
branch
2 times, most recently
from
August 24, 2026 17:10
e76ed28 to
f6bb661
Compare
… should be revisited entirely at some point. Fixed another issue uncovered by "accident". Stop a shutdown race where a pending debouce timer could make StopAsync throw an exception. Also remove the ~ handling; left over from previous troubleshooting and no longer needed. The lookup behavior remains the same.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request
Summary
This is a "follow on" PR for retro artwork changes (been waiting for the mega one to merge before adding more :-) ). This improves the server-side artwork handling for retro games, especially when ROMs are added or scanned after Jellyfin is already running.
on side note, there are oddball conditions like games that are the same but different titles that won't get fuzzy matched. Like 2600: Boom Bang/Crackpots etc... needs more work there also
Related Issues
None
Type of Change
Area
Client Impact
Does this need matching changes in a client repo (Core, Smart-TV, Roku)? No
Compatibility
bool?, an int toint?)Testing
Describe how this change was tested.
Test Steps
Screenshots (if applicable)
Include config page screenshots or request/response samples where relevant.
Checklist