Skip to content

[Doc] Describe connect across both protocol lifecycles - #564

Merged
koic merged 1 commit into
modelcontextprotocol:mainfrom
koic:describe_connect_as_lifecycle_negotiation
Sep 20, 2026
Merged

koic merged 1 commit into
modelcontextprotocol:mainfrom
koic:describe_connect_as_lifecycle_negotiation

Conversation

@koic

@koic koic commented Sep 20, 2026

Copy link
Copy Markdown
Member

Motivation and Context

The client documentation equated MCP::Client#connect with the MCP initialization handshake, which MCP 2026-07-28 removed: a modern connection probes server/discover instead. Calling connect first is still required on either era, since it settles the lifecycle and registers the identity, capabilities, and protocol version that modern requests carry in _meta, so the wording now names that rather than the handshake.

How Has This Been Tested?

Without connect a modern server rejects the first request; with it the same call succeeds. The documentation site builds with the new wording.

Breaking Changes

None. The change is documentation prose and example comments.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

## Motivation and Context

The client documentation equated `MCP::Client#connect` with the MCP initialization handshake,
which MCP 2026-07-28 removed: a modern connection probes `server/discover` instead.
Calling `connect` first is still required on either era, since it settles the lifecycle
and registers the identity, capabilities, and protocol version that modern requests carry in `_meta`,
so the wording now names that rather than the handshake.

## How Has This Been Tested?

Without `connect` a modern server rejects the first request; with it the same call succeeds.
The documentation site builds with the new wording.

## Breaking Changes

None. The change is documentation prose and example comments.
@koic
koic merged commit 5ce47b6 into modelcontextprotocol:main Sep 20, 2026
11 checks passed
@koic
koic deleted the describe_connect_as_lifecycle_negotiation branch September 20, 2026 13:25
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.

1 participant