@orion-js/logger 3.2.36 → 3.2.37

package/lib/formats.js

@@ -9,9 +9,15 @@ const util_1 = __importDefault(require("util"));
 const lodash_1 = require("lodash");
 const { metadata, timestamp, json, colorize, combine, printf } = winston_1.format;
 const metaError = (0, winston_1.format)(info => {
-    if (info.metadata && info.metadata.error && info.metadata.error instanceof Error) {
-        info.stack = info.metadata.error.stack;
-        info.metadata.error = info.metadata.error.message;
+    if (info?.metadata?.value?.error instanceof Error) {
+        info.stack = info?.metadata?.value?.error.stack;
+        info.errorMessage = info?.metadata?.value?.error.message;
+        delete info?.metadata?.value?.error;
+    }
+    if (info?.metadata?.value instanceof Error) {
+        info.stack = info?.metadata?.value.stack;
+        info.errorMessage = info?.metadata?.value.message;
+        delete info?.metadata?.value;
     }
     return info;
 });
@@ -25,8 +31,23 @@ exports.sentryFormat = (0, winston_1.format)(info => {
         }
     };
 });
+function getMetadataText(metadata) {
+    const { value, ...rest } = metadata;
+    if ((0, lodash_1.isEmpty)(rest)) {
+        if (typeof value === 'undefined')
+            return '';
+        return util_1.default.inspect(value);
+    }
+    return `${util_1.default.inspect(value)} ${util_1.default.inspect(rest)}`;
+}
 exports.textConsoleFormat = combine(colorize(), metadata({ fillExcept: ['fileName', 'level', 'message', 'stack'] }), metaError(), timestamp(), printf(info => {
-    return `[${info.level}] [${info.timestamp}] ${info.fileName ? `[${info.fileName}]` : ''} ${info.message} ${info.stack ? `\n${info.stack}` : ''} ${(0, lodash_1.isEmpty)(info.metadata?.value) ? '' : util_1.default.inspect(info.metadata?.value)} ${(0, lodash_1.isEmpty)(info.metadata?.parent) ? '' : util_1.default.inspect(info.metadata?.parent)}`;
+    // console.log(info)
+    const date = new Date(info.timestamp);
+    const timeLabel = `${date.getHours()}:${date.getMinutes()}:${date.getSeconds()}`;
+    const fileNameLabel = info.fileName ? `[${info.fileName}]` : '';
+    const stack = info.stack ? `\n${info.stack}` : '';
+    const value = getMetadataText(info.metadata);
+    return `[${info.level}] [${timeLabel}] ${fileNameLabel} ${info.message} ${value} ${stack}`;
 }));
 exports.textConsoleTransport = new winston_1.transports.Console({
     handleExceptions: true,

package/lib/index.test.js

@@ -13,6 +13,10 @@ describe('Log', () => {
         logger_1.logger.info('info', 'a string');
         logger_1.logger.info('info', ['array']);
     });
+    it('should log errors correctly', () => {
+        logger_1.logger.error('an error', new Error('message'));
+        logger_1.logger.error('an error2', { error: new Error('message'), info: 'of the log' });
+    });
     it('should pass the context', () => {
         const withMeta = logger_1.logger.addMetadata({ context: 'context' });
         withMeta.info('info', { notContext: 'notContext' });

package/lib/logger.js

@@ -31,21 +31,21 @@ const getLogger = (context) => {
 exports.getLogger = getLogger;
 const createLogger = (logger) => {
     return {
-        debug: (message, value = {}) => {
+        debug: (message, value) => {
             const fileName = (0, getFileName_1.getFileName)();
-            return logger.debug(message, { value, fileName });
+            return logger.debug({ message, fileName, value });
         },
-        info: (message, value = {}) => {
+        info: (message, value) => {
             const fileName = (0, getFileName_1.getFileName)();
-            return logger.info(message, { value, fileName });
+            return logger.info({ message, fileName, value });
         },
-        warn: (message, value = {}) => {
+        warn: (message, value) => {
             const fileName = (0, getFileName_1.getFileName)();
-            return logger.warn(message, { value, fileName });
+            return logger.warn({ message, fileName, value });
         },
-        error: (message, value = {}) => {
+        error: (message, value) => {
             const fileName = (0, getFileName_1.getFileName)();
-            return logger.error(message, { value, fileName });
+            return logger.error({ message, fileName, value });
         },
         addContext: (module) => {
             if (module.id) {
@@ -56,7 +56,7 @@ const createLogger = (logger) => {
             return createLogger(logger.child({}));
         },
         addMetadata: (metadata) => {
-            return createLogger(logger.child({ parent: metadata }));
+            return createLogger(logger.child(metadata));
         }
     };
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@orion-js/logger",
-  "version": "3.2.36",
+  "version": "3.2.37",
   "main": "lib/index.js",
   "author": "nicolaslopezj",
   "license": "MIT",
@@ -27,5 +27,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "892890d7fed7597fa41d18c113c4d18aa58546bc"
+  "gitHead": "f64b1082ff22931fc49c2df07bb477eb14ac9df2"
 }