Skip to content

tui: enter in :repos asks for a new box or the repo's own - #1258

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

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

Conversation

@cdalar

@cdalar cdalar commented Oct 26, 2026

Copy link
Copy Markdown
Owner

Summary

  • Since tui: enter in :repos starts Claude on a new box every time #1257 every enter on a repository in :repos made a new box. Now it asks first, in a choice dialog: n for a new box (<name>-<word>-<word>-<id>, passed as --box, so two pieces of work never share one), or r for the repository's own box, claude-<name>, which onctl claude --repo alone comes back to. The dialog says whether that box exists and in what state. esc (any other key) cancels.
  • tuiDialog grows choices beside its y/N form: a key, a footer label, and what the key does. Destroy's y/N dialog is unchanged.
  • Help text, the describe page and the docs say the same.

Test plan

  • go test ./... passes: the :repos test drives the dialog (contents, esc, n, r, the box-exists wording), plus the args and flash cases
  • golangci-lint run ./cmd/... clean

🤖 Generated with Claude Code

https://claude.ai/code/session_01XrkgRG7mxeo99tLWf4L7Se

Since #1257 every enter on a repository made a new box. Now it asks
first, in a choice dialog: n for a new box (<name>-<word>-<word>-<id>,
passed as --box, so two pieces of work never share one), or r for the
repository's own box, claude-<name>, which `onctl claude --repo` alone
comes back to -- the dialog says whether that box exists and in what
state. Any other key cancels.

tuiDialog grows choices beside its y/N form: a key, a label for the
footer, and what the key does.

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

codecov Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 88.37209% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 48.01%. Comparing base (8d5d518) to head (7ccfe41).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
cmd/tuiactions.go 63.63% 4 Missing ⚠️
cmd/tuirepos.go 95.23% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1258      +/-   ##
==========================================
+ Coverage   47.85%   48.01%   +0.16%     
==========================================
  Files          81       81              
  Lines        9360     9391      +31     
==========================================
+ Hits         4479     4509      +30     
- Misses       4881     4882       +1     

☔ 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 e4701ad into main Oct 26, 2026
13 of 14 checks passed
@cdalar
cdalar deleted the tui-repo-box-choice branch October 26, 2026 16:32
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

⌂ 换网址