From 02a9317a308bfb7eb3d8d06da41383af1189a17b Mon Sep 17 00:00:00 2001 From: Sameer Kankute Date: Fri, 15 May 2026 10:11:38 +0530 Subject: [PATCH] fix(mcp): dedupe load_servers_from_config alias block Removes accidental duplicate alias/mcp_aliases and get_server_prefix logic (fixes PLR0915 and avoids resetting alias after mapping). --- .../mcp_server/mcp_server_manager.py | 26 ------------------- 1 file changed, 26 deletions(-) diff --git a/litellm/proxy/_experimental/mcp_server/mcp_server_manager.py b/litellm/proxy/_experimental/mcp_server/mcp_server_manager.py index fd7bcf2fb5..bcf751ae4f 100644 --- a/litellm/proxy/_experimental/mcp_server/mcp_server_manager.py +++ b/litellm/proxy/_experimental/mcp_server/mcp_server_manager.py @@ -321,32 +321,6 @@ class MCPServerManager: )() name_for_prefix = get_server_prefix(temp_server) - # Use alias for name if present, else server_name - alias = server_config.get("alias", None) - - # Apply mcp_aliases mapping if provided - if mcp_aliases and alias is None: - # Check if this server_name has an alias in mcp_aliases - for alias_name, target_server_name in mcp_aliases.items(): - if ( - target_server_name == server_name - and alias_name not in used_aliases - ): - alias = alias_name - used_aliases.add(alias_name) - verbose_logger.debug( - f"Mapped alias '{alias_name}' to server '{server_name}'" - ) - break - - # Create a temporary server object to use with get_server_prefix utility - temp_server = type( - "TempServer", - (), - {"alias": alias, "server_name": server_name, "server_id": None}, - )() - name_for_prefix = get_server_prefix(temp_server) - server_url = server_config.get("url", None) or "" # Generate stable server ID based on parameters server_id = self._generate_stable_server_id(