🐝 Add command debug logging

rpedretti · Oct 19, 2024 · cd63be8 · cd63be8
1 parent 17c5577
commit cd63be8
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 12 deletions.
diff --git a/src/common/exec.ts b/src/common/exec.ts
@@ -47,18 +47,16 @@ export async function exec(options: { command: string; args: string[]; cwd?: str
     });
   }
 
-  if (result.stdout && result.stderr) {
-    commonLogger.warn(`Both stdout and stderr are not empty for "${options.command}" command`, {
-      stdout: result.stdout,
-      stderr: result.stderr,
-      command: options.command,
-      args: options.args,
-      cwd: cwd,
-    });
-    return result.stdout;
-  }
+  commonLogger.debug("Command executed", {
+    command: options.command,
+    args: options.args,
+    cwd: cwd,
+    stdout: result.stdout,
+    stderr: result.stderr,
+  });
 
-  if (result.stderr) {
+  // check error code
+  if (result.stderr && !result.stdout) {
     throw new ExecErrror(`Error executing "${options.command}" command`, {
       stderr: result.stderr,
       command: options.command,

diff --git a/src/common/logger.ts b/src/common/logger.ts
@@ -113,7 +113,7 @@ export class Logger {
   }
 
   error(message: string, context: Context & { error: unknown | null }) {
-    const stackTrace = context.error instanceof Error ? context.error.stack ?? "" : "";
+    const stackTrace = context.error instanceof Error ? (context.error.stack ?? "") : "";
     this.addMessage({
       message: message,
       level: LogLevel.error,