@crimsonsunset/jsg-logger 1.8.7 → 1.8.8

package/devtools/dist/vite.svg

@@ -0,0 +1 @@
+<svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" aria-hidden="true" role="img" class="iconify iconify--logos" width="31.88" height="32" preserveAspectRatio="xMidYMid meet" viewBox="0 0 256 257"><defs><linearGradient id="IconifyId1813088fe1fbc01fb466" x1="-.828%" x2="57.636%" y1="7.652%" y2="78.411%"><stop offset="0%" stop-color="#41D1FF"></stop><stop offset="100%" stop-color="#BD34FE"></stop></linearGradient><linearGradient id="IconifyId1813088fe1fbc01fb467" x1="43.376%" x2="50.316%" y1="2.242%" y2="89.03%"><stop offset="0%" stop-color="#FFEA83"></stop><stop offset="8.333%" stop-color="#FFDD35"></stop><stop offset="100%" stop-color="#FFA800"></stop></linearGradient></defs><path fill="url(#IconifyId1813088fe1fbc01fb466)" d="M255.153 37.938L134.897 252.976c-2.483 4.44-8.862 4.466-11.382.048L.875 37.958c-2.746-4.814 1.371-10.646 6.827-9.67l120.385 21.517a6.537 6.537 0 0 0 2.322-.004l117.867-21.483c5.438-.991 9.574 4.796 6.877 9.62Z"></path><path fill="url(#IconifyId1813088fe1fbc01fb467)" d="M185.432.063L96.44 17.501a3.268 3.268 0 0 0-2.634 3.014l-5.474 92.456a3.268 3.268 0 0 0 3.997 3.378l24.777-5.718c2.318-.535 4.413 1.507 3.936 3.838l-7.361 36.047c-.495 2.426 1.782 4.5 4.151 3.78l15.304-4.649c2.372-.72 4.652 1.36 4.15 3.788l-11.698 56.621c-.732 3.542 3.979 5.473 5.943 2.437l1.313-2.028l72.516-144.72c1.215-2.423-.88-5.186-3.54-4.672l-25.505 4.922c-2.396.462-4.435-1.77-3.759-4.114l16.646-57.705c.677-2.35-1.37-4.583-3.769-4.113Z"></path></svg>

package/index.js

@@ -17,35 +17,6 @@ import {buildLogEntry, dispatchToTransports} from './utils/transport-dispatcher.
 import {redactValue} from './utils/redaction.js';
 import packageJson from './package.json' with {type: 'json'};
 
-// Check default config for devtools at module load time
-// This allows bundlers to tree-shake if disabled
-const defaultDevtoolsEnabled = defaultConfig.devtools?.enabled ?? false;
-
-// Conditional static import.
-// The NODE_ENV check MUST come first: production bundlers (Next.js/webpack DefinePlugin)
-// replace it with `false`, making the whole condition statically false so the entire
-// block — and the devtools chunk graph — is dead-code-eliminated from client bundles.
-// A bare `if (defaultDevtoolsEnabled)` is NOT eliminable because the value comes from a
-// JSON property access webpack cannot constant-fold.
-// Use lazy initialization to avoid top-level await (which would make module async)
-let devtoolsModule = null;
-let devtoolsModulePromise = null;
-if (process.env.NODE_ENV !== 'production' && defaultDevtoolsEnabled) {
-    metaLog('[JSG-LOGGER] DevTools module pre-loading started (default config enabled)');
-    // Start loading immediately but don't await (non-blocking)
-    // Bundlers can still analyze this static import for tree-shaking
-    devtoolsModulePromise = import('@crimsonsunset/jsg-logger/devtools').then(module => {
-        devtoolsModule = module;
-        metaLog('[JSG-LOGGER] DevTools module pre-loaded successfully');
-        return module;
-    }).catch(error => {
-        metaError('[JSG-LOGGER] DevTools module pre-load failed:', error);
-        return null;
-    });
-} else {
-    metaLog('[JSG-LOGGER] DevTools module NOT pre-loaded (default config disabled - will tree-shake)');
-}
-
 /**
  * Main Logger Class
  * Manages logger instances and provides the public API
@@ -717,29 +688,15 @@ class JSGLogger {
                     }
 
                     try {
-                        // Use pre-loaded module (from default config) or load dynamically (consumer's runtime config override)
+                        const {loadDevtoolsModule} = await import('./utils/devtools-loader.js');
+                        const devtoolsModule = await loadDevtoolsModule();
+
                         if (!devtoolsModule) {
-                            // Check if we have a promise for pre-loading
-                            if (devtoolsModulePromise) {
-                                devtoolsLogger.info('Waiting for pre-loaded DevTools module...');
-                                // Wait for pre-load to complete
-                                devtoolsModule = await devtoolsModulePromise;
-                            } else if (process.env.NODE_ENV !== 'production') {
-                                // Runtime config override: consumer enabled devtools but default was disabled.
-                                // Load on demand via dynamic import. Guarded by NODE_ENV so production
-                                // bundlers (Next.js/webpack) statically eliminate the devtools subtree
-                                // instead of pulling its chunks into the client bundle.
-                                devtoolsLogger.info('Loading DevTools module dynamically (runtime config override)...');
-                                devtoolsModule = await import('@crimsonsunset/jsg-logger/devtools');
-                            } else {
-                                devtoolsLogger.warn('DevTools panel is not available in production builds');
-                                return null;
-                            }
-                        } else {
-                            devtoolsLogger.info('Using pre-loaded DevTools module');
+                            devtoolsLogger.warn('DevTools panel is not available in this build');
+                            return null;
                         }
-                        
-                        if (!devtoolsModule || !devtoolsModule.initializePanel) {
+
+                        if (!devtoolsModule.initializePanel) {
                             throw new Error('DevTools panel module missing initializePanel export');
                         }
                         

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@crimsonsunset/jsg-logger",
-  "version": "1.8.7",
+  "version": "1.8.8",
   "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",
@@ -94,12 +94,14 @@
     "dev": "vite",
     "dev:devtools": "cd devtools && npm run dev",
     "test:devtools": "vite",
-    "build:devtools": "cd devtools && npm run build",
+    "build:devtools": "cd devtools && npm ci && npm run build",
+    "prepublishOnly": "npm run build:devtools",
     "build:devtools:netlify": "npm install && cd devtools && npm install && npm run build:netlify",
     "preview:devtools": "cd devtools && npm run preview",
     "release:patch": "node scripts/release.js patch",
     "release:minor": "node scripts/release.js minor",
     "release:major": "node scripts/release.js major",
+    "publish:ci": "node scripts/publish.js",
     "publish:public": "npm publish --access public",
     "publish:github": "npm publish --registry=https://npm.pkg.github.com/ --access public",
     "check": "npm run test && echo 'Package ready for publishing'"

package/utils/devtools-loader.js

@@ -0,0 +1,43 @@
+/**
+ * Lazy devtools panel loader — isolated from index.js so the devtools Vite
+ * build (which imports index.js) does not try to resolve panel-entry.js
+ * before it exists.
+ */
+
+let devtoolsModule = null;
+let devtoolsModulePromise = null;
+
+/**
+ * Load the devtools panel bundle on demand.
+ * @returns {Promise<Object|null>}
+ */
+export async function loadDevtoolsModule() {
+    if (devtoolsModule) {
+        return devtoolsModule;
+    }
+
+    if (devtoolsModulePromise) {
+        return devtoolsModulePromise;
+    }
+
+    if (process.env.NODE_ENV === 'production') {
+        return null;
+    }
+
+    const entry = ['./devtools', 'dist', 'panel-entry.js'].join('/');
+
+    devtoolsModulePromise = import(
+        /* webpackIgnore: true */
+        /* @vite-ignore */
+        entry
+    ).then((module) => {
+        devtoolsModule = module;
+        return module;
+    }).catch((error) => {
+        console.error('[JSG-LOGGER] DevTools module load failed:', error);
+        devtoolsModulePromise = null;
+        return null;
+    });
+
+    return devtoolsModulePromise;
+}