mini_racer-csim 0.21.1.4 → 0.21.1.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4a64c6a3cd59b38b736212c4ef9685cbc7c6bc9e56d0aee12734e957e749fb7c
-  data.tar.gz: 50e7e8a671b90f3bf2e6563092f31ed00edf7d2c8093eb52eccbe502635773ca
+  metadata.gz: 18ed230356524c1098d8809a51fdee97a601f949e027f86ec7e393869ab8daf2
+  data.tar.gz: ab81fcad2a27c063af01ae4d6a8f66c9d2025fadd555f8e0036034f9e12adf38
 SHA512:
-  metadata.gz: 53cb234c4dca6643756f5f0558080dade542569bf64aef713e0f44477e05d2c7e14bc305f63671519d67131a63f35dcee65829893dd493011b5f2764552677bd
-  data.tar.gz: ee5e61767e56179bf2aec29c54ca81d0f3c6b3ad7d1788498bb45b84ee5f7d3b6dcb613cc5f297dbdcb4eae32f4005b594a6a7a562d51c9d1904b646537976cb
+  metadata.gz: 71fd179510efac57fdf0222cebb84bea43d678df1672a3281ed431ed11f7af1ada36de3ddfd832b2ccbdeb619ceb789f4a6f3d98671d8a7a830fababfef0c172
+  data.tar.gz: 20a5df4c74fbda09d2801d777ce4d205e696110216ed5d85107126228808a4da6074189a29c0a7da2ebb6f1d7a6af86877b7aa528eb063f2795f0b4d0e546229

data/CHANGELOG

@@ -1,3 +1,6 @@
+- 0.21.1.5 - 08-06-2026
+  - Fix a process abort when a reply value is still not cloneable after the safe-context filter runs (e.g. it contains a `Symbol`, which the filter passes through). The filter slow path returned `false` without re-throwing, so its local `TryCatch` cleared the pending exception and the caller hit `assert(try_catch.HasCaught())`. The clone failure is now propagated and surfaced as the usual `{"error" => "… could not be cloned"}` response. (Latent before 0.21.1.4; the 0.21.1.4 filter memoization made the path reachable for cyclic values.)
+
 - 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
 

data/README.md

@@ -483,7 +483,9 @@ Without `drain()` the order would be `["before", "after", "microtask"]` because
 
 When the drain has to happen from within JavaScript itself — for example between each listener in a synchronous `dispatchEvent` chain — the same checkpoint is available to JS as `drainMicrotasks()`. It runs inline on the V8 thread without the Ruby ↔ V8 round-trip, so no `attach` is required.
 
-It is exposed through an opt-in **host namespace** — a single object (in the spirit of Deno's `Deno` or Bun's `Bun`) that mini_racer hangs its non-standard helpers off. Pass `host_namespace:` to enable it; by default nothing is injected and the global stays clean:
+### Host namespace
+
+`mini_racer-csim` exposes its non-standard JS-callable helpers through a single opt-in object (in the spirit of Deno's `Deno` or Bun's `Bun`) rather than as bare globals, so allowing them is a one-time decision and `globalThis` otherwise stays clean. Pass `host_namespace:` to enable it; by default nothing is injected:
 
 ```ruby
 context = MiniRacerCsim::Context.new(host_namespace: "MiniRacer")
@@ -498,6 +500,8 @@ context.eval("log")
 # => ["before", "microtask", "after"]
 ```
 
+Beyond `drainMicrotasks()`, the namespace also carries the per-frame-realm JS helpers — `realmGlobal(id)`, `realmOf(fn)`, and `onUnhandledRejection(fn)` (see the **Per-frame realms** entry above). They live here for the same reason, so cross-realm wiring *in JS* requires `host_namespace:`; realms driven from Ruby (`Context#create_realm` + `Realm#eval`/`call`) do not.
+
 `host_namespace:` accepts a String (the global name to use — it must be a valid JavaScript identifier), `true` (the default name `"MiniRacer"`), or `nil`/`false` (the default — inject nothing). The namespace object is defined non-enumerable so it does not appear in `Object.keys(globalThis)`, while its methods are ordinary properties discoverable via `Object.keys(MiniRacer)`. Like `perform_microtask_checkpoint`, `drainMicrotasks()` is a no-op while a microtask checkpoint is already in progress, and it lets watchdog/out-of-memory termination propagate to the enclosing `eval`/`call`.
 
 ### ES modules

data/ext/mini_racer_csim_extension/mini_racer_v8.cc

@@ -273,9 +273,26 @@ bool reply(State& st, v8::Local<v8::Value> v)
         return false;
     }
     Serialized serialized(st, v);
-    if (serialized.data)
+    if (serialized.data) {
         v8_reply(st.ruby_context, serialized.data, serialized.size);
-    return serialized.data != nullptr; // exception pending if false
+        return true;
+    }
+    // The filtered value still is not serializable — e.g. it contains a Symbol,
+    // which the filter passes through unchanged (it only drops functions).
+    // ValueSerializer may signal this by returning false WITHOUT throwing, in
+    // which case the bare `return false` below would propagate no exception:
+    // this TryCatch would clear it on destruction and the caller would hit
+    // `assert(try_catch.HasCaught())`. Maintain the invariant that reply()==false
+    // implies a pending, propagated exception — synthesize a clone error when
+    // none was thrown — so the caller returns the {"error": ...} response
+    // instead of aborting. The "could not be cloned" wording matches the
+    // heuristic in the 3-arg reply().
+    if (!try_catch.HasCaught()) {
+        st.isolate->ThrowException(v8::Exception::Error(
+            v8::String::NewFromUtf8Literal(st.isolate, "value could not be cloned")));
+    }
+    try_catch.ReThrow();
+    return false;
 }
 
 bool reply(State& st, v8::Local<v8::Value> result, v8::Local<v8::Value> err)

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.4"
+  VERSION = "0.21.1.5"
   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.4
+  version: 0.21.1.5
 platform: ruby
 authors:
 - Keita Urashima