moxml 0.5.42 → 0.5.43

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d447f74ea4b1c546e83fbb94819c6043a1fa48e0f9932ac7202009cecff63e1e
-  data.tar.gz: 4c33c0af41d9c628ecc993d7015d4e827d1d451b266dd7ad4145f00b322670a3
+  metadata.gz: 0c8fd5df5889694c9d7a3ee99c4caad866ddc40edd13ba46945f55e4e9f77a9b
+  data.tar.gz: ce14f6e9199e22576c0c5149e716a672efb9e44ef3a9b68d1b78abad02754a92
 SHA512:
-  metadata.gz: ebdc376975c3893facd06d793a04eebdfa9387814b131b9f0c5aecbd047fb57dcfadba80c40eee8ff60e52b4a7cbf1e886e67909b5f9b46623ca91b75cdb04b5
-  data.tar.gz: 9cf48a7154d6336a24504df0ec1cd62d689859f1b8f9df85e313bf69f1db97a17191e813b7f2e10bdd8565644dae88f6b323b1cd109e64995e26ea8d537bb67a
+  metadata.gz: 93ac1cd541ed454ac89c10179e5e56f919a4cd309ae8f16abe02950ea5a6fedfdb03c53a7a8c23520616cd202a2b8f5384616ac8cd7b9e884d9f127c1d38bd98
+  data.tar.gz: ed75757fec960bd181921f9562650f4f3d44f2261fa0eb41a09437906d26581ad07c8b5446d0719a1d26b5308de2e1e25c60d2f0013da6861d144692d53eada4

data/lib/moxml/adapter/leptris.rb

@@ -89,7 +89,17 @@ module Moxml
       # sites must never depend on the layer having loaded
       # (issue #217 — binding-only installs crashed on the
       # unguarded bridges).
+      # Class-body level: a bare @ivar inside `class << self` lands
+      # on the singleton's singleton, invisible to the methods at
+      # call time (same shape as @native_doc_roots below).
+      @binding_of = ObjectSpace::WeakMap.new
+
       class << self
+        # native -> bridged binding node. The bridge object is
+        # identity-stable for the native's lifetime (the binding's
+        # per-document wrap cache), so repeat bridges — the write
+        # loop on a factory-created element, re-accessed attribute
+        # lists — skip the document resolution and pointer wrap.
         # Binding node for any native: identity for binding nodes
         # (and on installs without the native layer — the constant
         # check short-circuits), a Node.wrap over the shared C
@@ -98,11 +108,14 @@ module Moxml
           return node unless NATIVE_READ_LAYER &&
             node.is_a?(::Leptris::XML::NativeNode)
 
+          bridged = @binding_of[node]
+          return bridged if bridged
+
           doc = doc_for(node)
           ptr = ::FFI::Pointer.new(node.address)
-          return ::Leptris::XML::Node.wrap(ptr, doc) if doc
-
-          ::Leptris::XML::Node.wrap(ptr, nil)
+          bridged = ::Leptris::XML::Node.wrap(ptr, doc)
+          @binding_of[node] = bridged unless bridged.nil?
+          bridged
         end
       end
 
@@ -281,7 +294,7 @@ module Moxml
         end
 
         def set_root(doc, element)
-          doc.root = element
+          to_binding(doc).root = to_binding(element)
         end
 
         def bare_set_qname_safe?

data/lib/moxml/adapter/rexml.rb

@@ -140,6 +140,7 @@ module Moxml
         end
 
         def set_root(doc, element)
+          doc.delete_element(doc.root) if doc.root
           doc.add_element(element)
         end
 

data/lib/moxml/version.rb

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

data/spec/integration/shared_examples/integration_workflows.rb

@@ -143,4 +143,38 @@ RSpec.shared_examples "Moxml Integration" do
       )
     end
   end
+
+  describe "document root assignment" do
+    it "attaches a factory-created element as the root and serializes" do
+      doc = context.create_document
+      root = doc.create_element("root")
+      root["id"] = "1"
+      child = doc.create_element("child")
+      child.text = "data"
+      root.add_child(child)
+
+      doc.root = root
+
+      expect(doc.root.name).to eq("root")
+      expect(doc.root["id"]).to eq("1")
+      expect(doc.root.parent).to eq(doc)
+      expect(doc.to_xml).to include("<root id=\"1\">", "<child>data</child>")
+
+      reparsed = context.parse(doc.to_xml)
+      expect(reparsed.root["id"]).to eq("1")
+      expect(reparsed.at_xpath("//child").text).to eq("data")
+    end
+
+    it "replaces an existing root" do
+      doc = context.parse("<old><nested/></old>")
+      replacement = doc.create_element("new")
+      replacement.add_child("text")
+
+      doc.root = replacement
+
+      expect(doc.root.name).to eq("new")
+      expect(doc.to_xml).to include("<new>text</new>")
+      expect(doc.to_xml).not_to include("old")
+    end
+  end
 end

metadata

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