@robylon/react-native-sdk 2.1.1-staging.3 → 2.1.3-staging.0

package/lib/typescript/hooks/useChatbotEvents.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"useChatbotEvents.d.ts","sourceRoot":"","sources":["../../../src/hooks/useChatbotEvents.ts"],"names":[],"mappings":"AAGA,OAAO,EACL,gBAAgB,EAChB,aAAa,EACb,mBAAmB,EACpB,MAAM,iBAAiB,CAAC;AAGzB,UAAU,cAAc;IACtB,OAAO,EAAE,MAAM,CAAC;IAChB,aAAa,EAAE,GAAG,CAAC;IACnB,YAAY,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;IACnC,OAAO,CAAC,EAAE,mBAAmB,CAAC;IAC9B,UAAU,EAAE;QAAE,EAAE,EAAE,MAAM,CAAC;QAAC,OAAO,EAAE,MAAM,CAAA;KAAE,CAAC;CAC7C;AAED,eAAO,MAAM,gBAAgB,mEAM1B,cAAc;sBAwDN,gBAAgB,SAAS,GAAG;iCArDtB,aAAa,mBACR,OAAO,MAAM,EAAE,GAAG,CAAC;CAgExC,CAAC"}
+{"version":3,"file":"useChatbotEvents.d.ts","sourceRoot":"","sources":["../../../src/hooks/useChatbotEvents.ts"],"names":[],"mappings":"AAGA,OAAO,EACL,gBAAgB,EAChB,aAAa,EACb,mBAAmB,EACpB,MAAM,iBAAiB,CAAC;AAGzB,UAAU,cAAc;IACtB,OAAO,EAAE,MAAM,CAAC;IAChB,aAAa,EAAE,GAAG,CAAC;IACnB,YAAY,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;IACnC,OAAO,CAAC,EAAE,mBAAmB,CAAC;IAC9B,UAAU,EAAE;QAAE,EAAE,EAAE,MAAM,CAAC;QAAC,OAAO,EAAE,MAAM,CAAA;KAAE,CAAC;CAC7C;AAED,eAAO,MAAM,gBAAgB,mEAM1B,cAAc;sBA0EN,gBAAgB,SAAS,GAAG;iCAxDtB,aAAa,mBACR,OAAO,MAAM,EAAE,GAAG,CAAC;CAyExC,CAAC"}

package/lib/typescript/utils/session.d.ts

@@ -0,0 +1,21 @@
+/**
+ * Session id pass-through between the chatbot web app and the SDK client.
+ *
+ * The session id travels on its own message type so that it never reaches
+ * `onEvent`, `onMessage` or the internal `/users/sdk/record-logs/` endpoint.
+ * Its only exits from the SDK are the `onSession` prop and `getSessionId()`
+ * on the ref, both of which the client has to opt into explicitly.
+ */
+/** Message the web app posts whenever the active session id changes. */
+export declare const SESSION_UPDATED_MESSAGE_TYPE = "SESSION_UPDATED";
+/** Key carrying the resume target inside the `registerUserId` payload. */
+export declare const RESUME_SESSION_KEY = "sessionId";
+/**
+ * Pulls the session id out of a `SESSION_UPDATED` payload.
+ *
+ * Accepts the snake_case, camelCase and nested spellings so a naming slip on
+ * the web side degrades to a no-op rather than a broken build. Anything that
+ * is not a non-empty string is dropped.
+ */
+export declare const extractSessionId: (data: any) => string | undefined;
+//# sourceMappingURL=session.d.ts.map

package/lib/typescript/utils/session.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"session.d.ts","sourceRoot":"","sources":["../../../src/utils/session.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAEH,wEAAwE;AACxE,eAAO,MAAM,4BAA4B,oBAAoB,CAAC;AAE9D,0EAA0E;AAC1E,eAAO,MAAM,kBAAkB,cAAc,CAAC;AAE9C;;;;;;GAMG;AACH,eAAO,MAAM,gBAAgB,SAAU,GAAG,KAAG,MAAM,GAAG,SAGrD,CAAC"}

package/lib/typescript/versions/version.staging.d.ts

@@ -1,2 +1,2 @@
-export declare const SDK_VERSION = "2.1.1-staging.3";
+export declare const SDK_VERSION = "2.1.3-staging.0";
 //# sourceMappingURL=version.staging.d.ts.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@robylon/react-native-sdk",
-  "version": "2.1.1-staging.3",
+  "version": "2.1.3-staging.0",
   "description": "React Native SDK for Robylon",
   "main": "lib/commonjs/index.js",
   "module": "lib/module/index.js",
@@ -25,7 +25,7 @@
     "publish:staging": "npm version $(node scripts/get-next-version.js staging) && node scripts/update-version.js staging && npm run build:staging && npm publish --tag staging",
     "publish:production": "node scripts/validate-publish.js production && npm run build:production && npm publish",
     "publish:hotfix": "node scripts/validate-publish.js hotfix && npm version $(node scripts/get-next-version.js hotfix) && node scripts/update-version.js production && npm run build:production && npm publish --tag hotfix",
-    "postinstall": "node scripts/setup-git-hooks.js",
+    "postinstall": "node scripts/setup-git-hooks.js || exit 0",
     "branch": "node scripts/create-branch.js",
     "tag": "node scripts/create-version-tag.js"
   },

package/scripts/setup-git-hooks.js

@@ -2,17 +2,29 @@ const fs = require("fs");
 const path = require("path");
 const { execSync } = require("child_process");
 
-// Create .git/hooks directory if it doesn't exist
-const hooksDir = path.join(__dirname, "../.git/hooks");
-if (!fs.existsSync(hooksDir)) {
-  fs.mkdirSync(hooksDir, { recursive: true });
+// This runs from postinstall, which also fires when the SDK is installed as a
+// dependency. Bail out unless we are in the SDK's own checkout, so consumers
+// do not get a stray .git/hooks directory inside node_modules.
+const gitDir = path.join(__dirname, "../.git");
+if (!fs.existsSync(gitDir)) {
+  process.exit(0);
 }
 
-// Copy the validate-branch-name script
-const sourcePath = path.join(__dirname, "validate-branch-name.sh");
-const targetPath = path.join(hooksDir, "commit-msg");
+try {
+  const hooksDir = path.join(gitDir, "hooks");
+  if (!fs.existsSync(hooksDir)) {
+    fs.mkdirSync(hooksDir, { recursive: true });
+  }
 
-fs.copyFileSync(sourcePath, targetPath);
-fs.chmodSync(targetPath, "755");
+  // Copy the validate-branch-name script
+  const sourcePath = path.join(__dirname, "validate-branch-name.sh");
+  const targetPath = path.join(hooksDir, "commit-msg");
 
-console.log("Git hooks installed successfully!");
+  fs.copyFileSync(sourcePath, targetPath);
+  fs.chmodSync(targetPath, "755");
+
+  console.log("Git hooks installed successfully!");
+} catch (error) {
+  // Never fail an install over developer tooling.
+  console.warn("Skipped git hook setup:", error.message);
+}

package/src/Chatbotsdk.tsx

@@ -49,6 +49,11 @@ import {
 } from "./utils/webViewStorage";
 import { animateWebViewOpen, animateWebViewClose } from "./utils/animations";
 import { handleDownloadRequest, DownloadRequest } from "./utils/fileDownload";
+import {
+  SESSION_UPDATED_MESSAGE_TYPE,
+  RESUME_SESSION_KEY,
+  extractSessionId,
+} from "./utils/session";
 
 export interface ChatbotProps {
   api_key: string;
@@ -61,10 +66,16 @@ export interface ChatbotProps {
     is_test_user?: boolean;
     [key: string | number]: any;
   };
+  session_id?: string;
+  session_context?: Record<string, any>;
   onEvent?: ChatbotEventHandler;
   onOpen?: () => void;
   onClose?: () => void;
   onReady?: () => void;
+  onSession?: (
+    session_id: string,
+    session_context?: Record<string, any>,
+  ) => void;
 }
 
 export interface ChatbotRef {
@@ -72,6 +83,7 @@ export interface ChatbotRef {
   close: () => void;
   toggle: () => void;
   isReady: () => boolean;
+  getSessionId: () => string | undefined;
 }
 
 interface InternalChatbotProps extends ChatbotProps {
@@ -151,6 +163,9 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
       onClose: externalOnClose,
       onReady,
       user_profile,
+      session_id,
+      session_context,
+      onSession,
     },
     ref,
   ) => {
@@ -171,6 +186,18 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
     const [systemInfo, setSystemInfo] = useState({ os: "", browser: "" });
     const systemInfoRef = useRef({ os: "", browser: "" });
 
+    // Session id pass-through. All four are refs so that client-supplied values
+    // never enter handleMessage's dependency array: adding them would rebuild
+    // the handler on every client re-render, omitting them without a ref would
+    // capture a stale closure.
+    const sessionIdRef = useRef<string | undefined>(undefined);
+    const onSessionRef = useRef(onSession);
+    const sessionContextRef = useRef(session_context);
+    const resumeSessionRef = useRef(session_id);
+    onSessionRef.current = onSession;
+    sessionContextRef.current = session_context;
+    resumeSessionRef.current = session_id;
+
     // Initialize animated values
     const animatedValues = useRef({
       scale: new Animated.Value(1),
@@ -221,10 +248,37 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
           }
         },
         isReady: () => isInitialized,
+        getSessionId: () => sessionIdRef.current,
       }),
       [isInitialized, isWebViewVisible],
     );
 
+    // Hands the session id to the client. Deduped so that a re-emission on
+    // reopen does not fire client side effects twice, and wrapped so that a
+    // throwing client handler cannot break message handling.
+    const emitSession = useCallback((data: any) => {
+      const sessionId = extractSessionId(data);
+      if (!sessionId || sessionId === sessionIdRef.current) return;
+      sessionIdRef.current = sessionId;
+
+      try {
+        onSessionRef.current?.(sessionId, sessionContextRef.current ?? {});
+      } catch (error) {
+        errorTracker.trackError(
+          error instanceof Error
+            ? error
+            : new Error("onSession handler threw an error"),
+          "ChatbotSDK",
+          {
+            // The session id itself is deliberately left out of the report so
+            // that it stays confined to the two opt-in client channels.
+            type: ErrorTypes.RUNTIME_ERROR,
+            context: { source: SESSION_UPDATED_MESSAGE_TYPE },
+          },
+        );
+      }
+    }, []);
+
     const triggerAppInit = () => {
       if (webViewRef.current) {
         webViewRef.current?.injectJavaScript(getBrowserAndOSInfoScript());
@@ -308,6 +362,15 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
             return;
           }
 
+          // Returns before the switch below so the session id never reaches
+          // emitEvent (and through it onEvent and /users/sdk/record-logs/) or
+          // the default branch's onMessage. Clients who registered neither
+          // onSession nor getSessionId cannot observe it.
+          if (parsedData?.type === SESSION_UPDATED_MESSAGE_TYPE) {
+            emitSession(parsedData?.data);
+            return;
+          }
+
           if (parsedData?.type === "APP_READY") {
             emitEvent(ChatbotEventType.CHATBOT_APP_READY);
             if (webViewRef.current && isWebViewVisible) {
@@ -318,6 +381,7 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
                 action: "registerUserId",
                 data: {
                   userId: effectiveUserId,
+                  [RESUME_SESSION_KEY]: resumeSessionRef.current || undefined,
                   token: user_token ? `${user_token}` : undefined,
                   userProfile: {
                     ...user_profile,
@@ -328,7 +392,21 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
                   },
                 },
               };
-              logger.debug("messageData====>", JSON.stringify(messageData));
+              // logger.debug is an unconditional console.log, so the session id
+              // is redacted here rather than printed into every client's
+              // production console.
+              logger.debug(
+                "messageData====>",
+                JSON.stringify({
+                  ...messageData,
+                  data: {
+                    ...messageData.data,
+                    [RESUME_SESSION_KEY]: resumeSessionRef.current
+                      ? "[redacted]"
+                      : undefined,
+                  },
+                }),
+              );
               webViewRef.current?.injectJavaScript(`
             window.postMessage(${JSON.stringify({
               name: "openFrame",
@@ -411,6 +489,7 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
       },
       [
         emitEvent,
+        emitSession,
         isWebViewVisible,
         onMessage,
         systemInfo,
@@ -470,6 +549,11 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
           const endpointUrl = `${API_URL}/chat/chatbot/get/`;
           const payload = {
             client_user_id: effectiveUserId,
+            // Read from the ref rather than the prop so that a later change
+            // does not re-trigger the config fetch. Consistent with session_id
+            // being an initialization-time input. JSON.stringify drops the key
+            // when there is nothing to resume.
+            resumable_session_id: resumeSessionRef.current || undefined,
             org_id: api_key,
             token: user_token,
             extra_info: {},
@@ -728,7 +812,17 @@ const Chatbot = forwardRef<ChatbotRef, InternalChatbotProps>(
 
     return (
       <ErrorBoundary componentName="ChatbotSDK">
-        <View style={styles?.mainContainer}>
+        {/*
+          This root view is flex: 1 and stays mounted while the chat is closed.
+          Left hit-testable, it sits invisibly over whatever the host renders
+          behind it and swallows every tap. "box-none" exempts the container
+          itself while leaving its children interactive, which is what the
+          floating button and the dev-only debug button need. The WebView
+          wrapper below manages its own pointerEvents.
+        */}
+        <View
+          style={styles?.mainContainer}
+          pointerEvents={isWebViewVisible ? "auto" : "box-none"}>
           {!loading && (
             <>
               <ErrorBoundary componentName="FloatingButton">

package/src/hooks/useChatbotEvents.ts

@@ -1,4 +1,4 @@
-import { useCallback } from "react";
+import { useCallback, useRef } from "react";
 import { getSystemInfo } from "../utils/systemInfo";
 import { logger } from "../utils/logger";
 import {
@@ -23,6 +23,21 @@ export const useChatbotEvents = ({
   onEvent,
   systemInfo,
 }: EventHookProps) => {
+  // Held in refs so that emitEvent keeps a stable identity across renders.
+  // Host apps routinely pass an inline onEvent and an inline user_profile
+  // object literal; with those in the dependency arrays, emitEvent changed
+  // identity on every render, which re-ran the effects keyed on it, which
+  // re-emitted events, which re-rendered the host. That loop also POSTed to
+  // /users/sdk/record-logs/ on every iteration.
+  const onEventRef = useRef(onEvent);
+  const chatbotConfigRef = useRef(chatbotConfig);
+  const userProfileRef = useRef(user_profile);
+  const systemInfoRef = useRef(systemInfo);
+  onEventRef.current = onEvent;
+  chatbotConfigRef.current = chatbotConfig;
+  userProfileRef.current = user_profile;
+  systemInfoRef.current = systemInfo;
+
   const onInternalEvent = useCallback(
     async (
       eventType: AllEventTypes,
@@ -30,9 +45,12 @@ export const useChatbotEvents = ({
     ) => {
       if (!api_key) return;
 
+      const chatbotConfig = chatbotConfigRef.current;
+      const user_profile = userProfileRef.current;
+
       try {
         const apiUrl = API_URL;
-        const sysInfo = getSystemInfo(systemInfo);
+        const sysInfo = getSystemInfo(systemInfoRef.current);
 
         const eventData = {
           trigger_time: Date.now(),
@@ -74,16 +92,22 @@ export const useChatbotEvents = ({
         logger.error("Failed to send internal event:", error);
       }
     },
-    [api_key, chatbotConfig, user_profile, systemInfo]
+    [api_key]
   );
 
   const emitEvent = useCallback(
     (type: ChatbotEventType, data?: any) => {
       const timestamp = Date.now();
-      onEvent?.({ type, timestamp, data });
+      // Isolated so that a throwing host handler cannot prevent the internal
+      // event from being recorded, or break the caller.
+      try {
+        onEventRef.current?.({ type, timestamp, data });
+      } catch (error) {
+        logger.error("onEvent handler threw an error:", error);
+      }
       onInternalEvent(type, { ...data, timestamp });
     },
-    [onEvent, onInternalEvent]
+    [onInternalEvent]
   );
 
   return {

package/src/utils/session.ts

@@ -0,0 +1,26 @@
+/**
+ * Session id pass-through between the chatbot web app and the SDK client.
+ *
+ * The session id travels on its own message type so that it never reaches
+ * `onEvent`, `onMessage` or the internal `/users/sdk/record-logs/` endpoint.
+ * Its only exits from the SDK are the `onSession` prop and `getSessionId()`
+ * on the ref, both of which the client has to opt into explicitly.
+ */
+
+/** Message the web app posts whenever the active session id changes. */
+export const SESSION_UPDATED_MESSAGE_TYPE = "SESSION_UPDATED";
+
+/** Key carrying the resume target inside the `registerUserId` payload. */
+export const RESUME_SESSION_KEY = "sessionId";
+
+/**
+ * Pulls the session id out of a `SESSION_UPDATED` payload.
+ *
+ * Accepts the snake_case, camelCase and nested spellings so a naming slip on
+ * the web side degrades to a no-op rather than a broken build. Anything that
+ * is not a non-empty string is dropped.
+ */
+export const extractSessionId = (data: any): string | undefined => {
+  const raw = data?.session_id ?? data?.sessionId ?? data?.session?.id;
+  return typeof raw === "string" && raw.length > 0 ? raw : undefined;
+};

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.1.1-staging.3';
+export const SDK_VERSION = '2.1.3-staging.0';