expo-tvos-search 1.2.2 → 1.3.0

package/build/index.js

@@ -1,9 +1,9 @@
 import React from "react";
 import { Platform } from "react-native";
-// Safely try to load the native view - it may not be available if:
-// 1. Running on a non-tvOS platform
-// 2. Native module hasn't been built yet (needs expo prebuild)
-// 3. expo-modules-core isn't properly installed
+/**
+ * Native view component loaded at module initialization.
+ * Returns null on non-tvOS platforms or when the native module is unavailable.
+ */
 let NativeView = null;
 if (Platform.OS === "ios" && Platform.isTV) {
     try {
@@ -11,9 +11,38 @@ if (Platform.OS === "ios" && Platform.isTV) {
         if (typeof requireNativeViewManager === "function") {
             NativeView = requireNativeViewManager("ExpoTvosSearch");
         }
+        else {
+            console.warn("[expo-tvos-search] requireNativeViewManager is not a function. " +
+                "This usually indicates an incompatible expo-modules-core version. " +
+                "Try reinstalling expo-modules-core or updating to a compatible version.");
+        }
     }
-    catch {
-        // Native module not available - will fall back to React Native implementation
+    catch (error) {
+        // Categorize the error to help with debugging
+        const errorMessage = error instanceof Error ? error.message : String(error);
+        if (errorMessage.includes("expo-modules-core")) {
+            console.warn("[expo-tvos-search] Failed to load expo-modules-core. " +
+                "Make sure expo-modules-core is installed: npm install expo-modules-core\n" +
+                `Error: ${errorMessage}`);
+        }
+        else if (errorMessage.includes("ExpoTvosSearch")) {
+            console.warn("[expo-tvos-search] Native module ExpoTvosSearch not found. " +
+                "This usually means:\n" +
+                "1. You haven't run 'expo prebuild' yet, or\n" +
+                "2. The native project needs to be rebuilt (try 'expo prebuild --clean')\n" +
+                "3. You're not running on a tvOS simulator/device\n" +
+                `Error: ${errorMessage}`);
+        }
+        else {
+            // Unexpected error - log full details for debugging
+            console.warn("[expo-tvos-search] Unexpected error loading native module.\n" +
+                `Error: ${errorMessage}\n` +
+                "Please report this issue at: https://github.com/keiver/expo-tvos-search/issues");
+            // In development, log the full error for debugging
+            if (typeof __DEV__ !== "undefined" && __DEV__) {
+                console.error("[expo-tvos-search] Full error details:", error);
+            }
+        }
     }
 }
 /**
@@ -51,6 +80,25 @@ if (Platform.OS === "ios" && Platform.isTV) {
  */
 export function TvosSearchView(props) {
     if (!NativeView) {
+        // Warn in development when native module is unavailable
+        if (typeof __DEV__ !== "undefined" && __DEV__) {
+            const isRunningOnTvOS = Platform.OS === "ios" && Platform.isTV;
+            if (isRunningOnTvOS) {
+                // On tvOS but module failed to load - this is unexpected
+                console.warn("[expo-tvos-search] TvosSearchView is rendering null on tvOS. " +
+                    "This usually means:\n" +
+                    "1. The native module wasn't built properly (try 'expo prebuild --clean')\n" +
+                    "2. expo-modules-core is missing or incompatible\n" +
+                    "3. The app needs to be restarted after installing the module\n\n" +
+                    "Check the earlier console logs for specific error details.");
+            }
+            else {
+                // Not on tvOS - expected behavior, but developer might want to know
+                console.info("[expo-tvos-search] TvosSearchView is not available on " +
+                    `${Platform.OS}${Platform.isTV ? " (TV)" : ""}. ` +
+                    "Use isNativeSearchAvailable() to check before rendering this component.");
+            }
+        }
         return null;
     }
     return React.createElement(NativeView, { ...props });

package/ios/ExpoTvosSearchModule.swift

@@ -12,17 +12,32 @@ public class ExpoTvosSearchModule: Module {
         Name("ExpoTvosSearch")
 
         View(ExpoTvosSearchView.self) {
-            Events("onSearch", "onSelectItem")
+            Events("onSearch", "onSelectItem", "onError", "onValidationWarning")
 
             Prop("results") { (view: ExpoTvosSearchView, results: [[String: Any]]) in
                 // Limit results array size to prevent memory issues
                 let limitedResults = Array(results.prefix(Self.maxResults))
+                if results.count > Self.maxResults {
+                    view.onValidationWarning([
+                        "type": "value_clamped",
+                        "message": "Results array truncated from \(results.count) to \(Self.maxResults) items",
+                        "context": "maxResults=\(Self.maxResults)"
+                    ])
+                }
                 view.updateResults(limitedResults)
             }
 
             Prop("columns") { (view: ExpoTvosSearchView, columns: Int) in
                 // Clamp columns between min and max for safe grid layout
-                view.columns = min(max(Self.minColumns, columns), Self.maxColumns)
+                let clampedValue = min(max(Self.minColumns, columns), Self.maxColumns)
+                if clampedValue != columns {
+                    view.onValidationWarning([
+                        "type": "value_clamped",
+                        "message": "columns value \(columns) was clamped to range [\(Self.minColumns), \(Self.maxColumns)]",
+                        "context": "columns=\(clampedValue)"
+                    ])
+                }
+                view.columns = clampedValue
             }
 
             Prop("placeholder") { (view: ExpoTvosSearchView, placeholder: String) in
@@ -48,7 +63,15 @@ public class ExpoTvosSearchModule: Module {
 
             Prop("topInset") { (view: ExpoTvosSearchView, topInset: Double) in
                 // Clamp to non-negative values (max 500 points reasonable for any screen)
-                view.topInset = CGFloat(min(max(0, topInset), 500))
+                let clampedValue = min(max(0, topInset), 500)
+                if clampedValue != topInset {
+                    view.onValidationWarning([
+                        "type": "value_clamped",
+                        "message": "topInset value \(topInset) was clamped to range [0, 500]",
+                        "context": "topInset=\(clampedValue)"
+                    ])
+                }
+                view.topInset = CGFloat(clampedValue)
             }
 
             Prop("showTitleOverlay") { (view: ExpoTvosSearchView, show: Bool) in
@@ -61,7 +84,15 @@ public class ExpoTvosSearchModule: Module {
 
             Prop("marqueeDelay") { (view: ExpoTvosSearchView, delay: Double) in
                 // Clamp between 0 and maxMarqueeDelay seconds
-                view.marqueeDelay = min(max(0, delay), Self.maxMarqueeDelay)
+                let clampedValue = min(max(0, delay), Self.maxMarqueeDelay)
+                if clampedValue != delay {
+                    view.onValidationWarning([
+                        "type": "value_clamped",
+                        "message": "marqueeDelay value \(delay) was clamped to range [0, \(Self.maxMarqueeDelay)]",
+                        "context": "marqueeDelay=\(clampedValue)"
+                    ])
+                }
+                view.marqueeDelay = clampedValue
             }
 
             Prop("emptyStateText") { (view: ExpoTvosSearchView, text: String) in
@@ -79,6 +110,38 @@ public class ExpoTvosSearchModule: Module {
             Prop("noResultsHintText") { (view: ExpoTvosSearchView, text: String) in
                 view.noResultsHintText = String(text.prefix(Self.maxStringLength))
             }
+
+            Prop("textColor") { (view: ExpoTvosSearchView, colorHex: String?) in
+                view.textColor = colorHex
+            }
+
+            Prop("accentColor") { (view: ExpoTvosSearchView, colorHex: String) in
+                view.accentColor = colorHex
+            }
+
+            Prop("cardWidth") { (view: ExpoTvosSearchView, width: Double) in
+                view.cardWidth = CGFloat(max(50, min(1000, width)))  // Clamp to reasonable range
+            }
+
+            Prop("cardHeight") { (view: ExpoTvosSearchView, height: Double) in
+                view.cardHeight = CGFloat(max(50, min(1000, height)))  // Clamp to reasonable range
+            }
+
+            Prop("imageContentMode") { (view: ExpoTvosSearchView, mode: String) in
+                view.imageContentMode = mode
+            }
+
+            Prop("cardMargin") { (view: ExpoTvosSearchView, margin: Double) in
+                view.cardMargin = CGFloat(max(0, min(200, margin)))  // Clamp to reasonable range
+            }
+
+            Prop("cardPadding") { (view: ExpoTvosSearchView, padding: Double) in
+                view.cardPadding = CGFloat(max(0, min(100, padding)))  // Clamp to reasonable range
+            }
+
+            Prop("overlayTitleSize") { (view: ExpoTvosSearchView, size: Double) in
+                view.overlayTitleSize = CGFloat(max(8, min(72, size)))  // Clamp to reasonable font size range
+            }
         }
     }
 }