Skip to content

Conversation

Mansehej
Copy link

@Mansehej Mansehej commented Sep 8, 2025

Introduces a seconds-based initialization timeout for MCP. Supports both per‑server and global configuration with clear precedence.

Behavior

  • Precedence:
    • Per‑server: mcp_servers..initialize_timeout_s
    • Global: mcp_initialize_timeout_s
    • Default: 10 seconds

Changes

  • core
    • config_types: add initialize_timeout_s to McpServerConfig
    • config: add mcp_initialize_timeout_s to Config/ConfigToml and wire through
    • mcp_connection_manager: accept global timeout and apply precedence
    • codex: pass global timeout into connection manager
  • docs
    • config.md and advanced.md updated with new keys and examples

Testing

  • cargo fmt
  • cargo clippy --tests -p codex-core
  • cargo test --all-features

Related issues

and potentially more.

Copy link

github-actions bot commented Sep 8, 2025


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.

@Mansehej
Copy link
Author

Mansehej commented Sep 8, 2025

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

@Mansehej
Copy link
Author

Mansehej commented Sep 8, 2025

recheck

@Mansehej
Copy link
Author

Mansehej commented Sep 9, 2025

Closing because another implementation solving the same problem is now merged.

@Mansehej Mansehej closed this Sep 9, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Sep 9, 2025
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