coradoc 2.0.2 → 2.0.3

data/coradoc-html/lib/coradoc/html/converter_base.rb

@@ -118,25 +118,19 @@ module Coradoc
       # @param config [Hash, Object] Configuration options or object
       # @return [Object] Built configuration object
       def build_config(config)
-        # If config is already a Configuration object, validate and return it
-        if config.respond_to?(:validate!)
-          config.validate! if config.respond_to?(:validate!)
+        if config.public_methods.include?(:validate!)
+          config.validate!
           return config
         end
 
-        # Otherwise, build from hash (subclasses should override this)
         config
       end
 
-      # Extract document title
-      #
-      # @return [String] Document title
       def extract_document_title
-        # Handle CoreModel::StructuralElement (has title directly)
-        if @document.respond_to?(:title) && @document.title
+        if @document.is_a?(Coradoc::CoreModel::StructuralElement) && @document.title
           title = @document.title
           return title if title.is_a?(String)
-          return title.text if title.respond_to?(:text)
+          return title.text if title.is_a?(Coradoc::CoreModel::Base) && title.text
 
           return title.to_s
         end
@@ -144,10 +138,6 @@ module Coradoc
         'Untitled Document'
       end
 
-      # Extract text from content (array of inline elements)
-      #
-      # @param content [Array] Content elements
-      # @return [String] Extracted text
       def extract_text_from_content(content)
         case content
         when Array
@@ -157,7 +147,7 @@ module Coradoc
         when Coradoc::CoreModel::InlineElement
           content.content.to_s
         when Coradoc::CoreModel::Base
-          if content.respond_to?(:content)
+          if content.content
             extract_text_from_content(content.content)
           else
             content.to_s

data/coradoc-html/lib/coradoc/html/converters/base.rb

@@ -434,37 +434,28 @@ module Coradoc
 
           # Extract text from unknown model types as a fallback
           def extract_text_fallback(content)
-            # Try text attribute first
-            if content.respond_to?(:text) && content.text
-              text_val = content.text
-              return text_val if text_val.is_a?(String)
-              return text_val.to_s if text_val.respond_to?(:to_s)
-            end
+            if content.is_a?(Coradoc::CoreModel::Base)
+              if content.class.attributes.key?(:text) && content.text
+                text_val = content.text
+                return text_val if text_val.is_a?(String)
 
-            # Try content attribute
-            if content.respond_to?(:content) && content.content
-              content_val = content.content
-              if content_val.is_a?(String)
-                return content_val
-              elsif content_val.is_a?(Array)
-                return content_val.map { |item| convert_content_to_html(item, {}) }.join
+                return text_val.to_s
               end
-            end
-
-            # Try href attribute (for cross-references)
-            return content.href.to_s if content.respond_to?(:href) && content.href
 
-            # Try term attribute (for term references)
-            return content.term.to_s if content.respond_to?(:term) && content.term
-
-            # Try id attribute (for references with IDs)
-            return content.id.to_s if content.respond_to?(:id) && content.id
-
-            # Try name attribute
-            return content.name.to_s if content.respond_to?(:name) && content.name
+              if content.class.attributes.key?(:content) && content.content
+                content_val = content.content
+                if content_val.is_a?(String)
+                  return content_val
+                elsif content_val.is_a?(Array)
+                  return content_val.map { |item| convert_content_to_html(item, {}) }.join
+                end
+              end
 
-            # Try to_adoc if available (for AsciiDoc models)
-            return content.to_adoc.to_s if content.respond_to?(:to_adoc)
+              return content.href.to_s if content.class.attributes.key?(:href) && content.href
+              return content.term.to_s if content.class.attributes.key?(:term) && content.term
+              return content.id.to_s if content.class.attributes.key?(:id) && content.id
+              return content.name.to_s if content.class.attributes.key?(:name) && content.name
+            end
 
             ''
           end

data/coradoc-html/lib/coradoc/html/converters/comment_line.rb

@@ -12,9 +12,9 @@ module Coradoc
           return '' unless options[:preserve_comments]
 
           # Get comment text - check for content or text attribute
-          text = if comment.respond_to?(:content) && comment.content
+          text = if comment.content
                    comment.content
-                 elsif comment.respond_to?(:text) && comment.text
+                 elsif comment.text
                    comment.text
                  else
                    ''

data/coradoc-html/lib/coradoc/html/converters/link.rb

@@ -11,7 +11,7 @@ module Coradoc
           attrs[:id] = model.id if model.id
 
           # Get title from metadata
-          if model.respond_to?(:metadata) && model.metadata && model.metadata[:title]
+          if model.metadata && model.metadata[:title]
             attrs[:title] =
               model.metadata[:title]
           end

data/coradoc-html/lib/coradoc/html/converters/list_item.rb

@@ -50,18 +50,18 @@ module Coradoc
             parts = []
 
             # Convert main content - check children first (mixed content), then content
-            content_to_render = model.respond_to?(:children) && model.children&.any? ? model.children : model.content
+            content_to_render = model.children&.any? ? model.children : model.content
             parts << convert_content_to_html(content_to_render, state) if content_to_render
 
             # Convert attached content
-            if model.respond_to?(:attached) && model.attached && !model.attached.empty?
+            if model.attached && !model.attached.empty?
               model.attached.each do |attached_item|
                 parts << convert_content_to_html(attached_item, state)
               end
             end
 
             # Convert nested list
-            parts << convert_content_to_html(model.nested, state) if model.respond_to?(:nested) && model.nested
+            parts << convert_content_to_html(model.nested, state) if model.nested
 
             attrs = {}
             attrs[:id] = model.id if model.id

data/coradoc-html/lib/coradoc/html/converters/ordered.rb

@@ -42,7 +42,7 @@ module Coradoc
             attrs[:id] = model.id if model.id
 
             # Add start attribute if not starting from 1
-            attrs[:start] = model.start if model.respond_to?(:start) && model.start && model.start != 1
+            attrs[:start] = model.start if model.start && model.start != 1
 
             build_element('ol', "\n#{items_html}\n", attrs)
           end

data/coradoc-html/lib/coradoc/html/converters/span.rb

@@ -39,7 +39,7 @@ module Coradoc
           # @return [String] HTML string
           def to_html(model, state = {})
             # Prefer children for mixed content, fall back to content
-            content = if model.respond_to?(:children) && model.children&.any?
+            content = if model.children&.any?
                         model.children.map { |c| convert_content_to_html(c, state) }.join
                       else
                         escape_html(model.content || '')
@@ -47,7 +47,7 @@ module Coradoc
 
             # Build attributes from metadata
             attrs = {}
-            if model.respond_to?(:metadata) && model.metadata && model.metadata[:class]
+            if model.metadata && model.metadata[:class]
               attrs[:class] =
                 model.metadata[:class]
             end

data/coradoc-html/lib/coradoc/html/converters/table.rb

@@ -48,16 +48,16 @@ module Coradoc
           attrs = []
 
           # Add ID if present
-          attrs << %( id="#{escape_attribute(table.id)}") if table.respond_to?(:id) && table.id
+          attrs << %( id="#{escape_attribute(table.id)}") if table.id
 
           # CoreModel::Table with frame attribute
-          attrs << %( class="frame-#{escape_attribute(table.frame)}") if table.respond_to?(:frame) && table.frame
+          attrs << %( class="frame-#{escape_attribute(table.frame)}") if table.frame
 
           attrs.join
         end
 
         def self.build_caption(table)
-          return nil unless table.respond_to?(:title) && table.title
+          return nil unless table.title
 
           caption_text = table.title.to_s
           return nil if caption_text.empty?

data/coradoc-html/lib/coradoc/html/converters/table_cell.rb

@@ -13,7 +13,7 @@ module Coradoc
           return '' unless cell
 
           # Check if this is a header cell
-          is_header = cell.respond_to?(:header) && cell.header == true
+          is_header = cell.header == true
           tag = is_header ? 'th' : 'td'
 
           # Build cell attributes
@@ -52,33 +52,19 @@ module Coradoc
         def self.build_attributes(cell)
           attrs = []
 
-          # Add ID if present
-          attrs << %( id="#{escape_attribute(cell.id)}") if cell.respond_to?(:id) && cell.id
+          attrs << %( id="#{escape_attribute(cell.id)}") if cell.id
 
-          # Colspan and rowspan
-          attrs << %( colspan="#{cell.colspan}") if cell.respond_to?(:colspan) && cell.colspan
-          attrs << %( rowspan="#{cell.rowspan}") if cell.respond_to?(:rowspan) && cell.rowspan
+          attrs << %( colspan="#{cell.colspan}") if cell.colspan
+          attrs << %( rowspan="#{cell.rowspan}") if cell.rowspan
 
-          # Build style attribute for alignment, colors, etc.
           style_parts = []
 
-          # Horizontal alignment
-          style_parts << "text-align: #{escape_attribute(cell.alignment)}" if cell.respond_to?(:alignment) && cell.alignment
-
-          # Vertical alignment
-          style_parts << "vertical-align: #{escape_attribute(cell.vertical_alignment)}" if cell.respond_to?(:vertical_alignment) && cell.vertical_alignment
-
-          # Background color
-          style_parts << "background-color: #{escape_attribute(cell.bgcolor)}" if cell.respond_to?(:bgcolor) && cell.bgcolor
-
-          # Text color
-          style_parts << "color: #{escape_attribute(cell.color)}" if cell.respond_to?(:color) && cell.color
-
-          # Width
-          style_parts << "width: #{escape_attribute(cell.width)}" if cell.respond_to?(:width) && cell.width
-
-          # Height
-          style_parts << "height: #{escape_attribute(cell.height)}" if cell.respond_to?(:height) && cell.height
+          style_parts << "text-align: #{escape_attribute(cell.alignment)}" if cell.alignment
+          style_parts << "vertical-align: #{escape_attribute(cell.vertical_alignment)}" if cell.vertical_alignment
+          style_parts << "background-color: #{escape_attribute(cell.bgcolor)}" if cell.bgcolor
+          style_parts << "color: #{escape_attribute(cell.color)}" if cell.color
+          style_parts << "width: #{escape_attribute(cell.width)}" if cell.width
+          style_parts << "height: #{escape_attribute(cell.height)}" if cell.height
 
           # Add style attribute if we have any styles
           attrs << %( style="#{style_parts.join('; ')}") if style_parts.any?
@@ -88,7 +74,7 @@ module Coradoc
 
         # Wrap content with style tags based on cell style
         def self.wrap_with_style(content, cell)
-          return content unless cell.respond_to?(:style) && cell.style
+          return content unless cell.style
 
           case cell.style.to_s.downcase
           when 'strong', 's'
@@ -112,11 +98,11 @@ module Coradoc
           return '' if cell.nil?
 
           # Use renderable_content if available (prefers children over content)
-          content = if cell.respond_to?(:renderable_content)
+          content = if cell.renderable_content
                       cell.renderable_content
-                    elsif cell.respond_to?(:children) && cell.children.any?
+                    elsif cell.children.any?
                       cell.children
-                    elsif cell.respond_to?(:content)
+                    elsif cell.content
                       cell.content
                     else
                       cell

data/coradoc-html/lib/coradoc/html/converters/table_row.rb

@@ -9,7 +9,7 @@ module Coradoc
           return '' unless row
 
           # CoreModel::TableRow uses cells
-          cells = row.respond_to?(:cells) ? row.cells : []
+          cells = row.cells || []
           columns_html = cells.map do |cell|
             TableCell.to_html(cell)
           end.join("\n")
@@ -35,7 +35,7 @@ module Coradoc
           attrs = []
 
           # Add ID if present
-          attrs << %( id="#{escape_attribute(row.id)}") if row.respond_to?(:id) && row.id
+          attrs << %( id="#{escape_attribute(row.id)}") if row.id
 
           attrs.join
         end

data/coradoc-html/lib/coradoc/html/converters/text_element.rb

@@ -23,11 +23,7 @@ module Coradoc
           # @return [String] Plain text (escaped)
           def to_html(model, state = {})
             # Handle both string and model with content attribute
-            content = if model.respond_to?(:content)
-                        model.content
-                      else
-                        model
-                      end
+            content = model.content || model
 
             return '' if content.nil?
 

data/coradoc-html/lib/coradoc/html/input/converters/a.rb

@@ -33,7 +33,7 @@ module Coradoc
               ref_id = href.sub(/^#/, '').gsub(/\s/, '').gsub(/__+/, '_')
               # Convert content to string
               content_str = if content.is_a?(Array)
-                              content.map { |c| c.respond_to?(:content) ? c.content : c.to_s }.join
+                              content.map { |c| c.is_a?(Coradoc::CoreModel::Base) ? c.content : c.to_s }.join
                             else
                               content.to_s
                             end
@@ -52,7 +52,7 @@ module Coradoc
 
             # Convert content to string for the link
             content_str = if content.is_a?(Array)
-                            content.map { |c| c.respond_to?(:content) ? c.content : c.to_s }.join
+                            content.map { |c| c.is_a?(Coradoc::CoreModel::Base) && c.content ? c.content : c.to_s }.join
                           else
                             content.to_s
                           end

data/coradoc-html/lib/coradoc/html/input/converters/dl.rb

@@ -87,7 +87,7 @@ module Coradoc
               when Coradoc::CoreModel::InlineElement
                 item.content.to_s
               when Coradoc::CoreModel::Base
-                if item.respond_to?(:content)
+                if item.content
                   item.content.to_s
                 else
                   ''

data/coradoc-html/lib/coradoc/html/input/converters/figure.rb

@@ -38,9 +38,9 @@ module Coradoc
               when Coradoc::CoreModel::InlineElement
                 item.content.to_s
               when Coradoc::CoreModel::Base
-                if item.respond_to?(:content)
+                if item.content
                   item.content.to_s
-                elsif item.respond_to?(:title)
+                elsif item.title
                   item.title.to_s
                 else
                   ''

data/coradoc-html/lib/coradoc/html/input/converters/markup.rb

@@ -85,11 +85,11 @@ module Coradoc
           def extract_text_from_model(model)
             return '' if model.nil?
 
-            if model.respond_to?(:content) && model.content.is_a?(String)
+            if model.is_a?(Coradoc::CoreModel::Base) && model.content.is_a?(String)
               model.content
-            elsif model.respond_to?(:children) && model.children.is_a?(Array)
+            elsif model.is_a?(Coradoc::CoreModel::StructuralElement) && model.children.is_a?(Array)
               model.children.map { |c| extract_text_from_model(c) }.join
-            elsif model.respond_to?(:title) && model.title.is_a?(String)
+            elsif model.is_a?(Coradoc::CoreModel::Base) && model.title.is_a?(String)
               model.title
             else
               model.to_s

data/coradoc-html/lib/coradoc/html/input/converters/p.rb

@@ -66,7 +66,7 @@ module Coradoc
 
           # Check if InlineElement has meaningful nested content
           def item_has_nested_content?(item)
-            return false unless item.respond_to?(:nested_elements)
+            return false unless item.is_a?(Coradoc::CoreModel::InlineElement)
             return false if item.nested_elements.nil? || item.nested_elements.empty?
 
             true

data/coradoc-html/lib/coradoc/html/input/converters/td.rb

@@ -50,7 +50,7 @@ module Coradoc
               when Coradoc::CoreModel::InlineElement
                 item.content.to_s
               when Coradoc::CoreModel::Base
-                if item.respond_to?(:content)
+                if item.content
                   item.content.to_s
                 else
                   ''

data/coradoc-html/lib/coradoc/html/input/converters.rb

@@ -74,8 +74,7 @@ module Coradoc
         def self.lookup(tag_name)
           ensure_converters_loaded
           converter = @@converters[tag_name.to_sym] || default_converter(tag_name)
-          converter = converter.new if converter.respond_to? :new
-          converter
+          converter.is_a?(Class) ? converter.new : converter
         end
 
         # NOTE: process won't run plugin hooks

data/coradoc-html/lib/coradoc/html/input/html_converter.rb

@@ -41,7 +41,7 @@ module Coradoc
 
             plugin_instances.each do |plugin|
               plugin.html_tree = root
-              if plugin.respond_to?(:preprocess_html_tree)
+              if plugin.public_methods.include?(:preprocess_html_tree)
                 track_time "Preprocessing document with #{plugin.name} plugin" do
                   plugin.preprocess_html_tree
                 end
@@ -61,7 +61,7 @@ module Coradoc
             end
 
             plugin_instances.each do |plugin|
-              next unless plugin.respond_to?(:postprocess_coremodel_tree)
+              next unless plugin.public_methods.include?(:postprocess_coremodel_tree)
 
               plugin.coremodel_tree = coremodel
               track_time "Postprocessing CoreModel tree with #{plugin.name} plugin" do
@@ -131,7 +131,7 @@ module Coradoc
             end
 
             plugin_instances.each do |plugin|
-              next unless plugin.respond_to?(:postprocess_output_string)
+              next unless plugin.public_methods.include?(:postprocess_output_string)
 
               plugin.output_string = result
               track_time "Postprocessing output string with #{plugin.name} plugin" do

data/coradoc-html/lib/coradoc/html/input/plugin.rb

@@ -150,7 +150,7 @@ module Coradoc
         # @deprecated Use postprocess_coremodel_tree instead. Will be removed in v2.0.
         def postprocess_coradoc_tree
           warn '[DEPRECATION] `postprocess_coradoc_tree` is deprecated. Use `postprocess_coremodel_tree` instead.'
-          postprocess_coremodel_tree if respond_to?(:postprocess_coremodel_tree)
+          postprocess_coremodel_tree if public_methods.include?(:postprocess_coremodel_tree)
         end
 
         #### Output string functionalities
@@ -161,7 +161,7 @@ module Coradoc
         # @deprecated Use postprocess_output_string instead. Will be removed in v2.0.
         def postprocess_asciidoc_string
           warn '[DEPRECATION] `postprocess_asciidoc_string` is deprecated. Use `postprocess_output_string` instead.'
-          postprocess_output_string if respond_to?(:postprocess_output_string)
+          postprocess_output_string if public_methods.include?(:postprocess_output_string)
         end
       end
     end

data/coradoc-html/lib/coradoc/html/renderer.rb

@@ -179,22 +179,20 @@ module Coradoc
       # (lutaml-model may not create drops if Liquid was loaded after the model class)
       def ensure_drop_class(element)
         klass = element.class
-        if klass.respond_to?(:register_class_if_liquid_defined) &&
-           klass.respond_to?(:base_drop_class) && !klass.base_drop_class
+        if klass.public_methods.include?(:register_class_if_liquid_defined) &&
+           klass.public_methods.include?(:base_drop_class) && !klass.base_drop_class
           klass.register_class_if_liquid_defined
         end
       end
 
-      # Ensure all CoreModel drop classes are registered
-      # (lutaml-model only creates drops when Liquid is already loaded at class definition time)
       def ensure_core_model_drops
         return unless defined?(Coradoc::CoreModel)
 
         CoreModel.constants(false).each do |const_name|
           klass = CoreModel.const_get(const_name)
           next unless klass.is_a?(Class)
-          next unless klass.respond_to?(:register_class_if_liquid_defined)
-          next if klass.respond_to?(:base_drop_class) && klass.base_drop_class
+          next unless klass.public_methods.include?(:register_class_if_liquid_defined)
+          next if klass.public_methods.include?(:base_drop_class) && klass.base_drop_class
 
           klass.register_class_if_liquid_defined
         rescue StandardError

data/coradoc-html/lib/coradoc/html/theme/base.rb

@@ -158,11 +158,10 @@ module Coradoc
         #
         # @return [String] Document title
         def extract_document_title
-          # Handle CoreModel::StructuralElement (has title directly)
-          if @document.respond_to?(:title) && @document.title
+          if @document.is_a?(Coradoc::CoreModel::StructuralElement) && @document.title
             title = @document.title
             return title if title.is_a?(String)
-            return title.text if title.respond_to?(:text)
+            return title.text if title.is_a?(Coradoc::CoreModel::Base) && title.text
 
             return title.to_s
           end

data/coradoc-html/lib/coradoc/html/theme/classic_renderer.rb

@@ -235,9 +235,9 @@ module Coradoc
         # @return [Array] Array of section objects
         def extract_sections(doc)
           sections = []
-          return sections unless doc.respond_to?(:sections)
+          return sections unless doc.is_a?(Coradoc::CoreModel::StructuralElement)
 
-          collect_sections(doc.sections, sections, 1)
+          collect_sections(doc.children, sections, 1)
           sections
         end
 
@@ -260,10 +260,7 @@ module Coradoc
             }
 
             # Recursively collect subsections
-            if item.respond_to?(:sections) && item.sections
-              collect_sections(item.sections, section_data[:children],
-                               level + 1)
-            end
+            collect_sections(item.children, section_data[:children], level + 1)
 
             sections << section_data
           end
@@ -274,14 +271,14 @@ module Coradoc
         # @param section [Coradoc::CoreModel::StructuralElement] Section to extract title from
         # @return [String] Section title
         def extract_section_title(section)
-          if section.respond_to?(:title)
-            title = section.title
-            if title.respond_to?(:text)
+          title = section.title
+          if title
+            if title.is_a?(Coradoc::CoreModel::Base) && title.text
               title.text
-            elsif title.respond_to?(:content)
+            elsif title.is_a?(Coradoc::CoreModel::Base) && title.content
               Coradoc::Html::Converters::Base.get_text_content(title)
             elsif title.is_a?(Array)
-              title.map { |t| t.respond_to?(:text) ? t.text : t.to_s }.join
+              title.map { |t| t.is_a?(Coradoc::CoreModel::Base) && t.text ? t.text : t.to_s }.join
             else
               title.to_s
             end
@@ -295,7 +292,7 @@ module Coradoc
         # @param section [Coradoc::CoreModel::StructuralElement] Section to extract ID from
         # @return [String] Section ID
         def extract_section_id(section)
-          if section.respond_to?(:id) && section.id
+          if section.id
             section.id
           else
             # Generate ID from title

data/coradoc-html/lib/coradoc/html/theme/modern/serializers/document_serializer.rb

@@ -235,7 +235,7 @@ module Coradoc
               def serialize_table_row(row)
                 {
                   type: 'table_row',
-                  header: row.respond_to?(:header) && row.header,
+                  header: row.is_a?(Coradoc::CoreModel::TableRow) && row.header,
                   cells: (row.cells || []).map do |cell|
                     serialize_table_cell(cell)
                   end
@@ -318,10 +318,10 @@ module Coradoc
                 }
 
                 # Add link-specific attributes
-                result[:href] = element.target if element.inline_type == 'link' && element.respond_to?(:target)
+                result[:href] = element.target if element.inline_type == 'link' && element.target
 
                 # Add cross-ref specific attributes
-                result[:target] = element.target if element.inline_type == 'xref' && element.respond_to?(:target)
+                result[:target] = element.target if element.inline_type == 'xref' && element.target
 
                 result
               end

data/coradoc-markdown/lib/coradoc/markdown/model/base.rb

@@ -26,11 +26,12 @@ module Coradoc
         return element if element.nil?
 
         element = yield element, :pre
-        element = if element.respond_to?(:visit)
+        element = case element
+                  when self
                     element.visit(&block)
-                  elsif element.is_a?(Array)
+                  when Array
                     element.map { |child| visit(child, &block) }.flatten.compact
-                  elsif element.is_a?(Hash)
+                  when Hash
                     result = {}
                     element.each { |k, v| result[k] = visit(v, &block) }
                     result
@@ -59,12 +60,12 @@ module Coradoc
         when nil
           ''
         else
-          if content.respond_to?(:to_md)
+          if content.is_a?(Base)
             content.to_md
           else
             raise ArgumentError,
                   "Cannot serialize #{content.class.name} to Markdown. " \
-                  'Expected String or object responding to #to_md.'
+                  'Expected String or Base subclass.'
           end
         end
       end

data/coradoc-markdown/lib/coradoc/markdown/serializer.rb

@@ -102,12 +102,12 @@ module Coradoc
         when Emphasis, Strong, Code, Link, Image, FootnoteReference, Math, Extension, Strikethrough, Highlight
           serialize(element)
         else
-          if element.respond_to?(:to_md)
+          if element.is_a?(Base)
             element.to_md
           else
             raise ArgumentError,
                   "Cannot serialize inline content of type #{element.class}. " \
-                  'Expected String, known inline model, or object responding to #to_md.'
+                  'Expected String, known inline model, or Base subclass.'
           end
         end
       end