moxml 0.5.54 → 0.5.55

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ba51d35ca414232eb04fafd976a2f5d24b2f43fff1b0e4c4a69147162da324c7
-  data.tar.gz: 73df7c7f9da2ff33e4ab2d06f3b56b29b0a09e4784c78eb9907d3ec15f82c234
+  metadata.gz: ded5e928e12262125965a33ed378e35d8ecc85c3e65de4d514a1fefe0ddd9458
+  data.tar.gz: b86296c3c0402915fc455696b9bd76b7892a08f58ec8fa9d8ac043fcd36784f4
 SHA512:
-  metadata.gz: c30f3bef444d6a6da7416e5ee5493c5656a6bbca1f28c9d504c9405b3be54b083eb5bf9a218ed1e0ac1cb94b3551678c028adadf667fcc7b15b50a2e403341ab
-  data.tar.gz: cef805979f7168ec08ce5dc80624e74a2bef27225233d1389862d84b6d4d9cea56806b50a6231e078a626ba488ce90668e0fd8b40ad6cf11a650a3b154c24041
+  metadata.gz: 8a29c038b4e5290a47acab859cf99796e9b653eb991f12e34539871e3dc8f02a6d3e2c66b312db64d87488a308bbb93f365dd5e4294e75fa329a5bfcb77bb174
+  data.tar.gz: ddfecce2232d70546ed2ac42c9ba5eb000e1d98cee79a00047f0a2927fcb47e40b27b4b35fb282f8086b805428062e664a3652673195e67a6fa7596907579063

data/lib/moxml/adapter/base.rb

@@ -141,6 +141,29 @@ module Moxml
           nil
         end
 
+        # Prefixed-attribute value fast path: adapters whose engine
+        # resolves expanded-name (uri, local) lookups natively answer
+        # the value here; others fall back to the resolver's
+        # attribute-list match. Capability probe, not a class
+        # identity check — Moxml::Adapter::Leptris is only defined
+        # once that adapter loads (issue #242).
+        def expanded_attr_reads?
+          false
+        end
+
+        def expanded_attr_value(_element, _uri, _local)
+          nil
+        end
+
+        # Engine-side inclusive C14N delegation: adapters whose
+        # engine canonicalizes byte-identically to the Ruby reference
+        # answer the String here; others (and unsupported shapes)
+        # return nil and the Ruby reference runs. Capability probe —
+        # same reasoning as expanded_attr_reads? (issue #242).
+        def native_inclusive10(_native)
+          nil
+        end
+
         # Subtree walk capability: an adapter with a C-side pre-order
         # traversal (leptris >= 1.9.174.6 visit) walks descendants in
         # one dispatch; nil keeps the recursive children walk. Self

data/lib/moxml/adapter/leptris.rb

@@ -463,8 +463,27 @@ module Moxml
         # engine — the same match rule as the resolver (xmlns
         # declarations invisible, no-namespace never matches a
         # prefixed name), without materializing the attribute list.
+        def expanded_attr_reads?
+          true
+        end
+
+        def native_inclusive10(native)
+          return nil unless native_c14n_byte_safe?
+
+          to_binding(native).canonicalize(
+            ::Leptris::XML::FFI::C14N_1_0, nil,
+            mode: ::Leptris::XML::FFI::C14N_MODE_CANONICAL
+          )
+        end
+
         def expanded_attr_value(element, uri, local)
-          element.attribute_ns(uri, local)
+          # Bridge first: identity-mode wrappers hand a NativeNode
+          # (no attribute_ns); binding elements pass through as-is.
+          # The class-receiver is_a? this method replaced was always
+          # false — element.adapter IS the adapter class — so the
+          # branch sat dead since identity mode and the binding-only
+          # assumption went unnoticed (issue #242).
+          to_binding(element).attribute_ns(uri, local)
         end
 
         # Fast bare-name read: the binding call plus marker

data/lib/moxml/attribute_resolver.rb

@@ -42,9 +42,7 @@ module Moxml
       return nil if uri.nil?
 
       adapter = element.adapter
-      if adapter.is_a?(Moxml::Adapter::Leptris)
-        return adapter.expanded_attr_value(element.native, uri, local)
-      end
+      return adapter.expanded_attr_value(element.native, uri, local) if adapter.expanded_attr_reads?
 
       attr = resolve(element, name)
       attr&.value

data/lib/moxml/c14n.rb

@@ -62,14 +62,7 @@ module Moxml
       return nil if with_comments || !inclusive_namespaces.empty?
       return nil unless algorithm_shape_inclusive10?(node_or_xml)
 
-      adapter = node_or_xml.context.config.adapter
-      return nil unless adapter == Moxml::Adapter::Leptris &&
-        Moxml::Adapter::Leptris.native_c14n_byte_safe?
-
-      Moxml::Adapter::Leptris.to_binding(node_or_xml.native).canonicalize(
-        ::Leptris::XML::FFI::C14N_1_0, nil,
-        mode: ::Leptris::XML::FFI::C14N_MODE_CANONICAL
-      )
+      node_or_xml.context.config.adapter.native_inclusive10(node_or_xml.native)
     end
 
     def self.algorithm_shape_inclusive10?(node_or_xml)

data/lib/moxml/version.rb

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

data/spec/moxml/attribute_resolver_spec.rb

@@ -105,4 +105,22 @@ RSpec.describe Moxml::AttributeResolver do
              )).to be(false)
     end
   end
+
+  describe ".resolve_value capability dispatch (issue #242)" do
+    it "answers prefixed values through the adapter capability, not class identity" do
+      doc = Moxml.new(:nokogiri).parse(
+        '<w:f xmlns:w="urn:w"><w:n w:val="Arial"/></w:f>',
+      )
+      node = doc.root.children.first
+
+      expect(Moxml::Adapter::Base.expanded_attr_reads?).to be(false)
+      expect(Moxml::Adapter::Nokogiri.expanded_attr_reads?).to be(false)
+      expect(node["w:val"]).to eq("Arial")
+    end
+
+    it "keeps native c14n delegation behind the adapter protocol" do
+      expect(Moxml::Adapter::Base.native_inclusive10(nil)).to be_nil
+      expect(Moxml::Adapter::Nokogiri.native_inclusive10(nil)).to be_nil
+    end
+  end
 end

metadata

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