Merge branch 'etraut/next-turn-state-remove-override-context' into etraut/next-turn-state-core

This commit is contained in:
Eric Traut
2026-05-16 17:58:11 -07:00
2 changed files with 3 additions and 4 deletions

View File

@@ -281,10 +281,9 @@ impl CodexThread {
pub async fn update_turn_context_overrides(
&self,
overrides: CodexThreadTurnContextOverrides,
) -> ConstraintResult<ThreadConfigSnapshot> {
) -> ConstraintResult<()> {
let updates = self.turn_context_settings_update(overrides).await;
self.codex.session.update_settings(updates).await?;
Ok(self.config_snapshot().await)
self.codex.session.update_settings(updates).await
}
async fn turn_context_settings_update(

View File

@@ -70,7 +70,7 @@ For complete documentation of the `Op` and `EventMsg` variants, refer to [protoc
- `Op::Interrupt` Interrupts a running turn
- `Op::ExecApproval` Approve or deny code execution
- `Op::UserInputAnswer` Provide answers for a `request_user_input` tool call
- `Op::UserTurn` and `Op::UserInputWithTurnContext` accept an optional `personality` override that updates the models communication style
- `Op::UserInput` accepts an optional `personality` turn-context override that updates the models communication style
Valid `personality` values are `friendly`, `pragmatic`, and `none`. When `none` is selected, the personality placeholder is replaced with an empty string.