fix: Fix pwsh git hook exit codes. (#1764)

* Add exit code.

* Update changelog.

* Fix snap.

* Fix tests.

CHANGELOG.md

@@ -2,6 +2,10 @@
 
 ## Unreleased
 
+#### 🐞 Fixes
+
+- Fixed an issue with PowerShell Git hooks not bubbling up exit codes of failed commands.
+
 #### ⚙️ Internal
 
 - Updated proto to v0.44.1 (from 0.43.1).

crates/vcs-hooks/src/hooks_generator.rs

@@ -202,6 +202,8 @@ impl<'app> HooksGenerator<'app> {
                     "#!/usr/bin/env powershell"
                 },
                 "$ErrorActionPreference = 'Stop'",
+                // https://learn.microsoft.com/en-us/powershell/scripting/learn/experimental-features?view=powershell-7.4#psnativecommanderroractionpreference
+                "$PSNativeCommandErrorActionPreference = $true",
                 "",
             ]);
         }
@@ -216,7 +218,19 @@ impl<'app> HooksGenerator<'app> {
 
         for command in commands {
             contents.push(command);
+
+            // https://github.com/moonrepo/moon/issues/1761
+            if !self.is_bash_format() {
+                contents.extend([
+                    "",
+                    "if ($LASTEXITCODE -ne 0) {",
+                    "  exit $LASTEXITCODE",
+                    "}",
+                    "",
+                ]);
+            }
         }
+
         contents.push("\n");
 
         self.create_file(file_path, contents.join("\n"))?;

crates/vcs-hooks/tests/snapshots/hooks_generator_test__windows__creates_local_hook_files-2.snap

@@ -1,11 +1,16 @@
 ---
 source: crates/vcs-hooks/tests/hooks_generator_test.rs
-expression: "fs::read_to_string(post_push).unwrap()"
+expression: "clean_powershell(fs::read_to_string(post_push).unwrap())"
 ---
 #!/usr/bin/env pwsh
 $ErrorActionPreference = 'Stop'
+$PSNativeCommandErrorActionPreference = $true
 
 # Automatically generated by moon. DO NOT MODIFY!
 # https://moonrepo.dev/docs/guides/vcs-hooks
 
 moon check --all
+
+if ($LASTEXITCODE -ne 0) {
+  exit $LASTEXITCODE
+}

crates/vcs-hooks/tests/snapshots/hooks_generator_test__windows__creates_local_hook_files.snap

@@ -1,12 +1,22 @@
 ---
 source: crates/vcs-hooks/tests/hooks_generator_test.rs
-expression: "fs::read_to_string(pre_commit).unwrap()"
+expression: "clean_powershell(fs::read_to_string(pre_commit).unwrap())"
 ---
 #!/usr/bin/env pwsh
 $ErrorActionPreference = 'Stop'
+$PSNativeCommandErrorActionPreference = $true
 
 # Automatically generated by moon. DO NOT MODIFY!
 # https://moonrepo.dev/docs/guides/vcs-hooks
 
 moon run :lint
+
+if ($LASTEXITCODE -ne 0) {
+  exit $LASTEXITCODE
+}
+
 some-command
+
+if ($LASTEXITCODE -ne 0) {
+  exit $LASTEXITCODE
+}