Skip to content

fix: run detached portal summon --mode tmux off-TTY, add real tmux e2e - #1

Merged
Nurozen merged 1 commit into
weirwoodfrom
stave/tmux-e2e/stave
Aug 27, 2026
Merged

Nurozen merged 1 commit into
weirwoodfrom
stave/tmux-e2e/stave

Conversation

@Nurozen

@Nurozen Nurozen commented Aug 27, 2026

Copy link
Copy Markdown
Owner

Follow-up to the repo-tethers BF1 tmux summon (PR 1 of a 2-PR stack).

UX: non-interactive stave portal summon --mode tmux now runs the detached create instead of degrading to a preview. A detached tmux new-session -d needs no TTY, and the blocking attach stays gated off-TTY, so previewing was a wart. Guarded so --mode print/--print-command and real-TTY attach are unchanged.

e2e: new test_rig/portal-e2e leg drives --mode tmux against the ssh-host container (forced off-TTY via </dev/null) and asserts the tmux session exists, its pane shows the stub agent's marker, and re-summon is idempotent. The ssh-host image gains a kept-alive /usr/bin/codex stub. Assertion sshes carry ConnectTimeout so a wedge fails rather than stalls.

Deep-planned, deep-built, and reviewed (holistic + codex). Full CI gate green (build both platforms, vet, go test -race, golangci-lint 0, shellcheck).

🤖 Generated with Claude Code

Follow-up to the repo-tethers BF1 tmux summon.

- UX: non-interactive `stave portal summon --mode tmux` now runs the
  detached create instead of degrading to a preview. A detached
  `tmux new-session -d` needs no TTY, and the blocking attach stays
  gated off-TTY, so previewing was a wart.
- e2e: new test_rig/portal-e2e leg drives `--mode tmux` against the
  ssh-host container and asserts the tmux session exists, its pane shows
  the stub agent's marker, and re-summon is idempotent. The ssh-host
  image gains a kept-alive `codex` stub so a summoner exists remotely.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@Nurozen
Nurozen merged commit bdbeb88 into weirwood Aug 27, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant