mini_racer-csim 0.21.1.3 → 0.21.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b0e53581dc8063ed19c418dfc9ea450a7fc027adf5ea7732b32fb5dbabd6b0c7
-  data.tar.gz: 100100878b0966db7d6b0d8a38723993ac33eff9252bd6c48adccfcec65bf92c
+  metadata.gz: 4a64c6a3cd59b38b736212c4ef9685cbc7c6bc9e56d0aee12734e957e749fb7c
+  data.tar.gz: 50e7e8a671b90f3bf2e6563092f31ed00edf7d2c8093eb52eccbe502635773ca
 SHA512:
-  metadata.gz: c9a3e14d28a170e7d60201c490e0c695c92684e88f77935c733812206cd7a158d07cd96b76cb0bbfdc9a2f9804c3add1a33d189c47ef2b304873980448ce3ab2
-  data.tar.gz: c74e8ae6b9acac62be7a1099e7464564f08ac3e485f4e2f992b879d27d91c636d412c17506e0e972646e69780cc81dfd70e3de3bdbf961b4c1e3bcf19c808e84
+  metadata.gz: 53cb234c4dca6643756f5f0558080dade542569bf64aef713e0f44477e05d2c7e14bc305f63671519d67131a63f35dcee65829893dd493011b5f2764552677bd
+  data.tar.gz: ee5e61767e56179bf2aec29c54ca81d0f3c6b3ad7d1788498bb45b84ee5f7d3b6dcb613cc5f297dbdcb4eae32f4005b594a6a7a562d51c9d1904b646537976cb

data/CHANGELOG

@@ -1,3 +1,6 @@
+- 0.21.1.4 - 08-06-2026
+  - Memoize the safe-context reply filter (the recursive deep-copy fallback used when a reply value is not cloneable by V8's ValueSerializer — e.g. it contains a function or a host object). A visited-set (original → filtered copy) makes shared subgraphs clone once instead of being re-walked per reference path (O(N) instead of super-linear), terminates cycles (previously a cyclic value containing a non-cloneable member recursed forever), and preserves object identity — matching the ValueSerializer fast path it backs
+
 - 0.21.1.3 - 08-06-2026
   - **Breaking (realm JS API):** move the per-frame-realm JS helpers off bare `globalThis` globals onto the opt-in host namespace, so globalThis pollution is decided once (by `host_namespace:`) rather than per feature
     - `__mr_realmGlobal(id)` → `<host_namespace>.realmGlobal(id)`

data/ext/mini_racer_csim_extension/mini_racer_v8.cc

@@ -23,44 +23,61 @@ static const char safe_context_script_source[] = R"js(
 ;(function($globalThis) {
     const {Map: $Map, Set: $Set} = $globalThis
     const sentinel = {}
-    return function filter(v) {
-        if (typeof v === "function")
-            return sentinel
-        if (typeof v !== "object" || v === null)
-            return v
-        if (v instanceof $Map) {
-            const m = new Map()
-            for (let [k, t] of Map.prototype.entries.call(v)) {
-                t = filter(t)
-                if (t !== sentinel)
-                    m.set(k, t)
-            }
-            return m
-        } else if (v instanceof $Set) {
-            const s = new Set()
-            for (let t of Set.prototype.values.call(v)) {
-                t = filter(t)
-                if (t !== sentinel)
-                    s.add(t)
-            }
-            return s
-        } else {
-            const o = Array.isArray(v) ? [] : {}
-            const pds = Object.getOwnPropertyDescriptors(v)
-            for (const [k, d] of Object.entries(pds)) {
-                if (!d.enumerable)
-                    continue
-                let t = d.value
-                if (d.get) {
-                    // *not* d.get.call(...), may have been tampered with
-                    t = Function.prototype.call.call(d.get, v, k)
+    return function filter(root) {
+        // Memoize original -> filtered copy. Registered BEFORE recursing into a
+        // value's children so that a value reachable by many paths is cloned
+        // once (linear, not super-linear: e.g. a DOM node's ownerDocument is
+        // reachable from every node), cycles terminate (the in-progress copy is
+        // returned), and object identity is preserved. This matches V8's
+        // ValueSerializer (the fast path this is the fallback for), whose ref
+        // table also dedupes and handles cycles; without it the slow path both
+        // diverges (duplicates shared objects) and can recurse forever on a
+        // cyclic value that happens to contain a non-cloneable member.
+        const seen = new Map()
+        return (function rec(v) {
+            if (typeof v === "function")
+                return sentinel
+            if (typeof v !== "object" || v === null)
+                return v
+            if (seen.has(v))
+                return seen.get(v)
+            if (v instanceof $Map) {
+                const m = new Map()
+                seen.set(v, m)
+                for (let [k, t] of Map.prototype.entries.call(v)) {
+                    t = rec(t)
+                    if (t !== sentinel)
+                        m.set(k, t)
+                }
+                return m
+            } else if (v instanceof $Set) {
+                const s = new Set()
+                seen.set(v, s)
+                for (let t of Set.prototype.values.call(v)) {
+                    t = rec(t)
+                    if (t !== sentinel)
+                        s.add(t)
                 }
-                t = filter(t)
-                if (t !== sentinel)
-                    Object.defineProperty(o, k, {value: t, enumerable: true})
+                return s
+            } else {
+                const o = Array.isArray(v) ? [] : {}
+                seen.set(v, o)
+                const pds = Object.getOwnPropertyDescriptors(v)
+                for (const [k, d] of Object.entries(pds)) {
+                    if (!d.enumerable)
+                        continue
+                    let t = d.value
+                    if (d.get) {
+                        // *not* d.get.call(...), may have been tampered with
+                        t = Function.prototype.call.call(d.get, v, k)
+                    }
+                    t = rec(t)
+                    if (t !== sentinel)
+                        Object.defineProperty(o, k, {value: t, enumerable: true})
+                }
+                return o
             }
-            return o
-        }
+        })(root)
     }
 })
 )js";

data/lib/mini_racer_csim/version.rb

@@ -4,6 +4,6 @@ module MiniRacerCsim
   # mini_racer-csim fork: upstream version + a fork revision segment.
   # 0.21.1.0 = first fork release on upstream 0.21.1; bump the 4th segment for
   # fork-only changes, reset it when rebasing onto a new upstream version.
-  VERSION = "0.21.1.3"
+  VERSION = "0.21.1.4"
   LIBV8_NODE_VERSION = "~> 24.12.0.1"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mini_racer-csim
 version: !ruby/object:Gem::Version
-  version: 0.21.1.3
+  version: 0.21.1.4
 platform: ruby
 authors:
 - Keita Urashima