mirror of
https://github.com/codewhale-hq/Codewhale.git
synced 2026-11-07 10:36:05 -10:00
fix(doctor): a named route without a confirmed credential is not set up (U7)
05ff7c753 treated ProviderModel NeedsAction as provider-ready, so a fresh
home (default route named, no credential) read "first-run setup is
unfinished" instead of naming the provider step. NeedsAction means no
credential is confirmed for the route; only Verified now counts, and the
line reads "Not ready: no model provider set up → run /provider in
Codewhale, or `codewhale setup`."
No-Issue: 0.10.1 addendum U7 (UX-6)
Checks (local, built binary codewhale-tui, fresh CODEWHALE_HOME/HOME):
- `codewhale-tui doctor`: third line and last line are the verdict above;
0 "update checkpoint" rows
- cargo test -p codewhale-tui --lib -- a_fresh_home_is_not_ready: 1 passed
at 05ff7c753; unchanged assertion (starts "Not ready", names /provider)
Co-Authored-By: Claude Opus 5.5 (1M context) <[email protected]>
This commit is contained in:
1 parent
be2140e249
commit
d2b1430ccb
1 file changed
+6
-6
@@ -5458,14 +5458,14 @@ fn doctor_verdict(state: &codewhale_config::SetupState) -> &'static str {
|
||||
if state.first_run_ready() {
|
||||
return "Ready: setup is complete.";
|
||||
}
|
||||
let provider_ready = matches!(
|
||||
state.status(codewhale_config::SetupStep::ProviderModel),
|
||||
codewhale_config::StepStatus::Verified | codewhale_config::StepStatus::NeedsAction
|
||||
);
|
||||
if provider_ready {
|
||||
// NeedsAction means a route is named but no credential is confirmed for
|
||||
// it, which is still "no provider set up" from where the user sits.
|
||||
let provider_verified = state.status(codewhale_config::SetupStep::ProviderModel)
|
||||
== codewhale_config::StepStatus::Verified;
|
||||
if provider_verified {
|
||||
"Not ready: first-run setup is unfinished → run `codewhale setup`."
|
||||
} else {
|
||||
"Not ready: no model provider connected → run /provider in Codewhale, or `codewhale setup`."
|
||||
"Not ready: no model provider set up → run /provider in Codewhale, or `codewhale setup`."
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
Reference in new issue
Block a user