From 7078a0b676bb040a65f2cdde916a943314bb394c Mon Sep 17 00:00:00 2001 From: pakrym-oai Date: Fri, 2 Jan 2026 11:27:37 -0800 Subject: [PATCH] Log compaction request bodies (#8676) We already log request bodies for normal requests, logging for compaction helps with debugging. --- codex-rs/codex-client/src/transport.rs | 9 +++++++++ codex-rs/protocol/src/models.rs | 1 - 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/codex-rs/codex-client/src/transport.rs b/codex-rs/codex-client/src/transport.rs index 986ba3a67..abe6e29ee 100644 --- a/codex-rs/codex-client/src/transport.rs +++ b/codex-rs/codex-client/src/transport.rs @@ -69,6 +69,15 @@ impl ReqwestTransport { #[async_trait] impl HttpTransport for ReqwestTransport { async fn execute(&self, req: Request) -> Result { + if enabled!(Level::TRACE) { + trace!( + "{} to {}: {}", + req.method, + req.url, + req.body.as_ref().unwrap_or_default() + ); + } + let builder = self.build(req)?; let resp = builder.send().await.map_err(Self::map_error)?; let status = resp.status(); diff --git a/codex-rs/protocol/src/models.rs b/codex-rs/protocol/src/models.rs index 722e91561..5113dadd0 100644 --- a/codex-rs/protocol/src/models.rs +++ b/codex-rs/protocol/src/models.rs @@ -416,7 +416,6 @@ impl Serialize for FunctionCallOutputPayload { where S: Serializer, { - tracing::debug!("Function call output payload: {:?}", self); if let Some(items) = &self.content_items { items.serialize(serializer) } else {