@umituz/react-native-auth 1.8.1 → 1.10.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@umituz/react-native-auth",
-  "version": "1.8.1",
+  "version": "1.10.0",
   "description": "Authentication service for React Native apps - Secure, type-safe, and production-ready. Provider-agnostic design supports Firebase Auth and can be adapted for Supabase or other providers.",
   "main": "./src/index.ts",
   "types": "./src/index.ts",

package/src/application/ports/IAuthService.ts

@@ -37,6 +37,11 @@ export interface IAuthService {
    */
   setGuestMode(): Promise<void>;
 
+  /**
+   * Check if currently in guest mode
+   */
+  getIsGuestMode(): boolean;
+
   /**
    * Get current authenticated user
    */

package/src/infrastructure/providers/FirebaseAuthProvider.ts

@@ -19,6 +19,7 @@ import type {
 } from "../../application/ports/IAuthProvider";
 import type { AuthUser } from "../../domain/entities/AuthUser";
 import { mapFirebaseAuthError } from "../utils/AuthErrorMapper";
+import { mapToAuthUser } from "../utils/UserMapper";
 
 export class FirebaseAuthProvider implements IAuthProvider {
   private auth: Auth | null = null;
@@ -54,7 +55,7 @@ export class FirebaseAuthProvider implements IAuthProvider {
         credentials.email.trim(),
         credentials.password
       );
-      return this.mapFirebaseUser(userCredential.user);
+      return mapToAuthUser(userCredential.user) as AuthUser;
     } catch (error: unknown) {
       throw mapFirebaseAuthError(error);
     }
@@ -82,7 +83,7 @@ export class FirebaseAuthProvider implements IAuthProvider {
         }
       }
 
-      return this.mapFirebaseUser(userCredential.user);
+      return mapToAuthUser(userCredential.user) as AuthUser;
     } catch (error: unknown) {
       throw mapFirebaseAuthError(error);
     }
@@ -104,7 +105,7 @@ export class FirebaseAuthProvider implements IAuthProvider {
     if (!this.auth?.currentUser) {
       return null;
     }
-    return this.mapFirebaseUser(this.auth.currentUser);
+    return mapToAuthUser(this.auth.currentUser);
   }
 
   onAuthStateChange(callback: (user: AuthUser | null) => void): () => void {
@@ -114,18 +115,7 @@ export class FirebaseAuthProvider implements IAuthProvider {
     }
 
     return onAuthStateChanged(this.auth, (user) => {
-      callback(user ? this.mapFirebaseUser(user) : null);
+      callback(mapToAuthUser(user));
     });
   }
-
-  private mapFirebaseUser(user: User): AuthUser {
-    return {
-      uid: user.uid,
-      email: user.email,
-      displayName: user.displayName,
-      isAnonymous: user.isAnonymous,
-      emailVerified: user.emailVerified,
-      photoURL: user.photoURL,
-    };
-  }
 }

package/src/infrastructure/utils/UserMapper.ts

@@ -0,0 +1,27 @@
+/**
+ * User Mapper
+ * Single Source of Truth for user object transformations
+ */
+
+import type { AuthUser } from "../../domain/entities/AuthUser";
+
+interface FirebaseUserLike {
+  uid: string;
+  email: string | null;
+  displayName: string | null;
+  isAnonymous: boolean;
+  emailVerified: boolean;
+  photoURL: string | null;
+}
+
+export function mapToAuthUser(user: FirebaseUserLike | null): AuthUser | null {
+  if (!user) return null;
+  return {
+    uid: user.uid,
+    email: user.email,
+    displayName: user.displayName,
+    isAnonymous: user.isAnonymous,
+    emailVerified: user.emailVerified,
+    photoURL: user.photoURL,
+  };
+}

package/src/presentation/hooks/useAuthState.ts

@@ -3,26 +3,13 @@
  * Single Responsibility: Manage authentication state
  */
 
-import { useState, useEffect } from "react";
+import { useState, useEffect, useRef } from "react";
 import { DeviceEventEmitter } from "react-native";
 import { getAuthService } from "../../infrastructure/services/AuthService";
 import { useFirebaseAuth } from "@umituz/react-native-firebase-auth";
+import { mapToAuthUser } from "../../infrastructure/utils/UserMapper";
 import type { AuthUser } from "../../domain/entities/AuthUser";
 
-type FirebaseUser = ReturnType<typeof useFirebaseAuth>["user"];
-
-function mapToAuthUser(user: FirebaseUser): AuthUser | null {
-  if (!user) return null;
-  return {
-    uid: user.uid,
-    email: user.email,
-    displayName: user.displayName,
-    isAnonymous: user.isAnonymous,
-    emailVerified: user.emailVerified,
-    photoURL: user.photoURL,
-  };
-}
-
 export interface UseAuthStateResult {
   user: AuthUser | null;
   isAuthenticated: boolean;
@@ -46,9 +33,17 @@ export function useAuthState(): UseAuthStateResult {
   const [error, setError] = useState<string | null>(null);
   const [loading, setLoading] = useState(false);
 
+  // Ref to track latest isGuest value for event handlers
+  const isGuestRef = useRef(isGuest);
+
   const user = isGuest ? null : mapToAuthUser(firebaseUser);
   const isAuthenticated = !!user && !isGuest;
 
+  // Keep ref in sync with state
+  useEffect(() => {
+    isGuestRef.current = isGuest;
+  }, [isGuest]);
+
   // Reset guest mode when user signs in
   useEffect(() => {
     if (firebaseUser && isGuest) {
@@ -56,38 +51,28 @@ export function useAuthState(): UseAuthStateResult {
     }
   }, [firebaseUser, isGuest]);
 
-  // Sync isGuest state with service on mount
+  // Sync isGuest state with service on mount only
   useEffect(() => {
     const service = getAuthService();
     if (service) {
       const serviceIsGuest = service.getIsGuestMode();
-      if (serviceIsGuest !== isGuest) {
+      if (serviceIsGuest !== isGuestRef.current) {
         setIsGuest(serviceIsGuest);
       }
     }
   }, []);
 
-  // Listen for guest-mode-enabled event
+  // Listen for auth events - subscribe once on mount
   useEffect(() => {
     const guestSubscription = DeviceEventEmitter.addListener(
       "guest-mode-enabled",
-      () => {
-        /* eslint-disable-next-line no-console */
-        if (__DEV__) {
-          console.log("[useAuthState] Guest mode enabled event received");
-        }
-        setIsGuest(true);
-      }
+      () => setIsGuest(true)
     );
 
     const authSubscription = DeviceEventEmitter.addListener(
       "user-authenticated",
       () => {
-        /* eslint-disable-next-line no-console */
-        if (__DEV__) {
-          console.log("[useAuthState] User authenticated event received");
-        }
-        if (isGuest) {
+        if (isGuestRef.current) {
           setIsGuest(false);
         }
       }
@@ -97,7 +82,7 @@ export function useAuthState(): UseAuthStateResult {
       guestSubscription.remove();
       authSubscription.remove();
     };
-  }, [isGuest]);
+  }, []);
 
   return {
     user,

package/src/presentation/hooks/useLoginForm.ts

@@ -7,14 +7,7 @@ import { useState, useCallback } from "react";
 import { useLocalization } from "@umituz/react-native-localization";
 import { useAuth } from "./useAuth";
 import { getAuthErrorLocalizationKey } from "../utils/getAuthErrorMessage";
-
-/**
- * Validate email format
- */
-function validateEmail(emailValue: string): boolean {
-  const emailRegex = /^[^\s@]+@[^\s@]+\.[^\s@]+$/;
-  return emailRegex.test(emailValue);
-}
+import { validateEmail } from "../../infrastructure/utils/AuthValidation";
 
 export interface UseLoginFormResult {
   email: string;
@@ -66,10 +59,8 @@ export function useLoginForm(): UseLoginFormResult {
 
     let hasError = false;
 
-    if (!email.trim()) {
-      setEmailError(t("auth.errors.invalidEmail"));
-      hasError = true;
-    } else if (!validateEmail(email.trim())) {
+    const emailResult = validateEmail(email.trim());
+    if (!emailResult.isValid) {
       setEmailError(t("auth.errors.invalidEmail"));
       hasError = true;
     }

package/src/presentation/utils/getAuthErrorMessage.ts

@@ -3,14 +3,13 @@
  * Maps error codes to localization keys
  */
 
-import type { AuthError } from "../../domain/errors/AuthError";
+import { AuthError } from "../../domain/errors/AuthError";
 
 /**
  * Map AuthError code to localization key
  */
 export function getAuthErrorLocalizationKey(error: Error): string {
-  // Check if error has code property
-  const code = (error as any).code;
+  const code = error instanceof AuthError ? error.code : undefined;
 
   // Map error codes to localization keys
   const errorCodeMap: Record<string, string> = {
@@ -75,14 +74,3 @@ export function getAuthErrorLocalizationKey(error: Error): string {
   // Default to unknown error
   return "auth.errors.unknownError";
 }
-
-
-
-
-
-
-
-
-
-
-