Skip to content

Keep MCP event subscriptions alive after task unloading - #41899

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

Keep MCP event subscriptions alive after task unloading#41899
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/e102aa382e63121661ff0567f7523515e459922d

Conversation

@copyberry

@copyberry copyberry Bot commented Aug 31, 2026

Copy link
Copy Markdown

Keep MCP event subscriptions alive after task unloading

Why

Event subscriptions may need to outlive the MCP runtime associated with a task.

What changed

  • Add McpEventStreamOpener, which retains the hosted Apps MCP connection settings and opens each event stream with an independent client.
  • Keep subscription cancellation tied to account changes and removal of the task's event server, including when cancellation ownership is forwarded across runtime replacement.
  • Expose the opener and cancellation forwarding through McpResourceClient.

Testing

Add an HTTP transport test that shuts down the runtime, refreshes credentials for the same account, and verifies that a new event stream connects with the current authorization and receives notifications.

## Why

Event subscriptions may need to outlive the MCP runtime associated with a task.

## What changed

- Add `McpEventStreamOpener`, which retains the hosted Apps MCP connection settings and opens each event stream with an independent client.
- Keep subscription cancellation tied to account changes and removal of the task's event server, including when cancellation ownership is forwarded across runtime replacement.
- Expose the opener and cancellation forwarding through `McpResourceClient`.

## Testing

Add an HTTP transport test that shuts down the runtime, refreshes credentials for the same account, and verifies that a new event stream connects with the current authorization and receives notifications.

GitOrigin-RevId: e102aa382e63121661ff0567f7523515e459922d
@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/e102aa382e63121661ff0567f7523515e459922d branch from 2fb00bd to 53691ba Compare August 31, 2026 19:18
@copyberry
copyberry Bot merged commit 53691ba into main Aug 31, 2026
20 of 26 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/e102aa382e63121661ff0567f7523515e459922d branch August 31, 2026 19:20
@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.

2 participants