fix(i18n): clarify tool permission title as 'Pre-approved Tools'#13327
Merged
fix(i18n): clarify tool permission title as 'Pre-approved Tools'#13327
Conversation
…tics The section title "Tools" was misleading — it implied the switches control tool availability, when they actually control pre-approval permissions. Updated all 11 locale files to say "Pre-approved Tools" (and equivalents) to match the code fallback text. Closes #13319 Co-Authored-By: Claude Opus 4.6 <[email protected]> Signed-off-by: icarus <[email protected]>
DeJeune
approved these changes
Mar 9, 2026
![cherry-ai-bot[bot]](https://avatars.githubusercontent.com/in/3013849?s=60&v=4){kind=small}
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What this PR does
Before this PR:
The
agent.settings.toolsMcp.tools.titlei18n key showed "Tools" (or equivalents), which implied the switches control tool availability (whether a tool can be used at all).After this PR:
The title now reads "Pre-approved Tools" (and equivalents in all locales), clearly indicating the switches control pre-approval permissions — tools not in the whitelist are still usable but require per-use approval at runtime.
Fixes #13319
Why we need it and why it was done in this way
The code fallback text was already
"Pre-approved Tools"but the i18n values were simplified to just"Tools", losing the important semantic meaning. This PR aligns all 11 locale files with the intended wording.The following tradeoffs were made: None — this is a straightforward i18n text correction.
The following alternatives were considered: N/A
Breaking changes
None.
Special notes for your reviewer
agent.settings.tools.*(which have zero code references) were left untouched — cleanup can be done separately.agent.settings.tools.titlevalues where correct, and newly translated where they were also wrong.Checklist
Release note