fulgur_chart 0.7.0 → 0.9.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fc454d38dbab809eed4d7e7e22edc3630e74f5042308ceec29b6ed7042515ace
-  data.tar.gz: 770d479abd84d965265a031ec44250d47cf5d65a9d365d27593cf11fd87f0770
+  metadata.gz: 06d802ba139417a89422869f273ca4006b43acab13337bf45be42494ea02444f
+  data.tar.gz: 5301f5ac659389f8c334960e7239c1de6fa21e269c6929f690ab36a4e0232532
 SHA512:
-  metadata.gz: '09ecf263ac296b185c6c5c9fd04454cd518898af8b165f7a62ecb87f0ab0f3c702f66eb24e2d3f4d1eed072eba56d22e4ebee3fe92df0328397a666a54843207'
-  data.tar.gz: 8dded30443c6f3af4c5eb4490f53b154390c36f2469652bf5ebb3d2882a58212057e2b98d03aaea18b264181ca5ab7b3b0e62c81ca8f26ea006850e454fcd9bf
+  metadata.gz: 6963cd4c72110e3cf2e112414a3b130e5a68fcc3af6cea3f59bf71bb95967d5bc850dc10228b6a81ee3054f126c13c0bcbf76257e5a341c969525459e03359d7
+  data.tar.gz: c377c3c2ea5ab1fe325d2775cc3d9867cbfbb1847b94c867331bf4feeafa0ca51fa5d53aef63af8b6c30731472de64721ed63c6f350c6e45e1e005f0fd493ffd

data/ext/fulgur_chart/Cargo.toml

@@ -1,7 +1,7 @@
 [package]
 name = "fulgur_chart"
-version = "0.7.0"
-edition = "2021"
+version = "0.9.0"
+edition = "2024"
 publish = false
 
 [lib]
@@ -9,7 +9,11 @@ crate-type = ["cdylib"]
 
 [dependencies]
 magnus = "0.7"
-fulgur-chart = "0.7.0"
+# Raw libruby bindings for rb_thread_call_without_gvl (magnus 0.7 does not wrap it).
+# Already pulled in transitively by magnus; declaring it directly unifies to the same
+# version and makes the symbol importable.
+rb-sys = "0.9"
+fulgur-chart = "0.9.0"
 serde = { version = "1", features = ["derive"] }
 serde_json = "1"
 schemars = { version = "1.2", features = ["preserve_order"] }

data/ext/fulgur_chart/src/lib.rs

@@ -1,9 +1,8 @@
-use fulgur_chart::guard::{validate_spec, InputLimits};
+use fulgur_chart::guard::{InputLimits, validate_spec};
 use magnus::{
-    function,
+    Error, ExceptionClass, RHash, RString, Ruby, Value, function,
     prelude::*,
     scan_args::{get_kwargs, scan_args},
-    Error, ExceptionClass, RHash, RString, Ruby, Value,
 };
 
 // --- error helpers (classification is by CALL SITE, never by parsing the message) ---
@@ -167,6 +166,112 @@ fn build_ir(
     Ok(ir)
 }
 
+// --- GVL release ---
+
+/// Run `func` with the GVL released, returning `Some(value)` — or `None` if a pending
+/// interrupt prevented `func` from running at all (see below).
+///
+/// Uses `rb_thread_call_without_gvl2`, NOT `rb_thread_call_without_gvl`. The v2 variant does
+/// NOT process interrupts on GVL re-acquisition, so control ALWAYS returns to this Rust frame
+/// after `func` runs: it never raises / longjmps across the `extern "C"` boundary. The v1
+/// variant checks (and handles) pending interrupts right after `func` returns — that handling
+/// can raise, longjmp-ing past this frame and the caller's, skipping our box reclamation and
+/// the owned Rust data's Drop glue (a leak on every interrupted render, and an unsound
+/// longjmp across Rust frames). v2 avoids that: the render runs uninterrupted and a pending
+/// interrupt is honored by Ruby at its next checkpoint, after this call returns.
+///
+/// `func` MUST NOT call any Ruby C API — it executes while this thread does not hold the GVL,
+/// so building VALUEs / raising / allocating Ruby objects would be unsound. (If a global
+/// allocator that calls back into Ruby's GC — e.g. rb-sys's tracking allocator — were ever
+/// installed, even plain Rust heap allocation here would become unsound. None is today.)
+///
+/// `None`: v2 checks for a pending interrupt BEFORE releasing the GVL and, if one is set,
+/// returns without ever calling `func`. Callers fall back to running the work under the GVL,
+/// leaving the still-pending interrupt for Ruby to raise once control returns to it.
+///
+/// A panic inside `func` is caught and re-raised here so it never crosses `extern "C"` (UB).
+fn nogvl<F, R>(func: F) -> Option<R>
+where
+    F: FnOnce() -> R,
+{
+    use std::ffi::c_void;
+
+    unsafe extern "C" fn call<F, R>(arg: *mut c_void) -> *mut c_void
+    where
+        F: FnOnce() -> R,
+    {
+        // `arg` is the Box<F> leaked below; reconstitute and consume it exactly once.
+        let func = *unsafe { Box::from_raw(arg as *mut F) };
+        let result = std::panic::catch_unwind(std::panic::AssertUnwindSafe(func));
+        Box::into_raw(Box::new(result)) as *mut c_void
+    }
+
+    let arg = Box::into_raw(Box::new(func)) as *mut c_void;
+    // SAFETY: `call::<F, R>` has the rb_thread_call_without_gvl2 callback signature; `arg` is
+    // a valid Box<F>. NULL ubf → the render itself is never interrupted mid-flight.
+    let ret = unsafe {
+        rb_sys::rb_thread_call_without_gvl2(Some(call::<F, R>), arg, None, std::ptr::null_mut())
+    };
+    if ret.is_null() {
+        // v2 declined to run `func` (interrupt pending at entry), so `call` never ran and the
+        // closure box was not consumed — reclaim it here to avoid leaking it. (`func` always
+        // returns a non-null box pointer, so a null return unambiguously means "did not run".)
+        drop(unsafe { Box::from_raw(arg as *mut F) });
+        return None;
+    }
+    // `ret` is the Box<thread::Result<R>> leaked at the end of `call`.
+    match *unsafe { Box::from_raw(ret as *mut std::thread::Result<R>) } {
+        Ok(value) => Some(value),
+        Err(payload) => std::panic::resume_unwind(payload),
+    }
+}
+
+/// Output of the GVL-free render region. Carries only plain data (no Ruby VALUEs) so it
+/// can cross back into the VM once the GVL is re-acquired.
+enum Rendered {
+    Svg(String),
+    Png(Vec<u8>),
+}
+
+/// How a render failure is classified once back under the GVL. Mirrors the original
+/// call-site classification: SVG font/render failures → ParseError, raster failures →
+/// RenderError, unknown format → ParseError.
+enum RenderFail {
+    Parse(String),
+    Render(String),
+    UnsupportedFormat(String),
+}
+
+/// The heavy, Ruby-free rendering. Called inside `nogvl`, so it must touch ONLY the
+/// owned/borrowed plain Rust inputs (`ir`, `font`, `scale`) and never the Ruby C API.
+fn render_pure(
+    ir: &fulgur_chart::ir::ChartSpec,
+    format: &str,
+    font: Option<&[u8]>,
+    scale: f32,
+) -> Result<Rendered, RenderFail> {
+    match format {
+        "svg" => {
+            // Font present → render_chart_with_font (Err → ParseError on the SVG path);
+            // else the bundled-font render.
+            let svg = match font {
+                Some(bytes) => fulgur_chart::render::render_chart_with_font(ir, bytes)
+                    .map_err(RenderFail::Parse)?,
+                None => fulgur_chart::render::render_chart(ir),
+            };
+            Ok(Rendered::Svg(svg))
+        }
+        "png" => {
+            let fb = font.unwrap_or(fulgur_chart::font::DEFAULT_FONT);
+            // Invalid font on the image path → RenderError (the SVG path maps this to ParseError).
+            let png = fulgur_chart::raster_direct::render_chart_to_png(ir, scale, fb)
+                .map_err(RenderFail::Render)?;
+            Ok(Rendered::Png(png))
+        }
+        other => Err(RenderFail::UnsupportedFormat(other.to_string())),
+    }
+}
+
 // --- public API: render (low-level primitive; the FulgurChart::Builder is the intended API) ---
 
 /// `FulgurChart.render(spec_json, format, **opts)` → String.
@@ -182,28 +287,25 @@ fn render(ruby: &Ruby, args: &[Value]) -> Result<RString, Error> {
     let opts = parse_opts(ruby, scanned.keywords)?;
     let ir = build_ir(ruby, &spec_json, &opts)?;
 
-    match format.as_str() {
-        "svg" => {
-            // Font present → render_chart_with_font (Err → ParseError on the SVG path);
-            // else the bundled-font render.
-            let svg = match &opts.font {
-                Some(bytes) => fulgur_chart::render::render_chart_with_font(&ir, bytes)
-                    .map_err(|e| parse_err(ruby, e))?,
-                None => fulgur_chart::render::render_chart(&ir),
-            };
-            Ok(ruby.str_new(&svg)) // UTF-8 String
-        }
-        "png" => {
-            let fb: &[u8] = opts
-                .font
-                .as_deref()
-                .unwrap_or(fulgur_chart::font::DEFAULT_FONT);
-            // Invalid font on the image path → RenderError (the SVG path maps this to ParseError).
-            let png = fulgur_chart::raster_direct::render_chart_to_png(&ir, opts.scale, fb)
-                .map_err(|e| render_err(ruby, e))?;
-            Ok(ruby.str_from_slice(&png)) // ASCII-8BIT (BINARY) String
-        }
-        other => Err(parse_err(
+    // The heavy rendering touches no Ruby objects (ir/font are owned plain data; font was
+    // already copied out of the VM in parse_opts), so run it with the GVL released. Other
+    // Ruby threads — including other renders — then run truly in parallel. Ruby strings and
+    // exceptions are built BELOW, after the GVL is re-acquired; the closure must not touch
+    // the Ruby C API.
+    let font = opts.font; // Option<Vec<u8>>, owned
+    let scale = opts.scale;
+    // Release the GVL for the heavy render. If an interrupt is already pending, nogvl returns
+    // None without running the closure; fall back to rendering under the GVL and let Ruby
+    // raise the still-pending interrupt when control returns from this method.
+    let result = nogvl(|| render_pure(&ir, format.as_str(), font.as_deref(), scale))
+        .unwrap_or_else(|| render_pure(&ir, format.as_str(), font.as_deref(), scale));
+
+    match result {
+        Ok(Rendered::Svg(svg)) => Ok(ruby.str_new(&svg)), // UTF-8 String
+        Ok(Rendered::Png(png)) => Ok(ruby.str_from_slice(&png)), // ASCII-8BIT (BINARY) String
+        Err(RenderFail::Parse(m)) => Err(parse_err(ruby, m)),
+        Err(RenderFail::Render(m)) => Err(render_err(ruby, m)),
+        Err(RenderFail::UnsupportedFormat(other)) => Err(parse_err(
             ruby,
             format!("unsupported format '{other}' (supported: svg, png)"),
         )),
@@ -223,7 +325,7 @@ fn schema(ruby: &Ruby, dsl: Value) -> Result<String, Error> {
             return Err(parse_err(
                 ruby,
                 format!("unsupported DSL '{other}' (supported: chartjs, vegalite)"),
-            ))
+            ));
         }
     };
     serde_json::to_string(&s).map_err(|e| render_err(ruby, format!("schema serialization: {e}")))

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: fulgur_chart
 version: !ruby/object:Gem::Version
-  version: 0.7.0
+  version: 0.9.0
 platform: ruby
 authors:
 - Fulgur