Crash on startup with codecompanion v19 — nil passed to `pairs()` in `variables.lua`

[dtrckd]

Bug

mcphub crashes on startup when used with codecompanion v19 release. The error occurs
inside the vim.schedule callback registered by variables.lua.

Error

Error executing vim.schedule lua callback: .../mcphub/extensions/codecompanion/variables.lua:20:
bad argument #1 to 'pairs' (table expected, got nil)
stack traceback:
    [C]: in function 'pairs'
    .../mcphub/extensions/codecompanion/variables.lua:20: in function 'register'
    .../mcphub/extensions/codecompanion/variables.lua:92: in function 'fn'
    vim/_editor.lua:366: in function <vim/_editor.lua:365>

Environment

• mcphub.nvim: 7cd5db3
• codecompanion.nvim: v19 (e203cc6 / 19.0.0)

[powerman]

Temporary workaround is extensions.mcphub.opts.make_vars = false.

[git-into-it]

Looks like codecompanion removed variables from chat interactions.

[powerman]

Not removed, just renamed. Here is a quick fix (can be applied using e.g. https://github.com/polirritmico/lazy-local-patcher.nvim):

diff --git a/lua/mcphub/extensions/codecompanion/variables.lua b/lua/mcphub/extensions/codecompanion/variables.lua
index 2a15a55..20c7e14 100644
--- a/lua/mcphub/extensions/codecompanion/variables.lua
+++ b/lua/mcphub/extensions/codecompanion/variables.lua
@@ -14,7 +14,7 @@ function M.register(opts)
         return
     end
 
-    local cc_variables = config.interactions.chat.variables
+    local cc_variables = config.interactions.chat.editor_context
 
     -- Remove existing MCP variables
     for key, value in pairs(cc_variables) do

[Narong-Kanthanu]

this issue fixed in PR #276, Merges plz

[gunslingerfry]

Not removed, just renamed. Here is a quick fix (can be applied using e.g. https://github.com/polirritmico/lazy-local-patcher.nvim):

diff --git a/lua/mcphub/extensions/codecompanion/variables.lua b/lua/mcphub/extensions/codecompanion/variables.lua
index 2a15a55..20c7e14 100644
--- a/lua/mcphub/extensions/codecompanion/variables.lua
+++ b/lua/mcphub/extensions/codecompanion/variables.lua
@@ -14,7 +14,7 @@ function M.register(opts)
return
end

• local cc_variables = config.interactions.chat.variables

• local cc_variables = config.interactions.chat.editor_context

  -- Remove existing MCP variables
  for key, value in pairs(cc_variables) do

As annoying as this issue has been, it feels worth it to have learned about lazy-local-patcher.nvim! Very cool plugin!