@umituz/react-native-subscription 2.37.29 → 2.37.31

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@umituz/react-native-subscription",
-  "version": "2.37.29",
+  "version": "2.37.31",
   "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/revenuecat/infrastructure/services/userSwitchHandler.ts

@@ -9,7 +9,23 @@ declare const __DEV__: boolean;
 
 function buildSuccessResult(deps: InitializerDeps, customerInfo: CustomerInfo, offerings: any): InitializeResult {
   const isPremium = !!customerInfo.entitlements.active[deps.config.entitlementIdentifier];
-  return { success: true, offering: offerings.current, isPremium };
+  return { success: true, offering: offerings?.current ?? null, isPremium };
+}
+
+/**
+ * Fetch offerings separately - non-fatal if it fails.
+ * 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> {
+  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: {} };
+  }
 }
 
 function normalizeUserId(userId: string): string | null {
@@ -87,7 +103,7 @@ async function performUserSwitch(
 
     deps.setInitialized(true);
     deps.setCurrentUserId(normalizedUserId || undefined);
-    const offerings = await Purchases.getOfferings();
+    const offerings = await fetchOfferingsSafe();
 
     if (typeof __DEV__ !== 'undefined' && __DEV__) {
       console.log('[UserSwitchHandler] ✅ User switch completed successfully');
@@ -136,9 +152,11 @@ export async function handleInitialConfiguration(
       console.log('[UserSwitchHandler] ✅ Purchases.configure() successful');
     }
 
+    // Fetch customer info (critical) and offerings (non-fatal) separately.
+    // Empty offerings should NOT block initialization - SDK is still usable.
     const [customerInfo, offerings] = await Promise.all([
       Purchases.getCustomerInfo(),
-      Purchases.getOfferings(),
+      fetchOfferingsSafe(),
     ]);
 
     const currentUserId = await Purchases.getAppUserID();
@@ -198,11 +216,11 @@ export async function fetchCurrentUserData(deps: InitializerDeps): Promise<Initi
   try {
     const [customerInfo, offerings] = await Promise.all([
       Purchases.getCustomerInfo(),
-      Purchases.getOfferings(),
+      fetchOfferingsSafe(),
     ]);
     return buildSuccessResult(deps, customerInfo, offerings);
   } catch (error) {
-    console.error('[UserSwitchHandler] Failed to fetch customer info/offerings for initialized user', {
+    console.error('[UserSwitchHandler] Failed to fetch customer info for initialized user', {
       error
     });
     return FAILED_INITIALIZATION_RESULT;

package/src/domains/subscription/infrastructure/managers/SubscriptionManager.ts

@@ -74,9 +74,13 @@ class SubscriptionManagerImpl {
     this.serviceInstance = service ?? null;
     this.ensurePackageHandlerInitialized();
 
-    if (success) {
-      // Notify reactive state so React components re-render and enable their queries
-      const notifyUserId = (userId && userId.length > 0) ? userId : null;
+    // Always notify reactive state when service is available, even if offerings fetch failed.
+    // This ensures React components (useSubscriptionPackages, useSubscriptionStatus) are unblocked.
+    // The service IS configured and usable - empty offerings is not a fatal error.
+    const notifyUserId = (userId && userId.length > 0) ? userId : null;
+    if (service) {
+      initializationState.markInitialized(notifyUserId);
+    } else if (success) {
       initializationState.markInitialized(notifyUserId);
     }
 

package/src/domains/subscription/infrastructure/services/OfferingsFetcher.ts

@@ -54,7 +54,10 @@ export async function fetchOfferings(deps: OfferingsFetcherDeps): Promise<Purcha
         await new Promise<void>(resolve => setTimeout(resolve, FETCH_RETRY_DELAY_MS));
         continue;
       }
-      throw new Error(`Failed to fetch offerings: ${error instanceof Error ? error.message : "Unknown error"}`);
+      if (typeof __DEV__ !== "undefined" && __DEV__) {
+        console.warn('[OfferingsFetcher] Failed to fetch offerings after all retries:', error);
+      }
+      return null;
     }
   }
 

package/src/domains/subscription/presentation/useFeatureGate.ts

@@ -17,6 +17,11 @@ export function useFeatureGate(params: UseFeatureGateParams): UseFeatureGateResu
 
   const pendingActionRef = useRef<(() => void | Promise<void>) | null>(null);
   const prevCreditBalanceRef = useRef(creditBalance);
+  // Separate ref to track previous subscription state for canExecutePurchaseAction.
+  // NOTE: Must NOT use hasSubscriptionRef from useSyncedRefs here because useSyncedRefs
+  // effects run BEFORE this effect (React runs effects in definition order), so
+  // hasSubscriptionRef.current would already be the NEW value when we check it.
+  const prevHasSubscriptionRef = useRef(hasSubscription);
   const isWaitingForPurchaseRef = useRef(false);
   const isWaitingForAuthCreditsRef = useRef(false);
 
@@ -50,13 +55,14 @@ export function useFeatureGate(params: UseFeatureGateParams): UseFeatureGateResu
   }, [isCreditsLoaded, creditBalance, hasSubscription, requiredCredits, onShowPaywallRef, requiredCreditsRef]);
 
   useEffect(() => {
-
+    // Use prevHasSubscriptionRef (updated AFTER check) not hasSubscriptionRef from useSyncedRefs
+    // (which is already updated to new value before this effect runs - race condition fix)
     const shouldExecute = canExecutePurchaseAction(
       isWaitingForPurchaseRef.current,
       creditBalance,
       prevCreditBalanceRef.current ?? 0,
       hasSubscription,
-      hasSubscriptionRef.current,
+      prevHasSubscriptionRef.current,
       !!pendingActionRef.current
     );
 
@@ -67,9 +73,10 @@ export function useFeatureGate(params: UseFeatureGateParams): UseFeatureGateResu
       action();
     }
 
+    // Update AFTER check so next render has correct "prev" values
     prevCreditBalanceRef.current = creditBalance;
-    // hasSubscriptionRef is already synced by useSyncedRefs, no need to update manually
-    // eslint-disable-next-line react-hooks/exhaustive-deps
+    prevHasSubscriptionRef.current = hasSubscription;
+     
   }, [creditBalance, hasSubscription]);
 
   const requireFeature = useCallback(