inspector/core/mcp/index.ts
cliffhall fe5b622fd1 fix: salvage a list result's valid entries when one is malformed (#1909)
The SDK validates a list result as a whole, so a single non-conforming entry
rejected the entire response and the panel showed "Couldn't load resources"
with zero items — the exact opposite of what a debugging tool should do with a
server that is 90% right. Reported against a PHP server whose empty
`annotations` object reaches the wire as `[]` rather than `{}`.

`listAll*` now falls back to a lenient re-walk when — and only when — the SDK
rejects a result it received (`InvalidResult` / `UnsupportedResultType`), keeps
the entries that validate, and reports the rest. A conforming server takes the
strict, cache-aware path exactly as before.

Salvaging is not accepting: the dropped entries are named with their reason in
a warning above the list, and the Protocol entry is still marked rejected, so
the wire truth survives. When every entry validates, the original error is
rethrown rather than swallowed — the rejection was about something the
per-item pass cannot explain.

`isClientDecodeRejection` moves to the new module and is now shared with
managedListState, which had the only copy.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HDdo1rNRVQnVRRdsqSrvG3
Signed-off-by: cliffhall <cliff@futurescale.com>
2026-08-15 16:52:11 -04:00

60 lines
1.5 KiB
TypeScript

// Main MCP client module
// Re-exports the primary API for MCP client/server interaction
export { InspectorClient } from "./inspectorClient.js";
export type {
InspectorClientOptions,
InspectorClientEnvironment,
AppRendererClient,
} from "./types.js";
// Re-export type-safe event target types for consumers
export type { InspectorClientEventMap } from "./inspectorClientEventTarget.js";
export {
getOAuthServerUrl,
getServerType,
isOAuthCapableServerType,
} from "./config.js";
// Re-export types used by consumers
export type {
// Transport factory types (required by InspectorClient)
CreateTransport,
CreateTransportOptions,
CreateTransportResult,
// Config types
MCPConfig,
MCPServerConfig,
ServerType,
// Connection and state types (used by components and hooks)
ConnectionStatus,
StderrLogEntry,
MessageEntry,
FetchRequestEntry,
FetchRequestEntryBase,
FetchRequestCategory,
ServerState,
// Invocation types (returned from InspectorClient methods)
ResourceReadInvocation,
ResourceTemplateReadInvocation,
PromptGetInvocation,
ToolCallInvocation,
} from "./types.js";
// Per-item list salvage (#1909)
export type { MalformedListItem } from "./listSalvage.js";
// Re-export JSON utilities
export type { JsonValue } from "../json/jsonUtils.js";
export {
convertParameterValue,
convertToolParameters,
convertPromptArguments,
} from "../json/jsonUtils.js";
// Re-export session storage types
export type {
InspectorClientStorage,
InspectorClientSessionState,
} from "./sessionStorage.js";