Repository navigation
tui: pick the provider inside the tui - #1267
Merged
Merged
Conversation
With neither -p nor ONCTL_CLOUD, onctl / onctl tui opened a promptui list and then the tui. It now opens on the tui's providers view, and enter on a row restarts the tui on that provider. Until one is picked nothing is listed, and only navigation, :q, :help and :provider work. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1267 +/- ##
==========================================
+ Coverage 49.60% 49.75% +0.14%
==========================================
Files 83 83
Lines 9678 9706 +28
==========================================
+ Hits 4801 4829 +28
Misses 4877 4877 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
An error opened a popup and also flashed the same text in red at the bottom. The flash is hidden while the popup is open and cleared when it is closed. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Picking a provider on a machine with no .onctl directory ended in a "please run `onctl init` first" popup and no way on. The tui now asks, runs onctl init in the foreground on yes (it asks its own questions), and goes on to the provider that was picked. boxes needs no config and is not asked. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
The tui asks "Run onctl init?" itself, and then init asked its own question about a project-based .onctl folder on the terminal. --global sets up only ~/.onctl and doesn't ask; the tui runs init with it. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
A listing error (the ac provider without Apple's container CLI, an expired token) was only flashed and cut off at the screen's edge. It now opens the error popup the first time; the same failure on the next refresh is flashed only, one that arrives while a question or a form is open waits, and a listing that works lets the next failure open it again. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Testing
🤖 Generated with Claude Code