@umituz/react-native-firebase 1.13.27 → 1.13.29

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@umituz/react-native-firebase",
-  "version": "1.13.27",
+  "version": "1.13.29",
   "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",
@@ -35,14 +35,23 @@
     "expo-crypto": ">=13.0.0",
     "firebase": ">=10.0.0",
     "react": ">=18.2.0",
-    "react-native": ">=0.74.0"
+    "react-native": ">=0.74.0",
+    "zustand": ">=5.0.0"
   },
   "devDependencies": {
+    "@react-native-async-storage/async-storage": "^2.2.0",
+    "@sentry/react-native": "^7.8.0",
+    "@sentry/types": "^10.32.1",
+    "@types/jest": "^30.0.0",
     "@types/react": "~19.1.10",
+    "@umituz/react-native-sentry": "^1.4.2",
+    "expo-apple-authentication": "^8.0.8",
+    "expo-crypto": "^15.0.8",
     "firebase": "^12.6.0",
     "react": "19.1.0",
     "react-native": "0.81.5",
-    "typescript": "~5.9.2"
+    "typescript": "~5.9.2",
+    "zustand": "^5.0.0"
   },
   "publishConfig": {
     "access": "public"

package/src/auth/infrastructure/services/apple-auth.service.ts

@@ -167,7 +167,10 @@ export class AppleAuthService {
     let result = "";
 
     for (let i = 0; i < randomBytes.length; i++) {
-      result += chars.charAt(randomBytes[i] % chars.length);
+      const byte = randomBytes[i];
+      if (byte !== undefined) {
+        result += chars.charAt(byte % chars.length);
+      }
     }
 
     return result;

package/src/auth/infrastructure/services/reauthentication.service.ts

@@ -126,7 +126,10 @@ async function generateNonce(length: number = 32): Promise<string> {
   let result = "";
 
   for (let i = 0; i < randomBytes.length; i++) {
-    result += chars.charAt(randomBytes[i] % chars.length);
+    const byte = randomBytes[i];
+    if (byte !== undefined) {
+      result += chars.charAt(byte % chars.length);
+    }
   }
 
   return result;
@@ -223,7 +226,7 @@ export async function getAppleReauthCredential(): Promise<{
  */
 export async function reauthenticateWithApple(user: User): Promise<ReauthenticationResult> {
   const result = await getAppleReauthCredential();
-  
+
   if (!result.success || !result.credential) {
     return {
       success: false,

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

@@ -0,0 +1,73 @@
+/**
+ * Firebase Auth Store
+ * Shared Zustand store for Firebase Auth state
+ *
+ * CRITICAL: This store ensures only ONE auth listener is created,
+ * preventing performance issues from multiple subscriptions.
+ */
+
+import { create } from "zustand";
+import { onAuthStateChanged, type User, type Auth } from "firebase/auth";
+
+declare const __DEV__: boolean;
+
+interface AuthState {
+  user: User | null;
+  loading: boolean;
+  initialized: boolean;
+  listenerSetup: boolean;
+}
+
+interface AuthActions {
+  setupListener: (auth: Auth) => void;
+  cleanup: () => void;
+}
+
+type AuthStore = AuthState & AuthActions;
+
+let unsubscribe: (() => void) | null = null;
+
+export const useAuthStore = create<AuthStore>((set, get) => ({
+  user: null,
+  loading: true,
+  initialized: false,
+  listenerSetup: false,
+
+  setupListener: (auth: Auth) => {
+    const state = get();
+
+    // Only setup listener once
+    if (state.listenerSetup || unsubscribe) {
+      return;
+    }
+
+    set({ listenerSetup: true });
+
+    unsubscribe = onAuthStateChanged(auth, (currentUser: User | null) => {
+      if (__DEV__) {
+        console.log(
+          "[AuthStore] Auth state changed:",
+          currentUser?.uid || "null"
+        );
+      }
+      set({
+        user: currentUser,
+        loading: false,
+        initialized: true,
+      });
+    });
+  },
+
+  cleanup: () => {
+    if (unsubscribe) {
+      unsubscribe();
+      unsubscribe = null;
+    }
+    set({
+      user: null,
+      loading: true,
+      initialized: false,
+      listenerSetup: false,
+    });
+  },
+}));

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

@@ -2,15 +2,14 @@
  * useFirebaseAuth Hook
  * React hook for Firebase Auth state management
  *
- * Directly uses Firebase Auth's built-in state management via onAuthStateChanged
- * Simple, performant, no retry mechanism needed (auth is pre-initialized)
+ * Uses shared Zustand store to ensure only ONE auth listener exists.
+ * This prevents performance issues from multiple subscriptions.
  */
 
-import { useEffect, useState, useRef } from "react";
-import { onAuthStateChanged, type User } from "firebase/auth";
+import { useEffect } from "react";
+import type { User } from "firebase/auth";
 import { getFirebaseAuth } from "../../infrastructure/config/FirebaseAuthClient";
-
-declare const __DEV__: boolean;
+import { useAuthStore } from "../../infrastructure/stores/auth.store";
 
 export interface UseFirebaseAuthResult {
   /** Current authenticated user from Firebase Auth */
@@ -24,7 +23,7 @@ export interface UseFirebaseAuthResult {
 /**
  * Hook for Firebase Auth state management
  *
- * Directly uses Firebase Auth's built-in state management.
+ * Uses shared store to ensure only one listener is active.
  * Auth is pre-initialized in appInitializer, so no retry needed.
  *
  * @example
@@ -33,41 +32,18 @@ export interface UseFirebaseAuthResult {
  * ```
  */
 export function useFirebaseAuth(): UseFirebaseAuthResult {
-  const [user, setUser] = useState<User | null>(null);
-  const [loading, setLoading] = useState(true);
-  const [initialized, setInitialized] = useState(false);
-
-  const unsubscribeRef = useRef<(() => void) | null>(null);
+  const { user, loading, initialized, setupListener } = useAuthStore();
 
   useEffect(() => {
     const auth = getFirebaseAuth();
 
     if (!auth) {
-      // Auth not available (offline mode or error)
-      setLoading(false);
-      setUser(null);
-      setInitialized(false);
       return;
     }
 
-    // Subscribe to auth state changes
-    unsubscribeRef.current = onAuthStateChanged(auth, (currentUser: User | null) => {
-      if (__DEV__) {
-        console.log('[useFirebaseAuth] Auth state changed:', currentUser?.uid || 'null');
-      }
-      setUser(currentUser);
-      setLoading(false);
-      setInitialized(true);
-    });
-
-    // Cleanup on unmount
-    return () => {
-      if (unsubscribeRef.current) {
-        unsubscribeRef.current();
-        unsubscribeRef.current = null;
-      }
-    };
-  }, []); // Empty deps - subscribe once on mount
+    // Setup listener (will only run once due to store check)
+    setupListener(auth);
+  }, [setupListener]);
 
   return {
     user,
@@ -75,4 +51,3 @@ export function useFirebaseAuth(): UseFirebaseAuthResult {
     initialized,
   };
 }
-

package/src/firestore/__tests__/setup.ts

@@ -3,10 +3,11 @@
  */
 
 // Mock __DEV__ for tests
-export {};
+export { };
 
 declare global {
-  var __DEV__: boolean | undefined;
+  var mockFirestore: () => any;
+  var mockFirebaseError: (code: string, message: string) => any;
 }
 
 if (typeof (global as any).__DEV__ === 'undefined') {
@@ -16,20 +17,28 @@ if (typeof (global as any).__DEV__ === 'undefined') {
 // Mock console methods to avoid noise in tests
 const originalConsole = { ...console };
 
-beforeEach(() => {
-  // Restore console methods before each test
-  Object.assign(console, originalConsole);
-});
+const globalAny = global as any;
+
+/**
+ * We check if beforeEach is available before using it
+ * This avoids errors when running tsc or in environments without Jest
+ */
+if (typeof globalAny.beforeEach !== 'undefined') {
+  globalAny.beforeEach(() => {
+    // Restore console methods before each test
+    Object.assign(console, originalConsole);
+  });
+}
 
 // Set up global test utilities
-global.mockFirestore = () => ({
-  collection: jest.fn(),
-  doc: jest.fn(),
-  runTransaction: jest.fn(),
-  batch: jest.fn(),
+globalAny.mockFirestore = () => ({
+  collection: globalAny.jest?.fn() || (() => ({})),
+  doc: globalAny.jest?.fn() || (() => ({})),
+  runTransaction: globalAny.jest?.fn() || (() => Promise.resolve()),
+  batch: globalAny.jest?.fn() || (() => ({})),
 });
 
-global.mockFirebaseError = (code: string, message: string) => {
+globalAny.mockFirebaseError = (code: string, message: string) => {
   const error = new Error(message) as any;
   error.code = code;
   return error;