fix(cli): restore fabro-cli verification after merge

Resolve new clippy failures introduced by the merge and update the root
help snapshot to include the uninstall command so fabro-cli lint and
test verification return to green.
This commit is contained in:
Bryan Helmkamp 2026-04-08 16:41:13 -04:00
parent 9305b57bd2
commit b6dddfb6a3
No known key found for this signature in database
4 changed files with 16 additions and 19 deletions

View file

@ -119,14 +119,11 @@ async fn read_worker_control_stream<R>(
{
let mut lines = BufReader::new(reader).lines();
loop {
match lines.next_line().await {
Ok(Some(line)) => {
apply_worker_control_line(&interviewer, &cancel_token, &line).await;
}
Ok(None) | Err(_) => {
interviewer.interrupt_all().await;
break;
}
if let Ok(Some(line)) = lines.next_line().await {
apply_worker_control_line(&interviewer, &cancel_token, &line).await;
} else {
interviewer.interrupt_all().await;
break;
}
}
}

View file

@ -32,15 +32,15 @@ pub(crate) async fn execute(
None
};
let log_path = storage_dir
.as_ref()
.map(|dir| Storage::new(dir).server_state().log_path())
.unwrap_or_else(|| {
record_path
.parent()
.map(|parent| parent.join("server.log"))
.unwrap_or_else(|| PathBuf::from("server.log"))
});
let log_path = storage_dir.as_ref().map_or_else(
|| {
record_path.parent().map_or_else(
|| PathBuf::from("server.log"),
|parent| parent.join("server.log"),
)
},
|dir| Storage::new(dir).server_state().log_path(),
);
let pid = std::process::id();
serve::serve_command(serve_args, styles, storage_dir, move |resolved_bind| {

View file

@ -338,8 +338,7 @@ fn maybe_warn_host_port_fallback(requested: &BindRequest, resolved: &Bind) {
};
if addr.ip() == *host && addr.port() != DEFAULT_TCP_PORT {
eprintln!(
"Warning: TCP port {} is unavailable on {}; falling back to a random port.",
DEFAULT_TCP_PORT, host
"Warning: TCP port {DEFAULT_TCP_PORT} is unavailable on {host}; falling back to a random port."
);
}
}

View file

@ -32,6 +32,7 @@ fn help() {
server Server operations
doctor Check environment and integration health
install Set up the Fabro environment (LLMs, certs, GitHub)
uninstall Uninstall Fabro from this machine
pr Pull request operations
secret Manage server-owned secrets
settings Inspect effective settings