@umituz/react-native-firebase 1.13.116 → 1.13.117

package/src/firestore/utils/firestore-helper.ts

@@ -31,13 +31,10 @@ export function getDb(): Firestore | null {
  */
 export async function withFirestore<T>(
   operation: (db: Firestore) => Promise<FirestoreResult<T>>,
-  logTag?: string
+  _logTag?: string
 ): Promise<FirestoreResult<T>> {
   const db = getDb();
   if (!db) {
-    if (__DEV__ && logTag) {
-      console.log(`[${logTag}] No Firestore instance`);
-    }
     return NO_DB_ERROR as FirestoreResult<T>;
   }
   return operation(db);
@@ -48,13 +45,10 @@ export async function withFirestore<T>(
  */
 export async function withFirestoreVoid(
   operation: (db: Firestore) => Promise<void>,
-  logTag?: string
+  _logTag?: string
 ): Promise<void> {
   const db = getDb();
   if (!db) {
-    if (__DEV__ && logTag) {
-      console.log(`[${logTag}] No Firestore instance`);
-    }
     return;
   }
   return operation(db);
@@ -65,13 +59,10 @@ export async function withFirestoreVoid(
  */
 export async function withFirestoreBool(
   operation: (db: Firestore) => Promise<boolean>,
-  logTag?: string
+  _logTag?: string
 ): Promise<boolean> {
   const db = getDb();
   if (!db) {
-    if (__DEV__ && logTag) {
-      console.log(`[${logTag}] No Firestore instance`);
-    }
     return false;
   }
   return operation(db);
@@ -110,9 +101,14 @@ export async function runTransaction<T>(
 ): Promise<T> {
   const db = getDb();
   if (!db) {
-    throw new Error("[runTransaction] Firestore not initialized");
+    throw new Error("[runTransaction] Firestore database is not initialized. Please ensure Firebase is properly initialized before running transactions.");
+  }
+  try {
+    return await fbRunTransaction(db, updateFunction);
+  } catch (error) {
+    const errorMessage = error instanceof Error ? error.message : 'Unknown error';
+    throw new Error(`[runTransaction] Transaction failed: ${errorMessage}`);
   }
-  return fbRunTransaction(db, updateFunction);
 }
 
 /**

package/src/firestore/utils/quota-error-detector.util.ts

@@ -16,24 +16,35 @@ const QUOTA_ERROR_MESSAGES = [
     'too many requests',
 ];
 
+/**
+ * Type guard for error with code property
+ */
+function hasCodeProperty(error: unknown): error is { code: string } {
+    return typeof error === 'object' && error !== null && 'code' in error && typeof (error as { code: unknown }).code === 'string';
+}
+
+/**
+ * Type guard for error with message property
+ */
+function hasMessageProperty(error: unknown): error is { message: string } {
+    return typeof error === 'object' && error !== null && 'message' in error && typeof (error as { message: unknown }).message === 'string';
+}
+
 /**
  * Check if error is a Firestore quota error
  */
 export function isQuotaError(error: unknown): boolean {
     if (!error || typeof error !== 'object') return false;
 
-    const code = 'code' in error && typeof (error as Record<string, unknown>).code === 'string'
-        ? (error as Record<string, unknown>).code as string
-        : undefined;
-    const message = 'message' in error && typeof (error as Record<string, unknown>).message === 'string'
-        ? (error as Record<string, unknown>).message as string
-        : undefined;
-
-    if (code && QUOTA_ERROR_CODES.some((c) => code.includes(c))) {
-        return true;
+    if (hasCodeProperty(error)) {
+        const code = error.code;
+        if (QUOTA_ERROR_CODES.some((c) => code.includes(c))) {
+            return true;
+        }
     }
 
-    if (message) {
+    if (hasMessageProperty(error)) {
+        const message = error.message;
         const lowerMessage = message.toLowerCase();
         return QUOTA_ERROR_MESSAGES.some((m) => lowerMessage.includes(m));
     }
@@ -47,13 +58,13 @@ export function isQuotaError(error: unknown): boolean {
 export function isRetryableError(error: unknown): boolean {
     if (!error || typeof error !== 'object') return false;
 
-    const code = 'code' in error && typeof (error as Record<string, unknown>).code === 'string'
-        ? (error as Record<string, unknown>).code as string
-        : undefined;
-
-    const retryableCodes = ['unavailable', 'deadline-exceeded', 'aborted'];
+    if (hasCodeProperty(error)) {
+        const code = error.code;
+        const retryableCodes = ['unavailable', 'deadline-exceeded', 'aborted'];
+        return retryableCodes.some((c) => code.includes(c));
+    }
 
-    return code ? retryableCodes.some((c) => code.includes(c)) : false;
+    return false;
 }
 
 /**

package/src/index.ts

@@ -11,10 +11,6 @@
  * - @umituz/react-native-firebase/storage
  */
 
-if (__DEV__) {
-  console.log("📍 [LIFECYCLE] @umituz/react-native-firebase/index.ts - START");
-}
-
 // Core Errors
 export {
   FirebaseError,
@@ -22,12 +18,6 @@ export {
   FirebaseConfigurationError,
 } from "./domain/errors/FirebaseError";
 
-if (__DEV__) {
-  console.log(
-    "📍 [LIFECYCLE] @umituz/react-native-firebase/index.ts - Core errors exported",
-  );
-}
-
 // Core Types
 export type { FirebaseConfig } from "./domain/value-objects/FirebaseConfig";
 export type { IFirebaseClient } from "./application/ports/IFirebaseClient";

package/src/infrastructure/config/FirebaseClient.ts

@@ -16,8 +16,6 @@
  * - Dependency Inversion: Depends on abstractions (interfaces), not concrete implementations
  */
 
-if (__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';
@@ -144,13 +142,6 @@ export async function initializeAllFirebaseServices(
 
   const { auth } = await FirebaseServiceInitializer.initializeServices(options);
 
-  if (__DEV__) {
-    console.log('[Firebase] All services initialized:', {
-      app: !!app,
-      auth: !!auth,
-    });
-  }
-
   return {
     app,
     auth,

package/src/infrastructure/config/FirebaseConfigLoader.ts

@@ -62,6 +62,15 @@ function getExpoValue(key: ConfigKey, expoConfig: Record<string, string>): strin
   return expoConfig[mapping[key]] || '';
 }
 
+/**
+ * Validate Firebase API key format
+ */
+function validateApiKey(apiKey: string): boolean {
+  // Firebase API keys typically start with "AIza" followed by 35 characters
+  const apiKeyPattern = /^AIza[0-9A-Za-z_-]{35}$/;
+  return apiKeyPattern.test(apiKey);
+}
+
 /**
  * Load Firebase configuration from Constants and environment variables
  */
@@ -76,11 +85,33 @@ export function loadFirebaseConfig(): FirebaseConfig | null {
     config[key] = expoValue || getEnvValue(key);
   }
 
-  // Validate required fields
-  if (!config.apiKey?.trim() || !config.authDomain?.trim() || !config.projectId?.trim()) {
+  // Validate required fields with proper checks
+  const apiKey = config.apiKey?.trim();
+  const authDomain = config.authDomain?.trim();
+  const projectId = config.projectId?.trim();
+
+  if (!apiKey || !authDomain || !projectId) {
+    if (__DEV__) {
+      console.error('[FirebaseConfigLoader] Missing required configuration fields');
+    }
+    return null;
+  }
+
+  // Validate API key format
+  if (!validateApiKey(apiKey)) {
+    if (__DEV__) {
+      console.error('[FirebaseConfigLoader] Invalid API key format');
+    }
     return null;
   }
 
+  // Validate authDomain format (should be like "projectId.firebaseapp.com")
+  if (!authDomain.includes('.firebaseapp.com') && !authDomain.includes('.web.app')) {
+    if (__DEV__) {
+      console.warn('[FirebaseConfigLoader] Unusual authDomain format, expected "projectId.firebaseapp.com" or similar');
+    }
+  }
+
   return config as FirebaseConfig;
 }
 

package/src/infrastructure/config/orchestrators/FirebaseInitializationOrchestrator.ts

@@ -18,32 +18,18 @@ export class FirebaseInitializationOrchestrator {
     state: FirebaseClientState
   ): FirebaseApp | null {
     if (state.isInitialized()) {
-      if (__DEV__) {
-        console.log('[Firebase] Already initialized, returning existing instance');
-      }
       return state.getApp();
     }
 
     if (state.getInitializationError()) {
-      if (__DEV__) {
-        console.log('[Firebase] Previous initialization failed, skipping retry');
-      }
       return null;
     }
 
     try {
-      if (__DEV__) {
-        console.log('[Firebase] Initializing with projectId:', config.projectId);
-      }
-
       FirebaseConfigValidator.validate(config);
       const app = FirebaseAppInitializer.initialize(config);
       state.setApp(app);
 
-      if (__DEV__) {
-        console.log('[Firebase] Successfully initialized');
-      }
-
       return app;
     } catch (error) {
       const errorMessage =
@@ -67,16 +53,9 @@ export class FirebaseInitializationOrchestrator {
 
     const autoConfig = loadFirebaseConfig();
     if (autoConfig) {
-      if (__DEV__) {
-        console.log('[Firebase] Auto-initializing with environment config');
-      }
       return this.initialize(autoConfig, state);
     }
 
-    if (__DEV__) {
-      console.log('[Firebase] No configuration found for auto-initialization');
-    }
-
     return null;
   }
 }

package/src/infrastructure/config/services/FirebaseServiceInitializer.ts

@@ -22,31 +22,15 @@ export class FirebaseServiceInitializer {
   static async initializeServices(
     options?: ServiceInitializationOptions
   ): Promise<ServiceInitializationResult> {
-    if (__DEV__) {
-      console.log('[Firebase] Initializing auth service...');
-    }
-
     let auth: unknown = null;
     if (options?.authInitializer) {
       try {
         auth = await options.authInitializer();
-        if (__DEV__) {
-          console.log('[Firebase] Auth initialized via callback');
-        }
-      } catch (error) {
-        if (__DEV__) {
-          console.warn(
-            '[Firebase] Auth initialization failed:',
-            error instanceof Error ? error.message : 'Unknown error'
-          );
-        }
+      } catch {
+        // Silently fail, auth initialization is optional
       }
     }
 
-    if (__DEV__) {
-      console.log('[Firebase] Auth service initialized:', !!auth);
-    }
-
     return { auth };
   }
 }

package/src/init/createFirebaseInitModule.ts

@@ -52,7 +52,6 @@ export function createFirebaseInitModule(
         });
 
         if (__DEV__) {
-          console.log('[createFirebaseInitModule] Firebase initialized');
         }
 
         return true;

package/src/storage/deleter.ts

@@ -32,8 +32,7 @@ function extractStoragePath(downloadUrl: string): string | null {
         }
 
         return decodeURIComponent(pathMatch[1]);
-    } catch (error) {
-        if (__DEV__) console.warn('[StorageDeleter] Failed to parse URL:', error);
+    } catch {
         return null;
     }
 }
@@ -45,18 +44,9 @@ function extractStoragePath(downloadUrl: string): string | null {
 export async function deleteStorageFile(
     downloadUrlOrPath: string
 ): Promise<DeleteResult> {
-    if (__DEV__) {
-        console.log("[StorageDeleter] Deleting", { url: downloadUrlOrPath });
-    }
-
     const storagePath = extractStoragePath(downloadUrlOrPath);
 
     if (!storagePath) {
-        if (__DEV__) {
-            console.error("[StorageDeleter] Invalid URL", {
-                url: downloadUrlOrPath,
-            });
-        }
         return { success: false, storagePath: downloadUrlOrPath };
     }
 
@@ -69,20 +59,8 @@ export async function deleteStorageFile(
         const storageRef = ref(storage, storagePath);
         await deleteObject(storageRef);
 
-        if (__DEV__) {
-            console.log("[StorageDeleter] Deleted successfully", {
-                storagePath,
-            });
-        }
-
         return { success: true, storagePath };
-    } catch (error) {
-        if (__DEV__) {
-            console.error("[StorageDeleter] Delete failed", {
-                storagePath,
-                error,
-            });
-        }
+    } catch {
         return { success: false, storagePath };
     }
 }
@@ -97,10 +75,6 @@ export async function deleteStorageFile(
 export async function deleteStorageFiles(
     urlsOrPaths: string[]
 ): Promise<BatchDeleteResult> {
-    if (__DEV__) {
-        console.log("[StorageDeleter] Batch delete", { count: urlsOrPaths.length });
-    }
-
     const storage = getStorageInstance();
     if (!storage) {
         return {
@@ -131,12 +105,5 @@ export async function deleteStorageFiles(
         }
     });
 
-    if (__DEV__) {
-        console.log("[StorageDeleter] Batch delete complete", {
-            successful: successful.length,
-            failed: failed.length,
-        });
-    }
-
     return { successful, failed };
 }

package/src/storage/uploader.ts

@@ -20,9 +20,6 @@ export function getMimeType(base64: string): string {
     if (base64.startsWith("data:image/png")) return "image/png";
     if (base64.startsWith("data:image/webp")) return "image/webp";
     if (base64.startsWith("data:image/gif")) return "image/gif";
-    if (__DEV__) {
-        console.warn("[StorageUploader] Could not detect MIME type from base64 prefix, falling back to image/jpeg");
-    }
     return "image/jpeg";
 }
 
@@ -47,13 +44,6 @@ export async function uploadBase64Image(
     const mimeType = options?.mimeType ?? getMimeType(base64);
     const dataUrl = ensureDataUrl(base64, mimeType);
 
-    if (__DEV__) {
-        console.log("[StorageUploader] Starting base64 upload", {
-            storagePath,
-            mimeType,
-        });
-    }
-
     const storage = getStorageInstance();
     if (!storage) {
         throw new Error("Firebase Storage not initialized");
@@ -71,13 +61,6 @@ export async function uploadBase64Image(
     await uploadBytes(storageRef, blob, metadata);
     const downloadUrl = await getDownloadURL(storageRef);
 
-    if (__DEV__) {
-        console.log("[StorageUploader] Upload complete", {
-            storagePath,
-            downloadUrl,
-        });
-    }
-
     return {
         downloadUrl,
         storagePath,
@@ -92,13 +75,6 @@ export async function uploadFile(
     storagePath: string,
     options?: UploadOptions
 ): Promise<UploadResult> {
-    if (__DEV__) {
-        console.log("[StorageUploader] Starting file upload", {
-            uri,
-            storagePath,
-        });
-    }
-
     const storage = getStorageInstance();
     if (!storage) {
         throw new Error("Firebase Storage not initialized");
@@ -109,9 +85,6 @@ export async function uploadFile(
     const blob = await response.blob();
 
     const contentType = options?.mimeType ?? "image/jpeg";
-    if (!options?.mimeType && __DEV__) {
-        console.warn("[StorageUploader] No MIME type provided for file upload, falling back to image/jpeg");
-    }
     const metadata: UploadMetadata = {
         contentType,
         customMetadata: options?.customMetadata,
@@ -120,13 +93,6 @@ export async function uploadFile(
     await uploadBytes(storageRef, blob, metadata);
     const downloadUrl = await getDownloadURL(storageRef);
 
-    if (__DEV__) {
-        console.log("[StorageUploader] Upload complete", {
-            storagePath,
-            downloadUrl,
-        });
-    }
-
     return {
         downloadUrl,
         storagePath,