@yogiswara/honcho-editor-ui 2.6.6 → 2.6.7

package/dist/hooks/usePreset.js

@@ -79,9 +79,11 @@ export function usePreset(controller, firebaseUid, options = {}) {
         try {
             debugLog('Loading presets from backend...');
             const loadedPresets = await stableControllerRef.current.getPresets(stableFirebaseUidRef.current);
-            setPresets(loadedPresets);
+            // Ensure we always have an array
+            const presetsArray = Array.isArray(loadedPresets) ? loadedPresets : [];
+            setPresets(presetsArray);
             setIsInitialized(true);
-            debugLog('Presets loaded successfully', { count: loadedPresets.length });
+            debugLog('Presets loaded successfully', { count: presetsArray.length });
         }
         catch (err) {
             handleError('load presets', err);
@@ -101,11 +103,13 @@ export function usePreset(controller, firebaseUid, options = {}) {
         // Don't set loading state for background operations
         stableControllerRef.current.getPresets(stableFirebaseUidRef.current)
             .then((loadedPresets) => {
-            setPresets(loadedPresets);
+            // Ensure we always have an array
+            const presetsArray = Array.isArray(loadedPresets) ? loadedPresets : [];
+            setPresets(presetsArray);
             if (!isInitialized) {
                 setIsInitialized(true);
             }
-            debugLog('Background presets loaded successfully', { count: loadedPresets.length });
+            debugLog('Background presets loaded successfully', { count: presetsArray.length });
         })
             .catch((err) => {
             debugLog('Background load failed:', err);
@@ -123,11 +127,6 @@ export function usePreset(controller, firebaseUid, options = {}) {
             setError('Preset name cannot be empty');
             return null;
         }
-        // Check for duplicate names
-        if (presets.some(p => p.name.toLowerCase() === name.toLowerCase())) {
-            setError('A preset with this name already exists');
-            return null;
-        }
         setIsLoading(true);
         setError(null);
         try {
@@ -182,13 +181,13 @@ export function usePreset(controller, firebaseUid, options = {}) {
             setError('Preset name cannot be empty');
             return false;
         }
-        const existingPreset = presets.find(p => p.id === presetId);
+        const existingPreset = Array.isArray(presets) ? presets.find(p => p.id === presetId) : null;
         if (!existingPreset) {
             setError('Preset not found');
             return false;
         }
         // Check for duplicate names (excluding the current preset)
-        if (presets.some(p => p.id !== presetId && p.name.toLowerCase() === newName.toLowerCase())) {
+        if (Array.isArray(presets) && presets.some(p => p.id !== presetId && p.name.toLowerCase() === newName.toLowerCase())) {
             setError('A preset with this name already exists');
             return false;
         }
@@ -218,7 +217,7 @@ export function usePreset(controller, firebaseUid, options = {}) {
             debugLog('Delete skipped: missing controller or firebaseUid');
             return false;
         }
-        const existingPreset = presets.find(p => p.id === presetId);
+        const existingPreset = Array.isArray(presets) ? presets.find(p => p.id === presetId) : null;
         if (!existingPreset) {
             setError('Preset not found');
             return false;
@@ -275,10 +274,10 @@ export function usePreset(controller, firebaseUid, options = {}) {
         };
         debugLog('Finding preset by adjustments...', adjustments);
         // Find preset that matches the current adjustments
-        const matchingPreset = presets.find(preset => {
+        const matchingPreset = Array.isArray(presets) ? presets.find(preset => {
             const presetAdjustments = presetToAdjustmentState(preset);
             return adjustmentsMatch(presetAdjustments, adjustments);
-        });
+        }) : null;
         if (matchingPreset) {
             debugLog('Found matching preset:', matchingPreset);
             return matchingPreset;
@@ -288,13 +287,23 @@ export function usePreset(controller, firebaseUid, options = {}) {
             return null;
         }
     }, [presets, debugLog]);
-    // Auto-load presets on initialization - stable dependencies
+    // Auto-load presets on initialization or when firebaseUid becomes available
     useEffect(() => {
-        if (memoizedOptions.autoLoad && stableControllerRef.current && stableFirebaseUidRef.current && !isInitialized) {
-            debugLog('Auto-loading presets...');
+        const hasValidCredentials = stableControllerRef.current && stableFirebaseUidRef.current;
+        const shouldAutoLoad = memoizedOptions.autoLoad && hasValidCredentials;
+        if (shouldAutoLoad && !isInitialized) {
+            debugLog('Auto-loading presets on initialization...');
             load();
         }
-    }, [memoizedOptions.autoLoad, isInitialized, load]);
+        else if (shouldAutoLoad && isInitialized) {
+            // If we're already initialized but firebaseUid changed to a new valid value, reload
+            const firebaseUidChanged = prevStableFirebaseUidRef.current !== stableFirebaseUidRef.current;
+            if (firebaseUidChanged && stableFirebaseUidRef.current) {
+                debugLog('Auto-loading presets due to firebaseUid change...');
+                load();
+            }
+        }
+    }, [memoizedOptions.autoLoad, stableControllerRef.current, stableFirebaseUidRef.current, isInitialized, load]);
     // Clear state when controller or firebaseUid changes - but only when they actually change
     useEffect(() => {
         const controllerChanged = prevStableControllerRef.current !== stableControllerRef.current;
@@ -302,19 +311,22 @@ export function usePreset(controller, firebaseUid, options = {}) {
         if ((controllerChanged || firebaseUidChanged) && isInitialized) {
             debugLog('Controller or firebaseUid changed, clearing state');
             setError(null);
-            setIsInitialized(false);
-            // Don't clear presets to avoid flicker - they'll be reloaded
+            // Only reset initialization if firebaseUid became null/empty
+            if (firebaseUidChanged && !stableFirebaseUidRef.current) {
+                setIsInitialized(false);
+                setPresets([]); // Clear presets when user changes
+            }
         }
         prevStableControllerRef.current = stableControllerRef.current;
         prevStableFirebaseUidRef.current = stableFirebaseUidRef.current;
-    }, [stableControllerRef.current, stableFirebaseUidRef.current, isInitialized]);
+    }, [stableControllerRef.current, stableFirebaseUidRef.current, isInitialized, debugLog]);
     // Preset info object - memoized to prevent re-renders
     const info = useMemo(() => ({
         isLoading,
         error,
-        count: presets.length,
+        count: Array.isArray(presets) ? presets.length : 0,
         isInitialized
-    }), [isLoading, error, presets.length, isInitialized]);
+    }), [isLoading, error, presets, isInitialized]);
     // Actions object - memoized to prevent re-renders when functions don't change
     const actions = useMemo(() => ({
         create,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yogiswara/honcho-editor-ui",
-  "version": "2.6.6",
+  "version": "2.6.7",
   "description": "A complete UI component library for the Honcho photo editor.",
   "main": "./dist/index.js",
   "module": "./dist/index.mjs",