feat: Streamable HTTP support #643
Conversation
|
good work, let's get this added in! |
|
@rm-openai Can you please review this? |
src/agents/mcp/server.py
Outdated
| @@ -105,7 +106,14 @@ async def connect(self): | |||
| """Connect to the server.""" | |||
| try: | |||
| transport = await self.exit_stack.enter_async_context(self.create_streams()) | |||
| read, write = transport | |||
| # Handle different transport return values | |||
There was a problem hiding this comment.
Can you add some unit tests to make sure this works?
There was a problem hiding this comment.
@rm-openai Updated the check to discard the extra values and remove hard-coded checks. Do you want me to add unit tests right away or can I take that in a subsequent PR?
|
Also you'll probably need to increase the min version of |
It's already updated to 1.8.0 |
|
hmm can you run lint/fix typechecking? |
|
@rm-openai Fixed lint and typechecking issues. |
|
Been looking forward to this. Can't wait to try it in the next release. Thanks everyone. |
No description provided.