Skip to content

Comments

docs: document session management for streamable HTTP transport#674

Draft
DaleSeo wants to merge 1 commit intomodelcontextprotocol:mainfrom
DaleSeo:docs-session-management
Draft

docs: document session management for streamable HTTP transport#674
DaleSeo wants to merge 1 commit intomodelcontextprotocol:mainfrom
DaleSeo:docs-session-management

Conversation

@DaleSeo
Copy link
Contributor

@DaleSeo DaleSeo commented Feb 20, 2026

Motivation and Context

The SessionManager trait and the StreamableHttpService are key to how MCP servers manage stateful sessions over Streamable HTTP. However, the current documentation is lacking. There's no overview of the module, no explanation of what a "session" means here, and no examples on how to read the session ID or custom state in a tool handler.

This PR adds rustdoc documentation to close that gap.

How Has This Been Tested?

Documentation-only change. Verified that cargo doc builds without warnings and all existing tests continue to pass.

Breaking Changes

None.

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

Additional context

@DaleSeo DaleSeo self-assigned this Feb 20, 2026
@github-actions github-actions bot added T-core Core library changes T-examples Example code changes T-transport Transport layer changes labels Feb 20, 2026
@DaleSeo DaleSeo force-pushed the docs-session-management branch from b1d7e60 to cdedffd Compare February 20, 2026 21:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-core Core library changes T-examples Example code changes T-transport Transport layer changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant