Skip to content

fix: upgrade path fallback, provider progress tracking#48

Merged
LavX merged 4 commits intodevelopmentfrom
fix/upgrade-and-search-improvements
Mar 30, 2026
Merged

fix: upgrade path fallback, provider progress tracking#48
LavX merged 4 commits intodevelopmentfrom
fix/upgrade-and-search-improvements

Conversation

@LavX
Copy link
Copy Markdown
Owner

@LavX LavX commented Mar 30, 2026

Summary

  • Upgrade subtitle path fallback: when a subtitle file has been renamed or re-downloaded (e.g. .hu.hi.srt to .hu.srt), the upgrade system now finds the current file for the same language instead of silently dropping the candidate
  • Per-provider progress tracking: subtitle searches now show which provider is being queried in real-time in the jobs manager (e.g. "Searching opensubtitles (2/5)")
  • Throttle crash fix: time_until_midnight callers were passing datetime instead of timezone (regression from upstream cherry-pick)
  • Null score guard: prevents TypeError on manual download history entries with no score
  • Restart handler fix: moved terminate_child out of finally block

Test plan

  • Trigger upgrade on a movie with stale subtitle path in history, verify it finds the current file and actually searches providers
  • Trigger individual subtitle search, verify jobs manager shows per-provider progress
  • Verify provider throttling doesn't crash on titulky/legendasdivx limits

LavX added 3 commits March 30, 2026 10:58
…uard

- When a subtitle file has been renamed or re-downloaded (e.g. .hu.hi.srt
  to .hu.srt), the upgrade system now finds the current file for the same
  language instead of silently dropping the candidate.
- Fix time_until_midnight callers passing datetime instead of timezone
  (regression from upstream cherry-pick of 263d01c).
- Guard against None score in forced_minimum_score to prevent TypeError
  on manual download history entries.
Show which provider is being searched in real-time in the jobs manager
(e.g. "Searching opensubtitles (2/5)"). Adds a provider_progress_callback
on the provider pool that gets called before each provider query.
Both individual and specific subtitle searches now report progress.
The terminate/restart logic should not be inside finally, which
runs even if os.remove fails, masking the exception.
Repository owner deleted a comment from chatgpt-codex-connector bot Mar 30, 2026
Verify the scraper service is reachable during provider initialization
by hitting /health. If unreachable, raise ServiceUnavailable so the
provider is throttled for 20 minutes and shows the issue in
System > Providers, instead of silently failing on every search.
@LavX LavX merged commit 3003dbc into development Mar 30, 2026
4 checks passed
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