feat: add array logging

Author: araujogui

bin/commands/list.mjs

@@ -25,5 +25,5 @@ export default function list(type) {
         ? availableRules
         : availableTransports;
 
-  Logger.getInstance().info(list.join('\n'));
+  Logger.getInstance().info(list);
 }

src/logger/__tests__/logger.test.mjs

@@ -181,4 +181,36 @@ describe('createLogger', () => {
 
     strictEqual(transport.mock.callCount(), 0);
   });
+
+  it('should log all messages if message is a string array', t => {
+    const transport = t.mock.fn();
+
+    const logger = createLogger(transport, LogLevel.info);
+
+    logger.info(['Hello, 1!', 'Hello, 2!', 'Hello, 3!']);
+
+    strictEqual(transport.mock.callCount(), 3);
+  });
+
+  it.only('should log error message', t => {
+    t.mock.timers.enable({ apis: ['Date'] });
+
+    const transport = t.mock.fn();
+
+    const logger = createLogger(transport, LogLevel.error);
+
+    logger.error(new Error('Hello, World!'));
+
+    strictEqual(transport.mock.callCount(), 1);
+
+    const call = transport.mock.calls[0];
+    deepStrictEqual(call.arguments, [
+      {
+        level: LogLevel.error,
+        message: 'Hello, World!',
+        metadata: {},
+        timestamp: 0,
+      },
+    ]);
+  });
 });

src/logger/logger.mjs

@@ -2,6 +2,11 @@
 
 import { LogLevel } from './constants.mjs';
 
+/**
+ * @typedef {import('./types').Metadata} Metadata
+ * @typedef {import('./types').LogMessage} LogMessage
+ */
+
 /**
  * Creates a logger instance with the specified transport, log level and an
  * optional module name.
@@ -19,19 +24,27 @@ export const createLogger = (
    * Logs a message at the given level with optional metadata.
    *
    * @param {number} level - Log level for the message.
-   * @param {string} message - Message to log.
-   * @param {import('./types').Metadata} metadata - Additional metadata
+   * @param {LogMessage} message - Message to log.
+   * @param {Metadata} metadata - Additional metadata
+   * @returns {void}
    */
   const log = (level, message, metadata = {}) => {
     if (!shouldLog(level)) {
       return;
     }
 
+    if (Array.isArray(message)) {
+      return message.forEach(msg => log(level, msg, metadata));
+    }
+
     const timestamp = Date.now();
 
+    // Extract message string from Error object or use message as-is
+    const msg = message instanceof Error ? message.message : message;
+
     transport({
       level,
-      message,
+      message: msg,
       timestamp,
       metadata,
       ...(module && { module }),
@@ -41,8 +54,8 @@ export const createLogger = (
   /**
    * Logs an info message.
    *
-   * @param {string} message - Info message to log.
-   * @param {import('./types').Metadata} metadata - Additional metadata
+   * @param {LogMessage} message - Info message to log.
+   * @param {Metadata} metadata - Additional metadata
    * @returns {void}
    */
   const info = (message, metadata = {}) =>
@@ -51,8 +64,8 @@ export const createLogger = (
   /**
    * Logs a warning message.
    *
-   * @param {string} message - Warning message to log.
-   * @param {import('./types').Metadata} metadata - Additional metadata
+   * @param {LogMessage} message - Warning message to log.
+   * @param {Metadata} metadata - Additional metadata
    * @returns {void}
    */
   const warn = (message, metadata = {}) =>
@@ -61,34 +74,28 @@ export const createLogger = (
   /**
    * Logs an error message or Error object.
    *
-   * @param {string | Error} input - Error message or Error object to log.
-   * @param {import('./types').Metadata} metadata - Additional metadata
+   * @param {LogMessage} message - Error message or Error object to log.
+   * @param {Metadata} metadata - Additional metadata
    * @returns {void}
    */
-  const error = (input, metadata = {}) => {
-    const message = typeof input === 'string' ? input : input.message;
-
+  const error = (message, metadata = {}) =>
     log(LogLevel.error, message, metadata);
-  };
 
   /**
    * Logs a fatal error message or Error object.
    *
-   * @param {string | Error} input - Fatal error message or Error object to log.
-   * @param {import('./types').Metadata} metadata - Additional metadata
+   * @param {LogMessage} message - Fatal error message or Error object to log.
+   * @param {Metadata} metadata - Additional metadata
    * @returns {void}
    */
-  const fatal = (input, metadata = {}) => {
-    const message = typeof input === 'string' ? input : input.message;
-
+  const fatal = (message, metadata = {}) =>
     log(LogLevel.fatal, message, metadata);
-  };
 
   /**
    * Logs a debug message.
    *
-   * @param {string} message - Debug message to log.
-   * @param {import('./types').Metadata} metadata - Additional metadata
+   * @param {LogMessage} message - Debug message to log.
+   * @param {Metadata} metadata - Additional metadata
    * @returns {void}
    */
   const debug = (message, metadata = {}) =>

src/logger/types.d.ts

@@ -1,5 +1,7 @@
 export type LogLevel = 'info' | 'warn' | 'error' | 'fatal' | 'trace' | 'debug';
 
+export type LogMessage = string | Error | string[];
+
 export interface Position {
   start: { line: number };
   end: { line: number };