-
Notifications
You must be signed in to change notification settings - Fork 1.9k
feat: add individual tool controls to settings #5964
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
f825bff
to
276d61e
Compare
I love this! ive been running similar setup on my fork for a month almost but my UI isn't very good, additionally i have an option to make #MODES part also granular so each mode can select which mode should be visible in its system prompt ( this helps when u have multiple agents packaged with specific workflows so they do not conflict with each other ) |
This feature allows users to selectively disable tools they never plan to use, which removes those tool descriptions from the system prompt and saves context tokens. The implementation includes: - New settings schema for individual tool controls - New ToolsSettings UI component with checkboxes for each tool - Logic to filter tools based on these settings - Integration with the system prompt generation Fixes: #5963 Signed-off-by: Eric Wheeler <[email protected]>
This commit adds missing translations for the following locales: - French (fr) - Hindi (hi) - Indonesian (id) - Italian (it) - Japanese (ja) - Korean (ko) - Dutch (nl) - Polish (pl) - Brazilian Portuguese (pt-BR) - Russian (ru) - Turkish (tr) - Vietnamese (vi) - Simplified Chinese (zh-CN) - Traditional Chinese (zh-TW) All translations were added to the respective `settings.json` files and verified using the `find-missing-translations.js` script. Signed-off-by: Eric Wheeler <[email protected]>
276d61e
to
56049f5
Compare
I have not reviewed in depth but the MCP portion of the tools are already excluded if you have MCP disabled. This seems redundant. Same with apply_diff. |
Thanks for doing this @KJ7LNW ! We really need some way to start excluding tools at a project level. There are now hyper specific MCP's like "Apple Development Docs" - https://github.com/kimsungwhee/apple-docs-mcp that I really only want to use by flagging something for Swift projects only. |
Context
This PR adds the ability to selectively enable/disable individual tools in the settings UI. This allows users to remove tools they never plan to use from the system prompt, saving valuable context tokens.
Implementation
How to Test
Get in Touch
Discord: KJ7LNW
Fixes #5963
Important
This PR adds UI and backend support for enabling/disabling individual tools in settings, affecting system prompt generation and tool availability.
global-settings.ts
.ToolsSettings
component inToolsSettings.tsx
with checkboxes for each tool.ToolsSettings
intoSettingsView.tsx
.tools/index.ts
to filter tools based on new settings.generateSystemPrompt.ts
to incorporate tool settings.ExtensionStateContext.tsx
to include tool settings.webviewMessageHandler.ts
.settings.json
for tool-related strings.This description was created by
for f825bff. You can customize this summary. It will automatically update as commits are pushed.