Repository navigation
fix(ui): restore MCP catalog provider logos - #40781
Conversation
Greptile SummaryThis PR restores seven MCP catalog provider logos by replacing unavailable CDN URLs and adds registry-driven regression coverage for root and subpath deployments
Confidence Score: 5/5The PR appears safe to merge, with the changed logo paths present and correctly resolved across supported server roots No actionable failures remain; the packaged assets, runtime path handling, test discovery, and production rendering path align with the intended fix
|
| Filename | Overview |
|---|---|
| litellm/proxy/mcp_registry.json | Replaces seven broken provider icon URLs with available packaged or official logo sources |
| ui/litellm-dashboard/src/app/(dashboard)/mcp-servers/_components/mcp_discovery.integration.test.tsx | Adds registry-driven logo regression coverage and moves the component test into the configured integration suite |
Reviews (1): Last reviewed commit: "fix(ui): restore MCP catalog provider lo..." | Re-trigger Greptile
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
|
QA passed: real browser, proxy-served UI, before ae6a4a2 vs after f5bf2da. All seven logos load at root and under /litellm. Session Proof
Steps: http://localhost:4000/ui/mcp-servers/, click "+ Add New MCP Server", scroll to Search, Web & Browser, Cloud naturalWidth before -> after: Exa 0 -> 1024, Tavily 0 -> 1024, Slack 0 -> 150, Twilio 0 -> 150, Playwright 0 -> 400, Browserbase 0 -> 200, AWS 0 -> 200. GitHub and Notion 150 on both. Selecting Exa prefills name Note: on the standalone |
a73454b
into
litellm_internal_staging
TLDR
Problem this solves:
How it solves it:
User Flow
Before: seven providers show initials instead of logos
https://litellm-domain/litellm/ui/mcp-servers/After: all seven providers display their logos
https://litellm-domain/litellm/ui/mcp-servers/Linear ticket
Resolves LIT-7390
Pre-Submission checklist
Screenshots / Proof of Fix
Real browser, local proxy and seeded database,
SERVER_ROOT_PATH=/litellm; no mocked catalog or images. The local development launcher needed a routing-only adaptation outside the repository to serve the subpath. Screenshots are attached to LIT-7390 and require Linear access. Independent browser QA also passed with screenshots for the same base and headBefore (ae6a4a2)
http://127.0.0.1:65123/litellm/ui/mcp-servers/and click + Add New MCP ServernaturalWidth === 0; GitHub and Notion have nonzero widthsAfter (f5bf2da)
http://127.0.0.1:64345/litellm/ui/mcp-servers/and click + Add New MCP Server/litellm/ui/assets/logos/Supporting checks
http://127.0.0.1:65324/ui/mcp-servers/: all seven logos load; Search filtering and Exa selection retain the expected name and URL/litellmroots/litellmtest_anthropic_with_responses_apisendsprevious_response_id="hi"and receives 403. Replaying the identical request on merge baseae6a4a2fgives the identical 403; this comes from fix(proxy): authorize every Responses API id, not only the ones the proxy issued #39548, already in stagingkeyBudgetWindow.spec.ts). Windows checks passed. Final check summary: 127 passed, one skipped, one inherited endpoint-test failureType
Bug Fix
Caveats
Low
/uiasset routesFinal Attestation