moxml 0.5.103 → 0.5.105

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 566f2842eac4e01753965cb5f56eada53d6683b3ef6b576015eb4b0da572ae67
-  data.tar.gz: ccc15ef083358a018e1536983017c6681c70c11ccbb4678ac123e5b0b2c9aada
+  metadata.gz: 0c57aa62d4c524daa9096eb565711a4ebbed7525ba851cb6e327c98de842c4e4
+  data.tar.gz: 604a922868c9ba3fb656c41e84e7b2f848bdd20defcd5259d697da32e2fd7e0e
 SHA512:
-  metadata.gz: 2c29296b60bedabbc512c9f5c454b0195b3632dbf9952b0c397fc3bad6554135dd3096a04a34e1abddf9ce40088af4856b2ead7d8ca69a4eb251dc3db38286b3
-  data.tar.gz: fd1684a85d2ace2191b33bf996a12e8bea6cb535a340732cccee3318abc6649ba2e07548509d1469959e10ee711ecec650f8ec6b4525beb6a66e2c9a75a81c1d
+  metadata.gz: 80f4dc813942807549d58b93492e731dceeb3528e7cfcfd3f3a3d4c5fbd1a2c4d5b234cbc1cad9b8945efc1c280e4408b373ac8d0bf2e489051277b98716fc82
+  data.tar.gz: c91474a18a6877d6c0a012765f27c37c2ccd42260c810b130d62c3d69a6f55f2e08019a6472bb6f1b5d1f8649f99c3747f0139c72a80a606322850e56a98f134

data/README.adoc

@@ -1283,7 +1283,8 @@ reproduce:
 [source,shell]
 ----
 # One-time setup: install ragel + ruby-ll so the forks' gitignored
-# generated lexer/parser sources can be produced.
+# generated lexer/parser sources can be produced. ruby-ll must be
+# installed outside the bundle: vendor:prepare runs it outside Bundler.
 gem install ruby-ll --no-document
 # (Install ragel via your package manager, e.g. apt-get install ragel
 #  or brew install ragel.)

data/Rakefile

@@ -88,13 +88,17 @@ namespace :vendor do
        "#{ruby_ll}/lib/ll/parser.rll"],
     ]
 
-    generators.each do |cmd, output, source|
-      if File.exist?(output) && File.mtime(output) >= File.mtime(source)
-        next
+    # Under bundle exec, ruby-ll resolves to the vendored opal-ruby-ll
+    # binstub, which needs the not-yet-compiled libll.
+    Bundler.with_unbundled_env do
+      generators.each do |cmd, output, source|
+        if File.exist?(output) && File.mtime(output) >= File.mtime(source)
+          next
+        end
+
+        FileUtils.mkdir_p(File.dirname(output))
+        sh cmd
       end
-
-      FileUtils.mkdir_p(File.dirname(output))
-      sh cmd
     end
   end
 

data/lib/moxml/adapter/base.rb

@@ -131,11 +131,29 @@ module Moxml
         def children_accepts_entity_flag?
           return @children_accepts_entity_flag unless @children_accepts_entity_flag.nil?
 
-          kinds = %i[key keyrest keyreq]
+          # Opal records parameter data only when compiling with arity
+          # checking, which is off by default (opal-rspec turns it on, so
+          # the Opal spec run never sees this). Without it
+          # Method#parameters returns a bare JS undefined, which answers no
+          # method call (not even nil?), so it is matched with `case`,
+          # whose Module#=== tolerates it. Without the data the keyword is
+          # assumed: every built-in adapter's #children accepts it.
           @children_accepts_entity_flag =
-            method(:children).parameters.any? do |kind, _name|
-              kinds.include?(kind)
-            end
+            children_params_flag(method(:children).parameters)
+        end
+
+        # The decision over a parameters VALUE, so the Opal shape (a
+        # bare JS undefined, matched by `case`) is spec-able with real
+        # values. Without parameter data the keyword is assumed: every
+        # built-in adapter's #children accepts it.
+        KEYWORD_PARAM_KINDS = %i[key keyrest keyreq].freeze
+
+        def children_params_flag(params)
+          case params
+          when Array
+            params.any? { |kind, _| KEYWORD_PARAM_KINDS.include?(kind) }
+          else true
+          end
         end
 
         # Protocol-level native equality; engines with more than one

data/lib/moxml/adapter/leptris.rb

@@ -2008,7 +2008,16 @@ module Moxml
               )
             end
           else
-            node.unlink
+            begin
+              node.unlink
+            rescue ::Leptris::XML::Error => e
+              # "Not found" from unlink means the node is already in
+              # no parent list — detached (replace, children=, engine
+              # paths the wrapper layer cannot clear links for).
+              # Nokogiri's unlink is a no-op there; match it instead
+              # of crashing cleanup passes.
+              raise unless e.message.include?("Not found")
+            end
           end
         end
 

data/lib/moxml/attribute.rb

@@ -27,19 +27,28 @@ module Moxml
     end
 
     def value
-      val = @native.value.to_s
-      # Same guard as Element#text: entity-free documents skip the
-      # marker restore scans. The memo rides the owning element's
-      # (attr natives have no entity probe); a detached attribute
-      # wrapper falls back to the unconditional restore.
-      parent = @parent_node
-      if parent.nil? || parent.entity_bearing?
-        adapter.restore_entities(val)
-      else
-        val
+      # Engines allocate a fresh String per read (Nokogiri attr.value);
+      # hydration walks re-read the same wrappers, so the value is
+      # memoized. A held wrapper recomputes when its owner element's
+      # value generation moves (element-side writes bump it locally —
+      # a context-wide bump would evict every wrapper per bulk-build
+      # write). Parentless wrappers (xpath ResultAttrs) are immutable
+      # captures and memo unconditionally.
+      if @parent_node
+        generation = @parent_node.attribute_value_generation
+        @value = nil if @value_gen != generation
+        @value_gen = generation
+      end
+      @value ||= begin
+        val = @native.value.to_s
+        parent = @parent_node
+        if parent.nil? || parent.entity_bearing?
+          adapter.restore_entities(val)
+        else
+          val
+        end
       end
     end
-
     alias content value
 
     # Returns raw native value without entity marker restoration.
@@ -55,6 +64,7 @@ module Moxml
       else
         @parent_node&.invalidate_attribute_value_cache!
       end
+      @value = nil
       adapter.set_attribute_value(@native, new_value)
     end
 

data/lib/moxml/element.rb

@@ -189,6 +189,7 @@ module Moxml
                   raise ArgumentError, "children= accepts String, Node, or Array"
                 end
       adapter.replace_children(@native, natives)
+      @text = nil
       context.bump_children_generation
       invalidate_children_cache!
     end
@@ -329,16 +330,30 @@ module Moxml
     end
 
     def text
-      val = adapter.text_content(@native)
-      # Entity-free documents (the common case) skip the marker
-      # restore scans entirely — the per-wrapper entity_bearing?
-      # memo rides the adapter's serialize generation.
-      entity_bearing? ? adapter.restore_entities(val) : val
+      # Generation-gated memo: engines allocate a fresh String per read
+      # (Nokogiri node.text), and hydration walks re-read the same
+      # wrappers — Element#text was a top string allocation site in
+      # large-document metanorma compiles. Text only changes through
+      # this wrapper's writers (text=/children=), all of which clear
+      # the memo.
+      @text = nil if @text_gen != context.children_generation
+      @text_gen = context.children_generation
+      @text ||= begin
+        val = adapter.text_content(@native)
+        # Entity-free documents (the common case) skip the marker
+        # restore scans entirely — the per-wrapper entity_bearing?
+        # memo rides the adapter's serialize generation.
+        entity_bearing? ? adapter.restore_entities(val) : val
+      end
     end
 
     alias content text
 
     def text=(content)
+      @text = nil
+      # A text write replaces the whole child list; the generation
+      # bump also retires @elements/@children memos across wrappers.
+      context.bump_children_generation
       adapter.set_text_content(@native, normalize_xml_value(content))
       invalidate_children_cache!
     end
@@ -363,6 +378,11 @@ module Moxml
     def inner_xml=(xml)
       wrapper = "_moxml_inner_#{Process.pid}_#{object_id}"
       doc = context.parse("<#{wrapper}>#{xml}</#{wrapper}>")
+      @text = nil
+      # Whole-list replacement rides the children generation like
+      # children= does (moxml#310) — text and elements memos across
+      # wrappers recompute on next read.
+      context.bump_children_generation
       adapter.replace_children(@native, doc.root.children.map(&:native))
       invalidate_children_cache!
     end
@@ -415,13 +435,23 @@ module Moxml
     # too when the attribute set changes. No context generation
     # bump — that would evict every wrapper's caches document-wide
     # on each write of a bulk build.
+    # Local counter for held Attribute wrappers: value writes bump it
+    # (no context-wide bump — bulk builds would evict every wrapper's
+    # caches per write), and Attribute#value recomputes when its
+    # owner's counter moves.
+    def attribute_value_generation
+      @attribute_value_generation ||= 0
+    end
+
     def invalidate_attribute_value_cache!
       @attribute_cache = nil
+      @attribute_value_generation = attribute_value_generation + 1
     end
 
     def invalidate_local_attribute_cache!
       @attributes = nil
       @attribute_cache = nil
+      @attribute_value_generation = attribute_value_generation + 1
     end
 
     # Called by the namespace-scoped attribute paths (xmlns writes

data/lib/moxml/node.rb

@@ -254,6 +254,11 @@ module Moxml
       context.bump_children_generation
       invalidate_parent_children_cache!
       adapter.remove(@native)
+      # The wrapper-side parent link dies with the attach — leaving it
+      # stale makes a later #remove's guard trust a parent that no
+      # longer exists and push an already-detached node into the
+      # engine (leptris: "Not found").
+      @parent_node = nil
       invalidate_children_cache!
       # The detached subtree left its declaring ancestors behind
       invalidate_namespace_cache!
@@ -269,6 +274,9 @@ module Moxml
       node = prepare_node(node)
       invalidate_parent_children_cache!
       adapter.replace(@native, node.native)
+      # Self is detached by the replace — drop the stale parent link
+      # (see #remove).
+      @parent_node = nil
       invalidate_children_cache!
       self
     end

data/lib/moxml/text.rb

@@ -5,8 +5,14 @@ module Moxml
     include Node
 
     def content
-      text = raw_content
-      entity_bearing? ? adapter.restore_entities(text) : text
+      # Engines allocate a fresh String per read (Nokogiri
+      # node.text); hydration walks re-read the same text wrappers —
+      # this read was the single largest string allocation site in
+      # large-document metanorma compiles. Cleared by content=.
+      @content ||= begin
+        text = raw_content
+        entity_bearing? ? adapter.restore_entities(text) : text
+      end
     end
 
     # Returns raw content without entity marker restoration.
@@ -15,6 +21,7 @@ module Moxml
     end
 
     def content=(text)
+      @content = nil
       adapter.set_text_content(@native, normalize_xml_value(text))
     end
 

data/lib/moxml/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Moxml
-  VERSION = "0.5.103"
+  VERSION = "0.5.105"
 end

data/spec/moxml/adapter/base_spec.rb

@@ -46,5 +46,19 @@ RSpec.describe Moxml::Adapter::Base do
       expect(Moxml::Adapter::Leptris.children_accepts_entity_flag?).to be(true)
       expect(Moxml::Adapter::Nokogiri.children_accepts_entity_flag?).to be(true)
     end
+
+    it "assumes the keyword when Method#parameters returns no data" do
+      # Opal without arity checking answers a bare JS undefined for
+      # parameters — not even nil?. Exercise the decision through the
+      # real values the probe can see.
+      probe = described_class.children_params_flag(nil)
+      expect(probe).to be(true)
+      expect(described_class.children_params_flag([%i[req node]])).to be(false)
+      expect(
+        described_class.children_params_flag(
+          [%i[req node], %i[key entity_bearing]],
+        ),
+      ).to be(true)
+    end
   end
 end

data/spec/moxml/at_search_spec.rb

@@ -383,4 +383,52 @@ RSpec.describe "Nokogiri-compatible node sugar" do
       expect(doc.root.inner_xml).to eq("<a>x</a><b/>")
     end
   end
+
+  describe "Node#remove after replace with a parent link" do
+    it "is a no-op, not an engine error (stale wrapper parent link)" do
+      doc = Moxml.parse("<r><term><p>a</p><note>n</note></term></r>")
+      # children walks are parent-aware: the wrapper carries @parent_node
+      p_node = doc.at("//term").children.find { |c| c.name == "p" }
+      t = doc.create_element("definition")
+      p_node.replace(t)
+      expect { t << p_node.remove }.not_to raise_error
+      expect(doc.at("//term/definition/p")).not_to be_nil
+      expect(doc.at("//term/definition/p").text).to eq("a")
+    end
+  end
+
+  describe "wrapper read memos" do
+    it "Element#text memoizes and invalidates on text=/children=" do
+      doc = Moxml.parse("<r><a>x<b>y</b>z</a></r>")
+      a = doc.at("//a")
+      first = a.text
+      expect(a.text).to equal(first)
+      a.text = "new"
+      expect(a.text).to eq("new")
+      a.children = "<c>w</c>"
+      expect(a.text).to eq("w")
+    end
+
+    it "Attribute#value memoizes and invalidates on value=" do
+      doc = Moxml.parse(%(<r a="1"/>))
+      attr = doc.root.attributes.first
+      first = attr.value
+      expect(attr.value).to equal(first)
+      attr.value = "2"
+      expect(attr.value).to eq("2")
+      expect(doc.root["a"]).to eq("2")
+    end
+  end
+
+  describe "Text#content memo" do
+    it "memoizes and invalidates on content=" do
+      doc = Moxml.parse("<r>hello</r>")
+      t = doc.root.children.first
+      first = t.content
+      expect(t.content).to equal(first)
+      expect(t.text).to equal(first)
+      t.content = "bye"
+      expect(t.content).to eq("bye")
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: moxml
 version: !ruby/object:Gem::Version
-  version: 0.5.103
+  version: 0.5.105
 platform: ruby
 authors:
 - Ribose Inc.
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-10-02 00:00:00.000000000 Z
+date: 2026-10-03 00:00:00.000000000 Z
 dependencies: []
 description: |
   Moxml is a unified XML manipulation library that provides a common API