scriptoriumadmin/litellm
1
0
Fork 0
mirror of https://github.com/BerriAI/litellm.git synced 2026-09-15 23:31:29 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 165

fix: enable/disable all operates on full tool set, not just filtered subset

Browse source
This commit is contained in:
Ishaan Jaffer 2026-03-09 20:53:18 -07:00
parent 50f510d8f6
commit 691ecc685b
1 changed files with 7 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
ui/litellm-dashboard/src/components/mcp_tools/mcp_tool_configuration.tsx
View file

@ -296,24 +296,27 @@ const MCPToolConfiguration: React.FC<MCPToolConfigurationProps> = ({
};
const handleEnableSuggested = () => {
const suggestedNames = pinnedFiltered.map((t) => t.name);
// Enable ALL suggested tools (not just the currently filtered subset)
const suggestedNames = suggestedTools.map((t) => t.name);
const others = allowedTools.filter((n) => !suggestedToolNames.has(n));
onAllowedToolsChange([...others, ...suggestedNames]);
};
const handleDisableSuggested = () => {
// Disable ALL suggested tools (not just the currently filtered subset)
onAllowedToolsChange(allowedTools.filter((n) => !suggestedToolNames.has(n)));
};
const handleEnableRest = () => {
const restNames = restFiltered.map((t) => t.name);
// Enable ALL non-suggested tools (not just the currently filtered subset)
const restNames = tools.filter((t) => !suggestedToolNames.has(t.name)).map((t) => t.name);
const current = new Set(allowedTools);
onAllowedToolsChange([...allowedTools, ...restNames.filter((n) => !current.has(n))]);
};
const handleDisableRest = () => {
const restNameSet = new Set(restFiltered.map((t) => t.name));
onAllowedToolsChange(allowedTools.filter((n) => !restNameSet.has(n)));
// Disable ALL non-suggested tools (not just the currently filtered subset)
onAllowedToolsChange(allowedTools.filter((n) => suggestedToolNames.has(n)));
};
// Don't show anything if required fields aren't filled