feat: awkward builders

Author: KrLite

src/command/attach.rs

@@ -10,7 +10,7 @@ use async_trait::async_trait;
 ///
 /// Attach local standard input, output, and error streams to a running container.
 ///
-/// # Example
+/// # Examples
 ///
 /// ```no_run
 /// use docker_wrapper::AttachCommand;
@@ -46,7 +46,7 @@ pub struct AttachCommand {
 impl AttachCommand {
     /// Create a new attach command
     ///
-    /// # Example
+    /// # Examples
     ///
     /// ```
     /// use docker_wrapper::AttachCommand;
@@ -66,7 +66,7 @@ impl AttachCommand {
 
     /// Override the key sequence for detaching a container
     ///
-    /// # Example
+    /// # Examples
     ///
     /// ```
     /// use docker_wrapper::AttachCommand;
@@ -82,7 +82,7 @@ impl AttachCommand {
 
     /// Do not attach STDIN
     ///
-    /// # Example
+    /// # Examples
     ///
     /// ```
     /// use docker_wrapper::AttachCommand;
@@ -98,7 +98,7 @@ impl AttachCommand {
 
     /// Do not proxy signals
     ///
-    /// # Example
+    /// # Examples
     ///
     /// ```
     /// use docker_wrapper::AttachCommand;
@@ -120,7 +120,7 @@ impl AttachCommand {
     /// - The container doesn't exist
     /// - The container is not running
     ///
-    /// # Example
+    /// # Examples
     ///
     /// ```no_run
     /// use docker_wrapper::AttachCommand;

src/command/builder/build.rs

@@ -12,7 +12,7 @@ use async_trait::async_trait;
 /// This is essentially the same as `docker build` but accessed through
 /// the builder subcommand interface.
 ///
-/// # Example
+/// # Examples
 ///
 /// ```no_run
 /// use docker_wrapper::command::builder::BuilderBuildCommand;
@@ -33,64 +33,65 @@ use async_trait::async_trait;
 /// ```
 #[derive(Debug, Clone)]
 pub struct BuilderBuildCommand {
-    /// Underlying build command
+    /// Underlying build command.
     inner: BuildCommand,
 }
 
 impl BuilderBuildCommand {
-    /// Create a new builder build command
+    /// Creates a new builder build command.
     ///
     /// # Arguments
-    /// * `context` - Build context path (e.g., ".", "/path/to/dir")
+    ///
+    /// * `context` - Build context path (e.g., `.`, `/path/to/dir`).
     pub fn new(context: impl Into<String>) -> Self {
         Self {
             inner: BuildCommand::new(context),
         }
     }
 
-    /// Set the Dockerfile to use
+    /// Sets the Dockerfile to use.
     #[must_use]
     pub fn dockerfile(mut self, path: impl Into<String>) -> Self {
         self.inner = self.inner.file(path.into());
         self
     }
 
-    /// Tag the image
+    /// Tags the image.
     #[must_use]
     pub fn tag(mut self, tag: impl Into<String>) -> Self {
         self.inner = self.inner.tag(tag);
         self
     }
 
-    /// Do not use cache when building
+    /// Do not use cache when building.
     #[must_use]
     pub fn no_cache(mut self) -> Self {
         self.inner = self.inner.no_cache();
         self
     }
 
-    /// Set build-time variables
+    /// Sets build-time variables.
     #[must_use]
     pub fn build_arg(mut self, key: impl Into<String>, value: impl Into<String>) -> Self {
         self.inner = self.inner.build_arg(key, value);
         self
     }
 
-    /// Set target build stage
+    /// Sets target build stage.
     #[must_use]
     pub fn target(mut self, target: impl Into<String>) -> Self {
         self.inner = self.inner.target(target);
         self
     }
 
-    /// Set platform for multi-platform builds
+    /// Sets platform for multi-platform builds.
     #[must_use]
     pub fn platform(mut self, platform: impl Into<String>) -> Self {
         self.inner = self.inner.platform(platform);
         self
     }
 
-    /// Enable `BuildKit` backend
+    /// Enables `BuildKit` backend.
     #[must_use]
     pub fn buildkit(mut self) -> Self {
         // This would normally set DOCKER_BUILDKIT=1 environment variable
@@ -102,35 +103,35 @@ impl BuilderBuildCommand {
         self
     }
 
-    /// Enable quiet mode
+    /// Enables quiet mode.
     #[must_use]
     pub fn quiet(mut self) -> Self {
         self.inner = self.inner.quiet();
         self
     }
 
-    /// Always remove intermediate containers
+    /// Always removes intermediate containers.
     #[must_use]
     pub fn force_rm(mut self) -> Self {
         self.inner = self.inner.force_rm();
         self
     }
 
-    /// Remove intermediate containers after successful build (default)
+    /// Removes intermediate containers after successful build (default).
     #[must_use]
     pub fn rm(self) -> Self {
         // rm is the default behavior, this is a no-op
         self
     }
 
-    /// Do not remove intermediate containers after build
+    /// Does not remove intermediate containers after build.
     #[must_use]
     pub fn no_rm(mut self) -> Self {
         self.inner = self.inner.no_rm();
         self
     }
 
-    /// Always attempt to pull newer version of base image
+    /// Always attempts to pull newer version of base image.
     #[must_use]
     pub fn pull(mut self) -> Self {
         self.inner = self.inner.pull();
@@ -142,7 +143,7 @@ impl BuilderBuildCommand {
 impl DockerCommand for BuilderBuildCommand {
     type Output = BuildOutput;
 
-    fn command_name() -> &str {
+    fn command_name() -> &'static str {
         "builder build"
     }
 
@@ -155,15 +156,15 @@ impl DockerCommand for BuilderBuildCommand {
     }
 
     fn build_command_args(&self) -> Vec<String> {
-        self.inner.build_command_args();
+        self.inner.build_command_args()
     }
 
     async fn execute(&self) -> Result<Self::Output> {
-        // The builder build command has the same output as regular build
+        // the builder build command has the same output as regular build
         let args = self.build_command_args();
         let output = self.inner.executor.execute_command("docker", args).await?;
 
-        // Extract image ID from output
+        // extract image ID from output
         let image_id = extract_image_id(&output.stdout);
 
         Ok(BuildOutput {
@@ -175,9 +176,9 @@ impl DockerCommand for BuilderBuildCommand {
     }
 }
 
-/// Extract image ID from build output
+/// Extracts image ID from build output.
 fn extract_image_id(stdout: &str) -> Option<String> {
-    // Look for "Successfully built <id>" or "writing image sha256:<id>"
+    // look for "Successfully built <id>" or "writing image sha256:<id>"
     for line in stdout.lines().rev() {
         if line.contains("Successfully built") {
             return line.split_whitespace().last().map(String::from);

src/command/builder/mod.rs

@@ -1,4 +1,4 @@
-//! Docker builder commands for build cache management
+//! Docker builder commands for build cache management.
 //!
 //! This module provides support for Docker builder commands that manage
 //! the build subsystem and build cache.