@umituz/react-native-onboarding 3.0.2 → 3.0.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@umituz/react-native-onboarding",
-  "version": "3.0.2",
+  "version": "3.0.4",
   "description": "Advanced onboarding flow for React Native apps with personalization questions, theme-aware colors, animations, and customizable slides. SOLID, DRY, KISS principles applied.",
   "main": "./src/index.ts",
   "types": "./src/index.ts",

package/src/infrastructure/storage/OnboardingStore.ts

@@ -18,17 +18,36 @@ interface OnboardingStore extends OnboardingStoreState {
   setError: (error: string | null) => void;
   setState: (state: Partial<OnboardingStoreState>) => void;
   getState: () => OnboardingStoreState;
+  // Async actions for initialization (match OnboardingStoreActions signatures)
+  initialize: (storageKey?: string) => Promise<void>;
+  complete: (storageKey?: string) => Promise<void>;
+  skip: (storageKey?: string) => Promise<void>;
+  reset: (storageKey?: string) => Promise<void>;
+  saveAnswer: (questionId: string, answer: any) => Promise<void>;
+  setUserData: (data: any) => Promise<void>;
 }
 
-export const useOnboardingStore = create<OnboardingStore>((set, get) => ({
-  ...initialOnboardingState,
+export const useOnboardingStore = create<OnboardingStore>((set, get) => {
+  const actions = createOnboardingStoreActions(set as any, get);
 
-  setCurrentStep: (step) => set({ currentStep: step }),
-  setLoading: (loading) => set({ loading }),
-  setError: (error) => set({ error }),
-  setState: set,
-  getState: get,
-}));
+  return {
+    ...initialOnboardingState,
+
+    setCurrentStep: (step) => set({ currentStep: step }),
+    setLoading: (loading) => set({ loading }),
+    setError: (error) => set({ error }),
+    setState: set,
+    getState: get,
+
+    // Async actions from actions module
+    initialize: actions.initialize,
+    complete: actions.complete,
+    skip: actions.skip,
+    reset: actions.reset,
+    saveAnswer: actions.saveAnswer,
+    setUserData: actions.setUserData,
+  };
+});
 
 /**
  * Hook for accessing onboarding state
@@ -47,7 +66,7 @@ export const useOnboarding = () => {
     loading: store.loading,
     error: store.error,
     userData: store.userData,
-    
+
     // Actions
     initialize: actions.initialize,
     complete: actions.complete,
@@ -58,7 +77,7 @@ export const useOnboarding = () => {
     setError: store.setError,
     saveAnswer: actions.saveAnswer,
     setUserData: actions.setUserData,
-    
+
     // Selectors
     getAnswer: selectors.getAnswer,
     getUserData: selectors.getUserData,

package/src/infrastructure/storage/OnboardingStoreActions.ts

@@ -63,22 +63,32 @@ export function createOnboardingStoreActions(
       try {
         set({ loading: true, error: null });
 
+        // Ensure storage write completes before proceeding
         const result = await storageRepository.setString(storageKey, "true");
 
+        if (!result.success) {
+          throw new Error("Failed to save completion status to storage");
+        }
+
         const userData = { ...get().userData };
         userData.completedAt = new Date().toISOString();
 
-        await storageRepository.setItem(USER_DATA_STORAGE_KEY, userData);
+        const userDataResult = await storageRepository.setItem(USER_DATA_STORAGE_KEY, userData);
 
+        if (!userDataResult.success) {
+          throw new Error("Failed to save user data to storage");
+        }
+
+        // Only update state after storage write is confirmed
         set({
-          isOnboardingComplete: result.success,
+          isOnboardingComplete: true,
           userData,
           loading: false,
-          error: result.success ? null : "Failed to complete onboarding",
+          error: null,
         });
 
         if (__DEV__) {
-          console.log('[OnboardingStore] Onboarding completed successfully');
+          console.log('[OnboardingStore] Onboarding completed and persisted successfully');
         }
       } catch (error) {
         set({
@@ -89,6 +99,8 @@ export function createOnboardingStoreActions(
         if (__DEV__) {
           console.error('[OnboardingStore] Completion error:', error);
         }
+
+        throw error;
       }
     },
 
@@ -96,22 +108,33 @@ export function createOnboardingStoreActions(
       try {
         set({ loading: true, error: null });
 
+        // Ensure storage write completes before proceeding
         const result = await storageRepository.setString(storageKey, "true");
 
+        if (!result.success) {
+          throw new Error("Failed to save skip status to storage");
+        }
+
         const userData = { ...get().userData };
         userData.skipped = true;
         userData.completedAt = new Date().toISOString();
-        await storageRepository.setItem(USER_DATA_STORAGE_KEY, userData);
 
+        const userDataResult = await storageRepository.setItem(USER_DATA_STORAGE_KEY, userData);
+
+        if (!userDataResult.success) {
+          throw new Error("Failed to save user data to storage");
+        }
+
+        // Only update state after storage write is confirmed
         set({
-          isOnboardingComplete: result.success,
+          isOnboardingComplete: true,
           userData,
           loading: false,
-          error: result.success ? null : "Failed to skip onboarding",
+          error: null,
         });
 
         if (__DEV__) {
-          console.log('[OnboardingStore] Onboarding skipped successfully');
+          console.log('[OnboardingStore] Onboarding skipped and persisted successfully');
         }
       } catch (error) {
         set({
@@ -122,6 +145,8 @@ export function createOnboardingStoreActions(
         if (__DEV__) {
           console.error('[OnboardingStore] Skip error:', error);
         }
+
+        throw error;
       }
     },