feat: add debug and error logging to run_command function

- Add tracing::debug! macro to log stdout output at debug level
- Add tracing::error! macro to always log stderr output when not empty
- Improve observability by logging command outputs with structured context
- Enhance error visibility while maintaining debug-level stdout logging

Author: josecelano

src/shared/executor.rs

@@ -107,9 +107,28 @@ impl CommandExecutor {
                 source,
             })?;
 
+        let stderr = String::from_utf8_lossy(&output.stderr).to_string();
+        let stdout = String::from_utf8_lossy(&output.stdout).to_string();
+
+        // Always log stderr if it's not empty
+        if !stderr.trim().is_empty() {
+            tracing::error!(
+                operation = "command_execution",
+                command = %command_display,
+                stderr = %stderr,
+                "Command produced stderr output"
+            );
+        }
+
+        // Log stdout at debug level
+        tracing::debug!(
+            operation = "command_execution",
+            command = %command_display,
+            stdout = %stdout,
+            "Command output"
+        );
+
         if !output.status.success() {
-            let stderr = String::from_utf8_lossy(&output.stderr).to_string();
-            let stdout = String::from_utf8_lossy(&output.stdout).to_string();
             let exit_code = output
                 .status
                 .code()
@@ -123,7 +142,7 @@ impl CommandExecutor {
             });
         }
 
-        Ok(String::from_utf8_lossy(&output.stdout).to_string())
+        Ok(stdout)
     }
 }