feat: eliminate direct console output bypassing UserOutput

- Add blank_line(), steps(), and info_block() methods to UserOutput
- Replace all println! calls in template.rs with UserOutput methods
- Replace all eprintln! calls in handle_error with UserOutput methods
- Add comprehensive tests for new UserOutput methods
- All output now respects verbosity levels and goes through proper channels

Co-authored-by: josecelano <[email protected]>

Author: Copilot

src/presentation/commands/create/subcommands/template.rs

@@ -53,19 +53,24 @@ pub fn handle_template_generation(output_path: &Path) -> Result<(), CreateSubcom
         "Configuration template generated: {}",
         output_path.display()
     ));
-    println!();
-    println!("Next steps:");
-    println!("1. Edit the template file and replace placeholder values:");
-    println!("   - REPLACE_WITH_ENVIRONMENT_NAME: Choose a unique environment name (e.g., 'dev', 'staging')");
-    println!("   - REPLACE_WITH_SSH_PRIVATE_KEY_PATH: Path to your SSH private key");
-    println!("   - REPLACE_WITH_SSH_PUBLIC_KEY_PATH: Path to your SSH public key");
-    println!("2. Review default values:");
-    println!("   - username: 'torrust' (can be changed if needed)");
-    println!("   - port: 22 (standard SSH port)");
-    println!("3. Create the environment:");
-    println!(
-        "   torrust-tracker-deployer create environment --env-file {}",
-        output_path.display()
+
+    output.blank_line();
+
+    output.steps(
+        "Next steps:",
+        &[
+            "Edit the template file and replace placeholder values:",
+            "   - REPLACE_WITH_ENVIRONMENT_NAME: Choose a unique environment name (e.g., 'dev', 'staging')",
+            "   - REPLACE_WITH_SSH_PRIVATE_KEY_PATH: Path to your SSH private key",
+            "   - REPLACE_WITH_SSH_PUBLIC_KEY_PATH: Path to your SSH public key",
+            "Review default values:",
+            "   - username: 'torrust' (can be changed if needed)",
+            "   - port: 22 (standard SSH port)",
+            &format!(
+                "Create the environment:\n   torrust-tracker-deployer create environment --env-file {}",
+                output_path.display()
+            ),
+        ],
     );
 
     Ok(())

src/presentation/commands/mod.rs

@@ -119,7 +119,12 @@ pub fn execute(command: Commands, working_dir: &std::path::Path) -> Result<(), C
 /// # }
 /// ```
 pub fn handle_error(error: &CommandError) {
-    eprintln!("Error: {error}");
-    eprintln!("\nFor detailed troubleshooting:");
-    eprintln!("{}", error.help());
+    use crate::presentation::user_output::{UserOutput, VerbosityLevel};
+
+    let mut output = UserOutput::new(VerbosityLevel::Normal);
+    let help_text = error.help();
+
+    output.error(&format!("{error}"));
+    output.blank_line();
+    output.info_block("For detailed troubleshooting:", &[help_text]);
 }

src/presentation/user_output.rs

@@ -264,6 +264,86 @@ impl UserOutput {
     pub fn data(&mut self, data: &str) {
         writeln!(self.stdout_writer, "{data}").ok();
     }
+
+    /// Display a blank line to stderr (Normal level and above)
+    ///
+    /// Used for spacing between sections of output to improve readability.
+    ///
+    /// # Examples
+    ///
+    /// ```rust
+    /// use torrust_tracker_deployer_lib::presentation::user_output::{UserOutput, VerbosityLevel};
+    ///
+    /// let mut output = UserOutput::new(VerbosityLevel::Normal);
+    /// output.success("Configuration template generated");
+    /// output.blank_line();
+    /// output.progress("Starting next steps...");
+    /// ```
+    pub fn blank_line(&mut self) {
+        if self.verbosity >= VerbosityLevel::Normal {
+            writeln!(self.stderr_writer).ok();
+        }
+    }
+
+    /// Display a numbered list of steps to stderr (Normal level and above)
+    ///
+    /// Useful for displaying sequential instructions or action items.
+    ///
+    /// # Examples
+    ///
+    /// ```rust
+    /// use torrust_tracker_deployer_lib::presentation::user_output::{UserOutput, VerbosityLevel};
+    ///
+    /// let mut output = UserOutput::new(VerbosityLevel::Normal);
+    /// output.steps("Next steps:", &[
+    ///     "Edit the configuration file",
+    ///     "Review the settings",
+    ///     "Run the deploy command",
+    /// ]);
+    /// // Output to stderr:
+    /// // Next steps:
+    /// // 1. Edit the configuration file
+    /// // 2. Review the settings
+    /// // 3. Run the deploy command
+    /// ```
+    pub fn steps(&mut self, title: &str, steps: &[&str]) {
+        if self.verbosity >= VerbosityLevel::Normal {
+            writeln!(self.stderr_writer, "{title}").ok();
+            for (idx, step) in steps.iter().enumerate() {
+                writeln!(self.stderr_writer, "{}. {}", idx + 1, step).ok();
+            }
+        }
+    }
+
+    /// Display a multi-line information block to stderr (Normal level and above)
+    ///
+    /// Useful for displaying grouped information or detailed messages.
+    ///
+    /// # Examples
+    ///
+    /// ```rust
+    /// use torrust_tracker_deployer_lib::presentation::user_output::{UserOutput, VerbosityLevel};
+    ///
+    /// let mut output = UserOutput::new(VerbosityLevel::Normal);
+    /// output.info_block("Configuration options:", &[
+    ///     "  - username: 'torrust' (default)",
+    ///     "  - port: 22 (default SSH port)",
+    ///     "  - key_path: path/to/key",
+    /// ]);
+    /// // Output to stderr:
+    /// // Configuration options:
+    /// //   - username: 'torrust' (default)
+    /// //   - port: 22 (default SSH port)
+    /// //   - key_path: path/to/key
+    /// ```
+    pub fn info_block(&mut self, title: &str, lines: &[&str]) {
+        if self.verbosity >= VerbosityLevel::Normal {
+            writeln!(self.stderr_writer, "{title}").ok();
+            for line in lines {
+                writeln!(self.stderr_writer, "{line}").ok();
+            }
+        }
+    }
 }
 
 #[cfg(test)]
@@ -472,4 +552,101 @@ mod tests {
         assert!(normal >= VerbosityLevel::Normal);
         assert!(normal < VerbosityLevel::Verbose);
     }
+
+    #[test]
+    fn it_should_write_blank_line_to_stderr() {
+        let (mut output, stdout_buf, stderr_buf) = create_test_user_output(VerbosityLevel::Normal);
+
+        output.blank_line();
+
+        // Verify blank line went to stderr
+        let stderr_content = String::from_utf8(stderr_buf.lock().unwrap().clone()).unwrap();
+        assert_eq!(stderr_content, "\n");
+
+        // Verify stdout is empty
+        let stdout_content = String::from_utf8(stdout_buf.lock().unwrap().clone()).unwrap();
+        assert_eq!(stdout_content, "");
+    }
+
+    #[test]
+    fn it_should_not_write_blank_line_at_quiet_level() {
+        let (mut output, _stdout_buf, stderr_buf) = create_test_user_output(VerbosityLevel::Quiet);
+
+        output.blank_line();
+
+        // Verify no output at Quiet level
+        let stderr_content = String::from_utf8(stderr_buf.lock().unwrap().clone()).unwrap();
+        assert_eq!(stderr_content, "");
+    }
+
+    #[test]
+    fn it_should_write_steps_to_stderr() {
+        let (mut output, stdout_buf, stderr_buf) = create_test_user_output(VerbosityLevel::Normal);
+
+        output.steps(
+            "Next steps:",
+            &[
+                "Edit the configuration file",
+                "Review the settings",
+                "Run the deploy command",
+            ],
+        );
+
+        // Verify steps went to stderr with correct formatting
+        let stderr_content = String::from_utf8(stderr_buf.lock().unwrap().clone()).unwrap();
+        assert_eq!(
+            stderr_content,
+            "Next steps:\n1. Edit the configuration file\n2. Review the settings\n3. Run the deploy command\n"
+        );
+
+        // Verify stdout is empty
+        let stdout_content = String::from_utf8(stdout_buf.lock().unwrap().clone()).unwrap();
+        assert_eq!(stdout_content, "");
+    }
+
+    #[test]
+    fn it_should_not_write_steps_at_quiet_level() {
+        let (mut output, _stdout_buf, stderr_buf) = create_test_user_output(VerbosityLevel::Quiet);
+
+        output.steps("Next steps:", &["Step 1", "Step 2"]);
+
+        // Verify no output at Quiet level
+        let stderr_content = String::from_utf8(stderr_buf.lock().unwrap().clone()).unwrap();
+        assert_eq!(stderr_content, "");
+    }
+
+    #[test]
+    fn it_should_write_info_block_to_stderr() {
+        let (mut output, stdout_buf, stderr_buf) = create_test_user_output(VerbosityLevel::Normal);
+
+        output.info_block(
+            "Configuration options:",
+            &[
+                "  - username: 'torrust' (default)",
+                "  - port: 22 (default SSH port)",
+            ],
+        );
+
+        // Verify info block went to stderr
+        let stderr_content = String::from_utf8(stderr_buf.lock().unwrap().clone()).unwrap();
+        assert_eq!(
+            stderr_content,
+            "Configuration options:\n  - username: 'torrust' (default)\n  - port: 22 (default SSH port)\n"
+        );
+
+        // Verify stdout is empty
+        let stdout_content = String::from_utf8(stdout_buf.lock().unwrap().clone()).unwrap();
+        assert_eq!(stdout_content, "");
+    }
+
+    #[test]
+    fn it_should_not_write_info_block_at_quiet_level() {
+        let (mut output, _stdout_buf, stderr_buf) = create_test_user_output(VerbosityLevel::Quiet);
+
+        output.info_block("Info:", &["Line 1", "Line 2"]);
+
+        // Verify no output at Quiet level
+        let stderr_content = String::from_utf8(stderr_buf.lock().unwrap().clone()).unwrap();
+        assert_eq!(stderr_content, "");
+    }
 }