feat(): LibraryViewModel performance improvements

[phobos665]

Description

• Reduce sorting cpu time & memory usage (Cpu is now O(n) instead of O (n log n) e.g. exponential complexity for sorting) - Did this by creating a sortName on entry creation, rather than manipulating both the comparitor and the name on comparison.
• Created a Job & cache for fetching results so that we can cancel and check the cache before we fetch again. This way we don't have issues with multiple processes trying to receive the results and causing excessive redraws on the UI.

Overall, the UI should be vastly more responsive on initial and further retrievals for games going forward. This also means Carousel should be much more usable.

Recording

Type of Change

• Bug fix
• Performance / stability improvement
• Compatibility improvements
• Other (requires prior approval)

Checklist

• If I have access to #code-changes, I have discussed this change there and it has been green-lighted. If I do not have access, I have still provided clear context in this PR. If I skip both, I accept that this change may face delays in review, may not be reviewed at all, or may be closed.
• This change aligns with the current project scope (core functionality, stability, or performance). If not, it has been explicitly approved beforehand.
• I have attached a recording of the change.
• I have read and agree to the contribution guidelines in CONTRIBUTING.md.

---

Summary by cubic

Speeds up LibraryViewModel filtering, sorting, and pagination to reduce CPU work and UI redraws. The library and carousel feel more responsive on first load and when paging; pagination now cancels stale work.

• Performance
  • Reuse cached filtered+sorted results and paginate via applyPagination instead of re-filtering on page changes.
  • Add pageJob and filterJob with ensureActive(); cancel in-flight pagination on rebuild to avoid stale UI commits.
  • Precompute sortName per entry for cheaper comparisons.
  • Cache Steam base-game sizes per app/branch in a ConcurrentHashMap; batch installed-branch lookup with SteamService.getAllInstalledApps().
  • Check GOG/Epic/Amazon credentials once per run and reuse for filtering and badge counts.
  • Clear caches on refresh and when the Steam app list changes.

^{Written for commit 11187c0. Summary will update on new commits.}

Summary by CodeRabbit

• Performance
  • Improved library filtering and pagination by reusing cached filtered/sorted results and applying pagination-specific adjustments without re-running the full filtering pipeline.
  • Faster refreshes and page transitions, with in-progress filtering work canceled when superseded.
• Bug Fixes
  • Improved consistency of Steam-related filtering and size estimates.
  • More reliable source availability indicators (including badge counts) by evaluating credentials once per filtering run and reusing the results.

[coderabbitai]

📝 Walkthrough

Walkthrough

LibraryViewModel now cancels superseded filtering jobs, caches filtered results for pagination, memoizes Steam size calculations, precomputes sorting and credential data, and centralizes recommendation insertion.

Changes

Library filtering pipeline

Layer / File(s)	Summary
Filter cancellation and cached pagination app/src/main/java/app/gamenative/ui/model/LibraryViewModel.kt	Filtering jobs are synchronized and cancellable, while pagination reuses the cached filtered list and applies recommendation insertion through applyPagination.
Steam filtering and size memoization app/src/main/java/app/gamenative/ui/model/LibraryViewModel.kt	Steam processing precomputes installed branches and lowercase sort names, memoizes size calculations, and clears the size cache when Steam data or refresh state changes.
Credential-gated source inclusion app/src/main/java/app/gamenative/ui/model/LibraryViewModel.kt	GOG, Epic, and Amazon credential checks are evaluated once per run and reused for source inclusion and badge counts; cancellation checks are added around sorting and state updates.

Estimated code review effort: 3 (Moderate) | ~25 minutes

Sequence Diagram(s)

sequenceDiagram
  participant FilterRequest
  participant LibraryViewModel
  participant PaginationState
  FilterRequest->>LibraryViewModel: start filtering
  LibraryViewModel->>LibraryViewModel: cancel prior filter and page jobs
  LibraryViewModel->>PaginationState: cache full filtered list
  FilterRequest->>LibraryViewModel: change page
  LibraryViewModel->>PaginationState: slice cached list and apply pagination

Loading

Suggested reviewers: utkarshdalal

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title is concise and clearly matches the performance-focused LibraryViewModel changes.
Description check	✅ Passed	The description follows the template and gives a clear change summary, type, and checklist, but the Recording section is empty.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[cubic-dev-ai]

All reported issues were addressed across 1 file

<sub>Reply with feedback, questions, or to request a fix.

Re-trigger cubic</sub>

[phobos665]

Encompassed by: #1721

But feel free to merge this as a smaller step to reduce that PR's size.