Skip to content

Change leftover window titles & text#18

Merged
Egezenn merged 1 commit into
Silkroad-Developer-Community:mainfrom
Egezenn:names
Jun 22, 2026
Merged

Change leftover window titles & text#18
Egezenn merged 1 commit into
Silkroad-Developer-Community:mainfrom
Egezenn:names

Conversation

@Egezenn

@Egezenn Egezenn commented Jun 19, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • Chore
    • Updated application branding from "RSBot" to "OasisBot" in splash screen and client window title.
    • Updated temporary file naming conventions.

@coderabbitai

coderabbitai Bot commented Jun 19, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 3e76ad34-6403-4445-95dc-68b88e2e741a

📥 Commits

Reviewing files that changed from the base of the PR and between 796177c and 12e6269.

📒 Files selected for processing (5)
  • Application/OasisBot/Views/SplashScreen.Designer.cs
  • Application/OasisBot/Views/SplashScreen.cs
  • Library/RSBot.Core/Components/ClientManager.cs
  • Library/RSBot.Core/Network/Handler/Agent/Character/CharacterDataEndResponse.cs
  • Library/RSBot.Loader.Library/Library.cpp
💤 Files with no reviewable changes (1)
  • Application/OasisBot/Views/SplashScreen.cs

📝 Walkthrough

Walkthrough

Five string literals referencing "RSBot" are renamed to "OasisBot": the splash screen designer label text, the client window title suffix set on character load, the temp config filename prefix written by ClientManager.PrepareTempConfigFile, the matching read/delete path in the C++ LoadConfig(), and a redundant runtime label assignment is removed from the SplashScreen constructor.

Changes

RSBot → OasisBot Branding Rename

Layer / File(s) Summary
Splash screen label and constructor cleanup
Application/OasisBot/Views/SplashScreen.Designer.cs, Application/OasisBot/Views/SplashScreen.cs
logoLabel.Text is set to "OASISBOT" in the designer; the duplicate runtime assignment in the constructor is removed. Minor end-of-file formatting adjusted in the designer.
Client title and temp config filename rename
Library/RSBot.Core/Network/Handler/Agent/Character/CharacterDataEndResponse.cs, Library/RSBot.Core/Components/ClientManager.cs, Library/RSBot.Loader.Library/Library.cpp
Client window title suffix changed from RSBot to OasisBot. Temp config file prefix changed from RSBot_{pid}.tmp to OasisBot_{pid}.tmp in both the C# writer (ClientManager) and the C++ reader/deleter (LoadConfig).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Poem

🐇 Hop hop, the old name must go,
"RSBot" replaced with a brand-new glow!
The splash screen shines, the title's right,
The temp files renamed, all tidy and bright.
OasisBot blooms in the desert night! 🌵✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 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 purpose of the PR: systematically updating all remaining 'RSBot' references to 'OasisBot' in window titles, labels, and configuration filenames across multiple files.
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.

✏️ 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

Comment @coderabbitai help to get the list of available commands and usage tips.

@Egezenn Egezenn merged commit 3d3b8a2 into Silkroad-Developer-Community:main Jun 22, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant