build: fixed workflow

Author: devdammit

.github/workflows/ci.yml

@@ -22,12 +22,11 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
-      - uses: leafo/gh-actions-lua@v10
-        with:
-          luaVersion: "luajit-2.1.0-beta3"
-      - uses: leafo/gh-actions-luarocks@v4
-      - name: Install luacheck
-        run: luarocks install luacheck
+      - name: Install Lua and luacheck
+        run: |
+          sudo apt-get update
+          sudo apt-get install -y lua5.4 luarocks
+          sudo luarocks install luacheck
       - name: Run luacheck
         run: luacheck lua/ plugin/
 
@@ -45,11 +44,10 @@ jobs:
         with:
           neovim: true
           version: ${{ matrix.neovim }}
-      - name: Cache test deps
-        uses: actions/cache@v4
-        with:
-          path: .test-deps
-          key: test-deps-${{ matrix.neovim }}-${{ hashFiles('tests/minimal_init.lua') }}
+      - name: Install C compiler (for treesitter parsers)
+        run: |
+          sudo apt-get update
+          sudo apt-get install -y build-essential
       - name: Run headless smoke test
         env:
           OPENAPI_TEST_DEPS: ${{ github.workspace }}/.test-deps

.luacheckrc

@@ -3,7 +3,13 @@ std = "luajit"
 cache = true
 codes = true
 
--- Globals provided by the Neovim runtime / plugins.
+-- `vim` is writable (we set vim.g.* flags), so declare it as a mutable global
+-- with known writable fields rather than read-only.
+globals = {
+  "vim.g",
+}
+
+-- Globals provided by the Neovim runtime / plugins (read-only).
 read_globals = {
   "vim",
   "Snacks",

lua/openapi/parser/model.lua

@@ -56,7 +56,7 @@ local function parse_paths(paths_node, bufnr, ops)
   if not paths_node then
     return
   end
-  for path_key, path_val, path_text in ts.iter_pairs(paths_node, bufnr) do
+  for _, path_val, path_text in ts.iter_pairs(paths_node, bufnr) do
     if path_text and path_text:sub(1, 1) == "/" then
       for m_key, _, m_text in ts.iter_pairs(path_val, bufnr) do
         if m_text and HTTP_METHODS[m_text:lower()] then
@@ -85,9 +85,9 @@ local function parse_components(components_node, bufnr, comps, targets)
   if not components_node then
     return
   end
-  for _kind_key, kind_val, kind_text in ts.iter_pairs(components_node, bufnr) do
+  for _, kind_val, kind_text in ts.iter_pairs(components_node, bufnr) do
     if kind_text then
-      for name_key, _name_val, name_text in ts.iter_pairs(kind_val, bufnr) do
+      for name_key, _, name_text in ts.iter_pairs(kind_val, bufnr) do
         if name_text then
           local pointer = "/components/" .. kind_text .. "/" .. name_text
           local pos = mkpos(name_key)

tests/minimal_init.lua

@@ -36,3 +36,10 @@ vim.opt.runtimepath:prepend(root())
 if pcall(require, "nvim-treesitter.configs") then
   pcall(vim.cmd, "TSInstallSync! yaml")
 end
+
+-- Fail loudly if the yaml parser still isn't available, so CI surfaces the
+-- real cause instead of the smoke test silently doing nothing.
+if not pcall(vim.treesitter.language.add, "yaml") then
+  io.stderr:write("FATAL: yaml treesitter parser is not available\n")
+  vim.cmd("cquit 1")
+end