feat: compose commands

Author: KrLite

examples/docker_compose.rs

@@ -1,9 +1,9 @@
-//! Docker Compose example demonstrating multi-container application management
+//! Docker Compose example demonstrating multi-container application management.
 //!
 //! This example shows how to use the compose feature to manage
 //! multi-container applications with Docker Compose.
 //!
-//! Run with: cargo run --example docker_compose --features compose
+//! Run with: cargo run --example docker_compose --features compose.
 
 #[cfg(feature = "compose")]
 use docker_wrapper::compose::down::RemoveImages;

examples/exec_examples.rs

@@ -175,7 +175,7 @@ async fn env_exec_example(container_id: &str) {
         }
     }
 
-    // Multiple environment variables from HashMap
+    // multiple environment variables from HashMap
     let mut env_vars = HashMap::new();
     env_vars.insert("DEBUG".to_string(), "true".to_string());
     env_vars.insert("LOG_LEVEL".to_string(), "info".to_string());

src/command.rs

@@ -124,9 +124,10 @@ pub trait DockerCommand {
 }
 
 /// Progress output type for compose commands.
-#[derive(Debug, Clone, Copy)]
+#[derive(Debug, Default, Clone, Copy)]
 pub enum ProgressType {
     /// Auto-detects progress output.
+    #[default]
     Auto,
     /// TTY output.
     Tty,
@@ -151,13 +152,14 @@ impl std::fmt::Display for ProgressType {
 }
 
 /// ANSI control character mode.
-#[derive(Debug, Clone, Copy)]
+#[derive(Debug, Default, Clone, Copy)]
 pub enum AnsiMode {
     /// Never prints ANSI.
     Never,
     /// Always prints ANSI.
     Always,
     /// Auto-detects ANSI.
+    #[default]
     Auto,
 }
 

src/command/bake.rs

@@ -171,7 +171,7 @@ impl BakeCommand {
 
     /// Add a target to build
     ///
-    /// Multiple targets can be specified. If no targets are specified,
+    /// multiple targets can be specified. If no targets are specified,
     /// all targets defined in the bake file will be built.
     ///
     /// # Examples
@@ -941,9 +941,7 @@ mod tests {
     fn test_bake_command_extensibility() {
         let mut bake_cmd = BakeCommand::new();
         bake_cmd.executor_mut().add_arg("--experimental");
-        bake_cmd
-            .executor_mut()
-            .add_args(vec!["--custom", "value"]);
+        bake_cmd.executor_mut().add_args(vec!["--custom", "value"]);
 
         // Extensibility is handled through the executor's raw_args
         // The actual testing of raw args is done in command.rs tests

src/command/compose/attach.rs

@@ -1,41 +1,44 @@
 //! Docker Compose attach command implementation using unified trait pattern.
 
-use super::{CommandExecutor, ComposeCommand, ComposeConfig, DockerCommand};
-use crate::error::Result;
+use crate::{
+    compose::{ComposeCommand, ComposeConfig},
+    error::Result,
+    CommandExecutor, DockerCommand,
+};
 use async_trait::async_trait;
 
-/// Docker Compose attach command
+/// Docker Compose attach command.
 ///
 /// Attach to a running container's output.
 #[derive(Debug, Clone, Default)]
 pub struct ComposeAttachCommand {
-    /// Base command executor
+    /// Base command executor.
     pub executor: CommandExecutor,
-    /// Base compose configuration
+    /// Base compose configuration.
     pub config: ComposeConfig,
-    /// Service to attach to
+    /// Service to attach to.
     pub service: String,
-    /// Detach keys sequence
+    /// Detach keys sequence.
     pub detach_keys: Option<String>,
-    /// Container index if service has multiple instances
+    /// Container index if service has multiple instances.
     pub index: Option<u32>,
-    /// Don't stream STDIN
+    /// Doesn't stream STDIN.
     pub no_stdin: bool,
-    /// Use a pseudo-TTY
+    /// Uses a pseudo-TTY.
     pub sig_proxy: bool,
 }
 
-/// Result from attach command
+/// Result from attach command.
 #[derive(Debug, Clone)]
 pub struct AttachResult {
-    /// Output from the command
+    /// Output from the command.
     pub output: String,
-    /// Whether the operation succeeded
+    /// Whether the operation succeeded.
     pub success: bool,
 }
 
 impl ComposeAttachCommand {
-    /// Create a new attach command
+    /// Creates a new attach command.
     #[must_use]
     pub fn new(service: impl Into<String>) -> Self {
         Self {
@@ -47,28 +50,28 @@ impl ComposeAttachCommand {
         }
     }
 
-    /// Set detach keys
+    /// Sets detach keys.
     #[must_use]
     pub fn detach_keys(mut self, keys: impl Into<String>) -> Self {
         self.detach_keys = Some(keys.into());
         self
     }
 
-    /// Set container index
+    /// Sets container index.
     #[must_use]
     pub fn index(mut self, index: u32) -> Self {
         self.index = Some(index);
         self
     }
 
-    /// Don't attach to STDIN
+    /// Doesn't attach to STDIN.
     #[must_use]
     pub fn no_stdin(mut self) -> Self {
         self.no_stdin = true;
         self
     }
 
-    /// Disable signal proxy
+    /// Disables signal proxy.
     #[must_use]
     pub fn no_sig_proxy(mut self) -> Self {
         self.sig_proxy = false;
@@ -80,6 +83,10 @@ impl ComposeAttachCommand {
 impl DockerCommand for ComposeAttachCommand {
     type Output = AttachResult;
 
+    fn command_name() -> &'static str {
+        <Self as ComposeCommand>::command_name()
+    }
+
     fn executor(&self) -> &CommandExecutor {
         &self.executor
     }
@@ -89,7 +96,6 @@ impl DockerCommand for ComposeAttachCommand {
     }
 
     fn build_command_args(&self) -> Vec<String> {
-        // Use the ComposeCommand implementation explicitly
         <Self as ComposeCommand>::build_command_args(self)
     }
 
@@ -105,6 +111,10 @@ impl DockerCommand for ComposeAttachCommand {
 }
 
 impl ComposeCommand for ComposeAttachCommand {
+    fn subcommand_name() -> &'static str {
+        "attach"
+    }
+
     fn config(&self) -> &ComposeConfig {
         &self.config
     }
@@ -113,14 +123,10 @@ impl ComposeCommand for ComposeAttachCommand {
         &mut self.config
     }
 
-    fn subcommand(&self) -> &'static str {
-        "attach"
-    }
-
     fn build_subcommand_args(&self) -> Vec<String> {
         let mut args = Vec::new();
 
-        // Add flags
+        // add flags
         if let Some(ref keys) = self.detach_keys {
             args.push("--detach-keys".to_string());
             args.push(keys.clone());