@umituz/react-native-ai-fal-provider 2.1.3 → 2.1.5

package/src/infrastructure/utils/parsers/json-parsers.util.ts

@@ -12,7 +12,12 @@ export function safeJsonParse<T = unknown>(
 ): T {
   try {
     return JSON.parse(data) as T;
-  } catch {
+  } catch (error) {
+    console.warn(
+      '[json-parsers] Failed to parse JSON, using fallback:',
+      error instanceof Error ? error.message : String(error),
+      { dataPreview: data.substring(0, 100) }
+    );
     return fallback;
   }
 }
@@ -23,7 +28,12 @@ export function safeJsonParse<T = unknown>(
 export function safeJsonParseOrNull<T = unknown>(data: string): T | null {
   try {
     return JSON.parse(data) as T;
-  } catch {
+  } catch (error) {
+    console.warn(
+      '[json-parsers] Failed to parse JSON, returning null:',
+      error instanceof Error ? error.message : String(error),
+      { dataPreview: data.substring(0, 100) }
+    );
     return null;
   }
 }
@@ -37,7 +47,12 @@ export function safeJsonStringify(
 ): string {
   try {
     return JSON.stringify(data);
-  } catch {
+  } catch (error) {
+    console.warn(
+      '[json-parsers] Failed to stringify object, using fallback:',
+      error instanceof Error ? error.message : String(error),
+      { dataType: typeof data }
+    );
     return fallback;
   }
 }
@@ -50,6 +65,7 @@ export function isValidJson(data: string): boolean {
     JSON.parse(data);
     return true;
   } catch {
+    // Don't log here - this is expected to fail for validation checks
     return false;
   }
 }

package/src/infrastructure/utils/string-format.util.ts

@@ -12,62 +12,3 @@ export function truncateText(text: string, maxLength: number): string {
   }
   return text.slice(0, maxLength - 3) + "...";
 }
-
-/**
- * Capitalize first letter of string
- */
-export function capitalize(text: string): string {
-  if (!text) return text;
-  return text.charAt(0).toUpperCase() + text.slice(1);
-}
-
-/**
- * Convert string to title case
- */
-export function toTitleCase(text: string): string {
-  return text
-    .toLowerCase()
-    .split(" ")
-    .map((word) => capitalize(word))
-    .join(" ");
-}
-
-/**
- * Convert string to slug
- */
-export function toSlug(text: string): string {
-  return text
-    .toLowerCase()
-    .trim()
-    .replace(/[^\w\s-]/g, "")
-    .replace(/[\s_-]+/g, "-")
-    .replace(/^-+|-+$/g, "");
-}
-
-/**
- * Format list of items with conjunction
- */
-export function formatList(items: readonly string[], conjunction: string = "and"): string {
-  if (items.length === 0) return "";
-  if (items.length === 1) return items[0] ?? "";
-  if (items.length === 2) return items.join(` ${conjunction} `);
-
-  const allButLast = items.slice(0, -1);
-  const last = items[items.length - 1];
-  return `${allButLast.join(", ")}, ${conjunction} ${last}`;
-}
-
-/**
- * Pluralize word based on count
- */
-export function pluralize(word: string, count: number): string {
-  if (count === 1) return word;
-  return `${word}s`;
-}
-
-/**
- * Format count with plural word
- */
-export function formatCount(word: string, count: number): string {
-  return `${count} ${pluralize(word, count)}`;
-}

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

@@ -40,6 +40,8 @@ export function isValidApiKey(value: unknown): boolean {
 
 /**
  * Validate model ID format
+ * Pattern: org/model or org/model/version
+ * Allows dots for versions (e.g., v1.5) but prevents path traversal (..)
  */
 const MODEL_ID_PATTERN = /^[a-zA-Z0-9-_]+\/[a-zA-Z0-9-_.]+(\/[a-zA-Z0-9-_.]+)?$/;
 
@@ -48,6 +50,16 @@ export function isValidModelId(value: unknown): boolean {
     return false;
   }
 
+  // Prevent path traversal attacks
+  if (value.includes('..')) {
+    return false;
+  }
+
+  // Ensure it doesn't start or end with dots
+  if (value.startsWith('.') || value.endsWith('.')) {
+    return false;
+  }
+
   return MODEL_ID_PATTERN.test(value) && value.length >= MIN_MODEL_ID_LENGTH;
 }
 

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

@@ -35,13 +35,21 @@ export function useModels(props: UseModelsProps): UseModelsReturn {
   const [isLoading, setIsLoading] = useState(true);
   const [error, setError] = useState<string | null>(null);
 
+  // Memoize config to prevent unnecessary re-renders when parent re-renders
+  // Only recreate when actual config values change
+  const memoizedConfig = useMemo(() => config, [
+    config?.initialModelId,
+    config?.defaultCreditCost,
+    config?.defaultModelId,
+  ]);
+
   // Direct effect - no intermediate callback needed
   useEffect(() => {
     setIsLoading(true);
     setError(null);
 
     try {
-      const selectionData = falModelsService.getModelSelectionData(type, config);
+      const selectionData = falModelsService.getModelSelectionData(type, memoizedConfig);
       setModels(selectionData.models);
       setSelectedModel(selectionData.selectedModel);
     } catch (err) {
@@ -49,7 +57,7 @@ export function useModels(props: UseModelsProps): UseModelsReturn {
     } finally {
       setIsLoading(false);
     }
-  }, [type, config]); // Direct dependencies
+  }, [type, memoizedConfig]);
 
   // Separate refresh callback for manual reloads
   const loadModels = useCallback(() => {
@@ -57,7 +65,7 @@ export function useModels(props: UseModelsProps): UseModelsReturn {
     setError(null);
 
     try {
-      const selectionData = falModelsService.getModelSelectionData(type, config);
+      const selectionData = falModelsService.getModelSelectionData(type, memoizedConfig);
       setModels(selectionData.models);
       setSelectedModel(selectionData.selectedModel);
     } catch (err) {
@@ -65,7 +73,7 @@ export function useModels(props: UseModelsProps): UseModelsReturn {
     } finally {
       setIsLoading(false);
     }
-  }, [type, config]);
+  }, [type, memoizedConfig]);
 
   const selectModel = useCallback(
     (modelId: string) => {

package/src/infrastructure/utils/collections/array-reducers.util.ts

@@ -1,67 +0,0 @@
-/**
- * Array Reducer Utilities
- * Reduce, group, chunk, and aggregation operations
- */
-
-/**
- * Reduce array to sum of number property
- */
-export function sumByProperty<T>(
-  items: readonly T[],
-  numberProperty: keyof T
-): number {
-  return items.reduce((sum, item) => {
-    const value = item[numberProperty] as unknown as number;
-    return sum + (typeof value === "number" ? value : 0);
-  }, 0);
-}
-
-/**
- * Group array by property value
- */
-export function groupByProperty<T>(
-  items: readonly T[],
-  property: keyof T
-): Map<unknown, T[]> {
-  const groups = new Map<unknown, T[]>();
-  for (const item of items) {
-    const key = item[property];
-    const existing = groups.get(key);
-    if (existing) {
-      existing.push(item);
-    } else {
-      groups.set(key, [item]);
-    }
-  }
-  return groups;
-}
-
-/**
- * Chunk array into smaller arrays of specified size
- */
-export function chunkArray<T>(items: readonly T[], chunkSize: number): T[][] {
-  const result: T[][] = [];
-  for (let i = 0; i < items.length; i += chunkSize) {
-    result.push([...items.slice(i, i + chunkSize)]);
-  }
-  return result;
-}
-
-/**
- * Get distinct values of a property from array
- */
-export function distinctByProperty<T>(
-  items: readonly T[],
-  property: keyof T
-): unknown[] {
-  const seen = new Set<unknown>();
-  const result: unknown[] = [];
-  for (const item of items) {
-    const value = item[property];
-    if (!seen.has(value)) {
-      seen.add(value);
-      result.push(value);
-    }
-  }
-  return result;
-}

package/src/infrastructure/utils/helpers/function-helpers.util.ts

@@ -1,25 +0,0 @@
-/**
- * Function Helper Utilities
- * Common functional programming helpers
- */
-
-/**
- * No-op function
- */
-export function noop(): void {
-  // Intentionally empty
-}
-
-/**
- * Identity function
- */
-export function identity<T>(value: T): T {
-  return value;
-}
-
-/**
- * Constant function (returns same value regardless of input)
- */
-export function constant<T>(value: T): () => T {
-  return () => value;
-}

package/src/infrastructure/utils/parsers/number-helpers.util.ts

@@ -1,19 +0,0 @@
-/**
- * Number Helper Utilities
- * Number manipulation and formatting
- */
-
-/**
- * Clamp number between min and max
- */
-export function clampNumber(value: number, min: number, max: number): number {
-  return Math.min(Math.max(value, min), max);
-}
-
-/**
- * Round to decimal places
- */
-export function roundToDecimals(value: number, decimals: number): number {
-  const multiplier = Math.pow(10, decimals);
-  return Math.round(value * multiplier) / multiplier;
-}