Merge pull request #122 from aliev/feat/post-hook-print-stdout

feat: add option to print post-hook stdout

Author: aliev

README.md

@@ -684,6 +684,23 @@ This feature is especially helpful on Windows where scripts such as `.ps1`
 cannot be launched directly, and on Unix when you want to force a specific
 interpreter (e.g., Python, Node.js, Bash).
 
+### Displaying Post-hook Stdout
+
+By default, Baker stores `post` hook `stdout` in debug logs only. If you want
+users to see post-hook messages directly in the terminal (for example, welcome
+text or next-step instructions), enable `post_hook_print_stdout`:
+
+```yaml
+post_hook_filename: post.py
+post_hook_runner:
+  - python3
+post_hook_print_stdout: true
+```
+
+When enabled, Baker prints the post-hook `stdout` to the screen after the hook
+finishes. Keep in mind this output becomes visible in CI logs and terminal
+history, so hooks should avoid printing secrets.
+
 ### Available Platform Variables
 
 Baker provides these platform variables that can be used in templates and hook filenames:

src/cli/hooks.rs

@@ -40,6 +40,7 @@ pub fn run_hook<P: AsRef<Path>>(
     hook_path: P,
     answers: Option<&serde_json::Value>,
     runner: &[String],
+    inherit_stdout: bool,
 ) -> Result<Option<String>> {
     let hook_path = hook_path.as_ref();
 
@@ -70,7 +71,7 @@ pub fn run_hook<P: AsRef<Path>>(
 
     let mut child = command
         .stdin(Stdio::piped())
-        .stdout(Stdio::piped())
+        .stdout(if inherit_stdout { Stdio::inherit() } else { Stdio::piped() })
         .stderr(Stdio::inherit())
         .spawn()?;
 
@@ -151,6 +152,7 @@ mod tests {
             &script_path,
             None,
             &["sh".to_string()],
+            false,
         )
         .expect("hook execution")
         .expect("stdout");
@@ -177,10 +179,16 @@ mod tests {
             "-File".to_string(),
         ];
 
-        let output =
-            run_hook(temp_dir.path(), temp_dir.path(), &script_path, None, &runner)
-                .expect("hook execution")
-                .expect("stdout");
+        let output = run_hook(
+            temp_dir.path(),
+            temp_dir.path(),
+            &script_path,
+            None,
+            &runner,
+            false,
+        )
+        .expect("hook execution")
+        .expect("stdout");
 
         assert!(output.contains("windows_runner"));
     }

src/cli/processor.rs

@@ -288,6 +288,7 @@ mod tests {
                 pre_hook_filename: "pre".into(),
                 post_hook_runner: Vec::new(),
                 pre_hook_runner: Vec::new(),
+                post_hook_print_stdout: false,
                 follow_symlinks,
             },
             skip_confirms,

src/cli/runner.rs

@@ -112,6 +112,7 @@ impl Runner {
         )?;
         let pre_hook_runner = config.pre_hook_runner.clone();
         let post_hook_runner = config.post_hook_runner.clone();
+        let post_hook_print_stdout = config.post_hook_print_stdout;
 
         let execute_hooks = self.confirm_hook_execution(
             context.template_root(),
@@ -139,6 +140,7 @@ impl Runner {
             execute_hooks,
             pre_hook_runner,
             post_hook_runner,
+            post_hook_print_stdout,
         })
     }
 
@@ -170,6 +172,7 @@ impl Runner {
                 &hook_plan.pre_hook_file,
                 None,
                 &runner,
+                false,
             )
         } else {
             Ok(None)
@@ -236,6 +239,7 @@ impl Runner {
                 &hook_plan.post_hook_file,
                 Some(context.answers()),
                 &runner,
+                hook_plan.post_hook_print_stdout,
             )?;
 
             if let Some(result) = post_hook_stdout {
@@ -452,6 +456,7 @@ struct HookPlan {
     execute_hooks: bool,
     pre_hook_runner: Vec<String>,
     post_hook_runner: Vec<String>,
+    post_hook_print_stdout: bool,
 }
 
 fn render_hook_runner(

src/config/loader.rs

@@ -34,6 +34,8 @@ pub struct ConfigV1 {
     pub post_hook_runner: Vec<String>,
     #[serde(default = "get_default_pre_hook_runner")]
     pub pre_hook_runner: Vec<String>,
+    #[serde(default = "get_default_post_hook_print_stdout")]
+    pub post_hook_print_stdout: bool,
     #[serde(default = "get_default_follow_symlinks")]
     pub follow_symlinks: bool,
 }
@@ -110,6 +112,10 @@ fn get_default_pre_hook_runner() -> Vec<String> {
     Vec::new()
 }
 
+fn get_default_post_hook_print_stdout() -> bool {
+    false
+}
+
 fn get_default_loop_separator() -> String {
     DEFAULT_LOOP_SEPARATOR.to_string()
 }
@@ -172,6 +178,25 @@ questions: {}"#;
         assert!(!cfg.follow_symlinks);
     }
 
+    #[test]
+    fn post_hook_print_stdout_defaults_false() {
+        let raw = r#"schemaVersion: v1
+questions: {}"#;
+        let config: Config = serde_yaml::from_str(raw).expect("valid config");
+        let Config::V1(cfg) = config;
+        assert!(!cfg.post_hook_print_stdout);
+    }
+
+    #[test]
+    fn post_hook_print_stdout_parses_true() {
+        let raw = r#"schemaVersion: v1
+post_hook_print_stdout: true
+questions: {}"#;
+        let config: Config = serde_yaml::from_str(raw).expect("valid config");
+        let Config::V1(cfg) = config;
+        assert!(cfg.post_hook_print_stdout);
+    }
+
     #[test]
     fn follow_symlinks_parses_true() {
         let raw = r#"schemaVersion: v1

src/template/processor.rs

@@ -427,6 +427,7 @@ mod tests {
                 pre_hook_filename: "pre".into(),
                 post_hook_runner: Vec::new(),
                 pre_hook_runner: Vec::new(),
+                post_hook_print_stdout: false,
                 follow_symlinks: false,
             },
             Vec::new(),