@umituz/react-native-subscription 2.37.42 → 2.37.44

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@umituz/react-native-subscription",
-  "version": "2.37.42",
+  "version": "2.37.44",
   "description": "Complete subscription management with RevenueCat, paywall UI, and credits system for React Native apps",
   "main": "./src/index.ts",
   "types": "./src/index.ts",

package/src/domains/credits/application/creditOperationUtils.ts

@@ -11,7 +11,7 @@ import {
 import { PURCHASE_ID_PREFIXES, PROCESSED_PURCHASES_WINDOW } from "../core/CreditsConstants";
 
 
-export function calculateNewCredits({ metadata, existingData, creditLimit, purchaseId }: CalculateCreditsParams): number {
+export function calculateNewCredits({ metadata, existingData, creditLimit }: CalculateCreditsParams): number {
   const isExpired = metadata.expirationDate ? isPast(metadata.expirationDate) : false;
   const isPremium = metadata.isPremium;
   const status = resolveSubscriptionStatus({

package/src/domains/credits/infrastructure/operations/CreditsInitializer.ts

@@ -19,23 +19,26 @@ interface InitializeCreditsParams {
   type?: PurchaseType;
 }
 
-function isTransientError(error: any): boolean {
+function isTransientError(error: unknown): boolean {
+  const code = error instanceof Error ? (error as Error & { code?: string }).code : undefined;
+  const message = error instanceof Error ? error.message : '';
+
   return (
-    error?.code === 'already-exists' ||
-    error?.code === 'DEADLINE_EXCEEDED' ||
-    error?.code === 'UNAVAILABLE' ||
-    error?.code === 'RESOURCE_EXHAUSTED' ||
+    code === 'already-exists' ||
+    code === 'DEADLINE_EXCEEDED' ||
+    code === 'UNAVAILABLE' ||
+    code === 'RESOURCE_EXHAUSTED' ||
     // Firestore transaction contention: document was modified between our read and write.
     // runTransaction does not auto-retry on failed-precondition, so we do it here.
-    error?.code === 'failed-precondition' ||
-    error?.code === 'FAILED_PRECONDITION' ||
+    code === 'failed-precondition' ||
+    code === 'FAILED_PRECONDITION' ||
     // Firestore transaction aborted due to concurrent modification.
-    error?.code === 'aborted' ||
-    error?.code === 'ABORTED' ||
-    error?.message?.includes('already-exists') ||
-    error?.message?.includes('timeout') ||
-    error?.message?.includes('unavailable') ||
-    error?.message?.includes('failed-precondition')
+    code === 'aborted' ||
+    code === 'ABORTED' ||
+    message.includes('already-exists') ||
+    message.includes('timeout') ||
+    message.includes('unavailable') ||
+    message.includes('failed-precondition')
   );
 }
 
@@ -46,7 +49,7 @@ export async function initializeCreditsWithRetry(params: InitializeCreditsParams
   const cfg = { ...config, creditLimit };
 
   const maxRetries = 3;
-  let lastError: any;
+  let lastError: unknown;
 
   for (let attempt = 0; attempt < maxRetries; attempt++) {
     try {
@@ -76,7 +79,7 @@ export async function initializeCreditsWithRetry(params: InitializeCreditsParams
         data: result.finalData ? mapCreditsDocumentToEntity(result.finalData) : null,
         error: null,
       };
-    } catch (error: any) {
+    } catch (error: unknown) {
       lastError = error;
 
       if (isTransientError(error) && attempt < maxRetries - 1) {
@@ -98,7 +101,9 @@ export async function initializeCreditsWithRetry(params: InitializeCreditsParams
       ? lastError
       : 'Unknown error during credit initialization';
 
-  const errorCode = lastError?.code ?? 'UNKNOWN_ERROR';
+  const errorCode = lastError instanceof Error
+    ? (lastError as Error & { code?: string }).code ?? 'UNKNOWN_ERROR'
+    : 'UNKNOWN_ERROR';
 
   return {
     success: false,

package/src/domains/revenuecat/infrastructure/services/UserSwitchMutex.ts

@@ -18,7 +18,8 @@ class UserSwitchMutexImpl {
       }
       try {
         await this.activeSwitchPromise;
-      } catch {
+      } catch (_ignored) {
+        // Intentional: waiting for active switch to complete without failing
       }
 
       const timeSinceLastSwitch = Date.now() - this.lastSwitchTime;

package/src/domains/revenuecat/infrastructure/services/userSwitchHandler.ts

@@ -1,4 +1,4 @@
-import Purchases, { type CustomerInfo } from "react-native-purchases";
+import Purchases, { type CustomerInfo, type PurchasesOfferings } from "react-native-purchases";
 import type { InitializeResult } from "../../../../shared/application/ports/IRevenueCatService";
 import type { InitializerDeps } from "./RevenueCatInitializer.types";
 import { FAILED_INITIALIZATION_RESULT } from "./initializerConstants";
@@ -7,7 +7,7 @@ import { getPremiumEntitlement } from "../../core/types";
 
 declare const __DEV__: boolean;
 
-function buildSuccessResult(deps: InitializerDeps, customerInfo: CustomerInfo, offerings: any): InitializeResult {
+function buildSuccessResult(deps: InitializerDeps, customerInfo: CustomerInfo, offerings: PurchasesOfferings | null): InitializeResult {
   const isPremium = !!customerInfo.entitlements.active[deps.config.entitlementIdentifier];
   return { success: true, offering: offerings?.current ?? null, isPremium };
 }
@@ -17,14 +17,14 @@ function buildSuccessResult(deps: InitializerDeps, customerInfo: CustomerInfo, o
  * Empty offerings (no products configured in RevenueCat dashboard) should NOT
  * block SDK initialization. The SDK is still usable for premium checks, purchases, etc.
  */
-async function fetchOfferingsSafe(): Promise<any> {
+async function fetchOfferingsSafe(): Promise<PurchasesOfferings | null> {
   try {
     return await Purchases.getOfferings();
   } catch (error) {
     if (typeof __DEV__ !== 'undefined' && __DEV__) {
       console.warn('[UserSwitchHandler] Offerings fetch failed (non-fatal):', error);
     }
-    return { current: null, all: {} };
+    return null;
   }
 }
 
@@ -165,7 +165,7 @@ export async function handleInitialConfiguration(
       console.log('[UserSwitchHandler] ✅ Initial configuration completed:', {
         revenueCatUserId: currentUserId,
         activeEntitlements: Object.keys(customerInfo.entitlements.active),
-        offeringsCount: offerings.all ? Object.keys(offerings.all).length : 0,
+        offeringsCount: offerings?.all ? Object.keys(offerings.all).length : 0,
       });
     }
 

package/src/domains/subscription/application/SubscriptionSyncUtils.ts

@@ -17,8 +17,7 @@ export const extractRevenueCatData = (customerInfo: CustomerInfo, entitlementId:
     throw new Error('[extractRevenueCatData] entitlementId is required');
   }
 
-  const entitlement = customerInfo.entitlements.active[entitlementId]
-    ?? customerInfo.entitlements.all[entitlementId];
+  const entitlement = customerInfo.entitlements.active[entitlementId];
 
   const isPremium = !!customerInfo.entitlements.active[entitlementId];
 

package/src/domains/subscription/infrastructure/services/purchase/PurchaseErrorHandler.ts

@@ -24,18 +24,20 @@ export async function handleAlreadyPurchasedError(
 ): Promise<PurchaseResult> {
   try {
     const restoreResult = await handleRestore(deps, userId);
-    if (restoreResult.success && restoreResult.isPremium && restoreResult.customerInfo) {
-      await notifyPurchaseCompleted(
-        deps.config,
-        userId,
-        pkg.product.identifier,
-        restoreResult.customerInfo,
-        getSavedPurchase()?.source
-      );
+    if (restoreResult.success && restoreResult.customerInfo) {
+      if (restoreResult.isPremium) {
+        await notifyPurchaseCompleted(
+          deps.config,
+          userId,
+          pkg.product.identifier,
+          restoreResult.customerInfo,
+          getSavedPurchase()?.source
+        );
+      }
       clearSavedPurchase();
       return {
         success: true,
-        isPremium: true,
+        isPremium: restoreResult.isPremium,
         customerInfo: restoreResult.customerInfo,
         productId: restoreResult.productId || pkg.product.identifier,
       };

package/src/domains/subscription/infrastructure/services/purchase/PurchaseExecutor.ts

@@ -17,7 +17,12 @@ async function executeConsumablePurchase(
     clearSavedPurchase();
   }
 
-  await notifyPurchaseCompleted(config, userId, productId, customerInfo, source, packageType);
+  try {
+    await notifyPurchaseCompleted(config, userId, productId, customerInfo, source, packageType);
+  } catch (syncError) {
+    // Non-fatal: RevenueCat purchase succeeded, credits sync can be recovered on next session
+    console.error('[PurchaseExecutor] Post-purchase sync failed (purchase was successful):', syncError);
+  }
 
   return {
     success: true,
@@ -55,7 +60,12 @@ async function executeSubscriptionPurchase(
     });
   }
 
-  await notifyPurchaseCompleted(config, userId, productId, customerInfo, source, packageType);
+  try {
+    await notifyPurchaseCompleted(config, userId, productId, customerInfo, source, packageType);
+  } catch (syncError) {
+    // Non-fatal: RevenueCat purchase succeeded, credits sync can be recovered on next session
+    console.error('[PurchaseExecutor] Post-purchase sync failed (purchase was successful):', syncError);
+  }
 
   return {
     success: true,

package/src/domains/subscription/infrastructure/utils/trialEligibilityUtils.ts

@@ -44,7 +44,7 @@ export function createFallbackEligibilityMap(
   for (const productId of productIds) {
     result[productId] = {
       productId,
-      eligible: true,
+      eligible: false,
       trialDurationDays: DEFAULT_TRIAL_DURATION_DAYS,
     };
   }

package/src/domains/trial/infrastructure/DeviceTrialRepository.ts

@@ -22,23 +22,28 @@ export class DeviceTrialRepository {
     if (!this.db) return false;
     const ref = doc(this.db, DEVICE_TRIALS_COLLECTION, deviceId);
 
-    // Atomic check-then-act: ensure createdAt is set only once
-    await runTransaction(async (tx: Transaction) => {
-      const snap = await tx.get(ref);
-      const existingData = snap.data();
-
-      const updateData: Record<string, unknown> = {
-        ...data,
-        updatedAt: serverTimestamp(),
-      };
-
-      if (!existingData?.createdAt) {
-        updateData.createdAt = serverTimestamp();
-      }
-
-      tx.set(ref, updateData, { merge: true });
-    });
-
-    return true;
+    try {
+      // Atomic check-then-act: ensure createdAt is set only once
+      await runTransaction(async (tx: Transaction) => {
+        const snap = await tx.get(ref);
+        const existingData = snap.data();
+
+        const updateData: Record<string, unknown> = {
+          ...data,
+          updatedAt: serverTimestamp(),
+        };
+
+        if (!existingData?.createdAt) {
+          updateData.createdAt = serverTimestamp();
+        }
+
+        tx.set(ref, updateData, { merge: true });
+      });
+
+      return true;
+    } catch (error) {
+      console.error('[DeviceTrialRepository] Failed to save record:', error instanceof Error ? error.message : String(error));
+      return false;
+    }
   }
 }