@umituz/react-native-ai-fal-provider 2.0.17 → 2.0.19

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@umituz/react-native-ai-fal-provider",
-  "version": "2.0.17",
+  "version": "2.0.19",
   "description": "FAL AI provider for React Native - implements IAIProvider interface for unified AI generation",
   "main": "./src/index.ts",
   "types": "./src/index.ts",

package/src/domain/types/input-builders.types.ts

@@ -55,5 +55,5 @@ export interface TextToVideoOptions {
 }
 
 export interface FaceSwapOptions {
-  // No additional options
+  readonly enhanceFaces?: boolean;
 }

package/src/domain/types/provider.types.ts

@@ -39,8 +39,6 @@ export interface AIProviderConfig {
   textToImageModel?: string;
   imageEditModel?: string;
   videoGenerationModel?: string;
-  videoFeatureModels?: Partial<Record<VideoFeatureType, string>>;
-  imageFeatureModels?: Partial<Record<ImageFeatureType, string>>;
 }
 
 // =============================================================================
@@ -93,6 +91,7 @@ export interface SubscribeOptions<T = unknown> {
 
 export interface RunOptions {
   onProgress?: (progress: ProviderProgressInfo) => void;
+  signal?: AbortSignal;
 }
 
 // =============================================================================

package/src/exports/infrastructure.ts

@@ -14,20 +14,10 @@ export type { FalProviderType } from "../infrastructure/services";
 
 export {
   categorizeFalError,
-  falErrorMapper,
   mapFalError,
   isFalErrorRetryable,
   buildSingleImageInput,
   buildDualImageInput,
-  buildUpscaleInput,
-  buildPhotoRestoreInput,
-  buildVideoFromImageInput,
-  buildFaceSwapInput,
-  buildImageToImageInput,
-  buildRemoveBackgroundInput,
-  buildRemoveObjectInput,
-  buildReplaceBackgroundInput,
-  buildHDTouchUpInput,
 } from "../infrastructure/utils";
 
 export { CostTracker } from "../infrastructure/utils/cost-tracker";

package/src/infrastructure/services/fal-provider-subscription.ts

@@ -11,8 +11,6 @@ import { mapFalStatusToJobStatus } from "./fal-status-mapper";
 import { validateNSFWContent } from "../validators/nsfw-validator";
 import { NSFWContentError } from "./nsfw-content-error";
 
-declare const __DEV__: boolean | undefined;
-
 interface FalApiErrorDetail {
   msg?: string;
   type?: string;
@@ -59,22 +57,16 @@ export async function handleFalSubscription<T = unknown>(
   signal?: AbortSignal
 ): Promise<{ result: T; requestId: string | null }> {
   const timeoutMs = options?.timeoutMs ?? DEFAULT_FAL_CONFIG.defaultTimeoutMs;
+
+  if (timeoutMs <= 0 || timeoutMs > 3600000) {
+    throw new Error(`Invalid timeout: ${timeoutMs}ms. Must be between 1 and 3600000ms (1 hour)`);
+  }
+
   let timeoutId: ReturnType<typeof setTimeout> | null = null;
   let currentRequestId: string | null = null;
   let abortHandler: (() => void) | null = null;
+  let listenerAdded = false;
 
-  if (typeof __DEV__ !== "undefined" && __DEV__) {
-    console.log("[FalProvider] Subscribe started:", {
-      model,
-      timeoutMs,
-      inputKeys: Object.keys(input),
-      hasImageUrl: !!input.image_url,
-      hasPrompt: !!input.prompt,
-      promptPreview: input.prompt ? String(input.prompt).substring(0, 50) + "..." : "N/A",
-    });
-  }
-
-  // Check if already aborted before starting
   if (signal?.aborted) {
     throw new Error("Request cancelled by user");
   }
@@ -82,7 +74,6 @@ export async function handleFalSubscription<T = unknown>(
   let lastStatus = "";
 
   try {
-    // Create promises array conditionally to avoid unnecessary abort promise creation
     const promises: Promise<unknown>[] = [
       fal.subscribe(model, {
         input,
@@ -91,14 +82,13 @@ export async function handleFalSubscription<T = unknown>(
         onQueueUpdate: (update: { status: string; logs?: unknown[]; request_id?: string }) => {
           currentRequestId = update.request_id ?? currentRequestId;
           const jobStatus = mapFalStatusToJobStatus({
-            ...update as unknown as FalQueueStatus,
+            status: update.status as FalQueueStatus["status"],
             requestId: currentRequestId ?? "",
+            logs: update.logs as FalQueueStatus["logs"],
+            queuePosition: undefined,
           });
           if (jobStatus.status !== lastStatus) {
             lastStatus = jobStatus.status;
-            if (typeof __DEV__ !== "undefined" && __DEV__) {
-              console.log("[FalProvider] Status:", jobStatus.status, "RequestId:", currentRequestId);
-            }
           }
           options?.onQueueUpdate?.(jobStatus);
         },
@@ -110,55 +100,33 @@ export async function handleFalSubscription<T = unknown>(
       }),
     ];
 
-    // Add abort promise only if signal is provided and not already aborted
     if (signal && !signal.aborted) {
       const abortPromise = new Promise<never>((_, reject) => {
         abortHandler = () => {
           reject(new Error("Request cancelled by user"));
         };
         signal.addEventListener("abort", abortHandler);
+        listenerAdded = true;
       });
       promises.push(abortPromise);
     }
 
     const result = await Promise.race(promises);
 
-    if (typeof __DEV__ !== "undefined" && __DEV__) {
-      console.log("[FalProvider] Subscribe completed:", {
-        model,
-        requestId: currentRequestId,
-        resultKeys: result ? Object.keys(result as object) : "null",
-        hasVideo: !!(result as Record<string, unknown>)?.video,
-        hasOutput: !!(result as Record<string, unknown>)?.output,
-        hasData: !!(result as Record<string, unknown>)?.data,
-      });
-      // Log full result structure for debugging
-      console.log("[FalProvider] Result structure:", JSON.stringify(result, null, 2).substring(0, 1000));
-    }
-
     validateNSFWContent(result as Record<string, unknown>);
 
     options?.onResult?.(result as T);
     return { result: result as T, requestId: currentRequestId };
   } catch (error) {
-    // Preserve NSFWContentError type
     if (error instanceof NSFWContentError) {
-      if (typeof __DEV__ !== "undefined" && __DEV__) {
-        console.error("[FalProvider] NSFW content detected");
-      }
       throw error;
     }
 
-    // Parse FAL error and throw with user-friendly message
     const userMessage = parseFalError(error);
-    if (typeof __DEV__ !== "undefined" && __DEV__) {
-      console.error("[FalProvider] Error:", userMessage);
-    }
     throw new Error(userMessage);
   } finally {
     if (timeoutId) clearTimeout(timeoutId);
-    // Clean up abort listener to prevent memory leak (only if it was added)
-    if (abortHandler && signal && !signal.aborted) {
+    if (listenerAdded && abortHandler && signal) {
       signal.removeEventListener("abort", abortHandler);
     }
   }
@@ -174,36 +142,19 @@ export async function handleFalRun<T = unknown>(
 ): Promise<T> {
   options?.onProgress?.({ progress: 10, status: "IN_PROGRESS" as const });
 
-  if (typeof __DEV__ !== "undefined" && __DEV__) {
-    console.log("[FalProvider] run() model:", model, "inputKeys:", Object.keys(input));
-  }
-
   try {
     const result = await fal.run(model, { input });
 
-    if (typeof __DEV__ !== "undefined" && __DEV__) {
-      console.log("[FalProvider] run() raw result:", JSON.stringify(result, null, 2));
-      console.log("[FalProvider] run() result type:", typeof result);
-      console.log("[FalProvider] run() result keys:", result ? Object.keys(result as object) : "null");
-    }
-
     validateNSFWContent(result as Record<string, unknown>);
 
     options?.onProgress?.({ progress: 100, status: "COMPLETED" as const });
     return result as T;
   } catch (error) {
-    // Preserve NSFWContentError type
     if (error instanceof NSFWContentError) {
-      if (typeof __DEV__ !== "undefined" && __DEV__) {
-        console.error("[FalProvider] run() NSFW content detected");
-      }
       throw error;
     }
 
     const userMessage = parseFalError(error);
-    if (typeof __DEV__ !== "undefined" && __DEV__) {
-      console.error("[FalProvider] run() Error:", userMessage);
-    }
     throw new Error(userMessage);
   }
 }

package/src/infrastructure/services/fal-provider.constants.ts

@@ -14,17 +14,8 @@ export const DEFAULT_FAL_CONFIG = {
 } as const;
 
 export const FAL_CAPABILITIES: ProviderCapabilities = {
-  imageFeatures: [
-    "upscale",
-    "photo-restore",
-    "face-swap",
-    "anime-selfie",
-    "remove-background",
-    "remove-object",
-    "hd-touch-up",
-    "replace-background",
-  ] as const,
-  videoFeatures: ["image-to-video", "text-to-video"] as const,
+  imageFeatures: [] as const,
+  videoFeatures: [] as const,
   textToImage: true,
   textToVideo: true,
   imageToVideo: true,

package/src/infrastructure/services/fal-provider.ts

@@ -6,8 +6,8 @@
 import { fal } from "@fal-ai/client";
 import type {
   IAIProvider, AIProviderConfig, JobSubmission, JobStatus, SubscribeOptions,
-  RunOptions, ImageFeatureType, VideoFeatureType, ImageFeatureInputData,
-  VideoFeatureInputData, ProviderCapabilities,
+  RunOptions, ProviderCapabilities, ImageFeatureType, VideoFeatureType,
+  ImageFeatureInputData, VideoFeatureInputData,
 } from "../../domain/types";
 import type { CostTrackerConfig } from "../../domain/entities/cost-tracking.types";
 import { DEFAULT_FAL_CONFIG, FAL_CAPABILITIES } from "./fal-provider.constants";
@@ -18,11 +18,8 @@ import {
   removeRequest, cancelAllRequests, hasActiveRequests,
 } from "./request-store";
 import * as queueOps from "./fal-queue-operations";
-import * as featureModels from "./fal-feature-models";
 import { validateInput } from "../utils/input-validator.util";
 
-declare const __DEV__: boolean | undefined;
-
 export class FalProvider implements IAIProvider {
   readonly providerId = "fal";
   readonly providerName = "FAL AI";
@@ -30,13 +27,9 @@ export class FalProvider implements IAIProvider {
   private apiKey: string | null = null;
   private initialized = false;
   private costTracker: CostTracker | null = null;
-  private videoFeatureModels: Record<string, string> = {};
-  private imageFeatureModels: Record<string, string> = {};
 
   initialize(config: AIProviderConfig): void {
     this.apiKey = config.apiKey;
-    this.videoFeatureModels = config.videoFeatureModels ?? {};
-    this.imageFeatureModels = config.imageFeatureModels ?? {};
     fal.config({
       credentials: config.apiKey,
       retry: {
@@ -46,9 +39,6 @@ export class FalProvider implements IAIProvider {
       },
     });
     this.initialized = true;
-    if (typeof __DEV__ !== "undefined" && __DEV__) {
-      console.log("[FalProvider] Initialized");
-    }
   }
 
   enableCostTracking(config?: CostTrackerConfig): void {
@@ -75,10 +65,24 @@ export class FalProvider implements IAIProvider {
     return FAL_CAPABILITIES;
   }
 
-  isFeatureSupported(feature: ImageFeatureType | VideoFeatureType): boolean {
-    const caps = this.getCapabilities();
-    return caps.imageFeatures.includes(feature as ImageFeatureType) ||
-           caps.videoFeatures.includes(feature as VideoFeatureType);
+  isFeatureSupported(_feature: ImageFeatureType | VideoFeatureType): boolean {
+    return false;
+  }
+
+  getImageFeatureModel(_feature: ImageFeatureType): string {
+    throw new Error("Feature-specific models are not supported in this provider. Use the main app's feature implementations.");
+  }
+
+  buildImageFeatureInput(_feature: ImageFeatureType, _data: ImageFeatureInputData): Record<string, unknown> {
+    throw new Error("Feature-specific input building is not supported in this provider. Use the main app's feature implementations.");
+  }
+
+  getVideoFeatureModel(_feature: VideoFeatureType): string {
+    throw new Error("Feature-specific models are not supported in this provider. Use the main app's feature implementations.");
+  }
+
+  buildVideoFeatureInput(_feature: VideoFeatureType, _data: VideoFeatureInputData): Record<string, unknown> {
+    throw new Error("Feature-specific input building is not supported in this provider. Use the main app's feature implementations.");
   }
 
   private validateInit(): void {
@@ -116,9 +120,6 @@ export class FalProvider implements IAIProvider {
 
     const existing = getExistingRequest<T>(key);
     if (existing) {
-      if (typeof __DEV__ !== "undefined" && __DEV__) {
-        console.log(`[FalProvider] Dedup: returning existing promise for ${model}`);
-      }
       return existing.promise;
     }
 
@@ -142,6 +143,11 @@ export class FalProvider implements IAIProvider {
     validateInput(model, input);
     const processedInput = await preprocessInput(input);
 
+    const signal = options?.signal;
+    if (signal?.aborted) {
+      throw new Error("Request cancelled by user");
+    }
+
     return executeWithCostTracking({
       tracker: this.costTracker,
       model,
@@ -163,22 +169,6 @@ export class FalProvider implements IAIProvider {
   hasRunningRequest(): boolean {
     return hasActiveRequests();
   }
-
-  getImageFeatureModel(feature: ImageFeatureType): string {
-    return featureModels.getImageFeatureModel(this.imageFeatureModels, feature);
-  }
-
-  buildImageFeatureInput(feature: ImageFeatureType, data: ImageFeatureInputData): Record<string, unknown> {
-    return featureModels.buildImageFeatureInput(feature, data);
-  }
-
-  getVideoFeatureModel(feature: VideoFeatureType): string {
-    return featureModels.getVideoFeatureModel(this.videoFeatureModels, feature);
-  }
-
-  buildVideoFeatureInput(feature: VideoFeatureType, data: VideoFeatureInputData): Record<string, unknown> {
-    return featureModels.buildVideoFeatureInput(feature, data);
-  }
 }
 
 export const falProvider = new FalProvider();

package/src/infrastructure/services/fal-queue-operations.ts

@@ -37,14 +37,10 @@ export async function submitJob(model: string, input: Record<string, unknown>):
 export async function getJobStatus(model: string, requestId: string): Promise<JobStatus> {
   const status = await fal.queue.status(model, { requestId, logs: true });
 
-  // Validate the response structure before mapping
   if (!isValidFalQueueStatus(status)) {
-    // Fallback to default status if validation fails
-    return {
-      status: "IN_PROGRESS",
-      logs: [],
-      queuePosition: undefined,
-    };
+    throw new Error(
+      `Invalid FAL queue status response for model ${model}, requestId ${requestId}`
+    );
   }
 
   return mapFalStatusToJobStatus(status);
@@ -52,5 +48,12 @@ export async function getJobStatus(model: string, requestId: string): Promise<Jo
 
 export async function getJobResult<T = unknown>(model: string, requestId: string): Promise<T> {
   const result = await fal.queue.result(model, { requestId });
+
+  if (!result || typeof result !== 'object') {
+    throw new Error(
+      `Invalid FAL queue result for model ${model}, requestId ${requestId}`
+    );
+  }
+
   return result.data as T;
 }

package/src/infrastructure/services/fal-status-mapper.ts

@@ -24,13 +24,14 @@ export function mapFalStatusToJobStatus(status: FalQueueStatus): JobStatus {
 
   return {
     status: mappedStatus,
-    logs: status.logs?.map((log: FalLogEntry) => ({
-      message: log.message,
-      level: log.level ?? "info",
-      timestamp: log.timestamp ?? new Date().toISOString(),
-    })) ?? [],
+    logs: Array.isArray(status.logs)
+      ? status.logs.map((log: FalLogEntry) => ({
+          message: log.message,
+          level: log.level ?? "info",
+          timestamp: log.timestamp ?? new Date().toISOString(),
+        }))
+      : [],
     queuePosition: status.queuePosition ?? undefined,
-    // Preserve requestId from FalQueueStatus for use in hooks
     requestId: status.requestId,
   };
 }

package/src/infrastructure/services/request-store.ts

@@ -3,8 +3,6 @@
  * Survives hot reloads for React Native development
  */
 
-declare const __DEV__: boolean | undefined;
-
 export interface ActiveRequest<T = unknown> {
   promise: Promise<T>;
   abortController: AbortController;
@@ -13,9 +11,6 @@ export interface ActiveRequest<T = unknown> {
 const STORE_KEY = "__FAL_PROVIDER_REQUESTS__";
 type RequestStore = Map<string, ActiveRequest>;
 
-// Counter for generating unique request IDs
-let requestCounter = 0;
-
 export function getRequestStore(): RequestStore {
   if (!(globalThis as Record<string, unknown>)[STORE_KEY]) {
     (globalThis as Record<string, unknown>)[STORE_KEY] = new Map();
@@ -27,18 +22,20 @@ export function getRequestStore(): RequestStore {
  * Create a collision-resistant request key using combination of:
  * - Model name
  * - Input hash (for quick comparison)
- * - Unique counter (guarantees uniqueness)
+ * - Unique ID (guarantees uniqueness)
  */
 export function createRequestKey(model: string, input: Record<string, unknown>): string {
   const inputStr = JSON.stringify(input, Object.keys(input).sort());
-  // Use DJB2 hash for input fingerprinting (faster than crypto for dedup check)
+  // Use DJB2 hash for input fingerprinting
   let hash = 0;
   for (let i = 0; i < inputStr.length; i++) {
     const char = inputStr.charCodeAt(i);
     hash = ((hash << 5) - hash + char) | 0;
   }
-  // Add counter to guarantee uniqueness even with hash collisions
-  const uniqueId = `${requestCounter++}`;
+  // Use crypto.randomUUID() for guaranteed uniqueness without race conditions
+  const uniqueId = typeof crypto !== 'undefined' && crypto.randomUUID
+    ? crypto.randomUUID()
+    : `${Date.now()}-${Math.random().toString(36).slice(2)}`;
   return `${model}:${hash.toString(36)}:${uniqueId}`;
 }
 
@@ -56,10 +53,7 @@ export function removeRequest(key: string): void {
 
 export function cancelAllRequests(): void {
   const store = getRequestStore();
-  store.forEach((req, key) => {
-    if (typeof __DEV__ !== "undefined" && __DEV__) {
-      console.log(`[RequestStore] Cancelling: ${key}`);
-    }
+  store.forEach((req) => {
     req.abortController.abort();
   });
   store.clear();
@@ -81,7 +75,7 @@ export function cleanupRequestStore(_maxAge: number = 300000): void {
 
   // Requests are automatically removed when they complete (via finally block)
   // This function exists for future enhancements like time-based cleanup
-  if (store.size > 50 && typeof __DEV__ !== "undefined" && __DEV__) {
-    console.warn(`[RequestStore] Large request store size: ${store.size}. Consider investigating potential leaks.`);
+  if (store.size > 50) {
+    // Store size exceeds threshold - indicates potential memory leak
   }
 }

package/src/infrastructure/utils/cost-tracker.ts

@@ -10,8 +10,6 @@ import type {
 } from "../../domain/entities/cost-tracking.types";
 import { findModelById } from "../../domain/constants/default-models.constants";
 
-declare const __DEV__: boolean | undefined;
-
 interface CostSummary {
   totalEstimatedCost: number;
   totalActualCost: number;
@@ -69,14 +67,8 @@ export class CostTracker {
           currency: this.config.currency,
         };
       }
-
-      if (typeof __DEV__ !== "undefined" && __DEV__) {
-        console.warn("[CostTracker] No pricing found for model:", modelId);
-      }
     } catch (error) {
-      if (typeof __DEV__ !== "undefined" && __DEV__) {
-        console.warn("[CostTracker] Error finding model:", modelId, error);
-      }
+      // Silently return default cost info on error
     }
 
     return {
@@ -164,6 +156,7 @@ export class CostTracker {
   clearHistory(): void {
     this.costHistory = [];
     this.currentOperationCosts.clear();
+    this.operationCounter = 0;
   }
 
   getCostsByModel(modelId: string): GenerationCost[] {

package/src/infrastructure/utils/cost-tracking-executor.util.ts

@@ -5,8 +5,6 @@
 
 import type { CostTracker } from "./cost-tracker";
 
-declare const __DEV__: boolean | undefined;
-
 interface ExecuteWithCostTrackingOptions<T> {
   tracker: CostTracker | null;
   model: string;
@@ -37,9 +35,8 @@ export async function executeWithCostTracking<T>(
       const requestId = getRequestId?.(result);
       tracker.completeOperation(operationId, model, operation, requestId);
     } catch (costError) {
-      if (typeof __DEV__ !== "undefined" && __DEV__) {
-        console.warn("[CostTracking] Failed:", costError);
-      }
+      // Cost tracking failure shouldn't break the operation
+      // Log for debugging but don't throw
     }
 
     return result;
@@ -47,7 +44,7 @@ export async function executeWithCostTracking<T>(
     try {
       tracker.failOperation(operationId);
     } catch {
-      // Silently ignore cost tracking errors on failure path
+      // Cost tracking cleanup failure on error path - ignore
     }
     throw error;
   }

package/src/infrastructure/utils/error-mapper.ts

@@ -42,10 +42,3 @@ export function mapFalError(error: unknown): FalErrorInfo {
 export function isFalErrorRetryable(error: unknown): boolean {
   return categorizeFalError(error).retryable;
 }
-
-// Backward compatibility
-export const falErrorMapper = {
-  mapToErrorInfo: mapFalError,
-  isRetryable: isFalErrorRetryable,
-  getErrorType: (error: unknown) => categorizeFalError(error).type,
-};

package/src/infrastructure/utils/fal-storage.util.ts

@@ -7,12 +7,8 @@ import { fal } from "@fal-ai/client";
 import {
   base64ToTempFile,
   deleteTempFile,
-  getFileSize,
-  detectMimeType,
 } from "@umituz/react-native-design-system/filesystem";
 
-declare const __DEV__: boolean | undefined;
-
 /**
  * Upload base64 image to FAL storage
  * Uses design system's filesystem utilities for React Native compatibility
@@ -20,30 +16,22 @@ declare const __DEV__: boolean | undefined;
 export async function uploadToFalStorage(base64: string): Promise<string> {
   // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-assignment
   const tempUri = (await base64ToTempFile(base64));
-  // eslint-disable-next-line @typescript-eslint/no-unsafe-call
-  const fileSize = getFileSize(tempUri);
-  // eslint-disable-next-line @typescript-eslint/no-unsafe-call
-  const mimeType = detectMimeType(base64);
-
-  if (typeof __DEV__ !== "undefined" && __DEV__) {
-    console.log("[FalStorage] Uploading image", {
-      size: `${(fileSize / 1024).toFixed(1)}KB`,
-      type: mimeType,
-    });
-  }
 
-  const response = await fetch(tempUri);
-  const blob = await response.blob();
-  const url = await fal.storage.upload(blob);
-
-  // eslint-disable-next-line @typescript-eslint/no-unsafe-call
-  await deleteTempFile(tempUri);
-
-  if (typeof __DEV__ !== "undefined" && __DEV__) {
-    console.log("[FalStorage] Upload complete", { url: url.slice(0, 60) + "..." });
+  try {
+    const response = await fetch(tempUri);
+    const blob = await response.blob();
+    const url = await fal.storage.upload(blob);
+    return url;
+  } finally {
+    if (tempUri) {
+      try {
+        // eslint-disable-next-line @typescript-eslint/no-unsafe-call
+        await deleteTempFile(tempUri);
+      } catch {
+        // Silently ignore cleanup failures
+      }
+    }
   }
-
-  return url;
 }
 
 /**

package/src/infrastructure/utils/index.ts

@@ -4,7 +4,6 @@
 
 export { categorizeFalError } from "./error-categorizer";
 export {
-  falErrorMapper,
   mapFalError,
   isFalErrorRetryable,
 } from "./error-mapper";
@@ -12,15 +11,6 @@ export {
 export {
   buildSingleImageInput,
   buildDualImageInput,
-  buildUpscaleInput,
-  buildPhotoRestoreInput,
-  buildVideoFromImageInput,
-  buildFaceSwapInput,
-  buildImageToImageInput,
-  buildRemoveBackgroundInput,
-  buildRemoveObjectInput,
-  buildReplaceBackgroundInput,
-  buildHDTouchUpInput,
 } from "./input-builders.util";
 
 export {

package/src/infrastructure/utils/input-builders.util.ts

@@ -4,5 +4,3 @@
  */
 
 export * from "./base-builders.util";
-export * from "./image-feature-builders.util";
-export * from "./video-feature-builders.util";

package/src/infrastructure/utils/input-preprocessor.util.ts

@@ -5,8 +5,6 @@
 
 import { uploadToFalStorage } from "./fal-storage.util";
 
-declare const __DEV__: boolean | undefined;
-
 const IMAGE_URL_KEYS = [
   "image_url",
   "second_image_url",
@@ -37,23 +35,11 @@ export async function preprocessInput(
   for (const key of IMAGE_URL_KEYS) {
     const value = result[key];
     if (isBase64DataUri(value)) {
-      if (typeof __DEV__ !== "undefined" && __DEV__) {
-        console.log(`[FalPreprocessor] Uploading ${key} to storage...`);
-      }
-
       const uploadPromise = uploadToFalStorage(value)
         .then((url) => {
           result[key] = url;
-          if (typeof __DEV__ !== "undefined" && __DEV__) {
-            console.log(`[FalPreprocessor] ${key} uploaded`, {
-              url: url.slice(0, 50) + "...",
-            });
-          }
         })
         .catch((error) => {
-          if (typeof __DEV__ !== "undefined" && __DEV__) {
-            console.error(`[FalPreprocessor] Failed to upload ${key}:`, error);
-          }
           throw new Error(`Failed to upload ${key}: ${error instanceof Error ? error.message : "Unknown error"}`);
         });
 
@@ -62,52 +48,36 @@ export async function preprocessInput(
   }
 
   // Handle image_urls array (for multi-person generation)
-  if (Array.isArray(result.image_urls)) {
+  if (Array.isArray(result.image_urls) && result.image_urls.length > 0) {
     const imageUrls = result.image_urls as unknown[];
-    // Pre-initialize array with correct length to avoid sparse array
-    const processedUrls: string[] = new Array(imageUrls.length).fill("") as string[];
+    const processedUrls: string[] = [];
 
     for (let i = 0; i < imageUrls.length; i++) {
       const imageUrl = imageUrls[i];
       if (isBase64DataUri(imageUrl)) {
-        if (typeof __DEV__ !== "undefined" && __DEV__) {
-          console.log(`[FalPreprocessor] Uploading image_urls[${i}] to storage...`);
-        }
-
-        // Capture index in closure to ensure correct assignment
         const index = i;
         const uploadPromise = uploadToFalStorage(imageUrl)
           .then((url) => {
             processedUrls[index] = url;
-            if (typeof __DEV__ !== "undefined" && __DEV__) {
-              console.log(`[FalPreprocessor] image_urls[${index}] uploaded`, {
-                url: url.slice(0, 50) + "...",
-              });
-            }
           })
           .catch((error) => {
-            if (typeof __DEV__ !== "undefined" && __DEV__) {
-              console.error(`[FalPreprocessor] Failed to upload image_urls[${index}]:`, error);
-            }
             throw new Error(`Failed to upload image_urls[${index}]: ${error instanceof Error ? error.message : "Unknown error"}`);
           });
 
         uploadPromises.push(uploadPromise);
       } else if (typeof imageUrl === "string") {
         processedUrls[i] = imageUrl;
+      } else {
+        processedUrls[i] = "";
       }
     }
 
-    // Always set processed URLs after all uploads complete
     result.image_urls = processedUrls;
   }
 
   // Wait for ALL uploads to complete (both individual keys and array)
   if (uploadPromises.length > 0) {
     await Promise.all(uploadPromises);
-    if (typeof __DEV__ !== "undefined" && __DEV__) {
-      console.log(`[FalPreprocessor] All images uploaded (${uploadPromises.length})`);
-    }
   }
 
   return result;

package/src/infrastructure/utils/input-validator.util.ts

@@ -5,8 +5,6 @@
 
 import { isValidModelId, isValidPrompt } from "./type-guards.util";
 
-declare const __DEV__: boolean | undefined;
-
 export interface ValidationError {
   field: string;
   message: string;
@@ -75,18 +73,26 @@ export function validateInput(
 
   for (const field of imageFields) {
     const value = input[field];
-    if (value !== undefined && typeof value !== "string") {
-      errors.push({
-        field,
-        message: `${field} must be a string`,
-      });
+    if (value !== undefined) {
+      if (typeof value !== "string") {
+        errors.push({
+          field,
+          message: `${field} must be a string`,
+        });
+      } else if (value.length > 0) {
+        const isValidUrl = value.startsWith('http://') || value.startsWith('https://');
+        const isValidBase64 = value.startsWith('data:image/');
+        if (!isValidUrl && !isValidBase64) {
+          errors.push({
+            field,
+            message: `${field} must be a valid URL or base64 data URI`,
+          });
+        }
+      }
     }
   }
 
   if (errors.length > 0) {
-    if (typeof __DEV__ !== "undefined" && __DEV__) {
-      console.warn("[InputValidator] Validation errors:", errors);
-    }
     throw new InputValidationError(errors);
   }
 }

package/src/infrastructure/utils/type-guards.util.ts

@@ -92,9 +92,10 @@ export function isValidModelId(value: unknown): boolean {
     return false;
   }
 
-  // FAL model IDs typically follow the pattern: "owner/model-name" or "owner/model/version"
-  const modelIdPattern = /^[a-z0-9-]+\/[a-z0-9-]+(\/[a-z0-9.]+)?$/;
-  return modelIdPattern.test(value);
+  // FAL model IDs follow pattern: "owner/model-name" or "owner/model/version"
+  // Allow uppercase, dots, underscores, hyphens
+  const modelIdPattern = /^[a-zA-Z0-9-_]+\/[a-zA-Z0-9-_.]+(\/[a-zA-Z0-9-_.]+)?$/;
+  return modelIdPattern.test(value) && value.length >= 3;
 }
 
 /**

package/src/infrastructure/validators/nsfw-validator.ts

@@ -5,8 +5,6 @@
 
 import { NSFWContentError } from "../services/nsfw-content-error";
 
-declare const __DEV__: boolean | undefined;
-
 /**
  * Check if value indicates NSFW content
  */
@@ -33,9 +31,6 @@ export function validateNSFWContent(result: Record<string, unknown>): void {
   if (nsfwConcepts && Array.isArray(nsfwConcepts)) {
     const hasNSFW = nsfwConcepts.some((value) => value === true);
     if (hasNSFW) {
-      if (typeof __DEV__ !== "undefined" && __DEV__) {
-        console.log("[FalProvider] NSFW content detected (has_nsfw_concepts), rejecting result");
-      }
       throw new NSFWContentError();
     }
   }
@@ -43,27 +38,18 @@ export function validateNSFWContent(result: Record<string, unknown>): void {
   // Format 2: nsfw_detected boolean
   const nsfwDetected = result?.nsfw_detected;
   if (nsfwDetected !== undefined && isNSFWIndicator(nsfwDetected)) {
-    if (typeof __DEV__ !== "undefined" && __DEV__) {
-      console.log("[FalProvider] NSFW content detected (nsfw_detected), rejecting result");
-    }
     throw new NSFWContentError();
   }
 
   // Format 3: is_nsfw boolean
   const isNsfw = result?.is_nsfw;
   if (isNsfw !== undefined && isNSFWIndicator(isNsfw)) {
-    if (typeof __DEV__ !== "undefined" && __DEV__) {
-      console.log("[FalProvider] NSFW content detected (is_nsfw), rejecting result");
-    }
     throw new NSFWContentError();
   }
 
   // Format 4: nsfw_score number (> 0.5 threshold)
   const nsfwScore = result?.nsfw_score as number | undefined;
   if (typeof nsfwScore === "number" && nsfwScore > 0.5) {
-    if (typeof __DEV__ !== "undefined" && __DEV__) {
-      console.log("[FalProvider] NSFW content detected (nsfw_score: " + nsfwScore + "), rejecting result");
-    }
     throw new NSFWContentError();
   }
 
@@ -72,9 +58,6 @@ export function validateNSFWContent(result: Record<string, unknown>): void {
   if (policyViolation && typeof policyViolation === "object") {
     const type = (policyViolation.type || "").toLowerCase();
     if (type.includes("nsfw") || type.includes("adult") || type.includes("explicit")) {
-      if (typeof __DEV__ !== "undefined" && __DEV__) {
-        console.log("[FalProvider] Content policy violation detected:", policyViolation);
-      }
       throw new NSFWContentError();
     }
   }

package/src/init/createAiProviderInitModule.ts

@@ -5,8 +5,6 @@
 
 import { falProvider } from '../infrastructure/services';
 
-declare const __DEV__: boolean;
-
 /**
  * InitModule interface (from @umituz/react-native-design-system)
  */
@@ -24,18 +22,6 @@ export interface AiProviderInitModuleConfig {
    */
   getApiKey: () => string | undefined;
 
-  /**
-   * Video feature models mapping
-   * Maps feature types to FAL model IDs
-   */
-  videoFeatureModels?: Record<string, string>;
-
-  /**
-   * Image feature models mapping
-   * Maps feature types to FAL model IDs
-   */
-  imageFeatureModels?: Record<string, string>;
-
   /**
    * Whether this module is critical for app startup
    * @default false
@@ -63,10 +49,6 @@ export interface AiProviderInitModuleConfig {
  *     createFirebaseInitModule(),
  *     createAiProviderInitModule({
  *       getApiKey: () => getFalApiKey(),
- *       videoFeatureModels: {
- *         "image-to-video": "fal-ai/wan-25-preview/image-to-video",
- *         "text-to-video": "fal-ai/wan-25-preview/text-to-video",
- *       },
  *     }),
  *   ],
  * });
@@ -77,8 +59,6 @@ export function createAiProviderInitModule(
 ): InitModule {
   const {
     getApiKey,
-    videoFeatureModels,
-    imageFeatureModels,
     critical = false,
     dependsOn = ['firebase'],
   } = config;
@@ -92,30 +72,16 @@ export function createAiProviderInitModule(
         const apiKey = getApiKey();
 
         if (!apiKey) {
-          if (typeof __DEV__ !== 'undefined' && __DEV__) {
-            console.log('[createAiProviderInitModule] No API key - skipping');
-          }
-          return Promise.resolve(true); // Not an error, just skip
+          return Promise.resolve(false);
         }
 
-        // Initialize FAL provider
         falProvider.initialize({
           apiKey,
-          videoFeatureModels,
-          imageFeatureModels,
         });
 
-        if (typeof __DEV__ !== 'undefined' && __DEV__) {
-          console.log('[createAiProviderInitModule] FAL provider initialized');
-        }
-
         return Promise.resolve(true);
       } catch (error) {
-        if (typeof __DEV__ !== 'undefined' && __DEV__) {
-          console.error('[createAiProviderInitModule] Error:', error);
-        }
-        // Continue on error - AI provider is not critical
-        return Promise.resolve(true);
+        return Promise.resolve(false);
       }
     },
   };

package/src/presentation/hooks/use-fal-generation.ts

@@ -52,11 +52,9 @@ export function useFalGeneration<T = unknown>(
         const result = await falProvider.subscribe<T>(modelEndpoint, input, {
           timeoutMs: options?.timeoutMs,
           onQueueUpdate: (status) => {
-            // Update requestId ref when we receive it from status
             if (status.requestId) {
               currentRequestIdRef.current = status.requestId;
             }
-            // Map JobStatus to FalQueueStatus for backward compatibility
             options?.onProgress?.({
               status: status.status,
               requestId: status.requestId ?? currentRequestIdRef.current ?? "",
@@ -95,9 +93,6 @@ export function useFalGeneration<T = unknown>(
     if (falProvider.hasRunningRequest()) {
       setIsCancelling(true);
       falProvider.cancelCurrentRequest();
-      if (typeof __DEV__ !== "undefined" && __DEV__) {
-        console.log("[useFalGeneration] Request cancelled");
-      }
     }
   }, []);
 

package/src/presentation/hooks/use-models.ts

@@ -19,7 +19,7 @@ import type {
   UseModelsReturn,
 } from "../../domain/types/model-selection.types";
 
-declare const __DEV__: boolean;
+export type { UseModelsReturn } from "../../domain/types/model-selection.types";
 
 export interface UseModelsProps {
   /** Model type to fetch */
@@ -56,11 +56,6 @@ export function useModels(props: UseModelsProps): UseModelsReturn {
       setSelectedModel(initial);
     }
 
-    if (typeof __DEV__ !== "undefined" && __DEV__) {
-      // eslint-disable-next-line no-console
-      console.log(`[useModels] Loaded ${fetchedModels.length} ${type} models`);
-    }
-
     setIsLoading(false);
   }, [type, config?.initialModelId, defaultModelId]);
 
@@ -73,11 +68,6 @@ export function useModels(props: UseModelsProps): UseModelsReturn {
       const model = models.find((m) => m.id === modelId);
       if (model) {
         setSelectedModel(model);
-
-        if (typeof __DEV__ !== "undefined" && __DEV__) {
-          // eslint-disable-next-line no-console
-          console.log(`[useModels] Selected: ${model.name} (${model.id})`);
-        }
       }
     },
     [models],

package/src/infrastructure/builders/image-feature-builder.ts

@@ -1,64 +0,0 @@
-/**
- * Image Feature Input Builder
- * Builds inputs for image-based AI features
- */
-
-import type {
-  ImageFeatureType,
-  ImageFeatureInputData,
-} from "../../domain/types";
-import { buildSingleImageInput } from "../utils/base-builders.util";
-import {
-  buildUpscaleInput,
-  buildPhotoRestoreInput,
-  buildFaceSwapInput,
-  buildRemoveBackgroundInput,
-  buildRemoveObjectInput,
-  buildReplaceBackgroundInput,
-  buildKontextStyleTransferInput,
-} from "../utils/image-feature-builders.util";
-
-const DEFAULT_ANIME_SELFIE_PROMPT = "Transform this person into anime style illustration. Keep the same gender, face structure, hair color, eye color, and expression. Make it look like a high-quality anime character portrait with vibrant colors and clean lineart.";
-
-export function buildImageFeatureInput(
-  feature: ImageFeatureType,
-  data: ImageFeatureInputData,
-): Record<string, unknown> {
-  const { imageBase64, targetImageBase64, prompt, options } = data;
-
-  switch (feature) {
-    case "upscale":
-    case "hd-touch-up":
-      return buildUpscaleInput(imageBase64, options);
-
-    case "photo-restore":
-      return buildPhotoRestoreInput(imageBase64, options);
-
-    case "face-swap":
-      if (!targetImageBase64) {
-        throw new Error("Face swap requires target image");
-      }
-      return buildFaceSwapInput(imageBase64, targetImageBase64, options);
-
-    case "remove-background":
-      return buildRemoveBackgroundInput(imageBase64, options);
-
-    case "remove-object":
-      return buildRemoveObjectInput(imageBase64, { prompt, ...options });
-
-    case "replace-background":
-      if (!prompt) {
-        throw new Error("Replace background requires prompt");
-      }
-      return buildReplaceBackgroundInput(imageBase64, { prompt, ...options });
-
-    case "anime-selfie":
-      return buildKontextStyleTransferInput(imageBase64, {
-        prompt: prompt || (options?.prompt as string) || DEFAULT_ANIME_SELFIE_PROMPT,
-        guidance_scale: options?.guidance_scale as number | undefined,
-      });
-
-    default:
-      return buildSingleImageInput(imageBase64, options);
-  }
-}

package/src/infrastructure/builders/video-feature-builder.ts

@@ -1,52 +0,0 @@
-/**
- * Video Feature Input Builder
- * Builds inputs for video-based AI features
- */
-
-import type {
-  VideoFeatureType,
-  VideoFeatureInputData,
-} from "../../domain/types";
-import {
-  buildVideoFromImageInput,
-  buildTextToVideoInput,
-} from "../utils/video-feature-builders.util";
-
-const DEFAULT_VIDEO_PROMPTS: Partial<Record<VideoFeatureType, string>> = {
-  "image-to-video": "Animate this image with natural, smooth motion while preserving all details",
-  "text-to-video": "Generate a high-quality video based on the description, smooth motion",
-} as const;
-
-/**
- * Features that require image input
- */
-const IMAGE_REQUIRED_FEATURES: readonly VideoFeatureType[] = [
-  "image-to-video",
-] as const;
-
-function isImageRequiredFeature(feature: VideoFeatureType): boolean {
-  return IMAGE_REQUIRED_FEATURES.includes(feature);
-}
-
-export function buildVideoFeatureInput(
-  feature: VideoFeatureType,
-  data: VideoFeatureInputData,
-): Record<string, unknown> {
-  const { sourceImageBase64, prompt, options } = data;
-  const effectivePrompt = prompt || DEFAULT_VIDEO_PROMPTS[feature] || "Generate video";
-
-  if (isImageRequiredFeature(feature)) {
-    return buildVideoFromImageInput(sourceImageBase64 || "", {
-      prompt: effectivePrompt,
-      duration: options?.duration as number | undefined,
-      resolution: options?.resolution as string | undefined,
-    });
-  }
-
-  return buildTextToVideoInput({
-    prompt: effectivePrompt,
-    duration: options?.duration as number | undefined,
-    aspectRatio: options?.aspect_ratio as string | undefined,
-    resolution: options?.resolution as string | undefined,
-  });
-}

package/src/infrastructure/services/fal-feature-models.ts

@@ -1,48 +0,0 @@
-/**
- * FAL Feature Models - Model resolution and input building
- */
-
-import type {
-  ImageFeatureType,
-  VideoFeatureType,
-  ImageFeatureInputData,
-  VideoFeatureInputData,
-} from "../../domain/types";
-import {
-  buildImageFeatureInput as buildImageFeatureInputImpl,
-} from "../builders/image-feature-builder";
-import {
-  buildVideoFeatureInput as buildVideoFeatureInputImpl,
-} from "../builders/video-feature-builder";
-
-export function getImageFeatureModel(
-  imageFeatureModels: Record<string, string>,
-  feature: ImageFeatureType,
-): string {
-  const model = imageFeatureModels[feature];
-  if (!model) throw new Error(`No model for image feature: ${feature}`);
-  return model;
-}
-
-export function getVideoFeatureModel(
-  videoFeatureModels: Record<string, string>,
-  feature: VideoFeatureType,
-): string {
-  const model = videoFeatureModels[feature];
-  if (!model) throw new Error(`No model for video feature: ${feature}`);
-  return model;
-}
-
-export function buildImageFeatureInput(
-  feature: ImageFeatureType,
-  data: ImageFeatureInputData,
-): Record<string, unknown> {
-  return buildImageFeatureInputImpl(feature, data);
-}
-
-export function buildVideoFeatureInput(
-  feature: VideoFeatureType,
-  data: VideoFeatureInputData,
-): Record<string, unknown> {
-  return buildVideoFeatureInputImpl(feature, data);
-}

package/src/infrastructure/utils/image-feature-builders.util.ts

@@ -1,103 +0,0 @@
-/**
- * Image Feature Input Builders
- * Builder functions for specific image features
- */
-
-import type {
-  UpscaleOptions,
-  PhotoRestoreOptions,
-  RemoveBackgroundOptions,
-  RemoveObjectOptions,
-  ReplaceBackgroundOptions,
-  FaceSwapOptions,
-} from "../../domain/types";
-import { buildSingleImageInput } from "./base-builders.util";
-import { formatImageDataUri } from "./image-helpers.util";
-
-export function buildUpscaleInput(
-  base64: string,
-  options?: UpscaleOptions,
-): Record<string, unknown> {
-  return buildSingleImageInput(base64, {
-    scale: options?.scaleFactor ?? 2,
-    face_enhance: options?.enhanceFaces ?? false,
-  });
-}
-
-export function buildPhotoRestoreInput(
-  base64: string,
-  options?: PhotoRestoreOptions,
-): Record<string, unknown> {
-  return buildSingleImageInput(base64, {
-    face_enhance: options?.enhanceFaces ?? true,
-  });
-}
-
-export function buildFaceSwapInput(
-  sourceBase64: string,
-  targetBase64: string,
-  _options?: FaceSwapOptions,
-): Record<string, unknown> {
-  return {
-    base_image_url: formatImageDataUri(sourceBase64),
-    swap_image_url: formatImageDataUri(targetBase64),
-  };
-}
-
-export function buildRemoveBackgroundInput(
-  base64: string,
-  options?: RemoveBackgroundOptions & {
-    model?: string;
-    operating_resolution?: string;
-    output_format?: string;
-    refine_foreground?: boolean;
-  },
-): Record<string, unknown> {
-  return buildSingleImageInput(base64, {
-    model: options?.model ?? "General Use (Light)",
-    operating_resolution: options?.operating_resolution ?? "1024x1024",
-    output_format: options?.output_format ?? "png",
-    refine_foreground: options?.refine_foreground ?? true,
-  });
-}
-
-export function buildRemoveObjectInput(
-  base64: string,
-  options?: RemoveObjectOptions,
-): Record<string, unknown> {
-  return buildSingleImageInput(base64, {
-    mask_url: options?.mask,
-    prompt: options?.prompt || "Remove the object and fill with background",
-  });
-}
-
-export function buildReplaceBackgroundInput(
-  base64: string,
-  options: ReplaceBackgroundOptions,
-): Record<string, unknown> {
-  return buildSingleImageInput(base64, {
-    prompt: options.prompt,
-  });
-}
-
-export function buildHDTouchUpInput(
-  base64: string,
-  options?: UpscaleOptions,
-): Record<string, unknown> {
-  return buildUpscaleInput(base64, options);
-}
-
-export interface KontextStyleTransferOptions {
-  prompt: string;
-  guidance_scale?: number;
-}
-
-export function buildKontextStyleTransferInput(
-  base64: string,
-  options: KontextStyleTransferOptions,
-): Record<string, unknown> {
-  return buildSingleImageInput(base64, {
-    prompt: options.prompt,
-    guidance_scale: options.guidance_scale ?? 3.5,
-  });
-}

package/src/infrastructure/utils/video-feature-builders.util.ts

@@ -1,58 +0,0 @@
-/**
- * Video Feature Input Builders
- * Builder functions for video features
- */
-
-import type {
-  ImageToImagePromptConfig,
-  VideoFromImageOptions,
-  TextToVideoOptions,
-} from "../../domain/types";
-import { buildSingleImageInput } from "./base-builders.util";
-import { formatImageDataUri } from "./image-helpers.util";
-
-export function buildImageToImageInput(
-  base64: string,
-  promptConfig: ImageToImagePromptConfig,
-): Record<string, unknown> {
-  return buildSingleImageInput(base64, {
-    prompt: promptConfig.prompt,
-    negative_prompt: promptConfig.negativePrompt,
-    strength: promptConfig.strength ?? 0.85,
-    num_inference_steps: promptConfig.num_inference_steps ?? 50,
-    guidance_scale: promptConfig.guidance_scale ?? 7.5,
-  });
-}
-
-export function buildVideoFromImageInput(
-  base64: string,
-  options?: VideoFromImageOptions & {
-    enable_safety_checker?: boolean;
-    default_prompt?: string;
-  },
-): Record<string, unknown> {
-  return {
-    prompt: options?.prompt || options?.default_prompt || "Generate natural motion video",
-    image_url: formatImageDataUri(base64),
-    enable_safety_checker: options?.enable_safety_checker ?? false,
-    ...(options?.duration && { duration: options.duration }),
-    ...(options?.resolution && { resolution: options.resolution }),
-  };
-}
-
-/**
- * Build input for text-to-video generation (no image required)
- */
-export function buildTextToVideoInput(
-  options: TextToVideoOptions,
-): Record<string, unknown> {
-  const { prompt, duration, aspectRatio, resolution } = options;
-
-  return {
-    prompt,
-    enable_safety_checker: false,
-    ...(duration && { duration }),
-    ...(aspectRatio && { aspect_ratio: aspectRatio }),
-    ...(resolution && { resolution }),
-  };
-}