@stacksjs/cli 0.59.10 → 0.59.11

package/dist/index.js

@@ -37,9 +37,8 @@ async function exec(command, options) {
     stderr: options?.silent || options?.quiet ? "ignore" : options?.stderr || "inherit",
     detached: options?.background || false,
     cwd: options?.cwd || import.meta.dir,
-    onExit(_subprocess, exitCode, _signalCode, _error) {
-      if (exitCode && exitCode !== ExitCode.Success)
-        process2.exit(exitCode);
+    onExit(subprocess, exitCode, signalCode, error) {
+      exitHandler("spawn", subprocess, exitCode, signalCode, error);
     }
   });
   if (options?.stdin === "pipe" && options.input) {
@@ -67,13 +66,24 @@ async function execSync(command, options) {
     stdout: options?.stdout ?? "pipe",
     stderr: options?.stderr ?? "inherit",
     cwd: options?.cwd ?? import.meta.dir,
-    onExit(_subprocess, exitCode, _signalCode, _error) {
-      if (exitCode !== ExitCode.Success && exitCode)
-        process2.exit(exitCode);
+    onExit(subprocess, exitCode, signalCode, error) {
+      exitHandler("spawnSync", subprocess, exitCode, signalCode, error);
     }
   });
   return proc.stdout.toString();
 }
+var exitHandler = function(type, subprocess, exitCode, signalCode, error) {
+  log2.debug(`exitHandler: ${type}`);
+  log2.debug("subprocess", subprocess);
+  log2.debug("exitCode", exitCode);
+  log2.debug("signalCode", signalCode);
+  if (error) {
+    log2.error(error);
+    process2.exit(ExitCode.FatalError);
+  }
+  if (exitCode !== ExitCode.Success && exitCode)
+    process2.exit(exitCode);
+};
 
 // src/utils.ts
 import {collect} from "@stacksjs/collections";
@@ -520,7 +530,7 @@ import {log as log3} from "@stacksjs/logging";
 import {ExitCode as ExitCode2} from "@stacksjs/types";
 import {bgCyan as bgCyan2, bold as bold2, cyan as cyan2, dim as dim2, gray as gray2, green as green2, italic as italic2} from "kolorist";
 // package.json
-var version = "0.59.10";
+var version = "0.59.11";
 
 // src/helpers.ts
 async function intro(command2, options) {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@stacksjs/cli",
   "type": "module",
-  "version": "0.59.10",
+  "version": "0.59.11",
   "description": "TypeScript framework for CLI artisans. Build beautiful console apps with ease.",
   "author": "Chris Breuer",
   "license": "MIT",

package/src/exec.ts

@@ -43,9 +43,8 @@ export async function exec(command: string | string[], options?: CliOptions): Pr
     detached: options?.background || false,
     cwd: options?.cwd || import.meta.dir,
     // env: { ...e, ...options?.env },
-    onExit(_subprocess, exitCode, _signalCode, _error) {
-      if (exitCode && exitCode !== ExitCode.Success)
-        process.exit(exitCode)
+    onExit(subprocess, exitCode, signalCode, error) {
+      exitHandler('spawn', subprocess, exitCode, signalCode, error)
     },
   })
 
@@ -103,12 +102,26 @@ export async function execSync(command: string | string[], options?: CliOptions)
     stderr: options?.stderr ?? 'inherit',
     cwd: options?.cwd ?? import.meta.dir,
     // env: { ...Bun.env, ...options?.env },
-    onExit(_subprocess, exitCode, _signalCode, _error) {
-      // console.log('onExit', { subprocess, exitCode, signalCode, error })
-      if (exitCode !== ExitCode.Success && exitCode)
-        process.exit(exitCode)
+    onExit(subprocess, exitCode, signalCode, error) {
+      exitHandler('spawnSync', subprocess, exitCode, signalCode, error)
     },
   })
 
   return proc.stdout.toString()
 }
+
+// @ts-expect-error - missing types is okay here but can be improved later on
+function exitHandler(type: 'spawn' | 'spawnSync', subprocess, exitCode, signalCode, error) {
+  log.debug(`exitHandler: ${type}`)
+  log.debug('subprocess', subprocess)
+  log.debug('exitCode', exitCode)
+  log.debug('signalCode', signalCode)
+
+  if (error) {
+    log.error(error)
+    process.exit(ExitCode.FatalError)
+  }
+
+  if (exitCode !== ExitCode.Success && exitCode)
+    process.exit(exitCode)
+}