@orion-js/logger 3.2.0 → 3.2.36

package/lib/formats.js

@@ -6,6 +6,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.jsonConsoleTransport = exports.jsonConsoleFormat = exports.textConsoleTransport = exports.textConsoleFormat = exports.sentryFormat = void 0;
 const winston_1 = require("winston");
 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) {
@@ -25,7 +26,7 @@ exports.sentryFormat = (0, winston_1.format)(info => {
     };
 });
 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}` : ''} ${util_1.default.inspect(info.metadata)}`;
+    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)}`;
 }));
 exports.textConsoleTransport = new winston_1.transports.Console({
     handleExceptions: true,

package/lib/helpers/getFileName.d.ts

@@ -1,3 +1,2 @@
 export declare const getFileName: () => string;
 export declare const improveFileName: (path: string) => string;
-export declare const setFileName: (metadata: any) => void;

package/lib/helpers/getFileName.js

@@ -1,11 +1,11 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.setFileName = exports.improveFileName = exports.getFileName = void 0;
+exports.improveFileName = exports.getFileName = void 0;
 const getFileName = () => {
     try {
         const stack = new Error().stack;
         const lines = stack.split('\n');
-        const filePath = lines[4].split('(')[1].split(')')[0];
+        const filePath = lines[3].split('(')[1].split(')')[0];
         return (0, exports.improveFileName)(filePath);
     }
     catch (error) {
@@ -33,9 +33,3 @@ const improveFileName = (path) => {
     return path;
 };
 exports.improveFileName = improveFileName;
-const setFileName = (metadata) => {
-    if (metadata.fileName)
-        return;
-    metadata.fileName = (0, exports.getFileName)();
-};
-exports.setFileName = setFileName;

package/lib/index.test.js

@@ -7,5 +7,14 @@ describe('Log', () => {
         logger_1.logger.debug('debug');
         logger_1.logger.info('info');
         logger_1.logger.warn('warn');
+        logger_1.logger.warn('warn', { info: 'of the log' });
+    });
+    it('shouldnt fail when passing a non object to the metadata', () => {
+        logger_1.logger.info('info', 'a string');
+        logger_1.logger.info('info', ['array']);
+    });
+    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, metadata = {}) => {
-            (0, getFileName_1.setFileName)(metadata);
-            return logger.debug(message, metadata);
+        debug: (message, value = {}) => {
+            const fileName = (0, getFileName_1.getFileName)();
+            return logger.debug(message, { value, fileName });
         },
-        info: (message, metadata = {}) => {
-            (0, getFileName_1.setFileName)(metadata);
-            return logger.info(message, metadata);
+        info: (message, value = {}) => {
+            const fileName = (0, getFileName_1.getFileName)();
+            return logger.info(message, { value, fileName });
         },
-        warn: (message, metadata = {}) => {
-            (0, getFileName_1.setFileName)(metadata);
-            return logger.warn(message, metadata);
+        warn: (message, value = {}) => {
+            const fileName = (0, getFileName_1.getFileName)();
+            return logger.warn(message, { value, fileName });
         },
-        error: (message, metadata = {}) => {
-            (0, getFileName_1.setFileName)(metadata);
-            return logger.error(message, metadata);
+        error: (message, value = {}) => {
+            const fileName = (0, getFileName_1.getFileName)();
+            return logger.error(message, { value, fileName });
         },
         addContext: (module) => {
             if (module.id) {
@@ -56,7 +56,7 @@ const createLogger = (logger) => {
             return createLogger(logger.child({}));
         },
         addMetadata: (metadata) => {
-            return createLogger(logger.child(metadata));
+            return createLogger(logger.child({ parent: metadata }));
         }
     };
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@orion-js/logger",
-  "version": "3.2.0",
+  "version": "3.2.36",
   "main": "lib/index.js",
   "author": "nicolaslopezj",
   "license": "MIT",
@@ -13,11 +13,13 @@
     "upgrade-interactive": "yarn upgrade-interactive"
   },
   "dependencies": {
+    "lodash": "^4.17.21",
     "object-hash": "^2.2.0",
     "winston": "^3.6.0"
   },
   "devDependencies": {
     "@types/jest": "^27.0.2",
+    "@types/lodash": "^4.14.182",
     "jest": "27.3.1",
     "ts-jest": "27.0.7",
     "typescript": "^4.4.4"
@@ -25,5 +27,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "683cc5a018d45654ddf1299ea28e291fc7367dee"
+  "gitHead": "892890d7fed7597fa41d18c113c4d18aa58546bc"
 }