[Fix] MCP - Ensure internal users can access /mcp and /mcp/ routes

[ishaan-jaff]

[Fix] MCP - Ensure internal users can access /mcp and /mcp/ routes

This PR ensures internal users can access the /mcp and /mcp/ endpoints by extending route-validation logic, updating route definitions, and adding related unit tests.

Relevant issues

Pre-Submission checklist

Please complete all items before asking a LiteLLM maintainer to review your PR

• I have Added testing in the tests/litellm/ directory, Adding at least 1 test is a hard requirement - see details
• I have added a screenshot of my new test passing locally
• My PR passes all unit tests on make test-unit
• My PR's scope is as isolated as possible, it only solves 1 specific problem

Type

🐛 Bug Fix
✅ Test

Changes

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
litellm	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 27, 2025 3:55pm

[Copilot]

Pull Request Overview

This PR ensures internal users can access the /mcp and /mcp/ endpoints by extending route-validation logic, updating route definitions, and adding related unit tests.

• Extend is_llm_api_route to allow /mcp and /mcp/
• Define mcp_routes in the route enum and include them in the combined routes list
• Add unit tests for the new MCP routes

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File	Description
tests/proxy_admin_ui_tests/test_route_check_unit_tests.py	Added assertions for /mcp and /mcp/ routes
litellm/proxy/proxy_config.yaml	Removed unrelated configuration sections (general_settings, etc.)
litellm/proxy/auth/route_checks.py	Integrated mcp_routes into is_llm_api_route logic
litellm/proxy/_types.py	Defined mcp_routes list and included it in the master routes set

Comments suppressed due to low confidence (3)

tests/proxy_admin_ui_tests/test_route_check_unit_tests.py:92

• [nitpick] Consider adding unit tests for generic subpaths (e.g., /mcp/foo) to verify that the placeholder route /mcp/{subpath} is correctly handled.

    assert RouteChecks.is_llm_api_route("/mcp") is True

litellm/proxy/auth/route_checks.py:198

• The mcp_routes list includes a placeholder pattern (/mcp/{subpath}), but check_route_access may only perform exact matches; consider adding fuzzy matching logic or parsing placeholders similar to OpenAI routes.

        if RouteChecks.check_route_access(