From 9c765f1217dd2e152e135cacbdca7a7d810be5b4 Mon Sep 17 00:00:00 2001 From: jif-oai Date: Wed, 19 Nov 2025 15:21:58 +0000 Subject: [PATCH] Compilation nits --- codex-rs/core/src/saved_sessions.rs | 4 +--- codex-rs/exec/src/event_processor_with_human_output.rs | 1 + codex-rs/mcp-server/src/codex_tool_runner.rs | 1 + 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/codex-rs/core/src/saved_sessions.rs b/codex-rs/core/src/saved_sessions.rs index 4aaa136ee2..775d537c92 100644 --- a/codex-rs/core/src/saved_sessions.rs +++ b/codex-rs/core/src/saved_sessions.rs @@ -140,7 +140,5 @@ pub async fn resolve_rollout_path(codex_home: &Path, identifier: &str) -> Result entry.rollout_path.display() ); } - find_conversation_path_by_id_str(codex_home, identifier) - .await - .map_err(Into::into) // todo jif + Ok(find_conversation_path_by_id_str(codex_home, identifier).await?) } diff --git a/codex-rs/exec/src/event_processor_with_human_output.rs b/codex-rs/exec/src/event_processor_with_human_output.rs index 2d550fea46..ca22ac09c8 100644 --- a/codex-rs/exec/src/event_processor_with_human_output.rs +++ b/codex-rs/exec/src/event_processor_with_human_output.rs @@ -566,6 +566,7 @@ impl EventProcessor for EventProcessorWithHumanOutput { | EventMsg::ReasoningContentDelta(_) | EventMsg::ReasoningRawContentDelta(_) | EventMsg::UndoCompleted(_) + | EventMsg::SaveSessionResponse(_) | EventMsg::UndoStarted(_) => {} } CodexStatus::Running diff --git a/codex-rs/mcp-server/src/codex_tool_runner.rs b/codex-rs/mcp-server/src/codex_tool_runner.rs index 93dc7764dc..deacbabaac 100644 --- a/codex-rs/mcp-server/src/codex_tool_runner.rs +++ b/codex-rs/mcp-server/src/codex_tool_runner.rs @@ -301,6 +301,7 @@ async fn run_codex_tool_session_inner( | EventMsg::UndoStarted(_) | EventMsg::UndoCompleted(_) | EventMsg::ExitedReviewMode(_) + | EventMsg::SaveSessionResponse(_) | EventMsg::DeprecationNotice(_) => { // For now, we do not do anything extra for these // events. Note that