moxml 0.5.41 → 0.5.42

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d96e7e77957533fdb65727e5d244d038a6947b30a46563463b995114e4594faf
-  data.tar.gz: 11940d59f145433f37e0a5ea286055cf6d30167614291d007ff2d41dc68606a5
+  metadata.gz: d447f74ea4b1c546e83fbb94819c6043a1fa48e0f9932ac7202009cecff63e1e
+  data.tar.gz: 4c33c0af41d9c628ecc993d7015d4e827d1d451b266dd7ad4145f00b322670a3
 SHA512:
-  metadata.gz: 25d5f4a19555bd8802f978a3586d2d4c7fc9770d918fbfee1777203d548c6ed8d0d67fa42fd4dcddc047675f2034482f1323542379fe61193358209a4f8670d0
-  data.tar.gz: be203488271f744cb405f70a48af237408870c433a7b5922c8cafb1f303b2a4a451d0b0f6c3c80ce249a9fe75ec98904c330ff3a0cda212587fb231aac255f53
+  metadata.gz: ebdc376975c3893facd06d793a04eebdfa9387814b131b9f0c5aecbd047fb57dcfadba80c40eee8ff60e52b4a7cbf1e886e67909b5f9b46623ca91b75cdb04b5
+  data.tar.gz: 9cf48a7154d6336a24504df0ec1cd62d689859f1b8f9df85e313bf69f1db97a17191e813b7f2e10bdd8565644dae88f6b323b1cd109e64995e26ea8d537bb67a

data/docs/_pages/performance.adoc

@@ -293,12 +293,19 @@ scratch there. 1.9.163.2 moved the bulk into C and fixed the
 truncation — newer bindings use their C path (version-memoized,
 cheaper than a per-call count+copy).
 
-The native *builder* factories (native_create_element/text) are
-deliberately not adopted: native mutations do not advance the
-binding's document version, so the binding's memoized reads over
-the same tree go stale, and attaching the created natives pays a
-per-node `to_binding` bridge that costs more than the factory
-saves (create+attach measured 5816 -> 7697ns vs nokogiri 1750).
+The native builder factories (native_create_element/text) are
+adopted from 1.9.163.5, where native mutations became
+version-coherent with the binding (leptris-ruby#204/#208 — both
+surfaces' memos drop on mutation). The earlier rejection — stale
+binding memos plus a per-attach `to_binding` bridge costing more
+than the factory saved (5816 -> 7697ns vs nokogiri 1750) — no
+longer applies: `add_child` carries a both-native fast path with
+no bridge, and `doc_for` reads the 1.9.163.5 `NativeNode#document`
+accessor instead of the parent climb. create ×2 + attach:
+**4846 -> 2333ns (0.30x -> 0.78x of raw nokogiri)**. Attribute
+writes on native nodes still bridge (the native layer has no
+`[]=`), and the native `attributes` hash (name → String) does not
+fit the attribute-node contract — enumeration stays bridged.
 
 Serialize (document, 30KB): passing `encoding: "UTF-8"` when it
 equals the document's own ran a conversion pass for byte-identical

data/lib/moxml/adapter/leptris.rb

@@ -65,6 +65,15 @@ module Moxml
         defined?(::Leptris::XML::NativeNode) &&
         Gem::Version.new(::Leptris::VERSION) >= Gem::Version.new("1.9.163.2")
 
+      # 1.9.163.5 (leptris-ruby#204/#208): native mutations are
+      # version-coherent with the binding (both surfaces' memos
+      # drop on mutation) — the builder factories and native
+      # add_child are adoptable end-to-end; NativeNode grew a
+      # document accessor and line numbers.
+      NATIVE_MUTATIONS_COHERENT =
+        defined?(::Leptris::XML::NativeNode) &&
+        Gem::Version.new(::Leptris::VERSION) >= Gem::Version.new("1.9.163.5")
+
       # 1.9.163.2 moved the native bulk children into C
       # (Native.bulk_children) and fixed the 512 truncation
       # (leptris-ruby#202). The 162.6-163.1 window carries the
@@ -130,6 +139,10 @@ module Moxml
           # to the root (C-bound reads) and look the doc up in the
           # root registry. Detached subtrees answer nil.
           def doc_for(node)
+            # The 1.9.163.5 native layer exposes the owning document
+            # directly; the root climb serves older bindings.
+            return node.document if NATIVE_MUTATIONS_COHERENT
+
             current = node
             current = current.parent while current&.parent
             @native_doc_roots[current]
@@ -477,18 +490,24 @@ module Moxml
           ::Leptris::XML::Document.create
         end
 
-        # The native builder factories (native_create_element/text)
-        # are deliberately NOT adopted: native mutations do not
-        # advance the binding's document version, so the binding's
-        # memoized reads over the same tree go stale; attaching the
-        # created natives also pays a per-node to_binding bridge
-        # that costs more than the factory saves (create+attach
-        # measured 5816 -> 7697ns vs nokogiri 1750).
+        # Native builder factories: one C call and a TypedData wrap.
+        # Adopted from 1.9.163.5, where native mutations became
+        # version-coherent (leptris-ruby#204/#208) — the earlier
+        # rejection (stale binding memos + per-attach bridge cost)
+        # no longer applies: add_child carries a native fast path.
         def create_native_element(name, owner_doc = nil)
+          if NATIVE_MUTATIONS_COHERENT && owner_doc
+            return owner_doc.native_create_element(name.to_s)
+          end
+
           (owner_doc || create_document).create_element(name.to_s)
         end
 
         def create_native_text(content, owner_doc = nil)
+          if NATIVE_MUTATIONS_COHERENT && owner_doc
+            return owner_doc.native_create_text(content)
+          end
+
           (owner_doc || create_document).create_text_node(content)
         end
 
@@ -986,6 +1005,11 @@ module Moxml
         end
 
         def line_number(node)
+          if NATIVE_MUTATIONS_COHERENT &&
+              node.is_a?(::Leptris::XML::NativeNode)
+            line = node.line
+            return line.nil? || line.zero? ? nil : line
+          end
           return nil unless node.is_a?(::Leptris::XML::Node)
 
           line = node.line
@@ -1050,7 +1074,9 @@ module Moxml
         end
 
         def add_child(parent, child)
-          if NATIVE_READ_LAYER
+          if NATIVE_READ_LAYER && !(NATIVE_MUTATIONS_COHERENT &&
+                   parent.is_a?(::Leptris::XML::NativeNode) &&
+                   child.is_a?(::Leptris::XML::NativeNode))
             parent = to_binding(parent)
             child = to_binding(child)
           end
@@ -1070,7 +1096,10 @@ module Moxml
         end
 
         def add_previous_sibling(node, new_node)
-          node = to_binding(node) if NATIVE_READ_LAYER
+          if NATIVE_READ_LAYER
+            node = to_binding(node)
+            new_node = to_binding(new_node)
+          end
           # A PI inserted before the root lives at document level in
           # libleptris's model, not in the element tree.
           if new_node.is_a?(::Leptris::XML::ProcessingInstruction) &&
@@ -1082,7 +1111,10 @@ module Moxml
         end
 
         def add_next_sibling(node, new_node)
-          node = to_binding(node) if NATIVE_READ_LAYER
+          if NATIVE_READ_LAYER
+            node = to_binding(node)
+            new_node = to_binding(new_node)
+          end
           node.add_next_sibling(new_node)
         end
 

data/lib/moxml/version.rb

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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: moxml
 version: !ruby/object:Gem::Version
-  version: 0.5.41
+  version: 0.5.42
 platform: ruby
 authors:
 - Ribose Inc.