Skip to content

[1/8] Add MCP server environment config#17984

Open
aibrahim-oai wants to merge 1 commit intoopenai:mainfrom
aibrahim-oai:dev/remote-mcp-env-config
Open

[1/8] Add MCP server environment config#17984
aibrahim-oai wants to merge 1 commit intoopenai:mainfrom
aibrahim-oai:dev/remote-mcp-env-config

Conversation

@aibrahim-oai
Copy link
Copy Markdown
Collaborator

@aibrahim-oai aibrahim-oai commented Apr 15, 2026

Summary

  • Add an MCP server environment setting with local as the default.
  • Thread the default through config serialization, schema generation, and existing config fixtures.

Stack

o  #18027 [8/8] Fail exec client operations after disconnect
│
o  #18025 [7/8] Cover MCP stdio tests with executor placement
│
o  #17974 [6/8] Wire remote MCP stdio through executor
│
o  #17987 [5/8] Add executor process transport for MCP stdio
│
o  #17986 [4/8] Abstract MCP stdio server launching
│
o  #18020 [3/8] Add pushed exec process events
│
o  #17985 [2/8] Support piped stdin in exec process API
│
@  #17984 [1/8] Add MCP server environment config
│
o  main

Introduce the MCP server environment setting and thread the default local value through config serialization, schema generation, and existing config fixtures.

Co-authored-by: Codex <noreply@openai.com>
@aibrahim-oai aibrahim-oai changed the title Add MCP server environment config [1/5] Add MCP server environment config Apr 15, 2026
@aibrahim-oai aibrahim-oai changed the title [1/5] Add MCP server environment config [1/6] Add MCP server environment config Apr 15, 2026
@aibrahim-oai aibrahim-oai changed the title [1/6] Add MCP server environment config [1/7] Add MCP server environment config Apr 15, 2026
@aibrahim-oai aibrahim-oai changed the title [1/7] Add MCP server environment config [1/8] Add MCP server environment config Apr 16, 2026

/// Where Codex should start this MCP server.
#[serde(default, skip_serializing_if = "McpServerEnvironment::is_local")]
pub environment: McpServerEnvironment,
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

make it a string.
experimental_environment too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants