kramdown-ansi 0.7.0 → 0.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6d35fcfbc735d86a3f58f59c1eaded0ebdc4f706db3cedb0c807aa6f9f420fe8
-  data.tar.gz: cd32746d6d592ee48e59bda8211adf670bdef612a5c65da97e114234ef8f0e83
+  metadata.gz: bfce0110800a14de98f3322fa646e4432af3a51fcda9c278d5c631d322c2384d
+  data.tar.gz: fdad876925cd1ecab465d491b8cca2024dbc40b2e8d103794303b83e316aff3f
 SHA512:
-  metadata.gz: d7607e68a92ac9d968febb1179c60a291550306b56ada967fd29f2c9f1dd413f6d18981895b358b5a0ac34bdaae0291d1ed7e452a76dbe52329582fc11b11dc5
-  data.tar.gz: 07b078aad8c0e3d0b9cf1b509b9f21dc4a2cbdbf3101f4997a079349fb7272848fe58080a18ab4e79c4b13b07061cbe61dfb43c511b8630eb73c077a6ecb866b
+  metadata.gz: 624287e2ab60beda1ce2ad7ce0ae3e68feeb8ad430b46828c7af3ee4833dd7d92c8a726919c4ebf6289a35fe79600d9756983e735403ef2269fa1ed3fd00a4a2
+  data.tar.gz: 795ffcabcd29c3910d1da3fca407ab2c09d85a416f25b0122353f0176ab59dcb424e869d9b4fac149cde62a03c1792f3b3134d24fe85b0b2c507112e8a181d41

data/CHANGES.md

@@ -1,5 +1,34 @@
 # Changes
 
+## 2026-10-29 v0.8.0
+
+### Fixes
+
+- Fixed table width overflow by restructuring column width calculation from
+  per-row to per-table in `convert_table`, ensuring `Terminal::Table` is not
+  forced wider than the terminal by a single row's content distribution.
+- Fixed a bug in `Kramdown::ANSI::Width.wrap` where a newline was incorrectly
+  prepended to the first token when it exceeded the target length, eliminating
+  phantom blank lines in narrow table cells.
+- Subtracted table chrome (border, padding, and separator overhead) from the
+  column width budget in `convert_table` to prevent tables from exceeding
+  width.
+
+### Improvements
+
+- Simplified `convert_tr` to read pre-computed column widths and wrap each cell
+  to its fixed budget.
+
+### Build & CI
+
+- Updated the minimum version of the `gem_hadar` development dependency to
+  **2.21.0**.
+- Bumped `rubygems_version` to **4.0.23**.
+
+### Testing
+
+- Regenerated `spec/assets/tables.ansi` golden file.
+
 ## 2026-10-22 v0.7.0
 
 ### Fixes

data/kramdown-ansi.gemspec

@@ -1,9 +1,9 @@
 # -*- encoding: utf-8 -*-
-# stub: kramdown-ansi 0.7.0 ruby lib
+# stub: kramdown-ansi 0.8.0 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "kramdown-ansi".freeze
-  s.version = "0.7.0".freeze
+  s.version = "0.8.0".freeze
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0".freeze) if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib".freeze]
@@ -18,13 +18,13 @@ Gem::Specification.new do |s|
   s.licenses = ["MIT".freeze]
   s.rdoc_options = ["--title".freeze, "Kramdown-ansi - Output markdown in the terminal with ANSI escape sequences".freeze, "--main".freeze, "README.md".freeze]
   s.required_ruby_version = Gem::Requirement.new(">= 3.1".freeze)
-  s.rubygems_version = "4.0.22".freeze
+  s.rubygems_version = "4.0.23".freeze
   s.summary = "Output markdown in the terminal with ANSI escape sequences".freeze
   s.test_files = ["spec/kramdown/ansi/pager_spec.rb".freeze, "spec/kramdown/ansi/styles_spec.rb".freeze, "spec/kramdown/ansi/width_spec.rb".freeze, "spec/kramdown/ansi_golden_spec.rb".freeze, "spec/kramdown/ansi_spec.rb".freeze, "spec/kramdown/ansi_table_spec.rb".freeze, "spec/kramdown/render_spec.rb".freeze, "spec/spec_helper.rb".freeze]
 
   s.specification_version = 4
 
-  s.add_development_dependency(%q<gem_hadar>.freeze, [">= 2.20.1".freeze])
+  s.add_development_dependency(%q<gem_hadar>.freeze, [">= 2.21.0".freeze])
   s.add_development_dependency(%q<all_images>.freeze, ["~> 0.4".freeze])
   s.add_development_dependency(%q<rspec>.freeze, ["~> 3.2".freeze])
   s.add_development_dependency(%q<debug>.freeze, [">= 0".freeze])

data/lib/kramdown/ansi/width.rb

@@ -78,7 +78,9 @@ module Kramdown::ANSI::Width
       if $~[:nws]
         if length_without_ansi + token_length > length
           length_without_ansi = token_length
-          if result[-1] == ' '
+          if result.empty?
+            result = token_value
+          elsif result[-1] == ' '
             result[-1, 1] = "\n#{token_value}"
           else
             result << "\n#{token_value}"

data/lib/kramdown/ansi.rb

@@ -472,6 +472,29 @@ class Kramdown::ANSI < Kramdown::Converter::Base
       end
       return newline rows.join("\n")
     end
+    # Compute per-column budgets from the widest cell in each column
+    # across all rows. Terminal::Table sizes each column to its widest
+    # cell, so per-row ratios would produce columns wider than the
+    # terminal budget.
+    max_w = Hash.new(0)
+    el.children.each do |section|
+      next unless %i[thead tbody tfoot].include?(section.type)
+      section.children.each do |row|
+        row.children.each_with_index do |cell, i|
+          max_w[i] = [max_w[i], display_width(convert(cell, opts).strip)].max
+        end
+      end
+    end
+    n      = max_w.keys.map(&:to_i).max.to_i + 1
+    total  = max_w.values.sum
+    budget = width(percentage: 90) - (3 * n + 1)
+    col_w  = if total.zero?
+               same = (budget.to_f / n).to_i
+               Array.new(n, same)
+             else
+               (0...n).map { |i| ((max_w[i].to_f / total) * budget).to_i }
+             end
+    opts[:table_col_widths] = col_w
     table = Terminal::Table.new
     table.style = {
       all_separators: true,
@@ -533,33 +556,23 @@ class Kramdown::ANSI < Kramdown::Converter::Base
     ''
   end
 
-  # The convert_tr method processes a table row element by calculating column
-  # widths based on content size, wrapping text to fit within the terminal
-  # width, and adding the formatted row to the terminal table instance.
+  # The convert_tr method processes a table row element by wrapping each cell
+  # to the pre-computed column width and adding the formatted row to the
+  # terminal table instance.
   #
   # @param el [Kramdown::Element] the table row element to convert
   # @param opts [Hash] conversion options containing context for processing
+  #   (must include :table and :table_col_widths)
   # @return [String] an empty string to indicate successful processing and
   # addition of the row to the table instance
   def convert_tr(el, opts)
     return '' if el.children.empty?
-    cols = el.children.map do |c|
+    col_w = opts[:table_col_widths]
+    cols  = el.children.each_with_index.map do |c, i|
       stripped = convert(c, opts).strip
-      stripped.empty? ? ' ' : stripped
+      stripped.empty? ? ' ' : wrap(stripped, length: col_w[i])
     end
-    return '' if cols.empty?
-    col_widths     = cols.map { |c| display_width(c) }
-    col_widths_sum = col_widths.sum
-    full_width     = width(percentage: 90)
-    ratios         = cols.each_with_index.map do |c, i|
-      if col_widths_sum.zero?
-        1.0 / cols.size
-      else
-        col_widths[i] / col_widths_sum.to_f
-      end
-    end
-    wrapped_cols   = cols.zip(ratios).map { |c, r| wrap(c, length: r * full_width) }
-    opts[:table] << wrapped_cols
+    opts[:table] << cols
     ''
   end
 
@@ -662,5 +675,4 @@ class Kramdown::ANSI < Kramdown::Converter::Base
     @ansi_styles.apply(name, &block)
   end
 end
-
 Kramdown::Converter.const_set(:Ansi, Kramdown::ANSI)

data/lib/kramdown/version.rb

@@ -5,7 +5,7 @@
 # by hand; it is rewritten on every `rake version` invocation.
 module Kramdown
   # The current version of the Kramdown gem as a String.
-  VERSION         = '0.7.0'
+  VERSION         = '0.8.0'
   VERSION_ARRAY   = VERSION.split('.').map(&:to_i) # :nodoc:
   VERSION_MAJOR   = VERSION_ARRAY[0] # :nodoc:
   VERSION_MINOR   = VERSION_ARRAY[1] # :nodoc:

data/spec/assets/tables.ansi

@@ -286,60 +286,61 @@ a | b
 
 ────────────────────────────────────────────────────────────────────────────────
 
-╭──────┬────────────────────────────────────────────────────────────────╮
-│ Case │ What It Stresses                                               │
-├──────┼────────────────────────────────────────────────────────────────┤
-│ 01   │ Column count validation / overflow / underflow                 │
-├──────┼────────────────────────────────────────────────────────────────┤
-│ 02   │ Pipe-stripping logic (leading/trailing optional)               │
-├──────┼────────────────────────────────────────────────────────────────┤
-│ 03   │ Escape sequence handling (\|)                                  │
-├──────┼────────────────────────────────────────────────────────────────┤
-│ 04   │ Empty-string cell rendering, border alignment                  │
-├──────┼────────────────────────────────────────────────────────────────┤
-│ 05   │ Single-column edge (degenerate table)                          │
-├──────┼────────────────────────────────────────────────────────────────┤
-│ 06   │ Zero data rows — does it draw a header-only box?               │
-├──────┼────────────────────────────────────────────────────────────────┤
-│ 07   │ Consecutive tables — does the second become data of the first? │
-├──────┼────────────────────────────────────────────────────────────────┤
-│ 08   │ Unicode width calculation for column sizing                    │
-├──────┼────────────────────────────────────────────────────────────────┤
-│ 09   │ Inline markdown rendering inside a table cell                  │
-├──────┼────────────────────────────────────────────────────────────────┤
-│ 10   │ Alignment row with fewer columns than header                   │
-├──────┼────────────────────────────────────────────────────────────────┤
-│ 11   │ Blank line splitting one table into two                        │
-├──────┼────────────────────────────────────────────────────────────────┤
-│ 12   │ Whitespace normalization / trimming                            │
-├──────┼────────────────────────────────────────────────────────────────┤
-│ 13   │ Raw HTML + <br> in cell context                                │
-├──────┼────────────────────────────────────────────────────────────────┤
-│ 14   │ Cell with only whitespace — empty or not?                      │
-├──────┼────────────────────────────────────────────────────────────────┤
-│ 15   │ Very wide table — does it wrap or overflow?                    │
-├──────┼────────────────────────────────────────────────────────────────┤
-│ 16   │ Non-standard alignment markers                                 │
-├──────┼────────────────────────────────────────────────────────────────┤
-│ 17   │ Table at doc start, glued to following header                  │
-├──────┼────────────────────────────────────────────────────────────────┤
-│ 18   │ Table glued to preceding header (no blank before)              │
-├──────┼────────────────────────────────────────────────────────────────┤
-│ 19   │ Single col, missing closing pipe on header                     │
-├──────┼────────────────────────────────────────────────────────────────┤
-│ 20   │ Not a table: single pipe line                                  │
-├──────┼────────────────────────────────────────────────────────────────┤
-│ 21   │ Not a table: header + sep, no body, missing pipe               │
-├──────┼────────────────────────────────────────────────────────────────┤
-│ 22   │ Table after code block, no blank between                       │
-├──────┼────────────────────────────────────────────────────────────────┤
-│ 23   │ Table after list item, no blank between                        │
-├──────┼────────────────────────────────────────────────────────────────┤
-│ 24   │ No leading pipe on separator row                               │
-├──────┼────────────────────────────────────────────────────────────────┤
-│ 25   │ No leading pipe on body row                                    │
-├──────┼────────────────────────────────────────────────────────────────┤
-│ 26   │ Indented table inside ordered list item                        │
-├──────┼────────────────────────────────────────────────────────────────┤
-│ 27   │ Not a table just a pipe in a sentence                          │
-╰──────┴────────────────────────────────────────────────────────────────╯
+╭──────┬─────────────────────────────────────────────────────────╮
+│ Case │ What It Stresses                                        │
+├──────┼─────────────────────────────────────────────────────────┤
+│ 01   │ Column count validation / overflow / underflow          │
+├──────┼─────────────────────────────────────────────────────────┤
+│ 02   │ Pipe-stripping logic (leading/trailing optional)        │
+├──────┼─────────────────────────────────────────────────────────┤
+│ 03   │ Escape sequence handling (\|)                           │
+├──────┼─────────────────────────────────────────────────────────┤
+│ 04   │ Empty-string cell rendering, border alignment           │
+├──────┼─────────────────────────────────────────────────────────┤
+│ 05   │ Single-column edge (degenerate table)                   │
+├──────┼─────────────────────────────────────────────────────────┤
+│ 06   │ Zero data rows — does it draw a header-only box?        │
+├──────┼─────────────────────────────────────────────────────────┤
+│ 07   │ Consecutive tables — does the second become data of the │
+│      │ first?                                                  │
+├──────┼─────────────────────────────────────────────────────────┤
+│ 08   │ Unicode width calculation for column sizing             │
+├──────┼─────────────────────────────────────────────────────────┤
+│ 09   │ Inline markdown rendering inside a table cell           │
+├──────┼─────────────────────────────────────────────────────────┤
+│ 10   │ Alignment row with fewer columns than header            │
+├──────┼─────────────────────────────────────────────────────────┤
+│ 11   │ Blank line splitting one table into two                 │
+├──────┼─────────────────────────────────────────────────────────┤
+│ 12   │ Whitespace normalization / trimming                     │
+├──────┼─────────────────────────────────────────────────────────┤
+│ 13   │ Raw HTML + <br> in cell context                         │
+├──────┼─────────────────────────────────────────────────────────┤
+│ 14   │ Cell with only whitespace — empty or not?               │
+├──────┼─────────────────────────────────────────────────────────┤
+│ 15   │ Very wide table — does it wrap or overflow?             │
+├──────┼─────────────────────────────────────────────────────────┤
+│ 16   │ Non-standard alignment markers                          │
+├──────┼─────────────────────────────────────────────────────────┤
+│ 17   │ Table at doc start, glued to following header           │
+├──────┼─────────────────────────────────────────────────────────┤
+│ 18   │ Table glued to preceding header (no blank before)       │
+├──────┼─────────────────────────────────────────────────────────┤
+│ 19   │ Single col, missing closing pipe on header              │
+├──────┼─────────────────────────────────────────────────────────┤
+│ 20   │ Not a table: single pipe line                           │
+├──────┼─────────────────────────────────────────────────────────┤
+│ 21   │ Not a table: header + sep, no body, missing pipe        │
+├──────┼─────────────────────────────────────────────────────────┤
+│ 22   │ Table after code block, no blank between                │
+├──────┼─────────────────────────────────────────────────────────┤
+│ 23   │ Table after list item, no blank between                 │
+├──────┼─────────────────────────────────────────────────────────┤
+│ 24   │ No leading pipe on separator row                        │
+├──────┼─────────────────────────────────────────────────────────┤
+│ 25   │ No leading pipe on body row                             │
+├──────┼─────────────────────────────────────────────────────────┤
+│ 26   │ Indented table inside ordered list item                 │
+├──────┼─────────────────────────────────────────────────────────┤
+│ 27   │ Not a table just a pipe in a sentence                   │
+╰──────┴─────────────────────────────────────────────────────────╯

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: kramdown-ansi
 version: !ruby/object:Gem::Version
-  version: 0.7.0
+  version: 0.8.0
 platform: ruby
 authors:
 - Florian Frank
@@ -15,14 +15,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.20.1
+        version: 2.21.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.20.1
+        version: 2.21.0
 - !ruby/object:Gem::Dependency
   name: all_images
   requirement: !ruby/object:Gem::Requirement
@@ -233,7 +233,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.22
+rubygems_version: 4.0.23
 specification_version: 4
 summary: Output markdown in the terminal with ANSI escape sequences
 test_files: