Skip to content

Retain the MCP client for event streams - #41892

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/7d6c7b8ed9c5447d2d8cda4a077da845fb026173
Aug 31, 2026
Merged

Retain the MCP client for event streams#41892
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/7d6c7b8ed9c5447d2d8cda4a077da845fb026173

Conversation

@copyberry

@copyberry copyberry Bot commented Aug 31, 2026

Copy link
Copy Markdown

Retain the MCP client for event streams

What changed

  • Add McpEventStream::open to construct event stream requests and keep the
    underlying RmcpClient alive until the subscription completes or is canceled.
  • Stop retaining the broader McpServerConnection for the lifetime of an event
    stream, and remove the connection-returning lookup helper.
  • Use MCP event server terminology in event discovery and subscription errors.

## What changed

- Add `McpEventStream::open` to construct event stream requests and keep the
  underlying `RmcpClient` alive until the subscription completes or is canceled.
- Stop retaining the broader `McpServerConnection` for the lifetime of an event
  stream, and remove the connection-returning lookup helper.
- Use MCP event server terminology in event discovery and subscription errors.

GitOrigin-RevId: 7d6c7b8ed9c5447d2d8cda4a077da845fb026173
@github-actions

Copy link
Copy Markdown
Contributor


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/7d6c7b8ed9c5447d2d8cda4a077da845fb026173 branch from a9328d0 to e51b54b Compare August 31, 2026 18:24
@copyberry
copyberry Bot merged commit e51b54b into main Aug 31, 2026
26 of 32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/7d6c7b8ed9c5447d2d8cda4a077da845fb026173 branch August 31, 2026 18:26
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 31, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant