commenter 0.2.2 → 0.3.0

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.2
+  version: 0.3.0
 platform: ruby
 authors:
 - Ribose
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-06-04 00:00:00.000000000 Z
+date: 2026-08-26 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
@@ -127,14 +142,20 @@ files:
 - lib/commenter/filler.rb
 - lib/commenter/github_integration.rb
 - lib/commenter/parser.rb
+- lib/commenter/parser/osd_xlsx_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/github_integration_spec.rb
+- spec/commenter/osd_xlsx_parser_spec.rb
 - spec/commenter_spec.rb
 - spec/spec_helper.rb
+- spec/support/osd_fixtures.rb
+- spec/support/xlsx_builder.rb
 homepage: https://github.com/metanorma/commenter
 licenses:
 - BSD-2-Clause
@@ -160,5 +181,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: []