Add output and error handling to hook-runner

mirego · Mar 1, 2024 · 0ae10dd · 0ae10dd
1 parent d8e3a70
commit 0ae10dd
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 3 deletions.
diff --git a/cli/src/services/formatters/hook-runner.ts b/cli/src/services/formatters/hook-runner.ts
@@ -14,4 +14,11 @@ export default class HookRunnerFomatter {
     });
     console.log('');
   }
+  error(command: string, errors: string[]) {
+    console.log(chalk.red('⚠ '), chalk.bold(chalk.red(`${command}:`)));
+    errors.forEach((error) => {
+      console.log('  ', chalk.red(error));
+    });
+    console.log('');
+  }
 }
diff --git a/cli/src/services/hook-runner.ts b/cli/src/services/hook-runner.ts
@@ -23,9 +23,18 @@ export default class HookRunner {
     const hooks = this.hooks[name];
 
     if (hooks) {
-      new Formatter().log(name, hooks);
-
-      hooks.forEach(execSync);
+      const formatter = new Formatter()
+      formatter.log(name, hooks);
+
+      hooks.forEach((hook) => {
+        try {
+          const output: string = execSync(hook, { stdio: 'pipe' }).toString();
+          formatter.log(hook, [output])
+        } catch (error: any) {
+          formatter.error(hook, [error.stderr.toString()]);
+          throw new Error(`Hook execution failed for '${hook}': ${error.stderr.toString()}`);
+        }
+      });
     }
 
     return this.document.refreshPaths();