inspector/pr-screenshots
Repository files (latest commit first)
Filename Latest commit message Latest commit date
Cliff Hall e788bcbaca
web: pin Connection Info modal header while body scrolls (#1754) (#1755)
Same fix as #1698, applied to the Connection Info modal, which had the same
pattern: a custom header Group (title + close) inside a Stack in the modal body,
using the default `<Modal>` with no `scrollAreaComponent`. With a long server
`instructions` section (e.g. the everything server) the details overflow the
viewport and the whole modal scrolled, taking the header out of view.

Switch to the compound `Modal.Root`/`Content`/`Header`/`Body` API, move the
header into `Modal.Header` (sticky by design) and set
`scrollAreaComponent={ScrollArea.Autosize}` so only the body scrolls. The
fade-down transition is supplied app-wide by `ThemeModalRoot` (added in #1753),
so no `transitionProps` literal is needed here.

Closes #1754


Claude-Session: https://claude.ai/code/session_01XNdjEPKLG637X8YmhDiEk5

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
2026-07-23 22:08:04 -04:00
..
adv-ext-own-section-conninfo.png Extensions UI: own Advertised Extensions section + 'Client Advertised Extensions' heading (#1748) 2026-07-23 12:10:13 -04:00
adv-ext-own-section.png Extensions UI: own Advertised Extensions section + 'Client Advertised Extensions' heading (#1748) 2026-07-23 12:10:13 -04:00
advertised-ext-conninfo-legacy.png Extensions: Connection Info both eras + io.modelcontextprotocol/ui advertisement (Phase 3 of #1633) (#1744) 2026-07-23 10:11:36 -04:00
advertised-ext-conninfo-tasksoff.png Extensions: Connection Info both eras + io.modelcontextprotocol/ui advertisement (Phase 3 of #1633) (#1744) 2026-07-23 10:11:36 -04:00
advertised-ext-settings-tasks-off.png Extensions: advertised-extensions toggle in Server Settings (Phase 2 of #1633) (#1743) 2026-07-22 16:23:43 -04:00
advertised-ext-settings-tasks-on.png Extensions: advertised-extensions toggle in Server Settings (Phase 2 of #1633) (#1743) 2026-07-22 16:23:43 -04:00
advertised-ext-tools-tasks-off.png Extensions: advertised-extensions toggle in Server Settings (Phase 2 of #1633) (#1743) 2026-07-22 16:23:43 -04:00
advertised-ext-tools-tasks-on.png Extensions: advertised-extensions toggle in Server Settings (Phase 2 of #1633) (#1743) 2026-07-22 16:23:43 -04:00
ci-1754-header-pinned-scrolled.png web: pin Connection Info modal header while body scrolls (#1754) (#1755) 2026-07-23 22:08:04 -04:00
ci-1754-header-pinned-top.png web: pin Connection Info modal header while body scrolls (#1754) (#1755) 2026-07-23 22:08:04 -04:00
ema-client-settings-idp-fields.png Server Settings: disambiguate EMA OAuth credential fields (#1692) (#1749) 2026-07-23 16:51:24 -04:00
ema-oauth-fields-plain.png Server Settings: disambiguate EMA OAuth credential fields (#1692) (#1749) 2026-07-23 16:51:24 -04:00
ema-oauth-fields-resource-as.png Server Settings: disambiguate EMA OAuth credential fields (#1692) (#1749) 2026-07-23 16:51:24 -04:00
logging-legacy-era.png Logging: per-request log level on modern servers (era fork) (#1629) (#1724) 2026-07-19 17:08:22 -04:00
logging-modern-era-control.png Logging: per-request log level on modern servers (era fork) (#1629) (#1724) 2026-07-19 17:08:22 -04:00
logging-modern-era-loglevel-stamp.png Logging: per-request log level on modern servers (era fork) (#1629) (#1724) 2026-07-19 17:08:22 -04:00
logging-modern-server-setting.png Logging: per-request log level on modern servers (era fork) (#1629) (#1724) 2026-07-19 17:08:22 -04:00
README.md web: pin Connection Info modal header while body scrolls (#1754) (#1755) 2026-07-23 22:08:04 -04:00
ssm-1698-header-pinned-scrolled.png web: pin Server/Client Settings modal header while body scrolls (#1753) 2026-07-23 19:57:03 -04:00
ssm-1698-header-pinned-top.png web: pin Server/Client Settings modal header while body scrolls (#1753) 2026-07-23 19:57:03 -04:00
subscriptions-legacy-live-update.png Resources: subscriptions/listen model on modern servers (era fork) (#1630) (#1730) 2026-07-20 19:33:40 -04:00
subscriptions-legacy-resources-subscribe.png Resources: subscriptions/listen model on modern servers (era fork) (#1630) (#1730) 2026-07-20 19:33:40 -04:00
subscriptions-modern-header-badge.png Resources: subscriptions/listen model on modern servers (era fork) (#1630) (#1730) 2026-07-20 19:33:40 -04:00
subscriptions-modern-listen-acknowledged.png Resources: subscriptions/listen model on modern servers (era fork) (#1630) (#1730) 2026-07-20 19:33:40 -04:00
tasks-legacy-cancel-sticky.png Tasks: io.modelcontextprotocol/tasks extension on modern servers (era fork) (#1731) 2026-07-21 19:17:04 -04:00
tasks-legacy-run-as-task.png Tasks: io.modelcontextprotocol/tasks extension on modern servers (era fork) (#1731) 2026-07-21 19:17:04 -04:00
tasks-modern-completed.png Tasks: io.modelcontextprotocol/tasks extension on modern servers (era fork) (#1731) 2026-07-21 19:17:04 -04:00
tasks-modern-connected.png Tasks: io.modelcontextprotocol/tasks extension on modern servers (era fork) (#1731) 2026-07-21 19:17:04 -04:00
tasks-modern-input-completed.png Tasks: io.modelcontextprotocol/tasks extension on modern servers (era fork) (#1731) 2026-07-21 19:17:04 -04:00
tasks-modern-input-required.png Tasks: io.modelcontextprotocol/tasks extension on modern servers (era fork) (#1731) 2026-07-21 19:17:04 -04:00
tasks-modern-loop-cancelled.png Tasks: io.modelcontextprotocol/tasks extension on modern servers (era fork) (#1731) 2026-07-21 19:17:04 -04:00
tui-1751-after-select.png tui: detect array-of-enum on items.enum alone (#1752) 2026-07-23 19:12:45 -04:00
tui-1751-before-string.png tui: detect array-of-enum on items.enum alone (#1752) 2026-07-23 19:12:45 -04:00
tui-enumnames-after-cats.png web+tui: honor enumNames in enum forms (#1750) 2026-07-23 18:21:53 -04:00
tui-enumnames-before-raw.png web+tui: honor enumNames in enum forms (#1750) 2026-07-23 18:21:53 -04:00
xmcpheader-excluded-reason.png Tools: x-mcp-header annotation tooling + -32602 unknown-tool rendering (#1632) (#1735) 2026-07-22 11:42:54 -04:00
xmcpheader-excluded-tools.png Tools: x-mcp-header annotation tooling + -32602 unknown-tool rendering (#1632) (#1735) 2026-07-22 11:42:54 -04:00
xmcpheader-invalid-params.png Tools: x-mcp-header annotation tooling + -32602 unknown-tool rendering (#1632) (#1735) 2026-07-22 11:42:54 -04:00
xmcpheader-mirrored-headers.png Tools: x-mcp-header annotation tooling + -32602 unknown-tool rendering (#1632) (#1735) 2026-07-22 11:42:54 -04:00
xmcpheader-unknown-tool.png Tools: x-mcp-header annotation tooling + -32602 unknown-tool rendering (#1632) (#1735) 2026-07-22 11:42:54 -04:00

Connection Info modal: sticky header while body scrolls (#1754) — proof screenshots

Same fix as #1698, applied to the Connection Info modal. With a real server whose instructions are long (e.g. the everything server in a typical config), the connection details overflow the viewport; before the fix the whole modal scrolled and the header left the view. Captured from the ConnectionInfoModal Storybook story (WithOAuth, ~620px viewport) via Playwright.

Connection Info scrolled to the top — header pinned, Server Implementation visible

Top of scroll: the Connection Info header (title + close) sits above the first section.

Connection Info scrolled down — same header still pinned while the body shows Server Instructions / OAuth Details

Scrolled down (now showing Server Instructions → OAuth Details): the same header stays pinned. Measured in-page: the body scroll container is scrollHeight 1172 × clientHeight 558 (scrolls internally), header position: sticky flush to the dialog top.


Server/Client Settings modal: sticky header while body scrolls (#1698) — proof screenshots

Expanding enough accordion sections grew the Server Settings modal past the viewport, scrolling the whole modal and taking the header (title + close) out of view. The fix moves the header into a real Modal.Header (sticky by design) and adds scrollAreaComponent={ScrollArea.Autosize} so only Modal.Body scrolls. Captured from the ServerSettingsModal Storybook story (FullyConfigured, all sections expanded, ~680px viewport) via Playwright.

Modal scrolled to the top — header pinned, Options section visible

At the top of the scroll: the Server Settings header (collapse-all toggle, title, close) sits above the first section.

Modal scrolled down — same header still pinned while the body shows Custom Headers / Request Metadata

Scrolled down through the body (now showing Advertised Extensions → Custom Headers → Request Metadata): the exact same header stays pinned at the top. Measured in-page: the body scroll container is scrollHeight 2253 × clientHeight 612 (scrolls internally), and the header is position: sticky with its top flush to the dialog top after scrolling.


TUI array-of-enum renders as a select on items.enum alone (#1751) — proof screenshots

Follow-up to #1691. The TUI schemaToForm array-of-enum branch was nested under if (property.enum), so a standard array-of-enums schema — { type: "array", items: { enum, enumNames } } with no top-level enum — fell through to a plain text input. The fix keys the array case on items.enum alone (matching the web SchemaForm). Captured non-interactively (a standalone render of the real ink-form built by the real schemaToForm).

The demo schema has a Sizes array-of-enum field (enum: [s,m,l] + enumNames: [Small, Medium, Large], default: "m", no top-level enum) and a Priority single-enum control (has a top-level enum).

Before #1751 — the array-of-enum field is a plain text input showing the raw default "m"

Before the fix (reproducing the old nesting): Sizes is a plain text field showing the raw default m — enumNames never applies because it isn't a select at all.

With the #1751 fix — the array-of-enum field is a select resolving "m" to its title "Medium"

With the fix: Sizes is now a select; the default m resolves to its title Medium. The Priority control (which has a top-level enum) shows High in both — unchanged, confirming the reorder only affects the intended items.enum-only case.


TUI enum forms honor enumNames (#1691) — proof screenshots

The TUI tool-test form (schemaToForm → ink-form) rendered against a legacy titled enum: enum: [pet-1 … pet-5] paired with the non-standard enumNames: [Cats … Reptiles], default: "pet-1". Captured non-interactively (a standalone render of the real ink-form built by the real schemaToForm; the Ink TUI can't take live keystrokes in a headless pty).

TUI form with the #1691 fix — the required field resolves pet-1 to its title "Cats"

With the fix: the required Favorite Pet field resolves the default pet-1 to its human title "Cats". The submitted value stays the raw pet-1 — only the display label changes. (Plain Enum, which has no enumNames, is unaffected.)

The same form before #1691 — the field shows the raw wire value "pet-1"

Before the fix (same schema with enumNames stripped, reproducing the old behavior): the field shows the opaque wire value "pet-1" with no indication it means "Cats".

Ground-truth options from the real schemaToForm confirm the label↔value pairing and the length-guarded fallbacks: pet → Cats=>pet-1 … Reptiles=>pet-5; array-of-enum sizes → Small=>s | Medium=>m | Large=>l; a mismatched-length enumNames (2 names, 3 values) falls back to raw a=>a | b=>b | c=>c. The web side of this fix is covered by SchemaForm.test.tsx.


Server Settings OAuth field disambiguation under EMA (#1692) — proof screenshots

The Server Settings → OAuth Settings fields, rendered from the ServerSettingsForm Storybook stories. Under enterprise-managed authorization (EMA) these fields hold the resource authorization server credentials (leg 3 — its registered client), not the app/IdP pair configured in Client Settings. The unqualified labels sat directly under a toggle naming the enterprise IdP, making it easy to paste the wrong pair. See #1692.

OAuth Settings with EMA off — plain Client ID / Client Secret

Enterprise-managed authorization off (the common case): the fields keep the plain Client ID / Client Secret labels. The Scopes field now documents its delimiter — "Space-separated OAuth scopes (RFC 6749). Do not use commas — a comma-separated entry is sent as one invalid token and rejected by the authorization server." — with a mcp tools:read env:read placeholder.

OAuth Settings with EMA on — Resource AS Client ID / Secret

Enterprise-managed authorization on: the same two fields relabel to Resource AS Client ID / Resource AS Client Secret, each gaining a description that names the resource authorization server (its registered client, EMA leg 3) and points the app client id/secret back to Client Settings.

Client Settings modal — IdP Client ID / IdP Client Secret

The reciprocal side in the Client Settings modal (Enterprise-Managed Authorization section). Its credential fields are relabeled IdP Client ID / IdP Client Secret — parallel to Server Settings' Resource AS … — and each description now points the other way: these are the enterprise IdP pair (EMA legs 1–2), not the per-server resource authorization server credentials, which go in Server Settings → OAuth Settings. Together the two modals close the loop: each names its pair and where the other lives.


Connection Info extensions + io.modelcontextprotocol/ui (#1740) — proof screenshots

End-to-end verification of the Phase 3 Connection Info UI against the legacy advertised-extensions-http.json test server (port 3220), driven in a real browser through the web client's remote-proxy transport.

Connection Info on a legacy connection — Advertised Extensions shows tasks + ui

The Connection Info modal on a legacy connection (Era: LEGACY, no Discovery section). The new Server Extensions / Advertised Extensions two-column section renders — proving it is era-transparent (before #1740, extensions only appeared in the modern-only Discovery section). Advertised Extensions shows io.modelcontextprotocol/tasks, io.modelcontextprotocol/ui, confirming the Inspector now advertises the MCP Apps ui extension. (Server Extensions is — because this test server advertises none server-side.)

Connection Info after toggling Tasks off — Advertised Extensions shows only ui

After unchecking Tasks in Server Settings → Advertised Extensions (Phase 2) and reconnecting, the Advertised Extensions column shows only io.modelcontextprotocol/ui — the display reflects the per-server override live, and the ui advertisement persists (it is a separate registry entry).

Note: the Connection Info modal is populated from the connection's initializeResult, which requires serverInfo. On the modern test server used here serverInfo wasn't populated, so the modal stays closed on that modern connection — a pre-existing connection-layer behavior, independent of this PR (which only changes content inside the modal). The era-transparent Server Extensions path is covered by unit tests (ConnectionInfoContent.test.tsx renders legacy server extensions with no discoverResult).


Advertised-extensions toggle (#1739) — proof screenshots

End-to-end verification of the advertised-extensions debugging knob against a real legacy HTTP test server (test-servers/configs/advertised-extensions-http.json on port 3220), driven in a real browser through the web client's remote-proxy transport. The server registers echo (always) and get_weather gated on the io.modelcontextprotocol/tasks extension (extensionGatedTools): the tool is enabled on notifications/initialized only when the connected client declared that extension.

Advertised Extensions — Tasks advertised (default)

Server Settings → Advertised Extensions (the new section). With no override, Tasks (io.modelcontextprotocol/tasks) is checked — the registry default — so the Inspector advertises it in its client capabilities.

Tools with Tasks advertised

Connected with Tasks advertised. The server sees the declared extension on initialized and enables the gated tool, so tools/list returns both echo and get_weather.

Advertised Extensions — Tasks unchecked

Unchecking Tasks writes the per-server advertisedExtensions override. The change takes effect on the next connect (as the section note says).

Tools with Tasks not advertised

After reconnecting, the client advertises no extensions, so the server never enables the gated tool — tools/list now returns only echo. The server's tool registration demonstrably changed based on what the client advertised, exactly the acceptance criterion for #1739.


x-mcp-header Tools tooling (#1632) — proof screenshots

End-to-end verification of the SEP-2243 x-mcp-header Tools tooling against a real modern (2026-07-28) HTTP test server (test-servers/configs/xmcpheader-modern-http.json on port 3120), driven in a real browser through the web client's remote-proxy transport.

Excluded tools in the sidebar

Connected with Protocol Era = Modern. invalid_header_tool is dropped from tools/list by the SDK (its x-mcp-header annotation is invalid), so the Inspector re-lists the raw list and surfaces it struck-through under an "Excluded (SEP-2243)" divider — showing why a tool vanished rather than silently omitting it.

Exclusion reason on hover

Hovering the excluded tool shows the exact scan reason: the header name "Bad Header" contains a space, so it is not a valid RFC 9110 token.

Mirrored request headers

get_weather's detail panel shows the "Mirrored request headers (SEP-2243)" section: its city argument mirrors to Mcp-Param-City, with the note that the SDK omits Mcp-Param-* on the browser transport.

Unknown tool -32602

Calling a tool the server no longer recognizes rejects with -32602 (SDK v2) instead of an isError result, and renders as an "Unknown Tool" error panel with a targeted hint (reproduced by swapping the sessionless server to one without echo while the cached list still showed it).

Invalid params -32602

-32602 is the generic Invalid params code, so a known tool rejected for bad arguments throws the same code as an unknown tool. The panel disambiguates from the message: a -32602 that does not name an unknown tool renders under "Invalid Parameters" (with a schema hint) rather than "Unknown Tool". Triggered live via the trigger_invalid_params tool, which returns a real -32602 JSON-RPC error whose message is not about a missing tool.


Tasks extension era fork (#1631) — proof screenshots

End-to-end verification of the Tasks era fork against two real test servers (test-servers/configs/tasks-modern-http.json on port 3222 and tasks-legacy-http.json on 3223), driven in a real browser through the web client's remote-proxy transport.

Modern era (2026-07-28, io.modelcontextprotocol/tasks extension)

Modern connected, Tasks tab present

Connected with Protocol Era = Modern (MCP 2026-07-28). The Tasks tab appears in the monitoring sidebar because the io.modelcontextprotocol/tasks extension was negotiated (it is empty until a task runs).

Modern task completed

Connected with Protocol Era = Modern (MCP 2026-07-28). The Tasks tab is gated on the negotiated io.modelcontextprotocol/tasks extension (not capabilities.tasks). Running modern_task with Run as task on issues a tools/call that returns a CreateTaskResult (resultType: "task"), then polls tasks/get (no tasks/list); the completed task inlines its result (no blocking tasks/result) — shown both in the Results panel and the Tasks card.

Modern input_required → tasks/update

modern_input_task moves to input_required: the tasks/get response's inputRequests map (visible in the task's Full Task Object) carries an embedded elicitation/create, surfaced through the same pending-request modal the MRTR path uses — note the accurate wording "your answer is submitted via a tasks/update request (SEP-2663), not a retry". Answering it sends tasks/update with the inputResponses, and the next poll completes the task:

Modern input task completed

Legacy era (2025-11-25, contrast — unchanged)

Legacy run-as-task

Connected with Protocol Era = Legacy. The Tasks tab is gated on capabilities.tasks; simple_task is taskSupport: "required" so Run as task is forced on. The legacy flow uses tasks/list to populate the list, tasks/get to poll, and the blocking tasks/result to fetch the payload ({ "message": "Task completed: no message", "taskId": … }). Note the legacy-only Logs tab (this server advertises logging), absent from the modern monitor set.

Notes

  • SDK v2 removed all tasks support and era-gates the tasks/* spec methods out of the 2026-07-28 era on both the client (outbound) and server (inbound), and its codec rejects a resultType: "task" result outright. So the Inspector drives the extension itself: the task-creation frame is rewritten at the transport into a CallToolResult carrying the handle (the true resultType: "task" frame is still logged to the Protocol/Network tabs), and tasks/get / tasks/update / tasks/cancel ride a raw-wire request channel that carries the full modern envelope and is consumed by the transport before the SDK Client sees it. The test server serves tasks/* from an Express interceptor ahead of the SDK handler (the SDK's modern leg would answer them -32601).
  • On modern, task creation is server-directed (SEP-2663): the client declares the extension once and any tool may return a task, so the Tools screen offers Run as task for every tool on a modern connection (rather than gating on the legacy per-tool taskSupport).