lutaml-model 0.8.53 → 0.8.54

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 15a10289cc84edab04063cf83dac41f6923500898b0b65e2c72db51421f8a2ad
-  data.tar.gz: 4e6941d70effad354e4798013e27413eb0fd8d8b85643ca5621acb1e009b9247
+  metadata.gz: ffb19b7e87ec763167dc806fb0fd0b37a1ca292e7627333ba1238f06c3e5f17e
+  data.tar.gz: 340c6f96fb82f0d972ca654f548db51bdef12155bc94ae95e3eaf6eacf96af29
 SHA512:
-  metadata.gz: 149a8dc04992492d770567285e43fdea303035e93a340554966cf0ff5c844cfb83c03023f214cbc1f50fe3f9e4a20bbce9e540656ed97092876c0b450924e4bd
-  data.tar.gz: 82fc85442bef8b6f0014f03ac10373b03dd99b5be30631f45fbd2e924d02cc6baafbcd75fbb17249c648f9e8711609ba2d5d11494e80f9c6ab734d3cba517bf9
+  metadata.gz: d79e2b5b7fce4f6ba3115cdb190365b8a1fc1212922f00cabf2e9855fa85c2350808c0844f78c55015de4e8c8cf480cbdf5390e8afaa0a401f01ad6e2a1d93a3
+  data.tar.gz: c6a29c349be599ddfcbcb53087c17178de7de17aa1afa46fb5e2fc6eb2b3c4b0c0189b68a26aa77d0e2d561966c4001c0bd04abacf2d6e987d8846185b6add29

data/lib/lutaml/model/version.rb

@@ -2,6 +2,6 @@
 
 module Lutaml
   module Model
-    VERSION = "0.8.53"
+    VERSION = "0.8.54"
   end
 end

data/lib/lutaml/xml/model_transform.rb

@@ -731,18 +731,29 @@ _effective_register)
         # each_value scan below is dead whenever the index exists.
         return nil if root_index && !candidates
 
-        matched_attr = if candidates
-                         candidates.find do |attr|
-                           local_name_match?(attr, local_name, rule_uri,
-                                             flexible_local)
-                         end
-                       else
-                         doc.root.attributes.each_value.find do |attr|
-                           local_name_match?(attr, local_name, rule_uri,
-                                             flexible_local)
-                         end
-                       end
-        matched_attr&.value
+        matched = if candidates
+                    candidates.select do |attr|
+                      local_name_match?(attr, local_name, rule_uri,
+                                        flexible_local)
+                    end
+                  else
+                    doc.root.attributes.each_value.select do |attr|
+                      local_name_match?(attr, local_name, rule_uri,
+                                        flexible_local)
+                    end
+                  end
+        # Attribute identity is (namespace URI, local name) — #841. The
+        # lenient probe is a recovery mechanism for the single
+        # undeclared-prefix case; several distinct matches are an
+        # ambiguity the document order must not arbitrate.
+        if matched.size > 1
+          identities = matched.map(&:namespaced_name).join(", ")
+          warn "[Lutaml::Model] ambiguous attribute for rule " \
+               "#{rule_name.inspect}: #{identities} all match by local " \
+               "name on <#{doc.root.unprefixed_name}>; no value bound"
+          return nil
+        end
+        matched.first&.value
       end
 
       # Namespace discipline of the lenient local-name fallback

data/lib/lutaml/xml/xml_element.rb

@@ -385,11 +385,24 @@ module Lutaml
         ensure_attribute_index
 
         if attribute_name.is_a?(Array)
+          # Alias list: at most one spelling variant may be present.
+          # Several simultaneous matches are an authoring ambiguity —
+          # attribute identity is (namespace URI, local name) — #841 —
+          # and array order must not arbitrate.
+          found = nil
           attribute_name.each do |name|
             val = @attribute_index[name]
-            return val unless val.nil?
+            next if val.nil?
+
+            if found
+              warn "[Lutaml::Model] ambiguous attribute aliases " \
+                   "#{found[0].inspect} and #{name.inspect} both present " \
+                   "on <#{unprefixed_name}>; no value bound"
+              return nil
+            end
+            found = [name, val]
           end
-          nil
+          found&.last
         else
           @attribute_index[attribute_name]
         end

data/spec/lutaml/xml/attribute_ambiguity_surface_spec.rb

@@ -0,0 +1,81 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+# Attribute identity is (namespace URI, local name) — lutaml-model#841.
+# Where matching previously resolved silently by order (document order
+# in the lenient recovery, array order in alias lists), several
+# distinct matches are an ambiguity: no value binds and the collision
+# is surfaced on stderr.
+module AttributeAmbiguitySurfaceSpec
+  # Lenient recovery: sole-claimant rule spelled URI:local while the
+  # document's prefixes stay undeclared (#754 recovery case).
+  class LenientProbe < Lutaml::Model::Serializable
+    attribute :ext, :string
+
+    xml do
+      element "probe"
+      map_attribute "urn:example:v:ext", to: :ext
+    end
+  end
+
+  # Alias list: spelling variants of one name — at most one may occur.
+  class AliasProbe < Lutaml::Model::Serializable
+    attribute :status, :string
+
+    xml do
+      element "probe"
+      map_attribute %w[status product-status], to: :status
+    end
+  end
+end
+
+RSpec.describe "Ambiguous attribute matches surface instead of resolving by order" do
+  describe "lenient local-name recovery" do
+    it "recovers the single undeclared-prefix attribute" do
+      probe = AttributeAmbiguitySurfaceSpec::LenientProbe.from_xml(
+        %(<probe v:ext="1"/>),
+      )
+      expect(probe.ext).to eq("1")
+    end
+
+    it "binds no value when several same-local attributes match" do
+      expect do
+        probe = AttributeAmbiguitySurfaceSpec::LenientProbe.from_xml(
+          %(<probe v:ext="1" w:ext="2"/>),
+        )
+        expect(probe.ext).to be_nil
+      end.to output(/ambiguous attribute/i).to_stderr
+    end
+
+    it "prefers the exact qualified match when the prefix IS declared" do
+      probe = AttributeAmbiguitySurfaceSpec::LenientProbe.from_xml(
+        %(<probe xmlns:v="urn:example:v" v:ext="1" w:ext="2"/>),
+      )
+      expect(probe.ext).to eq("1")
+    end
+  end
+
+  describe "alias lists" do
+    it "reads the single present alias" do
+      probe = AttributeAmbiguitySurfaceSpec::AliasProbe.from_xml(
+        %(<probe product-status="draft"/>),
+      )
+      expect(probe.status).to eq("draft")
+
+      probe = AttributeAmbiguitySurfaceSpec::AliasProbe.from_xml(
+        %(<probe status="open"/>),
+      )
+      expect(probe.status).to eq("open")
+    end
+
+    it "binds no value when two aliases are simultaneously present" do
+      expect do
+        probe = AttributeAmbiguitySurfaceSpec::AliasProbe.from_xml(
+          %(<probe status="open" product-status="draft"/>),
+        )
+        expect(probe.status).to be_nil
+      end.to output(/ambiguous attribute aliases/i).to_stderr
+    end
+  end
+end

data/spec/lutaml/xml/xml_element_guard_spec.rb

@@ -40,11 +40,19 @@ RSpec.describe "XmlElement performance guard specs" do
         expect(element.find_attribute_value("missing")).to be_nil
       end
 
-      it "finds first match from array of names" do
-        result = element.find_attribute_value(["missing1", "class", "id"])
+      it "finds the single present name from array of names" do
+        result = element.find_attribute_value(["missing1", "class", "missing2"])
         expect(result).to eq("foo")
       end
 
+      it "binds no value when two names in the array are both present" do
+        # Attribute identity is (namespace URI, local name) — #841;
+        # array order must not arbitrate an ambiguity.
+        expect do
+          expect(element.find_attribute_value(["class", "id"])).to be_nil
+        end.to output(/ambiguous attribute aliases/i).to_stderr
+      end
+
       it "returns nil when no name in array matches" do
         expect(element.find_attribute_value(["missing1", "missing2"])).to be_nil
       end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lutaml-model
 version: !ruby/object:Gem::Version
-  version: 0.8.53
+  version: 0.8.54
 platform: ruby
 authors:
 - Ribose Inc.
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-09-21 00:00:00.000000000 Z
+date: 2026-09-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: base64
@@ -1902,6 +1902,7 @@ files:
 - spec/lutaml/xml/adapter/ox_adapter_spec.rb
 - spec/lutaml/xml/adapter/rexml_adapter_spec.rb
 - spec/lutaml/xml/adapter/xml_namespace_spec.rb
+- spec/lutaml/xml/attribute_ambiguity_surface_spec.rb
 - spec/lutaml/xml/attribute_element_namespace_disjoint_spec.rb
 - spec/lutaml/xml/attribute_namespace_disjointness_spec.rb
 - spec/lutaml/xml/builder/builder_spec.rb