From aff1060512f59206ea76f18d246aeaea1a217438 Mon Sep 17 00:00:00 2001 From: YutaSaito <36355491+uc4w6c@users.noreply.github.com> Date: Tue, 11 Nov 2025 12:50:13 +0900 Subject: [PATCH] fix: unable to delete MCP server from permission settings #16124 (#16407) --- .../src/components/team/team_info.tsx | 31 +++++++++---------- 1 file changed, 15 insertions(+), 16 deletions(-) diff --git a/ui/litellm-dashboard/src/components/team/team_info.tsx b/ui/litellm-dashboard/src/components/team/team_info.tsx index 34446c51664..0e3c5e92e6b 100644 --- a/ui/litellm-dashboard/src/components/team/team_info.tsx +++ b/ui/litellm-dashboard/src/components/team/team_info.tsx @@ -357,23 +357,22 @@ const TeamInfoView: React.FC = ({ servers: [], accessGroups: [], }; - const mcpToolPermissions = values.mcp_tool_permissions || {}; + const serverIds = new Set(servers || []); + const mcpToolPermissions = Object.fromEntries( + Object.entries(values.mcp_tool_permissions || {}).filter(([serverId]) => + serverIds.has(serverId) + ) + ); - if ( - (servers && servers.length > 0) || - (accessGroups && accessGroups.length > 0) || - Object.keys(mcpToolPermissions).length > 0 - ) { - updateData.object_permission = {}; - if (servers && servers.length > 0) { - updateData.object_permission.mcp_servers = servers; - } - if (accessGroups && accessGroups.length > 0) { - updateData.object_permission.mcp_access_groups = accessGroups; - } - if (Object.keys(mcpToolPermissions).length > 0) { - updateData.object_permission.mcp_tool_permissions = mcpToolPermissions; - } + updateData.object_permission = {}; + if (servers) { + updateData.object_permission.mcp_servers = servers; + } + if (accessGroups) { + updateData.object_permission.mcp_access_groups = accessGroups; + } + if (mcpToolPermissions) { + updateData.object_permission.mcp_tool_permissions = mcpToolPermissions; } delete values.mcp_servers_and_groups; delete values.mcp_tool_permissions;