Skip to content

docs(config): clarify default agent thread limit#9

Draft
middleDuckAi wants to merge 1 commit intoupstream-main-20260324from
middleDuck/codex-10517-extend-the-configuration-reference-with-the
Draft

docs(config): clarify default agent thread limit#9
middleDuckAi wants to merge 1 commit intoupstream-main-20260324from
middleDuck/codex-10517-extend-the-configuration-reference-with-the

Conversation

@middleDuckAi
Copy link
Copy Markdown
Owner

What?

  • clarify the agents.max_threads default in the config source-of-truth comment
  • regenerate codex-rs/core/config.schema.json

Why?

  • current runtime config resolution defaults agents.max_threads to Some(6)
  • the generated schema still said omitting the field enforced no limit
  • the duplicate thread for #8702 points at docs-site content, but this repo-owned schema mismatch still exists on current main

How?

  • update the AgentsToml::max_threads doc comment in codex-rs/core/src/config/mod.rs
  • regenerate the checked-in schema fixture

Testing

  • cargo run -p codex-core --bin codex-write-config-schema
  • cargo test -p codex-core config_schema_matches_fixture -- --nocapture

Refs openai#10517.

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.

1 participant