From 3026f4a9938c55d81040a36c03939dc7fd7b928c Mon Sep 17 00:00:00 2001 From: Teknium <127238744+teknium1@users.noreply.github.com> Date: Wed, 9 Sep 2026 04:46:11 -0700 Subject: [PATCH] test(compression): trim batch clarify coverage to two invariants Drop the sentinel-only batch test: a batch sentinel is already rejected by the shared _is_clarify_non_response_sentinel list check that the existing sentinel tests pin, so the case adds no new contract. Also add the contributor email mapping for the cherry-picked commit so release CI can attribute it. --- contributors/emails/gaoanze888@gmail.com | 2 ++ tests/agent/test_context_compressor.py | 12 ------------ 2 files changed, 2 insertions(+), 12 deletions(-) create mode 100644 contributors/emails/gaoanze888@gmail.com diff --git a/contributors/emails/gaoanze888@gmail.com b/contributors/emails/gaoanze888@gmail.com new file mode 100644 index 0000000000..d277865ea1 --- /dev/null +++ b/contributors/emails/gaoanze888@gmail.com @@ -0,0 +1,2 @@ +gaoanze888 +# PR #106089 salvage diff --git a/tests/agent/test_context_compressor.py b/tests/agent/test_context_compressor.py index 0e83f80b72..8c81452b8f 100644 --- a/tests/agent/test_context_compressor.py +++ b/tests/agent/test_context_compressor.py @@ -266,18 +266,6 @@ class TestSummarizeToolResultClarify: assert "Choice A" in summary assert "Choice B" in summary - def test_batch_with_only_sentinel_falls_back_to_generic(self): - """A batch where every response is a timeout sentinel must not be quoted as a user answer.""" - content = json.dumps({ - "responses": [ - {"question": "Q?", "user_response": "[user did not respond within 15m]"}, - ] - }) - - summary = _summarize_tool_result("clarify", "{}", content) - - assert summary == "[clarify] asked user a question" - class TestShouldCompress: def test_below_threshold(self, compressor):