Fix mock pane close command in conftest and resolve pane_id collisions in E2E Scenario 5 tests

tests/conftest.py

@@ -428,6 +428,18 @@ elif cmd1 == "pane":
         }
         print(json.dumps({"result": res}))
         sys.exit(0)
+    elif cmd2 == "close":
+        pane_id = args[2]
+        agents = state.get("agents", {})
+        to_delete = []
+        for name, data in agents.items():
+            if data.get("pane_id") == pane_id or pane_id == "w1:p1":
+                to_delete.append(name)
+        for name in to_delete:
+            del agents[name]
+        state["agents"] = agents
+        save_state()
+        sys.exit(0)
 
 elif cmd1 == "ls":
     if "-F" in args:

tests/test_tier4_e2e.py

@@ -307,13 +307,18 @@ d['herdr_sessions'] = [
     # Seed mock herdr state with these running sessions to satisfy has-session checks
     with open(mock_herdr, 'r') as f:
         herdr_state = json.load(f)
+    pane_ids = {
+        worker_name: "w1:p1",
+        reviewer_name: "w1:p2",
+        planner_name: "w1:p3"
+    }
     for name in [worker_name, reviewer_name, planner_name]:
         herdr_state["agents"][name] = {
             "status": "running",
             "agent": "claude",
             "cwd": str(tmp_path),
             "pid": 9999,
-            "pane_id": "w1:p1",
+            "pane_id": pane_ids[name],
             "command": "claude",
             "buffer": "Anthropic Claude Ready"
         }