From d3e8d29cfb8aea478ef6fc8b609ceb1661100909 Mon Sep 17 00:00:00 2001
From: ZdenekSrotyr <zdenek.srotyr@keboola.com>
Date: Thu, 7 May 2026 06:42:48 +0200
Subject: [PATCH] =?UTF-8?q?test(hooks):=20pin=20v0.43.0=20chained-entry=20?=
 =?UTF-8?q?=E2=86=92=20v0.44.0=20two-entry=20upgrade=20path?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 tests/test_lib_hooks.py | 40 ++++++++++++++++++++++++++++++++++++++++
 1 file changed, 40 insertions(+)

diff --git a/tests/test_lib_hooks.py b/tests/test_lib_hooks.py
index 7ee8d36..b64c759 100644
--- a/tests/test_lib_hooks.py
+++ b/tests/test_lib_hooks.py
@@ -108,6 +108,46 @@ def test_install_replaces_prior_single_pull_entry(tmp_path):
     assert any("agnes refresh-marketplace" in c for c in starts)
 
 
+def test_install_replaces_v0_43_chained_self_upgrade_pull_entry(tmp_path):
+    """Workspaces bootstrapped on v0.43.0 had a single SessionStart entry
+    chaining `agnes self-upgrade; agnes pull` in one shell line. Upgrading
+    those workspaces to v0.44.0+ must collapse that entry and re-install
+    the new two-entry layout — not stack the v0.44 entries on top of the
+    v0.43 chained one (which would re-run self-upgrade twice on every
+    session and leave the old format around forever).
+    """
+    settings_path = tmp_path / ".claude" / "settings.json"
+    settings_path.parent.mkdir(parents=True)
+    settings_path.write_text(json.dumps({
+        "hooks": {
+            "SessionStart": [
+                {"hooks": [{"type": "command", "command": (
+                    "agnes self-upgrade --quiet 2>/dev/null || true; "
+                    "agnes pull --quiet 2>/dev/null || true"
+                )}]},
+            ],
+            "SessionEnd": [
+                {"hooks": [{"type": "command", "command": "agnes push --quiet 2>/dev/null || true"}]},
+            ],
+        }
+    }))
+    install_claude_hooks(tmp_path)
+    cfg = _read_settings(tmp_path)
+    starts = _commands_for(cfg, "SessionStart")
+    # Exactly two entries — the v0.43 chained line was replaced, not stacked.
+    assert len(starts) == 2, starts
+    chain = next(
+        (c for c in starts if "agnes self-upgrade" in c and "agnes pull" in c),
+        None,
+    )
+    assert chain is not None
+    assert any("agnes refresh-marketplace" in c for c in starts)
+    # SessionEnd untouched (single push entry).
+    ends = _commands_for(cfg, "SessionEnd")
+    assert len(ends) == 1
+    assert "agnes push --quiet" in ends[0]
+
+
 def test_install_preserves_third_party_hooks(tmp_path):
     settings_path = tmp_path / ".claude" / "settings.json"
     settings_path.parent.mkdir(parents=True)