Skip to content

Expose negotiated MCP protocol version#1330

Closed
shusingh wants to merge 1 commit into
modelcontextprotocol:mainfrom
shusingh:expose-mcp-protocol-version
Closed

Expose negotiated MCP protocol version#1330
shusingh wants to merge 1 commit into
modelcontextprotocol:mainfrom
shusingh:expose-mcp-protocol-version

Conversation

@shusingh
Copy link
Copy Markdown

Summary

  • Return mcpProtocolVersion from useConnection
  • Pass the negotiated protocol version through App to the sidebar
  • Show the protocol version in the connected server metadata when available
  • Add hook coverage for reading mcp-protocol-version from response headers

Fixes #1324

Tests

  • npm test -- --runTestsByPath src/lib/hooks/__tests__/useConnection.test.tsx --runInBand
  • npm run lint in client
  • npm run build in client
  • npx prettier --check client/src/App.tsx client/src/components/Sidebar.tsx client/src/lib/hooks/useConnection.ts client/src/lib/hooks/__tests__/useConnection.test.tsx

@cliffhall
Copy link
Copy Markdown
Member

cliffhall commented May 20, 2026

Hi @shusingh

Sorry, the issue you referenced is against our v2/main branch, a totally different codebase. We're currently not accepting feature PRs for V1, only security and bug fixes.

@cliffhall cliffhall closed this May 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Surface negotiated protocolVersion through useInspectorClient

2 participants