neovim

commit e00d67ef310cab43ac60610c710d36c1c0903759
parent debabaf884fc26e9dc257aa5b62581d3846f80f1
Author: Eike <eike.rackwitz@mail.de>
Date:   Mon, 10 Mar 2025 21:06:29 +0100

feat(checkhealth): group parsers by name and path in output

Problem:
  :checkhealth vim.treesitter sorts parser entries solely by path,
  splitting duplicates and reducing clarity.

Solution:
  Sort entries first by name, then by path, so that duplicates are grouped.

Diffstat:
M
runtime/lua/vim/treesitter/health.lua
 | 
31
+++++++++++++++++++++++++------

1 file changed, 25 insertions(+), 6 deletions(-)
diff --git a/runtime/lua/vim/treesitter/health.lua b/runtime/lua/vim/treesitter/health.lua
@@ -19,23 +19,42 @@ function M.check()
 
   health.start('Treesitter parsers')
   local parsers = vim.api.nvim_get_runtime_file('parser/*', true)
-  for _, parser in pairs(parsers) do
+
+  ---@class ParserEntry
+  ---@field name string
+  ---@field path string
+
+  local sorted_parsers = {} ---@type ParserEntry[]
+
+  for _, parser in ipairs(parsers) do
     local parsername = vim.fn.fnamemodify(parser, ':t:r')
-    local is_loadable, err_or_nil = pcall(ts.language.add, parsername)
+    table.insert(sorted_parsers, { name = parsername, path = parser })
+  end
+
+  table.sort(sorted_parsers, function(a, b)
+    if a.name == b.name then
+      return a.path < b.path
+    else
+      return a.name < b.name
+    end
+  end)
+
+  for _, parser in ipairs(sorted_parsers) do
+    local is_loadable, err_or_nil = pcall(ts.language.add, parser.name)
 
     if not is_loadable then
       health.error(
         string.format(
           'Parser "%s" failed to load (path: %s): %s',
-          parsername,
-          parser,
+          parser.name,
+          parser.path,
           err_or_nil or '?'
         )
       )
     else
-      local lang = ts.language.inspect(parsername)
+      local lang = ts.language.inspect(parser.name)
       health.ok(
-        string.format('Parser: %-20s ABI: %d, path: %s', parsername, lang.abi_version, parser)
+        string.format('Parser: %-20s ABI: %d, path: %s', parser.name, lang.abi_version, parser.path)
       )
     end
   end