@robylon/react-native-sdk 2.0.32-staging.4 → 2.1.1-staging.3

package/src/Chatbotsdk.tsx

@@ -87,30 +87,9 @@ interface ChatbotMessage {
   level?: string;
   url?: string;
   filename?: string;
+  inPlace?: boolean;
 }
 
-const getOriginFromUrl = (url?: string): string | null => {
-  if (!url) return null;
-  try {
-    return new URL(url)?.origin ?? null;
-  } catch {
-    return null;
-  }
-};
-
-const warmupHttpOrigin = async (origin?: string | null) => {
-  if (!origin) return;
-  try {
-    await fetch(origin, { method: "HEAD" });
-  } catch {
-    try {
-      await fetch(origin, { method: "GET" });
-    } catch {
-      // Network warmup should be best-effort and non-blocking.
-    }
-  }
-};
-
 export enum ChatbotInterfaceType {
   WIDGET = "WIDGET",
   POPOVER = "POPOVER",
@@ -173,11 +152,11 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
       onReady,
       user_profile,
     },
-    ref
+    ref,
   ) => {
     const [isWebViewVisible, setIsWebViewVisible] = useState(false);
     const [chatbotConfig, setChatbotConfig] = useState<ChatbotConfig | null>(
-      null
+      null,
     );
     const [loading, setLoading] = useState(true);
     const [isInitialized, setIsInitialized] = useState(false);
@@ -206,7 +185,7 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
         isAnonymous: !user_id && user_id !== 0,
         // Add other config properties as needed
       }),
-      [effectiveUserId, user_id]
+      [effectiveUserId, user_id],
     );
 
     // Initialize event handling
@@ -243,7 +222,7 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
         },
         isReady: () => isInitialized,
       }),
-      [isInitialized, isWebViewVisible]
+      [isInitialized, isWebViewVisible],
     );
 
     const triggerAppInit = () => {
@@ -265,7 +244,7 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
           pendingStorageOps.current.push(operation);
         }
       },
-      [isStorageReady]
+      [isStorageReady],
     );
 
     // Handle WebView messages
@@ -278,7 +257,7 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
           // Handle storage messages
           if (
             Object.values(StorageMessageType).includes(
-              parsedData.type as StorageMessageType
+              parsedData.type as StorageMessageType,
             )
           ) {
             const storageMessage = parsedData as StorageMessage;
@@ -291,8 +270,8 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
                 webViewRef.current?.injectJavaScript(
                   createStorageMessage(
                     StorageMessageType.GET_STORAGE,
-                    "rblyn_anon"
-                  )
+                    "rblyn_anon",
+                  ),
                 );
                 break;
               case StorageMessageType.STORAGE_RESPONSE:
@@ -375,6 +354,7 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
                 const downloadData: DownloadRequest = parsedData?.data || {
                   url: parsedData?.url,
                   filename: parsedData?.filename,
+                  inPlace: parsedData?.inPlace,
                 };
                 handleDownloadRequest(downloadData);
               }
@@ -391,7 +371,7 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
             case "CHAT_INITIALIZATION_FAILED":
               emitEvent(
                 ChatbotEventType.CHAT_INITIALIZATION_FAILED,
-                parsedData?.data
+                parsedData?.data,
               );
               break;
             case "REQUEST_MIC_PERMISSION":
@@ -404,7 +384,6 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
                     ? "MIC_PERMISSION_GRANTED"
                     : "MIC_PERMISSION_DENIED",
                 };
-                logger.debug("messageData====>", JSON.stringify(messageData));
                 webViewRef.current?.injectJavaScript(`
                   window.postMessage(${JSON.stringify(messageData)}, '*');`);
               }
@@ -426,7 +405,7 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
             {
               type: ErrorTypes.RUNTIME_ERROR,
               context: { messageData: data },
-            }
+            },
           );
         }
       },
@@ -438,7 +417,7 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
         effectiveUserId,
         user_token,
         user_profile,
-      ]
+      ],
     );
 
     // Initialize user ID
@@ -457,8 +436,8 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
                 createStorageMessage(
                   StorageMessageType.SET_STORAGE,
                   "rblyn_anon",
-                  userId!
-                )
+                  userId!,
+                ),
               );
             });
           }
@@ -479,7 +458,7 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
         // Try to get stored ID once storage is ready
         executeStorageOp(() => {
           webViewRef.current?.injectJavaScript(
-            createStorageMessage(StorageMessageType.GET_STORAGE, "rblyn_anon")
+            createStorageMessage(StorageMessageType.GET_STORAGE, "rblyn_anon"),
           );
         });
       }
@@ -565,7 +544,7 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
             {
               type: ErrorTypes.NETWORK_ERROR,
               context: { api_key, user_id },
-            }
+            },
           );
         } finally {
           setLoading(false);
@@ -577,20 +556,6 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
       }
     }, [api_key, effectiveUserId, user_token]);
 
-    useEffect(() => {
-      const warmupOrigins = async () => {
-        const apiOrigin = getOriginFromUrl(API_URL);
-        const chatbotOrigin = getOriginFromUrl(BASE_CHATBOT_URL);
-
-        await Promise.allSettled([
-          warmupHttpOrigin(apiOrigin),
-          warmupHttpOrigin(chatbotOrigin),
-        ]);
-      };
-
-      warmupOrigins();
-    }, []);
-
     function constructUrl(): string {
       if (!effectiveUserId) return "";
       const params = new URLSearchParams({
@@ -631,7 +596,8 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
         webViewRef.current.postMessage(
           JSON.stringify({
             name: "closeFrame",
-          })
+            domain: "app-domain.com",
+          }),
         );
       }
 
@@ -682,7 +648,7 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
         }
         // Check if already granted
         const alreadyGranted = await PermissionsAndroid.check(
-          PermissionsAndroid.PERMISSIONS.RECORD_AUDIO
+          PermissionsAndroid.PERMISSIONS.RECORD_AUDIO,
         );
 
         if (alreadyGranted) {
@@ -696,7 +662,7 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
             message: "This app uses your microphone for voice features.",
             buttonPositive: "Allow",
             buttonNegative: "Deny",
-          }
+          },
         );
 
         const granted = result === PermissionsAndroid.RESULTS.GRANTED;
@@ -705,7 +671,7 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
           // Optional UX
           Alert.alert(
             "Microphone required",
-            "Voice features will not work unless you allow microphone access."
+            "Voice features will not work unless you allow microphone access.",
           );
         }
 
@@ -829,7 +795,7 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
                         return (
                           request.url.startsWith(BASE_CHATBOT_URL) ||
                           request.url.startsWith(
-                            chatbotConfig?.chat_iframe_url || ""
+                            chatbotConfig?.chat_iframe_url || "",
                           )
                         );
                       }}
@@ -919,7 +885,7 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
                               url: nativeEvent.url,
                               code: nativeEvent.code,
                             },
-                          }
+                          },
                         );
                       }}
                     />
@@ -933,7 +899,7 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
         </View>
       </ErrorBoundary>
     );
-  }
+  },
 );
 
 Chatbot.displayName = "Chatbot";

package/src/utils/fileDownload.ts

@@ -3,43 +3,167 @@
  * Utility functions for handling file downloads
  */
 
-import { Platform, Linking } from "react-native";
+import { Linking, NativeModules, Platform } from "react-native";
 import { logger } from "./logger";
 
 export interface DownloadRequest {
   url: string;
   filename?: string;
+  inPlace?: boolean;
 }
 
+const normalizeUrl = (url: string): string => {
+  return url?.trim();
+};
+
+const getUrlPathTail = (url?: string): string => {
+  const cleanedUrl = url?.split("?")?.[0] || "";
+  return cleanedUrl?.split("/")?.pop()?.trim() || "";
+};
+
+const getExtensionFromName = (name?: string): string => {
+  if (!name?.includes(".")) return "";
+  return name.substring(name.lastIndexOf("."));
+};
+
+const resolveDownloadFileName = (filename?: string, url?: string): string => {
+  const trimmedName = filename?.trim();
+  if (trimmedName) {
+    const sanitizedName = trimmedName.replace(/[<>:"/\\|?*\x00-\x1F]/g, "_");
+    const urlExtension = getExtensionFromName(getUrlPathTail(url));
+    if (getExtensionFromName(sanitizedName) || !urlExtension)
+      return sanitizedName;
+    return `${sanitizedName}${urlExtension}`;
+  }
+  const tailName = getUrlPathTail(url);
+  if (tailName) return tailName.replace(/[<>:"/\\|?*\x00-\x1F]/g, "_");
+  return `download-robylon-${Date.now()}`;
+};
+
+type RobylonDownloadNativeModule = {
+  downloadFile: (url: string, filename: string) => Promise<string>;
+};
+
+const isValidHttpUrl = (url: string): boolean => {
+  return /^https?:\/\//i.test(url);
+};
+
+const getNativeDownloadModule = (): RobylonDownloadNativeModule | null => {
+  return (
+    (NativeModules?.RobylonDownloadModule as RobylonDownloadNativeModule) ||
+    null
+  );
+};
+
+const openUrlWithFallback = async (url: string): Promise<void> => {
+  try {
+    await Linking.openURL(url);
+    return;
+  } catch (primaryError) {
+    logger.warn("Primary URL open failed, trying browser fallback", {
+      url,
+      primaryError:
+        primaryError instanceof Error
+          ? primaryError?.message
+          : String(primaryError),
+    });
+  }
+
+  await Linking.openURL(
+    `https://docs.google.com/viewer?url=${encodeURIComponent(url)}`,
+  );
+};
+
+const downloadOnAndroid = async (
+  url: string,
+  filename: string,
+): Promise<void> => {
+  const module = getNativeDownloadModule();
+  if (!module?.downloadFile) {
+    throw new Error("Robylon native download module is not linked on Android");
+  }
+  await module.downloadFile(url, filename);
+};
+
+const downloadOniOS = async (url: string, filename: string): Promise<void> => {
+  const module = getNativeDownloadModule();
+  if (!module?.downloadFile) {
+    throw new Error("Robylon native download module is not linked on iOS");
+  }
+  await module.downloadFile(url, filename);
+};
+
+const downloadWithNativeStorage = async (
+  url: string,
+  filename?: string,
+): Promise<void> => {
+  const safeName = resolveDownloadFileName(filename, url);
+  if (Platform.OS === "android") {
+    await downloadOnAndroid(url, safeName);
+    return;
+  }
+  if (Platform.OS === "ios") {
+    await downloadOniOS(url, safeName);
+    return;
+  }
+  await openUrlWithFallback(url);
+};
+
+const shouldUseInPlaceDownload = (
+  downloadRequest?: DownloadRequest,
+): boolean => {
+  return downloadRequest?.inPlace === true;
+};
+
 /**
- * Downloads a file using the platform's default mechanism
+ * Downloads a file using native storage when possible
  * @param url - The URL of the file to download
  * @param filename - Optional filename for the download
  */
 export const downloadFile = async (
   url: string,
-  filename?: string
+  filename?: string,
+  inPlace?: boolean,
 ): Promise<void> => {
   try {
-    logger.debug("Initiating file download", { url, filename });
+    const normalizedUrl = normalizeUrl(url);
+    logger.debug("Initiating file download", { url: normalizedUrl, filename });
 
     // Validate URL
-    if (!url || typeof url !== "string") {
+    if (!normalizedUrl || typeof normalizedUrl !== "string") {
       throw new Error("Invalid URL provided for download");
     }
 
-    // For React Native, we'll use Linking to open the URL
-    // This will use the device's default browser/download manager
-    const supported = await Linking.canOpenURL(url);
+    if (!isValidHttpUrl(normalizedUrl)) {
+      throw new Error("Download URL must be an http/https URL");
+    }
 
-    if (!supported) {
-      throw new Error("Cannot open the provided URL");
+    if (inPlace) {
+      try {
+        await downloadWithNativeStorage(normalizedUrl, filename);
+        logger.debug("File downloaded with native storage flow");
+      } catch (nativeDownloadError) {
+        logger.warn("Native download failed, falling back to URL open", {
+          url: normalizedUrl,
+          nativeDownloadError:
+            nativeDownloadError instanceof Error
+              ? nativeDownloadError?.message
+              : String(nativeDownloadError),
+        });
+        await openUrlWithFallback(normalizedUrl);
+        logger.debug("File download opened with fallback browser flow");
+      }
+      return;
     }
 
-    await Linking.openURL(url);
-    logger.debug("File download initiated successfully");
+    await openUrlWithFallback(normalizedUrl);
+    logger.debug("File download opened with legacy URL flow");
   } catch (error) {
-    logger.error("File download failed", { url, filename, error });
+    logger.error("File download failed", {
+      url,
+      filename,
+      error: error instanceof Error ? error?.message : String(error),
+    });
     throw error;
   }
 };
@@ -49,7 +173,7 @@ export const downloadFile = async (
  * @param downloadRequest - The download request object
  */
 export const handleDownloadRequest = async (
-  downloadRequest: DownloadRequest
+  downloadRequest: DownloadRequest,
 ): Promise<void> => {
   const { url, filename } = downloadRequest;
 
@@ -59,7 +183,11 @@ export const handleDownloadRequest = async (
   }
 
   try {
-    await downloadFile(url, filename);
+    await downloadFile(
+      url,
+      filename,
+      shouldUseInPlaceDownload(downloadRequest),
+    );
   } catch (error) {
     logger.error("Failed to handle download request", {
       downloadRequest,

package/src/versions/version.staging.ts

@@ -1,2 +1,2 @@
 // This file is auto-generated. Do not modify it manually.
-export const SDK_VERSION = '2.0.32-staging.4';
+export const SDK_VERSION = '2.1.1-staging.3';