commenter 0.2.3 → 0.3.1

data/spec/support/osd_fixtures.rb

@@ -0,0 +1,94 @@
+# frozen_string_literal: true
+
+require "tmpdir"
+require_relative "xlsx_builder"
+
+# Fixture data and helpers mirroring ISO OSD XLSX exports for specs.
+module OsdFixtures
+  RESOLVED_HEADERS = [
+    "Comment ID", "User name", "Clause nb", "Clause Title", "Type", "Subtype",
+    "Comment", "Proposal on Text", "Proposed change", "Feedbacks", "Topic",
+    "Tags", "Created Date", "Resolution status", "Motivation", "Resolution Date", "Stage code"
+  ].freeze
+
+  UNRESOLVED_HEADERS = [
+    "User name", "Clause nb", "Clause Title", "Type", "Comment type",
+    "Comment/Motivation", "Comment on text", "Proposal on Text", "Proposed change",
+    "Replies", "Resolution status", "Justification", "Resolution Date", "Date", "Comment number"
+  ].freeze
+
+  METADATA_ROWS = [
+    ["Date", "Reference", nil, nil, "Title EN", nil, nil, "Title FR"],
+    ["2026-04-21", "ISO/DIS 5843-6(en)", nil, nil, "Aerodromes and heliports",
+     nil, nil, "Aerodromes and heliports (FR)"]
+  ].freeze
+
+  RESOLVED_DATA_ROWS = [
+    [1, "John Doe", "5.2.1", "Requirements", "Comment", "Editorial",
+     "The values in Table 3 are inconsistent.", "Proposal on the text",
+     "Correct the values in column 2.", "Feedback from the working group",
+     "Topic A", "tag1", "2026-04-01", "Accepted",
+     "Reviewed and approved by the working group.", "2026-04-10", "40.20"],
+    [],
+    [2, "Jane Roe", nil, nil, "Comment", "Technical", "General remark on the introduction.",
+     nil, nil, nil, nil, nil, "2026-04-02", "Rejected", nil, nil, "40.20"]
+  ].freeze
+
+  def resolved_rows(gap: false)
+    headers = RESOLVED_HEADERS.dup
+    data_rows = RESOLVED_DATA_ROWS.map(&:dup)
+    if gap
+      headers.insert(5, nil)
+      data_rows = data_rows.map { |row| row.empty? ? row : row.insert(5, nil) }
+    end
+
+    METADATA_ROWS + [[]] + [headers] + data_rows
+  end
+
+  def unresolved_workbook_sheets
+    [
+      ["Comments (3)",
+       header_block(UNRESOLVED_HEADERS) + [
+         ["John Doe", "5.2.1", "Requirements", "Comment", "Editorial",
+          "The scope is too narrow.", "On text remark", "Proposal on the text",
+          "Widen the scope to include heliports.", "Reply from secretariat",
+          nil, "Justification text", nil, "2026-04-05", 7],
+         ["Jane Roe", "3.1", "Terms", "Comment", "Technical", "Define the term consistently.",
+          nil, nil, "Add a definition.", nil, nil, nil, nil, "2026-04-06", 8]
+       ]],
+      ["Unresolved comments (1)",
+       header_block(UNRESOLVED_HEADERS) + [
+         ["Jack Black", "4.2", "Design", "Comment", "General", "Improve the design section.",
+          nil, nil, nil, nil, nil, nil, nil, "2026-04-07", 9]
+       ]],
+      ["Resolved comments (1)",
+       header_block(RESOLVED_HEADERS) + [
+         [10, "Jill Hill", "6.1", "Testing", "Comment", "Technical", "Add test requirements.",
+          nil, nil, nil, nil, nil, "2026-04-08", "Partially accepted",
+          "Compromise reached.", "2026-04-12", "40.92"]
+       ]]
+    ]
+  end
+
+  def with_resolved_xlsx(rows = nil)
+    Dir.mktmpdir do |dir|
+      path = File.join(dir, "comments.xlsx")
+      XlsxBuilder.write(path, [["Comments (2)", rows || resolved_rows]])
+      yield path
+    end
+  end
+
+  def with_unresolved_workbook
+    Dir.mktmpdir do |dir|
+      path = File.join(dir, "workbook.xlsx")
+      XlsxBuilder.write(path, unresolved_workbook_sheets)
+      yield path
+    end
+  end
+
+  private
+
+  def header_block(headers)
+    METADATA_ROWS + [[]] + [headers]
+  end
+end

data/spec/support/redline_docx_builder.rb

@@ -0,0 +1,88 @@
+# frozen_string_literal: true
+
+require "zip"
+
+# Builds minimal redline DOCX files for specs, without binary fixtures. The
+# document body is assembled from a paragraph DSL:
+#
+#   { heading: "1", level: 1, text: "Scope" }
+#   { text: "Body text " }
+#   { change: :ins, id: "1", author: "CHEN Yvonne", date: "2025-12-05T20:10:00Z",
+#     text: "inserted" }
+#   { marker: true }  # self-closing paragraph-mark insertion inside rPr
+#   { anchor: "74", text: "Anchored text" }  # w:commentRangeStart/End around text
+#
+# Multiple entries in one paragraph can be given via :parts.
+#
+# comments: [{ id: "74", author: "CHEN Yvonne", date: "...", text: "remark" }]
+# are written to word/comments.xml.
+module RedlineDocxBuilder
+  W_NS = "http://schemas.openxmlformats.org/wordprocessingml/2006/main"
+  private_constant :W_NS
+
+  HEADING_LEVEL_STYLE = { 1 => "Heading1", 2 => "Heading2", 3 => "Heading3" }.freeze
+  CHANGE_TAG = { ins: "w:ins", del: "w:del", moveFrom: "w:moveFrom", moveTo: "w:moveTo" }.freeze
+  private_constant :HEADING_LEVEL_STYLE, :CHANGE_TAG
+
+  def self.write(path, paragraphs, comments: [])
+    zip = Zip::File.new(path, create: true)
+    zip.get_output_stream("word/document.xml") { |stream| stream.write(document_xml(paragraphs)) }
+    zip.get_output_stream("word/comments.xml") { |stream| stream.write(comments_xml(comments)) }
+    zip.close
+    path
+  end
+
+  def self.document_xml(paragraphs)
+    <<~XML
+      <?xml version="1.0" encoding="UTF-8"?>
+      <w:document xmlns:w="#{W_NS}"><w:body>#{paragraphs.map { |p| paragraph_xml(p) }.join}</w:body></w:document>
+    XML
+  end
+
+  def self.comments_xml(comments)
+    entries = comments.map do |c|
+      attrs = %(w:id="#{c[:id]}" w:author="#{c[:author]}" w:date="#{c[:date]}" w:initials="X")
+      %(<w:comment #{attrs}><w:p><w:r><w:t>#{c[:text]}</w:t></w:r></w:p></w:comment>)
+    end
+
+    <<~XML
+      <?xml version="1.0" encoding="UTF-8"?>
+      <w:comments xmlns:w="#{W_NS}">#{entries.join}</w:comments>
+    XML
+  end
+
+  def self.paragraph_xml(paragraph)
+    if paragraph[:marker]
+      return %(<w:p><w:pPr><w:rPr><w:ins w:id="#{paragraph[:id]}" w:author="#{paragraph[:author]}"/></w:rPr></w:pPr></w:p>)
+    end
+
+    inner = if paragraph[:parts]
+              paragraph[:parts].map { |part| part_xml(part) }.join
+            elsif paragraph[:heading]
+              %(<w:r><w:t>#{paragraph[:text]}</w:t></w:r>)
+            elsif paragraph[:change]
+              change_xml(paragraph)
+            else
+              %(<w:r><w:t>#{paragraph[:text]}</w:t></w:r>)
+            end
+
+    ppr = paragraph[:heading] ? %(<w:pPr><w:pStyle w:val="#{HEADING_LEVEL_STYLE.fetch(paragraph[:level], 1)}"/></w:pPr>) : ""
+    anchor = paragraph[:anchor] ? %(<w:commentRangeStart w:id="#{paragraph[:anchor]}"/>) : ""
+    anchor_end = paragraph[:anchor] ? %(<w:commentRangeEnd w:id="#{paragraph[:anchor]}"/>) : ""
+    "<w:p>#{ppr}#{anchor}#{inner}#{anchor_end}</w:p>"
+  end
+
+  def self.part_xml(part)
+    part[:change] ? change_xml(part) : %(<w:r><w:t>#{part[:text]}</w:t></w:r>)
+  end
+
+  def self.change_xml(change)
+    tag = CHANGE_TAG.fetch(change[:change])
+    attrs = %(w:id="#{change[:id]}" w:author="#{change[:author]}" w:date="#{change[:date]}")
+
+    return %(<#{tag} #{attrs}>#{change[:nested].map { |child| change_xml(child) }.join}</#{tag}>) if change[:nested]
+
+    text_tag = change[:change] == :del ? "w:delText" : "w:t"
+    %(<#{tag} #{attrs}><w:r><#{text_tag}>#{change[:text]}</#{text_tag}></w:r></#{tag}>)
+  end
+end

data/spec/support/xlsx_builder.rb

@@ -0,0 +1,147 @@
+# frozen_string_literal: true
+
+require "zip"
+
+# Builds minimal single-workbook XLSX files for specs, without needing a
+# spreadsheet-writing dependency. Strings are stored via sharedStrings; numbers
+# are written as plain numeric cells. Empty/nil cells are omitted (sparse
+# rows), matching how real exports represent empty cells.
+module XlsxBuilder
+  SHEET_NS = "http://schemas.openxmlformats.org/spreadsheetml/2006/main"
+  RELS_NS = "http://schemas.openxmlformats.org/package/2006/relationships"
+  OFFICE_REL_NS = "http://schemas.openxmlformats.org/officeDocument/2006/relationships"
+  private_constant :SHEET_NS, :RELS_NS, :OFFICE_REL_NS
+
+  # sheets: array of [name, rows] pairs, where rows is an array of arrays.
+  # Cell values: String or Numeric. nil and "" cells are skipped.
+  def self.write(path, sheets)
+    shared = []
+    worksheets = sheets.map { |name, rows| [name, worksheet_xml(rows, shared)] }
+
+    zip = Zip::File.new(path, create: true)
+    write_entry(zip, "[Content_Types].xml", content_types_xml(worksheets.size))
+    write_entry(zip, "_rels/.rels", root_rels_xml)
+    write_entry(zip, "xl/workbook.xml", workbook_xml(worksheets.map(&:first)))
+    write_entry(zip, "xl/_rels/workbook.xml.rels", workbook_rels_xml(worksheets.size))
+    write_entry(zip, "xl/sharedStrings.xml", shared_strings_xml(shared))
+    write_entry(zip, "xl/styles.xml", styles_xml)
+    worksheets.each_with_index do |(_name, xml), index|
+      write_entry(zip, "xl/worksheets/sheet#{index + 1}.xml", xml)
+    end
+    zip.close
+    path
+  end
+
+  def self.write_entry(zip, name, content)
+    zip.get_output_stream(name) { |stream| stream.write(content) }
+  end
+
+  def self.worksheet_xml(rows, shared)
+    row_elements = rows.each_with_index.map do |values, row_number|
+      cells = values.each_with_index.map do |value, column_index|
+        cell_xml(value, row_number + 1, column_index, shared)
+      end
+      "<row r=\"#{row_number + 1}\">#{cells.compact.join}</row>"
+    end
+
+    <<~XML
+      <?xml version="1.0" encoding="UTF-8" standalone="yes"?>
+      <worksheet xmlns="#{SHEET_NS}"><sheetData>#{row_elements.join}</sheetData></worksheet>
+    XML
+  end
+
+  def self.cell_xml(value, row_number, column_index, shared)
+    return if value.nil? || (value.is_a?(String) && value.empty?)
+
+    ref = "#{column_name(column_index)}#{row_number}"
+    return "<c r=\"#{ref}\"><v>#{value}</v></c>" if value.is_a?(Numeric)
+
+    text = value.to_s
+    index = shared.index(text)
+    if index.nil?
+      shared << text
+      index = shared.length - 1
+    end
+    "<c r=\"#{ref}\" t=\"s\"><v>#{index}</v></c>"
+  end
+
+  def self.column_name(index)
+    name = +""
+    remainder = index
+    loop do
+      name.prepend((remainder % 26 + 65).chr)
+      remainder = remainder / 26 - 1
+      break if remainder.negative?
+    end
+    name
+  end
+
+  def self.shared_strings_xml(shared)
+    entries = shared.map { |text| "<si><t>#{escape(text)}</t></si>" }
+    <<~XML
+      <?xml version="1.0" encoding="UTF-8" standalone="yes"?>
+      <sst xmlns="#{SHEET_NS}" count="#{shared.length}" uniqueCount="#{shared.length}">#{entries.join}</sst>
+    XML
+  end
+
+  def self.content_types_xml(sheet_count)
+    overrides = [
+      "<Override PartName=\"/xl/workbook.xml\" " \
+        "ContentType=\"application/vnd.openxmlformats-officedocument.spreadsheetml.sheet.main+xml\"/>",
+      "<Override PartName=\"/xl/sharedStrings.xml\" " \
+        "ContentType=\"application/vnd.openxmlformats-officedocument.spreadsheetml.sharedStrings+xml\"/>",
+      "<Override PartName=\"/xl/styles.xml\" " \
+        "ContentType=\"application/vnd.openxmlformats-officedocument.spreadsheetml.styles+xml\"/>"
+    ]
+    (1..sheet_count).each do |index|
+      overrides << "<Override PartName=\"/xl/worksheets/sheet#{index}.xml\" " \
+        "ContentType=\"application/vnd.openxmlformats-officedocument.spreadsheetml.worksheet+xml\"/>"
+    end
+    <<~XML
+      <?xml version="1.0" encoding="UTF-8" standalone="yes"?>
+      <Types xmlns="http://schemas.openxmlformats.org/package/2006/content-types"><Default Extension="rels" ContentType="application/vnd.openxmlformats-package.relationships+xml"/><Default Extension="xml" ContentType="application/xml"/>#{overrides.join}</Types>
+    XML
+  end
+
+  def self.root_rels_xml
+    <<~XML
+      <?xml version="1.0" encoding="UTF-8" standalone="yes"?>
+      <Relationships xmlns="#{RELS_NS}"><Relationship Id="rId1" Type="#{OFFICE_REL_NS}/officeDocument" Target="xl/workbook.xml"/></Relationships>
+    XML
+  end
+
+  def self.workbook_xml(sheet_names)
+    sheets = sheet_names.each_with_index.map do |name, index|
+      "<sheet name=\"#{escape(name)}\" sheetId=\"#{index + 1}\" r:id=\"rId#{index + 1}\"/>"
+    end
+    <<~XML
+      <?xml version="1.0" encoding="UTF-8" standalone="yes"?>
+      <workbook xmlns="#{SHEET_NS}" xmlns:r="#{OFFICE_REL_NS}"><sheets>#{sheets.join}</sheets></workbook>
+    XML
+  end
+
+  def self.workbook_rels_xml(sheet_count)
+    relationships = (1..sheet_count).map do |index|
+      "<Relationship Id=\"rId#{index}\" Type=\"#{OFFICE_REL_NS}/worksheet\" Target=\"worksheets/sheet#{index}.xml\"/>"
+    end
+    relationships << "<Relationship Id=\"rId#{sheet_count + 1}\" Type=\"#{OFFICE_REL_NS}/sharedStrings\" " \
+      "Target=\"sharedStrings.xml\"/>"
+    relationships << "<Relationship Id=\"rId#{sheet_count + 2}\" Type=\"#{OFFICE_REL_NS}/styles\" " \
+      "Target=\"styles.xml\"/>"
+    <<~XML
+      <?xml version="1.0" encoding="UTF-8" standalone="yes"?>
+      <Relationships xmlns="#{RELS_NS}">#{relationships.join}</Relationships>
+    XML
+  end
+
+  def self.styles_xml
+    <<~XML
+      <?xml version="1.0" encoding="UTF-8" standalone="yes"?>
+      <styleSheet xmlns="#{SHEET_NS}"><fonts count="1"><font><sz val="11"/><name val="Calibri"/></font></fonts><fills count="1"><fill><patternFill patternType="none"/></fill></fills><borders count="1"><border/></borders><cellStyleXfs count="1"><xf numFmtId="0" fontId="0" fillId="0" borderId="0"/></cellStyleXfs><cellXfs count="1"><xf numFmtId="0" fontId="0" fillId="0" borderId="0" xfId="0"/></cellXfs></styleSheet>
+    XML
+  end
+
+  def self.escape(text)
+    text.gsub("&", "&amp;").gsub("<", "&lt;").gsub(">", "&gt;").gsub('"', "&quot;")
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: commenter
 version: !ruby/object:Gem::Version
-  version: 0.2.3
+  version: 0.3.1
 platform: ruby
 authors:
 - Ribose
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-03-16 00:00:00.000000000 Z
+date: 2026-08-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: base64
@@ -80,6 +80,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '6.0'
+- !ruby/object:Gem::Dependency
+  name: roo
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '2.10'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '2.10'
 - !ruby/object:Gem::Dependency
   name: thor
   requirement: !ruby/object:Gem::Requirement
@@ -94,8 +108,8 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.0'
-description: Convert between ISO comment sheet DOCX and structured YAML with schema
-  validation.
+description: Convert between ISO comment sheets (DOCX and XLSX) and structured YAML
+  with schema validation.
 email:
 - open.source@ribose.com
 executables:
@@ -109,6 +123,7 @@ files:
 - ".rspec"
 - ".rubocop.yml"
 - ".rubocop_todo.yml"
+- CLAUDE.md
 - CODE_OF_CONDUCT.md
 - Gemfile
 - README.adoc
@@ -124,17 +139,29 @@ files:
 - lib/commenter/cli.rb
 - lib/commenter/comment.rb
 - lib/commenter/comment_sheet.rb
+- lib/commenter/comment_type.rb
 - lib/commenter/filler.rb
 - lib/commenter/github_integration.rb
+- lib/commenter/github_session.rb
 - lib/commenter/parser.rb
+- lib/commenter/parser/osd_xlsx_parser.rb
+- lib/commenter/parser/track_change_docx_parser.rb
 - lib/commenter/version.rb
 - schema/iso_comment_2012-03.yaml
+- schema/iso_comment_osd.yaml
 - sig/commenter.rbs
+- spec/commenter/cli_spec.rb
 - spec/commenter/comment_sheet_spec.rb
 - spec/commenter/comment_spec.rb
+- spec/commenter/comment_type_spec.rb
 - spec/commenter/github_integration_spec.rb
+- spec/commenter/osd_xlsx_parser_spec.rb
+- spec/commenter/track_change_docx_parser_spec.rb
 - spec/commenter_spec.rb
 - spec/spec_helper.rb
+- spec/support/osd_fixtures.rb
+- spec/support/redline_docx_builder.rb
+- spec/support/xlsx_builder.rb
 homepage: https://github.com/metanorma/commenter
 licenses:
 - BSD-2-Clause
@@ -160,5 +187,5 @@ requirements: []
 rubygems_version: 3.5.22
 signing_key:
 specification_version: 4
-summary: Library to work with ISO comment sheets in DOCX format.
+summary: Library to work with ISO comment sheets in DOCX and XLSX formats.
 test_files: []