Skip to content

tui: :providers shows each provider's full name and where it runs - #1262

Merged
cdalar merged 1 commit into
mainfrom
tui-providers-full-names
Oct 27, 2026
Merged

cdalar merged 1 commit into
mainfrom
tui-providers-full-names

Conversation

@cdalar

@cdalar cdalar commented Oct 27, 2026

Copy link
Copy Markdown
Owner

What

The :providers view listed only short names (ac, ch, fc, …) plus the current one. It now adds two columns, PROVIDER and WHERE, taken from providerPlaces. These match the provider list onctl tui shows at startup (#1261).

NAME     PROVIDER                  WHERE               CURRENT
aws      Amazon Web Services       cloud, BYO account
hetzner  Hetzner Cloud             cloud, BYO account  *
fc       Firecracker microVM       local
ch       Cloud Hypervisor microVM  local
ac       Apple Container           local
static   Static host (onctl.yaml)  remote, BYO host
boxes    Boxes service             cloud, SaaS

Shift-P and shift-W sort by the new columns.

Testing

  • TestTUIProviderSwitch checks the full rows: the current provider's row and a local one. It fails against the old two-column view.
  • It also fails if a provider is added to cloudProviderList without a full name.
  • go test ./... passes with GIT_CONFIG_GLOBAL=/dev/null, and golangci-lint is clean.
  • Locally, TestCloneScript fails without that setting, because the global git config requires GPG commit signing. That failure predates this change.

🤖 Generated with Claude Code

The view listed only the short names (ac, ch, fc...) and the current
one. It now has PROVIDER and WHERE columns from providerPlaces, as the
pick onctl tui starts with shows them: "ac  Apple Container  local".

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@codecov

codecov Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 48.00%. Comparing base (423056a) to head (d4567ba).
⚠️ Report is 12 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1262   +/-   ##
=======================================
  Coverage   47.99%   48.00%           
=======================================
  Files          81       81           
  Lines        9424     9425    +1     
=======================================
+ Hits         4523     4524    +1     
  Misses       4901     4901           

☔ 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 7b7124e into main Oct 27, 2026
13 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

⌂ 换网址