fix: match app language of the string resource in SteamService

[joshuatam]

Description

Since #1775 , downloadInfo is now showing the depot information from string resources, when the OS is chinese and the app is english, it will show chinese in the string. This fix it to match app language to SteamService.

Recording

screen-20260814-234155.1.mp4

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

Applies the app-selected language in SteamService during attachBaseContext so its string resources (e.g., downloadInfo/depot text) follow the app locale instead of the device locale. Previously, with OS Chinese and app English, downloadInfo used the OS language; now it matches the app.

• Review by running with mismatched OS/app locales and verifying SteamService strings follow the app language.
• No API/data changes or migrations.

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

Summary by CodeRabbit

• New Features
  • Added support for applying the configured application language when initializing the service.
  • Service-related content now respects the selected locale.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: ca04165d-184f-494d-b14e-d296b0bb7a94

📥 Commits

Reviewing files that changed from the base of the PR and between 337d243 and de203c3.

📒 Files selected for processing (1)

• app/src/main/java/app/gamenative/service/SteamService.kt

🚧 Files skipped from review as they are similar to previous changes (1)

• app/src/main/java/app/gamenative/service/SteamService.kt

Included review availability: Your plan includes up to 8 reviews per rolling hour; 7 remain after this review.

---

📝 Walkthrough

Walkthrough

SteamService now applies the configured application language in attachBaseContext before passing the localized context to the superclass.

Changes

Locale-aware service context

Layer / File(s)	Summary
Apply configured locale during service startup app/src/main/java/app/gamenative/service/SteamService.kt	SteamService initializes PrefManager, reads appLanguage, applies it through LocaleHelper, and attaches the localized context.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Merge Risk: ⚪ Minimal · up to de203

This localized change makes SteamService display string resources in the app-selected language instead of the device language; no actionable merge-blocking risk remains beyond normal checks and review.

Suggested reviewers: utkarshdalal

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
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 clearly identifies the main change: matching SteamService string resources to the app language.
Description check	✅ Passed	The description explains the bug and fix, includes a recording, selects bug fix, and completes all checklist items.

✨ 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]

No issues found across 1 file

_{Re-trigger cubic}

[utkarshdalal]

Looks good and nicely aligned with legendary.

Main issue: the keep-alive doesn't actually save the exit sync. It runs in viewModelScope, so swiping the app away cancels it anyway and now EpicService is left running with nothing to stop it. The sync needs to run in a scope that outlives the ViewModel (e.g. the service's own scope), with the service stopping itself when work finishes.

Also, downloadChunksParallel holds every decompressed chunk in memory. for the save-heavy games this targets that's an OOM risk. In resolveConflict you only need the chunks referenced by toDownload files, and checking completeness against just those also stops downloadSuccess going false (and the sync timestamp never updating) over chunks we didn't need.