openai/codex
1
0
Fork 0
mirror of https://github.com/openai/codex.git synced 2026-11-14 21:40:39 +00:00
Code Issues Packages Projects Releases Wiki Activity

chore: ResponseItem::AgentMessage should be assigned an id with amsg_ prefix

Browse Source
This commit is contained in:
Michael Bolin
2026-06-23 17:22:12 -07:00
parent 32b65bbf7a
commit b6791c8aea
2 changed files with 4 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
codex-rs/core/src/session/mod.rs
+2 -3
View File
@@ -2738,9 +2738,8 @@ impl Session {
ResponseItem::WebSearchCall { .. } => "ws",
ResponseItem::ImageGenerationCall { .. } => "ig",
ResponseItem::Compaction { .. } | ResponseItem::ContextCompaction { .. } => "cmp",
ResponseItem::AgentMessage { .. }
| ResponseItem::CompactionTrigger { .. }
| ResponseItem::Other => continue,
ResponseItem::AgentMessage { .. } => "amsg",
ResponseItem::CompactionTrigger { .. } | ResponseItem::Other => continue,
};
item.set_id(Some(format!("{prefix}_{}", Uuid::now_v7())));
}
codex-rs/core/src/session/tests.rs
+2 -2
View File
@@ -210,7 +210,7 @@ fn user_message(text: &str) -> ResponseItem {
}
#[test]
fn assign_missing_response_item_ids_skips_agent_messages() {
fn assign_missing_response_item_ids_assigns_agent_message_ids() {
let items = Cow::Owned(vec![
ResponseItem::AgentMessage {
id: None,
@@ -226,7 +226,7 @@ fn assign_missing_response_item_ids_skips_agent_messages() {
let items = Session::assign_missing_response_item_ids(items);
assert_eq!(items[0].id(), None);
assert!(items[0].id().is_some_and(|id| id.starts_with("amsg_")));
assert!(items[1].id().is_some_and(|id| id.starts_with("msg_")));
}