fix(mcp): preserve source_url in GET /v1/mcp/server list responses

[shivamrawat1]

Resolves LIT-3425

Description
source_url was saved correctly when creating MCP servers, but GET /v1/mcp/server always returned "source_url": null. The single-server endpoint (GET /v1/mcp/server/{id}) worked because it reads directly from the database.

This PR threads source_url through the in-memory registry roundtrip used by the list endpoint.

Cause
The list endpoint builds its response from the in-memory MCP registry via:

LiteLLM_MCPServerTable → MCPServer → LiteLLM_MCPServerTable

source_url existed on the DB/API model but was never added to the runtime MCPServer model, so it was dropped when loading servers into the registry and when rebuilding list responses in _build_mcp_server_table().

Fix
Add source_url to the runtime MCPServer model
Map source_url in build_mcp_server_from_table() when loading from DB
Map source_url in _build_mcp_server_table() when building list responses
Add regression test: test_round_trip_source_url_preserved

Before:

After:

[greptile-apps]

Greptile Summary

This PR fixes GET /v1/mcp/server always returning "source_url": null by threading the field through the in-memory registry round-trip that the list endpoint relies on, while leaving the single-server GET /v1/mcp/server/{id} path (which reads directly from the DB) untouched.

• Adds source_url: Optional[str] = None to the runtime MCPServer model so it survives the LiteLLM_MCPServerTable → MCPServer → LiteLLM_MCPServerTable conversion chain.
• Maps source_url in both build_mcp_server_from_table() (DB → registry load) and _build_mcp_server_table() (registry → list response), mirroring the pattern used by neighboring fields like byok_api_key_help_url.
• Adds a focused in-memory regression test (test_round_trip_source_url_preserved) that exercises the full round-trip without real network calls.

Confidence Score: 5/5

Safe to merge — the change is a minimal, isolated field propagation fix with no side effects on other code paths.

All three touch points (type model, DB-to-registry loader, registry-to-response builder) are updated consistently, matching the pattern of every other field in those same conversion functions. The regression test covers the exact round-trip that was broken and runs entirely in-memory.

No files require special attention.

Important Files Changed

Filename	Overview
litellm/types/mcp_server/mcp_server_manager.py	Adds source_url: Optional[str] = None to the runtime MCPServer model, consistent with placement in _types.py.
litellm/proxy/_experimental/mcp_server/mcp_server_manager.py	Threads source_url through both conversion paths: build_mcp_server_from_table() (DB → registry) and _build_mcp_server_table() (registry → list response). Fix is minimal and targeted.
tests/test_litellm/proxy/_experimental/mcp_server/test_mcp_server_manager.py	Adds test_round_trip_source_url_preserved — a pure in-memory regression test covering the full LiteLLM_MCPServerTable → MCPServer → LiteLLM_MCPServerTable round-trip with no real network calls.

_{Reviews (1): Last reviewed commit: "fix(mcp): preserve source_url in GET /v1..." | Re-trigger Greptile}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[mateo-berri]

LGTM; thanks!