From 602ba17888c25b4a2b782cf733f7270a31db73bd Mon Sep 17 00:00:00 2001 From: Drew Ritter Date: Thu, 6 Aug 2026 15:04:25 -0700 Subject: [PATCH] fix(e2e): gate tmux ownership on creation success Make the copyable recipe independent of ambient set -e behavior. A failed tmux new-session now reports the failure and exits while session_owned is still false; only a successful creation reaches the ownership assignment. A real collision reproduced the prior snippet killing its sentinel after marking a failed creation as owned. The guarded version preserved the exact sentinel identity, and three refined fresh-agent runs again preserved IDs 17, 18, and 19 while cleaning only their fallback sessions. --- skills/agentic-end-to-end-testing/driving-cli-tui.md | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/skills/agentic-end-to-end-testing/driving-cli-tui.md b/skills/agentic-end-to-end-testing/driving-cli-tui.md index 8a5ea48a..cd6f1319 100644 --- a/skills/agentic-end-to-end-testing/driving-cli-tui.md +++ b/skills/agentic-end-to-end-testing/driving-cli-tui.md @@ -19,7 +19,10 @@ cleanup() { } trap cleanup EXIT -tmux new-session -d -s "$session" -x 200 -y 50 " 2>\"$stderr_log\"" +if ! tmux new-session -d -s "$session" -x 200 -y 50 " 2>\"$stderr_log\""; then + echo "failed to create tmux session: $session" >&2 + exit 1 +fi session_owned=1 tmux send-keys -t "$session" -l "literal text" # -l = no key-name parsing (paths, slashes) tmux send-keys -t "$session" Enter