@repliqo/sdk-react-native 0.3.5 → 0.3.6

package/dist/components/RepliqoScrollView.js

@@ -49,6 +49,12 @@ const THROTTLE_MS = 400;
  * where the user never pauses long enough for the debounce to fire.
  */
 const SCROLL_INTERVAL_MS = 1200;
+/**
+ * During scroll, throttle window-bounds re-measurement to at most once per
+ * this many ms. Bounds rarely change while scrolling, but we still want to
+ * catch orientation changes / keyboard / overlay animations.
+ */
+const REMEASURE_THROTTLE_MS = 500;
 let scrollViewIdCounter = 0;
 const nextScrollViewId = () => `repliqo-scroll-${++scrollViewIdCounter}`;
 /**
@@ -64,16 +70,24 @@ const nextScrollViewId = () => `repliqo-scroll-${++scrollViewIdCounter}`;
 exports.RepliqoScrollView = react_1.default.forwardRef(({ onScroll, onMomentumScrollEnd, onScrollEndDrag, onLayout, onContentSizeChange, ...props }, ref) => {
     const debounceTimer = (0, react_1.useRef)(null);
     const scrollIntervalTimer = (0, react_1.useRef)(null);
+    const initialLayoutTimer = (0, react_1.useRef)(null);
     const lastCaptureTime = (0, react_1.useRef)(0);
     const lastCaptureY = (0, react_1.useRef)(-999);
+    const lastMeasureTime = (0, react_1.useRef)(0);
     const initialCaptured = (0, react_1.useRef)(false);
     const realContentHeight = (0, react_1.useRef)(0);
     const scrollViewId = (0, react_1.useRef)(nextScrollViewId());
     const innerRef = (0, react_1.useRef)(null);
+    const isMounted = (0, react_1.useRef)(true);
     const scrollState = (0, react_1.useRef)(null);
-    // Forward the inner ref to the parent's ref (if provided)
+    // Forward the inner ref to the parent's ref (if provided).
+    // IMPORTANT: do NOT clear innerRef on cleanup — RN sometimes calls the
+    // ref callback with null mid-update, and clearing would break async
+    // measurement callbacks that fire after the ref reattaches.
     const setRef = (0, react_1.useCallback)((node) => {
-        innerRef.current = node;
+        if (node !== null) {
+            innerRef.current = node;
+        }
         if (typeof ref === 'function') {
             ref(node);
         }
@@ -81,12 +95,21 @@ exports.RepliqoScrollView = react_1.default.forwardRef(({ onScroll, onMomentumSc
             ref.current = node;
         }
     }, [ref]);
-    /** Measure window-relative bounds and register with SDK. */
+    /**
+     * Measure window-relative bounds and register with SDK.
+     * Idempotent + safe to call after unmount (guards via isMounted).
+     */
     const measureAndRegister = (0, react_1.useCallback)(() => {
+        if (!isMounted.current)
+            return;
         const node = innerRef.current;
         if (!node || typeof node.measureInWindow !== 'function')
             return;
         node.measureInWindow((x, y, width, height) => {
+            if (!isMounted.current)
+                return;
+            // RN sometimes returns 0/0 dims before the view is positioned —
+            // ignore those, the next onLayout/onScroll will re-measure.
             if (width <= 0 || height <= 0)
                 return;
             try {
@@ -97,20 +120,29 @@ exports.RepliqoScrollView = react_1.default.forwardRef(({ onScroll, onMomentumSc
             catch { }
         });
     }, []);
-    // Cleanup timers + unregister on unmount
+    // Cleanup all timers + unregister + listeners on unmount
     (0, react_1.useEffect)(() => {
         const id = scrollViewId.current;
+        isMounted.current = true;
+        // Re-measure when keyboard appears/disappears (causes layout shift)
+        const showSub = react_native_1.Keyboard.addListener('keyboardDidShow', measureAndRegister);
+        const hideSub = react_native_1.Keyboard.addListener('keyboardDidHide', measureAndRegister);
         return () => {
+            isMounted.current = false;
             if (debounceTimer.current)
                 clearTimeout(debounceTimer.current);
             if (scrollIntervalTimer.current)
                 clearInterval(scrollIntervalTimer.current);
+            if (initialLayoutTimer.current)
+                clearTimeout(initialLayoutTimer.current);
+            showSub.remove();
+            hideSub.remove();
             try {
                 client_1.AppAnalytics.getInstance().unregisterScrollView(id);
             }
             catch { }
         };
-    }, []);
+    }, [measureAndRegister]);
     /**
      * Core tile capture logic. Reads from refs (stable across renders).
      * Checks throttle + delta before capturing.
@@ -118,6 +150,8 @@ exports.RepliqoScrollView = react_1.default.forwardRef(({ onScroll, onMomentumSc
      * @param force Skip throttle check (used for interval captures)
      */
     const doCapture = (force = false) => {
+        if (!isMounted.current)
+            return;
         const state = scrollState.current;
         if (!state || state.viewportHeight <= 0)
             return;
@@ -160,6 +194,13 @@ exports.RepliqoScrollView = react_1.default.forwardRef(({ onScroll, onMomentumSc
             sdk.updateScrollViewOffset(scrollViewId.current, contentOffset.y);
         }
         catch { }
+        // Re-measure bounds occasionally during scroll — catches layout
+        // shifts from animations, keyboard, orientation changes, etc.
+        const now = Date.now();
+        if (now - lastMeasureTime.current > REMEASURE_THROTTLE_MS) {
+            lastMeasureTime.current = now;
+            measureAndRegister();
+        }
         // Update scroll state — use realContentHeight if available
         const bestContentHeight = realContentHeight.current > 0
             ? realContentHeight.current
@@ -177,10 +218,12 @@ exports.RepliqoScrollView = react_1.default.forwardRef(({ onScroll, onMomentumSc
         startScrollInterval();
         // Capture initial tile (Y~0) on first scroll event with real dims
         if (!initialCaptured.current && contentOffset.y < 10) {
-            setTimeout(() => doCapture(), 200);
+            if (initialLayoutTimer.current)
+                clearTimeout(initialLayoutTimer.current);
+            initialLayoutTimer.current = setTimeout(() => doCapture(), 200);
         }
         onScroll?.(event);
-    }, [onScroll]);
+    }, [onScroll, measureAndRegister]);
     const handleScrollEndDrag = (0, react_1.useCallback)((event) => {
         // User lifted finger — capture immediately
         stopScrollInterval();
@@ -197,7 +240,6 @@ exports.RepliqoScrollView = react_1.default.forwardRef(({ onScroll, onMomentumSc
         onMomentumScrollEnd?.(event);
     }, [onMomentumScrollEnd]);
     const handleLayout = (0, react_1.useCallback)((event) => {
-        // Capture initial tile after layout (gets real viewportHeight)
         const { height } = event.nativeEvent.layout;
         if (!initialCaptured.current && height > 0) {
             scrollState.current = {
@@ -205,12 +247,15 @@ exports.RepliqoScrollView = react_1.default.forwardRef(({ onScroll, onMomentumSc
                 viewportHeight: height,
                 contentHeight: realContentHeight.current || height,
             };
-            setTimeout(() => doCapture(), 1000);
+            // Clear any prior pending initial-capture timer before scheduling
+            if (initialLayoutTimer.current)
+                clearTimeout(initialLayoutTimer.current);
+            initialLayoutTimer.current = setTimeout(() => doCapture(), 1000);
         }
-        // Register window-relative bounds with the SDK.
-        // Defer measurement to next tick — onLayout fires before the view
-        // is positioned in the window on some Android versions.
-        setTimeout(measureAndRegister, 50);
+        // Register window-relative bounds immediately (no setTimeout).
+        // measureInWindow itself is async; if it returns 0/0 dims, the
+        // next onLayout / onScroll will retry — no race window.
+        measureAndRegister();
         onLayout?.(event);
     }, [onLayout, measureAndRegister]);
     /**

package/dist/core/client.d.ts

@@ -32,6 +32,20 @@ export declare class AppAnalytics {
     static destroy(): void;
     startSession(deviceId: string, deviceInfo?: DeviceInfo): Promise<string>;
     endSession(): Promise<void>;
+    /**
+     * Track a touch event for heatmap analysis.
+     *
+     * @param x Touch X coordinate, in WINDOW coordinates (must match the
+     *          coordinate system of `measureInWindow` — typically `pageX`
+     *          from a React Native gesture event).
+     * @param y Touch Y coordinate, in WINDOW coordinates (must match the
+     *          coordinate system of `measureInWindow` — typically `pageY`
+     *          from a React Native gesture event). On Android with edge-to-edge
+     *          enabled, ensure `pageY` excludes the status bar to stay
+     *          consistent with `measureInWindow`.
+     * @param screenName Optional screen name (defaults to current screen).
+     * @param extra Optional extra data to attach to the event.
+     */
     trackTouch(x: number, y: number, screenName?: string, extra?: Record<string, any>): void;
     trackNavigation(fromScreen: string, toScreen: string): void;
     trackCustomEvent(eventName: string, data?: Record<string, any>): void;

package/dist/core/client.js

@@ -167,6 +167,20 @@ class AppAnalytics {
             this.snapshotCapture.setSessionId(null);
         }
     }
+    /**
+     * Track a touch event for heatmap analysis.
+     *
+     * @param x Touch X coordinate, in WINDOW coordinates (must match the
+     *          coordinate system of `measureInWindow` — typically `pageX`
+     *          from a React Native gesture event).
+     * @param y Touch Y coordinate, in WINDOW coordinates (must match the
+     *          coordinate system of `measureInWindow` — typically `pageY`
+     *          from a React Native gesture event). On Android with edge-to-edge
+     *          enabled, ensure `pageY` excludes the status bar to stay
+     *          consistent with `measureInWindow`.
+     * @param screenName Optional screen name (defaults to current screen).
+     * @param extra Optional extra data to attach to the event.
+     */
     trackTouch(x, y, screenName, extra) {
         if (!this.sessionId) {
             this.logger.warn('Cannot track touch: no active session');
@@ -176,29 +190,48 @@ class AppAnalytics {
             return;
         }
         // Auto-detect: find which (if any) registered ScrollView contains
-        // this touch. If inside, adjust Y by that ScrollView's scroll offset.
-        // If outside all ScrollViews, treat as fixed UI (no adjustment).
-        let activeScrollOffset = 0;
-        let isFixed = true;
-        if (this.scrollViewRegistry.size > 0) {
-            for (const info of this.scrollViewRegistry.values()) {
-                const { bounds } = info;
-                if (x >= bounds.x &&
-                    x <= bounds.x + bounds.width &&
-                    y >= bounds.y &&
-                    y <= bounds.y + bounds.height) {
-                    activeScrollOffset = info.scrollOffsetY;
-                    isFixed = false;
-                    break;
+        // this touch. If multiple match (nested ScrollViews, e.g. FlatList
+        // inside ScrollView), pick the SMALLEST by area — the most-nested
+        // one is the actual scroll context for that touch.
+        let bestMatchOffset = 0;
+        let bestMatchArea = Infinity;
+        let foundMatch = false;
+        for (const info of this.scrollViewRegistry.values()) {
+            const { bounds } = info;
+            if (x >= bounds.x &&
+                x <= bounds.x + bounds.width &&
+                y >= bounds.y &&
+                y <= bounds.y + bounds.height) {
+                const area = bounds.width * bounds.height;
+                if (area < bestMatchArea) {
+                    bestMatchArea = area;
+                    bestMatchOffset = info.scrollOffsetY;
+                    foundMatch = true;
                 }
             }
         }
-        else {
-            // No ScrollViews registered → fall back to legacy behavior
-            // (use the manually-set scrollOffsetY, treat as content touch).
-            activeScrollOffset = this.scrollOffsetY;
+        let activeScrollOffset;
+        let isFixed;
+        if (foundMatch) {
+            // Touch is inside a registered ScrollView → content touch
+            activeScrollOffset = bestMatchOffset;
             isFixed = false;
         }
+        else if (this.scrollViewRegistry.size > 0) {
+            // ScrollViews ARE registered, but this touch is outside all of them
+            // → it's on fixed UI (nav bar, header, etc.)
+            activeScrollOffset = 0;
+            isFixed = true;
+        }
+        else {
+            // No ScrollViews registered at all. Two possible cases:
+            //   (a) The screen has no scrollable content → touch is fixed
+            //   (b) RepliqoScrollView hasn't measured yet (first ~1 frame)
+            // Either way, marking as fixed is safer than guessing content
+            // position with stale scrollOffsetY from a previous screen.
+            activeScrollOffset = 0;
+            isFixed = true;
+        }
         const adjustedY = y + activeScrollOffset;
         const event = {
             type: 'touch',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@repliqo/sdk-react-native",
-  "version": "0.3.5",
+  "version": "0.3.6",
   "description": "Session replay & analytics SDK for React Native. Captures screenshots (including modals/alerts), navigation, screen visits, and custom events.",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/src/components/RepliqoScrollView.tsx

@@ -1,6 +1,7 @@
 import React, { useCallback, useRef, useEffect } from 'react';
 import {
   ScrollView,
+  Keyboard,
   type ScrollViewProps,
   type NativeSyntheticEvent,
   type NativeScrollEvent,
@@ -20,6 +21,12 @@ const THROTTLE_MS = 400;
  * where the user never pauses long enough for the debounce to fire.
  */
 const SCROLL_INTERVAL_MS = 1200;
+/**
+ * During scroll, throttle window-bounds re-measurement to at most once per
+ * this many ms. Bounds rarely change while scrolling, but we still want to
+ * catch orientation changes / keyboard / overlay animations.
+ */
+const REMEASURE_THROTTLE_MS = 500;
 
 let scrollViewIdCounter = 0;
 const nextScrollViewId = () => `repliqo-scroll-${++scrollViewIdCounter}`;
@@ -38,21 +45,29 @@ export const RepliqoScrollView = React.forwardRef<ScrollView, ScrollViewProps>(
   ({ onScroll, onMomentumScrollEnd, onScrollEndDrag, onLayout, onContentSizeChange, ...props }, ref) => {
     const debounceTimer = useRef<ReturnType<typeof setTimeout> | null>(null);
     const scrollIntervalTimer = useRef<ReturnType<typeof setInterval> | null>(null);
+    const initialLayoutTimer = useRef<ReturnType<typeof setTimeout> | null>(null);
     const lastCaptureTime = useRef(0);
     const lastCaptureY = useRef(-999);
+    const lastMeasureTime = useRef(0);
     const initialCaptured = useRef(false);
     const realContentHeight = useRef(0);
     const scrollViewId = useRef<string>(nextScrollViewId());
     const innerRef = useRef<ScrollView | null>(null);
+    const isMounted = useRef(true);
     const scrollState = useRef<{
       offsetY: number;
       viewportHeight: number;
       contentHeight: number;
     } | null>(null);
 
-    // Forward the inner ref to the parent's ref (if provided)
+    // Forward the inner ref to the parent's ref (if provided).
+    // IMPORTANT: do NOT clear innerRef on cleanup — RN sometimes calls the
+    // ref callback with null mid-update, and clearing would break async
+    // measurement callbacks that fire after the ref reattaches.
     const setRef = useCallback((node: ScrollView | null) => {
-      innerRef.current = node;
+      if (node !== null) {
+        innerRef.current = node;
+      }
       if (typeof ref === 'function') {
         ref(node);
       } else if (ref) {
@@ -60,11 +75,18 @@ export const RepliqoScrollView = React.forwardRef<ScrollView, ScrollViewProps>(
       }
     }, [ref]);
 
-    /** Measure window-relative bounds and register with SDK. */
+    /**
+     * Measure window-relative bounds and register with SDK.
+     * Idempotent + safe to call after unmount (guards via isMounted).
+     */
     const measureAndRegister = useCallback(() => {
+      if (!isMounted.current) return;
       const node = innerRef.current as any;
       if (!node || typeof node.measureInWindow !== 'function') return;
       node.measureInWindow((x: number, y: number, width: number, height: number) => {
+        if (!isMounted.current) return;
+        // RN sometimes returns 0/0 dims before the view is positioned —
+        // ignore those, the next onLayout/onScroll will re-measure.
         if (width <= 0 || height <= 0) return;
         try {
           AppAnalytics.getInstance().registerScrollView(scrollViewId.current, {
@@ -74,17 +96,27 @@ export const RepliqoScrollView = React.forwardRef<ScrollView, ScrollViewProps>(
       });
     }, []);
 
-    // Cleanup timers + unregister on unmount
+    // Cleanup all timers + unregister + listeners on unmount
     useEffect(() => {
       const id = scrollViewId.current;
+      isMounted.current = true;
+
+      // Re-measure when keyboard appears/disappears (causes layout shift)
+      const showSub = Keyboard.addListener('keyboardDidShow', measureAndRegister);
+      const hideSub = Keyboard.addListener('keyboardDidHide', measureAndRegister);
+
       return () => {
+        isMounted.current = false;
         if (debounceTimer.current) clearTimeout(debounceTimer.current);
         if (scrollIntervalTimer.current) clearInterval(scrollIntervalTimer.current);
+        if (initialLayoutTimer.current) clearTimeout(initialLayoutTimer.current);
+        showSub.remove();
+        hideSub.remove();
         try {
           AppAnalytics.getInstance().unregisterScrollView(id);
         } catch {}
       };
-    }, []);
+    }, [measureAndRegister]);
 
     /**
      * Core tile capture logic. Reads from refs (stable across renders).
@@ -93,6 +125,7 @@ export const RepliqoScrollView = React.forwardRef<ScrollView, ScrollViewProps>(
      * @param force Skip throttle check (used for interval captures)
      */
     const doCapture = (force = false) => {
+      if (!isMounted.current) return;
       const state = scrollState.current;
       if (!state || state.viewportHeight <= 0) return;
 
@@ -144,6 +177,14 @@ export const RepliqoScrollView = React.forwardRef<ScrollView, ScrollViewProps>(
           sdk.updateScrollViewOffset(scrollViewId.current, contentOffset.y);
         } catch {}
 
+        // Re-measure bounds occasionally during scroll — catches layout
+        // shifts from animations, keyboard, orientation changes, etc.
+        const now = Date.now();
+        if (now - lastMeasureTime.current > REMEASURE_THROTTLE_MS) {
+          lastMeasureTime.current = now;
+          measureAndRegister();
+        }
+
         // Update scroll state — use realContentHeight if available
         const bestContentHeight = realContentHeight.current > 0
           ? realContentHeight.current
@@ -163,12 +204,13 @@ export const RepliqoScrollView = React.forwardRef<ScrollView, ScrollViewProps>(
 
         // Capture initial tile (Y~0) on first scroll event with real dims
         if (!initialCaptured.current && contentOffset.y < 10) {
-          setTimeout(() => doCapture(), 200);
+          if (initialLayoutTimer.current) clearTimeout(initialLayoutTimer.current);
+          initialLayoutTimer.current = setTimeout(() => doCapture(), 200);
         }
 
         onScroll?.(event);
       },
-      [onScroll],
+      [onScroll, measureAndRegister],
     );
 
     const handleScrollEndDrag = useCallback(
@@ -196,7 +238,6 @@ export const RepliqoScrollView = React.forwardRef<ScrollView, ScrollViewProps>(
 
     const handleLayout = useCallback(
       (event: LayoutChangeEvent) => {
-        // Capture initial tile after layout (gets real viewportHeight)
         const { height } = event.nativeEvent.layout;
         if (!initialCaptured.current && height > 0) {
           scrollState.current = {
@@ -204,12 +245,14 @@ export const RepliqoScrollView = React.forwardRef<ScrollView, ScrollViewProps>(
             viewportHeight: height,
             contentHeight: realContentHeight.current || height,
           };
-          setTimeout(() => doCapture(), 1000);
+          // Clear any prior pending initial-capture timer before scheduling
+          if (initialLayoutTimer.current) clearTimeout(initialLayoutTimer.current);
+          initialLayoutTimer.current = setTimeout(() => doCapture(), 1000);
         }
-        // Register window-relative bounds with the SDK.
-        // Defer measurement to next tick — onLayout fires before the view
-        // is positioned in the window on some Android versions.
-        setTimeout(measureAndRegister, 50);
+        // Register window-relative bounds immediately (no setTimeout).
+        // measureInWindow itself is async; if it returns 0/0 dims, the
+        // next onLayout / onScroll will retry — no race window.
+        measureAndRegister();
         onLayout?.(event);
       },
       [onLayout, measureAndRegister],

package/src/core/client.ts

@@ -250,6 +250,20 @@ export class AppAnalytics {
     }
   }
 
+  /**
+   * Track a touch event for heatmap analysis.
+   *
+   * @param x Touch X coordinate, in WINDOW coordinates (must match the
+   *          coordinate system of `measureInWindow` — typically `pageX`
+   *          from a React Native gesture event).
+   * @param y Touch Y coordinate, in WINDOW coordinates (must match the
+   *          coordinate system of `measureInWindow` — typically `pageY`
+   *          from a React Native gesture event). On Android with edge-to-edge
+   *          enabled, ensure `pageY` excludes the status bar to stay
+   *          consistent with `measureInWindow`.
+   * @param screenName Optional screen name (defaults to current screen).
+   * @param extra Optional extra data to attach to the event.
+   */
   trackTouch(
     x: number,
     y: number,
@@ -266,30 +280,50 @@ export class AppAnalytics {
     }
 
     // Auto-detect: find which (if any) registered ScrollView contains
-    // this touch. If inside, adjust Y by that ScrollView's scroll offset.
-    // If outside all ScrollViews, treat as fixed UI (no adjustment).
-    let activeScrollOffset = 0;
-    let isFixed = true;
-
-    if (this.scrollViewRegistry.size > 0) {
-      for (const info of this.scrollViewRegistry.values()) {
-        const { bounds } = info;
-        if (
-          x >= bounds.x &&
-          x <= bounds.x + bounds.width &&
-          y >= bounds.y &&
-          y <= bounds.y + bounds.height
-        ) {
-          activeScrollOffset = info.scrollOffsetY;
-          isFixed = false;
-          break;
+    // this touch. If multiple match (nested ScrollViews, e.g. FlatList
+    // inside ScrollView), pick the SMALLEST by area — the most-nested
+    // one is the actual scroll context for that touch.
+    let bestMatchOffset = 0;
+    let bestMatchArea = Infinity;
+    let foundMatch = false;
+
+    for (const info of this.scrollViewRegistry.values()) {
+      const { bounds } = info;
+      if (
+        x >= bounds.x &&
+        x <= bounds.x + bounds.width &&
+        y >= bounds.y &&
+        y <= bounds.y + bounds.height
+      ) {
+        const area = bounds.width * bounds.height;
+        if (area < bestMatchArea) {
+          bestMatchArea = area;
+          bestMatchOffset = info.scrollOffsetY;
+          foundMatch = true;
         }
       }
-    } else {
-      // No ScrollViews registered → fall back to legacy behavior
-      // (use the manually-set scrollOffsetY, treat as content touch).
-      activeScrollOffset = this.scrollOffsetY;
+    }
+
+    let activeScrollOffset: number;
+    let isFixed: boolean;
+
+    if (foundMatch) {
+      // Touch is inside a registered ScrollView → content touch
+      activeScrollOffset = bestMatchOffset;
       isFixed = false;
+    } else if (this.scrollViewRegistry.size > 0) {
+      // ScrollViews ARE registered, but this touch is outside all of them
+      // → it's on fixed UI (nav bar, header, etc.)
+      activeScrollOffset = 0;
+      isFixed = true;
+    } else {
+      // No ScrollViews registered at all. Two possible cases:
+      //   (a) The screen has no scrollable content → touch is fixed
+      //   (b) RepliqoScrollView hasn't measured yet (first ~1 frame)
+      // Either way, marking as fixed is safer than guessing content
+      // position with stale scrollOffsetY from a previous screen.
+      activeScrollOffset = 0;
+      isFixed = true;
     }
 
     const adjustedY = y + activeScrollOffset;