moxml 0.5.99 → 0.5.100

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e0799669e81215c8738217d5d9f75a5cc8842a9a35c92553d07f3f60ee6a8196
-  data.tar.gz: 86aeb7370ef541cc898fa1f3c9a63ba3da154a7ffb09a68d0236f82a1241fc10
+  metadata.gz: a421da0e82158a69de2d25328346153be526d74c6d2bc2bad732427763b175c1
+  data.tar.gz: c9d982a98b2ce4984b3ee8659e89f2a4d86484d7e39fec4c04221bd2767473eb
 SHA512:
-  metadata.gz: '0932b236b7f6e452705f49c6328848edd315774b35ae47d732a90fa65e5ed98612265765fab4c5bde3ca7eaad79b583457da72e051dbbac3d1df027986105435'
-  data.tar.gz: 1afbad96dc4a770d0b7c74a23fa55d6d80b2c80a3276bec9ae4b83d10cbda9cb59ae22649634ebfc4c6b49ab7715364396bf387236234372fc93212e4a3d0777
+  metadata.gz: 2a3c9a536530f058a8c0a7721a3f192d89be195df277e959d746f2b6360db2da3587c2467fc21222aff85b96c8c3483a6a59406554804e073b25336c6d5dfe17
+  data.tar.gz: 750f9751134a3c077bd103edb9b18b0de7ea179d0c27839f1dff22654d372d7eb989a55c154c09ee155ec64d386547885013320eefa4044cb7dccc994d309806

data/lib/moxml/adapter/leptris.rb

@@ -1793,8 +1793,14 @@ module Moxml
           parent_doc = parent.is_a?(NN) ? NN_DOCUMENT.bind_call(parent) : parent.document
           return if parent_doc.nil? || child_doc.equal?(parent_doc)
 
+          # moxml#308: one source document's children attach
+          # node-by-node — dedup by identity or the pin list grows
+          # linearly with append count (GB-scale malloc on
+          # large-document hydration). Distinct owners stay few.
           owner = child_doc.is_a?(NN) ? to_binding(child_doc) : child_doc
           docs = attachments.get(parent_doc, :adopted_docs) || []
+          return if docs.include?(owner)
+
           docs << owner
           attachments.set(parent_doc, :adopted_docs, docs)
         end

data/lib/moxml/version.rb

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

data/spec/moxml/adapter/leptris_append_gc_spec.rb

@@ -49,4 +49,18 @@ RSpec.describe "leptris cross-document append under GC pressure" do
       expect(out).to include("<annex")
     end
   end
+
+  # moxml#308: the pin list must dedup by identity — one source
+  # document's children attach node-by-node, and without dedup the
+  # list grows linearly with append count (GB-scale malloc on
+  # large-document hydration; OOM at 43-75M slots).
+  it "pins each adopted document once regardless of append count" do
+    target = ctx.parse("<r/>")
+    frag = ctx.parse("<m>#{Array.new(300) { |i| "<c#{i}>x</c#{i}>" }.join}</m>")
+    frag.root.children.each { |c| target.root.add_child(c) }
+
+    docs = ctx.config.adapter.attachments.get(target.native.document, :adopted_docs)
+    expect(docs.size).to eq(1)
+    expect(target.root.to_xml).to include("<c299")
+  end
 end

metadata

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