mini_racer_rs 1.0.1 → 1.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: aeb35a422c03ec1f8366cc6794daefa2cbacd69d8e113c0e1cb1d369a274cf07
-  data.tar.gz: e672b7b588ec4eb10e95317af8eea2a5d03aaf7340200b70c307f6a74e988606
+  metadata.gz: dfd3987a3aebfd95013e67e552b2041b991fd7b0da4933f98ae92335f7caa3a4
+  data.tar.gz: 51ccdf38b39ab3f33f771f3cf7fdbd7422bda16037e8bc026cfd64dc6fdfbdae
 SHA512:
-  metadata.gz: 95f7422df0c8a5f74675fdcdcf9ba1d9e9c3e7278a67b1bd57a4964ee38a022570b2b968db0a3b81af8e6320b2d2bbb9d98489bdfd87b3a8f78edb5971a595f9
-  data.tar.gz: 5288702eb114fe82c194a5071e15c99f36e6d2dbfb7c6541813074b89fb774a446dd84f4198c494306401a26cfda8932a082d8904f28bd468f01cda11bc29dcd
+  metadata.gz: ca5321d48ea8913f86d8966b6f475857d11b962c1f109e15f049fed590a517233f2cc3772fad99a9ff189f110995d3b2a0268f54e9f90aa48fcdd3cc1ac2b916
+  data.tar.gz: 3c2a6c804148cfd0bdf198d5a63ef4b440d63c667d7468ab646e8b85b277eb15ec0fa26f5d9877f426c01427e8f841723cf31d00bec0ee546881c895d373c61b

data/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+## [1.0.3](https://github.com/andymai/mini_racer_rs/compare/v1.0.2...v1.0.3) (2026-10-18)
+
+
+### Performance
+
+* **decode:** intern hash keys, build each Hash at once, and transcode strings in one pass ([#87](https://github.com/andymai/mini_racer_rs/issues/87)) ([220bfe7](https://github.com/andymai/mini_racer_rs/commit/220bfe7e6a9c8fa5a38befefcb4995756c377944))
+
+## [1.0.2](https://github.com/andymai/mini_racer_rs/compare/v1.0.1...v1.0.2) (2026-10-18)
+
+
+### Performance
+
+* **engine:** share one copy of a large repeated source across contexts ([#85](https://github.com/andymai/mini_racer_rs/issues/85)) ([9fe050a](https://github.com/andymai/mini_racer_rs/commit/9fe050a7a612314264e255f9bcc40ca146e93571))
+
 ## [1.0.1](https://github.com/andymai/mini_racer_rs/compare/v1.0.0...v1.0.1) (2026-10-18)
 
 

data/COMPATIBILITY.md

@@ -4,7 +4,7 @@
 
 | Engine | Done | Planned | Deviation |
 |---|---|---|---|
-| CRuby | 54 | 0 | 25 |
+| CRuby | 54 | 0 | 26 |
 | TruffleRuby | 58 | 0 | 0 |
 
 ## Surface
@@ -180,6 +180,12 @@ mini_racer_rs supports glibc Linux on x86_64 and aarch64, plus macOS on arm64 an
 
 Approved 2026-10-16.
 
+### `behavior:shared-source-heap-stats@ruby`
+
+`Context#heap_stats` can report large source text differently from mini_racer 0.22.1. When a context evaluates a source of 1 MiB or more that the process has evaluated before under the same file name and startup snapshot, mini_racer_rs gives V8 one copy shared by every such context and held outside V8's heap. It reports that source in `:external_memory` and omits it from `:used_heap_size`. V8 counts the shared source by its length in characters, so two-byte text reports half its bytes. mini_racer 0.22.1 reports `:external_memory` as 0 and counts each context's own copy in `:used_heap_size`. JavaScript sees the same source text in both gems. Contexts using `max_memory:` and sources that are not valid UTF-8 retain individual copies. Setting `MINI_RACER_RS_CODE_CACHE=off` before creating the first context disables sharing and makes `heap_stats` count every source as mini_racer does.
+
+Approved 2026-10-18.
+
 ### `behavior:snapshot-filter-context@ruby`
 
 mini_racer_rs returns the same filtered results from a snapshot-based context as from a context without a snapshot. In a context made from `MiniRacer::Snapshot.new("Object.getOwnPropertyDescriptors = () => ({})")`, `({a: 1, f() {}})` returns `{"a" => 1}`, and `[1, function() {}]` returns `[1]`. mini_racer 0.22.1 returns `nil` for both. Both gems copy results that V8 refuses to serialize as is through a filter in a separate safe context. mini_racer_rs keeps an untouched context in each snapshot blob and uses it for that filter, while mini_racer 0.22.1 builds the safe context from the snapshot, exposing the filter's built-ins to snapshot code. User code still sees the replacement in both gems: `Object.getOwnPropertyDescriptors({a: 1})` returns `{}`.

data/Cargo.lock

@@ -458,7 +458,7 @@ checksum = "cf8baf1c55e62ffcace7a9f06f4bd9cd3f0c4beb022d3b367256b91b87513d98"
 
 [[package]]
 name = "mini_racer_engine"
-version = "1.0.1"
+version = "1.0.3"
 dependencies = [
  "deno_core_icudata",
  "libc",
@@ -470,7 +470,7 @@ dependencies = [
 
 [[package]]
 name = "mini_racer_rs"
-version = "1.0.1"
+version = "1.0.3"
 dependencies = [
  "magnus",
  "mini_racer_engine",
@@ -481,7 +481,7 @@ dependencies = [
 
 [[package]]
 name = "mini_racer_sync"
-version = "1.0.1"
+version = "1.0.3"
 dependencies = [
  "loom",
  "thiserror",
@@ -489,7 +489,7 @@ dependencies = [
 
 [[package]]
 name = "mini_racer_wire"
-version = "1.0.1"
+version = "1.0.3"
 dependencies = [
  "proptest",
  "thiserror",

data/Cargo.toml

@@ -14,7 +14,7 @@ license = "MIT"
 publish = false
 repository = "https://github.com/andymai/mini_racer_rs"
 rust-version = "1.91"
-version = "1.0.1"
+version = "1.0.3"
 
 [workspace.dependencies]
 deno_core_icudata = "=0.78.0"

data/README.md

@@ -183,7 +183,7 @@ bundle exec ruby bench/run.rb --cpus 0-7
 
 | Engine | Done | Planned | Deviation |
 |---|---|---|---|
-| CRuby | 54 | 0 | 25 |
+| CRuby | 54 | 0 | 26 |
 | TruffleRuby | 58 | 0 | 0 |
 
 See [COMPATIBILITY.md](COMPATIBILITY.md).

data/crates/mini_racer_engine/src/process/code_cache.rs

@@ -77,15 +77,26 @@ impl Drop for Building {
 
 /// How the runner compiles a large script.
 pub(crate) enum Plan {
-    /// From source: a first sighting, a key being built elsewhere, a refused
-    /// cache, or the cache is off.
-    Compile,
+    /// From source: a first sighting or the cache is off, or with its key, a
+    /// key being built elsewhere or a refused cache.
+    Compile(Option<Key>),
     /// From source, then a cache made once the dispatch is over.
     Make(Building),
     /// From the entry, falling back to source if V8 refuses it.
     Consume(Arc<Entry>, Key),
 }
 
+impl Plan {
+    /// The key of a script the process has seen before.
+    pub(crate) const fn key(&self) -> Option<&Key> {
+        match self {
+            Self::Compile(key) => key.as_ref(),
+            Self::Make(building) => Some(&building.key),
+            Self::Consume(_, key) => Some(key),
+        }
+    }
+}
+
 impl CodeCache {
     pub(crate) fn plan(
         &'static self,
@@ -94,7 +105,7 @@ impl CodeCache {
         snapshot: Option<u64>,
     ) -> Plan {
         if !enabled() {
-            return Plan::Compile;
+            return Plan::Compile(None);
         }
         let sighting = self.sighting(filename, source, snapshot);
         let seen = touch(&self.sightings, sighting, MAX_SIGHTINGS);
@@ -103,7 +114,7 @@ impl CodeCache {
             .iter()
             .any(|entry| entry.key.sighting == sighting);
         if !seen && !cached {
-            return Plan::Compile;
+            return Plan::Compile(None);
         }
         let key = Key {
             filename: filename.clone(),
@@ -121,9 +132,10 @@ impl CodeCache {
         }
         drop(entries);
         if lock(&self.refused).contains(&key) {
-            return Plan::Compile;
+            return Plan::Compile(Some(key));
         }
-        self.begin(key).map_or(Plan::Compile, Plan::Make)
+        self.begin(key.clone())
+            .map_or(Plan::Compile(Some(key)), Plan::Make)
     }
 
     /// Claims `key` for this context to build, unless another is building it.
@@ -300,7 +312,7 @@ mod tests {
         let cache = cache();
         assert!(matches!(
             cache.plan(&name(), &source(1), None),
-            Plan::Compile
+            Plan::Compile(None)
         ));
         cached(cache, &source(1), Box::from(&b"compiled"[..]));
         assert!(matches!(
@@ -309,11 +321,11 @@ mod tests {
         ));
         assert!(matches!(
             cache.plan(&name(), &source(1), Some(7)),
-            Plan::Compile
+            Plan::Compile(None)
         ));
         assert!(matches!(
             cache.plan(&JsText::from("other.js"), &source(1), None),
-            Plan::Compile
+            Plan::Compile(None)
         ));
     }
 
@@ -355,10 +367,9 @@ mod tests {
         let _ = cache.plan(&name(), &source(3), None);
         let first = cache.plan(&name(), &source(3), None);
         assert!(matches!(first, Plan::Make(..)));
-        assert!(matches!(
-            cache.plan(&name(), &source(3), None),
-            Plan::Compile
-        ));
+        let elsewhere = cache.plan(&name(), &source(3), None);
+        assert!(matches!(elsewhere, Plan::Compile(Some(_))));
+        assert!(elsewhere.key() == first.key());
         drop(first);
         assert!(matches!(
             cache.plan(&name(), &source(3), None),
@@ -402,7 +413,7 @@ mod tests {
         assert_eq!(cache.stored(), 1);
         assert!(matches!(
             cache.plan(&name(), &source(7), None),
-            Plan::Compile
+            Plan::Compile(Some(_))
         ));
         assert!(matches!(
             cache.plan(&name(), &source(6), None),

data/crates/mini_racer_engine/src/process/fork.rs

@@ -7,6 +7,7 @@ use mini_racer_sync::{ForkGate, Phase, lock};
 
 use crate::control::ControlBlock;
 use crate::process::code_cache::CodeCache;
+use crate::process::sources::Sources;
 use crate::process::tasks::TaskRegistry;
 use crate::process::watchdog::Watchdog;
 use crate::process::workers::Workers;
@@ -51,13 +52,14 @@ pub struct Generation {
     watchdog: Mutex<Option<Arc<Watchdog>>>,
     workers: Workers,
     code_cache: CodeCache,
+    sources: Sources,
     next: OnceLock<Box<Self>>,
 }
 
 static ROOT: OnceLock<Generation> = OnceLock::new();
 
 impl Generation {
-    fn new(epoch: u64, quiesced: bool, code_cache: CodeCache) -> Self {
+    fn new(epoch: u64, quiesced: bool, code_cache: CodeCache, sources: Sources) -> Self {
         Self {
             epoch,
             quiesced,
@@ -69,6 +71,7 @@ impl Generation {
             watchdog: Mutex::new(None),
             workers: Workers::default(),
             code_cache,
+            sources,
             next: OnceLock::new(),
         }
     }
@@ -94,6 +97,11 @@ impl Generation {
         &self.code_cache
     }
 
+    #[must_use]
+    pub const fn sources(&self) -> &Sources {
+        &self.sources
+    }
+
     pub fn register(&self, block: &Arc<ControlBlock>) {
         let mut blocks = lock(&self.blocks);
         blocks.retain(|block| block.strong_count() > 0);
@@ -115,7 +123,8 @@ impl Generation {
 
 pub fn current() -> &'static Generation {
     let epoch = epoch();
-    let mut generation = ROOT.get_or_init(|| Generation::new(epoch, true, CodeCache::default()));
+    let mut generation =
+        ROOT.get_or_init(|| Generation::new(epoch, true, CodeCache::default(), Sources::default()));
     while generation.epoch != epoch {
         let quiesced = generation.quiesced && generation.prepared.load(Ordering::SeqCst);
         let next = generation.epoch.saturating_add(1);
@@ -125,6 +134,7 @@ pub fn current() -> &'static Generation {
                 next,
                 quiesced,
                 parent.code_cache.inherited(),
+                parent.sources.inherited(),
             ))
         });
     }