From 6f77491e954cd23eeb2202a46aa4814c0dce5bad Mon Sep 17 00:00:00 2001 From: pakrym-oai Date: Tue, 16 Jun 2026 19:16:16 -0700 Subject: [PATCH] thread-store: fix response fixture compilation (#28642) ## Why A `codex-thread-store` test fixture still constructs `ResponseItem::FunctionCallOutput` without its required `metadata` field, preventing the crate's test targets from compiling on `main`. ## What changed - Set the fixture's response-item metadata to `None`. ## Testing - `cargo check -p codex-thread-store --tests` --- codex-rs/thread-store/src/local/mod.rs | 1 + 1 file changed, 1 insertion(+) diff --git a/codex-rs/thread-store/src/local/mod.rs b/codex-rs/thread-store/src/local/mod.rs index 5670e5638..550e7d56b 100644 --- a/codex-rs/thread-store/src/local/mod.rs +++ b/codex-rs/thread-store/src/local/mod.rs @@ -512,6 +512,7 @@ mod tests { RolloutItem::ResponseItem(ResponseItem::FunctionCallOutput { call_id: "call-1".to_string(), output: FunctionCallOutputPayload::from_text("tool output".to_string()), + metadata: None, }), RolloutItem::EventMsg(EventMsg::TokenCount( codex_protocol::protocol::TokenCountEvent {