From 2c6995ca4dfc23b93db311b59c1b4ead464658b1 Mon Sep 17 00:00:00 2001 From: Jeremy Rose <172423086+nornagon-openai@users.noreply.github.com> Date: Mon, 15 Dec 2025 11:40:40 -0800 Subject: [PATCH] exec-server: additional context for errors (#7935) Add a .context() on some exec-server errors for debugging CI flakes. Also, "login": false in the test to make the test not affected by user profile. --- codex-rs/exec-server/src/posix/escalate_client.rs | 5 ++++- codex-rs/exec-server/tests/suite/accept_elicitation.rs | 1 + 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/codex-rs/exec-server/src/posix/escalate_client.rs b/codex-rs/exec-server/src/posix/escalate_client.rs index bea4b6fa5..0594cd90f 100644 --- a/codex-rs/exec-server/src/posix/escalate_client.rs +++ b/codex-rs/exec-server/src/posix/escalate_client.rs @@ -51,7 +51,10 @@ pub(crate) async fn run(file: String, argv: Vec) -> anyhow::Result }) .await .context("failed to send EscalateRequest")?; - let message = client.receive::().await?; + let message = client + .receive::() + .await + .context("failed to receive EscalateResponse")?; match message.action { EscalateAction::Escalate => { // TODO: maybe we should send ALL open FDs (except the escalate client)? diff --git a/codex-rs/exec-server/tests/suite/accept_elicitation.rs b/codex-rs/exec-server/tests/suite/accept_elicitation.rs index a9c6978a0..9632800d6 100644 --- a/codex-rs/exec-server/tests/suite/accept_elicitation.rs +++ b/codex-rs/exec-server/tests/suite/accept_elicitation.rs @@ -104,6 +104,7 @@ prefix_rule( name: Cow::Borrowed("shell"), arguments: Some(object(json!( { + "login": false, "command": "git init .", "workdir": project_root_path.to_string_lossy(), }