fix(client): modify message for public and discoverable toggle on engine settings

[johbaxter]

…ine settings

Description

Changes Made

How to Test

1. Steps to reproduce/test the behavior
2. Expected outcomes

Notes

[github-actions]

@CodiumAI-Agent /describe

[QodoAI-Agent]

Title

fix(client): modify message for public and discoverable toggle on engine settings

---

User description

…ine settings

Description

Changes Made

How to Test

1. Steps to reproduce/test the behavior
2. Expected outcomes

Notes

---

PR Type

• Enhancement

---

Description

• update notification messages dynamically

• toggle message text for discoverable feature

• toggle message text for global setting

---

Changes walkthrough 📝

	Relevant files
Enhancement	SettingsTiles.tsx Dynamic toggles for notification messages                                packages/client/src/components/settings/SettingsTiles.tsx Changed success message using conditional text. Updated error message using conditional text. Adjust messages for discoverable and global toggles. +12/-5
Additional files	pnpm-lock.yaml +653/-654

---

Need help?

Type /help how to ... in the comments thread for any questions about PR-Agent usage.

Check out the documentation for more information.

[github-actions]

@CodiumAI-Agent /review

[github-actions]

@CodiumAI-Agent /improve

[QodoAI-Agent]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Message Clarity Review that the notification message for the discoverable toggle correctly reflects the state change. Confirm that using the previous state to form the message does not confuse users. message: `Successfully made ${name} ${ discoverable ? 'undiscoverable' : 'discoverable' }`, }); } else { Message Clarity Validate that the global toggle messages accurately describe the result of the state change. Ensure consistency in behavior compared to similar toggles across the system. message: `Successfully made ${name} ${ global ? 'non-global' : 'global' }`, }); } else { notification.add({ color: 'error', message: `Error making ${name} ${ global ? 'non-global' : 'global' }`, });

[QodoAI-Agent]

PR Code Suggestions ✨

Category	Suggestion	Impact
General	Use toggled state for message Define and use a new variable holding the toggled state instead of relying on the current state for the message. packages/client/src/components/settings/SettingsTiles.tsx [283-288] +const newDiscoverable = !discoverable; +setDiscoverable(newDiscoverable); notification.add({ color: 'success', - message: `Successfully made ${name} ${ - discoverable ? 'undiscoverable' : 'discoverable' - }`, + message: `Successfully made ${name} ${newDiscoverable ? 'discoverable' : 'undiscoverable'}`, }); Suggestion importance[1-10]: 5 __ Why: The suggestion refactors the code by computing the toggled state explicitly, which improves clarity without altering behavior. However, it's a minor enhancement rather than a critical fix.	Low
	Use toggled state for global Similarly, compute the new global state in a variable and use it in the notification message for clarity. packages/client/src/components/settings/SettingsTiles.tsx [345-350] +const newGlobal = !global; +setGlobal(newGlobal); notification.add({ color: 'success', - message: `Successfully made ${name} ${ - global ? 'non-global' : 'global' - }`, + message: `Successfully made ${name} ${newGlobal ? 'global' : 'non-global'}`, }); Suggestion importance[1-10]: 5 __ Why: This suggestion applies the same clarity improvement to the global state logic, making the code easier to understand. It’s a modest improvement that maintains the intended functionality.	Low

[github-actions]

@CodiumAI-Agent /update_changelog

[QodoAI-Agent]

Changelog updates: 🔄

2025-03-17 *

Fixed

• Updated toggle notification messages in engine settings and bumped dependency versions.

to commit the new content to the CHANGELOG.md file, please type:
'/update_changelog --pr_update_changelog.push_changelog_changes=true'