Skip to content

tui: enter in :repos starts Claude on a new box every time - #1257

Merged
cdalar merged 1 commit into
mainfrom
tui-repo-fresh-box
Oct 26, 2026
Merged

cdalar merged 1 commit into
mainfrom
tui-repo-fresh-box

Conversation

@cdalar

@cdalar cdalar commented Oct 26, 2026

Copy link
Copy Markdown
Owner

Summary

  • enter on a repository in :repos ran onctl claude --repo owner/name, which comes back to the repository's one box (claude-<name>), so a second piece of work on the same repo landed in the first one's box.
  • The tui now names a new box for every enter (<name>-<word>-<word>-<id>, as generated box names go, never one of the VMs listed) and passes it as --box, so two pieces of work never share a box. The box is kept when Claude exits; the flash names it, and :vms pauses or destroys it.
  • The "come back with" hint onctl claude --repo prints carries the --box when one was given, since --repo alone would go to claude-<name>.
  • Help text, describe page and docs updated to match.

Test plan

  • go test ./... passes, including the new TestTUIRepoBoxName and the extended TestRepoAttachHint / TestTUIClaudeDone
  • golangci-lint run ./cmd/... clean

🤖 Generated with Claude Code

https://claude.ai/code/session_01XrkgRG7mxeo99tLWf4L7Se

Selecting a repository in :repos ran `onctl claude --repo owner/name`,
which comes back to the repository's one box, claude-<name>, so a
second piece of work on the same repository landed in the first one's
box. Now the tui names a new box for every enter -- <name>-<word>-
<word>-<id>, as generated box names go, never one of the VMs listed --
and passes it as --box, so two pieces of work never share a box. The
box is kept when Claude exits (the flash names it), for :vms to pause
or destroy.

The "come back with" hint `onctl claude --repo` prints now carries the
--box when one was given, since --repo alone would go to claude-<name>.

Claude-Session: https://claude.ai/code/session_01XrkgRG7mxeo99tLWf4L7Se
@codecov

codecov Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 74.07407% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 47.85%. Comparing base (454eacc) to head (080ec31).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
cmd/tuiactions.go 0.00% 3 Missing ⚠️
cmd/tuirepos.go 83.33% 3 Missing ⚠️
cmd/clauderepo.go 80.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1257      +/-   ##
==========================================
+ Coverage   47.78%   47.85%   +0.06%     
==========================================
  Files          81       81              
  Lines        9342     9360      +18     
==========================================
+ Hits         4464     4479      +15     
- Misses       4878     4881       +3     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@cdalar
cdalar merged commit 8d5d518 into main Oct 26, 2026
13 checks passed
@cdalar
cdalar deleted the tui-repo-fresh-box branch October 26, 2026 16:15
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

⌂ 换网址