From 90320fc51a760841d84acd1566f12bab57ddd092 Mon Sep 17 00:00:00 2001 From: Won Park Date: Mon, 6 Apr 2026 23:49:47 -0700 Subject: [PATCH] collapse dev message into one (#16988) collapse image-gen dev message into one --- codex-rs/core/src/codex_tests.rs | 11 +++-------- codex-rs/core/src/stream_events_utils.rs | 9 ++------- 2 files changed, 5 insertions(+), 15 deletions(-) diff --git a/codex-rs/core/src/codex_tests.rs b/codex-rs/core/src/codex_tests.rs index b0437edbf..0bfb0c303 100644 --- a/codex-rs/core/src/codex_tests.rs +++ b/codex-rs/core/src/codex_tests.rs @@ -4089,18 +4089,13 @@ async fn handle_output_item_done_records_image_save_history_message() { let image_output_dir = image_output_path .parent() .expect("generated image path should have a parent"); - let save_message: ResponseItem = DeveloperInstructions::new(format!( - "Generated images are saved to {} as {} by default.", + let image_message: ResponseItem = DeveloperInstructions::new(format!( + "Generated images are saved to {} as {} by default.\nIf you need to use a generated image at another path, copy it and leave the original in place unless the user explicitly asks you to delete it.", image_output_dir.display(), image_output_path.display(), )) .into(); - let copy_message: ResponseItem = DeveloperInstructions::new( - "If you need to use a generated image at another path, copy it and leave the original in place unless the user explicitly asks you to delete it." - .to_string(), - ) - .into(); - assert_eq!(history.raw_items(), &[save_message, copy_message, item]); + assert_eq!(history.raw_items(), &[image_message, item]); assert_eq!( std::fs::read(&expected_saved_path).expect("saved file"), b"foo" diff --git a/codex-rs/core/src/stream_events_utils.rs b/codex-rs/core/src/stream_events_utils.rs index 75eb1a8c2..ac4334d68 100644 --- a/codex-rs/core/src/stream_events_utils.rs +++ b/codex-rs/core/src/stream_events_utils.rs @@ -379,17 +379,12 @@ pub(crate) async fn handle_non_tool_response_item( .parent() .unwrap_or(turn_context.config.codex_home.as_path()); let message: ResponseItem = DeveloperInstructions::new(format!( - "Generated images are saved to {} as {} by default.", + "Generated images are saved to {} as {} by default.\nIf you need to use a generated image at another path, copy it and leave the original in place unless the user explicitly asks you to delete it.", image_output_dir.display(), image_output_path.display(), )) .into(); - let copy_message: ResponseItem = DeveloperInstructions::new( - "If you need to use a generated image at another path, copy it and leave the original in place unless the user explicitly asks you to delete it." - .to_string(), - ) - .into(); - sess.record_conversation_items(turn_context, &[message, copy_message]) + sess.record_conversation_items(turn_context, &[message]) .await; } Err(err) => {