fix(): Ignore the steam game check when non-steam game is prelaunched.#470
Conversation
| return@launch | ||
| } | ||
| } catch (_: Exception) { /* ignore persona read errors */ } | ||
| val isSteamGame = ContainerUtils.extractGameSourceFromContainerId(appId) == GameSource.STEAM |
There was a problem hiding this comment.
The only changes here is to check isSteamGame and only execute if isSteamGame = true
📝 WalkthroughWalkthroughThe launch flow in PluviaMain.kt is refined to restrict a Steam "another game running" prompt to Steam games only, while Custom Games now bypass Steam Cloud synchronization and proceed directly to launch. Non-Steam games retain their existing behavior. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
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. Comment |
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
2 participants
Summary by cubic
Skip the Steam “currently playing” check when prelaunching non‑Steam games. This removes the false “account session active” prompt and lets non‑Steam titles start without interruption.
Written for commit cebf5a6. Summary will update on new commits.
Summary by CodeRabbit
Bug Fixes
New Features