@goliapkg/sentori-react-native 5.1.2 → 5.1.3

package/android/src/main/java/com/sentori/SentoriReplayCapture.kt

@@ -120,7 +120,8 @@ object SentoriReplayCapture {
         val nodes = JSONArray()
         val rootLoc = IntArray(2).also { root.getLocationInWindow(it) }
         val ctx = WalkContext(rootLoc = rootLoc, maskedSet = maskedSet)
-        walk(root, depth = 0, parentMasked = false, ctx = ctx, nodes = nodes)
+        val rootClip = android.graphics.Rect(0, 0, root.width, root.height)
+        walk(root, depth = 0, parentMasked = false, ctx = ctx, clip = rootClip, nodes = nodes)
 
         val payload = JSONObject().apply {
             put("ts", System.currentTimeMillis())
@@ -173,6 +174,7 @@ object SentoriReplayCapture {
         depth: Int,
         parentMasked: Boolean,
         ctx: WalkContext,
+        clip: android.graphics.Rect,
         nodes: JSONArray,
     ) {
         if (nodes.length() >= MAX_NODES) return
@@ -188,68 +190,94 @@ object SentoriReplayCapture {
         val w = view.width
         val h = view.height
 
+        // Insight 2026-08-01: nothing was clipped here at all — a
+        // scroll container's full content width shipped as-is and
+        // drew past the device in the replay. `clip` accumulates
+        // every clipping ancestor (clipChildren defaults to true, so
+        // scroll viewports narrow it); a node ships as its visible
+        // intersection, an invisible one not at all. This is the
+        // root-cause fix the 05-18 decorView→content anchor swap
+        // only sidestepped.
+        var childClip = clip
+
         // Emit a node ONLY when the view has visual extent. A zero-
         // size view contributes nothing to render but its subtree
-        // might; recurse below regardless.
+        // might; recurse below regardless (and without narrowing the
+        // clip — transient zero-size Fabric wrappers do not really
+        // clip their laid-out descendants).
         if (w > 0 && h > 0) {
             val loc = IntArray(2)
             view.getLocationInWindow(loc)
             val x = loc[0] - ctx.rootLoc[0]
             val y = loc[1] - ctx.rootLoc[1]
 
-            val node = JSONObject().apply {
-                put("x", x)
-                put("y", y)
-                put("w", w)
-                put("h", h)
+            val selfRect = android.graphics.Rect(x, y, x + w, y + h)
+            val visible = android.graphics.Rect(selfRect)
+            if (!visible.intersect(clip)) {
+                // Fully outside every clipping ancestor: the view
+                // shows nothing, and if it clips its own children the
+                // whole subtree is gone with it.
+                if (view is ViewGroup && view.clipChildren) return
+                visible.setEmpty()
             }
-
-            var kindEmitted = false
-            when {
-                masked -> {
-                    node.put("kind", "mask")
-                    kindEmitted = true
-                }
-                view is TextView && !view.text.isNullOrEmpty() -> {
-                    node.put("kind", "text")
-                    val text = view.text.toString().let { if (it.length > 200) it.substring(0, 200) else it }
-                    node.put("text", text)
-                    node.put("color", colorToHex(view.currentTextColor))
-                    kindEmitted = true
-                }
-                view is EditText -> {
-                    node.put("kind", "text")
-                    val text = (view.text ?: "").toString().let { if (it.length > 200) it.substring(0, 200) else it }
-                    node.put("text", text)
-                    kindEmitted = true
-                }
-                view is ImageView -> {
-                    node.put("kind", "image")
-                    kindEmitted = true
+            if (view is ViewGroup && view.clipChildren) {
+                childClip = android.graphics.Rect(visible)
+            }
+            if (!visible.isEmpty) {
+                val node = JSONObject().apply {
+                    put("x", visible.left)
+                    put("y", visible.top)
+                    put("w", visible.width())
+                    put("h", visible.height())
                 }
-                view.background != null -> {
-                    // rc.8 — backgrounds backed by a BitmapDrawable
-                    // (a View whose backgroundImage / drawable
-                    // resource is a raster) emit as `image` kind so
-                    // the dashboard renders them as a media region,
-                    // not as a grey rect. Everything else stays
-                    // `rect` and tries to extract a fill colour.
-                    val bg = view.background
-                    if (bg is BitmapDrawable) {
+
+                var kindEmitted = false
+                when {
+                    masked -> {
+                        node.put("kind", "mask")
+                        kindEmitted = true
+                    }
+                    view is TextView && !view.text.isNullOrEmpty() -> {
+                        node.put("kind", "text")
+                        val text = view.text.toString().let { if (it.length > 200) it.substring(0, 200) else it }
+                        node.put("text", text)
+                        node.put("color", colorToHex(view.currentTextColor))
+                        kindEmitted = true
+                    }
+                    view is EditText -> {
+                        node.put("kind", "text")
+                        val text = (view.text ?: "").toString().let { if (it.length > 200) it.substring(0, 200) else it }
+                        node.put("text", text)
+                        kindEmitted = true
+                    }
+                    view is ImageView -> {
                         node.put("kind", "image")
-                    } else {
-                        node.put("kind", "rect")
-                        val color = extractDrawableColor(bg)
-                        if (color != null && (color shr 24 and 0xff) != 0) {
-                            node.put("color", colorToHex(color))
+                        kindEmitted = true
+                    }
+                    view.background != null -> {
+                        // rc.8 — backgrounds backed by a BitmapDrawable
+                        // (a View whose backgroundImage / drawable
+                        // resource is a raster) emit as `image` kind so
+                        // the dashboard renders them as a media region,
+                        // not as a grey rect. Everything else stays
+                        // `rect` and tries to extract a fill colour.
+                        val bg = view.background
+                        if (bg is BitmapDrawable) {
+                            node.put("kind", "image")
+                        } else {
+                            node.put("kind", "rect")
+                            val color = extractDrawableColor(bg)
+                            if (color != null && (color shr 24 and 0xff) != 0) {
+                                node.put("color", colorToHex(color))
+                            }
                         }
+                        kindEmitted = true
                     }
-                    kindEmitted = true
                 }
-            }
 
-            if (kindEmitted) {
-                nodes.put(node)
+                if (kindEmitted) {
+                    nodes.put(node)
+                }
             }
         }
 
@@ -257,7 +285,7 @@ object SentoriReplayCapture {
         // descendants (the rc.3 fix).
         if (!masked && view is ViewGroup) {
             for (i in 0 until view.childCount) {
-                walk(view.getChildAt(i), depth + 1, masked, ctx, nodes)
+                walk(view.getChildAt(i), depth + 1, masked, ctx, childClip, nodes)
             }
         }
     }

package/ios/SentoriReplayCapture.swift

@@ -74,6 +74,7 @@ import UIKit
             parentMasked: false,
             maskedIds: maskedIds,
             window: window,
+            clip: window.bounds,
             nodes: &nodes
         )
         lastDiagNodes = nodes.count
@@ -195,6 +196,7 @@ import UIKit
         parentMasked: Bool,
         maskedIds: Set<String>,
         window: UIWindow,
+        clip: CGRect,
         nodes: inout [[String: Any]]
     ) {
         if nodes.count >= MAX_NODES { return }
@@ -207,17 +209,29 @@ import UIKit
             .map { maskedIds.contains($0) } ?? false
         let masked = parentMasked || isThisMasked
 
+        // Insight 2026-08-01: the old check culled only nodes FULLY
+        // outside the window, so a 2000pt scroll ruler shipped at
+        // 2000pt and drew past the phone. `clip` accumulates every
+        // clipping ancestor (scroll views, clipsToBounds containers)
+        // from the window down; a node ships as the part of it that
+        // is actually visible, and a subtree behind a clipping edge
+        // ships not at all. This replaces the 05-18 anchor-swap class
+        // of fix: clipping is the root cause, anchors were symptoms.
         let frame = view.convert(view.bounds, to: window)
-        // Skip nodes outside the window bounds (off-screen recyclers).
-        if !frame.intersects(window.bounds) {
-            return
+        let visible = frame.intersection(clip)
+        let clipsChildren = view.clipsToBounds
+        if visible.isNull || visible.isEmpty {
+            // Nothing of this view shows. Its children can still show
+            // when this view does not clip them (transform-heavy or
+            // zero-size wrappers); a clipping view ends the subtree.
+            if clipsChildren { return }
         }
 
         var node: [String: Any] = [
-            "x": Double(frame.origin.x),
-            "y": Double(frame.origin.y),
-            "w": Double(frame.size.width),
-            "h": Double(frame.size.height),
+            "x": Double(visible.origin.x),
+            "y": Double(visible.origin.y),
+            "w": Double(visible.size.width),
+            "h": Double(visible.size.height),
         ]
 
         if masked {
@@ -239,11 +253,14 @@ import UIKit
         }
         // else: invisible container — skip emitting but recurse.
 
-        if node["kind"] != nil {
+        if node["kind"] != nil, !visible.isNull, !visible.isEmpty {
             nodes.append(node)
         }
 
         if !masked {
+            // A clipping view narrows the clip for everything below
+            // it; a non-clipping one passes the inherited clip on.
+            let childClip = clipsChildren ? visible : clip
             // Don't expose internals of masked subtrees.
             for sub in view.subviews {
                 walk(
@@ -253,6 +270,7 @@ import UIKit
                     parentMasked: masked,
                     maskedIds: maskedIds,
                     window: window,
+                    clip: childClip,
                     nodes: &nodes
                 )
             }

package/lib/transport.js

@@ -15,7 +15,7 @@ const MAX_RETRY = 3;
 const STORAGE_KEY = '@sentori/pending';
 const MAX_PERSISTED = 1000;
 // Pinned to package.json by a test — bump both together.
-const SDK_VERSION = '5.1.2';
+const SDK_VERSION = '5.1.3';
 let _queue = [];
 let _assertStats = new Map();
 let _flushTimer = null;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@goliapkg/sentori-react-native",
-  "version": "5.1.2",
+  "version": "5.1.3",
   "description": "Sentori SDK for React Native \u2014 JS-layer error capture, native crash handlers (iOS / Android), batched transport, fetch + react-navigation tracing.",
   "license": "Apache-2.0 OR MIT",
   "author": "GOLIA K.K. <takagi@golia.jp> (https://golia.jp)",

package/src/transport.ts

@@ -20,7 +20,7 @@ const STORAGE_KEY = '@sentori/pending';
 const MAX_PERSISTED = 1000;
 
 // Pinned to package.json by a test — bump both together.
-const SDK_VERSION = '5.1.2';
+const SDK_VERSION = '5.1.3';
 
 let _queue: WireEvent[] = [];
 let _assertStats = new Map<string, AssertStat>();