moxml 0.5.103 → 0.5.104

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 566f2842eac4e01753965cb5f56eada53d6683b3ef6b576015eb4b0da572ae67
-  data.tar.gz: ccc15ef083358a018e1536983017c6681c70c11ccbb4678ac123e5b0b2c9aada
+  metadata.gz: 3bbf00c0bac03066dd77a5bb5f952ac786a893d51dd0b18365a67500bdad7a15
+  data.tar.gz: 79d1609f628fcf8b69044ee5a9c990ae347cf0ff11c8b5e2ed2f812bee80ae0e
 SHA512:
-  metadata.gz: 2c29296b60bedabbc512c9f5c454b0195b3632dbf9952b0c397fc3bad6554135dd3096a04a34e1abddf9ce40088af4856b2ead7d8ca69a4eb251dc3db38286b3
-  data.tar.gz: fd1684a85d2ace2191b33bf996a12e8bea6cb535a340732cccee3318abc6649ba2e07548509d1469959e10ee711ecec650f8ec6b4525beb6a66e2c9a75a81c1d
+  metadata.gz: 756a39e9fc200fc4b1a9880f820489684169c3206739d02bb44aad987bfa9b20140a65ae923b29745609fd47407927be9d369f4ed4a64d4129e0f3168f3aec56
+  data.tar.gz: 6efc68d693d8c3b12f90ee4bfed2bf801162b89c7f3a5b0b1fc21eed45183a1ea962d4f2916a30932314208cd5099245a4977a0020b28892a83f4f019b40e03e

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/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/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/version.rb

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

data/spec/moxml/at_search_spec.rb

@@ -383,4 +383,17 @@ 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
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: moxml
 version: !ruby/object:Gem::Version
-  version: 0.5.103
+  version: 0.5.104
 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