@trojs/logger 2.4.0 → 2.5.1

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@trojs/logger",
   "description": "Winston logger for TroJS",
-  "version": "2.4.0",
+  "version": "2.5.1",
   "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) => {
@@ -81,39 +83,14 @@ export default ({ loggers = defaultLoggers, level = 'info', meta = {} } = {}) =>
     if (typeof logger[lvl] === 'function') wrapLevel(lvl)
   })
 
+  // Winston handles uncaughtException and unhandledRejection via transport
+  // handleExceptions and handleRejections options.
+  // exitOnError controls whether the process exits after logging.
+
+  // Only attach warning handler (not handled by Winston transports)
   if (!process.__trojsLoggerHandlersAttached) {
     process.__trojsLoggerHandlersAttached = true
 
-    process.on('uncaughtException', (err) => {
-      try {
-        logger.error(err instanceof Error ? err : new Error(String(err)))
-      } catch {
-        // eslint-disable-next-line no-console
-        console.error('UNCAUGHT_EXCEPTION', err)
-      }
-    })
-
-    process.on('unhandledRejection', (reason) => {
-      let err
-      if (reason instanceof Error) {
-        err = reason
-      } else if (typeof reason === 'string') {
-        err = new Error(reason)
-      } else {
-        try {
-          err = new Error(JSON.stringify(reason))
-        } catch {
-          err = new Error(String(reason))
-        }
-      }
-      try {
-        logger.error(err)
-      } catch {
-        // eslint-disable-next-line no-console
-        console.error('UNHANDLED_REJECTION', err)
-      }
-    })
-
     process.on('warning', (warning) => {
       try {
         logger.warn(

package/src/loggers/combined-file.js

@@ -1,4 +1,6 @@
 export default ({ winston, logger }) =>
   new winston.transports.File({
-    filename: logger?.location || 'combined.log'
+    filename: logger?.location || 'combined.log',
+    handleExceptions: true,
+    handleRejections: true
   })

package/src/loggers/error-file.js

@@ -1,5 +1,7 @@
 export default ({ winston, logger }) =>
   new winston.transports.File({
     filename: logger?.location || 'error.log',
-    level: logger?.level || 'error'
+    level: logger?.level || 'error',
+    handleExceptions: true,
+    handleRejections: true
   })

package/src/loggers/sentry.js

@@ -11,7 +11,9 @@ export default ({ logger }) => {
       sampleRate: logger?.sampleRate || 1,
       tracesSampleRate: logger?.tracesSampleRate || 1
     },
-    level: logger?.level || 'info'
+    level: logger?.level || 'info',
+    handleExceptions: true,
+    handleRejections: true
   }
 
   return new SentryTransport(options)