Improve bit Boilerplate AI assistant theme toggler (#12547)

[yasmoradi]

closes #12547

Summary by CodeRabbit

• Bug Fixes
  • Theme changes now report whether a new theme was actually applied, instead of always assuming success.
  • When requesting a theme change, the app now confirms if the theme was updated or was already set, giving clearer feedback to the user.

[coderabbitai]

Important

Review skipped

Auto incremental reviews are disabled on this repository.

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.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 6f855919-1513-472f-bd77-dc995da26d2d

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

Walkthrough

The CHANGE_THEME SignalR handler in the client now returns true only when the theme actually changes, and false when it is already set. The server-side chatbot tool captures this boolean and returns an "already set" message instead of always reporting success.

Changes

Theme Change Idempotency

Layer / File(s)	Summary
CHANGE_THEME handler and chatbot tool response src/.../AppClientCoordinator.cs, src/.../AppChatbot.Tools.cs	Client handler tracks and returns a themeChanged boolean (false when theme already matches); server tool captures the result and branches its response to report "already set" when no change occurred.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐇 A theme was asked, already dressed the same,
No need to toggle, nothing changed but the claim.
The rabbit checked twice, "It's already that hue!"
Now the server replies with something more true.
Less fibbing, more facts — that's the cottontail way! 🎨

🚥 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 clearly summarizes the main change: improving the AI assistant theme toggler.
Linked Issues check	✅ Passed	The changes align with issue #12547 by improving the theme toggler behavior and feedback.
Out of Scope Changes check	✅ Passed	The PR stays focused on theme-toggle handling and related user feedback, with no clear unrelated changes.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

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