@umituz/react-native-firebase 1.13.29 → 1.13.31

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@umituz/react-native-firebase",
-  "version": "1.13.29",
+  "version": "1.13.31",
   "description": "Unified Firebase package for React Native apps - Auth and Firestore services using Firebase JS SDK (no native modules).",
   "main": "./src/index.ts",
   "types": "./src/index.ts",

package/src/auth/infrastructure/config/initializers/FirebaseAuthInitializer.ts

@@ -5,7 +5,11 @@
  * Platform-agnostic: Works on all platforms (Web, iOS, Android)
  */
 
-import { initializeAuth, getAuth } from 'firebase/auth';
+import {
+  initializeAuth,
+  getAuth,
+  getReactNativePersistence,
+} from 'firebase/auth';
 import type { Auth } from 'firebase/auth';
 import AsyncStorage from '@react-native-async-storage/async-storage';
 import type { FirebaseApp } from 'firebase/app';
@@ -30,7 +34,6 @@ export class FirebaseAuthInitializer {
     addPackageBreadcrumb('firebase-auth', 'Initializing Firebase Auth');
 
     try {
-      // Try React Native persistence first (works on all platforms)
       const auth = this.initializeWithPersistence(app, config);
 
       if (auth) {
@@ -38,57 +41,54 @@ export class FirebaseAuthInitializer {
       }
 
       return auth;
-    } catch (error: any) {
-      // If already initialized or persistence fails, get existing instance
-      if (error.code === 'auth/already-initialized') {
-        trackPackageWarning(
-          'firebase-auth',
-          'Auth already initialized, returning existing instance',
-          { errorCode: error.code }
-        );
-
-        try {
-          return getAuth(app);
-        } catch (getAuthError) {
-          const errorObj = getAuthError instanceof Error ? getAuthError : new Error(String(getAuthError));
-          trackPackageError(errorObj, {
-            packageName: 'firebase-auth',
-            operation: 'getAuth-after-already-initialized',
-          });
-
-          if (__DEV__) {
-            console.warn('Firebase Auth: Failed to get existing auth instance:', getAuthError);
-          }
-          return null;
-        }
-      }
+    } catch (error: unknown) {
+      return this.handleInitializationError(error, app);
+    }
+  }
+
+  private static handleInitializationError(error: unknown, app: FirebaseApp): Auth | null {
+    const errorObj = error instanceof Error ? error : new Error(String(error));
+    const errorCode = (error as { code?: string })?.code;
+
+    if (errorCode === 'auth/already-initialized') {
+      trackPackageWarning(
+        'firebase-auth',
+        'Auth already initialized, returning existing instance',
+        { errorCode }
+      );
+      return this.getExistingAuth(app);
+    }
+
+    trackPackageError(errorObj, {
+      packageName: 'firebase-auth',
+      operation: 'initialize',
+      errorCode,
+    });
+
+    if (__DEV__) {
+      console.warn('Firebase Auth initialization error:', error);
+    }
+
+    return this.getExistingAuth(app);
+  }
+
+  private static getExistingAuth(app: FirebaseApp): Auth | null {
+    try {
+      return getAuth(app);
+    } catch (getAuthError) {
+      const errorObj = getAuthError instanceof Error
+        ? getAuthError
+        : new Error(String(getAuthError));
 
-      const errorObj = error instanceof Error ? error : new Error(String(error));
       trackPackageError(errorObj, {
         packageName: 'firebase-auth',
-        operation: 'initialize',
-        errorCode: error.code,
+        operation: 'getAuth-fallback',
       });
 
       if (__DEV__) {
-        console.warn('Firebase Auth initialization error:', error);
-      }
-
-      // Try to get auth instance as fallback
-      try {
-        return getAuth(app);
-      } catch (getAuthError) {
-        const fallbackError = getAuthError instanceof Error ? getAuthError : new Error(String(getAuthError));
-        trackPackageError(fallbackError, {
-          packageName: 'firebase-auth',
-          operation: 'getAuth-fallback',
-        });
-
-        if (__DEV__) {
-          console.warn('Firebase Auth: Failed to get auth instance after initialization error:', getAuthError);
-        }
-        return null;
+        console.warn('Firebase Auth: Failed to get auth instance:', getAuthError);
       }
+      return null;
     }
   }
 
@@ -97,25 +97,6 @@ export class FirebaseAuthInitializer {
     config?: FirebaseAuthConfig
   ): Auth | null {
     try {
-      const authModule = require('firebase/auth');
-      const getReactNativePersistence = authModule.getReactNativePersistence;
-
-      if (!getReactNativePersistence) {
-        trackPackageWarning(
-          'firebase-auth',
-          'getReactNativePersistence not available, using default persistence'
-        );
-
-        if (__DEV__) {
-          console.log('Firebase Auth: getReactNativePersistence not available, using default persistence');
-        }
-        try {
-          return getAuth(app);
-        } catch {
-          return null;
-        }
-      }
-
       const storage = config?.authStorage || {
         getItem: (key: string) => AsyncStorage.getItem(key),
         setItem: (key: string, value: string) => AsyncStorage.setItem(key, value),
@@ -137,12 +118,8 @@ export class FirebaseAuthInitializer {
       if (__DEV__) {
         console.warn('Firebase Auth: Persistence initialization failed:', error);
       }
-      try {
-        return getAuth(app);
-      } catch {
-        return null;
-      }
+
+      return this.getExistingAuth(app);
     }
   }
 }
-

package/src/auth/infrastructure/stores/auth.store.ts

@@ -1,9 +1,10 @@
 /**
  * Firebase Auth Store
- * Shared Zustand store for Firebase Auth state
+ * Basic Zustand store for Firebase Auth state
  *
- * CRITICAL: This store ensures only ONE auth listener is created,
- * preventing performance issues from multiple subscriptions.
+ * NOTE: For comprehensive auth state management including
+ * user types and guest mode, use @umituz/react-native-auth package.
+ * This store provides minimal state for low-level Firebase operations.
  */
 
 import { create } from "zustand";
@@ -27,7 +28,7 @@ type AuthStore = AuthState & AuthActions;
 
 let unsubscribe: (() => void) | null = null;
 
-export const useAuthStore = create<AuthStore>((set, get) => ({
+export const useFirebaseAuthStore = create<AuthStore>((set, get) => ({
   user: null,
   loading: true,
   initialized: false,
@@ -36,7 +37,6 @@ export const useAuthStore = create<AuthStore>((set, get) => ({
   setupListener: (auth: Auth) => {
     const state = get();
 
-    // Only setup listener once
     if (state.listenerSetup || unsubscribe) {
       return;
     }
@@ -44,9 +44,10 @@ export const useAuthStore = create<AuthStore>((set, get) => ({
     set({ listenerSetup: true });
 
     unsubscribe = onAuthStateChanged(auth, (currentUser: User | null) => {
-      if (__DEV__) {
+      if (typeof __DEV__ !== "undefined" && __DEV__) {
+        // eslint-disable-next-line no-console
         console.log(
-          "[AuthStore] Auth state changed:",
+          "[FirebaseAuthStore] Auth state changed:",
           currentUser?.uid || "null"
         );
       }

package/src/auth/presentation/hooks/useFirebaseAuth.ts

@@ -1,38 +1,33 @@
 /**
  * useFirebaseAuth Hook
- * React hook for Firebase Auth state management
+ * React hook for raw Firebase Auth state
  *
- * Uses shared Zustand store to ensure only ONE auth listener exists.
- * This prevents performance issues from multiple subscriptions.
+ * NOTE: For comprehensive auth management (user types, guest mode, sign in/out),
+ * use useAuth from @umituz/react-native-auth package instead.
+ * This hook provides minimal Firebase Auth access.
  */
 
 import { useEffect } from "react";
 import type { User } from "firebase/auth";
 import { getFirebaseAuth } from "../../infrastructure/config/FirebaseAuthClient";
-import { useAuthStore } from "../../infrastructure/stores/auth.store";
+import { useFirebaseAuthStore } from "../../infrastructure/stores/auth.store";
 
 export interface UseFirebaseAuthResult {
-  /** Current authenticated user from Firebase Auth */
+  /** Current Firebase user */
   user: User | null;
-  /** Whether auth state is loading (initial check) */
+  /** Whether auth state is loading */
   loading: boolean;
   /** Whether Firebase Auth is initialized */
   initialized: boolean;
 }
 
 /**
- * Hook for Firebase Auth state management
+ * Hook for raw Firebase Auth state
  *
  * Uses shared store to ensure only one listener is active.
- * Auth is pre-initialized in appInitializer, so no retry needed.
- *
- * @example
- * ```typescript
- * const { user, loading } = useFirebaseAuth();
- * ```
  */
 export function useFirebaseAuth(): UseFirebaseAuthResult {
-  const { user, loading, initialized, setupListener } = useAuthStore();
+  const { user, loading, initialized, setupListener } = useFirebaseAuthStore();
 
   useEffect(() => {
     const auth = getFirebaseAuth();
@@ -41,7 +36,6 @@ export function useFirebaseAuth(): UseFirebaseAuthResult {
       return;
     }
 
-    // Setup listener (will only run once due to store check)
     setupListener(auth);
   }, [setupListener]);
 

package/src/firestore/infrastructure/config/FirestoreClient.ts

@@ -13,6 +13,9 @@
  * - Dependency Inversion: Depends on Firebase App from @umituz/react-native-firebase
  */
 
+// eslint-disable-next-line no-console
+if (typeof __DEV__ !== "undefined" && __DEV__) console.log("📍 [LIFECYCLE] FirestoreClient.ts - Module loading");
+
 import type { Firestore } from 'firebase/firestore';
 import { getFirebaseApp } from '../../../infrastructure/config/FirebaseClient';
 import { FirebaseFirestoreInitializer } from './initializers/FirebaseFirestoreInitializer';

package/src/firestore/infrastructure/repositories/BaseRepository.ts

@@ -14,6 +14,9 @@
  * It provides a consistent interface for Firestore operations.
  */
 
+// eslint-disable-next-line no-console
+if (typeof __DEV__ !== "undefined" && __DEV__) console.log("📍 [LIFECYCLE] BaseRepository.ts - Module loading");
+
 import type { Firestore } from "firebase/firestore";
 import { getFirestore } from "../config/FirestoreClient";
 import {

package/src/infrastructure/config/FirebaseClient.ts

@@ -16,6 +16,9 @@
  * - Dependency Inversion: Depends on abstractions (interfaces), not concrete implementations
  */
 
+// eslint-disable-next-line no-console
+if (typeof __DEV__ !== "undefined" && __DEV__) console.log("📍 [LIFECYCLE] FirebaseClient.ts - Module loading");
+
 import type { FirebaseConfig } from '../../domain/value-objects/FirebaseConfig';
 import type { IFirebaseClient } from '../../application/ports/IFirebaseClient';
 import type { FirebaseApp } from './initializers/FirebaseAppInitializer';