Error starting LSP client

[pkropachev]

Hello!

By some reason today I get the following error while opening nvim 0.11.3.

Error  03:56:11 PM msg_show.lua_error Error executing vim.schedule lua callback: ...ols/squashfs-root/usr/share/nvim/runtime/lua/vim/lsp.lua:1020: client_id: expected number, got nil
stack traceback:
	[C]: in function 'error'
	vim/shared.lua: in function 'validate'
	...ols/squashfs-root/usr/share/nvim/runtime/lua/vim/lsp.lua:1020: in function 'buf_attach_client'
	...are/LazyVim/lazy/copilot.lua/lua/copilot/client/init.lua:69: in function 'buf_attach'
	...are/LazyVim/lazy/copilot.lua/lua/copilot/client/init.lua:149: in function ''
	vim/_editor.lua: in function <vim/_editor.lua:0>
	[C]: in function 'wait'
	...i/.local/share/LazyVim/lazy/lazy.nvim/lua/lazy/async.lua:113: in function 'wait'
	...share/LazyVim/lazy/lazy.nvim/lua/lazy/manage/process.lua:235: in function <...share/LazyVim/lazy/lazy.nvim/lua/lazy/manage/process.lua:232>
	[C]: in function 'pcall'
	...cal/share/LazyVim/lazy/lazy.nvim/lua/lazy/manage/git.lua:192: in function 'get_tag_refs'
	...cal/share/LazyVim/lazy/lazy.nvim/lua/lazy/manage/git.lua:161: in function 'ref'
	...cal/share/LazyVim/lazy/lazy.nvim/lua/lazy/manage/git.lua:149: in function <...cal/share/LazyVim/lazy/lazy.nvim/lua/lazy/manage/git.lua:118>
	[C]: in function 'pcall'
	...share/LazyVim/lazy/lazy.nvim/lua/lazy/manage/checker.lua:43: in function 'fast_check'
	...share/LazyVim/lazy/lazy.nvim/lua/lazy/manage/checker.lua:15: in function 'start'
	...al/share/LazyVim/lazy/lazy.nvim/lua/lazy/core/config.lua:345: in function ''
	vim/_editor.lua: in function ''
	vim/_editor.lua: in function <vim/_editor.lua:0>

Do you have any idea what can be reason?

[AntoineGS]

Hmm it is surely related to my last commit, can you share your configuration file? It seems related to the should_attach function.

[AntoineGS]

I added an additional validation prior to where the error occurred, let me know if the issue persists!

[neonvoidx]

still getting same error with newest commit 045daa5

config:

  {
    "zbirenbaum/copilot.lua",
    opts = {
      filetypes = { markdown = false, help = false }
    },
  },

@AntoineGS

[pkropachev]

Actually, now it's more better on my side. I'm doing the following:

# nv test

where test file is not exist.

I still see notice about error staring LSP client, but Copilot is running. And I don't see msg_show.lua_error message like in my initial message.

[AntoineGS]

Ok the issue seems to be with using the plugin without event = "InsertEnter", can you try by adding it to your configuration?

  {
    "zbirenbaum/copilot.lua",
    event = "InsertEnter",
    opts = {
      filetypes = { markdown = false, help = false }
    },
  },

I am working on a fix!

[neonvoidx]

Ok the issue seems to be with using the plugin without event = "InsertEnter", can you try by adding it to your configuration?

{
"zbirenbaum/copilot.lua",
event = "InsertEnter",
opts = {
filetypes = { markdown = false, help = false }
},
},

I am working on a fix!

No difference, tried that as well, same error

[pkropachev]

This is my config:

return {
  -- copilot.lua
  {
    "zbirenbaum/copilot.lua",
    cmd = "Copilot",
    event = "InsertEnter",
    config = function()
      vim.g.copilot_proxy = "myproxy"
      require("copilot").setup({})
    end,
  },
  {
    "folke/snacks.nvim",
    -- Updated this section to align with approach taken in other LazyVim snacks configs
    ---@diagnostic disable-next-line: unused-local
    opts = function(_, opts)
      -- Credit for this section goes to exsesx
      -- https://github.com/LazyVim/LazyVim/discussions/4232#discussioncomment-11191278
      local snacks = require("snacks")
      -- Check whether Copilot is installed
      if pcall(require, "copilot") then
        -- Workaround to keep track of state
        vim.g.snacks_copilot_enabled = true
        snacks
          .toggle({
            name = "Copilot Completion",
            color = {
              enabled = "azure",
              disabled = "orange",
            },
            get = function()
              return vim.g.snacks_copilot_enabled
            end,
            set = function(state)
              if state then
                vim.g.snacks_copilot_enabled = true
                require("copilot.command").enable()
              else
                vim.g.snacks_copilot_enabled = false
                require("copilot.command").disable()
              end
            end,
          })
          :map("<leader>at")
      end
    end,
  },
}

Can it be related with update Copilot LSP 0e1348e?

[AntoineGS]

Oh I was able to replicate by adding lazy = false to my configuration!

[neonvoidx]

Oh I was able to replicate by adding lazy = false to my configuration!

Ya I don't think we should technically have to have lazy=true for this to work, it was fine before, this is a regression bug for sure

[AntoineGS]

Don't worry it was not intentional :)

[AntoineGS]

I pushed another change which should fix it, let me know if the issue persists