@umituz/react-native-subscription 2.27.4 → 2.27.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@umituz/react-native-subscription",
-  "version": "2.27.4",
+  "version": "2.27.7",
   "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/presentation/hooks/useCredits.ts

@@ -3,11 +3,11 @@
  *
  * Fetches user credits with TanStack Query best practices.
  * Uses status-based state management for reliable loading detection.
- * Auto-initializes free credits for registered users only.
+ * Free credits initialization is delegated to useFreeCreditsInit hook.
  */
 
 import { useQuery } from "@umituz/react-native-design-system";
-import { useCallback, useMemo, useEffect, useState } from "react";
+import { useCallback, useMemo } from "react";
 import {
   useAuthStore,
   selectUserId,
@@ -19,6 +19,7 @@ import {
   getCreditsConfig,
   isCreditsRepositoryConfigured,
 } from "../../infrastructure/repositories/CreditsRepositoryProvider";
+import { useFreeCreditsInit } from "./useFreeCreditsInit";
 
 declare const __DEV__: boolean;
 
@@ -27,8 +28,6 @@ export const creditsQueryKeys = {
   user: (userId: string) => ["credits", userId] as const,
 };
 
-const freeCreditsInitAttempted = new Set<string>();
-
 export type CreditsLoadStatus = "idle" | "loading" | "initializing" | "ready" | "error";
 
 export interface UseCreditsResult {
@@ -59,7 +58,6 @@ export const useCredits = (): UseCreditsResult => {
   const userId = useAuthStore(selectUserId);
   const isAnonymous = useAuthStore(selectIsAnonymous);
   const isRegisteredUser = !!userId && !isAnonymous;
-  const [isInitializingFreeCredits, setIsInitializingFreeCredits] = useState(false);
 
   const isConfigured = isCreditsRepositoryConfigured();
   const config = getCreditsConfig();
@@ -96,47 +94,18 @@ export const useCredits = (): UseCreditsResult => {
   });
 
   const credits = data ?? null;
-  const freeCredits = config.freeCredits ?? 0;
-  const autoInit = config.autoInitializeFreeCredits !== false && freeCredits > 0;
   const querySuccess = status === "success";
-
-  useEffect(() => {
-    const shouldInitFreeCredits =
-      querySuccess &&
-      userId &&
-      isRegisteredUser &&
-      isConfigured &&
-      !credits &&
-      autoInit &&
-      !freeCreditsInitAttempted.has(userId);
-
-    if (shouldInitFreeCredits) {
-      freeCreditsInitAttempted.add(userId);
-      setIsInitializingFreeCredits(true);
-
-      if (typeof __DEV__ !== "undefined" && __DEV__) {
-        console.log("[useCredits] Initializing free credits:", userId.slice(0, 8));
-      }
-
-      const repository = getCreditsRepository();
-      repository.initializeFreeCredits(userId).then((result) => {
-        setIsInitializingFreeCredits(false);
-
-        if (result.success) {
-          if (typeof __DEV__ !== "undefined" && __DEV__) {
-            console.log("[useCredits] Free credits initialized:", result.data?.credits);
-          }
-          refetch();
-        } else if (typeof __DEV__ !== "undefined" && __DEV__) {
-          console.warn("[useCredits] Free credits init failed:", result.error?.message);
-        }
-      });
-    } else if (querySuccess && userId && isAnonymous && !credits && autoInit) {
-      if (typeof __DEV__ !== "undefined" && __DEV__) {
-        console.log("[useCredits] Skipping free credits - anonymous user must register first");
-      }
-    }
-  }, [querySuccess, userId, isRegisteredUser, isAnonymous, isConfigured, credits, autoInit, refetch]);
+  const hasCreditsData = (credits?.credits ?? 0) > 0;
+
+  // Delegate free credits initialization to dedicated hook
+  const { isInitializing, needsInit } = useFreeCreditsInit({
+    userId,
+    isRegisteredUser,
+    isAnonymous,
+    hasCredits: hasCreditsData,
+    querySuccess,
+    onInitComplete: refetch,
+  });
 
   const derivedValues = useMemo(() => {
     const has = (credits?.credits ?? 0) > 0;
@@ -149,7 +118,12 @@ export const useCredits = (): UseCreditsResult => {
     [credits]
   );
 
-  const loadStatus = deriveLoadStatus(status, isInitializingFreeCredits, queryEnabled);
+  // Include needsInit in initializing state for accurate loading detection
+  const loadStatus = deriveLoadStatus(
+    status,
+    isInitializing || needsInit,
+    queryEnabled
+  );
   const isCreditsLoaded = loadStatus === "ready";
   const isLoading = loadStatus === "loading" || loadStatus === "initializing";
 

package/src/presentation/hooks/useFreeCreditsInit.ts

@@ -0,0 +1,87 @@
+/**
+ * useFreeCreditsInit Hook
+ *
+ * Handles free credits initialization for newly registered users.
+ * Separated from useCredits for better maintainability and testability.
+ */
+
+import { useState, useEffect, useCallback } from "react";
+import {
+  getCreditsRepository,
+  getCreditsConfig,
+  isCreditsRepositoryConfigured,
+} from "../../infrastructure/repositories/CreditsRepositoryProvider";
+
+declare const __DEV__: boolean;
+
+const freeCreditsInitAttempted = new Set<string>();
+
+export interface UseFreeCreditsInitParams {
+  userId: string | null | undefined;
+  isRegisteredUser: boolean;
+  isAnonymous: boolean;
+  hasCredits: boolean;
+  querySuccess: boolean;
+  onInitComplete: () => void;
+}
+
+export interface UseFreeCreditsInitResult {
+  isInitializing: boolean;
+  needsInit: boolean;
+}
+
+export function useFreeCreditsInit(params: UseFreeCreditsInitParams): UseFreeCreditsInitResult {
+  const { userId, isRegisteredUser, isAnonymous, hasCredits, querySuccess, onInitComplete } = params;
+  const [isInitializing, setIsInitializing] = useState(false);
+
+  const isConfigured = isCreditsRepositoryConfigured();
+  const config = getCreditsConfig();
+  const freeCredits = config.freeCredits ?? 0;
+  const autoInit = config.autoInitializeFreeCredits !== false && freeCredits > 0;
+
+  const needsInit =
+    querySuccess &&
+    !!userId &&
+    isRegisteredUser &&
+    isConfigured &&
+    !hasCredits &&
+    autoInit &&
+    !freeCreditsInitAttempted.has(userId);
+
+  const initializeFreeCredits = useCallback(async (uid: string) => {
+    freeCreditsInitAttempted.add(uid);
+    setIsInitializing(true);
+
+    if (typeof __DEV__ !== "undefined" && __DEV__) {
+      console.log("[useFreeCreditsInit] Initializing free credits:", uid.slice(0, 8));
+    }
+
+    const repository = getCreditsRepository();
+    const result = await repository.initializeFreeCredits(uid);
+    setIsInitializing(false);
+
+    if (result.success) {
+      if (typeof __DEV__ !== "undefined" && __DEV__) {
+        console.log("[useFreeCreditsInit] Free credits initialized:", result.data?.credits);
+      }
+      onInitComplete();
+    } else if (typeof __DEV__ !== "undefined" && __DEV__) {
+      console.warn("[useFreeCreditsInit] Free credits init failed:", result.error?.message);
+    }
+  }, [onInitComplete]);
+
+  useEffect(() => {
+    if (needsInit && userId) {
+      initializeFreeCredits(userId);
+    } else if (querySuccess && userId && isAnonymous && !hasCredits && autoInit) {
+      if (typeof __DEV__ !== "undefined" && __DEV__) {
+        console.log("[useFreeCreditsInit] Skipping - anonymous user must register first");
+      }
+    }
+  }, [needsInit, userId, querySuccess, isAnonymous, hasCredits, autoInit, initializeFreeCredits]);
+
+  return {
+    isInitializing,
+    needsInit,
+  };
+}