From 515f7054265806325ff9f3d6c938770daacd1678 Mon Sep 17 00:00:00 2001 From: Alex Daley Date: Mon, 15 Jun 2026 20:08:39 -0400 Subject: [PATCH] [codex] Fix missing response item metadata in tests (#28415) Summary - Add the two missing `metadata: None` initializers after #28355 made response-item metadata required. - Restore test compilation for `codex-core` and `codex-api` on main. Validation - `git diff --check` - `just fmt` (Rust formatting passed; unrelated Python formatter steps could not use the sandboxed shared `uv` cache) - Focused crate tests are running after PR creation. --- codex-rs/codex-api/tests/clients.rs | 1 + codex-rs/core/src/guardian/tests.rs | 1 + 2 files changed, 2 insertions(+) diff --git a/codex-rs/codex-api/tests/clients.rs b/codex-rs/codex-api/tests/clients.rs index 624d008e5..e362560ca 100644 --- a/codex-rs/codex-api/tests/clients.rs +++ b/codex-rs/codex-api/tests/clients.rs @@ -312,6 +312,7 @@ async fn responses_client_stream_request_preserves_exact_json_body() -> Result<( role: "user".into(), content: vec![ContentItem::InputText { text: "hi".into() }], phase: None, + metadata: None, }], tools: Vec::new(), tool_choice: "auto".into(), diff --git a/codex-rs/core/src/guardian/tests.rs b/codex-rs/core/src/guardian/tests.rs index da03eb95a..fefe15cb7 100644 --- a/codex-rs/core/src/guardian/tests.rs +++ b/codex-rs/core/src/guardian/tests.rs @@ -1696,6 +1696,7 @@ async fn guardian_review_request_layout_matches_model_visible_request_snapshot() ), }], phase: None, + metadata: None, }], ) .await;