@crimsonsunset/jsg-logger 1.7.5 → 1.7.7

package/CHANGELOG.md

@@ -5,6 +5,82 @@ 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.7.7] - 2025-11-07 🔧 **DevTools Import Path Fix**
+
+### Fixed
+- **DevTools Panel Import Resolution** - Fixed DevTools panel failing to load in Vite-based projects
+  - Changed from relative path imports (`./devtools/dist/panel-entry.js`) to package export path (`@crimsonsunset/jsg-logger/devtools`)
+  - Resolves Vite module resolution and caching issues with relative paths from `node_modules`
+  - Works consistently across Vite dev server, production builds, and other bundlers
+  - No longer requires excluding logger from Vite's dependency optimization
+  - Fixes 404 errors and "Could not resolve import" errors when calling `enableDevPanel()`
+
+### Changed
+- **DevTools Import Strategy** - Updated to use package export paths for better bundler compatibility
+  - Pre-load import (line 30): Now uses `@crimsonsunset/jsg-logger/devtools` instead of relative path
+  - Dynamic import (line 534): Now uses `@crimsonsunset/jsg-logger/devtools` instead of relative path
+  - Package export path resolves through `package.json` exports field for explicit, unambiguous resolution
+  - Eliminates Vite caching issues that occurred with relative path resolution
+
+### Technical Details
+- **Files Modified**: `index.js` (lines 30, 534)
+- **Root Cause**: Relative paths from within `node_modules` can cause Vite's dependency optimizer to cache incorrect resolutions or fail to resolve paths correctly
+- **Solution**: Package export paths are explicitly defined in `package.json` and resolved consistently by all modern bundlers
+- **Impact**: DevTools panel now loads reliably in all Vite-based projects without special configuration
+
+## [1.7.6] - 2025-11-06 🎛️ **DevTools Singleton Fix & Logging Improvements**
+
+### Added
+- **Static `getControls()` Method** - Added `JSGLogger.getControls()` static method for accessing singleton controls
+  - Returns `controls` object from existing singleton instance without triggering initialization
+  - Provides clean access point for DevTools panel to get logger controls
+  - Returns `null` if singleton not yet initialized
+  - Prevents DevTools from creating new logger instance when accessing controls
+
+### Fixed
+- **DevTools Singleton Re-initialization** - Fixed DevTools panel creating new logger instance on open
+  - DevTools panel now uses `JSGLogger.getControls()` and `JSGLogger.getInstanceSync()` instead of top-level imports
+  - Prevents loss of component loggers when DevTools opens
+  - Component loggers now persist correctly across DevTools open/close cycles
+  - Fixed issue where opening DevTools would reset logger to default config with only "core" component
+- **Component List Duplicates** - Fixed duplicate component entries in DevTools UI
+  - `listComponents()` now returns only kebab-case names from config (e.g., `devtools-ui`)
+  - Removed camelCase aliases from component list display
+  - CamelCase access still available via `logger.components.camelCase()` getters for backward compatibility
+  - UI now shows clean, non-duplicated component list
+- **Circular Reference Errors** - Fixed circular reference errors when logging objects
+  - Added circular reference detection in browser formatter `displayContextData()` function
+  - Removed problematic object logging (theme objects, logger instances, panel instances)
+  - Error logging now extracts only safe properties (`message`, `stack`) to avoid serialization issues
+  - DevTools panel initialization no longer throws circular reference errors
+- **DevTools Logging Format** - Fixed DevTools logs to use proper JSG logger formatting
+  - Replaced all `console.log()` calls in DevTools components with JSG logger calls
+  - DevTools logs now show timestamps, emojis, colors, and structured output
+  - All DevTools component logs (DevToolsPanel, GlobalControls, DisplayControls) use `devtools-ui` component logger
+  - Logs now match standard JSG logger format instead of plain `[JSG-DEVTOOLS]` console messages
+
+### Changed
+- **DevTools Panel Logger Access** - Updated DevTools panel to use singleton access pattern
+  - `panel-entry.jsx` now imports `{ JSGLogger }` class instead of default export
+  - Uses `JSGLogger.getInstanceSync()` for full logger instance access
+  - Uses `JSGLogger.getControls()` for controls API access
+  - Prevents module-level initialization from creating new singleton instance
+- **Component Alias Creation** - Disabled camelCase aliases in `this.loggers` object
+  - Commented out `createAliases()` calls in `init()` and `initSync()` methods
+  - CamelCase aliases no longer added to `this.loggers` (prevents duplicates in `listComponents()`)
+  - CamelCase access still works via `components` getters (e.g., `logger.components.devtoolsUi()`)
+
+### Technical Details
+- **Files Modified**: 
+  - `index.js` - Added `getControls()` static method, disabled `createAliases()` calls, updated `listComponents()` to use `configManager.getAvailableComponents()`
+  - `devtools/src/panel-entry.jsx` - Changed to use singleton access pattern, removed top-level logger import
+  - `devtools/src/components/DevToolsPanel.jsx` - Replaced `console.log` with JSG logger calls
+  - `devtools/src/components/GlobalControls.jsx` - Replaced `console.log` with JSG logger calls
+  - `devtools/src/components/DisplayControls.jsx` - Replaced all `console.log/warn` with JSG logger calls
+  - `devtools/src/App.jsx` - Fixed circular reference errors in error logging
+  - `formatters/browser-formatter.js` - Added circular reference detection in `displayContextData()`
+- **Result**: DevTools panel now correctly uses singleton instance, component loggers persist, no duplicate entries, proper logging format throughout
+
 ## [1.7.5] - 2025-11-06 🎛️ **DevTools Panel UX Improvements**
 
 ### Added

package/devtools/dist/__vite-browser-external-2Ng8QIWW-2Ng8QIWW-2Ng8QIWW.js

@@ -0,0 +1,5 @@
+const __viteBrowserExternal = {};
+export {
+  __viteBrowserExternal as default
+};
+//# sourceMappingURL=__vite-browser-external-2Ng8QIWW-2Ng8QIWW-2Ng8QIWW.js.map

package/devtools/dist/__vite-browser-external-2Ng8QIWW-2Ng8QIWW-2Ng8QIWW.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"__vite-browser-external-2Ng8QIWW-2Ng8QIWW-2Ng8QIWW.js","sources":["__vite-browser-external-2Ng8QIWW-2Ng8QIWW.js"],"sourcesContent":["const __viteBrowserExternal = {};\nexport {\n  __viteBrowserExternal as default\n};\n//# sourceMappingURL=__vite-browser-external-2Ng8QIWW-2Ng8QIWW.js.map\n"],"names":[],"mappings":"AAAK,MAAC,wBAAwB,CAAA;"}

package/devtools/dist/{panel-entry-BZ56bT66.js → panel-entry-BZ56bT66-BMyD75mc.js}

@@ -12068,7 +12068,7 @@ class ConfigManager {
       return module.default || module;
     } catch (error) {
       try {
-        const fs = await import("./__vite-browser-external-2Ng8QIWW-2Ng8QIWW.js");
+        const fs = await import("./__vite-browser-external-2Ng8QIWW-2Ng8QIWW-2Ng8QIWW.js");
         const fileContent = await fs.readFile(path, "utf-8");
         return JSON.parse(fileContent);
       } catch (fsError) {
@@ -12867,7 +12867,7 @@ let devtoolsModule = null;
 let devtoolsModulePromise = null;
 if (defaultDevtoolsEnabled) {
   metaLog("[JSG-LOGGER] DevTools module pre-loading started (default config enabled)");
-  devtoolsModulePromise = import("./panel-entry-DzGsV7s--DzGsV7s-.js").then((module) => {
+  devtoolsModulePromise = import("./panel-entry-DzGsV7s--DzGsV7s--DzGsV7s-.js").then((module) => {
     devtoolsModule = module;
     metaLog("[JSG-LOGGER] DevTools module pre-loaded successfully");
     return module;
@@ -13240,7 +13240,7 @@ const _JSGLogger = class _JSGLogger2 {
                 devtoolsModule = await devtoolsModulePromise;
               } else {
                 devtoolsLogger2.info("Loading DevTools module dynamically (runtime config override)...");
-                devtoolsModule = await import("./panel-entry-DzGsV7s--DzGsV7s-.js");
+                devtoolsModule = await import("./panel-entry-DzGsV7s--DzGsV7s--DzGsV7s-.js");
               }
             } else {
               devtoolsLogger2.info("Using pre-loaded DevTools module");
@@ -13599,4 +13599,4 @@ export {
   initializePanel,
   togglePanel
 };
-//# sourceMappingURL=panel-entry-BZ56bT66.js.map
+//# sourceMappingURL=panel-entry-BZ56bT66-BMyD75mc.js.map