chore: better error logs

jamesgeorge007 · Sep 26, 2023 · dbd3749 · dbd3749
1 parent 762c517
commit dbd3749
Show file tree

Hide file tree

Showing 2 changed files with 40 additions and 18 deletions.
diff --git a/dist/index.js b/dist/index.js
@@ -1840,20 +1840,31 @@ const toUrlFormat = (item) => {
 
 const exec = (cmd, args = []) =>
   new Promise((resolve, reject) => {
-    const app = spawn(cmd, args, { stdio: "pipe" });
+    const app = spawn(cmd, args, { stdio: "inherit" });
+
     let stdout = "";
-    app.stdout.on("data", (data) => {
-      stdout = data;
-    });
+    if (app.stdout) {
+      app.stdout.on("data", (data) => {
+        stdout = data;
+      });
+    }
+
+    let stderr = "";
+    if (app.stderr) {
+      app.stderr.on("data", (data) => {
+        stderr = data;
+      });
+    }
+
     app.on("close", (code) => {
       if (code !== 0 && !stdout.includes("nothing to commit")) {
-        err = new Error(`Invalid status code: ${code}`);
-        err.code = code;
-        return reject(err);
+        return reject({ code, stderr });
       }
-      return resolve(code);
+
+      return resolve({ code, stdout });
     });
-    app.on("error", reject);
+
+    app.on("error", () => reject({ code, stderr }));
   });
 
 /**

diff --git a/index.js b/index.js
@@ -57,20 +57,31 @@ const toUrlFormat = (item) => {
 
 const exec = (cmd, args = []) =>
   new Promise((resolve, reject) => {
-    const app = spawn(cmd, args, { stdio: "pipe" });
+    const app = spawn(cmd, args, { stdio: "inherit" });
+
     let stdout = "";
-    app.stdout.on("data", (data) => {
-      stdout = data;
-    });
+    if (app.stdout) {
+      app.stdout.on("data", (data) => {
+        stdout = data;
+      });
+    }
+
+    let stderr = "";
+    if (app.stderr) {
+      app.stderr.on("data", (data) => {
+        stderr = data;
+      });
+    }
+
     app.on("close", (code) => {
       if (code !== 0 && !stdout.includes("nothing to commit")) {
-        err = new Error(`Invalid status code: ${code}`);
-        err.code = code;
-        return reject(err);
+        return reject({ code, stderr });
       }
-      return resolve(code);
+
+      return resolve({ code, stdout });
     });
-    app.on("error", reject);
+
+    app.on("error", () => reject({ code, stderr }));
   });
 
 /**