@eaprelsky/nocturna-wheel 3.0.0 → 3.0.1

package/dist/nocturna-wheel.bundle.js

@@ -89,47 +89,16 @@
         }
         
         /**
-         * Gets or creates an IconProvider instance
-         * @param {string} basePath - Optional base path for SVG assets
-         * @returns {Object} The IconProvider instance
+         * Gets the IconProvider instance
+         * NOTE: IconProvider should be registered externally before use
+         * This method only retrieves, does not create
+         * @returns {Object|undefined} The IconProvider instance if registered
          */
-        static getIconProvider(basePath = './assets/svg/zodiac/') {
+        static getIconProvider() {
             if (!this.has('iconProvider')) {
-                // Create a simple icon provider
-                const iconProvider = {
-                    basePath: basePath,
-                    
-                    getPlanetIconPath(planetName) {
-                        return `${this.basePath}zodiac-planet-${planetName.toLowerCase()}.svg`;
-                    },
-                    
-                    getZodiacIconPath(signName) {
-                        return `${this.basePath}zodiac-sign-${signName.toLowerCase()}.svg`;
-                    },
-                    
-                    getAspectIconPath(aspectType) {
-                        return `${this.basePath}zodiac-aspect-${aspectType.toLowerCase()}.svg`;
-                    },
-                    
-                    createTextFallback(svgUtils, options, text) {
-                        const { x, y, size = '16px', color = '#000000', className = 'icon-fallback' } = options;
-                        
-                        const textElement = svgUtils.createSVGElement("text", {
-                            x: x,
-                            y: y,
-                            'text-anchor': 'middle',
-                            'dominant-baseline': 'middle',
-                            'font-size': size,
-                            'class': className,
-                            'fill': color
-                        });
-                        
-                        textElement.textContent = text;
-                        return textElement;
-                    }
-                };
-                
-                this.register('iconProvider', iconProvider);
+                console.warn('ServiceRegistry: IconProvider not registered. Icons may not work correctly.');
+                console.warn('ServiceRegistry: This should be initialized in main.js with inline IconData.');
+                return null;
             }
             return this.get('iconProvider');
         }
@@ -142,8 +111,8 @@
             // Initialize SvgUtils
             this.getSvgUtils();
             
-            // Initialize IconProvider with the assets base path
-            this.getIconProvider(options.assetBasePath || './assets/svg/zodiac/');
+            // Note: IconProvider should be registered externally in main.js
+            // with inline IconData before calling this method
             
             console.log("ServiceRegistry: Core services initialized");
         }
@@ -159,17 +128,6 @@
      * 2. External mode: Uses external file paths (for custom icons)
      */
 
-    // Try to import IconData if available (after build)
-    let InlineIconData = null;
-    try {
-        // This will be replaced by rollup during build
-        // During development or if IconData doesn't exist, this will fail gracefully
-        InlineIconData = null; // Will be set up after IconData.js is generated
-    } catch (e) {
-        // IconData not available yet
-        InlineIconData = null;
-    }
-
     class IconProvider {
         /**
          * Constructor
@@ -190,6 +148,9 @@
             
             // Lazy-load IconData on first use
             this.inlineData = null;
+            
+            // Track if warning was already shown (to avoid spam)
+            this._warnedAboutMissingData = false;
         }
         
         /**
@@ -197,9 +158,6 @@
          * @private
          */
         _getInlineData() {
-            if (!this.inlineData && InlineIconData) {
-                this.inlineData = InlineIconData;
-            }
             return this.inlineData;
         }
         
@@ -209,7 +167,11 @@
          */
         setInlineData(iconData) {
             this.inlineData = iconData;
-            InlineIconData = iconData;
+            console.log('IconProvider.setInlineData(): Data loaded successfully',
+                'planets:', Object.keys(iconData?.planets || {}).length,
+                'signs:', Object.keys(iconData?.signs || {}).length,
+                'aspects:', Object.keys(iconData?.aspects || {}).length
+            );
         }
 
         /**
@@ -230,6 +192,16 @@
                 const inlineData = this._getInlineData();
                 if (inlineData?.planets?.[name]) {
                     return inlineData.planets[name];
+                } else {
+                    // Debug: log when inline data is not available (only once)
+                    if (!this._warnedAboutMissingData) {
+                        this._warnedAboutMissingData = true;
+                        if (!inlineData) {
+                            console.warn(`IconProvider: Inline data not loaded, falling back to external paths. Call setInlineData() to load inline icons.`);
+                        } else if (!inlineData.planets) {
+                            console.warn(`IconProvider: Inline data missing 'planets' category`);
+                        }
+                    }
                 }
             }
             
@@ -321,7 +293,7 @@
     /**
      * IconData.js
      * Auto-generated module containing inline SVG icons as data URLs
-     * Generated at: 2025-11-13T13:46:12.823Z
+     * Generated at: 2025-11-14T10:26:57.781Z
      * 
      * This file is automatically generated by the build process.
      * Do not edit manually - changes will be overwritten.
@@ -1968,11 +1940,18 @@
          */
         constructor(options) {
             super(options);
-            if (!options.assetBasePath) {
-                throw new Error("ZodiacRenderer: Missing required option assetBasePath");
+            
+            // Store the icon provider
+            this.iconProvider = options.iconProvider;
+            
+            // assetBasePath is required only if IconProvider is not available
+            if (!options.assetBasePath && !this.iconProvider) {
+                throw new Error("ZodiacRenderer: Missing required option assetBasePath or iconProvider");
             }
-
-            this.iconProvider = options.iconProvider; // Store the icon provider
+            
+            // Set default assetBasePath if not provided (for backward compatibility)
+            this.assetBasePath = options.assetBasePath || './assets/';
+            
             this.signIconRadius = (this.outerRadius + this.middleRadius) / 2;
             this.signIconSize = 30;
         }
@@ -2083,10 +2062,17 @@
                     iconHref = this.iconProvider.getZodiacIconPath(signName);
                 } else {
                     // Fallback to old path construction
-                    iconHref = `${this.options.assetBasePath}svg/zodiac/zodiac-sign-${signName}.svg`;
+                    const basePath = this.options.assetBasePath || this.assetBasePath || './assets/';
+                    iconHref = `${basePath}svg/zodiac/zodiac-sign-${signName}.svg`;
                 }
                 
-                console.log(`Loading zodiac sign: ${iconHref}`);
+                // Log icon path for debugging (only log first time for each sign to avoid spam)
+                if (!this._loggedSigns) this._loggedSigns = {};
+                if (!this._loggedSigns[signName]) {
+                    console.log(`ZodiacRenderer: Loading sign '${signName}':`, 
+                        iconHref.startsWith('data:') ? 'inline data URL' : iconHref);
+                    this._loggedSigns[signName] = true;
+                }
                 
                 const icon = this.svgUtils.createSVGElement("image", {
                     x: point.x - this.signIconSize / 2, // Offset to center the icon
@@ -2405,12 +2391,17 @@
          */
         constructor(options) {
             super(options);
-            if (!options.assetBasePath) {
-                throw new Error(`${this.constructor.name}: Missing required option assetBasePath`);
-            }
             
             // Store the icon provider
             this.iconProvider = options.iconProvider;
+            
+            // assetBasePath is required only if IconProvider is not available
+            if (!options.assetBasePath && !this.iconProvider) {
+                throw new Error(`${this.constructor.name}: Missing required option assetBasePath or iconProvider`);
+            }
+            
+            // Set default assetBasePath if not provided (for backward compatibility)
+            this.assetBasePath = options.assetBasePath || './assets/';
         }
 
         /**
@@ -2512,7 +2503,8 @@
                 iconPath = this.iconProvider.getPlanetIconPath(planet.name);
             } else {
                 // Fallback to old path construction if IconProvider is not available
-                iconPath = `${this.options.assetBasePath}svg/zodiac/zodiac-planet-${planet.name.toLowerCase()}.svg`;
+                const basePath = this.options.assetBasePath || this.assetBasePath || './assets/';
+                iconPath = `${basePath}svg/zodiac/zodiac-planet-${planet.name.toLowerCase()}.svg`;
             }
             
             // Calculate top-left position of the icon (centered on the calculated point)
@@ -3514,7 +3506,7 @@
             this.renderedAspects = []; // Store calculated aspects
             this._aspectCacheKey = null; // Cache key for aspect calculations
             this._aspectCache = [];      // Cached aspect results
-            this.assetBasePath = options.assetBasePath || ''; // Store asset base path
+            this.assetBasePath = options.assetBasePath || './assets/'; // Store asset base path with fallback
             this.iconProvider = options.iconProvider; // Store the icon provider
             
             // Define major aspects and their angles (can be overridden/extended by config)
@@ -4043,7 +4035,12 @@
             this.config = config;
             this.svgNS = svgNS;
             this.svgUtils = ServiceRegistry.getSvgUtils();
-            this.iconProvider = ServiceRegistry.getIconProvider(config.assets?.basePath);
+            this.iconProvider = ServiceRegistry.getIconProvider();
+            
+            // If IconProvider is null, log error for debugging
+            if (!this.iconProvider) {
+                console.error('RendererFactory: IconProvider not available. Icons will not render correctly.');
+            }
         }
         
         /**
@@ -5627,17 +5624,31 @@
      * See examples/factory-injection-example.js for more advanced usage patterns.
      */
 
-    // Initialize core services immediately
-    ServiceRegistry.initializeServices();
+    // CRITICAL: Register IconProvider BEFORE initializing other services
+    // This ensures renderers get the inline-capable provider, not a legacy stub
+    const iconProvider = new IconProvider({
+        useInline: true,
+        basePath: './assets/svg/zodiac/'  // Fallback for external mode
+    });
 
-    // Initialize IconProvider with inline data if available
+    // Initialize with inline data if available
     if (IconData && Object.keys(IconData.planets || {}).length > 0) {
-        const iconProvider = ServiceRegistry.getIconProvider();
-        if (iconProvider && typeof iconProvider.setInlineData === 'function') {
-            iconProvider.setInlineData(IconData);
-        }
+        iconProvider.setInlineData(IconData);
+        console.log('IconProvider: Initialized with inline icons -', 
+            'Planets:', Object.keys(IconData.planets).length,
+            'Signs:', Object.keys(IconData.signs).length,
+            'Aspects:', Object.keys(IconData.aspects).length
+        );
+    } else {
+        console.warn('IconProvider: IconData not loaded, will use external paths');
     }
 
+    // Register IconProvider BEFORE initializing services
+    ServiceRegistry.register('iconProvider', iconProvider);
+
+    // Now initialize other core services
+    ServiceRegistry.initializeServices();
+
     // Library version
     const VERSION = '0.2.0';