Skip to content

fix(cli): size auto-detected interactive exec terminals - #3084

Open
elezar wants to merge 1 commit into
mainfrom
3080-fix-auto-tty-sizing/el
Open

fix(cli): size auto-detected interactive exec terminals#3084
elezar wants to merge 1 commit into
mainfrom
3080-fix-auto-tty-sizing/el

Conversation

@elezar

@elezar elezar commented Sep 1, 2026

Copy link
Copy Markdown
Member

Summary

Route auto-detected interactive sandbox exec sessions through the existing bidirectional protocol so the remote PTY receives the local terminal dimensions and later resize events.

Related Issue

Closes #3080

Changes

  • Route every resolved interactive terminal with a terminal stdin through ExecSandboxInteractive.

Testing

  • git diff --check passes
  • mise run pre-commit (cargo, rustfmt, and mise are unavailable in this workspace)
  • Test suite run (Rust toolchain unavailable in this workspace)
  • E2E tests run (not applicable to this focused routing decision)

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)

Signed-off-by: Evan Lezar <elezar@nvidia.com>
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.

bug(cli): sandbox exec requests a PTY without sending terminal size on the auto-detected TTY path (v0.0.106)

1 participant