From 390ed46993695a91cd14092115f7cd5723963bd2 Mon Sep 17 00:00:00 2001 From: Shin Bot Date: Sat, 31 Jan 2026 19:28:25 +0000 Subject: [PATCH] litellm_fix(lint): remove unused ToolNameValidationResult imports Fixes ruff F401 errors in check_code_and_doc_quality CI job. **Regression introduced in:** 41ec820 (fix files) - added files with unused imports ## Problem ToolNameValidationResult is imported but never used in: - litellm/proxy/_experimental/mcp_server/mcp_server_manager.py - litellm/proxy/management_endpoints/mcp_management_endpoints.py ## Fix ```diff - ToolNameValidationResult, ``` Removed from both import statements. ## Changes - mcp_server_manager.py: -1 line (removed unused import) - mcp_management_endpoints.py: -1 line (removed unused import) --- litellm/proxy/_experimental/mcp_server/mcp_server_manager.py | 1 - litellm/proxy/management_endpoints/mcp_management_endpoints.py | 1 - 2 files changed, 2 deletions(-) diff --git a/litellm/proxy/_experimental/mcp_server/mcp_server_manager.py b/litellm/proxy/_experimental/mcp_server/mcp_server_manager.py index 0087bc05b25..5ae5b610805 100644 --- a/litellm/proxy/_experimental/mcp_server/mcp_server_manager.py +++ b/litellm/proxy/_experimental/mcp_server/mcp_server_manager.py @@ -67,7 +67,6 @@ from litellm.types.utils import CallTypes try: from mcp.shared.tool_name_validation import ( # type: ignore SEP_986_URL, - ToolNameValidationResult, validate_tool_name, ) except ImportError: diff --git a/litellm/proxy/management_endpoints/mcp_management_endpoints.py b/litellm/proxy/management_endpoints/mcp_management_endpoints.py index 1e06c1eace4..3ba841f07fd 100644 --- a/litellm/proxy/management_endpoints/mcp_management_endpoints.py +++ b/litellm/proxy/management_endpoints/mcp_management_endpoints.py @@ -60,7 +60,6 @@ except ImportError as e: if MCP_AVAILABLE: try: from mcp.shared.tool_name_validation import ( # type: ignore - ToolNameValidationResult, validate_tool_name, ) except ImportError: