Moved prelaunch steps to "install dependencies" option

[unbelievableflavour]

This separate "install dependencies" to a executable phase before launch phase. Causing them to install the vc redists (and other prelaunchsteps) in the container instead of wine command This will make them actually create registry keys.

[coderabbitai]

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

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

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.

---

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]

2 issues found across 12 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="app/src/main/java/app/gamenative/utils/PreLaunchSteps.kt">

<violation number="1" location="app/src/main/java/app/gamenative/utils/PreLaunchSteps.kt:41">
P2: buildChain always appends a trailing " & " but the only call site executes the chain directly, leaving a dangling command separator that can trigger a cmd syntax error.</violation>
</file>

<file name="app/src/main/java/app/gamenative/ui/PluviaMain.kt">

<violation number="1" location="app/src/main/java/app/gamenative/ui/PluviaMain.kt:1083">
P2: Changing XServer’s route to a query-template breaks exact route comparisons elsewhere (e.g., MainViewModel.setCurrentScreen), causing the XServer screen to be misclassified.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}