Skip to content

[MCP caching] split MCP connections to a backend + lightweight handle#15256

Open
jgershen-oai wants to merge 2 commits intomainfrom
codex/jgershen-mcp-backend-session-boundary
Open

[MCP caching] split MCP connections to a backend + lightweight handle#15256
jgershen-oai wants to merge 2 commits intomainfrom
codex/jgershen-mcp-backend-session-boundary

Conversation

@jgershen-oai
Copy link
Collaborator

@jgershen-oai jgershen-oai commented Mar 20, 2026

Summary

Split MCP connection ownership into a (to-be-shared) backend and a lightweight session-local client.
This PR keeps all other behavior unchanged; the motivation is to be able to cache the connection.

Testing

  • cargo check -p codex-core
  • cargo test -p codex-core mcp_connection_manager::tests::

@jgershen-oai jgershen-oai changed the title refactor(core): split MCP backend from session state [MCP caching] split MCP connections to a shared backend + client frontend Mar 20, 2026
@jgershen-oai jgershen-oai changed the title [MCP caching] split MCP connections to a shared backend + client frontend [MCP caching] split MCP connections to a backend + lightweight client Mar 20, 2026
@etraut-openai etraut-openai added the oai PRs contributed by OpenAI employees label Mar 20, 2026
@jgershen-oai jgershen-oai force-pushed the codex/jgershen-mcp-backend-session-boundary branch from 55e4394 to 7591ac9 Compare March 20, 2026 07:11
@jgershen-oai jgershen-oai marked this pull request as ready for review March 20, 2026 07:40
@jgershen-oai jgershen-oai requested a review from pakrym-oai March 20, 2026 07:41
@jgershen-oai jgershen-oai changed the title [MCP caching] split MCP connections to a backend + lightweight client [MCP caching] split MCP connections to a backend + lightweight handle Mar 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

oai PRs contributed by OpenAI employees

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants