Skip to content

feat: add tmux runtime adapter#11

Merged
harshitsinghbhandari merged 3 commits into
mainfrom
aa-8-tmux-worktree-adapters
May 27, 2026
Merged

feat: add tmux runtime adapter#11
harshitsinghbhandari merged 3 commits into
mainfrom
aa-8-tmux-worktree-adapters

Conversation

@harshitsinghbhandari

@harshitsinghbhandari harshitsinghbhandari commented May 27, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add a real tmux implementation of ports.Runtime
  • validate tmux session IDs and use the AO session ID as the tmux session name
  • wrap launch commands with env exports, PATH preservation, and an interactive shell keep-alive
  • support short send-keys input and buffer-paste for multiline/long messages
  • add unit-tested command builders plus a tmux integration test that skips when tmux is unavailable

Scope

This PR intentionally implements only the tmux Runtime adapter. git-worktree Workspace is left for follow-up per Harshit direction.

Refs #9

Validation

From backend/:

  • gofmt -l .
  • go build ./...
  • go vet ./...
  • go test -race ./...

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

Comment thread backend/internal/adapters/runtime/tmux/tmux.go Outdated
Comment thread backend/internal/adapters/runtime/tmux/tmux.go Outdated
Comment thread backend/internal/adapters/runtime/tmux/tmux.go
@harshitsinghbhandari harshitsinghbhandari merged commit f9bb4ce into main May 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.

2 participants