moxml 0.5.102 → 0.5.104

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d182614a13bd3934c6d6dc1bb9b137015dc1daa124aeac24df40246ea3109103
-  data.tar.gz: 51d96b5f964b185772d4378913c5e48e4775c41cf5fd1a3e5cf942576fa33183
+  metadata.gz: 3bbf00c0bac03066dd77a5bb5f952ac786a893d51dd0b18365a67500bdad7a15
+  data.tar.gz: 79d1609f628fcf8b69044ee5a9c990ae347cf0ff11c8b5e2ed2f812bee80ae0e
 SHA512:
-  metadata.gz: f4e9ae7331917e4fb295aa541028cb233d889127e4e32982d24ad1d696df668f52d223b70a1c6f2e96360f72138abf36a2eb055f15b9f48c069cb7b421191cfe
-  data.tar.gz: '0362854ca321c56e434a0f030232088d200d948dfe231c2d2cf4d2017c3af7316610e3f18ece3ca85a9a9a2c0d98f431c9f3bb465f186fe39ae23b63ea2e9a7b'
+  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

@@ -507,7 +507,37 @@ module Moxml
         raise Moxml::XsltError, "stylesheet compile failed: #{e.message}"
       end
 
+      # Bindings >= 1.9.290.0 thread top-level xsl:param overrides
+      # (leptris#1478); older bindings raise the typed error (the
+      # #360 face gap, face-probed so a misrelease degrades instead
+      # of NoMethodError-ing — the 1.9.199.0 lesson).
+      NATIVE_XSLT_PARAMS =
+        NATIVE_READ_LAYER &&
+        Gem::Version.new(::Leptris::VERSION) >= Gem::Version.new("1.9.290.0") &&
+        ::Leptris::XML::XSLT::Stylesheet.instance_method(:apply_to)
+          .parameters.size > 1
+
+      # The seam contract passes the libxslt flat quoted array; the
+      # binding wants the raw name => value hash (leptris#1478
+      # matches nokogiri's quoted-string semantics internally).
+      def self.flat_params_to_hash(params)
+        return params if params.is_a?(Hash)
+        return {} if params.nil? || params.empty?
+
+        params.each_slice(2).to_h { |name, value| [name, unquote_param(value)] }
+      end
+
+      def self.unquote_param(value)
+        str = value.to_s
+        str.start_with?("'") && str.end_with?("'") && str.length >= 2 ? str[1..-2] : str
+      end
+
       def self.xslt_apply_document(sheet, document_native, params)
+        if NATIVE_XSLT_PARAMS
+          return sheet.apply_to(document_native,
+                                params: flat_params_to_hash(params))
+        end
+
         assert_xslt_no_params(params)
         sheet.apply_to(document_native)
       rescue ::Leptris::XML::XPathError => e
@@ -515,6 +545,11 @@ module Moxml
       end
 
       def self.xslt_apply_string(sheet, document_native, params)
+        if NATIVE_XSLT_PARAMS
+          return sheet.serialize(document_native,
+                                 params: flat_params_to_hash(params))
+        end
+
         assert_xslt_no_params(params)
         sheet.serialize(document_native)
       rescue ::Leptris::XML::XPathError => e
@@ -525,8 +560,9 @@ module Moxml
         return if params.nil? || params.empty?
 
         raise Moxml::XsltError,
-              "XSLT params are not threaded by the leptris engine yet " \
-              "(leptris-ruby#360); pass an empty params hash"
+              "XSLT params are not threaded by this binding " \
+              "(leptris-ruby#360); upgrade to leptris >= 1.9.290.0 or " \
+              "pass an empty params hash"
       end
 
       def self.native_c14n_byte_safe?
@@ -1972,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
 
@@ -2016,11 +2061,23 @@ module Moxml
           new_node
         end
 
+        NATIVE_APPEND_CHILDREN =
+          NATIVE_READ_LAYER &&
+          Gem::Version.new(::Leptris::VERSION) >= Gem::Version.new("1.9.290.0")
+
         def replace_children(node, new_children)
           if NATIVE_READ_LAYER
             node = to_binding(node)
             new_children = new_children.map { |child| to_binding(child) }
           end
+          if NATIVE_APPEND_CHILDREN && !new_children.empty?
+            # One crossing to clear, one to append N (leptris-ruby#366's
+            # bulk face; the old path paid an append crossing per child).
+            ::Leptris::XML::FFI.check_status(
+              ::Leptris::XML::FFI.leptris_element_remove_children(node.c_ptr),
+            )
+            return node.append_children(new_children)
+          end
           node.children = new_children
         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.102"
+  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

data/spec/moxml/xslt/stylesheet_spec.rb

@@ -4,8 +4,9 @@ require "spec_helper"
 
 # The Moxml::XSLT contract: compile once, apply many. Only engines
 # with an XSLT processor implement it (nokogiri/libxslt; leptris
-# covering XSLT 1.0-3.0). leptris does not thread top-level params
-# yet (leptris-ruby#360) — non-empty params raise Moxml::XsltError.
+# covering XSLT 1.0-3.0). Bindings >= 1.9.290.0 thread top-level
+# params (leptris#1478); older bindings raise Moxml::XsltError
+# citing leptris-ruby#360.
 SHEET_COUNT = <<~XSL
   <xsl:stylesheet xmlns:xsl="http://www.w3.org/1999/XSL/Transform" version="1.0">
     <xsl:template match="/"><out><xsl:value-of select="count(//i)"/></out></xsl:template>
@@ -77,10 +78,22 @@ RSpec.describe "Moxml::XSLT" do
       expect(sheet.apply_to_string(doc)).to include("plain text result")
     end
 
-    it "raises XsltError on non-empty params (leptris-ruby#360)" do
-      sheet = ctx.xslt(SHEET_COUNT)
-      expect { sheet.apply_to(doc, params: { "n" => "7" }) }
-        .to raise_error(Moxml::XsltError, /params.*#360/)
+    if Gem::Version.new(Leptris::VERSION) >= Gem::Version.new("1.9.290.0")
+      it "threads top-level params (leptris#1478)" do
+        sheet = ctx.xslt(<<~XSL)
+          <xsl:stylesheet xmlns:xsl="http://www.w3.org/1999/XSL/Transform" version="1.0">
+            <xsl:param name="n" select="'0'"/>
+            <xsl:template match="/"><out><xsl:value-of select="$n"/></out></xsl:template>
+          </xsl:stylesheet>
+        XSL
+        expect(sheet.apply_to_string(doc, params: { "n" => "7" })).to include("7")
+      end
+    else
+      it "raises XsltError on non-empty params (leptris-ruby#360)" do
+        sheet = ctx.xslt(SHEET_COUNT)
+        expect { sheet.apply_to(doc, params: { "n" => "7" }) }
+          .to raise_error(Moxml::XsltError, /params.*#360/)
+      end
     end
 
     it "reports support" do

metadata

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