markdown-merge 1.0.3 → 7.1.1

data/lib/markdown/merge/smart_merger.rb

@@ -59,6 +59,38 @@ module Markdown
     # @see FileAnalysis
     # @see SmartMergerBase
     class SmartMerger < SmartMergerBase
+      VALID_BACKENDS = %i[auto commonmarker markly kramdown].freeze
+
+      class << self
+        def default_backend
+          :auto
+        end
+
+        def default_freeze_token
+          FileAnalysis::DEFAULT_FREEZE_TOKEN
+        end
+
+        def default_inner_merge_code_blocks
+          false
+        end
+
+        def default_parser_options
+          {}
+        end
+
+        def file_analysis_class
+          FileAnalysis
+        end
+
+        def template_parse_error_class
+          TemplateParseError
+        end
+
+        def destination_parse_error_class
+          DestinationParseError
+        end
+      end
+
       # @return [Symbol] The backend being used (:commonmarker, :markly)
       attr_reader :backend
 
@@ -94,6 +126,11 @@ module Markdown
       #   - `false` (default) - Disable inner-merge (use standard conflict resolution)
       #   - `CodeBlockMerger` instance - Use custom CodeBlockMerger
       #
+      # @param remove_template_missing_nodes [Boolean] Controls whether destination-only
+      #   structural nodes should be removed instead of preserved. Standalone HTML
+      #   comment-only fragments, freeze blocks, and link reference definitions remain
+      #   preserved when enabled.
+      #
       # @param freeze_token [String] Token to use for freeze block markers.
       #   Default: "markdown-merge"
       #   Looks for: <!-- markdown-merge:freeze --> / <!-- markdown-merge:unfreeze -->
@@ -114,18 +151,22 @@ module Markdown
       def initialize(
         template_content,
         dest_content,
-        backend: :auto,
+        backend: self.class.default_backend,
         signature_generator: nil,
         preference: :destination,
         add_template_only_nodes: false,
-        inner_merge_code_blocks: false,
-        freeze_token: FileAnalysis::DEFAULT_FREEZE_TOKEN,
+        inner_merge_code_blocks: self.class.default_inner_merge_code_blocks,
+        inner_merge_lists: false,
+        remove_template_missing_nodes: false,
+        freeze_token: self.class.default_freeze_token,
         match_refiner: nil,
         node_typing: nil,
         **parser_options
       )
+        validate_backend!(backend)
+
         @requested_backend = backend
-        @parser_options = parser_options
+        @parser_options = self.class.default_parser_options.merge(parser_options)
 
         super(
           template_content,
@@ -134,6 +175,8 @@ module Markdown
           preference: preference,
           add_template_only_nodes: add_template_only_nodes,
           inner_merge_code_blocks: inner_merge_code_blocks,
+          inner_merge_lists: inner_merge_lists,
+          remove_template_missing_nodes: remove_template_missing_nodes,
           freeze_token: freeze_token,
           match_refiner: match_refiner,
           node_typing: node_typing,
@@ -152,12 +195,12 @@ module Markdown
       # @param options [Hash] Analysis options
       # @return [FileAnalysis] File analysis instance
       def create_file_analysis(content, **opts)
-        FileAnalysis.new(
+        self.class.file_analysis_class.new(
           content,
           backend: opts[:backend] || @requested_backend,
           freeze_token: opts[:freeze_token],
           signature_generator: opts[:signature_generator],
-          **@parser_options,
+          **@parser_options
         )
       end
 
@@ -165,14 +208,14 @@ module Markdown
       #
       # @return [Class] Markdown::Merge::TemplateParseError
       def template_parse_error_class
-        TemplateParseError
+        self.class.template_parse_error_class
       end
 
       # Returns the DestinationParseError class to use.
       #
       # @return [Class] Markdown::Merge::DestinationParseError
       def destination_parse_error_class
-        DestinationParseError
+        self.class.destination_parse_error_class
       end
 
       # Convert a node to its source text.
@@ -185,14 +228,10 @@ module Markdown
       # @return [String] Source text
       def node_to_source(node, analysis)
         # Check for any FreezeNode type (base class or subclass)
-        if node.is_a?(Ast::Merge::FreezeNodeBase)
-          return node.full_text
-        end
+        return node.full_text if node.is_a?(Ast::Merge::FreezeNodeBase)
 
         # Handle gap line nodes (created for blank lines and link definitions)
-        if node.is_a?(LinkDefinitionNode) || node.is_a?(GapLineNode)
-          return node.content
-        end
+        return node.content if node.is_a?(LinkDefinitionNode) || node.is_a?(GapLineNode)
 
         # Unwrap if needed to access source_position
         raw_node = Ast::Merge::NodeTyping.unwrap(node)
@@ -216,6 +255,15 @@ module Markdown
           source
         end
       end
+
+      private
+
+      def validate_backend!(backend)
+        normalized_backend = backend.respond_to?(:to_sym) ? backend.to_sym : backend
+        return if VALID_BACKENDS.include?(normalized_backend)
+
+        raise ArgumentError, "Unknown backend: #{backend}"
+      end
     end
   end
 end