openai/codex
1
0
Fork 0
mirror of https://github.com/openai/codex.git synced 2026-09-15 12:08:01 +00:00
Code Issues Packages Projects Releases Wiki Activity

Keep only user+compaction items in compacted history

Browse Source
This commit is contained in:
Charles Cunningham
2026-02-17 19:35:22 -08:00
parent 6663b690b7
commit b781a15a42
3 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
codex-rs/core/src/compact.rs
View File

@@ -485,6 +485,7 @@ fn should_keep_compacted_history_item(item: &ResponseItem) -> bool {
Some(TurnItem::UserMessage(_))
)
}
// Keep compaction records for local/remote history continuity and token accounting.
ResponseItem::Compaction { .. } => true,
ResponseItem::Reasoning { .. }
| ResponseItem::LocalShellCall { .. }

5
codex-rs/core/tests/suite/compact.rs
View File

@@ -2961,11 +2961,6 @@ async fn auto_compact_counts_encrypted_reasoning_before_last_user() {
"second turn should not include compacted history"
);
let third_request_body = requests[2].body_json().to_string();
assert!(
third_request_body.contains("REMOTE_COMPACT_SUMMARY")
|| third_request_body.contains(FINAL_REPLY),
"third turn should include compacted history"
);
assert!(
third_request_body.contains("ENCRYPTED_COMPACTION_SUMMARY"),
"third turn should include compaction summary item"

6
codex-rs/core/tests/suite/compact_remote.rs
View File

@@ -1051,11 +1051,15 @@ async fn remote_compact_persists_replacement_history_in_rollout() -> Result<()>
)
});
if has_compacted_user_summary && has_compaction_item && has_compacted_assistant_note {
if has_compacted_user_summary && has_compaction_item {
assert!(
!has_permissions_developer_message,
"manual remote compact rollout replacement history should not inject permissions context"
);
assert!(
!has_compacted_assistant_note,
"manual remote compact rollout replacement history should drop assistant notes"
);
saw_compacted_history = true;
break;
}