moxml 0.5.111 → 0.5.113

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 24c16db3331672007afc4d08d042c3c44b52cfd00b3c20a026d88646afc842a7
-  data.tar.gz: 412edab11fe3895ef32a0f0520fb819fd2b5ea01b4436f08d715c0b38b132d1f
+  metadata.gz: c61331ea3536aa41170080eb6233643b3b48da707d268da9185ed11fbdcad87d
+  data.tar.gz: ef1765b2e2651438c8d38012ea374e2a8b9fe6ebcaff63d2afaf476eb0f727ab
 SHA512:
-  metadata.gz: dc6537c232494b1b33ec4f4f821e392ca38e8f274e01d0973f5725449b6688c5ba5ec598b50ef74dbb3a9d0b26fce15f411bde9d93d02744b0d663482e603d05
-  data.tar.gz: '019eb6bacde42ea30d76d6207da5f95d5c05e907c6f144e7f4c21cf92790bc150780f9289ac88fdc5ac94223f862ba198096bd040c90d26f146f54e52c6dc797'
+  metadata.gz: 2058949ca13a105d38f86b5c38acf3fb974ca9445faedda64ea7d857bf8f10561d5b6df42d7b56ede501d0fcb9186f0347349faee29dd66868b83ec47a94ac4c
+  data.tar.gz: c7c7b4f0b0d529ec793e790072d2115fe6ff7c17a389e08a84abadc2ab324e058f4f49d1e5d5a6e1143b0f688ac893b5ebae083638f1482b9b526b99d7c636fd

data/lib/moxml/adapter/leptris.rb

@@ -1893,6 +1893,44 @@ module Moxml
           installed
         end
 
+        # moxml#335: the engine's document resolution can mis-report
+        # pool ownership under spec-order state (the #1242 TLS-memo
+        # family), so a plain add can -1 for ANY node kind — not just
+        # elements and text. Elements rebuild structurally (the #376
+        # contract); every other kind carries no cross-document
+        # identity worth preserving, so reconstruct it in the parent's
+        # document through the binding's create faces.
+        def rebuild_foreign_child(parent_nn, child_nn)
+          doc = NN_DOCUMENT.bind_call(parent_nn)
+          rebuilt =
+            case child_nn.node_type
+            when :element
+              return structural_adopt(parent_nn, child_nn)
+            when :text
+              doc.create_text_node(child_nn.text)
+            when :comment
+              doc.create_comment(child_nn.text)
+            when :cdata
+              doc.create_cdata(child_nn.text)
+            when :processing_instruction, :pi
+              doc.create_processing_instruction(
+                child_nn.target, child_nn.content.to_s
+              )
+            else
+              raise Moxml::DocumentStructureError.new(
+                "cannot rebuild a #{child_nn.node_type} node",
+              )
+            end
+          # The raw engine add rejects binding-family children —
+          # append through the FFI face like structural_adopt.
+          ::Leptris::XML::FFI.check_status(
+            ::Leptris::XML::FFI.leptris_element_append_child(
+              ::FFI::Pointer.new(parent_nn.address), rebuilt.c_ptr
+            ),
+          )
+          rebuilt
+        end
+
         def copy_binding_subtree(doc, src)
           sub = doc.create_element(src.name.to_s)
           (src.respond_to?(:attributes) ? src.attributes : {}).each do |k, v|
@@ -2107,15 +2145,15 @@ module Moxml
               child = doc.create_text_node(child) if child.is_a?(String)
               # The engine's document resolution can mis-report pool
               # ownership under spec-order state (the #1242 TLS-memo
-              # family) — an -1 rejection falls back to the structural
-              # adopt for elements rather than raising through the flow.
+              # family) — an -1 rejection rebuilds the child in the
+              # parent's document for every reconstructible kind
+              # (moxml#335), rather than raising through the flow.
               begin
                 return NN_ADD_CHILD.bind_call(parent, child)
               rescue ::Leptris::XML::Error, ::RuntimeError => e
                 raise unless e.message.include?("append_child failed")
-                raise unless child.node_type == :element
 
-                structural_adopt(parent, child)
+                rebuild_foreign_child(parent, child)
               end
             end
 

data/lib/moxml/version.rb

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

data/spec/moxml/adapter/leptris_adoption_spec.rb

@@ -10,6 +10,7 @@ require "spec_helper"
 # names, doc-owned).
 RSpec.describe "leptris cross-document adoption" do
   let(:ctx) { Moxml.new(:leptris) }
+  let(:ad) { Moxml::Adapter::Leptris }
 
   it "add_child lands the adopted copy in the tree" do
     doc = ctx.parse("<r><a/></r>")
@@ -100,4 +101,49 @@ RSpec.describe "leptris cross-document adoption" do
       expect(mid.root.to_xml).not_to include("<deep")
     end
   end
+
+  # moxml#335: the -1 fallback must not re-raise by node kind — the
+  # engine's document resolution can mis-report pool ownership for
+  # ANY child (the #1242 TLS-memo family), and only elements have a
+  # cross-document identity worth a structural rebuild. The helper
+  # is specced directly: the engine failure is CI-only, so forcing
+  # the -1 locally is not reproducible.
+  context "when the add_child -1 rebuild fallback fires" do
+    it "rebuilds comments in the parent's document" do
+      doc = ctx.parse("<r/>")
+      other = ctx.parse("<other><!-- note --></other>")
+      ad.rebuild_foreign_child(doc.root.native, other.root.children.first.native)
+      expect(doc.root.to_xml).to include("<!-- note -->")
+      expect(other.root.to_xml).to include("<!-- note -->")
+    end
+
+    it "rebuilds CDATA in the parent's document" do
+      doc = ctx.parse("<r/>")
+      other = ctx.parse("<other><![CDATA[x < y]]></other>")
+      ad.rebuild_foreign_child(doc.root.native, other.root.children.first.native)
+      expect(doc.root.to_xml).to include("<![CDATA[x < y]]>")
+    end
+
+    it "rebuilds processing instructions in the parent's document" do
+      doc = ctx.parse("<r/>")
+      other = ctx.parse("<other><?tgt data='1'?></other>")
+      ad.rebuild_foreign_child(doc.root.native, other.root.children.first.native)
+      expect(doc.root.to_xml).to include("<?tgt data='1'?>")
+    end
+
+    it "rebuilds text in the parent's document" do
+      doc = ctx.parse("<r/>")
+      other = ctx.parse("<other>t</other>")
+      ad.rebuild_foreign_child(doc.root.native, other.root.children.first.native)
+      expect(doc.root.to_xml).to include(">t<")
+    end
+
+    it "rebuilds elements structurally" do
+      doc = ctx.parse("<r/>")
+      other = ctx.parse("<other><e k='v'>t</e></other>")
+      ad.rebuild_foreign_child(doc.root.native, other.root.children.first.native)
+      expect(doc.at("//r/e")["k"]).to eq("v")
+      expect(doc.at("//r/e").text).to eq("t")
+    end
+  end
 end

metadata

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