chore(logging): include buffer information when logging unhandled type errors

## Details

Occasionally I'll see unhandled type errors for table rows, but I have
no idea what kind of tables actually produce them. To make this easier
`log.unhandled` now takes a buffer id which is included in the output.
Once I have the file names that cause this I'll be able to handle it.

I've also updated all the log entry names to upper camel case.

Author: MeanderingProgrammer

doc/render-markdown.txt

@@ -1,4 +1,4 @@
-*render-markdown.txt*         For NVIM v0.11.1        Last change: 2025 May 23
+*render-markdown.txt*         For NVIM v0.11.1        Last change: 2025 May 27
 
 ==============================================================================
 Table of Contents                          *render-markdown-table-of-contents*

lua/render-markdown/core/handlers.lua

@@ -53,22 +53,22 @@ end
 ---@param language string
 ---@return render.md.Mark[]
 function M.tree(context, ctx, language)
-    log.buf('debug', 'language', ctx.buf, language)
+    log.buf('debug', 'Language', ctx.buf, language)
     if not context.view:overlaps(ctx.root) then
         return {}
     end
     local marks = {} ---@type render.md.Mark[]
     local custom = M.config.custom[language]
     if custom then
-        log.buf('debug', 'handler', ctx.buf, 'custom')
+        log.buf('debug', 'Handler', ctx.buf, 'custom')
         vim.list_extend(marks, custom.parse(ctx))
         if not custom.extends then
             return marks
         end
     end
     local builtin = M.builtin[language]
     if builtin then
-        log.buf('debug', 'handler', ctx.buf, 'builtin')
+        log.buf('debug', 'Handler', ctx.buf, 'builtin')
         vim.list_extend(marks, builtin.parse(ctx))
     end
     return marks

lua/render-markdown/core/log.lua

@@ -74,21 +74,19 @@ end
 ---@param capture string
 ---@param node render.md.Node
 function M.node(capture, node)
-    M.add('debug', 'node', {
+    M.add('debug', 'Node', {
         capture = capture,
         text = node.text,
         rows = { node.start_row, node.end_row },
         cols = { node.start_col, node.end_col },
     })
 end
 
----Encountered if new type is seen for a particular group
----@param language string
----@param group string
----@param value string
-function M.unhandled_type(language, group, value)
-    local message = ('%s -> %s -> %s'):format(language, group, value)
-    M.add('error', 'unhandled type', message)
+---Encountered if new type is seen in a particular node
+---@param buf integer
+---@param ... string
+function M.unhandled(buf, ...)
+    M.buf('error', 'UnhandledType', buf, table.concat({ ... }, ' -> '))
 end
 
 ---@param level render.md.log.Level

lua/render-markdown/core/manager.lua

@@ -160,23 +160,23 @@ end
 ---@param buf integer
 ---@return boolean
 function M.should_attach(buf)
-    log.buf('info', 'attach', buf, 'start')
+    log.buf('info', 'Attach', buf, 'start')
 
     if M.attached(buf) then
-        log.buf('info', 'attach', buf, 'skip', 'already attached')
+        log.buf('info', 'Attach', buf, 'skip', 'already attached')
         return false
     end
 
     if not vim.api.nvim_buf_is_valid(buf) then
-        log.buf('info', 'attach', buf, 'skip', 'invalid')
+        log.buf('info', 'Attach', buf, 'skip', 'invalid')
         return false
     end
 
     local file_type = Env.buf.get(buf, 'filetype')
     local file_types = M.config.file_types
     if not vim.tbl_contains(file_types, file_type) then
         local reason = file_type .. ' /∈ ' .. vim.inspect(file_types)
-        log.buf('info', 'attach', buf, 'skip', 'file type', reason)
+        log.buf('info', 'Attach', buf, 'skip', 'file type', reason)
         return false
     end
 
@@ -185,16 +185,16 @@ function M.should_attach(buf)
     local max_file_size = config.max_file_size
     if file_size > max_file_size then
         local reason = ('%f > %f'):format(file_size, max_file_size)
-        log.buf('info', 'attach', buf, 'skip', 'file size', reason)
+        log.buf('info', 'Attach', buf, 'skip', 'file size', reason)
         return false
     end
 
     if M.config.ignore(buf) then
-        log.buf('info', 'attach', buf, 'skip', 'user ignore')
+        log.buf('info', 'Attach', buf, 'skip', 'user ignore')
         return false
     end
 
-    log.buf('info', 'attach', buf, 'success')
+    log.buf('info', 'Attach', buf, 'success')
     M.buffers[#M.buffers + 1] = buf
     return true
 end

lua/render-markdown/core/ui.lua

@@ -50,7 +50,7 @@ end
 ---@param event string
 ---@param change boolean
 function M.update(buf, win, event, change)
-    log.buf('info', 'update', buf, event, ('change %s'):format(change))
+    log.buf('info', 'Update', buf, event, ('change %s'):format(change))
     M.updater.new(buf, win, change):start()
 end
 
@@ -119,7 +119,7 @@ function Updater:run()
         and self.config.resolved:render(self.mode)
         and not Env.win.get(self.win, 'diff')
         and Env.win.view(self.win).leftcol == 0
-    log.buf('info', 'render', self.buf, render)
+    log.buf('info', 'Render', self.buf, render)
     local next_state = render and 'rendered' or 'default'
     for _, window in ipairs(Env.buf.windows(self.buf)) do
         for name, value in pairs(self.config.win_options) do
@@ -163,7 +163,7 @@ end
 function Updater:get_extmarks()
     local has_parser, parser = pcall(vim.treesitter.get_parser, self.buf)
     if not has_parser or not parser then
-        log.buf('error', 'fail', self.buf, 'no treesitter parser found')
+        log.buf('error', 'Fail', self.buf, 'no treesitter parser found')
         return {}
     end
     -- reset buffer context

lua/render-markdown/handler/html.lua

@@ -37,7 +37,7 @@ function Handler:run(root)
     local marks = Marks.new(self.context, true)
     self.context.view:nodes(root, self.query, function(capture, node)
         local render = self.renders[capture]
-        assert(render ~= nil, 'unhandled html capture: ' .. capture)
+        assert(render, 'unhandled html capture: ' .. capture)
         render:execute(self.context, marks, node)
     end)
     return marks:get()

lua/render-markdown/handler/latex.lua

@@ -32,7 +32,7 @@ function Handler:run(root)
         return {}
     end
     if vim.fn.executable(self.config.converter) ~= 1 then
-        log.add('debug', 'executable not found', self.config.converter)
+        log.add('debug', 'ConverterNotFound', self.config.converter)
         return {}
     end
 
@@ -83,7 +83,7 @@ function Handler:convert(text)
         local converter = self.config.converter
         result = vim.fn.system(converter, text)
         if vim.v.shell_error == 1 then
-            log.add('error', converter, result)
+            log.add('error', 'ConverterFailed', converter, result)
             result = 'error'
         end
         Handler.cache[text] = result

lua/render-markdown/handler/markdown.lua

@@ -63,7 +63,7 @@ function Handler:run(root)
     local marks = Marks.new(self.context, false)
     self.context.view:nodes(root, self.query, function(capture, node)
         local render = self.renders[capture]
-        assert(render ~= nil, 'unhandled markdown capture: ' .. capture)
+        assert(render, 'unhandled markdown capture: ' .. capture)
         render:execute(self.context, marks, node)
     end)
     return marks:get()

lua/render-markdown/handler/markdown_inline.lua

@@ -48,7 +48,7 @@ function Handler:run(root)
     local marks = Marks.new(self.context, true)
     self.context.view:nodes(root, self.query, function(capture, node)
         local render = self.renders[capture]
-        assert(render ~= nil, 'unhandled inline capture: ' .. capture)
+        assert(render, 'unhandled inline capture: ' .. capture)
         render:execute(self.context, marks, node)
     end)
     return marks:get()

lua/render-markdown/health.lua

@@ -5,7 +5,7 @@ local state = require('render-markdown.state')
 local M = {}
 
 ---@private
-M.version = '8.4.6'
+M.version = '8.4.7'
 
 function M.check()
     M.start('version')