StreamableHttp - stateless server support #554
Conversation
examples/servers/simple-streamablehttp-stateless/mcp_simple_streamablehttp_stateless/server.py
Outdated
Show resolved
Hide resolved
src/mcp/server/lowlevel/server.py
Outdated
| # When True, the server will wait for the client to send an initialization | ||
| # message before processing any other messages. | ||
| # False should be used for stateless servers. | ||
| require_initialization: bool = True, |
There was a problem hiding this comment.
[nit / question] should we just spoof the initialization on these requests with capabilities that a stateless server would have (i.e. capabilities={})? that way the life cycle of the mcp server is still spec compliant - it's a bit of a pointless dance though, so I'm happy to leave it as you have it.
The code as it stands feels like this could be interpreted as initialization being optional - that's my main concern
There was a problem hiding this comment.
I think I'd probaly change the name, require_initialization sounds like we are skipping the protocol requirement on intialization
|
Can we change the module name please, to something that is pep 8 compatible, like |
ihrpr
dismissed stale reviews from dsp-ant and jerome3o-anthropic
The base branch was changed.
Tested with multi-client test script in Typescript introduced in this PR
uv run mcp-simple-streamablehttp-stateless --log-level DEBUGuv run mcp-simple-streamablehttp-stateless --json-responseFollow ups
Stacked on top of #553