canon 0.3.2 → 0.3.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b000e9e89684d66d4f037a950a2121a88717dd6c72d975648742f32b5a83fb9f
-  data.tar.gz: c5e8b5ecfdcb8435559be3e6c940e8e8bec99202054b90fa857f9e3afd3097e2
+  metadata.gz: 52174949c77e8f33286b2875ac5e4052e6f5671b64e53e48a3b0b3128bb598a1
+  data.tar.gz: 0b2efb1c4fa7e060f6d54f1c149bd775d45fab66e09ac7bbf306f0a6142360fa
 SHA512:
-  metadata.gz: '0804e7a2a07ff2687e363ee2bd0a1e2d00a5fe826e45ae5bd69fe26749bf2e2b1517a6fb2fcd3518e41d7d9bf6bf0b76fd8e704c288293724d27cbc3613978f4'
-  data.tar.gz: 6d9dca5aa3c29bbde5d067ca02e2a22c4d7bee098635e0fd8b48df16fbf08102fa713608f1965d198f3ace1c064743916b57d34d7a2cc2a6b845902e1553a045
+  metadata.gz: 4e78d8618af59c255979bf3987a1be3e7419e009354b73ca64df75600ce8c2f1cc90aa515eb14be97149fe5426243efacda4ff42eab63b5fc8f291407baa046e
+  data.tar.gz: 2596286ac94921e6021590fe6f1ece71d827145743cc993f703cfca079fb91472845c67f25a3e9ad03aecd471c6cc482cd78854a5e9ce85d16bd8f0ea5643636

data/lib/canon/version.rb

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

data/lib/canon/xml/data_model.rb

@@ -228,7 +228,9 @@ inherited_namespaces: nil)
       def self.build_text_node(nokogiri_text, preserve_whitespace: false)
         content = nokogiri_text.content
 
-        if !preserve_whitespace && content.strip.empty? && nokogiri_text.parent.is_a?(Nokogiri::XML::Element)
+        unless WhitespacePolicy.keep_dom_text?(content,
+                                               preserve_whitespace: preserve_whitespace,
+                                               element_parent: nokogiri_text.parent.is_a?(Nokogiri::XML::Element))
           return nil
         end
 
@@ -394,7 +396,7 @@ preserve_whitespace: false)
       # whitespace-only skip rule matches the wrapper path exactly.
       def self.build_moxml_text_from_record(record, preserve_whitespace)
         content = record[:text].to_s
-        return nil if !preserve_whitespace && content.strip.empty?
+        return nil unless WhitespacePolicy.keep_dom_text?(content, preserve_whitespace: preserve_whitespace)
 
         Nodes::TextNode.new(value: content, original: content)
       end
@@ -502,7 +504,9 @@ inherited_namespaces: nil)
       def self.build_moxml_text_node(moxml_text, preserve_whitespace: false)
         content = moxml_text.content
 
-        if !preserve_whitespace && content.strip.empty? && moxml_text.parent.is_a?(Moxml::Element)
+        unless WhitespacePolicy.keep_dom_text?(content,
+                                               preserve_whitespace: preserve_whitespace,
+                                               element_parent: moxml_text.parent.is_a?(Moxml::Element))
           return nil
         end
 

data/lib/canon/xml/sax.rb

@@ -10,13 +10,16 @@ module Canon
     # API to that protocol. OCP: a new engine is a new driver plus one line
     # here; the builder never changes.
     #
-    # Engine choice: leptris SAX outperforms Nokogiri SAX (1.10x min-of-N
-    # on canon's builder, 53KB doc) through moxml 0.5.12's
-    # interest-proportional callbacks — BUT the batch-attribute transport
-    # reads uninitialized memory at certain buffer offsets, corrupting the
-    # first N attributes of an element (leptris-ruby#95). Until that is
-    # fixed, CRuby keeps the Nokogiri driver; flipping is the one-line
-    # change below.
+    # Engine choice: the leptris SAX flip is armed but blocked twice over.
+    # The batch-attribute corruption (leptris-ruby#95) is answered by a
+    # correctness-first path in leptris 1.9.36 — attributes are correct,
+    # but that path DROPS namespace declarations from SAX events
+    # entirely (raw `["<root xmlns:a=.../>", []]` — xmlns pairs vanish),
+    # blinding namespace comparison. Until xmlns reporting is restored
+    # (and then the fast path returns with the C fix), CRuby keeps the
+    # Nokogiri driver; the flip is the one-line change below. With
+    # leptris absent (moxml resolving nokogiri) the driver calls Nokogiri
+    # directly — the wrapper layer only adds overhead there.
     module Sax
       autoload :NokogiriDriver, "canon/xml/sax/nokogiri_driver"
       autoload :MoxmlDriver, "canon/xml/sax/moxml_driver"

data/lib/canon/xml/sax_builder.rb

@@ -198,22 +198,12 @@ strip_doctype: false)
           return
         end
 
-        # Skip whitespace-only text nodes unless:
-        # 1. preserve_whitespace is true, OR
-        # 2. The content contains CR (from 
 entities) which must be preserved for C14N, OR
-        # 3. The content contains non-ASCII whitespace (NBSP U+00A0, ideographic
-        #    space U+3000, etc.) — those are semantically meaningful content,
-        #    not pretty-print indentation, and must survive parsing so the
-        #    comparator can detect Unicode whitespace-type differences.
-        #
-        # Strip only when the node is pure ASCII whitespace (space, tab, CR, LF).
-        # This lets pretty-printed fixtures work (indent nodes stripped) while
-        # preserving NBSP-only text nodes.
-        if !@preserve_whitespace && decoded_string.gsub(/[ \t\r\n]/,
-                                                        "").empty? && parent.node_type == :element && !decoded_string.include?("\r")
-          # Only skip if parent is an element (not root)
-          return
-        end
+        # Skip whitespace-only text nodes unless preserving, per the SAX
+        # policy (CR-bearing content always survives for C14N;
+        # non-ASCII whitespace is meaningful content).
+        return unless WhitespacePolicy.keep_sax_text?(decoded_string,
+                                                      preserve_whitespace: @preserve_whitespace,
+                                                      element_parent: parent.node_type == :element)
 
         text = Nodes::TextNode.new(value: decoded_string, original: raw_string)
         parent.add_child(text)

data/lib/canon/xml/whitespace_policy.rb

@@ -0,0 +1,37 @@
+# frozen_string_literal: true
+
+module Canon
+  module Xml
+    # Parse-time whitespace policy: whether a character-data node
+    # survives conversion. One home for the keep/strip rules so the
+    # DOM/SAX differences are visible here instead of implied by
+    # copy-paste across conversion sites.
+    module WhitespacePolicy
+      module_function
+
+      # DOM conversion rule: whitespace-only text is dropped unless
+      # preserving. Non-ASCII whitespace (NBSP, U+3000) survives —
+      # String#strip only removes ASCII whitespace.
+      #
+      # NOTE: CR-only nodes are dropped on this path; the SAX rule keeps
+      # them (character references must survive for C14N).
+      def keep_dom_text?(content, preserve_whitespace:, element_parent: true)
+        return true if preserve_whitespace
+        return true unless element_parent
+
+        !content.strip.empty?
+      end
+
+      # SAX rule: same shape, plus CR-bearing content is always kept
+      # (
 must survive parsing for C14N) — only runs of pure ASCII
+      # whitespace (space, tab, CR, LF) are dropped when not preserving.
+      def keep_sax_text?(content, preserve_whitespace:, element_parent: true)
+        return true if preserve_whitespace
+        return true unless element_parent
+        return true if content.include?("\r")
+
+        !content.gsub(/[ \t\r\n]/, "").empty?
+      end
+    end
+  end
+end

data/lib/canon/xml.rb

@@ -28,6 +28,7 @@ module Canon
     autoload :Sax, "canon/xml/sax"
     autoload :SaxBuilder, "canon/xml/sax_builder"
     autoload :WhitespaceNormalizer, "canon/xml/whitespace_normalizer"
+    autoload :WhitespacePolicy, "canon/xml/whitespace_policy"
     autoload :XmlBaseHandler, "canon/xml/xml_base_handler"
     autoload :XPathEngine, "canon/xml/xpath_engine"
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: canon
 version: !ruby/object:Gem::Version
-  version: 0.3.2
+  version: 0.3.3
 platform: ruby
 authors:
 - Ribose Inc.
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-08-28 00:00:00.000000000 Z
+date: 2026-08-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: diff-lcs
@@ -401,6 +401,7 @@ files:
 - lib/canon/xml/sax/nokogiri_driver.rb
 - lib/canon/xml/sax_builder.rb
 - lib/canon/xml/whitespace_normalizer.rb
+- lib/canon/xml/whitespace_policy.rb
 - lib/canon/xml/xml_base_handler.rb
 - lib/canon/xml/xpath_engine.rb
 - lib/canon/xml_backend.rb