coq_nvim icon indicating copy to clipboard operation
coq_nvim copied to clipboard

Custom sources not working

Open williamboman opened this issue 4 years ago • 4 comments

Hey! There seem to have been a regression with custom 3rd party sources, related to #293 (more context can be found in that thread). Whenever I have a custom source registered I run into the following error:

Error executing luv callback:
[string "<nvim>"]:62: client_id: expected number, got string
stack traceback:
        [string "<nvim>"]:62: in function 'on_resp_old'
        [string "<nvim>"]:82: in function 'on_resp'
        [string "<nvim>"]:234: in function 'callback'
        /Users/williamboman/.config/nvim/lua/wb/coq_3p/uuid.lua:43: in function </Users/williamboman/.config/nvim/lua/wb/coq_3p/uuid.lua:29>

You may find the implementation of the source here.

williamboman avatar Oct 19 '21 12:10 williamboman

Getting back on the discussion of the other ticket, 3p words because they don't use the COQsources variable as I looked in. So I guess that they are managed in a different way compared to our custom sources.

Mte90 avatar Oct 21 '21 14:10 Mte90

@ms-jpq any help for this?

Mte90 avatar Jan 03 '22 11:01 Mte90

another ping, I have no idea how to patch it

Mte90 avatar Mar 11 '22 18:03 Mte90

I removed that validation on https://github.com/ms-jpq/coq_nvim/blob/4cda35c003e646b5f6bdb312dab70106027fc3e5/coq/lsp/requests/lsp.lua#L62 and the custom source works. I will do a PR for that but I am not sure if fix the issue or why that code is here but atleast after 7 months I am able to use my custom source again...

Mte90 avatar May 26 '22 09:05 Mte90

So my patch for custom source was merged and now they should be working again. Someone can double check?

Mte90 avatar Aug 18 '22 12:08 Mte90

Cool :+1:! I don't use coq_nvim anymore so I won't be able to test, so I'll close this

williamboman avatar Aug 18 '22 15:08 williamboman