Repository navigation
Small defects in web, TUI and default catalog (tracking list from a v2.13 review) #2827
Copy link
Copy link
Closed
Labels
Milestone
Description
Which version line?
v2 — current (@modelcontextprotocol/inspector@latest)
Which client?
All / shared core
Inspector version
2.13.0 (v2/main @ d7f6cee, built from source)
Node version
v22.23.4
Operating system (and browser, for the web client)
Linux (Debian), Chrome for Testing 153 (headless, via Playwright)
Transport
stdio
MCP server under inspection
The repo's own stdio test server, default config:
node test-servers/build/test-server-stdio.js (built with npm run test-servers:build in clients/web).
Steps to reproduce
Each item lists its own repro; all on v2.13.0 built from source.
- Web fonts from Google at runtime. Load the web UI with no internet access → console shows
ERR_TUNNEL_CONNECTION_FAILEDforfonts.googleapis.com(clients/web/index.htmllines 7–9). With internet, every launch sends the user's IP to Google. - Sandbox listener (6275) robustness.
curl -I http://127.0.0.1:6275/sandbox→ 404;curl 'http://127.0.0.1:6275/sandbox?x=1'→ 404;curl -H 'Host: rebind.evil.com' http://127.0.0.1:6275/sandbox→ 200 (no Host allow-list, unlike the main server).sandbox-controller.tsmatchesreq.urlexactly. - SPA fallback prefix.
curl http://127.0.0.1:6274/apixyz→ 404; the fallback testspath.startsWith("/api")instead of"/api/"(clients/web/server/server.ts). - Ad-hoc server name differs by client. Same launch command: web names it
node, TUI names itdefault. - Default seed on Windows.
DEFAULT_SEED_CONFIGincore/mcp/serverList.tsseedsfilesystem-server-defaultwith/tmp, which does not exist on Windows. - No
<h1>after connecting. axe-corepage-has-heading-oneon Tools/Prompts/Resources/Apps; the home screen passes. - List order differs. Tools are listed in server order in the TUI, alphabetically in the web client.
- TUI nits. The tool form shows
Test Tool: <name>twice; a comment nearroots: cleanRoots(...)inclients/tui/src/App.tsxsays the TUI has no roots editor, but one shipped in TUI has no UI for Tasks, resource subscriptions, or Roots #2432. - Weak custom token accepted silently.
MCP_INSPECTOR_API_TOKEN=testtokis accepted with no warning.
Expected behavior
- Fonts are bundled with the app; no third-party requests.
- The sandbox answers HEAD and ignores a query string on /sandbox, and applies the same Host allow-list as the main server.
- Non-API deep links get the SPA; only /api/ paths are excluded.
- One naming rule for ad-hoc servers across clients.
- The seed uses a platform temp dir (os.tmpdir()).
- Every screen has an
.
- One list order in both clients (or a sort toggle in both).
- A single title on the tool form; the comment reflects the roots editor.
- A startup warning for a custom token shorter than 32 characters.
Actual behavior
See each numbered item in Steps to reproduce.
Logs, errors, or screenshots
axe-core scans were run with the copy shipped in clients/web/node_modules; all other axe rules passed on all five web tabs.
Already prototyped a fix?
No response
Before you submit
- I searched existing issues and this is not a duplicate.
- This is not a security vulnerability report (those go through the private advisory process).
Activity
cliffhall commentedon Oct 29, 2026
Triage: Priority Low (total 5)
buglabel (no milestone, assignee, epic, engagement or reporter-set priority)Board: #28, Status
Incoming(awaiting maintainer review — no milestone yet).fix: review round 1 — stable sorted TUI tools, Apps messages h2, bund…
Merge pull request #2856 from modelcontextprotocol/v2/fix/2827-v213-r…
cliffhall commentedon Oct 29, 2026
Fixed by #2856, merged into
v2/main; verified together with the rest of v2.14.0 in the rollup #2858.