fix(): Ignore the steam game check when non-steam game is prelaunched.

[phobos665]

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.

• Bug Fixes
  • Run the Steam session check only if the container source is Steam (using ContainerUtils.extractGameSourceFromContainerId).

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

Summary by CodeRabbit

• Bug Fixes

  • Fixed unwanted "another game running" prompts appearing for non-Steam games. The alert now displays only when launching Steam games.

• New Features

  • Custom Games now launch with a streamlined process that bypasses Steam Cloud synchronization, enabling faster game startup.

[phobos665]

The only changes here is to check isSteamGame and only execute if isSteamGame = true

[coderabbitai]

📝 Walkthrough

Walkthrough

The 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

Cohort / File(s)	Summary
Game Launch Flow Logic app/src/main/java/app/gamenative/ui/PluviaMain.kt	Added conditional check to restrict Steam's "another game running" prompt to Steam games only (isSteamGame check). Introduced new bypass path for Custom Games to skip Steam Cloud sync and proceed directly to launch.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• Removed checking for custom game exes to speed things up #286: Modifies PluviaMain.kt to alter custom-game launch flow by bypassing executable-selection/validation paths
• Don't show "continue offline" when user is offline and a game is running #174: Changes behavior when a game is running by preventing Steam reconnect attempts based on SteamService.isGameRunning state

Poem

🐰 Steam games alone hear the warning bell,
Custom adventures skip the sync and spell,
A hop through the launch, a path less well-trod,
Different routes for different game mod! 🎮✨

🚥 Pre-merge checks | ✅ 2 | ❌ 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 (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: ignoring Steam game checks for non-Steam games during prelaunch, which aligns with the AI-generated summary showing restriction of the Steam prompt to Steam games only and addition of a bypass path for Custom Games.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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 and usage tips.}

[cubic-dev-ai]

No issues found across 1 file