From 802e0b1daebad8a60da496db1a070fe13ddc45bd Mon Sep 17 00:00:00 2001 From: Teknium <127238744+teknium1@users.noreply.github.com> Date: Mon, 5 Oct 2026 00:58:05 -0700 Subject: [PATCH] test(approval): compare only the approval pair now that human-input hooks fire around the same CLI prompts --- tests/tools/test_approval_plugin_hooks.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tests/tools/test_approval_plugin_hooks.py b/tests/tools/test_approval_plugin_hooks.py index 92f5bb96d7..22680ae803 100644 --- a/tests/tools/test_approval_plugin_hooks.py +++ b/tests/tools/test_approval_plugin_hooks.py @@ -344,6 +344,9 @@ def _capture_hooks(run): with patch("hermes_cli.plugins.invoke_hook", side_effect=lambda name, **kw: captured.append((name, kw)) or []): result = run() + # on_human_input_* fire around the same prompts with their own payload contract + # (tests/tools/test_human_input_hooks.py); this helper compares the approval pair. + captured = [(name, kw) for name, kw in captured if not name.startswith("on_human_input_")] for _, kw in captured: for key in ("turn_id", "tool_call_id", "session_id"): kw.pop(key, None)