src: log error stack trace if provided (#451)

* src: log error stack trace if provided

Signed-off-by: flakey5 <[email protected]>

* fixup! src: log error stack trace if provided

Signed-off-by: flakey5 <[email protected]>

---------

Signed-off-by: flakey5 <[email protected]>

Author: flakey5

src/logger/__tests__/logger.test.mjs

@@ -203,7 +203,8 @@ describe('createLogger', () => {
 
     const logger = createLogger(transport, LogLevel.error);
 
-    logger.error(new Error('Hello, World!'));
+    const error = new Error('Hello, World!');
+    logger.error(error);
 
     strictEqual(transport.mock.callCount(), 1);
 
@@ -212,7 +213,9 @@ describe('createLogger', () => {
       {
         level: LogLevel.error,
         message: 'Hello, World!',
-        metadata: {},
+        metadata: {
+          stack: error.stack,
+        },
         module: undefined,
         timestamp: 0,
       },

src/logger/logger.mjs

@@ -40,7 +40,13 @@ export const createLogger = (
     const timestamp = Date.now();
 
     // Extract message string from Error object or use message as-is
-    const msg = message instanceof Error ? message.message : message;
+    let msg;
+    if (message instanceof Error) {
+      msg = message.message;
+      metadata.stack = message.stack;
+    } else {
+      msg = message;
+    }
 
     transport({
       level,

src/logger/transports/console.mjs

@@ -10,7 +10,7 @@ import { prettifyTimestamp } from '../utils/time.mjs';
  * @returns {void}
  */
 const console = ({ level, message, timestamp, metadata = {}, module }) => {
-  const { file } = metadata;
+  const { file, stack } = metadata;
 
   const time = prettifyTimestamp(timestamp);
 
@@ -37,6 +37,10 @@ const console = ({ level, message, timestamp, metadata = {}, module }) => {
   }
 
   process.stdout.write('\n');
+
+  if (stack) {
+    process.stdout.write(stack);
+  }
 };
 
 export default console;

src/logger/types.d.ts

@@ -14,6 +14,8 @@ export interface File {
 
 interface Metadata {
   file?: File;
+  // Stack trace if the message is an error
+  stack?: string;
 }
 
 interface TransportContext {