@trojs/logger 2.3.0 → 2.5.0

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@trojs/logger",
   "description": "Winston logger for TroJS",
-  "version": "2.3.0",
+  "version": "2.5.0",
   "author": {
     "name": "Pieter Wigboldus",
     "url": "https://trojs.org/"

package/src/logger.js

@@ -29,6 +29,7 @@ const levels = {
  * @param {Array<{[key: string]: string}>} [options.loggers=defaultLoggers] - Array of logger transport configurations.
  * @param {string} [options.level='info'] - Minimum log level for the logger.
  * @param {object} [options.meta={}] - Default metadata to include in all log messages.
+ * @param {boolean} [options.exitOnError=false] - Whether the logger should exit on error.
  * @returns {LoggerType} Winston logger instance with custom level wrappers.
  * These handlers will log errors and warnings using the logger, and are only attached once per process.
  * @example
@@ -36,14 +37,15 @@ const levels = {
  * const logger = createLogger({ level: 'debug', meta: { service: 'api' } });
  * logger.info('Service started');
  */
-export default ({ loggers = defaultLoggers, level = 'info', meta = {} } = {}) => {
+export default ({ loggers = defaultLoggers, level = 'info', meta = {}, exitOnError = false } = {}) => {
   const winstonLoggers = makeLoggers({ winston, loggers })
 
   const logger = winston.createLogger({
     level,
     levels,
     defaultMeta: meta,
-    transports: winstonLoggers
+    transports: winstonLoggers,
+    exitOnError
   })
 
   const wrapLevel = (lvl) => {

package/src/loggers/winston-transport-sentry-node.js

@@ -85,6 +85,26 @@ export class SentryTransport extends TransportStream {
 
     const sentryLevel = this.levelsMap[winstonLevel]
 
+    // Normalize message to string
+    let normalizedMessage = message
+    if (!normalizedMessage || normalizedMessage === '') {
+      // Try to get message from stack or error
+      if (meta.stack) {
+        normalizedMessage = (meta.stack.split('\n')[0] || '').trim()
+      } else if (meta.error instanceof Error) {
+        normalizedMessage = meta.error.message || meta.error.toString()
+      } else {
+        normalizedMessage = 'Empty log message'
+      }
+    } else if (typeof normalizedMessage !== 'string') {
+      // Stringify non-string messages
+      try {
+        normalizedMessage = JSON.stringify(normalizedMessage)
+      } catch {
+        normalizedMessage = String(normalizedMessage)
+      }
+    }
+
     return Sentry.withScope((scope) => {
       if (tags !== undefined && SentryTransport.isObject(tags)) {
         scope.setTags(tags)
@@ -100,11 +120,11 @@ export class SentryTransport extends TransportStream {
       if (SentryTransport.shouldLogException(sentryLevel)) {
         const error
           = Object.values(info).find((value) => value instanceof Error)
-            ?? new ExtendedError(info)
+            ?? new ExtendedError({ ...info, message: normalizedMessage })
         Sentry.captureException(error, { tags, level: sentryLevel })
       } else {
         // Capturing Messages
-        Sentry.captureMessage(message, sentryLevel)
+        Sentry.captureMessage(normalizedMessage, sentryLevel)
       }
     })
   }