Skip to content

Commit a286130

Browse files
celia-oaibolinfest
andauthored
Update codex-rs/app-server/src/codex_message_processor.rs
Co-authored-by: Michael Bolin <[email protected]>
1 parent 3a4a023 commit a286130

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

codex-rs/app-server/src/codex_message_processor.rs

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3366,9 +3366,11 @@ fn errors_to_info(
33663366
/// Precedence (lowest to highest):
33673367
/// - `cli_overrides`: process-wide startup `--config` flags.
33683368
/// - `request_overrides`: per-request dotted-path overrides (`params.config`), converted JSON->TOML.
3369-
/// - `forced_overrides`: strongly-typed overrides derived from the conversation/thread parameters
3370-
/// (e.g. `cwd`, sandbox/approval policy, model/provider, instructions), plus a few app-server
3371-
/// supplied values (e.g. `codex_linux_sandbox_exe`).
3369+
/// - `typesafe_overrides`: Request objects such as `NewConversationParams` and
3370+
/// `ThreadStartParams` support a limited set of _explicit_ config overrides, so
3371+
/// `typesafe_overrides` is a `ConfigOverrides` derived from the respective request object.
3372+
/// Because the overrides are defined explicitly in the `*Params`, this takes priority over
3373+
/// the more general "bag of config options" provided by `cli_overrides` and `request_overrides`.
33723374
async fn derive_config_from_params(
33733375
cli_overrides: &[(String, TomlValue)],
33743376
request_overrides: Option<HashMap<String, serde_json::Value>>,

0 commit comments

Comments
 (0)