fix: refactor pulseaudio sink implementation to auto failover

[joshuatam]

Description

fix: refactor pulseaudio sink implementation to auto failover

• related pulseaudio commit: GameNative/pulseaudio-android@6a2f1da

Recording

None

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 CodeRabbit

• Chores
  • Updated bundled PulseAudio component to the latest version for improved compatibility and performance.

[coderabbitai]

📝 Walkthrough

Walkthrough

Updated the PulseAudio component archive version reference in XServerScreen.kt from the June 9th build to the June 10th build within the component extraction mapping used during environment initialization.

Changes

PulseAudio Component Asset Update

Layer / File(s)	Summary
PulseAudio archive version bump app/src/main/java/app/gamenative/ui/screen/xserver/XServerScreen.kt	Updated the bundled PulseAudio archive filename from pulseaudio-gamenative-20260609.tzst to pulseaudio-gamenative-20260610.tzst in the extractionPairs mapping within refreshComponentsFiles.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~1 minute

Possibly related PRs

• utkarshdalal/GameNative#1484: Updates XServerScreen.refreshComponentsFiles() to extract/load a newer pulseaudio-gamenative-*.tzst PulseAudio component archive.
• utkarshdalal/GameNative#1536: Modifies XServerScreen.refreshComponentsFiles() to change which pulseaudio-gamenative-*.tzst asset version is extracted/loaded.
• utkarshdalal/GameNative#1527: Updates the PulseAudio component archive version used in XServerScreen.refreshComponentsFiles(...) with a different pulseaudio-gamenative-*.tzst date.

Suggested reviewers

• utkarshdalal

Poem

🐰 A tzst file hops from day to day,
Component assets leading the way,
June tenth arrives with a brand new build—
PulseAudio's sounds shall surely be thrilled! 🎵

🚥 Pre-merge checks | ✅ 2 | ❌ 3

❌ Failed checks (2 warnings, 1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The title mentions 'refactor pulseaudio sink implementation to auto failover', but the actual change only updates a PulseAudio asset version from 20260609 to 20260610, with no refactoring evident.	Update the title to accurately reflect the actual change, such as 'chore: update pulseaudio-gamenative asset to version 20260610' or provide the promised refactoring implementation.
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.
Description check	❓ Inconclusive	The description lacks key implementation details: it references a related commit but provides no explanation of what the auto-failover refactoring entails or how it improves the sink implementation.	Add a detailed explanation of what auto-failover behavior was implemented, how it works, and how the asset update relates to this refactoring. Also clarify why the recording is not attached.

✅ Passed checks (2 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.

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

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

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@app/src/main/java/app/gamenative/ui/screen/xserver/XServerScreen.kt`:
- Line 4506: PulseAudio runtime paths are constructed with new
File(context.getFilesDir(), "/pulseaudio") (PulseAudioComponent, occurrences
around lines ~259/302/345) which yields an absolute "/pulseaudio" path, while
XServerScreen.kt extracts the archive into File(context.filesDir, "pulseaudio")
(<filesDir>/pulseaudio); fix by normalizing the runtime path to match the
extraction target: change the child path from "/pulseaudio" to "pulseaudio" in
PulseAudioComponent so it uses new File(context.getFilesDir(), "pulseaudio") (or
otherwise build the path via File(context.filesDir, "pulseaudio")) ensuring
/pulseaudio is not treated as an absolute path and the extracted pactl/modules
and generated default.pa are found.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 2fa2cbb4-96de-47bc-923c-8a541a17bdb8

📥 Commits

Reviewing files that changed from the base of the PR and between bfe5bf5 and bb343b8.

📒 Files selected for processing (3)

• app/src/main/assets/pulseaudio-gamenative-20260609.tzst
• app/src/main/assets/pulseaudio-gamenative-20260610.tzst
• app/src/main/java/app/gamenative/ui/screen/xserver/XServerScreen.kt

[cubic-dev-ai]

No issues found across 3 files

_{Re-trigger cubic}