@umituz/react-native-auth 2.4.0 → 2.5.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@umituz/react-native-auth",
-  "version": "2.4.0",
+  "version": "2.5.1",
   "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/adapters/StorageProviderAdapter.ts

@@ -15,11 +15,9 @@ export class StorageProviderAdapter implements IStorageProvider {
   async get(key: string): Promise<string | null> {
     try {
       if (this.storage.getString) {
-        // @umituz/react-native-storage format
         const result = await this.storage.getString(key, null);
         return result?.value ?? null;
       } else if (this.storage.getItem) {
-        // AsyncStorage format
         return await this.storage.getItem(key);
       } else {
         throw new Error("Unsupported storage implementation");
@@ -30,44 +28,24 @@ export class StorageProviderAdapter implements IStorageProvider {
   }
 
   async set(key: string, value: string): Promise<void> {
-    try {
-      if (this.storage.setString) {
-        // @umituz/react-native-storage format
-        await this.storage.setString(key, value);
-      } else if (this.storage.setItem) {
-        // AsyncStorage format
-        await this.storage.setItem(key, value);
-      } else {
-        throw new Error("Unsupported storage implementation");
-      }
-    } catch (error) {
-      if (__DEV__) {
-        console.warn("[StorageProviderAdapter] Failed to set value:", error);
-      }
-      throw error;
+    if (this.storage.setString) {
+      await this.storage.setString(key, value);
+    } else if (this.storage.setItem) {
+      await this.storage.setItem(key, value);
+    } else {
+      throw new Error("Unsupported storage implementation");
     }
   }
 
   async remove(key: string): Promise<void> {
-    try {
-      if (this.storage.removeItem) {
-        // Both AsyncStorage and @umituz/react-native-storage support removeItem
-        await this.storage.removeItem(key);
-      } else {
-        throw new Error("Unsupported storage implementation");
-      }
-    } catch (error) {
-      if (__DEV__) {
-        console.warn("[StorageProviderAdapter] Failed to remove value:", error);
-      }
-      throw error;
+    if (this.storage.removeItem) {
+      await this.storage.removeItem(key);
+    } else {
+      throw new Error("Unsupported storage implementation");
     }
   }
 }
 
-/**
- * Create storage provider from various storage implementations
- */
 export function createStorageProvider(storage: any): IStorageProvider {
   return new StorageProviderAdapter(storage);
-}
+}

package/src/infrastructure/services/AuthEventService.ts

@@ -85,13 +85,11 @@ export class AuthEventService {
   private notifyListeners(event: string, payload: AuthEventPayload): void {
     const eventListeners = this.listeners.get(event);
     if (eventListeners) {
-      eventListeners.forEach(listener => {
+      eventListeners.forEach((listener) => {
         try {
           listener(payload);
-        } catch (error) {
-          if (__DEV__) {
-            console.error(`Error in auth event listener for ${event}:`, error);
-          }
+        } catch {
+          // Silent fail for listener errors
         }
       });
     }

package/src/infrastructure/storage/GuestModeStorage.ts

@@ -3,24 +3,15 @@
  * Single Responsibility: Manage guest mode persistence
  */
 
-import type { IStorageProvider } from "../services/GuestModeService";
 import { getAuthPackage } from "../services/AuthPackage";
 
-/**
- * Load guest mode from storage
- */
 export async function loadGuestMode(storageKey?: string): Promise<boolean> {
   try {
     const packageConfig = getAuthPackage()?.getConfig();
     const key = storageKey ?? packageConfig?.storageKeys.guestMode ?? "@auth_guest_mode";
-    
+
     const storageProvider = getAuthPackage()?.getStorageProvider();
-    if (!storageProvider) {
-      if (__DEV__) {
-        console.warn("[GuestModeStorage] No storage provider available");
-      }
-      return false;
-    }
+    if (!storageProvider) return false;
 
     const value = await storageProvider.get(key);
     return value === "true";
@@ -29,53 +20,34 @@ export async function loadGuestMode(storageKey?: string): Promise<boolean> {
   }
 }
 
-/**
- * Save guest mode to storage
- */
 export async function saveGuestMode(isGuest: boolean, storageKey?: string): Promise<void> {
   try {
     const packageConfig = getAuthPackage()?.getConfig();
     const key = storageKey ?? packageConfig?.storageKeys.guestMode ?? "@auth_guest_mode";
-    
+
     const storageProvider = getAuthPackage()?.getStorageProvider();
-    if (!storageProvider) {
-      if (__DEV__) {
-        console.warn("[GuestModeStorage] No storage provider available");
-      }
-      return;
-    }
+    if (!storageProvider) return;
 
     if (isGuest) {
       await storageProvider.set(key, "true");
-      if (__DEV__) {
-        console.log("[GuestModeStorage] Guest mode persisted to storage");
-      }
     } else {
       await storageProvider.remove(key);
     }
-  } catch (error) {
-    if (__DEV__) {
-      console.warn("[GuestModeStorage] Failed to persist guest mode:", error);
-    }
+  } catch {
+    // Ignore storage errors
   }
 }
 
-/**
- * Clear guest mode from storage
- */
 export async function clearGuestMode(storageKey?: string): Promise<void> {
   try {
     const packageConfig = getAuthPackage()?.getConfig();
     const key = storageKey ?? packageConfig?.storageKeys.guestMode ?? "@auth_guest_mode";
-    
+
     const storageProvider = getAuthPackage()?.getStorageProvider();
-    if (!storageProvider) {
-      return;
-    }
+    if (!storageProvider) return;
 
     await storageProvider.remove(key);
   } catch {
     // Ignore storage errors
   }
 }
-

package/src/infrastructure/utils/AuthEventEmitter.ts

@@ -5,25 +5,10 @@
 
 import { DeviceEventEmitter } from "react-native";
 
-/**
- * Emit user authenticated event
- */
 export function emitUserAuthenticated(userId: string): void {
-  /* eslint-disable-next-line no-console */
-  if (__DEV__) {
-    console.log("[AuthEventEmitter] Emitting user-authenticated event");
-  }
   DeviceEventEmitter.emit("user-authenticated", { userId });
 }
 
-/**
- * Emit guest mode enabled event
- */
 export function emitGuestModeEnabled(): void {
-  /* eslint-disable-next-line no-console */
-  if (__DEV__) {
-    console.log("[AuthEventEmitter] Emitting guest-mode-enabled event");
-  }
   DeviceEventEmitter.emit("guest-mode-enabled");
 }
-

package/src/presentation/components/AuthBottomSheet.tsx

@@ -47,7 +47,6 @@ export const AuthBottomSheet: React.FC<AuthBottomSheetProps> = ({
   onTermsPress,
   onPrivacyPress,
 }) => {
-  if (__DEV__) console.log("[AuthBottomSheet] Component rendering");
   const tokens = useAppDesignTokens();
   const { t } = useLocalization();
   const modalRef = useRef<BottomSheetModal>(null);
@@ -55,7 +54,6 @@ export const AuthBottomSheet: React.FC<AuthBottomSheetProps> = ({
   const { isVisible, mode, hideAuthModal, setMode, executePendingCallback, clearPendingCallback } =
     useAuthModalStore();
   const { isAuthenticated, isGuest } = useAuth();
-  if (__DEV__) console.log("[AuthBottomSheet] isVisible:", isVisible, "isAuthenticated:", isAuthenticated);
 
   // Present/dismiss modal based on visibility state
   useEffect(() => {

package/src/presentation/hooks/useAuth.ts

@@ -47,10 +47,8 @@ export interface UseAuthResult {
  * ```
  */
 export function useAuth(): UseAuthResult {
-  if (__DEV__) console.log("[useAuth] Hook called");
   const state = useAuthState();
   const actions = useAuthActions(state);
-  if (__DEV__) console.log("[useAuth] isAuthenticated:", state.isAuthenticated, "isGuest:", state.isGuest);
 
   return {
     user: state.user,

package/src/presentation/hooks/useAuthActions.ts

@@ -14,85 +14,65 @@ export interface UseAuthActionsResult {
   continueAsGuest: () => Promise<void>;
 }
 
-/**
- * Hook for authentication actions
- */
-export function useAuthActions(
-  state: UseAuthStateResult
-): UseAuthActionsResult {
+export function useAuthActions(state: UseAuthStateResult): UseAuthActionsResult {
   const { isGuest, setIsGuest, setLoading, setError } = state;
 
-  const signUp = useCallback(async (
-    email: string,
-    password: string,
-    displayName?: string
-  ) => {
-    const service = getAuthService();
-    if (!service) {
-      const err = "Auth service is not initialized";
-      setError(err);
-      throw new Error(err);
-    }
-    try {
-      setLoading(true);
-      setError(null);
-      await service.signUp({ email, password, displayName });
-      if (isGuest) {
-        setIsGuest(false);
+  const signUp = useCallback(
+    async (email: string, password: string, displayName?: string) => {
+      const service = getAuthService();
+      if (!service) {
+        const err = "Auth service is not initialized";
+        setError(err);
+        throw new Error(err);
       }
-    } catch (err: unknown) {
-      const errorMessage = err instanceof Error ? err.message : "Sign up failed";
-      setError(errorMessage);
-      throw err;
-    } finally {
-      setLoading(false);
-    }
-  }, [isGuest, setIsGuest, setLoading, setError]);
-
-  const signIn = useCallback(async (email: string, password: string) => {
-    /* eslint-disable-next-line no-console */
-    if (__DEV__) {
-      console.log("[useAuthActions] signIn called with email:", email);
-    }
-    const service = getAuthService();
-    if (!service) {
-      const err = "Auth service is not initialized";
-      setError(err);
-      throw new Error(err);
-    }
-    try {
-      setLoading(true);
-      setError(null);
-      /* eslint-disable-next-line no-console */
-      if (__DEV__) {
-        console.log("[useAuthActions] Calling service.signIn()");
+      try {
+        setLoading(true);
+        setError(null);
+        await service.signUp({ email, password, displayName });
+        if (isGuest) {
+          setIsGuest(false);
+        }
+      } catch (err: unknown) {
+        const errorMessage = err instanceof Error ? err.message : "Sign up failed";
+        setError(errorMessage);
+        throw err;
+      } finally {
+        setLoading(false);
       }
-      await service.signIn({ email, password });
-      /* eslint-disable-next-line no-console */
-      if (__DEV__) {
-        console.log("[useAuthActions] Service.signIn() completed successfully");
+    },
+    [isGuest, setIsGuest, setLoading, setError],
+  );
+
+  const signIn = useCallback(
+    async (email: string, password: string) => {
+      const service = getAuthService();
+      if (!service) {
+        const err = "Auth service is not initialized";
+        setError(err);
+        throw new Error(err);
       }
-      if (isGuest) {
-        /* eslint-disable-next-line no-console */
-        if (__DEV__) {
-          console.log("[useAuthActions] Clearing guest mode after sign in");
+      try {
+        setLoading(true);
+        setError(null);
+        await service.signIn({ email, password });
+        if (isGuest) {
+          setIsGuest(false);
         }
-        setIsGuest(false);
+      } catch (err: unknown) {
+        const errorMessage = err instanceof Error ? err.message : "Sign in failed";
+        setError(errorMessage);
+        throw err;
+      } finally {
+        setLoading(false);
       }
-} catch (err: unknown) {
-      const errorMessage = err instanceof Error ? err.message : "Sign in failed";
-      setError(errorMessage);
-      throw err;
-    } finally {
-      setLoading(false);
-    }
-  }, [isGuest, setIsGuest, setLoading, setError]);
+    },
+    [isGuest, setIsGuest, setLoading, setError],
+  );
 
   const signOut = useCallback(async () => {
     const service = getAuthService();
-    if (!service) {
-      return;
-    }
+    if (!service) return;
+
     try {
       setLoading(true);
       await service.signOut();
@@ -102,76 +82,22 @@ export function useAuthActions(
   }, [setLoading]);
 
   const continueAsGuest = useCallback(async () => {
-    /* eslint-disable-next-line no-console */
-    if (__DEV__) {
-      console.log("========================================");
-      console.log("[useAuthActions] 🎯 continueAsGuest() CALLED");
-      console.log("[useAuthActions] Current state:", {
-        isGuest,
-        loading: state.loading,
-      });
-      console.log("========================================");
-    }
-    
     const service = getAuthService();
     if (!service) {
-      /* eslint-disable-next-line no-console */
-      if (__DEV__) {
-        console.log("[useAuthActions] ⚠️ No service available, setting isGuest directly");
-      }
       setIsGuest(true);
       return;
     }
-    
+
     try {
-      /* eslint-disable-next-line no-console */
-      if (__DEV__) {
-        console.log("[useAuthActions] Setting loading to true...");
-      }
       setLoading(true);
-      
-      /* eslint-disable-next-line no-console */
-      if (__DEV__) {
-        console.log("[useAuthActions] 📞 Calling service.setGuestMode()...");
-      }
       await service.setGuestMode();
-      
-      /* eslint-disable-next-line no-console */
-      if (__DEV__) {
-        console.log("[useAuthActions] ✅ Service.setGuestMode() completed successfully");
-        console.log("[useAuthActions] Setting isGuest to true...");
-      }
-      
       setIsGuest(true);
-      
-      /* eslint-disable-next-line no-console */
-      if (__DEV__) {
-        console.log("[useAuthActions] ✅ isGuest set to true");
-      }
-    } catch (error: unknown) {
-      const errorMessage = error instanceof Error ? error.message : "Failed to continue as guest";
-      setError(errorMessage);
-      if (__DEV__) {
-        console.error("[useAuthActions] ❌ ERROR in continueAsGuest:", error);
-        console.error("[useAuthActions] Error details:", {
-          message: errorMessage,
-          stack: error instanceof Error ? error.stack : undefined,
-        });
-      }
+    } catch {
       setIsGuest(true);
     } finally {
-      /* eslint-disable-next-line no-console */
-      if (__DEV__) {
-        console.log("[useAuthActions] Setting loading to false...");
-      }
       setLoading(false);
-      /* eslint-disable-next-line no-console */
-      if (__DEV__) {
-        console.log("[useAuthActions] ✅ continueAsGuest() FINISHED");
-        console.log("========================================");
-      }
     }
-  }, [isGuest, setIsGuest, setLoading, state.loading]);
+  }, [setIsGuest, setLoading]);
 
   return {
     signUp,
@@ -180,4 +106,3 @@ export function useAuthActions(
     continueAsGuest,
   };
 }
-

package/src/presentation/hooks/useAuthState.ts

@@ -25,9 +25,7 @@ export interface UseAuthStateResult {
  * Hook for managing authentication state
  */
 export function useAuthState(): UseAuthStateResult {
-  if (__DEV__) console.log("[useAuthState] Hook called");
   const { user: firebaseUser, loading: firebaseLoading } = useFirebaseAuth();
-  if (__DEV__) console.log("[useAuthState] firebaseUser:", firebaseUser?.uid, "firebaseLoading:", firebaseLoading);
   const [isGuest, setIsGuest] = useState(() => {
     const service = getAuthService();
     return service ? service.getIsGuestMode() : false;
@@ -44,7 +42,8 @@ export function useAuthState(): UseAuthStateResult {
     return mapToAuthUser(firebaseUser);
   }, [isGuest, firebaseUser?.uid]);
 
-  const isAuthenticated = !!user && !isGuest;
+  // Anonymous users are NOT authenticated - they need to register/login
+  const isAuthenticated = !!user && !isGuest && !user.isAnonymous;
 
   // Keep ref in sync with state
   useEffect(() => {

package/src/presentation/hooks/useLoginForm.ts

@@ -23,9 +23,6 @@ export interface UseLoginFormResult {
   displayError: string | null;
 }
 
-/**
- * Hook for login form logic
- */
 export function useLoginForm(): UseLoginFormResult {
   const { t } = useLocalization();
   const { signIn, loading, error, continueAsGuest } = useAuth();
@@ -36,23 +33,25 @@ export function useLoginForm(): UseLoginFormResult {
   const [passwordError, setPasswordError] = useState<string | null>(null);
   const [localError, setLocalError] = useState<string | null>(null);
 
-  const handleEmailChange = useCallback((text: string) => {
-    setEmail(text);
-    if (emailError) setEmailError(null);
-    if (localError) setLocalError(null);
-  }, [emailError, localError]);
-
-  const handlePasswordChange = useCallback((text: string) => {
-    setPassword(text);
-    if (passwordError) setPasswordError(null);
-    if (localError) setLocalError(null);
-  }, [passwordError, localError]);
+  const handleEmailChange = useCallback(
+    (text: string) => {
+      setEmail(text);
+      if (emailError) setEmailError(null);
+      if (localError) setLocalError(null);
+    },
+    [emailError, localError],
+  );
+
+  const handlePasswordChange = useCallback(
+    (text: string) => {
+      setPassword(text);
+      if (passwordError) setPasswordError(null);
+      if (localError) setLocalError(null);
+    },
+    [passwordError, localError],
+  );
 
   const handleSignIn = useCallback(async () => {
-    /* eslint-disable-next-line no-console */
-    if (__DEV__) {
-      console.log("[useLoginForm] handleSignIn called");
-    }
     setEmailError(null);
     setPasswordError(null);
     setLocalError(null);
@@ -73,29 +72,11 @@ export function useLoginForm(): UseLoginFormResult {
       hasError = true;
     }
 
-    if (hasError) {
-      /* eslint-disable-next-line no-console */
-      if (__DEV__) {
-        console.log("[useLoginForm] Validation errors, returning early");
-      }
-      return;
-    }
+    if (hasError) return;
 
     try {
-      /* eslint-disable-next-line no-console */
-      if (__DEV__) {
-        console.log("[useLoginForm] Calling signIn with email:", email.trim());
-      }
       await signIn(email.trim(), password);
-      /* eslint-disable-next-line no-console */
-      if (__DEV__) {
-        console.log("[useLoginForm] signIn completed successfully");
-      }
-    } catch (err: any) {
-      /* eslint-disable-next-line no-console */
-      if (__DEV__) {
-        console.error("[useLoginForm] signIn error:", err);
-      }
+    } catch (err: unknown) {
       const localizationKey = getAuthErrorLocalizationKey(err);
       const errorMessage = t(localizationKey);
       setLocalError(errorMessage);
@@ -103,39 +84,12 @@ export function useLoginForm(): UseLoginFormResult {
   }, [email, password, t, signIn]);
 
   const handleContinueAsGuest = useCallback(async () => {
-    /* eslint-disable-next-line no-console */
-    if (__DEV__) {
-      console.log("========================================");
-      console.log("[useLoginForm] 🎯 Continue as Guest button PRESSED");
-      console.log("[useLoginForm] Current loading state:", loading);
-      console.log("[useLoginForm] Current error state:", error);
-      console.log("========================================");
-    }
-    
     try {
-      /* eslint-disable-next-line no-console */
-      if (__DEV__) {
-        console.log("[useLoginForm] Calling continueAsGuest() function...");
-      }
-      
       await continueAsGuest();
-      
-      /* eslint-disable-next-line no-console */
-      if (__DEV__) {
-        console.log("[useLoginForm] ✅ continueAsGuest() completed successfully");
-        console.log("[useLoginForm] Current loading state after:", loading);
-      }
-    } catch (err) {
-      /* eslint-disable-next-line no-console */
-      if (__DEV__) {
-        console.error("[useLoginForm] ❌ ERROR in continueAsGuest:", err);
-        console.error("[useLoginForm] Error details:", {
-          message: err instanceof Error ? err.message : String(err),
-          stack: err instanceof Error ? err.stack : undefined,
-        });
-      }
+    } catch {
+      // Silent fail - guest mode should always work
     }
-  }, [continueAsGuest, loading, error]);
+  }, [continueAsGuest]);
 
   const displayError = localError || error;
 
@@ -153,4 +107,3 @@ export function useLoginForm(): UseLoginFormResult {
     displayError,
   };
 }
-