diff --git a/src/main.rs b/src/main.rs index 33d0c08..0a452a3 100644 --- a/src/main.rs +++ b/src/main.rs @@ -3,7 +3,7 @@ use std::io::{BufRead, BufReader}; use std::net::SocketAddr; use std::path::PathBuf; use tokio::signal; -use tracing::{debug, info}; +use tracing::{debug, info, warn}; use clawshell::cli::{Cli, Commands}; use clawshell::config::Config; @@ -498,19 +498,31 @@ fn cmd_onboard() -> Result<(), Box> { "clawshell:clawshell" }; - let _ = std::process::Command::new("chmod") + if let Err(e) = std::process::Command::new("chmod") .args(["0700", &config_dir.to_string_lossy()]) - .status(); - let _ = std::process::Command::new("chown") + .status() + { + warn!(path = %config_dir.display(), error = %e, "Failed to chmod config directory"); + } + if let Err(e) = std::process::Command::new("chown") .args(["-R", chown_spec, &config_dir.to_string_lossy()]) - .status(); - let _ = std::process::Command::new("chown") + .status() + { + warn!(path = %config_dir.display(), error = %e, "Failed to chown config directory"); + } + if let Err(e) = std::process::Command::new("chown") .args(["-R", chown_spec, &log_dir_path.to_string_lossy()]) - .status(); + .status() + { + warn!(path = %log_dir_path.display(), error = %e, "Failed to chown log directory"); + } if let Some(pid_parent) = pid_path.parent() { - let _ = std::process::Command::new("chown") + if let Err(e) = std::process::Command::new("chown") .args([chown_spec, &pid_parent.to_string_lossy()]) - .status(); + .status() + { + warn!(path = %pid_parent.display(), error = %e, "Failed to chown PID directory"); + } } tui::print_step_done(3, TOTAL_STEPS, "Permissions set");