streamer mode bugfix

[ryanbarlow97]

Description:

fixes
#3572

streamer mode bufix

Please complete the following:

• I have added screenshots for all UI updates
• I process any text displayed to the user through translateText() and I've added it to the en.json file
• I have added relevant tests to the test directory
• I confirm I have thoroughly tested these changes and take full responsibility for any bugs introduced

Please put your Discord username so you can be contacted if a bug or regression is found:

w.o.n

[coderabbitai]

Walkthrough

HostLobbyModal now reads UserSettings to decide whether to expose the lobby ID in the browser URL or hide it behind /streamer-mode. A new updateLobbyHistory method replaces calls to updateHistory at key flow points: lobby opening and game-config updates.

Changes

Lobby URL Visibility Control

Layer / File(s)	Summary
Setup: UserSettings integration src/client/HostLobbyModal.ts	UserSettings is imported and a userSettings field is initialized on HostLobbyModal to support visibility-aware history updates.
Core implementation: Lobby history visibility control src/client/HostLobbyModal.ts	updateHistory is modified to early-return on CrazyGames, and updateLobbyHistory is added to rewrite the browser history to either the lobby URL or /streamer-mode depending on userSettings.lobbyIdVisibility().
Integration: Apply visibility settings at call sites src/client/HostLobbyModal.ts	Calls to updateHistory(url) in the lobby-open flow and game-config-update flow are replaced with updateLobbyHistory(url) to ensure the visibility setting persists across all history updates.

🎯 2 (Simple) | ⏱️ ~8 minutes

A lobby hides or shows its name, 🎭
Streamer-mode now stands your ground,
Settings make it tame. ✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'streamer mode bugfix' is related to the changeset, which modifies HostLobbyModal to incorporate UserSettings for lobbyIdVisibility and streamer mode functionality.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
Description check	✅ Passed	The PR description mentions fixing issue #3572 and references 'streamer mode bugfix', which aligns with the changeset that modifies HostLobbyModal to incorporate UserSettings for lobby ID visibility in streamer mode.

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

---

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.}

[Copilot]

Pull request overview

Fixes the “streamer mode” URL regression for lobby hosts by ensuring the browser URL remains /streamer-mode whenever the user setting indicates lobby IDs should be hidden (instead of being overwritten back to a lobby URL during host-side updates).

Changes:

• Added UserSettings usage in HostLobbyModal to read lobbyIdVisibility().
• Introduced updateLobbyHistory() to apply /streamer-mode when lobby IDs are hidden.
• Replaced host-side history updates (on open + config updates) to respect streamer mode behavior.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.