@umituz/react-native-auth 4.3.81 → 4.3.82

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@umituz/react-native-auth",
-  "version": "4.3.81",
+  "version": "4.3.82",
   "description": "Authentication service for React Native apps - Secure, type-safe, and production-ready. Provider-agnostic design with dependency injection, configurable validation, and comprehensive error handling.",
   "main": "./src/index.ts",
   "types": "./src/index.ts",

package/src/infrastructure/services/AnonymousModeService.ts

@@ -54,16 +54,26 @@ export class AnonymousModeService {
   }
 
   async enable(storageProvider: IStorageProvider): Promise<void> {
-    // Save to storage first, then update memory to maintain consistency.
-    // This prevents TOCTOU: memory is never set to true unless storage confirms the write.
-    const saveSuccess = await this.save(storageProvider, true);
+    const previousState = this.isAnonymousMode;
 
-    if (!saveSuccess) {
-      throw new Error('Failed to save anonymous mode state');
-    }
+    try {
+      // Save to storage first, then update memory to maintain consistency.
+      // This prevents TOCTOU: memory is never set to true unless storage confirms the write.
+      const saveSuccess = await this.save(storageProvider, true);
+
+      if (!saveSuccess) {
+        // Rollback to previous state on failure
+        this.isAnonymousMode = previousState;
+        throw new Error('Failed to save anonymous mode state');
+      }
 
-    this.isAnonymousMode = true;
-    emitAnonymousModeEnabled();
+      this.isAnonymousMode = true;
+      emitAnonymousModeEnabled();
+    } catch (error) {
+      // Ensure state is rolled back on any error
+      this.isAnonymousMode = previousState;
+      throw error;
+    }
   }
 
   getIsAnonymousMode(): boolean {

package/src/infrastructure/services/AuthService.ts

@@ -38,8 +38,8 @@ export class AuthService {
 
     if (this.initialized) return;
 
-    // Create and store initialization promise to prevent concurrent initialization
-    this.initializationPromise = (async () => {
+    // Store reference to current promise to prevent race condition
+    const initPromise = (async () => {
       this.repository = new AuthRepository(this.config);
 
       if (this.storageProvider) {
@@ -48,10 +48,15 @@ export class AuthService {
       this.initialized = true;
     })();
 
+    this.initializationPromise = initPromise;
+
     try {
-      await this.initializationPromise;
+      await initPromise;
     } finally {
-      this.initializationPromise = null;
+      // Only null out if it's still the same promise (prevents race condition)
+      if (this.initializationPromise === initPromise) {
+        this.initializationPromise = null;
+      }
     }
   }
 

package/src/infrastructure/utils/listener/listenerState.util.ts

@@ -123,11 +123,12 @@ export function resetListenerState(): void {
         console.error('[ListenerState] Error during unsubscribe:', error);
       }
     }
+    // Always set to null even if unsubscribe fails to prevent retry loops
+    state.unsubscribe = null;
   }
   state.initialized = false;
   state.refCount = 0;
   state.initializationInProgress = false;
   state.anonymousSignInInProgress = false;
   state.cleanupInProgress = false;
-  state.unsubscribe = null;
 }

package/src/presentation/hooks/useAuthHandlers.ts

@@ -59,7 +59,8 @@ export const useAuthHandlers = (appInfo: AuthHandlersAppInfo, translations?: Aut
       if (__DEV__) {
         console.error("[useAuthHandlers] Failed to open app store:", error);
       }
-      Alert.alert(translations?.common || "", translations?.failedToOpenAppStore || "");
+      const errorMessage = error instanceof Error ? error.message : String(error);
+      Alert.alert(translations?.common || "", translations?.failedToOpenAppStore || errorMessage);
     }
   }, [appInfo.appStoreUrl, translations]);
 
@@ -70,9 +71,10 @@ export const useAuthHandlers = (appInfo: AuthHandlersAppInfo, translations?: Aut
       if (__DEV__) {
         console.error("[useAuthHandlers] Sign out failed:", error);
       }
+      const errorMessage = error instanceof Error ? error.message : "Unknown error";
       AlertService.createErrorAlert(
         translations?.common || "",
-        translations?.unknown || ""
+        translations?.unknown || errorMessage
       );
     }
   }, [signOut, translations]);

package/src/presentation/navigation/AuthNavigator.tsx

@@ -38,18 +38,39 @@ export const AuthNavigator: React.FC<AuthNavigatorProps> = ({
   >(undefined);
 
   useEffect(() => {
+    let isMounted = true;
+    const abortController = new AbortController();
+
     const checkInitialRoute = async () => {
-      const result = await storageRepository.getString(SHOW_REGISTER_KEY, "false");
-      const value = unwrap(result, "false");
-      if (value === "true") {
-        setInitialRouteName("Register");
-        void storageRepository.removeItem(SHOW_REGISTER_KEY);
-      } else {
-        setInitialRouteName("Login");
+      try {
+        const result = await storageRepository.getString(SHOW_REGISTER_KEY, "false");
+        // Check if component is still mounted before updating state
+        if (!isMounted || abortController.signal.aborted) return;
+
+        const value = unwrap(result, "false");
+        if (value === "true") {
+          setInitialRouteName("Register");
+          void storageRepository.removeItem(SHOW_REGISTER_KEY);
+        } else {
+          setInitialRouteName("Login");
+        }
+      } catch (error) {
+        // Silently fail - will default to Login screen
+        if (__DEV__) {
+          console.error('[AuthNavigator] Failed to check initial route:', error);
+        }
+        if (isMounted && !abortController.signal.aborted) {
+          setInitialRouteName("Login");
+        }
       }
     };
 
     void checkInitialRoute();
+
+    return () => {
+      isMounted = false;
+      abortController.abort();
+    };
   }, []);
 
   // Memoize nested translation objects to prevent screen wrapper recreation