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:
CodeWhale BotandClaude Opus 5.5 committed 2026-09-22 18:52:55 -07:00
1 parent be2140e249
commit d2b1430ccb
1 file changed
+6 -6
+6 -6
View File
@@ -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`."
}
}