lutaml-model 0.8.51 → 0.8.52

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bd76b1c0377a6e86d547f5ca29fe5d164d390bc7f1d88ba7051a5930b119911f
-  data.tar.gz: 22e2dea3ad97c502647a49ec748113e939234b9136c02634b99c722ce443df10
+  metadata.gz: 92e0193fa58094112522b5a7717aed2f568d77096eb0ae3bc1a246be28c05bf0
+  data.tar.gz: 20dcd124c22c695cc0a8fca65d04422ac84f468515ef153d909e5a6dcf5eac67
 SHA512:
-  metadata.gz: '0883bebaf58efeeda222ddc83f74661f396184786944f48f1a9ae2ecb5cbb4c15059cb5dd93df791d101acd734ca597e849b5c0d4488b104c59b7afa01f05e40'
-  data.tar.gz: 46e13698b95bff244e8d31627bbd3c1fd2a9fc667b3741a9707bac2a85256be426a9606c3c38ac848722ae668c488da39643154f5387aa6a60ee006dd57056b3
+  metadata.gz: 0cc3e71dceb6ae87db00453efbd905de28dc10d528cd38426b29c2c0da1a67d64d41a59095e45a9da1daa97888a4d001c8211b0ad64a3a011ef46949e71206a6
+  data.tar.gz: c444800a986c0e34bc1502fd671f53d7ff4776377d89b62225b857c923f4b97058042094145caf0d0a54db37576bedd546a733e114a444118ff2e13241092d83

data/docs/_guides/native-engines.adoc

@@ -57,9 +57,31 @@ RUBY_YJIT_ENABLE=1 ruby app.rb
 RubyVM::YJIT.enable
 ----
 
-The opt-in XML plan fast path (see below) moves work out of Ruby and
-into the engine, which shrinks the YJIT win — enabling both is still
-net faster than either alone.
+The XML plan fast path moves work out of Ruby and into the engine,
+which shrinks the YJIT win — enabling both is still net faster than
+either alone.
+
+== The XML plan fast path (default on)
+
+When the XML adapter resolves to leptris, XML (de)serialization
+compiles the model's mapping into a leptris descriptor plan and runs
+the whole document through one native pass — measured 6.7x parse and
+4x serialize with -76% / -80% allocations on the plan-path corpus.
+Models the plan cannot express exactly (same-name discriminator
+partitions, namespace-qualified models with deferred-rule shapes) and
+shapes with custom semantics (custom methods, polymorphism, unions,
+ordered children) hydrate through the interpretive pipeline or a
+per-subtree fragment of it, unchanged.
+
+The path is on by default and is a no-op without leptris (standard
+adapters keep the classic pipeline). Opt out for the whole process:
+
+[source,ruby]
+----
+Lutaml::Model::Config.configure do |config|
+  config.xml_plan_fast_path = false
+end
+----
 
 == Selecting adapters explicitly
 

data/lib/lutaml/model/configuration.rb

@@ -25,7 +25,7 @@ module Lutaml
         @default_register = :default
         @conversion_cache = nil
         @configured = false
-        @xml_plan_fast_path = false
+        @xml_plan_fast_path = true
       end
 
       def configure

data/lib/lutaml/model/version.rb

@@ -2,6 +2,6 @@
 
 module Lutaml
   module Model
-    VERSION = "0.8.51"
+    VERSION = "0.8.52"
   end
 end

data/lib/lutaml/xml/plan_compiler.rb

@@ -39,7 +39,21 @@ module Lutaml
           key = [model_class, register]
           return PLAN_CACHE[key] if PLAN_CACHE.key?(key)
 
-          PLAN_CACHE[key] = build(model_class, register)
+          # Cycle guard: a self-referential model (JATS sec-in-sec) must
+          # resolve to nil — the interpretive pipeline owns it. The guard
+          # is THREAD-LOCAL (a shared in-progress set races: a concurrent
+          # same-key compile would cache false permanently — the #828
+          # lesson); the nil at the cycle point is NOT cached — the
+          # outermost build completes and caches the real verdict.
+          stack = (Thread.current[:plan_compiler_stack] ||= [])
+          return nil if stack.include?(key)
+
+          stack.push(key)
+          begin
+            PLAN_CACHE[key] = build(model_class, register)
+          ensure
+            stack.pop
+          end
         end
 
         private

data/spec/lutaml/xml/plan_fast_path_spec.rb

@@ -38,8 +38,42 @@ RSpec.describe "XML plan fast path" do
     Lutaml::Model::Config.instance.xml_plan_fast_path = old
   end
 
-  it "is opt-in and off by default" do
-    expect(Lutaml::Model::Configuration.new.xml_plan_fast_path).to be(false)
+  # Real-corpus crash (niso-jats): sec-in-sec nesting is a
+  # self-referential model — compile must resolve it to nil instead of
+  # recursing to SystemStackError.
+  it "opts self-referential models out instead of recursing" do
+    node = Class.new(Lutaml::Model::Serializable) do
+      attribute :text, :string
+      xml do
+        element "n"
+        map_content to: :text
+      end
+    end
+    node.attribute :child, node
+    node.xml do
+      element "n"
+      map_content to: :text
+      map_element "child", to: :child
+    end
+    stub_const("PlanFastPath::SelfRef", node)
+
+    expect(Lutaml::Xml::PlanCompiler.compile(node, :default)).to be_nil
+
+    xml = "<n>#{'<n>t</n>' * 200}</n>"
+    parsed = node.from_xml(xml)
+    expect(parsed).to be_a(node)
+  end
+
+  it "is on by default and can be disabled" do
+    expect(Lutaml::Model::Configuration.new.xml_plan_fast_path).to be(true)
+
+    # Opt-out restores the interpretive pipeline for the whole process.
+    Lutaml::Model::Config.instance.xml_plan_fast_path = false
+    item = item_class.new(id: 1, name: "a", tags: %w[x])
+    parsed = root_class.from_xml(root_class.new(item: [item]).to_xml)
+    expect(parsed.item.first.id).to eq(1)
+  ensure
+    Lutaml::Model::Config.instance.xml_plan_fast_path = true
   end
 
   it "hydrates equal to the interpretive path" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lutaml-model
 version: !ruby/object:Gem::Version
-  version: 0.8.51
+  version: 0.8.52
 platform: ruby
 authors:
 - Ribose Inc.