canon 0.2.11 → 0.3.0

data/lib/canon/diff/diff_node_mapper.rb

@@ -1,6 +1,5 @@
 # frozen_string_literal: true
 
-require_relative "diff_line"
 require "set"
 
 module Canon
@@ -159,8 +158,6 @@ module Canon
       # @param diff_lines [Array<DiffLine>] The diff lines to update
       # @param lcs_diffs [Array<Diff::LCS::Change>] The LCS changes
       def apply_block_formatting!(diff_lines, lcs_diffs)
-        require_relative "formatting_detector"
-
         blocks = group_change_blocks(lcs_diffs)
 
         blocks.each do |block|
@@ -265,7 +262,6 @@ module Canon
       # @param line2 [String] Second line
       # @return [Boolean] true if formatting-only difference
       def formatting_only_line?(line1, line2)
-        require_relative "formatting_detector"
         FormattingDetector.formatting_only?(line1, line2)
       end
 

data/lib/canon/diff/diff_report_builder.rb

@@ -1,9 +1,5 @@
 # frozen_string_literal: true
 
-require_relative "diff_report"
-require_relative "diff_block_builder"
-require_relative "diff_context_builder"
-
 module Canon
   module Diff
     # Builds a complete DiffReport from DiffLines

data/lib/canon/diff/formatting_detector.rb

@@ -99,7 +99,6 @@ module Canon
       def self.decode_xml_entities(text)
         return text unless text.include?("&")
 
-        require_relative "../tree_diff/core/xml_entity_decoder"
         Canon::TreeDiff::Core::XmlEntityDecoder.decode_xml_entities(text)
       end
 

data/lib/canon/diff/node_serializer.rb

@@ -1,12 +1,5 @@
 # frozen_string_literal: true
 
-require_relative "../xml/data_model"
-require_relative "../xml/nodes/text_node"
-require_relative "../xml/nodes/comment_node"
-require_relative "../xml/nodes/element_node"
-require_relative "../xml/nodes/processing_instruction_node"
-require_relative "../xml/nodes/root_node"
-
 module Canon
   module Diff
     # Serializes nodes from different parsing libraries into canonical strings

data/lib/canon/diff.rb

@@ -0,0 +1,39 @@
+# frozen_string_literal: true
+
+module Canon
+  # Diff representation and pipeline construction.
+  #
+  # This namespace holds the structural artifacts produced by the diff
+  # pipeline — DiffNode, DiffLine, DiffBlock, DiffContext, DiffReport —
+  # plus the builders that assemble them (DiffNodeMapper, DiffLineBuilder,
+  # DiffBlockBuilder, DiffContextBuilder, DiffReportBuilder) and the
+  # supporting services (PathBuilder, NodeSerializer, DiffClassifier,
+  # FormattingDetector, SourceLocator, TextDecomposer, DiffCharRange,
+  # DiffNodeEnricher, XmlSerializationFormatter).
+  #
+  # All children are autoloaded from this file — never `require_relative`
+  # them from sibling files. Reference the constant and let autoload
+  # resolve it on first use.
+  module Diff
+    autoload :DiffBlock, "canon/diff/diff_block"
+    autoload :DiffBlockBuilder, "canon/diff/diff_block_builder"
+    autoload :DiffCharRange, "canon/diff/diff_char_range"
+    autoload :DiffClassifier, "canon/diff/diff_classifier"
+    autoload :DiffContext, "canon/diff/diff_context"
+    autoload :DiffContextBuilder, "canon/diff/diff_context_builder"
+    autoload :DiffLine, "canon/diff/diff_line"
+    autoload :DiffLineBuilder, "canon/diff/diff_line_builder"
+    autoload :DiffNode, "canon/diff/diff_node"
+    autoload :DiffNodeEnricher, "canon/diff/diff_node_enricher"
+    autoload :DiffNodeMapper, "canon/diff/diff_node_mapper"
+    autoload :DiffReport, "canon/diff/diff_report"
+    autoload :DiffReportBuilder, "canon/diff/diff_report_builder"
+    autoload :FormattingDetector, "canon/diff/formatting_detector"
+    autoload :NodeSerializer, "canon/diff/node_serializer"
+    autoload :PathBuilder, "canon/diff/path_builder"
+    autoload :SourceLocator, "canon/diff/source_locator"
+    autoload :TextDecomposer, "canon/diff/text_decomposer"
+    autoload :XmlSerializationFormatter,
+             "canon/diff/xml_serialization_formatter"
+  end
+end

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

@@ -2,8 +2,6 @@
 
 require "diff/lcs" unless RUBY_ENGINE == "opal"
 require "diff/lcs/hunk" unless RUBY_ENGINE == "opal"
-require_relative "../debug_output"
-require_relative "../theme"
 
 module Canon
   class DiffFormatter
@@ -22,10 +20,8 @@ module Canon
         def self.for_format(format, **options)
           case format
           when :xml
-            require_relative "xml_formatter"
             XmlFormatter.new(**options)
           when :html, :html4, :html5
-            require_relative "html_formatter"
             # Determine HTML version from format
             version = case format
                       when :html5 then :html5
@@ -34,18 +30,15 @@ module Canon
                       end
             HtmlFormatter.new(html_version: version, **options)
           when :json
-            require_relative "json_formatter"
             JsonFormatter.new(**options)
           when :yaml
-            require_relative "yaml_formatter"
             YamlFormatter.new(**options)
           else
-            require_relative "simple_formatter"
             SimpleFormatter.new(**options)
           end
         end
 
-        # rubocop:disable Metrics/ParameterLists
+        # rubocop:disable-next Metrics/ParameterLists
         def initialize(use_color: true, context_lines: 3,
                        diff_grouping_lines: nil, visualization_map: nil,
                        show_diffs: :all, differences: [],
@@ -65,7 +58,6 @@ module Canon
           @theme = theme
           @character_visualization = character_visualization
         end
-        # rubocop:enable Metrics/ParameterLists
 
         # Get the resolved theme hash
         # @return [Hash] Theme hash
@@ -279,7 +271,7 @@ module Canon
 
           differences.select do |diff|
             # Handle both DiffNode objects and legacy Hash format
-            is_normative = if diff.respond_to?(:normative?)
+            is_normative = if diff.is_a?(Canon::Diff::DiffNode)
                              diff.normative?
                            elsif diff.is_a?(Hash) && diff.key?(:normative)
                              diff[:normative]
@@ -394,8 +386,6 @@ module Canon
         # @param diffs [Array] LCS diff array
         # @return [Array<Canon::Diff::DiffBlock>] Array of diff blocks
         def identify_diff_blocks(diffs)
-          require_relative "../../diff/diff_block"
-
           blocks = []
           current_start = nil
           current_types = []
@@ -467,8 +457,6 @@ module Canon
         # @return [Array<Canon::Diff::DiffContext>] Array of diff contexts
         def expand_contexts_with_context_lines(contexts, context_lines,
                                                 total_lines)
-          require_relative "../../diff/diff_context"
-
           contexts.map do |context|
             first_block = context.first
             last_block = context.last
@@ -736,10 +724,8 @@ module Canon
             next unless match.status == :matched
 
             [match.elem1, match.elem2].compact.each do |elem|
-              next unless elem.respond_to?(:children)
-
               elem.children.each do |child|
-                children.add(child) if child.respond_to?(:name)
+                children.add(child) if Canon::Comparison::NodeInspector.element_node?(child)
               end
             end
           end
@@ -778,8 +764,8 @@ module Canon
           return true if elements_with_semantic_diffs.include?(element)
 
           # Check all descendants
-          if element.respond_to?(:children)
-            element.children.any? do |child|
+          if Canon::Comparison::NodeInspector.element_node?(element)
+            Canon::Comparison::NodeInspector.children(element).any? do |child|
               has_semantic_diff_in_subtree?(child, elements_with_semantic_diffs)
             end
           else

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

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require_relative "base_formatter"
-require_relative "../legend"
 require "set"
 
 module Canon
@@ -12,7 +10,7 @@ module Canon
       class HtmlFormatter < BaseFormatter
         attr_reader :html_version
 
-        # rubocop:disable Metrics/ParameterLists
+        # rubocop:disable-next Metrics/ParameterLists
         def initialize(use_color: true, context_lines: 3,
                        diff_grouping_lines: nil, visualization_map: nil,
                        html_version: :html4, show_diffs: :all, differences: [],
@@ -27,7 +25,6 @@ module Canon
                 character_visualization: character_visualization)
           @html_version = html_version
         end
-        # rubocop:enable Metrics/ParameterLists
 
         # Format DOM-guided HTML diff
         #
@@ -53,11 +50,6 @@ module Canon
             return ""
           end
 
-          require_relative "../../html/data_model"
-          require_relative "../../xml/element_matcher"
-          require_relative "../../xml/line_range_mapper"
-          require_relative "../../pretty_printer/html"
-
           output = []
 
           begin
@@ -107,7 +99,6 @@ module Canon
             end
 
             output << ""
-            require_relative "simple_formatter"
             simple = SimpleFormatter.new(
               use_color: @use_color,
               context_lines: @context_lines,
@@ -122,9 +113,6 @@ module Canon
 
         # Format using new DiffReportBuilder pipeline
         def format_with_pipeline(doc1, doc2)
-          require_relative "../../diff/diff_node_mapper"
-          require_relative "../../diff/diff_report_builder"
-
           # Layer 2: Map DiffNodes to DiffLines
           diff_lines = Canon::Diff::DiffNodeMapper.map(@differences, doc1, doc2)
 
@@ -350,15 +338,14 @@ module Canon
 
           # Second pass: skip children of elements with diffs
           elements_with_diffs.each do |elem|
-            if elem.respond_to?(:parent)
-              current = elem.parent
-              while current
-                if current.respond_to?(:name) && elements_with_diffs.include?(current)
-                  elements_to_skip.add(elem)
-                  break
-                end
-                current = current.respond_to?(:parent) ? current.parent : nil
+            current = Canon::Comparison::NodeInspector.parent(elem)
+            while current
+              if Canon::Comparison::NodeInspector.element_node?(current) &&
+                  elements_with_diffs.include?(current)
+                elements_to_skip.add(elem)
+                break
               end
+              current = Canon::Comparison::NodeInspector.parent(current)
             end
           end
 

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

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require_relative "base_formatter"
-require_relative "../legend"
 require "strscan"
 
 module Canon
@@ -38,7 +36,6 @@ module Canon
             output << colorize(
               "Warning: JSON parsing failed (#{e.message}), using simple diff", :yellow
             )
-            require_relative "simple_formatter"
             simple = SimpleFormatter.new(
               use_color: @use_color,
               context_lines: @context_lines,

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

@@ -1,8 +1,5 @@
 # frozen_string_literal: true
 
-require_relative "base_formatter"
-require_relative "../legend"
-
 module Canon
   class DiffFormatter
     module ByLine

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

@@ -1,8 +1,5 @@
 # frozen_string_literal: true
 
-require_relative "base_formatter"
-require_relative "../legend"
-require_relative "../../tree_diff/core/xml_entity_decoder"
 require "set"
 require "strscan"
 
@@ -44,10 +41,6 @@ module Canon
             return ""
           end
 
-          require_relative "../../diff/diff_node_enricher"
-          require_relative "../../diff/diff_line_builder"
-          require_relative "../../diff/diff_report_builder"
-
           # Compute line number width BEFORE formatting
           compute_line_num_width(doc1, doc2)
 
@@ -201,10 +194,6 @@ module Canon
             return ""
           end
 
-          require_relative "../../xml/data_model"
-          require_relative "../../xml/element_matcher"
-          require_relative "../../xml/line_range_mapper"
-
           output = []
 
           begin
@@ -247,7 +236,6 @@ module Canon
             end
 
             output << ""
-            require_relative "simple_formatter"
             simple = SimpleFormatter.new(
               use_color: @use_color,
               context_lines: @context_lines,
@@ -622,15 +610,14 @@ module Canon
 
           # Second pass: skip children of elements with diffs
           elements_with_diffs.each do |elem|
-            if elem.respond_to?(:parent)
-              current = elem.parent
-              while current
-                if current.respond_to?(:name) && elements_with_diffs.include?(current)
-                  elements_to_skip.add(elem)
-                  break
-                end
-                current = current.respond_to?(:parent) ? current.parent : nil
+            current = Canon::Comparison::NodeInspector.parent(elem)
+            while current
+              if Canon::Comparison::NodeInspector.element_node?(current) &&
+                  elements_with_diffs.include?(current)
+                elements_to_skip.add(elem)
+                break
               end
+              current = Canon::Comparison::NodeInspector.parent(current)
             end
           end
 
@@ -721,8 +708,6 @@ module Canon
 
         # Identify contiguous diff blocks
         def identify_diff_blocks(diffs)
-          require_relative "../../diff/diff_block"
-
           blocks = []
           current_start = nil
           current_types = []
@@ -784,8 +769,6 @@ module Canon
         # Expand contexts with context lines
         def expand_contexts_with_context_lines(contexts, context_lines,
                                                 total_lines)
-          require_relative "../../diff/diff_context"
-
           contexts.map do |context|
             first_block = context.first
             last_block = context.last
@@ -803,8 +786,6 @@ module Canon
 
         # Format a context
         def format_context(context, diffs, base_line1, base_line2)
-          require_relative "../../diff/formatting_detector"
-
           # Pre-compute block-level formatting for multi-line changes
           formatting_indices = detect_block_formatting(context, diffs)
 

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

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require_relative "base_formatter"
-require_relative "../legend"
 require "strscan"
 
 module Canon
@@ -37,7 +35,6 @@ module Canon
             output << colorize(
               "Warning: YAML parsing failed (#{e.message}), using simple diff", :yellow
             )
-            require_relative "simple_formatter"
             simple = SimpleFormatter.new(
               use_color: @use_color,
               context_lines: @context_lines,

data/lib/canon/diff_formatter/by_line_formatter.rb

@@ -9,7 +9,7 @@ module Canon
     # Receives preprocessed document strings from the DiffFormatter facade
     # and delegates to format-specific ByLine formatters (XML, HTML, JSON, YAML).
     class ByLineFormatter
-      # rubocop:disable Metrics/ParameterLists
+      # rubocop:disable-next Metrics/ParameterLists
       def initialize(use_color:, visualization_map:, context_lines:,
                      diff_grouping_lines:, show_diffs:, character_visualization:,
                      legacy_terminal:, diff_mode:)
@@ -22,7 +22,6 @@ module Canon
         @legacy_terminal = legacy_terminal
         @diff_mode = diff_mode
       end
-      # rubocop:enable Metrics/ParameterLists
 
       # Format a line-by-line diff between two documents.
       #

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

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require "set"
-require_relative "../theme"
 
 module Canon
   class DiffFormatter
@@ -76,19 +75,16 @@ show_diffs: :all, theme: nil)
 show_diffs: :all, theme: nil)
           case format
           when :xml, :html
-            require_relative "xml_formatter"
             XmlFormatter.new(use_color: use_color,
                              visualization_map: visualization_map,
                              show_diffs: show_diffs,
                              theme: theme)
           when :json
-            require_relative "json_formatter"
             JsonFormatter.new(use_color: use_color,
                               visualization_map: visualization_map,
                               show_diffs: show_diffs,
                               theme: theme)
           when :yaml
-            require_relative "yaml_formatter"
             YamlFormatter.new(use_color: use_color,
                               visualization_map: visualization_map,
                               show_diffs: show_diffs,
@@ -221,23 +217,20 @@ show_diffs: :all, theme: nil)
           current = node
           visited = Set.new
 
-          while current.respond_to?(:name)
-            # Prevent infinite loops by tracking visited nodes
+          while current
             break if visited.include?(current.object_id)
 
             visited << current.object_id
 
-            parts.unshift(current.name) if current.name
+            break if Canon::Comparison::NodeInspector.document?(current) ||
+              Canon::Comparison::NodeInspector.document_fragment?(current)
+
+            name = Canon::Comparison::NodeInspector.name(current)
+            break if name.nil?
 
-            # Stop at document or fragment roots
-            break if current.is_a?(Nokogiri::XML::Document) ||
-              current.is_a?(Nokogiri::HTML4::Document) ||
-              current.is_a?(Nokogiri::HTML5::Document) ||
-              current.is_a?(Nokogiri::XML::DocumentFragment) ||
-              current.is_a?(Nokogiri::HTML4::DocumentFragment) ||
-              current.is_a?(Nokogiri::HTML5::DocumentFragment)
+            parts.unshift(name) unless name.empty?
 
-            current = current.parent if current.respond_to?(:parent)
+            current = Canon::Comparison::NodeInspector.parent(current)
           end
 
           parts.empty? ? diff_node.dimension.to_s : parts.join(".")

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

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require_relative "base_formatter"
-
 module Canon
   class DiffFormatter
     module ByObject

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

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require_relative "base_formatter"
-
 module Canon
   class DiffFormatter
     module ByObject

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

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require_relative "json_formatter"
-
 module Canon
   class DiffFormatter
     module ByObject

data/lib/canon/diff_formatter/debug_output.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require_relative "diff_detail_formatter"
-
 module Canon
   class DiffFormatter
     # Verbose diff output helper for CANON_VERBOSE mode