From 708c25cdc8f00fcc8f8532f0da5944896dc4cfe1 Mon Sep 17 00:00:00 2001 From: Charles Cunningham Date: Thu, 12 Feb 2026 14:36:13 -0800 Subject: [PATCH] update snaps --- ...t__manual_compact_with_history_shapes.snap | 10 +++---- ...anual_compact_with_history_shapes.snap.new | 22 --------------- ...mpact__mid_turn_compaction_shapes.snap.new | 25 ----------------- ..._compaction_including_incoming_shapes.snap | 17 ++++++------ ...paction_including_incoming_shapes.snap.new | 27 ------------------- ...te_manual_compact_with_history_shapes.snap | 10 +++---- ...anual_compact_with_history_shapes.snap.new | 21 --------------- ...te__remote_mid_turn_compaction_shapes.snap | 8 +++--- ...remote_mid_turn_compaction_shapes.snap.new | 21 --------------- ...action_context_window_exceeded_shapes.snap | 1 + ...on_context_window_exceeded_shapes.snap.new | 14 ---------- ..._compaction_including_incoming_shapes.snap | 10 +++---- ...paction_including_incoming_shapes.snap.new | 25 ----------------- 13 files changed, 29 insertions(+), 182 deletions(-) delete mode 100644 codex-rs/core/tests/suite/snapshots/all__suite__compact__manual_compact_with_history_shapes.snap.new delete mode 100644 codex-rs/core/tests/suite/snapshots/all__suite__compact__mid_turn_compaction_shapes.snap.new delete mode 100644 codex-rs/core/tests/suite/snapshots/all__suite__compact__pre_turn_compaction_including_incoming_shapes.snap.new delete mode 100644 codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_manual_compact_with_history_shapes.snap.new delete mode 100644 codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_mid_turn_compaction_shapes.snap.new delete mode 100644 codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_pre_turn_compaction_context_window_exceeded_shapes.snap.new delete mode 100644 codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_pre_turn_compaction_including_incoming_shapes.snap.new diff --git a/codex-rs/core/tests/suite/snapshots/all__suite__compact__manual_compact_with_history_shapes.snap b/codex-rs/core/tests/suite/snapshots/all__suite__compact__manual_compact_with_history_shapes.snap index bcca5d3575..c88ff33122 100644 --- a/codex-rs/core/tests/suite/snapshots/all__suite__compact__manual_compact_with_history_shapes.snap +++ b/codex-rs/core/tests/suite/snapshots/all__suite__compact__manual_compact_with_history_shapes.snap @@ -13,9 +13,9 @@ Scenario: Manual /compact with prior user history compacts existing history and 05:message/user: ## Local Post-Compaction History Layout -00:message/developer: -01:message/user: -02:message/user:> -03:message/user:first manual turn -04:message/user:\nFIRST_MANUAL_SUMMARY +00:message/user:first manual turn +01:message/user:\nFIRST_MANUAL_SUMMARY +02:message/developer: +03:message/user: +04:message/user:> 05:message/user:second manual turn diff --git a/codex-rs/core/tests/suite/snapshots/all__suite__compact__manual_compact_with_history_shapes.snap.new b/codex-rs/core/tests/suite/snapshots/all__suite__compact__manual_compact_with_history_shapes.snap.new deleted file mode 100644 index 6fa78f6e20..0000000000 --- a/codex-rs/core/tests/suite/snapshots/all__suite__compact__manual_compact_with_history_shapes.snap.new +++ /dev/null @@ -1,22 +0,0 @@ ---- -source: core/tests/suite/compact.rs -assertion_line: 2395 -expression: "format_labeled_requests_snapshot(\"Manual /compact with prior user history compacts existing history and the follow-up turn includes the compact summary plus new user message.\",\n&[(\"Local Compaction Request\", &requests[1]),\n(\"Local Post-Compaction History Layout\", &requests[2]),])" ---- -Scenario: Manual /compact with prior user history compacts existing history and the follow-up turn includes the compact summary plus new user message. - -## Local Compaction Request -00:message/developer: -01:message/user: -02:message/user:> -03:message/user:first manual turn -04:message/assistant:FIRST_REPLY -05:message/user: - -## Local Post-Compaction History Layout -00:message/user:first manual turn -01:message/user:\nFIRST_MANUAL_SUMMARY -02:message/developer: -03:message/user: -04:message/user:> -05:message/user:second manual turn diff --git a/codex-rs/core/tests/suite/snapshots/all__suite__compact__mid_turn_compaction_shapes.snap.new b/codex-rs/core/tests/suite/snapshots/all__suite__compact__mid_turn_compaction_shapes.snap.new deleted file mode 100644 index 9d87e6a9e1..0000000000 --- a/codex-rs/core/tests/suite/snapshots/all__suite__compact__mid_turn_compaction_shapes.snap.new +++ /dev/null @@ -1,25 +0,0 @@ ---- -source: core/tests/suite/compact.rs -assertion_line: 2657 -expression: "format_labeled_requests_snapshot(\"Mid-turn continuation compaction after tool output: compact request includes tool artifacts and follow-up request includes the summary.\",\n&[(\"Local Compaction Request\", &auto_compact_mock.single_request()),\n(\"Local Post-Compaction History Layout\",\n&post_auto_compact_mock.single_request()),])" ---- -Scenario: Mid-turn continuation compaction after tool output: compact request includes tool artifacts and follow-up request includes the summary. - -## Local Compaction Request -00:message/developer: -01:message/user: -02:message/user:> -03:message/user:function call limit push -04:function_call/test_tool -05:function_call_output:unsupported call: test_tool -06:message/assistant:FINAL_REPLY -07:message/user:FOLLOW_UP_AFTER_LIMIT -08:message/user: - -## Local Post-Compaction History Layout -00:message/user:function call limit push -01:message/developer: -02:message/user: -03:message/user:> -04:message/user:FOLLOW_UP_AFTER_LIMIT -05:message/user:\nAUTO_SUMMARY diff --git a/codex-rs/core/tests/suite/snapshots/all__suite__compact__pre_turn_compaction_including_incoming_shapes.snap b/codex-rs/core/tests/suite/snapshots/all__suite__compact__pre_turn_compaction_including_incoming_shapes.snap index e586c8521a..553ee5b520 100644 --- a/codex-rs/core/tests/suite/snapshots/all__suite__compact__pre_turn_compaction_including_incoming_shapes.snap +++ b/codex-rs/core/tests/suite/snapshots/all__suite__compact__pre_turn_compaction_including_incoming_shapes.snap @@ -13,13 +13,14 @@ Scenario: Pre-turn auto-compaction with a context override emits the context dif 05:message/user:USER_TWO 06:message/assistant:SECOND_REPLY 07:message/user: -08:message/user: +08:message/user: | | USER_THREE +09:message/user: ## Local Post-Compaction History Layout -00:message/developer: -01:message/user: -02:message/user: -03:message/user:USER_ONE -04:message/user:USER_TWO -05:message/user:\nPRE_TURN_SUMMARY -06:message/user: | | | USER_THREE +00:message/user:USER_ONE +01:message/user:USER_TWO +02:message/developer: +03:message/user: +04:message/user: +05:message/user: | | USER_THREE +06:message/user:\nPRE_TURN_SUMMARY diff --git a/codex-rs/core/tests/suite/snapshots/all__suite__compact__pre_turn_compaction_including_incoming_shapes.snap.new b/codex-rs/core/tests/suite/snapshots/all__suite__compact__pre_turn_compaction_including_incoming_shapes.snap.new deleted file mode 100644 index 0458d88072..0000000000 --- a/codex-rs/core/tests/suite/snapshots/all__suite__compact__pre_turn_compaction_including_incoming_shapes.snap.new +++ /dev/null @@ -1,27 +0,0 @@ ---- -source: core/tests/suite/compact.rs -assertion_line: 3026 -expression: "format_labeled_requests_snapshot(\"Pre-turn auto-compaction with a context override emits the context diff in the compact request while the incoming user message is still excluded.\",\n&[(\"Local Compaction Request\", &requests[2]),\n(\"Local Post-Compaction History Layout\", &requests[3]),])" ---- -Scenario: Pre-turn auto-compaction with a context override emits the context diff in the compact request while the incoming user message is still excluded. - -## Local Compaction Request -00:message/developer: -01:message/user: -02:message/user:> -03:message/user:USER_ONE -04:message/assistant:FIRST_REPLY -05:message/user:USER_TWO -06:message/assistant:SECOND_REPLY -07:message/user: -08:message/user: | | USER_THREE -09:message/user: - -## Local Post-Compaction History Layout -00:message/user:USER_ONE -01:message/user:USER_TWO -02:message/developer: -03:message/user: -04:message/user: -05:message/user: | | USER_THREE -06:message/user:\nPRE_TURN_SUMMARY diff --git a/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_manual_compact_with_history_shapes.snap b/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_manual_compact_with_history_shapes.snap index bc795a19f4..f332d65d79 100644 --- a/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_manual_compact_with_history_shapes.snap +++ b/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_manual_compact_with_history_shapes.snap @@ -13,9 +13,9 @@ Scenario: Remote manual /compact where remote compact output is summary-only: fo 04:message/assistant:FIRST_REMOTE_REPLY ## Remote Post-Compaction History Layout -00:message/developer: -01:message/user: -02:message/user:> -03:message/user:REMOTE_COMPACTED_SUMMARY -04:compaction:encrypted=true +00:message/user:REMOTE_COMPACTED_SUMMARY +01:compaction:encrypted=true +02:message/developer: +03:message/user: +04:message/user:> 05:message/user:after compact diff --git a/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_manual_compact_with_history_shapes.snap.new b/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_manual_compact_with_history_shapes.snap.new deleted file mode 100644 index f9e4ad23aa..0000000000 --- a/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_manual_compact_with_history_shapes.snap.new +++ /dev/null @@ -1,21 +0,0 @@ ---- -source: core/tests/suite/compact_remote.rs -assertion_line: 174 -expression: "format_labeled_requests_snapshot(\"Remote manual /compact with prior user history compacts existing history and follow-up includes compact summary plus new user message.\",\n&[(\"Remote Compaction Request\", &compact_request),\n(\"Remote Post-Compaction History Layout\", follow_up_request),])" ---- -Scenario: Remote manual /compact with prior user history compacts existing history and follow-up includes compact summary plus new user message. - -## Remote Compaction Request -00:message/developer: -01:message/user: -02:message/user:> -03:message/user:hello remote compact -04:message/assistant:FIRST_REMOTE_REPLY - -## Remote Post-Compaction History Layout -00:message/user:REMOTE_COMPACTED_SUMMARY -01:compaction:encrypted=true -02:message/developer: -03:message/user: -04:message/user:> -05:message/user:after compact diff --git a/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_mid_turn_compaction_shapes.snap b/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_mid_turn_compaction_shapes.snap index 8ef8701673..488bee0bef 100644 --- a/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_mid_turn_compaction_shapes.snap +++ b/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_mid_turn_compaction_shapes.snap @@ -13,8 +13,8 @@ Scenario: Remote mid-turn continuation compaction after tool output: compact req 05:function_call_output:unsupported call: test_tool ## Remote Post-Compaction History Layout -00:message/user:USER_ONE -01:message/developer: -02:message/user: -03:message/user:> +00:message/developer: +01:message/user: +02:message/user:> +03:message/user:USER_ONE 04:message/user:\nREMOTE_MID_TURN_SUMMARY diff --git a/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_mid_turn_compaction_shapes.snap.new b/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_mid_turn_compaction_shapes.snap.new deleted file mode 100644 index e7621c9540..0000000000 --- a/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_mid_turn_compaction_shapes.snap.new +++ /dev/null @@ -1,21 +0,0 @@ ---- -source: core/tests/suite/compact_remote.rs -assertion_line: 1578 -expression: "format_labeled_requests_snapshot(\"Remote mid-turn continuation compaction after tool output: compact request includes tool artifacts and follow-up request includes the summary.\",\n&[(\"Remote Compaction Request\", &compact_request),\n(\"Remote Post-Compaction History Layout\", &requests[1]),])" ---- -Scenario: Remote mid-turn continuation compaction after tool output: compact request includes tool artifacts and follow-up request includes the summary. - -## Remote Compaction Request -00:message/developer: -01:message/user: -02:message/user:> -03:message/user:USER_ONE -04:function_call/test_tool -05:function_call_output:unsupported call: test_tool - -## Remote Post-Compaction History Layout -00:message/developer: -01:message/user: -02:message/user:> -03:message/user:USER_ONE -04:message/user:\nREMOTE_MID_TURN_SUMMARY diff --git a/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_pre_turn_compaction_context_window_exceeded_shapes.snap b/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_pre_turn_compaction_context_window_exceeded_shapes.snap index e718bfda88..2bac15dfda 100644 --- a/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_pre_turn_compaction_context_window_exceeded_shapes.snap +++ b/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_pre_turn_compaction_context_window_exceeded_shapes.snap @@ -10,3 +10,4 @@ Scenario: Remote pre-turn auto-compaction context-window failure: compaction req 02:message/user:> 03:message/user:USER_ONE 04:message/assistant:REMOTE_FIRST_REPLY +05:message/user:USER_TWO diff --git a/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_pre_turn_compaction_context_window_exceeded_shapes.snap.new b/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_pre_turn_compaction_context_window_exceeded_shapes.snap.new deleted file mode 100644 index 01bbca3bba..0000000000 --- a/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_pre_turn_compaction_context_window_exceeded_shapes.snap.new +++ /dev/null @@ -1,14 +0,0 @@ ---- -source: core/tests/suite/compact_remote.rs -assertion_line: 1503 -expression: "format_labeled_requests_snapshot(\"Remote pre-turn auto-compaction context-window failure: compaction request excludes the incoming user message and the turn errors.\",\n&[(\"Remote Compaction Request (Incoming User Excluded)\",\n&include_attempt_request),])" ---- -Scenario: Remote pre-turn auto-compaction context-window failure: compaction request excludes the incoming user message and the turn errors. - -## Remote Compaction Request (Incoming User Excluded) -00:message/developer: -01:message/user: -02:message/user:> -03:message/user:USER_ONE -04:message/assistant:REMOTE_FIRST_REPLY -05:message/user:USER_TWO diff --git a/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_pre_turn_compaction_including_incoming_shapes.snap b/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_pre_turn_compaction_including_incoming_shapes.snap index 00aaaeaa92..88fa34fa85 100644 --- a/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_pre_turn_compaction_including_incoming_shapes.snap +++ b/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_pre_turn_compaction_including_incoming_shapes.snap @@ -13,12 +13,12 @@ Scenario: Remote pre-turn auto-compaction with a context override emits the cont 05:message/user:USER_TWO 06:message/assistant:REMOTE_SECOND_REPLY 07:message/user: +08:message/user:USER_THREE ## Remote Post-Compaction History Layout 00:message/user:USER_ONE -01:message/user:USER_TWO -02:message/developer: -03:message/user: -04:message/user: +01:message/developer: +02:message/user: +03:message/user: +04:message/user:USER_TWO 05:message/user:\nREMOTE_PRE_TURN_SUMMARY -06:message/user:USER_THREE diff --git a/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_pre_turn_compaction_including_incoming_shapes.snap.new b/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_pre_turn_compaction_including_incoming_shapes.snap.new deleted file mode 100644 index a770633530..0000000000 --- a/codex-rs/core/tests/suite/snapshots/all__suite__compact_remote__remote_pre_turn_compaction_including_incoming_shapes.snap.new +++ /dev/null @@ -1,25 +0,0 @@ ---- -source: core/tests/suite/compact_remote.rs -assertion_line: 1398 -expression: "format_labeled_requests_snapshot(\"Remote pre-turn auto-compaction with a context override emits the context diff in the compact request while excluding the incoming user message.\",\n&[(\"Remote Compaction Request\", &compact_request),\n(\"Remote Post-Compaction History Layout\", &requests[2]),])" ---- -Scenario: Remote pre-turn auto-compaction with a context override emits the context diff in the compact request while excluding the incoming user message. - -## Remote Compaction Request -00:message/developer: -01:message/user: -02:message/user:> -03:message/user:USER_ONE -04:message/assistant:REMOTE_FIRST_REPLY -05:message/user:USER_TWO -06:message/assistant:REMOTE_SECOND_REPLY -07:message/user: -08:message/user:USER_THREE - -## Remote Post-Compaction History Layout -00:message/user:USER_ONE -01:message/developer: -02:message/user: -03:message/user: -04:message/user:USER_TWO -05:message/user:\nREMOTE_PRE_TURN_SUMMARY