coradoc 2.0.2 → 2.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b2a167bf671c487f25b37d0df4019bee6bd324636fdf9a681b799764f4a14b2d
-  data.tar.gz: 89345da1186b5ad1fc3ed0315b536a09e445015b1744094fd3ff187382023818
+  metadata.gz: 7f2699bddcd303d5c58e2f3e6a7d4c4341a8f06dc06df44785a97aa48ad7e055
+  data.tar.gz: 8e696cec10f047229d67806d5d93d21bcd02465ef57da243295f13906ccfc91d
 SHA512:
-  metadata.gz: 5a7ead748469ceecbe303de9fd0e65b0081c4765cb8cda371f11cdd76173dcb60372f89ff2c5fc1d30b74fab0bff349fd0149b62ddd65df5b944a350dea14cc9
-  data.tar.gz: 41c5de35c416b7bfcc74ba8043bc26771e4bc8a332f59211de6f0c7f6bae7019fa096458ab705bd7e54b0c248c352df70c04467679f156c315e02c00ef4fdf5d
+  metadata.gz: 8b313846dc7365da7c399117c33119c28009df2009d478345157839c29e9bf30a152fb0000ab56c7202ea7cb784e661ba2a3b61bfa7a40f7616440182994f718
+  data.tar.gz: 42118e1947a3c609110786b7f906412080a79e518b02e101a4cc19d696e790849d93cdc049334829837a44a18308a34a03e3d6caf8fac83628a8119c1d35cfcc

data/.rubocop_todo.yml

@@ -1,6 +1,6 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2026-05-06 19:17:34 UTC using RuboCop version 1.75.8.
+# on 2026-05-07 12:19:03 UTC using RuboCop version 1.75.8.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
@@ -15,6 +15,28 @@ Gemspec/RequiredRubyVersion:
     - 'coradoc-html/coradoc-html.gemspec'
     - 'coradoc.gemspec'
 
+# Offense count: 1
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: EnforcedStyle, IndentationWidth.
+# SupportedStyles: with_first_argument, with_fixed_indentation
+Layout/ArgumentAlignment:
+  Exclude:
+    - 'coradoc-adoc/lib/coradoc/asciidoc/serializer/serializers/document.rb'
+
+# Offense count: 1
+# This cop supports safe autocorrection (--autocorrect).
+Layout/ClosingParenthesisIndentation:
+  Exclude:
+    - 'lib/coradoc/document_manipulator.rb'
+
+# Offense count: 1
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: EnforcedStyle, IndentationWidth.
+# SupportedStyles: consistent, consistent_relative_to_receiver, special_for_inner_method_call, special_for_inner_method_call_in_parentheses
+Layout/FirstArgumentIndentation:
+  Exclude:
+    - 'lib/coradoc/document_manipulator.rb'
+
 # Offense count: 6
 # This cop supports safe autocorrection (--autocorrect).
 # Configuration parameters: AllowHeredoc, AllowURI, URISchemes, IgnoreCopDirectives, AllowedPatterns, SplitStrings.
@@ -22,6 +44,21 @@ Gemspec/RequiredRubyVersion:
 Layout/LineLength:
   Max: 235
 
+# Offense count: 1
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: EnforcedStyle.
+# SupportedStyles: symmetrical, new_line, same_line
+Layout/MultilineMethodCallBraceLayout:
+  Exclude:
+    - 'lib/coradoc/document_manipulator.rb'
+
+# Offense count: 1
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: AllowInHeredoc.
+Layout/TrailingWhitespace:
+  Exclude:
+    - 'coradoc-adoc/lib/coradoc/asciidoc/serializer/serializers/document.rb'
+
 # Offense count: 1
 Lint/BinaryOperatorWithIdenticalOperands:
   Exclude:
@@ -96,7 +133,7 @@ Lint/Void:
   Exclude:
     - 'coradoc-html/lib/coradoc/html.rb'
 
-# Offense count: 237
+# Offense count: 236
 # Configuration parameters: AllowedMethods, AllowedPatterns, CountRepeatedAttributes.
 Metrics/AbcSize:
   Max: 178
@@ -117,12 +154,12 @@ Metrics/BlockNesting:
 Metrics/ClassLength:
   Max: 556
 
-# Offense count: 186
+# Offense count: 181
 # Configuration parameters: AllowedMethods, AllowedPatterns.
 Metrics/CyclomaticComplexity:
   Max: 32
 
-# Offense count: 383
+# Offense count: 384
 # Configuration parameters: CountComments, CountAsOne, AllowedMethods, AllowedPatterns.
 Metrics/MethodLength:
   Max: 246
@@ -138,7 +175,7 @@ Metrics/ParameterLists:
   Max: 7
   MaxOptionalParameters: 4
 
-# Offense count: 125
+# Offense count: 123
 # Configuration parameters: AllowedMethods, AllowedPatterns.
 Metrics/PerceivedComplexity:
   Max: 27
@@ -380,6 +417,12 @@ Style/HashLikeCase:
     - 'coradoc-html/lib/coradoc/html/input/converters/table.rb'
     - 'coradoc-html/lib/coradoc/html/input/converters/td.rb'
 
+# Offense count: 1
+# This cop supports safe autocorrection (--autocorrect).
+Style/MultilineIfModifier:
+  Exclude:
+    - 'coradoc-adoc/lib/coradoc/asciidoc/serializer/serializers/document.rb'
+
 # Offense count: 1
 # This cop supports unsafe autocorrection (--autocorrect-all).
 # Configuration parameters: EnforcedStyle.

data/coradoc-docx/lib/coradoc/docx/transform/context.rb

@@ -62,7 +62,7 @@ module Coradoc
         private
 
         def extract_paragraph_text(paragraph)
-          return '' unless paragraph.respond_to?(:text)
+          return '' unless paragraph.is_a?(Uniword::Wordprocessingml::Paragraph) || paragraph.is_a?(Uniword::Wordprocessingml::Run)
 
           paragraph.text.to_s
         end

data/coradoc-docx/lib/coradoc/docx/transform/from_core_model.rb

@@ -242,11 +242,19 @@ module Coradoc
         end
 
         def add_list_to_builder(list_block, builder)
-          method = list_block.marker_type == 'numbered' ? :numbered_list : :bullet_list
+          list_method = list_block.marker_type == 'numbered' ? :numbered_list : :bullet_list
 
-          builder.send(method) do |list|
-            list_block.items.each do |item|
-              list.item(item.flat_text)
+          if list_method == :numbered_list
+            builder.numbered_list do |list|
+              list_block.items.each do |item|
+                list.item(item.flat_text)
+              end
+            end
+          else
+            builder.bullet_list do |list|
+              list_block.items.each do |item|
+                list.item(item.flat_text)
+              end
             end
           end
         end
@@ -334,8 +342,7 @@ module Coradoc
             va.value = 'superscript'
             props.vertical_align = va
           when 'monospace'
-            # Monospace via font
-            props.font = 'Courier New' if props.respond_to?(:font=)
+            props.fonts = Uniword::Wordprocessingml::RFonts.new(ascii: 'Courier New')
           when 'link'
             # Links need to be at the paragraph level (w:hyperlink)
             # Return the run; caller should wrap in Hyperlink

data/coradoc-docx/lib/coradoc/docx/transform/numbering_resolver.rb

@@ -61,8 +61,8 @@ module Coradoc
         def build_num_map(config)
           return {} unless config
 
-          instances = config.respond_to?(:instances) ? config.instances : []
-          return {} if instances.empty?
+          instances = config.instances
+          return {} if instances.nil? || instances.empty?
 
           map = {}
           instances.each do |inst|
@@ -73,8 +73,8 @@ module Coradoc
             definition = find_definition(config, abstract_num_id)
             next unless definition
 
-            levels = definition.respond_to?(:levels) ? definition.levels : []
-            level = levels.first
+            levels = definition.levels
+            level = levels&.first
             next unless level
 
             fmt = extract_num_fmt(level)
@@ -93,11 +93,13 @@ module Coradoc
           aid = instance.abstract_num_id
           return nil unless aid
 
-          aid.respond_to?(:value) ? aid.value : aid
+          aid.is_a?(Uniword::Wordprocessingml::AbstractNumId) ? aid.val : aid
         end
 
         def find_definition(config, abstract_num_id)
-          defs = config.respond_to?(:definitions) ? config.definitions : []
+          defs = config.definitions
+          return [] unless defs
+
           defs.find { |d| d.abstract_num_id == abstract_num_id }
         end
 
@@ -105,7 +107,7 @@ module Coradoc
           nf = level.numFmt
           return 'bullet' unless nf
 
-          nf.respond_to?(:val) ? nf.val.to_s : nf.to_s
+          nf.is_a?(Uniword::Wordprocessingml::NumFmt) ? nf.val.to_s : nf.to_s
         end
 
         def marker_type_for(fmt)

data/coradoc-docx/lib/coradoc/docx/transform/ordered_content.rb

@@ -18,7 +18,7 @@ module Coradoc
         # @param context [Context] transform context with registry
         # @return [Array] transformed content (Strings, InlineElements, etc.)
         def transform_paragraph_content(paragraph, context)
-          order = paragraph.respond_to?(:element_order) ? paragraph.element_order : nil
+          order = paragraph.is_a?(Uniword::Wordprocessingml::Paragraph) ? paragraph.element_order : nil
 
           if order && !order.empty?
             transform_ordered(paragraph, order, context)
@@ -49,7 +49,7 @@ module Coradoc
 
           result = []
           order.each do |entry|
-            name = entry.respond_to?(:name) ? entry.name : entry.to_s
+            name = entry.is_a?(String) ? entry : entry.name
             idx = counters[name]
             counters[name] = idx + 1
 

data/coradoc-docx/lib/coradoc/docx/transform/rules/image_rule.rb

@@ -109,7 +109,10 @@ module Coradoc
           def extent_to_px(extent, dimension)
             return nil unless extent
 
-            value = extent.respond_to?(dimension) ? extent.send(dimension) : nil
+            value = case dimension
+                    when :cx then extent.cx
+                    when :cy then extent.cy
+                    end
             return nil unless value
 
             px = value.to_i / EMU_PER_PX

data/coradoc-docx/lib/coradoc/docx/transform/rules/math_rule.rb

@@ -66,7 +66,8 @@ module Coradoc
           end
 
           def element_to_xml(element)
-            return '' unless element.respond_to?(:to_xml)
+            return '' unless element.is_a?(Uniword::Wordprocessingml::AlternateContent) ||
+                             element.is_a?(Uniword::Wordprocessingml::OMath)
 
             element.to_xml
           end

data/coradoc-docx/lib/coradoc/docx/transform/rules/run_rule.rb

@@ -45,10 +45,8 @@ module Coradoc
           private
 
           def effective_props(run)
-            if run.respond_to?(:effective_run_properties)
-              ep = run.effective_run_properties
-              return ep if ep
-            end
+            ep = run.effective_run_properties
+            return ep if ep
 
             run.properties
           end
@@ -66,15 +64,12 @@ module Coradoc
               result << context.transform(drawing)
             end
 
-            # Tab character
-            result << "\t" if run.respond_to?(:tab) && run.tab
+            result << "\t" if run.tab
 
-            # Inline math (m:oMath within a run)
-            result << context.transform(run.o_math) if run.respond_to?(:o_math) && run.o_math
+            result << context.transform(run.o_math) if run.class.attributes.key?(:o_math) && run.o_math
 
-            # Deleted text (tracked changes)
-            if run.respond_to?(:del_text) && run.del_text
-              text = run.del_text.respond_to?(:content) ? run.del_text.content.to_s : run.del_text.to_s
+            if run.del_text
+              text = run.del_text.is_a?(Uniword::Wordprocessingml::DeletedText) ? run.del_text.content.to_s : run.del_text.to_s
               unless text.empty?
                 result << CoreModel::InlineElement.new(
                   format_type: 'strikethrough',
@@ -83,24 +78,18 @@ module Coradoc
               end
             end
 
-            # Symbol (w:sym)
-            if run.respond_to?(:sym) && run.sym
-              sym = run.sym
-              char = sym.respond_to?(:char) ? sym.char : nil
+            if run.sym
+              char = run.sym.char
               result << char.to_s if char && !char.empty?
             end
 
-            # No-break hyphen (U+2011)
-            result << "\u2011" if run.respond_to?(:no_break_hyphen) && run.no_break_hyphen
+            result << "\u2011" if run.no_break_hyphen
 
-            # Soft hyphen (U+00AD)
-            result << "\u00AD" if run.respond_to?(:soft_hyphen) && run.soft_hyphen
+            result << "\u00AD" if run.class.attributes.key?(:soft_hyphen) && run.soft_hyphen
 
-            # Carriage return (w:cr) — treat as line break
-            result << CoreModel::InlineElement.new(format_type: 'hard_line_break') if run.respond_to?(:carriage_return) && run.carriage_return
+            result << CoreModel::InlineElement.new(format_type: 'hard_line_break') if run.class.attributes.key?(:carriage_return) && run.carriage_return
 
-            # Alternate content — extract preferred/fallback content
-            if run.respond_to?(:alternate_content) && run.alternate_content
+            if run.alternate_content
               result << extract_alternate_content(run.alternate_content,
                                                   context)
             end
@@ -109,19 +98,17 @@ module Coradoc
           end
 
           def extract_alternate_content(ac, context)
-            # Prefer the fallback content (wc:w:bdoContent or mc:Fallback)
-            content = if ac.respond_to?(:fallback) && ac.fallback
+            content = if ac.fallback
                         ac.fallback
-                      elsif ac.respond_to?(:choice) && ac.choice
+                      elsif ac.choice
                         ac.choice
                       end
 
             return nil unless content
 
-            # If the fallback/choice has paragraphs or runs, transform them
-            if content.respond_to?(:runs)
+            if content.is_a?(Uniword::Wordprocessingml::Run)
               content.runs&.each { |r| context.transform(r) }
-            elsif content.respond_to?(:paragraphs)
+            elsif content.is_a?(Uniword::Wordprocessingml::Paragraph)
               content.paragraphs&.flat_map { |p| context.transform(p) }
             end
           end
@@ -147,7 +134,7 @@ module Coradoc
             return nil unless props
 
             # Check rStyle for semantic role
-            if context.style_resolver.respond_to?(:run_semantic_role)
+            if context.style_resolver.is_a?(Coradoc::Docx::Transform::StyleResolver)
               role = context.style_resolver.run_semantic_role(run)
               case role
               when :monospace then return 'monospace'

data/coradoc-docx/lib/coradoc/docx/transform/rules/simple_field_rule.rb

@@ -62,19 +62,17 @@ module Coradoc
           private
 
           def field_text(field)
-            # SimpleField may have runs with resolved text
-            if field.respond_to?(:runs) && field.runs && !field.runs.empty?
+            if field.runs && !field.runs.empty?
               return field.runs.map do |r|
                 r.text&.content.to_s
               end.join
             end
 
-            # Fall back to text attribute
-            field.respond_to?(:text) ? field.text.to_s : nil
+            nil
           end
 
           def field_instruction(field)
-            instr = field.respond_to?(:instr) ? field.instr : nil
+            instr = field.instr
             instr.to_s
           end
 

data/coradoc-docx/lib/coradoc/docx/transform/rules/table_rule.rb

@@ -32,7 +32,7 @@ module Coradoc
           def transform_row(row, context)
             CoreModel::TableRow.new(
               cells: row.cells.map { |c| transform_cell(c, context) },
-              header: row.respond_to?(:header?) ? row.header? : false
+              header: row.header?
             )
           end
 
@@ -69,15 +69,14 @@ module Coradoc
           end
 
           def cell_paragraphs(cell)
-            cell.respond_to?(:paragraphs) ? (cell.paragraphs || []) : []
+            cell.paragraphs || []
           end
 
           def header_cell?(cell)
             return false unless cell.properties
-            return false unless cell.properties.respond_to?(:v_merge)
 
             vm = cell.properties.v_merge
-            vm.respond_to?(:value) ? vm.value.to_s == 'restart' : false
+            vm&.value.to_s == 'restart'
           end
         end
       end

data/coradoc-docx/lib/coradoc/docx/transform/style_resolver.rb

@@ -51,18 +51,16 @@ module Coradoc
           style_name = resolve_style_name(paragraph)
           return true if style_name && HEADING_PATTERN.match?(style_name)
 
-          # Check outline_level on paragraph properties
           ol = paragraph.properties.outline_level
           if ol
-            ol_level = ol.respond_to?(:value) ? ol.value.to_i : ol.to_i
+            ol_level = ol.is_a?(Uniword::Wordprocessingml::OutlineLevel) ? ol.value.to_i : ol.to_i
             return true if ol_level.positive?
           end
 
-          # Check outline_level from style definition
           style = find_style_for_paragraph(paragraph)
-          if style.respond_to?(:outline_level) && style.outline_level
+          if style&.outline_level
             ol_val = style.outline_level
-            ol_val = ol_val.respond_to?(:value) ? ol_val.value.to_i : ol_val.to_i
+            ol_val = ol_val.is_a?(Uniword::Wordprocessingml::OutlineLevel) ? ol_val.value.to_i : ol_val.to_i
             return true if ol_val.positive?
           end
 
@@ -82,7 +80,7 @@ module Coradoc
           # Check outline_level on paragraph properties
           ol = paragraph.properties&.outline_level
           if ol
-            level = ol.respond_to?(:value) ? ol.value.to_i : ol.to_i
+            level = ol.is_a?(Uniword::Wordprocessingml::OutlineLevel) ? ol.value.to_i : ol.to_i
             return level if level.positive?
           end
 
@@ -138,10 +136,9 @@ module Coradoc
           style_ref = paragraph.properties&.style
           return nil unless style_ref
 
-          value = style_ref.respond_to?(:value) ? style_ref.value : style_ref.to_s
+          value = style_ref.is_a?(Uniword::Wordprocessingml::PStyle) ? style_ref.val : style_ref.to_s
           return nil unless value
 
-          # Check local style map first (for custom style aliases)
           mapped = @style_map[value]
           return mapped if mapped
 
@@ -152,7 +149,7 @@ module Coradoc
           style_ref = run.properties.style
           return nil unless style_ref
 
-          value = style_ref.respond_to?(:value) ? style_ref.value : style_ref.to_s
+          value = style_ref.is_a?(Uniword::Wordprocessingml::PStyle) ? style_ref.val : style_ref.to_s
           return nil unless value
 
           mapped = @style_map[value]
@@ -165,23 +162,21 @@ module Coradoc
           style_id = style_id_from_paragraph(paragraph)
           return nil unless style_id
 
-          if @config.respond_to?(:style_by_id)
-            @config.style_by_id(style_id)
-          elsif @config.respond_to?(:styles)
-            @config.styles.find { |s| s.styleId == style_id }
-          end
+          return unless @config.is_a?(Uniword::Wordprocessingml::StylesConfiguration)
+
+          @config.style_by_id(style_id)
         end
 
         def style_id_from_paragraph(paragraph)
           style_ref = paragraph.properties&.style
           return nil unless style_ref
 
-          style_ref.respond_to?(:value) ? style_ref.value : style_ref.to_s
+          style_ref.is_a?(Uniword::Wordprocessingml::PStyle) ? style_ref.val : style_ref.to_s
         end
 
         def build_style_map(config)
           return {} unless config
-          return {} unless config.respond_to?(:styles)
+          return {} unless config.is_a?(Uniword::Wordprocessingml::StylesConfiguration)
 
           map = {}
           config.styles.each do |style|
@@ -191,7 +186,6 @@ module Coradoc
 
             map[id] = name
 
-            # Detect custom heading styles by basedOn chain
             if heading_by_based_on?(config, style)
               level = heading_level_from_chain(config, style)
               map[id] = "Heading#{level}"
@@ -202,26 +196,27 @@ module Coradoc
         end
 
         def extract_style_name(style)
-          return style.style_name if style.respond_to?(:style_name)
+          sn = style.style_name
+          return sn if sn
 
           name = style.name
           return nil unless name
 
-          name.respond_to?(:val) ? name.val.to_s : name.to_s
+          name.is_a?(Uniword::Wordprocessingml::StyleName) ? name.val.to_s : name.to_s
         end
 
         def heading_by_based_on?(config, style)
-          based_on = style.respond_to?(:based_on) ? style.based_on : nil
+          based_on = style.based_on
           return false unless based_on
 
           visited = Set.new
           current = style
           while current && !visited.include?(current.styleId)
             visited << current.styleId
-            parent_id = current.respond_to?(:based_on) ? current.based_on : nil
+            parent_id = current.based_on
             return true if parent_id && HEADING_PATTERN.match?(parent_id)
 
-            break unless parent_id && config.respond_to?(:styles)
+            break unless parent_id
 
             current = config.styles.find { |s| s.styleId == parent_id }
           end
@@ -240,8 +235,8 @@ module Coradoc
               return match[2].to_i if match
             end
 
-            parent_id = current.respond_to?(:based_on) ? current.based_on : nil
-            break unless parent_id && config.respond_to?(:styles)
+            parent_id = current.based_on
+            break unless parent_id
 
             current = config.styles.find { |s| s.styleId == parent_id }
           end

data/coradoc-docx/lib/coradoc/docx/transform/to_core_model.rb

@@ -129,14 +129,13 @@ module Coradoc
         end
 
         def section_break?(paragraph)
-          return false unless paragraph.respond_to?(:properties)
+          return false unless paragraph.is_a?(Uniword::Wordprocessingml::Paragraph)
           return false unless paragraph.properties
 
           sect_pr = paragraph.properties.section_properties
           return false unless sect_pr
 
-          # A section break exists if sectPr has a type (nextPage, continuous, etc.)
-          sect_pr.respond_to?(:type) && sect_pr.type
+          sect_pr.type ? true : false
         end
 
         def section_break_element(paragraph, context)
@@ -182,12 +181,12 @@ module Coradoc
         end
 
         def body_ordered_elements(body)
-          order = body.respond_to?(:element_order) ? body.element_order : nil
+          order = body.is_a?(Uniword::Wordprocessingml::Body) ? body.element_order : nil
           return body.elements if order.nil? || order.empty?
 
           p_idx = tbl_idx = sdt_idx = 0
           order.filter_map do |entry|
-            name = entry.respond_to?(:name) ? entry.name : entry.to_s
+            name = entry.is_a?(String) ? entry : entry.name
             case name
             when 'p'
               para = body.paragraphs[p_idx]
@@ -216,7 +215,7 @@ module Coradoc
           doc_footnotes = document.footnotes
           if doc_footnotes.is_a?(Hash)
             doc_footnotes.each do |id, fn|
-              paragraphs = fn.respond_to?(:content) ? fn.content : []
+              paragraphs = fn.is_a?(Uniword::Wordprocessingml::Footnote) ? fn.paragraphs : fn[:content]
               footnotes[id.to_s] = Array(paragraphs)
             end
           end
@@ -280,7 +279,10 @@ module Coradoc
         end
 
         def extract_from_parts(document, method, prefix, core_doc)
-          parts = document.respond_to?(method) ? document.send(method) : nil
+          parts = case method
+                  when :headers then document.headers
+                  when :footers then document.footers
+                  end
           return unless parts
 
           Array(parts).each_with_index do |part, idx|
@@ -288,13 +290,18 @@ module Coradoc
             next if text.nil? || text.strip.empty?
             next if layout_only_text?(text)
 
-            core_doc.set_metadata("docx.#{prefix}.#{part.respond_to?(:type) ? part.type : idx}",
-                                  text.strip)
+            part_type = if part.is_a?(Uniword::Wordprocessingml::Header) ||
+                           part.is_a?(Uniword::Wordprocessingml::Footer)
+                          part.type
+                        else
+                          idx
+                        end
+            core_doc.set_metadata("docx.#{prefix}.#{part_type}", text.strip)
           end
         end
 
         def extract_part_text(part)
-          paragraphs = part.respond_to?(:paragraphs) ? part.paragraphs : []
+          paragraphs = part.paragraphs || []
           return nil unless paragraphs
 
           paragraphs.map do |para|
@@ -303,7 +310,7 @@ module Coradoc
         end
 
         def extract_paragraph_text_content(para)
-          runs = para.respond_to?(:runs) ? para.runs : []
+          runs = para.runs || []
           return nil unless runs
 
           runs.map { |r| r.text&.content.to_s }.join

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

@@ -125,14 +125,10 @@ module Coradoc
         def extract_attributes(model)
           attrs = {}
 
-          # Extract ID if available
-          attrs[:id] = model.id if model.respond_to?(:id) && model.id
+          attrs[:id] = model.id if model.id
+          attrs[:title] = model.title if model.title
 
-          # Extract title if available
-          attrs[:title] = model.title if model.respond_to?(:title) && model.title
-
-          # Extract class/role if available
-          if model.respond_to?(:metadata) && model.metadata
+          if model.is_a?(Coradoc::CoreModel::StructuralElement) && model.metadata
             attrs[:class] = model.metadata[:class] || model.metadata[:role]
             attrs.merge!(model.metadata.except(:class, :role))
           end