From a46fbafe314d37c25ab59aed765a2b11802c2f93 Mon Sep 17 00:00:00 2001
From: kshitijk4poor <82637225+kshitijk4poor@users.noreply.github.com>
Date: Tue, 21 Jul 2026 16:05:25 +0530
Subject: [PATCH] test: drop source-grep change-detector from #68480

The three behavior tests (control proves dup, boundary is noop, tail-only
write) fully cover the flush semantics. The source-grep test reading
cli.py + cli_commands_mixin.py as text and asserting a string appears is
a change-detector that breaks on benign refactors without adding coverage.
---
 ...test_session_rotation_flush_cold_resume_68454.py | 13 -------------
 1 file changed, 13 deletions(-)

diff --git a/tests/agent/test_session_rotation_flush_cold_resume_68454.py b/tests/agent/test_session_rotation_flush_cold_resume_68454.py
index cba7d7269255..c7650fa631da 100644
--- a/tests/agent/test_session_rotation_flush_cold_resume_68454.py
+++ b/tests/agent/test_session_rotation_flush_cold_resume_68454.py
@@ -108,16 +108,3 @@ def test_rotation_flush_writes_only_new_tail(tmp_path: Path) -> None:
         "new unpersisted turn",
     ]
 
-
-def test_call_sites_pass_conversation_history_kwarg() -> None:
-    """Source guard: the three rotation flushes named in #68454 stay fixed."""
-    import pathlib
-
-    root = pathlib.Path(__file__).resolve().parents[2]
-    cli = (root / "cli.py").read_text()
-    mixin = (root / "hermes_cli" / "cli_commands_mixin.py").read_text()
-
-    # /new site uses conversation_history=self.conversation_history
-    assert "conversation_history=self.conversation_history" in cli
-    # /resume and /branch sites in the mixin
-    assert mixin.count("conversation_history=self.conversation_history") >= 2