neovim/node-host

Track msgpack fix for array of empty string mcollina/msgpack5/issues/48

snoe opened this issue · 2 comments

snoe commented

One of my users uncovered and fixed a bug in the underlying msgpack lib. Which has since been released under msgpack 3.4.1

I'm raising an issue here as a sort of master tracking issue, hoping we can coordinate with @tarruda and @nhynes to get the versions bumped in https://github.com/tarruda/node-msgpack5rpc and https://github.com/neovim/node-client and finally here.

mcollina/msgpack5#48
clojure-vim/clj-refactor.nvim#8

Seems reasonable. You might get a more prompt response if you make a PR/issue in node-msgpack5rpc [and then reference the contrib here so that it gets updated automatically].

snoe commented

Looking at this again this morning, it looks like updating node-host to use neovim-client 2.1.0 actually does bring in the msgpack 3.4 patch. So #17 should fix this as well.