feature: fire an event when a NES is available

[rachartier]

Did you check the docs?

• I have read all the sidekick.nvim docs

Is your feature request related to a problem? Please describe.

Hello,
Thank you for this plugin.

In my plugin, https://github.com/rachartier/tiny-inline-diagnostic.nvim, when a NES is displayed, there are some visual "bugs":

instead of:

I do not think it is easy to resolve this.

Describe the solution you'd like

I was wondering if it is possible to fire an event when a NES is found and displayed (or cancelled), so that users can disable whatever they want if needed.

In my case, I could add an autocmd to this event and disable diagnostics display.

What do you think?

Describe alternatives you've considered

Using

vim.lsp.config("copilot", {
  handlers = {
    didChangeStatus = function(err, res, ctx)
      if err then
        return
      end
      vim.print(res)
    end,
  },
})

But I think it will break your implementation

EDIT: tested overriding didChangeStatus, it does not work

Additional context

No response

[rachartier]

A quick workaround for those interested, it is quite ugly to do it like that, but for now, I don't see any other possibility:

local tiny_diags_disabled_by_nes = false

return {
  {
    "folke/sidekick.nvim",
    enabled = true,
    lazy = true,
    opts = {
      nes = {
        enabled = true,
      },
    },
    config = function(_, opts)
      local sidekick = require("sidekick.nes")
      local inline_diag = require("tiny-inline-diagnostic")

      local old_handler = sidekick._handler
      local old_clear = sidekick.clear

      local new_handler = function(err, result, ctx, config)
        local res = result or { edits = {} }
        if not vim.tbl_isempty(res.edits) then
          tiny_diags_disabled_by_nes = true
          inline_diag.disable()
        end

        return old_handler(err, result, ctx, config)
      end

      local new_clear = function()
        if tiny_diags_disabled_by_nes then
          inline_diag.enable()
        end
        old_clear()
      end

      require("sidekick.nes")._handler = new_handler
      require("sidekick.nes").clear = new_clear

      require("sidekick").setup(opts)
    end,
  },
}

[folke]

Just added this!

[rachartier]

Works perfectly, thank you!