hermodlog 1.10.0 → 2.0.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hermodlog",
-  "version": "1.10.0",
+  "version": "2.0.0",
   "description": "Simple contextual logger to simplify log display in heavy load context and provide easy parsing",
   "main": "src/Logger.js",
   "type": "module",

package/src/Logger.js

@@ -32,7 +32,7 @@ class Logger {
                 enumerable: false
             },
             historyLimit: {
-                value: props.historyLimit ?? 100,
+                value: props.historyLimit ?? 0,
                 writable: true,
                 enumerable: true
             },
@@ -81,7 +81,6 @@ class Logger {
                 writable: true,
                 enumerable: true
             }
-
         });
 
         if (props.date) {
@@ -114,11 +113,13 @@ class Logger {
         const keepListener = opts?.keepListener ?? true;
         const keepObject = opts?.keepObject ?? true;
         const keepHandler = opts?.keepHandler ?? true;
+        const historyLimit = opts?.historyLimit ?? null;
 
         return new Logger({
             level: (keepLevel) ? this.level : opts.level ?? this.level,
             date: (keepDate) ? this.date : opts.date ?? null,
             history: keepHistory ? this.history : [],
+            historyLimit: historyLimit ?? this.historyLimit,
             colors: (keepColors) ? this.LOG_COLORS : opts.colors ?? this.LOG_COLORS,
             contextName: (keepContext) ? this.contextName : opts.contextName ?? null,
             methodName: (keepMethod) ? this.methodName : opts.methodName ?? null,

package/src/Logger.spec.js

@@ -5,7 +5,7 @@ describe('Logger', () => {
     let silentLogger
     let silentModuleLogger
     it('should create an logger', () => {
-        silentLogger = new Logger({date:new Date('2023-07-29T01:38:00.482Z')});
+        silentLogger = new Logger({date:new Date('2023-07-29T01:38:00.482Z'), historyLimit: 100});
         silentLogger._log = () => {}
         expect(silentLogger.history.length).toBe(0)
         silentLogger.log('Hello');
@@ -42,7 +42,7 @@ describe('Logger', () => {
         expect(silentMethodLogger.history[0]).toBe(expected)
     })
     it('should handle level', function () {
-        const levelLogger = new Logger({level: 'error'})
+        const levelLogger = new Logger({level: 'error', historyLimit: 100});
         levelLogger._log = () => {}
         levelLogger.error('Hello');
         expect(levelLogger.history.length).toBe(1)
@@ -67,4 +67,18 @@ describe('Logger', () => {
         const expected =  `[\u001b[90m2023-07-29T01:38:00.482Z\u001b[0m] module:\u001b[90mtest-module\u001b[0m - \u001b[32mReceived from ws client\u001b[0m \u001b[32mObject(\u001b[0m\u001b[32m{\n  \"x\": 42,\n  \"y\": {\n    \"b\": [\n      1,\n      2,\n      3\n    ]\n  },\n  \"req\": \"getCar\"\n}\u001b[0m\u001b[32m)\u001b[0m`;
         expect(silentModuleLogger.history[1]).toBe(expected)
     });
+
+    it('should transmit history config to child loggers', () => {
+        const parentLogger = new Logger({date:new Date('2023-07-29T01:38:00.482Z'), historyLimit: 123});
+        parentLogger._log = () => {}
+        parentLogger.log('hello');
+        expect(parentLogger.historyLimit).toBe(123)
+        expect(parentLogger.history.length).toBe(1)
+
+        const childLogger = parentLogger.listener('onTest')
+        expect(childLogger.historyLimit).toBe(123)
+        expect(childLogger.history.length).toBe(0)
+        childLogger.log('Hello');
+        expect(childLogger.history.length).toBe(1)
+    })
 })

package/src/utils/log.js

@@ -166,10 +166,12 @@ export default function log(level, args, context) {
         }
     }
 
+    const historyLimit = context?.historyLimit;
+    if (historyLimit) {
     context.history.push(message);
-    const historyLimit = context.historyLimit || 100;
-    if (context.history.length > historyLimit) {
-        context.history.shift();
+        if (context.history.length > historyLimit) {
+            context.history.shift();
+        }
     }
 
     context._log(message);