Skip to content

fix: recover cached XboxUnity game names - #21

Merged
TrapEmAll merged 1 commit into
mainfrom
fix/catalog-name-enrichment
Jul 24, 2026
Merged

fix: recover cached XboxUnity game names#21
TrapEmAll merged 1 commit into
mainfrom
fix/catalog-name-enrichment

Conversation

@TrapEmAll

Copy link
Copy Markdown
Owner

Summary

  • resolves library names directly from the cached XboxUnity catalog when the primary title row is blank, unknown, or repeats its TitleID
  • applies the same fallback to library search and the game detail pane
  • enriches matching library rows after every catalog page instead of waiting for all pages to finish
  • records page-level catalog sync progress
  • marks abandoned running syncs as interrupted when a new refresh begins
  • repairs already-cached unknown names whenever the database starts
  • preserves user-provided and higher-quality known names

Root cause

The catalog cache correctly contained 53510804 as Hitman: Absolution, but the local refresh had stopped after 4,800 of 5,159 titles. Name enrichment ran only after all 52 pages completed, leaving the existing library row blank despite the correct title already being cached.

Validation

  • reproduced against the user's local database
  • confirmed both Library list and details resolve 53510804 as Hitman: Absolution
  • safely repaired the existing row with XboxUnity title catalog provenance
  • 74 tests passed
  • Ruff passed across the repository
  • focused mypy checks passed
  • compileall and diff checks passed
  • tests cover interrupted refreshes, page-level progress, read-through fallback, startup recovery, and preservation of preferred names

Transport

XboxUnity access remains explicitly HTTP-only.

@TrapEmAll
TrapEmAll marked this pull request as ready for review July 24, 2026 04:57
@TrapEmAll
TrapEmAll merged commit 54bca72 into main Jul 24, 2026
6 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