Refactor Command into mode-based API

- Introduce ExecutionMode with Silent and Live
- Add Command<Mode> generic and mode types Capture and InheritStreams
- Implement build and call paths for Capture and InheritStreams
- Update Docker and HPC executor usage to new API
- Add tests for new modes and IO behavior

Author: lyskov

src/executor/docker.rs

@@ -36,21 +36,21 @@ impl Executor {
             fs::create_dir_all(&d)?;
         }
 
-        let mut command = util::Command::new("docker");
-
-        command
+        let command = util::Command::new("docker")
             .arg("run")
             .args(options.split(' '))
             .arg(&spec.image.0)
             .args(spec.args.clone())
             .message(format!(
                 "Executing {} with arguments: {:?}",
                 self.app, spec.args
-            ));
+            ))
+            .live();
 
         println!("Running {command}");
 
-        let result = command.live().call();
+        //let result = command.live().call();
+        let result = command.call();
 
         // let command_line = format!(
         //     "docker run {options} {} {}",

src/executor/hpc_container.rs

@@ -41,21 +41,20 @@ impl Executor {
             fs::create_dir_all(&d)?;
         }
 
-        let mut command = util::Command::new(engine.0);
-
-        command
+        let command = util::Command::new(engine.0)
             .arg("run")
             .args(options.split(' '))
             .arg(image_path.to_string_lossy())
             .args(spec.args.clone())
             .message(format!(
                 "Executing {} with arguments: {:?}",
                 self.app, spec.args
-            ));
+            ))
+            .live();
 
         println!("Running {command}");
 
-        let result = command.live().call();
+        let result = command.call();
 
         // println!("{}", result.stdout.bright_black());
         // eprintln!("{}", result.stderr.bright_red());