refactor: improve save file error handling

- add error and info to tracing imports
- refactor save logic to use match for better error handling
- remove direct stdout message on save success

Signed-off-by: mingcheng <[email protected]>

Author: mingcheng

src/main.rs

@@ -26,7 +26,7 @@ use clap::Parser;
 use std::error::Error;
 use std::fs;
 use std::io::Write;
-use tracing::{Level, debug, trace};
+use tracing::{Level, debug, error, info, trace};
 
 use aigitcommit::utils::{
     OutputFormat, check_env_variables, format_openai_error, get_env, save_to_file, should_signoff,
@@ -185,10 +185,16 @@ async fn main() -> std::result::Result<(), Box<dyn Error>> {
     // If the --save option is enabled, save the commit message to a file
     if !cli.save.is_empty() {
         trace!("save option is enabled, will save the commit message to a file");
-        debug!("the save file path is {:?}", &cli.save);
 
-        save_to_file(&cli.save, &message)?;
-        writeln!(std::io::stdout(), "commit message saved to {}", cli.save)?;
+        // Save the commit message to the specified file
+        match save_to_file(&cli.save, &message) {
+            Ok(f) => {
+                info!("commit message saved to file: {:?}", f);
+            }
+            Err(e) => {
+                error!("failed to save commit message to file: {}", e);
+            }
+        }
     }
 
     Ok(())