chore(refactor): width calculations & version updates

## Details

- Add `widths` method to `Node` class to get width of all lines
- For code blocks always round padding / margin to nearest `tab` if the
  text contains a tab, regardless of which side is being calculated
- Make `buf` & `win` fields public in `Context` class
- Remove cache around window width and compute as needed
- Simplify state of `code` renderer by moving logic to where fields are
  used, only keep around the width information in `self.data`
- Update help doc neovim version to `0.11.0`
- Set the recommended version to `0.11.0`, minimum remains `0.9.0`
- Other minor internal naming changes

Author: MeanderingProgrammer

doc/render-markdown.txt

@@ -1,4 +1,4 @@
-*render-markdown.txt*           For 0.10.0          Last change: 2025 March 30
+*render-markdown.txt*           For 0.11.0          Last change: 2025 March 31
 
 ==============================================================================
 Table of Contents                          *render-markdown-table-of-contents*

justfile

@@ -10,7 +10,7 @@ update:
   ../../open-source/panvimdoc/panvimdoc.sh \
     --project-name render-markdown \
     --input-file README.md \
-    --vim-version 0.10.0
+    --vim-version 0.11.0
 
 test:
   just busted "tests"

lua/render-markdown/core/conceal.lua

@@ -21,14 +21,12 @@ local Conceal = {}
 Conceal.__index = Conceal
 
 ---@param context render.md.Context
----@param buf integer
----@param win integer
 ---@return render.md.Conceal
-function Conceal.new(context, buf, win)
+function Conceal.new(context)
     local self = setmetatable({}, Conceal)
     self.context = context
-    self.buf = buf
-    self.level = Env.win.get(win, 'conceallevel')
+    self.buf = context.buf
+    self.level = Env.win.get(context.win, 'conceallevel')
     self.computed = false
     self.lines = {}
     return self

lua/render-markdown/core/context.lua

@@ -16,13 +16,12 @@ local log = require('render-markdown.core.log')
 ---@field width integer
 
 ---@class render.md.Context
----@field private buf integer
----@field private win integer
 ---@field private ranges render.md.Range[]
 ---@field private callouts table<integer, render.md.CustomCallout>
 ---@field private checkboxes table<integer, render.md.CustomCheckbox>
 ---@field private offsets table<integer, render.md.context.Offset[]>
----@field private window_width? integer
+---@field buf integer
+---@field win integer
 ---@field mode string
 ---@field top_level_mode boolean
 ---@field conceal render.md.Conceal
@@ -35,23 +34,21 @@ Context.__index = Context
 ---@return render.md.Context
 function Context.new(props, offset)
     local self = setmetatable({}, Context)
-    self.buf = props.buf
-    self.win = props.win
 
     local ranges = {}
-    for _, window in ipairs(Env.buf.windows(self.buf)) do
-        table.insert(ranges, Context.compute_range(self.buf, window, offset))
+    for _, window in ipairs(Env.buf.windows(props.buf)) do
+        table.insert(ranges, Context.compute_range(props.buf, window, offset))
     end
     self.ranges = Range.coalesce(ranges)
-
     self.callouts = {}
     self.checkboxes = {}
     self.offsets = {}
-    self.window_width = nil
 
+    self.buf = props.buf
+    self.win = props.win
     self.mode = props.mode
     self.top_level_mode = props.top_level_mode
-    self.conceal = Conceal.new(self, self.buf, self.win)
+    self.conceal = Conceal.new(self)
     self.last_heading = nil
 
     return self
@@ -93,11 +90,6 @@ function Context:skip(component)
     return not Env.mode.is(self.mode, component.render_modes)
 end
 
----@return integer
-function Context:get_buf()
-    return self.buf
-end
-
 ---@param row integer
 ---@return render.md.CustomCallout?
 function Context:get_callout(row)
@@ -162,26 +154,18 @@ function Context:get_offset(node)
     return result
 end
 
----@param offset number
----@param width integer
+---@param value number
+---@param used integer
 ---@return integer
-function Context:to_width(offset, width)
-    if offset <= 0 then
+function Context:percent(value, used)
+    if value <= 0 then
         return 0
-    elseif offset < 1 then
-        return math.floor(((self:get_width() - width) * offset) + 0.5)
+    elseif value < 1 then
+        local available = Env.win.width(self.win) - used
+        return math.floor((available * value) + 0.5)
     else
-        return offset
-    end
-end
-
----@private
----@return integer
-function Context:get_width()
-    if self.window_width == nil then
-        self.window_width = Env.win.width(self.win)
+        return value
     end
-    return self.window_width
 end
 
 ---@param win integer

lua/render-markdown/core/ui.lua

@@ -132,7 +132,8 @@ function M.run_update(buf, win, change)
     end
 
     if next_state == 'rendered' then
-        if not buffer:has_marks() or parse then
+        local initial = not buffer:has_marks()
+        if initial or parse then
             M.clear(buf, buffer)
             buffer:set_marks(M.parse_buffer({
                 buf = buf,

lua/render-markdown/health.lua

@@ -5,12 +5,12 @@ local state = require('render-markdown.state')
 local M = {}
 
 ---@private
-M.version = '8.1.23'
+M.version = '8.1.24'
 
 function M.check()
     M.start('version')
     vim.health.ok('plugin ' .. M.version)
-    M.neovim('0.9', '0.10')
+    M.neovim('0.9', '0.11')
 
     M.start('configuration')
     local errors = state.validate()

lua/render-markdown/lib/node.lua

@@ -1,3 +1,6 @@
+local Iter = require('render-markdown.lib.iter')
+local Str = require('render-markdown.lib.str')
+
 ---@class render.md.Node
 ---@field private buf integer
 ---@field private node TSNode
@@ -27,13 +30,6 @@ function Node.new(buf, node)
     return self
 end
 
----@private
----@param node TSNode
----@return render.md.Node
-function Node:create(node)
-    return Node.new(self.buf, node)
-end
-
 ---@param a render.md.Node
 ---@param b render.md.Node
 ---@return boolean
@@ -45,16 +41,18 @@ function Node.__lt(a, b)
     end
 end
 
+---@private
+---@param node TSNode
+---@return render.md.Node
+function Node:create(node)
+    return Node.new(self.buf, node)
+end
+
 ---@return TSNode
 function Node:get()
     return self.node
 end
 
----@return boolean
-function Node:has_error()
-    return self.node:has_error()
-end
-
 ---@return integer[]
 function Node:sections()
     local result, levels, section = {}, 0, self:parent('section')
@@ -169,6 +167,12 @@ function Node:for_each_child(callback)
     end
 end
 
+---@return string?
+function Node:after()
+    local row, col = self.end_row, self.end_col
+    return vim.api.nvim_buf_get_text(self.buf, row, col, row, col + 1, {})[1]
+end
+
 ---@param position 'above'|'first'|'below'|'last'
 ---@param by integer
 ---@return string?
@@ -186,15 +190,10 @@ function Node:line(position, by)
     return row ~= nil and vim.api.nvim_buf_get_lines(self.buf, row, row + 1, false)[1] or nil
 end
 
----@return string[]
-function Node:lines()
-    return vim.api.nvim_buf_get_lines(self.buf, self.start_row, self.end_row, false)
-end
-
----@return string?
-function Node:after()
-    local row, col = self.end_row, self.end_col
-    return vim.api.nvim_buf_get_text(self.buf, row, col, row, col + 1, {})[1]
+---@return integer[]
+function Node:widths()
+    local lines = vim.api.nvim_buf_get_lines(self.buf, self.start_row, self.end_row, false)
+    return Iter.list.map(lines, Str.width)
 end
 
 return Node