canon 0.1.14 → 0.1.15

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 81995d22ec29adb9b2fb60f0ed8bc0219fe28e468c89a2001901b0f4521c757b
-  data.tar.gz: fabc6e6c77e92848783e747459377caa787330d3360f83f544b6372cc68ba227
+  metadata.gz: '04860609f8d3300ccebf84a0f2208510600dcfaac4b3f54f698eb2de7eed0493'
+  data.tar.gz: fc50abe2a915d7d7ff1cd630c0a5a50849b6fd5780664cb6bb419300b2388743
 SHA512:
-  metadata.gz: d33e2fcd54ae3b5cab9fdcfe980b1a8d1f2f97b1389ea430ecfda093b275e77f93e49e5a4f1171797df3fcb7f8d0ef28654301dbb846c16a4eb751018ea10129
-  data.tar.gz: 85ffc85bf577b631c9aee7e16f81e0be163de2025154dd197962943c533cd3a1aa0d79799d084194780ff8654e766b0ac3ac36b635425d47e64009c71a4edb6d
+  metadata.gz: d1bcc3ad7439fdd7f65627c53cd0dc4b92d781bdbdaf330d2bb8ecc89b3319a4fc02e78de8e961cdb69854661881db39bf59c664210c7a59b7a011355b1db71b
+  data.tar.gz: cca1b3f2eee48054b5431118350e64acb46f485096cd8c88798e0935210985d63eb93de4727a42d17cfaad8ef03c55e7d1745922500f3a3495a791d86fd60676

data/lib/canon/color_detector.rb

@@ -82,7 +82,7 @@ module Canon
       # @return [Boolean] true if colors appear to be supported
       def detect_from_env
         # Check TERM variable
-        term = ENV["TERM"]
+        term = ENV.fetch("TERM", nil)
         if term && NO_COLOR_TERMS.any? { |t| term.include?(t) }
           # Known no-color terminals
           return false
@@ -100,7 +100,7 @@ module Canon
         end
 
         # Check for known color-capable terminals
-        colorterm = ENV["COLORTERM"]
+        colorterm = ENV.fetch("COLORTERM", nil)
         return true if COLOR_TERM_VALUES.include?(colorterm)
 
         # Default: assume colors are supported on modern terminals
@@ -125,7 +125,7 @@ module Canon
       # - Generic CI: check for specific TeamCity/Terminal variables
       #
       # @return [Boolean] true if CI environment likely supports colors
-      def detect_ci_colors
+      def detect_ci_colors # rubocop:disable Naming/PredicateMethod
         # Most modern CI systems support ANSI colors
         # Only disable for explicitly known non-color CI
         return false if ENV["TERM"] == "dumb"

data/lib/canon/comparison/comparison_result.rb

@@ -44,10 +44,8 @@ html_version: nil, match_options: nil, algorithm: :dom, original_strings: nil)
           if diff.is_a?(Canon::Diff::DiffNode)
             diff.normative?
           # Legacy Hash format - always considered normative (structural differences)
-          elsif diff.is_a?(Hash)
-            true
           else
-            false
+            diff.is_a?(Hash)
           end
         end
       end

data/lib/canon/comparison/dimensions/attribute_order_dimension.rb

@@ -37,7 +37,7 @@ module Canon
         # @param order1 [Array<Symbol>] First attribute order
         # @param order2 [Array<Symbol>] Second attribute order
         # @return [Boolean] true if attribute order is exactly the same
-        def compare_strict(order1, order2)
+        def compare_strict(order1, order2) # rubocop:disable Naming/PredicateMethod
           order1 == order2
         end
 

data/lib/canon/comparison/dimensions/attribute_presence_dimension.rb

@@ -37,7 +37,7 @@ module Canon
         # @param names1 [Array<Symbol>] First attribute names
         # @param names2 [Array<Symbol>] Second attribute names
         # @return [Boolean] true if attribute names are exactly equal
-        def compare_strict(names1, names2)
+        def compare_strict(names1, names2) # rubocop:disable Naming/PredicateMethod
           names1.sort == names2.sort
         end
 

data/lib/canon/comparison/dimensions/comments_dimension.rb

@@ -37,7 +37,7 @@ module Canon
         # @param comments1 [Array<String>] First comments array
         # @param comments2 [Array<String>] Second comments array
         # @return [Boolean] true if comments are exactly equal
-        def compare_strict(comments1, comments2)
+        def compare_strict(comments1, comments2) # rubocop:disable Naming/PredicateMethod
           comments1 == comments2
         end
 
@@ -48,7 +48,7 @@ module Canon
         # @param comments1 [Array<String>] First comments array
         # @param comments2 [Array<String>] Second comments array
         # @return [Boolean] true if normalized comments are equal
-        def compare_normalize(comments1, comments2)
+        def compare_normalize(comments1, comments2) # rubocop:disable Naming/PredicateMethod
           normalize_comments(comments1) == normalize_comments(comments2)
         end
 

data/lib/canon/comparison/dimensions/element_position_dimension.rb

@@ -39,7 +39,7 @@ module Canon
         # @param pos1 [Integer] First position
         # @param pos2 [Integer] Second position
         # @return [Boolean] true if positions are equal
-        def compare_strict(pos1, pos2)
+        def compare_strict(pos1, pos2) # rubocop:disable Naming/PredicateMethod
           pos1 == pos2
         end
 

data/lib/canon/comparison/dimensions/registry.rb

@@ -67,7 +67,7 @@ module Canon
         # @param node2 [Object] Second node
         # @param behavior [Symbol] Comparison behavior
         # @return [Boolean] true if nodes match for this dimension
-        def self.compare(dimension_name, node1, node2, behavior)
+        def self.compare(dimension_name, node1, node2, behavior) # rubocop:disable Naming/PredicateMethod
           dimension = get(dimension_name)
           dimension.equivalent?(node1, node2, behavior)
         end

data/lib/canon/comparison/dimensions/structural_whitespace_dimension.rb

@@ -41,7 +41,7 @@ module Canon
         # @param ws1 [Array<String>] First whitespace array
         # @param ws2 [Array<String>] Second whitespace array
         # @return [Boolean] true if structural whitespace is exactly equal
-        def compare_strict(ws1, ws2)
+        def compare_strict(ws1, ws2) # rubocop:disable Naming/PredicateMethod
           ws1 == ws2
         end
 
@@ -52,7 +52,7 @@ module Canon
         # @param ws1 [Array<String>] First whitespace array
         # @param ws2 [Array<String>] Second whitespace array
         # @return [Boolean] true if normalized structural whitespace is equal
-        def compare_normalize(ws1, ws2)
+        def compare_normalize(ws1, ws2) # rubocop:disable Naming/PredicateMethod
           normalize_whitespace(ws1) == normalize_whitespace(ws2)
         end
 

data/lib/canon/comparison/dimensions/text_content_dimension.rb

@@ -37,7 +37,7 @@ module Canon
         # @param text1 [String, nil] First text
         # @param text2 [String, nil] Second text
         # @return [Boolean] true if texts are exactly equal
-        def compare_strict(text1, text2)
+        def compare_strict(text1, text2) # rubocop:disable Naming/PredicateMethod
           text1.to_s == text2.to_s
         end
 
@@ -49,7 +49,7 @@ module Canon
         # @param text1 [String, nil] First text
         # @param text2 [String, nil] Second text
         # @return [Boolean] true if normalized texts are equal
-        def compare_normalize(text1, text2)
+        def compare_normalize(text1, text2) # rubocop:disable Naming/PredicateMethod
           normalized1 = normalize_text(text1)
           normalized2 = normalize_text(text2)
 

data/lib/canon/comparison/format_detector.rb

@@ -52,7 +52,7 @@ module Canon
         # @return [Symbol] Format type
         def detect_string(str)
           # Use cache for format detection
-          Cache.fetch(:format_detect, Cache.key_for_format_detection(str)) do
+          Cache.fetch(:format_detect, Cache.key_for_format_detection(str)) do # rubocop:disable Lint/UselessDefaultValueArgument
             detect_string_uncached(str)
           end
         end

data/lib/canon/comparison/html_comparator.rb

@@ -134,9 +134,7 @@ module Canon
           # Classify DiffNodes as normative/informative if we have verbose output
           if opts[:verbose] && !differences.empty?
             classifier = Canon::Diff::DiffClassifier.new(match_opts)
-            classifier.classify_all(differences.select do |d|
-              d.is_a?(Canon::Diff::DiffNode)
-            end)
+            classifier.classify_all(differences.grep(Canon::Diff::DiffNode))
           end
 
           if opts[:verbose]
@@ -153,9 +151,7 @@ module Canon
             # Non-verbose mode: check equivalence
             # If comparison found differences, classify them to determine if normative
             classifier = Canon::Diff::DiffClassifier.new(match_opts)
-            classifier.classify_all(differences.select do |d|
-              d.is_a?(Canon::Diff::DiffNode)
-            end)
+            classifier.classify_all(differences.grep(Canon::Diff::DiffNode))
             # Equivalent if no normative differences (matches semantic algorithm)
             differences.none?(&:normative?)
           else

data/lib/canon/comparison/markup_comparator.rb

@@ -116,13 +116,13 @@ module Canon
 
           # Canon::Xml::Node ElementNode
           if node.is_a?(Canon::Xml::Nodes::ElementNode)
-            node.attribute_nodes.each_with_object({}) do |attr, hash|
-              hash[attr.name] = attr.value
+            node.attribute_nodes.to_h do |attr|
+              [attr.name, attr.value]
             end
           # Nokogiri nodes
           elsif node.respond_to?(:attributes)
-            node.attributes.each_with_object({}) do |(_, attr), hash|
-              hash[attr.name] = attr.value
+            node.attributes.to_h do |_, attr|
+              [attr.name, attr.value]
             end
           else
             {}
@@ -231,9 +231,9 @@ module Canon
         # @param node [Object] Node to check
         # @return [Boolean] true if node is a text node
         def text_node?(node)
-          node.respond_to?(:text?) && node.text? &&
-            !node.respond_to?(:element?) ||
-            node.respond_to?(:node_type) && node.node_type == :text
+          (node.respond_to?(:text?) && node.text? &&
+            !node.respond_to?(:element?)) ||
+            (node.respond_to?(:node_type) && node.node_type == :text)
         end
 
         # Get text content from a node

data/lib/canon/comparison/strategies/semantic_tree_match_strategy.rb

@@ -146,8 +146,8 @@ module Canon
           # MUST match DOM diff preprocessing EXACTLY (xml_comparator.rb:106-109)
           # Simple pattern: add newline between adjacent tags
           [
-            xml1.gsub(/></, ">\n<"),
-            xml2.gsub(/></, ">\n<"),
+            xml1.gsub("><", ">\n<"),
+            xml2.gsub("><", ">\n<"),
           ]
         end
 
@@ -185,7 +185,7 @@ module Canon
 
           # KEY FIX: Use simple gsub, NOT Canon.format
           # This ensures proper line-by-line display matching DOM diff format
-          [html1.gsub(/></, ">\n<"), html2.gsub(/></, ">\n<")]
+          [html1.gsub("><", ">\n<"), html2.gsub("><", ">\n<")]
         end
 
         # Preprocess JSON documents

data/lib/canon/comparison/xml_comparator/child_comparison.rb

@@ -241,9 +241,9 @@ diff_children, differences)
               smaller_set = children2
             end
 
-            smaller_set_names = smaller_set.map do |c|
+            smaller_set_names = smaller_set.filter_map do |c|
               c.respond_to?(:name) ? c.name : nil
-            end.compact
+            end
 
             new_larger_set = []
             max_len = larger_set.length

data/lib/canon/comparison/xml_comparator/node_parser.rb

@@ -26,7 +26,7 @@ module Canon
           end
 
           # Apply preprocessing to XML string before parsing
-          xml_string = apply_preprocessing(node, preprocessing)
+          xml_string = apply_preprocessing(node, preprocessing).strip
 
           # Use Canon::Xml::DataModel for parsing to get Canon::Xml::Node instances
           Canon::Xml::DataModel.from_xml(xml_string,

data/lib/canon/comparison/xml_comparator.rb

@@ -123,17 +123,15 @@ module Canon
           # Classify DiffNodes as normative/informative if we have verbose output
           if opts[:verbose] && !differences.empty?
             classifier = Canon::Diff::DiffClassifier.new(match_opts)
-            classifier.classify_all(differences.select do |d|
-              d.is_a?(Canon::Diff::DiffNode)
-            end)
+            classifier.classify_all(differences.grep(Canon::Diff::DiffNode))
           end
 
           if opts[:verbose]
             # Serialize parsed nodes for consistent formatting
             # This ensures both sides formatted identically, showing only real differences
             preprocessed = [
-              serialize_node(node1).gsub(/></, ">\n<"),
-              serialize_node(node2).gsub(/></, ">\n<"),
+              serialize_node(node1).gsub("><", ">\n<"),
+              serialize_node(node2).gsub("><", ">\n<"),
             ]
 
             ComparisonResult.new(
@@ -148,9 +146,7 @@ module Canon
             # Non-verbose mode: check equivalence
             # If comparison found differences, classify them to determine if normative
             classifier = Canon::Diff::DiffClassifier.new(match_opts)
-            classifier.classify_all(differences.select do |d|
-              d.is_a?(Canon::Diff::DiffNode)
-            end)
+            classifier.classify_all(differences.grep(Canon::Diff::DiffNode))
             # Equivalent if no normative differences (matches semantic algorithm)
             differences.none?(&:normative?)
           else

data/lib/canon/comparison/xml_node_comparison.rb

@@ -281,9 +281,9 @@ diff_children, differences)
       # @param node [Object] Node to check
       # @return [Boolean] true if node is a text node
       def self.text_node?(node)
-        node.respond_to?(:text?) && node.text? &&
-          !node.respond_to?(:element?) ||
-          node.respond_to?(:node_type) && node.node_type == :text
+        (node.respond_to?(:text?) && node.text? &&
+          !node.respond_to?(:element?)) ||
+          (node.respond_to?(:node_type) && node.node_type == :text)
       end
 
       # Extract text content from a node

data/lib/canon/comparison.rb

@@ -495,7 +495,7 @@ module Canon
 
         # Use cache for string documents
         Cache.fetch(:document_parse,
-                    Cache.key_for_document(doc, format, preprocessing)) do
+                    Cache.key_for_document(doc, format, preprocessing)) do # rubocop:disable Lint/UselessDefaultValueArgument
           yield doc
         end
       end

data/lib/canon/config/env_provider.rb

@@ -37,12 +37,12 @@ module Canon
           attributes.each do |attr|
             # Try format-specific ENV var first
             env_key = EnvSchema.env_key(format, config_type, attr)
-            value = ENV[env_key]
+            value = ENV.fetch(env_key, nil)
 
             # Fall back to global ENV var if format-specific not set
             if value.nil?
               global_key = EnvSchema.global_env_key(attr)
-              value = ENV[global_key]
+              value = ENV.fetch(global_key, nil)
             end
 
             # Convert and store if value exists
@@ -57,7 +57,7 @@ module Canon
           result = {}
           attributes.each do |attr|
             global_key = EnvSchema.global_env_key(attr)
-            value = ENV[global_key]
+            value = ENV.fetch(global_key, nil)
 
             if value
               result[attr] = TypeConverter.convert(attr, value)

data/lib/canon/diff/diff_block_builder.rb

@@ -66,13 +66,13 @@ module Canon
       # Create a DiffBlock from lines
       def create_block(start_idx, end_idx, diff_lines)
         # Determine types from diff_lines
-        types = diff_lines.map(&:type).uniq.map do |t|
+        types = diff_lines.map(&:type).uniq.filter_map do |t|
           case t
           when :added then "+"
           when :removed then "-"
           when :changed then "!"
           end
-        end.compact
+        end
 
         # Create block
         block = DiffBlock.new(

data/lib/canon/diff/diff_context_builder.rb

@@ -33,7 +33,7 @@ grouping_lines: nil)
         grouped_blocks = if @grouping_lines
                            group_nearby_blocks(@diff_blocks, @grouping_lines)
                          else
-                           @diff_blocks.map { |block| [block] }
+                           @diff_blocks.zip
                          end
 
         # Create contexts with context lines

data/lib/canon/diff/diff_node_mapper.rb

@@ -182,7 +182,7 @@ module Canon
             if node.respond_to?(:name) && node.name == line_element_name
               true
             # Check if the node's parent has the matching name (for TextNode diffs)
-            elsif node.respond_to?(:parent) && node.parent.respond_to?(:name) && node.parent.name == line_element_name
+            elsif node.respond_to?(:parent) && node.parent.respond_to?(:name) && node.parent.name == line_element_name # rubocop:disable Style/IfWithBooleanLiteralBranches
               true
             else
               false

data/lib/canon/diff_formatter/by_line/base_formatter.rb

@@ -160,7 +160,7 @@ module Canon
 
           # Keep only context_lines after the last change
           keep_until = [last_change_pos + context_lines, hunk.length - 1].min
-          hunk.slice!(keep_until + 1..-1) if keep_until < hunk.length - 1
+          hunk.slice!((keep_until + 1)..-1) if keep_until < hunk.length - 1
         end
 
         # Colorize text if color is enabled
@@ -454,7 +454,7 @@ module Canon
           # Try to get from config if available
           config = Canon::Config.instance
           # Default to 10,000 if config not available
-          config&.xml&.diff&.max_diff_lines || 10_000
+          config&.xml&.diff&.max_diff_lines || 10_000 # rubocop:disable Style/SafeNavigationChainLength
         end
 
         # Build set of children of matched parents

data/lib/canon/diff_formatter/by_line/html_formatter.rb

@@ -294,9 +294,9 @@ module Canon
                                                            @diff_grouping_lines)
                               format_diff_groups(groups)
                             else
-                              diff_sections.map do |s|
+                              diff_sections.filter_map do |s|
                                 s[:formatted]
-                              end.compact.join("\n\n")
+                              end.join("\n\n")
                             end
 
           output << formatted_diffs

data/lib/canon/diff_formatter/by_line/xml_formatter.rb

@@ -289,9 +289,9 @@ module Canon
                                                            @diff_grouping_lines)
                               format_diff_groups(groups)
                             else
-                              diff_sections.map do |s|
+                              diff_sections.filter_map do |s|
                                 s[:formatted]
-                              end.compact.join("\n\n")
+                              end.join("\n\n")
                             end
 
           output << formatted_diffs

data/lib/canon/diff_formatter/by_object/base_formatter.rb

@@ -298,7 +298,7 @@ show_diffs: :all)
           # Try to get from config if available
           config = Canon::Config.instance
           # Default to 10,000 if config not available
-          config&.xml&.diff&.max_diff_lines || 10_000
+          config&.xml&.diff&.max_diff_lines || 10_000 # rubocop:disable Style/SafeNavigationChainLength
         end
       end
     end

data/lib/canon/diff_formatter/by_object/json_formatter.rb

@@ -293,7 +293,9 @@ module Canon
             if value.empty?
               "{}"
             else
-              "{Hash with #{value.keys.length} keys: #{value.keys.take(3).map(&:to_s).join(', ')}#{value.keys.length > 3 ? '...' : ''}}"
+              "{Hash with #{value.keys.length} keys: " \
+                "#{value.keys.take(3).join(', ')}" \
+                "#{'...' if value.keys.length > 3}}"
             end
           else
             value.inspect

data/lib/canon/diff_formatter/by_object/xml_formatter.rb

@@ -73,7 +73,7 @@ module Canon
             path = extract_dom_path(diff)
           end
 
-          super(tree, path, diff)
+          super
         end
 
         private

data/lib/canon/diff_formatter/debug_output.rb

@@ -27,9 +27,9 @@ module Canon
 
           output = []
           output << ""
-          output << "=" * 80
+          output << ("=" * 80)
           output << "CANON VERBOSE MODE - DETAILED OPTIONS"
-          output << "=" * 80
+          output << ("=" * 80)
           output << ""
 
           # Show match options as a table
@@ -44,7 +44,7 @@ module Canon
           output << format_comparison_summary(comparison_result)
           output << ""
 
-          output << "=" * 80
+          output << ("=" * 80)
           output << ""
 
           output.join("\n")
@@ -63,9 +63,7 @@ module Canon
           internal_keys = %i[tree_diff_operations tree_diff_statistics
                              tree_diff_matching]
 
-          rows = comparison_result.match_options.reject do |dimension, _behavior|
-            internal_keys.include?(dimension)
-          end.map do |dimension, behavior|
+          rows = comparison_result.match_options.except(*internal_keys).map do |dimension, behavior|
             {
               dimension: dimension.to_s,
               behavior: behavior.to_s,

data/lib/canon/diff_formatter/diff_detail_formatter/dimension_formatter.rb

@@ -148,7 +148,7 @@ module Canon
             changed_str = changed.map do |prefix|
               attr_name = prefix.empty? ? "xmlns" : "xmlns:#{prefix}"
               "#{ColorHelper.colorize(attr_name, :cyan, use_color)}: " \
-              "\"#{ns_decls1[prefix]}\" → \"#{ns_decls2[prefix]}\""
+                "\"#{ns_decls1[prefix]}\" → \"#{ns_decls2[prefix]}\""
             end.join(", ")
             changes_parts << "Changed: #{changed_str}"
           end

data/lib/canon/diff_formatter/diff_detail_formatter/node_utils.rb

@@ -144,6 +144,11 @@ module Canon
           elsif node.respond_to?(:get_attribute)
             attr = node.get_attribute(attr_name)
             attr.respond_to?(:value) ? attr.value : attr
+          elsif node.respond_to?(:attribute_nodes)
+            attribute_node = node.attribute_nodes.find do |attr|
+              attr.name == attr_name.to_s
+            end
+            attribute_node&.value
           end
         end
 
@@ -162,6 +167,8 @@ module Canon
             node.inner_text
           elsif node.respond_to?(:value)
             node.value
+          elsif node.respond_to?(:node_info)
+            node.node_info
           elsif node.respond_to?(:to_s)
             node.to_s
           else

data/lib/canon/diff_formatter/diff_detail_formatter.rb

@@ -109,6 +109,12 @@ module Canon
           output << "#{colorize('Location:', :cyan, use_color,
                                 bold: true)}  #{colorize(location, :blue,
                                                          use_color)}"
+
+          # show reason if available
+          if diff.respond_to?(:reason) && diff.reason
+            output << "#{colorize('Reason:', :cyan, use_color,
+                                  bold: true)}  #{colorize(diff.reason, :yellow, use_color)}"
+          end
           output << ""
 
           # Dimension-specific details

data/lib/canon/diff_formatter.rb

@@ -410,7 +410,7 @@ module Canon
       when :xml
         [
           Canon::Xml::C14n.canonicalize(expected, with_comments: false).gsub(
-            /></, ">\n<"
+            "><", ">\n<"
           ),
           Canon::Xml::C14n.canonicalize(actual, with_comments: false).gsub(
             />\s+$/, ""
@@ -510,11 +510,11 @@ module Canon
       output << colorize("=== ORIGINAL INPUTS (Raw) ===", :cyan, :bold)
       output << ""
       output << colorize("EXPECTED:", :yellow, :bold)
-      output << "-" * 70
+      output << ("-" * 70)
       output << raw1
       output << ""
       output << colorize("RECEIVED:", :yellow, :bold)
-      output << "-" * 70
+      output << ("-" * 70)
       output << raw2
       output << ""
       output << ""
@@ -543,11 +543,11 @@ preprocessing_info = nil)
       end
       output << ""
       output << colorize("EXPECTED:", :yellow, :bold)
-      output << "-" * 70
+      output << ("-" * 70)
       output << preprocessed1
       output << ""
       output << colorize("RECEIVED:", :yellow, :bold)
-      output << "-" * 70
+      output << ("-" * 70)
       output << preprocessed2
       output << ""
       output << ""

data/lib/canon/errors.rb

@@ -83,13 +83,13 @@ module Canon
       case limit_type
       when :file_size
         "File size (#{format_bytes(actual)}) exceeds limit (#{format_bytes(limit)}). " \
-          "Increase limit via CANON_MAX_FILE_SIZE or config.diff.max_file_size"
+        "Increase limit via CANON_MAX_FILE_SIZE or config.diff.max_file_size"
       when :node_count
         "Tree node count (#{actual}) exceeds limit (#{limit}). " \
-          "Increase limit via CANON_MAX_NODE_COUNT or config.diff.max_node_count"
+        "Increase limit via CANON_MAX_NODE_COUNT or config.diff.max_node_count"
       when :diff_lines
         "Diff output (#{actual} lines) exceeds limit (#{limit} lines). " \
-          "Output truncated. Increase limit via CANON_MAX_DIFF_LINES or config.diff.max_diff_lines"
+        "Output truncated. Increase limit via CANON_MAX_DIFF_LINES or config.diff.max_diff_lines"
       else
         "Size limit exceeded: #{limit_type} (#{actual} > #{limit})"
       end

data/lib/canon/rspec_matchers.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "canon" unless defined?(::Canon)
+require "canon" unless defined?(Canon)
 require "canon/comparison"
 require "canon/diff_formatter"
 require "canon/config"
@@ -137,7 +137,7 @@ module Canon
         @target
       end
 
-      def diffable
+      def diffable # rubocop:disable Naming/PredicateMethod
         false
       end
 

data/lib/canon/tree_diff/adapters/json_adapter.rb

@@ -166,13 +166,9 @@ module Canon
         # @param tree_node [Core::TreeNode] Array tree node
         # @return [Array] Reconstructed array
         def build_array(tree_node)
-          array = []
-
-          tree_node.children.each do |child|
-            array << from_tree(child)
+          tree_node.children.map do |child|
+            from_tree(child)
           end
-
-          array
         end
 
         # Parse value from value TreeNode

data/lib/canon/tree_diff/adapters/yaml_adapter.rb

@@ -169,13 +169,9 @@ module Canon
         # @param tree_node [Core::TreeNode] Array tree node
         # @return [Array] Reconstructed array
         def build_array(tree_node)
-          array = []
-
-          tree_node.children.each do |child|
-            array << from_tree(child)
+          tree_node.children.map do |child|
+            from_tree(child)
           end
-
-          array
         end
 
         # Parse value from value TreeNode

data/lib/canon/tree_diff/core/matching.rb

@@ -28,7 +28,7 @@ module Canon
         # @param node1 [TreeNode] Node from tree 1
         # @param node2 [TreeNode] Node from tree 2
         # @return [Boolean] true if added, false if violates constraints
-        def add(node1, node2)
+        def add(node1, node2) # rubocop:disable Naming/PredicateMethod
           return false unless valid_pair?(node1, node2)
 
           @pairs << [node1, node2]
@@ -43,7 +43,7 @@ module Canon
         # @param node1 [TreeNode] Node from tree 1
         # @param node2 [TreeNode] Node from tree 2
         # @return [Boolean] true if removed, false if not found
-        def remove(node1, node2)
+        def remove(node1, node2) # rubocop:disable Naming/PredicateMethod
           removed = @pairs.delete([node1, node2])
           return false unless removed
 

data/lib/canon/tree_diff/core/node_signature.rb

@@ -81,12 +81,10 @@ module Canon
         #
         # @return [Array<String>] Path components
         def compute_path
-          components = []
-
           # Build path from root to node
           ancestors = @node.ancestors.reverse
-          ancestors.each do |ancestor|
-            components << path_component(ancestor)
+          components = ancestors.map do |ancestor|
+            path_component(ancestor)
           end
 
           # Add the node itself

data/lib/canon/tree_diff/core/tree_node.rb

@@ -292,7 +292,7 @@ module Canon
           attr_diff = attribute_difference(other)
 
           # Weighted combination
-          depth_diff * 0.3 + content_diff * 0.5 + attr_diff * 0.2
+          (depth_diff * 0.3) + (content_diff * 0.5) + (attr_diff * 0.2)
         end
 
         # Get content as a set for similarity calculation

data/lib/canon/tree_diff/operation_converter.rb

@@ -352,7 +352,7 @@ module Canon
       #
       # @param dimension [Symbol] Match dimension
       # @return [Boolean] true if normative (should be shown)
-      def determine_normative(dimension)
+      def determine_normative(dimension) # rubocop:disable Naming/PredicateMethod
         # Check match options behavior for this dimension
         behavior = @match_options.behavior_for(dimension)
 

data/lib/canon/tree_diff/operation_converter_helpers/reason_builder.rb

@@ -160,7 +160,7 @@ module Canon
           text = text.to_s
           return text if text.length <= max_length
 
-          "#{text[0...max_length - 3]}..."
+          "#{text[0...(max_length - 3)]}..."
         end
       end
     end

data/lib/canon/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Canon
-  VERSION = "0.1.14"
+  VERSION = "0.1.15"
 end

data/lib/canon/xml/line_range_mapper.rb

@@ -40,7 +40,7 @@ module Canon
       # Line range for an element
       LineRange = Struct.new(:start_line, :end_line, :elem) do
         def contains?(line_num)
-          line_num >= start_line && line_num <= end_line
+          line_num.between?(start_line, end_line)
         end
 
         def length

data/lib/canon/xml/nodes/attribute_node.rb

@@ -48,6 +48,10 @@ module Canon
         def xml_base?
           xml_attribute? && name == "base"
         end
+
+        def node_info
+          "name: #{name} value: #{value} namespace_uri: #{namespace_uri} prefix: #{prefix}"
+        end
       end
     end
   end

data/lib/canon/xml/nodes/element_node.rb

@@ -50,6 +50,10 @@ module Canon
             [attr.namespace_uri.to_s, attr.local_name]
           end
         end
+
+        def node_info
+          "name: #{name} namespace_uri: #{namespace_uri} prefix: #{prefix}"
+        end
       end
     end
   end

data/lib/canon/xml/nodes/namespace_node.rb

@@ -32,6 +32,10 @@ module Canon
         def xml_namespace?
           prefix == "xml" && uri == "http://www.w3.org/XML/1998/namespace"
         end
+
+        def node_info
+          "prefix: #{prefix} uri: #{uri}"
+        end
       end
     end
   end

data/lib/canon/xml/nodes/processing_instruction_node.rb

@@ -18,6 +18,10 @@ module Canon
         def node_type
           :processing_instruction
         end
+
+        def node_info
+          "data: #{data} target: #{target}"
+        end
       end
     end
   end

data/lib/canon.rb

@@ -13,7 +13,7 @@ require_relative "canon/formatters/html4_formatter"
 require_relative "canon/formatters/html5_formatter"
 require_relative "canon/comparison"
 
-require_relative "canon/rspec_matchers" if defined?(::RSpec)
+require_relative "canon/rspec_matchers" if defined?(RSpec)
 
 module Canon
   SUPPORTED_FORMATS = %i[xml yaml json html html4 html5 string].freeze

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: canon
 version: !ruby/object:Gem::Version
-  version: 0.1.14
+  version: 0.1.15
 platform: ruby
 authors:
 - Ribose Inc.
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-02-25 00:00:00.000000000 Z
+date: 2026-03-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: diff-lcs
@@ -351,6 +351,7 @@ metadata:
   homepage_uri: https://github.com/lutaml/canon
   source_code_uri: https://github.com/lutaml/canon
   changelog_uri: https://github.com/lutaml/canon
+  rubygems_mfa_required: 'true'
 post_install_message:
 rdoc_options: []
 require_paths: