hb-smart-logger 1.0.5 → 1.0.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hb-smart-logger",
-  "version": "1.0.5",
+  "version": "1.0.7",
   "description": "Production-ready Winston logger with daily rotation, safe JSON handling, and logger.log() alias",
   "main": "src/index.js",
   "publishConfig": {

package/src/index.js

@@ -1,12 +1,19 @@
 const fs = require("fs");
 const path = require("path");
-const winston = require("winston");
+
+/* ---------------------------------------------------
+   LOAD ENV (dotenv)
+--------------------------------------------------- */
 const dotenv = require("dotenv");
-const DailyRotateFile = require("winston-daily-rotate-file");
 dotenv.config();
 
-// Debug flag (explicit opt-in)
+const winston = require("winston");
+const DailyRotateFile = require("winston-daily-rotate-file");
+
+// Explicit debug opt-in
 const DEBUG_ENABLED = String(process.env.DEBUG).toLowerCase() === "true";
+const FILE_SIZE = String(process.env.LOG_FILE_SIZE).toLowerCase() === "50m";
+const FILE_TIME = String(process.env.LOG_FILE_TIME).toLowerCase() === "30d";
 
 // Ensure logs directory exists
 const logDir = path.resolve("logs");
@@ -60,10 +67,10 @@ const logFormat = winston.format.printf(
 );
 
 /* ---------------------------------------------------
-   BASE LOGGER (INFO ALWAYS ENABLED)
+   BASE LOGGER
 --------------------------------------------------- */
 const baseLogger = winston.createLogger({
-  level: "info", // 🔒 never suppress info
+  level: "debug", // allow all levels internally
   format: winston.format.combine(
     winston.format.timestamp({ format: "YYYY-MM-DD HH:mm:ss" }),
     winston.format.errors({ stack: true }),
@@ -73,23 +80,23 @@ const baseLogger = winston.createLogger({
 });
 
 /* ---------------------------------------------------
-   ROTATING FILES
+   FILE TRANSPORTS
 --------------------------------------------------- */
 
-// Combined logs → info, warn, error ALWAYS
+// Combined logs
 baseLogger.add(
   new DailyRotateFile({
     dirname: path.join(logDir, "combined"),
     filename: "combined-%DATE%.log",
-    level: "info",
+    level: DEBUG_ENABLED ? "debug" : "info",
     datePattern: "YYYY-MM-DD",
     zippedArchive: true,
-    maxSize: "10m",
-    maxFiles: "30d",
+    maxSize: FILE_SIZE,
+    maxFiles: FILE_TIME,
   })
 );
 
-// Per-level logs (always enabled)
+// Always-on per-level logs
 ["error", "warn", "info"].forEach((level) => {
   baseLogger.add(
     new DailyRotateFile({
@@ -98,13 +105,13 @@ baseLogger.add(
       level,
       datePattern: "YYYY-MM-DD",
       zippedArchive: true,
-      maxSize: "10m",
-      maxFiles: "30d",
+      maxSize: FILE_SIZE,
+      maxFiles: FILE_TIME,
     })
   );
 });
 
-// Debug logs ONLY if enabled
+// Debug file ONLY when enabled
 if (DEBUG_ENABLED) {
   baseLogger.add(
     new DailyRotateFile({
@@ -113,20 +120,18 @@ if (DEBUG_ENABLED) {
       level: "debug",
       datePattern: "YYYY-MM-DD",
       zippedArchive: true,
-      maxSize: "10m",
-      maxFiles: "30d",
+      maxSize: FILE_SIZE,
+      maxFiles: FILE_TIME,
     })
   );
 }
 
 /* ---------------------------------------------------
-   CONSOLE OUTPUT
+   CONSOLE TRANSPORT (NO DUPLICATES)
 --------------------------------------------------- */
-
-// Info+ always to console
 baseLogger.add(
   new winston.transports.Console({
-    level: "info",
+    level: DEBUG_ENABLED ? "debug" : "info",
     format: winston.format.combine(
       winston.format.colorize(),
       winston.format.simple()
@@ -134,19 +139,6 @@ baseLogger.add(
   })
 );
 
-// Debug to console ONLY if enabled
-if (DEBUG_ENABLED) {
-  baseLogger.add(
-    new winston.transports.Console({
-      level: "debug",
-      format: winston.format.combine(
-        winston.format.colorize(),
-        winston.format.simple()
-      ),
-    })
-  );
-}
-
 /* ---------------------------------------------------
    WRAPPER (MULTI-ARGS + logger.log)
 --------------------------------------------------- */
@@ -154,10 +146,10 @@ const wrapLogger = (logger) => {
   const levels = ["info", "warn", "error", "debug"];
 
   for (const level of levels) {
-    const original = logger[level]?.bind(logger);
+    const original = logger[level].bind(logger);
 
     logger[level] = (...args) => {
-      // Hard-disable debug when DEBUG is false
+      // Hard-disable debug if DEBUG=false
       if (level === "debug" && !DEBUG_ENABLED) return;
 
       const merged = args