@gratheon/log-lib 2.2.5 → 2.2.7

package/dist/logger.js

@@ -34,6 +34,7 @@ const fs = __importStar(require("fs"));
 const path = __importStar(require("path"));
 let conn = null;
 let dbInitialized = false;
+let initPromise = null;
 const LOG_LEVELS = {
     debug: 0,
     info: 1,
@@ -107,7 +108,7 @@ async function initializeConnection(config) {
         INDEX idx_level (level)
       ) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_general_ci;
     `);
-        // Run migrations: Add stacktrace column if it doesn't exist (for existing tables)
+        // Run migrations: Add stacktrace column if it doesn't exist (for existing tables created before v2.2.0)
         try {
             const columns = await conn.query((0, mysql_1.sql) `SHOW COLUMNS FROM \`logs\` LIKE 'stacktrace'`);
             if (columns.length === 0) {
@@ -115,12 +116,16 @@ async function initializeConnection(config) {
                 await conn.query((0, mysql_1.sql) `ALTER TABLE \`logs\` ADD COLUMN \`stacktrace\` TEXT AFTER \`meta\``);
                 console.log('[log-lib] Migration complete: stacktrace column added');
             }
+            else {
+                console.log('[log-lib] Migration check: stacktrace column already exists');
+            }
         }
         catch (migrationErr) {
             console.error('[log-lib] Migration failed (non-critical):', migrationErr);
             // Don't fail initialization if migration fails
         }
         dbInitialized = true;
+        console.log('[log-lib] Database initialization complete');
     }
     catch (err) {
         console.error('Failed to initialize logs database:', err);
@@ -312,26 +317,39 @@ function safeMeta(meta) {
     return meta;
 }
 function storeInDB(level, message, meta, stacktrace) {
-    if (!conn || !dbInitialized) {
-        // Database not ready yet, skip DB logging
+    if (!conn) {
+        // Database not configured, skip DB logging
         return;
     }
-    try {
-        const msg = safeToStringMessage(message);
-        const metaObj = safeMeta(meta);
-        const metaStr = (0, fast_safe_stringify_1.default)(metaObj).slice(0, 2000);
-        const stackStr = stacktrace || '';
-        // Fire and forget; avoid awaiting in hot path. Catch errors to avoid unhandled rejection.
-        conn.query((0, mysql_1.sql) `INSERT INTO \`logs\` (level, message, meta, stacktrace, timestamp) VALUES (${level}, ${msg}, ${metaStr}, ${stackStr}, NOW())`).catch(e => {
+    // Wait for initialization to complete before writing
+    const doStore = async () => {
+        if (initPromise) {
+            await initPromise.catch(() => { }); // Wait but ignore errors
+        }
+        if (!dbInitialized) {
+            // Initialization failed, skip DB logging
+            return;
+        }
+        try {
+            const msg = safeToStringMessage(message);
+            const metaObj = safeMeta(meta);
+            const metaStr = (0, fast_safe_stringify_1.default)(metaObj).slice(0, 2000);
+            const stackStr = stacktrace || '';
+            await conn.query((0, mysql_1.sql) `INSERT INTO \`logs\` (level, message, meta, stacktrace, timestamp) VALUES (${level}, ${msg}, ${metaStr}, ${stackStr}, NOW())`);
+        }
+        catch (e) {
             // fallback console output only - but don't spam
             if (process.env.ENV_ID === 'dev') {
                 console.error('Failed to persist log to DB', e);
             }
-        });
-    }
-    catch (e) {
-        console.error('Unexpected failure preparing log for DB', e);
-    }
+        }
+    };
+    // Fire and forget
+    doStore().catch(e => {
+        if (process.env.ENV_ID === 'dev') {
+            console.error('Unexpected failure preparing log for DB', e);
+        }
+    });
 }
 function createLogger(config = {}) {
     // Set up log level filtering
@@ -340,11 +358,9 @@ function createLogger(config = {}) {
         process.env.LOG_LEVEL ||
         (process.env.ENV_ID === 'dev' ? 'debug' : 'info');
     currentLogLevel = LOG_LEVELS[configuredLevel] ?? LOG_LEVELS.info;
-    // Start initialization asynchronously but don't wait for it (only if MySQL config provided)
+    // Start initialization asynchronously (only if MySQL config provided)
     if (config.mysql) {
-        initializeConnection(config).catch(err => {
-            console.error('Error during log database initialization:', err);
-        });
+        initPromise = initializeConnection(config);
     }
     const logger = {
         info: (message, meta) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gratheon/log-lib",
-  "version": "2.2.5",
+  "version": "2.2.7",
   "description": "Logging library with console and MySQL database persistence",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/src/logger.ts

@@ -7,6 +7,7 @@ import { LoggerConfig, Logger, FastifyLogger, LogMetadata, LogLevel } from "./ty
 
 let conn: ConnectionPool | null = null;
 let dbInitialized = false;
+let initPromise: Promise<void> | null = null;
 
 const LOG_LEVELS: Record<LogLevel, number> = {
   debug: 0,
@@ -91,13 +92,15 @@ async function initializeConnection(config: LoggerConfig) {
       ) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_general_ci;
     `);
     
-    // Run migrations: Add stacktrace column if it doesn't exist (for existing tables)
+    // Run migrations: Add stacktrace column if it doesn't exist (for existing tables created before v2.2.0)
     try {
       const columns = await conn.query(sql`SHOW COLUMNS FROM \`logs\` LIKE 'stacktrace'`);
       if (columns.length === 0) {
         console.log('[log-lib] Running migration: Adding stacktrace column...');
         await conn.query(sql`ALTER TABLE \`logs\` ADD COLUMN \`stacktrace\` TEXT AFTER \`meta\``);
         console.log('[log-lib] Migration complete: stacktrace column added');
+      } else {
+        console.log('[log-lib] Migration check: stacktrace column already exists');
       }
     } catch (migrationErr) {
       console.error('[log-lib] Migration failed (non-critical):', migrationErr);
@@ -105,6 +108,7 @@ async function initializeConnection(config: LoggerConfig) {
     }
     
     dbInitialized = true;
+    console.log('[log-lib] Database initialization complete');
   } catch (err) {
     console.error('Failed to initialize logs database:', err);
     // Don't throw - allow the service to start even if logging DB fails
@@ -294,25 +298,43 @@ function safeMeta(meta: any): any {
 }
 
 function storeInDB(level: string, message: any, meta?: any, stacktrace?: string) {
-  if (!conn || !dbInitialized) {
-    // Database not ready yet, skip DB logging
+  if (!conn) {
+    // Database not configured, skip DB logging
     return;
   }
-  try {
-    const msg = safeToStringMessage(message);
-    const metaObj = safeMeta(meta);
-    const metaStr = jsonStringify(metaObj).slice(0, 2000);
-    const stackStr = stacktrace || '';
-    // Fire and forget; avoid awaiting in hot path. Catch errors to avoid unhandled rejection.
-    conn.query(sql`INSERT INTO \`logs\` (level, message, meta, stacktrace, timestamp) VALUES (${level}, ${msg}, ${metaStr}, ${stackStr}, NOW())`).catch(e => {
+  
+  // Wait for initialization to complete before writing
+  const doStore = async () => {
+    if (initPromise) {
+      await initPromise.catch(() => {}); // Wait but ignore errors
+    }
+    
+    if (!dbInitialized) {
+      // Initialization failed, skip DB logging
+      return;
+    }
+    
+    try {
+      const msg = safeToStringMessage(message);
+      const metaObj = safeMeta(meta);
+      const metaStr = jsonStringify(metaObj).slice(0, 2000);
+      const stackStr = stacktrace || '';
+      
+      await conn!.query(sql`INSERT INTO \`logs\` (level, message, meta, stacktrace, timestamp) VALUES (${level}, ${msg}, ${metaStr}, ${stackStr}, NOW())`);
+    } catch (e: any) {
       // fallback console output only - but don't spam
       if (process.env.ENV_ID === 'dev') {
         console.error('Failed to persist log to DB', e);
       }
-    });
-  } catch (e) {
-    console.error('Unexpected failure preparing log for DB', e);
-  }
+    }
+  };
+  
+  // Fire and forget
+  doStore().catch(e => {
+    if (process.env.ENV_ID === 'dev') {
+      console.error('Unexpected failure preparing log for DB', e);
+    }
+  });
 }
 
 export function createLogger(config: LoggerConfig = {}): { logger: Logger; fastifyLogger: FastifyLogger } {
@@ -324,11 +346,9 @@ export function createLogger(config: LoggerConfig = {}): { logger: Logger; fasti
   
   currentLogLevel = LOG_LEVELS[configuredLevel] ?? LOG_LEVELS.info;
   
-  // Start initialization asynchronously but don't wait for it (only if MySQL config provided)
+  // Start initialization asynchronously (only if MySQL config provided)
   if (config.mysql) {
-    initializeConnection(config).catch(err => {
-      console.error('Error during log database initialization:', err);
-    });
+    initPromise = initializeConnection(config);
   }
 
   const logger: Logger = {