refactor: restructure SSH module into dedicated submodules

- Convert src/command_wrappers/ssh.rs file into folder structure
- Create separate submodules for SSH components:
  - credentials.rs: SshCredentials struct and implementations
  - connection.rs: SshConnection struct and implementations
  - client.rs: SshClient struct and implementations + tests
  - mod.rs: SshError enum and module exports
- Move SshCredentials and SshConnection from src/config/ssh.rs to command_wrappers
- Update all imports throughout codebase to use new module paths
- Maintain backwards compatibility via config module re-exports
- Update documentation examples to reference new module paths

This improves code organization by grouping SSH-related types together
and creating focused, single-responsibility modules for better maintainability.

Author: josecelano

src/actions/cloud_init.rs

@@ -3,7 +3,7 @@ use tracing::info;
 
 use crate::actions::{RemoteAction, RemoteActionError};
 use crate::command_wrappers::ssh::SshClient;
-use crate::config::ssh::SshConnection;
+use crate::command_wrappers::ssh::SshConnection;
 
 /// Action that checks if cloud-init has completed successfully on the server
 pub struct CloudInitValidator {

src/actions/docker.rs

@@ -3,7 +3,7 @@ use tracing::{info, warn};
 
 use crate::actions::{RemoteAction, RemoteActionError};
 use crate::command_wrappers::ssh::SshClient;
-use crate::config::ssh::SshConnection;
+use crate::command_wrappers::ssh::SshConnection;
 
 /// Action that validates Docker installation and daemon status on the server
 pub struct DockerValidator {

src/actions/docker_compose.rs

@@ -3,7 +3,7 @@ use tracing::{info, warn};
 
 use crate::actions::{RemoteAction, RemoteActionError};
 use crate::command_wrappers::ssh::SshClient;
-use crate::config::ssh::SshConnection;
+use crate::command_wrappers::ssh::SshConnection;
 
 /// Action that validates Docker Compose installation and basic functionality on the server
 pub struct DockerComposeValidator {

src/command_wrappers/ssh.rs src/command_wrappers/ssh/client.rssrc/command_wrappers/ssh.rs renamed to src/command_wrappers/ssh/client.rs

@@ -1,29 +1,10 @@
 use std::time::Duration;
-use thiserror::Error;
 
 use tracing::info;
 
 use crate::command::{CommandError, CommandExecutor};
-use crate::config::ssh::SshConnection;
 
-/// Errors that can occur during SSH operations
-#[derive(Error, Debug)]
-pub enum SshError {
-    /// SSH connectivity could not be established within the timeout period
-    #[error("SSH connectivity to '{host_ip}' could not be established after {attempts} attempts ({timeout_seconds} seconds)")]
-    ConnectivityTimeout {
-        host_ip: String,
-        attempts: u32,
-        timeout_seconds: u32,
-    },
-
-    /// Underlying command execution failed
-    #[error("SSH command execution failed: {source}")]
-    CommandFailed {
-        #[source]
-        source: CommandError,
-    },
-}
+use super::{SshConnection, SshError};
 
 /// A specialized SSH client with predefined security settings
 ///
@@ -246,8 +227,8 @@ impl SshClient {
 
 #[cfg(test)]
 mod tests {
+    use super::super::SshCredentials;
     use super::*;
-    use crate::config::ssh::SshCredentials;
     use std::net::{IpAddr, Ipv4Addr};
     use std::path::PathBuf;
 

src/command_wrappers/ssh/connection.rs

@@ -0,0 +1,64 @@
+use std::net::IpAddr;
+use std::path::PathBuf;
+
+use super::SshCredentials;
+
+/// SSH connection configuration for a specific remote instance.
+///
+/// Contains both the SSH credentials and the target host IP address,
+/// representing everything needed to establish an SSH connection.
+#[derive(Clone)]
+pub struct SshConnection {
+    /// SSH authentication credentials.
+    pub credentials: SshCredentials,
+
+    /// IP address of the target host for SSH connections.
+    ///
+    /// This is the IP address of the remote instance that the SSH client
+    /// will connect to.
+    pub host_ip: IpAddr,
+}
+
+impl SshConnection {
+    /// Creates a new SSH connection configuration with the provided parameters.
+    ///
+    /// ```rust
+    /// # use std::net::{IpAddr, Ipv4Addr};
+    /// # use std::path::PathBuf;
+    /// # use torrust_tracker_deploy::command_wrappers::ssh::{SshCredentials, SshConnection};
+    /// let credentials = SshCredentials::new(
+    ///     PathBuf::from("/home/user/.ssh/deploy_key"),
+    ///     PathBuf::from("/home/user/.ssh/deploy_key.pub"),
+    ///     "ubuntu".to_string(),
+    /// );
+    /// let connection = SshConnection::new(
+    ///     credentials,
+    ///     IpAddr::V4(Ipv4Addr::new(192, 168, 1, 100)),
+    /// );
+    /// ```
+    #[must_use]
+    pub fn new(credentials: SshCredentials, host_ip: IpAddr) -> Self {
+        Self {
+            credentials,
+            host_ip,
+        }
+    }
+
+    /// Access the SSH private key path.
+    #[must_use]
+    pub fn ssh_priv_key_path(&self) -> &PathBuf {
+        &self.credentials.ssh_priv_key_path
+    }
+
+    /// Access the SSH public key path.
+    #[must_use]
+    pub fn ssh_pub_key_path(&self) -> &PathBuf {
+        &self.credentials.ssh_pub_key_path
+    }
+
+    /// Access the SSH username.
+    #[must_use]
+    pub fn ssh_username(&self) -> &str {
+        &self.credentials.ssh_username
+    }
+}

src/config/ssh.rs src/command_wrappers/ssh/credentials.rssrc/config/ssh.rs renamed to src/command_wrappers/ssh/credentials.rs

@@ -1,6 +1,8 @@
 use std::net::IpAddr;
 use std::path::PathBuf;
 
+use super::SshConnection;
+
 /// SSH credentials for remote instance authentication.
 ///
 /// Contains the static SSH authentication information that is known
@@ -33,7 +35,7 @@ impl SshCredentials {
     ///
     /// ```rust
     /// # use std::path::PathBuf;
-    /// # use torrust_tracker_deploy::config::ssh::SshCredentials;
+    /// # use torrust_tracker_deploy::command_wrappers::ssh::SshCredentials;
     /// let credentials = SshCredentials::new(
     ///     PathBuf::from("/home/user/.ssh/deploy_key"),
     ///     PathBuf::from("/home/user/.ssh/deploy_key.pub"),
@@ -65,63 +67,3 @@ impl SshCredentials {
         }
     }
 }
-
-/// SSH connection configuration for a specific remote instance.
-///
-/// Contains both the SSH credentials and the target host IP address,
-/// representing everything needed to establish an SSH connection.
-#[derive(Clone)]
-pub struct SshConnection {
-    /// SSH authentication credentials.
-    pub credentials: SshCredentials,
-
-    /// IP address of the target host for SSH connections.
-    ///
-    /// This is the IP address of the remote instance that the SSH client
-    /// will connect to.
-    pub host_ip: IpAddr,
-}
-
-impl SshConnection {
-    /// Creates a new SSH connection configuration with the provided parameters.
-    ///
-    /// ```rust
-    /// # use std::net::{IpAddr, Ipv4Addr};
-    /// # use std::path::PathBuf;
-    /// # use torrust_tracker_deploy::config::ssh::{SshCredentials, SshConnection};
-    /// let credentials = SshCredentials::new(
-    ///     PathBuf::from("/home/user/.ssh/deploy_key"),
-    ///     PathBuf::from("/home/user/.ssh/deploy_key.pub"),
-    ///     "ubuntu".to_string(),
-    /// );
-    /// let connection = SshConnection::new(
-    ///     credentials,
-    ///     IpAddr::V4(Ipv4Addr::new(192, 168, 1, 100)),
-    /// );
-    /// ```
-    #[must_use]
-    pub fn new(credentials: SshCredentials, host_ip: IpAddr) -> Self {
-        Self {
-            credentials,
-            host_ip,
-        }
-    }
-
-    /// Access the SSH private key path.
-    #[must_use]
-    pub fn ssh_priv_key_path(&self) -> &PathBuf {
-        &self.credentials.ssh_priv_key_path
-    }
-
-    /// Access the SSH public key path.
-    #[must_use]
-    pub fn ssh_pub_key_path(&self) -> &PathBuf {
-        &self.credentials.ssh_pub_key_path
-    }
-
-    /// Access the SSH username.
-    #[must_use]
-    pub fn ssh_username(&self) -> &str {
-        &self.credentials.ssh_username
-    }
-}

src/command_wrappers/ssh/mod.rs

@@ -0,0 +1,30 @@
+pub mod client;
+pub mod connection;
+pub mod credentials;
+
+pub use client::SshClient;
+pub use connection::SshConnection;
+pub use credentials::SshCredentials;
+
+use thiserror::Error;
+
+use crate::command::CommandError;
+
+/// Errors that can occur during SSH operations
+#[derive(Error, Debug)]
+pub enum SshError {
+    /// SSH connectivity could not be established within the timeout period
+    #[error("SSH connectivity to '{host_ip}' could not be established after {attempts} attempts ({timeout_seconds} seconds)")]
+    ConnectivityTimeout {
+        host_ip: String,
+        attempts: u32,
+        timeout_seconds: u32,
+    },
+
+    /// Underlying command execution failed
+    #[error("SSH command execution failed: {source}")]
+    CommandFailed {
+        #[source]
+        source: CommandError,
+    },
+}

src/config/mod.rs

@@ -1,7 +1,6 @@
 use std::path::PathBuf;
 
-pub mod ssh;
-pub use ssh::{SshConnection, SshCredentials};
+pub use crate::command_wrappers::ssh::{SshConnection, SshCredentials};
 
 /// Configuration parameters for deployment environments.
 ///

src/steps/infrastructure/wait_ssh_connectivity.rs

@@ -1,7 +1,6 @@
 use tracing::info;
 
-use crate::command_wrappers::ssh::{SshClient, SshError};
-use crate::config::SshConnection;
+use crate::command_wrappers::ssh::{SshClient, SshConnection, SshError};
 
 /// Step that waits for SSH connectivity to be established on a remote host
 pub struct WaitForSSHConnectivityStep {
@@ -54,7 +53,7 @@ impl WaitForSSHConnectivityStep {
 mod tests {
     use std::net::{IpAddr, Ipv4Addr};
 
-    use crate::config::ssh::SshCredentials;
+    use crate::command_wrappers::ssh::SshCredentials;
 
     use super::*;