canon 0.3.38 → 0.3.40

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: dc55555287eb70fb4fc5201029aea89c2af55d8a9df26549d1d17b639a6de5de
-  data.tar.gz: 59ac3ba881495f29ee1355eda305da05d03d62a5611aa738404bf029f66d1721
+  metadata.gz: 7ceb43d90065cc39a64f8c74b6317dd9f8752aaa807ec7869ae3188c9e494b43
+  data.tar.gz: d6375f29977244331faff0d8dd54396d52494133d77e4286011804a63bd1d23f
 SHA512:
-  metadata.gz: 6b2d535b82b40ece128bea902b603b1b43c06c7bf4f40535f651e3b43c24674345de4e0c0767380fd6fc50d2986dc80ecf0e2c1b2d5286a7dd02524d65641920
-  data.tar.gz: '09af782d9c914502a54ca9b961448c89502bb20c3cffa93ec9268b7afc72099bcea2dcc2f7f8e36ccbf7fd6a6f8e3320ced5b7edfd67f7d70d0f5f95e7cf8d85'
+  metadata.gz: bed4791632c2d75f6db4e89e5c991b05a92593fdfaa16d7b20c21dbf5be3070914b0b4214567e9ca3045db8f9d3a8312a218acca28574214d76d817ee09706fd
+  data.tar.gz: 0d3f8b7bad01bd3bd1e089f5488dac9cd23ca9761ab287ad3f5f0340e918262e5b4fcf63c4bb64ddb427e19c3e1bec878b0bef9aaf93b596c5a2a1f125dadd99

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

@@ -67,8 +67,8 @@ module Canon
           output = []
 
           # Detect non-ASCII characters
-          all_text = (lines1 + lines2).join
-          non_ascii = Legend.detect_non_ascii(all_text, @visualization_map)
+          non_ascii = Legend.detect_non_ascii(doc1, @visualization_map)
+          non_ascii.merge!(Legend.detect_non_ascii(doc2, @visualization_map))
 
           # Add Unicode legend if needed
           unless non_ascii.empty?

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

@@ -19,8 +19,8 @@ module Canon
           lines2 = doc2.split("\n", -1)
 
           # Detect non-ASCII characters in the diff
-          all_text = (lines1 + lines2).join
-          non_ascii = Legend.detect_non_ascii(all_text, @visualization_map)
+          non_ascii = Legend.detect_non_ascii(doc1, @visualization_map)
+          non_ascii.merge!(Legend.detect_non_ascii(doc2, @visualization_map))
 
           # Add Unicode legend if any non-ASCII characters detected
           unless non_ascii.empty?

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

@@ -72,9 +72,11 @@ module Canon
 
           output = []
 
-          # Detect non-ASCII characters
-          all_text = (lines1 + lines2).join
-          non_ascii = Legend.detect_non_ascii(all_text, @visualization_map)
+          # Detect non-ASCII characters — scan the source strings
+          # directly; joining the split lines back into one document
+          # per render was pure garbage.
+          non_ascii = Legend.detect_non_ascii(doc1, @visualization_map)
+          non_ascii.merge!(Legend.detect_non_ascii(doc2, @visualization_map))
 
           # Add Unicode legend if needed
           unless non_ascii.empty?

data/lib/canon/version.rb

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

data/lib/canon/xml/digest_gate.rb

@@ -78,7 +78,7 @@ module Canon
         return nil unless root
 
         skeleton = doc.children.filter_map do |child|
-          next if child.equal?(root)
+          next if Canon::XmlParsing.same_engine_node?(child, root)
 
           case child
           when Moxml::Comment, Moxml::ProcessingInstruction then child.to_s

data/lib/canon/xml/tree_builder.rb

@@ -149,7 +149,7 @@ module Canon
       def add_document_children(root, children, document_element,
                                 skip_types = [])
         children.each do |child|
-          next if child.equal?(document_element)
+          next if Canon::XmlParsing.same_engine_node?(child, document_element)
           next if skip_types.any? { |type| child.is_a?(type) }
 
           node = yield child

data/lib/canon/xml_parsing.rb

@@ -63,6 +63,33 @@ module Canon
         end
       end
 
+      # Same underlying engine node? Wrapper identity first (correct
+      # everywhere wrappers are unified). Fallback: moxml 0.5.36-0.5.39
+      # wrap Document#root and Document#children entries in DIFFERENT
+      # wrapper classes for one node (NativeNode vs the FFI Element —
+      # moxml#216), so identity fails exactly where consumers exclude
+      # the document element from children; the shared C pointer
+      # address is the stable identity. Inert once wrappers unify.
+      def same_engine_node?(left, right)
+        return true if left.equal?(right)
+        return false unless defined?(::Moxml::Node) &&
+          left.is_a?(::Moxml::Node) && right.is_a?(::Moxml::Node)
+
+        address_left = moxml_node_address(left)
+        address_right = moxml_node_address(right)
+        !address_left.nil? && address_left == address_right
+      end
+
+      def moxml_node_address(wrapper)
+        native = wrapper.native
+        case native
+        when ::Leptris::XML::NativeNode then native.address
+        when ::Leptris::XML::Node then native.c_ptr.address
+        end
+      rescue StandardError
+        nil
+      end
+
       # --- Type checks (any recognized engine node) ---
 
       def document?(obj)

data/lib/tasks/benchmark_runner.rb

@@ -225,6 +225,13 @@ compare_against: nil)
       failure_means: "Slow formatting affects serialization performance. C14N is critical for digital signatures and XML canonicalization.",
       compare_against: "Previous branch (main).",
     },
+    by_line_rendering: {
+      name: "By-Line Rendering",
+      icon: "🖨️",
+      description: "Verbose by_line diff rendering (enricher, line builder, formatters). Referees the #86 rendering lane.",
+      failure_means: "Slow diff display affects developer-facing CLI output and CI failure reports.",
+      compare_against: "Previous branch (main). Inputs: documents with scattered value differences.",
+    },
     data_comparison: {
       name: "Data Comparison",
       icon: "🧮",
@@ -272,6 +279,9 @@ compare_against: nil)
       { name: "JSON", method: :json_format, desc: "JSON formatting" },
       { name: "YAML", method: :yaml_format, desc: "YAML formatting" },
     ],
+    by_line_rendering: [
+      { name: "XML", method: :by_line_render_xml, desc: "by_line XML render" },
+    ],
     data_comparison: [
       { name: "JSON", method: :json_compare_equivalent,
         desc: "JSON equivalence" },
@@ -568,6 +578,18 @@ compare_against: nil)
       yaml = DataGenerator.generate_yaml(items: @items)
       data = YAML.safe_load(yaml, permitted_classes: [Time])
       measure { Canon.format_yaml(data) }
+    when :by_line_render_xml
+      xml1 = DataGenerator.generate_xml(items: @items)
+      xml2 = DataGenerator.generate_xml(items: @items)
+      formatter = Canon::DiffFormatter.new(mode: :by_line,
+                                           display_preprocessing: :pretty_print,
+                                           use_color: false)
+      result = Canon::Comparison.equivalent?(xml1, xml2, format: :xml,
+                                                         verbose: true)
+      measure do
+        formatter.format(result.differences, :xml,
+                         doc1: xml1, doc2: xml2)
+      end
     when :json_compare_equivalent
       json1 = DataGenerator.generate_json(items: @items)
       json2 = DataGenerator.generate_json(items: @items)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: canon
 version: !ruby/object:Gem::Version
-  version: 0.3.38
+  version: 0.3.40
 platform: ruby
 authors:
 - Ribose Inc.
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-09-14 00:00:00.000000000 Z
+date: 2026-09-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: diff-lcs
@@ -44,14 +44,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.5.0
+        version: 0.5.41
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.5.0
+        version: 0.5.41
 - !ruby/object:Gem::Dependency
   name: nokogiri
   requirement: !ruby/object:Gem::Requirement