From 6fd423ea8840ccaa2ee0f0efd8d3fba8929e3e30 Mon Sep 17 00:00:00 2001 From: Katia Bazzi Date: Wed, 10 Jun 2026 17:27:25 -0700 Subject: [PATCH] Fix Guardian timeout message assertions --- codex-rs/core/src/mcp_tool_call_tests.rs | 2 +- codex-rs/core/tests/suite/guardian_review.rs | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/codex-rs/core/src/mcp_tool_call_tests.rs b/codex-rs/core/src/mcp_tool_call_tests.rs index 723948b7be..76a7d1e523 100644 --- a/codex-rs/core/src/mcp_tool_call_tests.rs +++ b/codex-rs/core/src/mcp_tool_call_tests.rs @@ -1701,7 +1701,7 @@ async fn guardian_review_decision_maps_to_mcp_tool_decision() { else { panic!("guardian timeout should carry a timeout message"); }; - assert!(message.contains("did not finish before its deadline")); + assert!(message.contains("Automatic approval review timed out")); assert!(!message.contains("unacceptable risk")); assert_eq!( mcp_tool_approval_decision_from_guardian( diff --git a/codex-rs/core/tests/suite/guardian_review.rs b/codex-rs/core/tests/suite/guardian_review.rs index 8f5d7e1adb..fb6ff08bec 100644 --- a/codex-rs/core/tests/suite/guardian_review.rs +++ b/codex-rs/core/tests/suite/guardian_review.rs @@ -147,7 +147,7 @@ async fn guardian_timeout_falls_back_to_manual_approval_end_to_end() -> Result<( approval .reason .as_deref() - .is_some_and(|reason| reason.contains("did not finish before its deadline")) + .is_some_and(|reason| reason.contains("Automatic approval review timed out")) ); test.codex