@crimsonsunset/jsg-logger 1.8.0 → 1.8.2

package/CHANGELOG.md

@@ -5,6 +5,21 @@ All notable changes to the JSG Logger project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.8.2] - 2026-03-27 🛡️ **Reinit Guard + configure() API**
+
+### Added
+- **`JSGLogger.configure(partialConfig)`** — new static method for post-init config updates. Merges partial config into the running instance without reinitializing or touching registered transports. Exposed on the default export as `logger.configure()` and on the class as `JSGLogger.configure()`. If called before any initialization has occurred, delegates to `getInstanceSync(partialConfig)`.
+
+### Fixed
+- **Reinit guard in `getInstanceSync()`** — calling `getInstanceSync(options)` on an already-initialized instance now returns the existing singleton instead of reinitializing. Previously this would wipe registered transports (e.g. PostHog transport) on every call. A `console.warn` is emitted to surface the misuse at call sites.
+- **Reinit guard in `getInstance()`** — same protection applied to the async path.
+
+### Changed
+- Removed stale comment "// If options are provided, we need to reinitialize even if global instance exists" — the behavior is now the opposite by design.
+
+### Migration
+No breaking changes. If you were relying on calling `getInstanceSync(options)` multiple times to update config post-init (which was always a footgun), switch to `logger.configure(partialConfig)` instead.
+
 ## [1.7.7] - 2025-11-07 🔧 **DevTools Import Path Fix**
 
 ### Fixed

package/index.js

@@ -81,14 +81,17 @@ class JSGLogger {
                 window.__JSG_Logger_Enhanced__ = JSGLogger._enhancedLoggers;
             }
         } else if (hasOptions) {
-            // Instance exists but new options provided - reinitialize
-            JSGLogger._enhancedLoggers = await JSGLogger._instance.init(options);
-            
-            // Make runtime controls available globally in browser for debugging
-            // (same as getInstanceSync behavior)
-            if (isBrowser() && typeof window !== 'undefined' && JSGLogger._enhancedLoggers?.controls) {
-                window.JSG_Logger = JSGLogger._enhancedLoggers.controls;
-                window.__JSG_Logger_Enhanced__ = JSGLogger._enhancedLoggers;
+            if (JSGLogger._instance.initialized) {
+                console.warn(
+                    '[JSGLogger] getInstance() called with options on an already-initialized instance — ' +
+                    'options were ignored to preserve registered transports. Use configure() to update settings post-init.'
+                );
+            } else {
+                JSGLogger._enhancedLoggers = await JSGLogger._instance.init(options);
+                if (isBrowser() && typeof window !== 'undefined' && JSGLogger._enhancedLoggers?.controls) {
+                    window.JSG_Logger = JSGLogger._enhancedLoggers.controls;
+                    window.__JSG_Logger_Enhanced__ = JSGLogger._enhancedLoggers;
+                }
             }
         }
 
@@ -104,21 +107,29 @@ class JSGLogger {
     static getInstanceSync(options = {}) {
         const hasOptions = options && Object.keys(options).length > 0;
         
-        // If options are provided, we need to reinitialize even if global instance exists
-        // This ensures config changes (like devtools.enabled) are applied
         if (hasOptions) {
-            // Reinitialize with new options - this will update the global references
+            if (JSGLogger._instance?.initialized) {
+                console.warn(
+                    '[JSGLogger] getInstanceSync() called with options on an already-initialized instance — ' +
+                    'options were ignored to preserve registered transports. Use configure() to update settings post-init.'
+                );
+                return JSGLogger._enhancedLoggers;
+            }
+
             if (!JSGLogger._instance) {
                 JSGLogger._instance = new JSGLogger();
             }
             JSGLogger._enhancedLoggers = JSGLogger._instance.initSync(options);
             
-            // Update global references after reinitialization
             if (isBrowser() && typeof window !== 'undefined' && JSGLogger._enhancedLoggers?.controls) {
                 window.JSG_Logger = JSGLogger._enhancedLoggers.controls;
                 window.__JSG_Logger_Enhanced__ = JSGLogger._enhancedLoggers;
             }
             
+            if (!isBrowser()) {
+                globalThis.__JSG_Logger_Enhanced__ = JSGLogger._enhancedLoggers;
+            }
+            
             return JSGLogger._enhancedLoggers;
         }
         
@@ -143,6 +154,13 @@ class JSGLogger {
             };
         }
 
+        // Server equivalent: check globalThis for cross-module-instance singleton
+        // Same problem as browser cross-bundle, but for Node.js bundled server chunks
+        if (!isBrowser() && globalThis.__JSG_Logger_Enhanced__) {
+            JSGLogger._enhancedLoggers = globalThis.__JSG_Logger_Enhanced__;
+            return JSGLogger._enhancedLoggers;
+        }
+
         // No options and no global instance - first time initialization
         if (!JSGLogger._instance) {
             JSGLogger._instance = new JSGLogger();
@@ -153,6 +171,11 @@ class JSGLogger {
                 window.JSG_Logger = JSGLogger._enhancedLoggers.controls;
                 window.__JSG_Logger_Enhanced__ = JSGLogger._enhancedLoggers;
             }
+
+            // Server equivalent: persist to globalThis for cross-bundle access
+            if (!isBrowser()) {
+                globalThis.__JSG_Logger_Enhanced__ = JSGLogger._enhancedLoggers;
+            }
         }
 
         return JSGLogger._enhancedLoggers;
@@ -542,6 +565,9 @@ class JSGLogger {
             // Expose config manager for runtime configuration
             configManager: configManager,
 
+            // Post-init config updater — safe to call after transports are registered
+            configure: (partialConfig) => JSGLogger.configure(partialConfig),
+
             // Log store for popup/debugging
             logStore: this.logStore,
 
@@ -970,6 +996,29 @@ class JSGLogger {
         }
     }
 
+    /**
+     * Update logger configuration post-initialization without reinitializing.
+     * Merges partialConfig into the current config without touching registered transports.
+     * Transports can only be registered at init time — this method cannot add or remove them.
+     * If called before any initialization has occurred, delegates to getInstanceSync(partialConfig).
+     * @param {Object} partialConfig - Partial config to merge into the current config
+     * @returns {Object} Enhanced logger exports
+     */
+    static configure(partialConfig = {}) {
+        if (!JSGLogger._instance?.initialized) {
+            return JSGLogger.getInstanceSync(partialConfig);
+        }
+
+        const currentTransports = JSGLogger._instance.transports;
+        const normalized = configManager._normalizeConfigStructure(partialConfig);
+        configManager.config = configManager.mergeConfigs(configManager.config, normalized);
+        configManager.config.transports = currentTransports;
+        JSGLogger._instance.transports = currentTransports;
+        JSGLogger._instance.refreshLoggers();
+
+        return JSGLogger._enhancedLoggers;
+    }
+
     /**
      * Get singleton controls without triggering initialization
      * Checks window.JSG_Logger first to ensure singleton works across separate bundles
@@ -1002,6 +1051,7 @@ if (isBrowser() && typeof window !== 'undefined') {
 // Add static methods to the enhanced loggers for convenience
 enhancedLoggers.getInstance = JSGLogger.getInstance;
 enhancedLoggers.getInstanceSync = JSGLogger.getInstanceSync;
+enhancedLoggers.configure = JSGLogger.configure.bind(JSGLogger);
 enhancedLoggers.logPerformance = JSGLogger.logPerformance;
 enhancedLoggers.JSGLogger = JSGLogger;
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@crimsonsunset/jsg-logger",
-  "version": "1.8.0",
+  "version": "1.8.2",
   "type": "module",
   "description": "Multi-environment logger with smart detection, file-level overrides, and beautiful console formatting. Test it live: https://logger.joesangiorgio.com/",
   "main": "index.js",