Skip to content

fix(tui): refresh valid cached ROM lists - #104

Draft
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/critical-bug-management-258d
Draft

cursor[bot] wants to merge 1 commit into
mainfrom
cursor/critical-bug-management-258d

Conversation

@cursor

@cursor cursor Bot commented Sep 29, 2026

Copy link
Copy Markdown
Contributor

Summary

Keep cached ROM lists as an immediate paint path, but continue the deferred network refresh even when the cache count still matches so same-count server changes do not leave the TUI permanently stale.

Crate(s)

  • romm-api
  • romm-cli
  • romm-tui
  • docs / CI only

Checklist

  • cargo fmt --check passes
  • Both cargo clippy runs pass (--all-features and --no-default-features, -D warnings)
  • cargo test --workspace passes (or scoped tests with reason)
    • Scoped to cargo test -p romm-tui because the change is isolated to TUI ROM-list cache refresh behavior.
  • PR title uses Conventional Commits with the right scope for the changelog
  • No secrets, tokens, or personal config in the diff
  • User-facing changes documented in the relevant docs/*.md if needed
    • No docs update needed; this restores intended refresh behavior.
Open in Web View Automation 

Co-authored-by: Patrick Mill <patricksmill@users.noreply.github.com>
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