-
Notifications
You must be signed in to change notification settings - Fork 7
List argument must be an Array of Buffers #10
Comments
Huh, I'm not getting this issue on the most recent [dev] build of neovim. Perhaps if you have time you could try installing from neovim HEAD? Otherwise my question is whether you have any node rplugins. The API for node-host recently changed in a very backwards-incompatible way. |
I use nvim-refactor.js, but that is a recent plugin, and I believe using the latest api
Before update, nvim --version output:
Post update, output:
Still unable to run. |
Hmm, what was the API change to node-host? I'm the author of a nvim plugin using node-host, so it may be that my plugin is broken because of this change... |
@nhynes I think I narrowed this down to a change between node 5.0.0 and 5.6.0 - 5.0 works and 5.6 gives this error. |
It's working for me on node 5.4.1.. on mac at least. I'm not using any *Sync calls, so I don't think that can be the cause. Also, the plugin is working with newest node-host, so I don't think it can be that. |
Arch Linux node versions 5.5 and 5.6 do not work for me. It's possible 5.4 introduced the issues, but I was using this with 5.5 not long ago (I believe, I don't have 5.4 in my package cache). |
This looks like the node commit that breaks it looks like the bl module in breaking because of it. which is part of Neovim itself. so more of a Neovim issue than nodehost one? Perhaps an issue should be filed there instead. |
Yeah, that issue is definitely relevant. That check that was introduced into the node code is incompatible with bl. It's a major flaw in bl that it doesn't work. Now that node are checking types more seriously, it's stopped bl from working in that function. The issue is downstream. It's possibly fixed in the latest version of msgpack, but msgpack5rpc is using an outdated version. I feel some pressure may need applying to @tarruda to make this happen. https://github.com/tarruda/node-msgpack5rpc |
I've made a few PRs, and made a few forks, you can fix this locally if you're in a pickle like I was:
In writing: Go to node-host, rm your node_modules directory (clean the old and bad cached modules), switch to my node-client fork temporarily, and then install all your modules. Disclaimer: When node-host is fixed, to update you will need to do this:
before updating. |
Latest version of node-client is published, updating the version will fix this issue. |
yes, updating to latest node-client works for me, too. Thanks, @SevereOverfl0w ! |
Yep, sorry for the delay. I'm scheduled to do node-* stuff this afternoon. |
🎈 No worries, thank you for helping us track this down and resolve it. |
Done. Thanks again for finding and fixing the bug! |
When trying to
:UpdateRemotePlugins
node-host has started failing for js, I see this in my .nvimlog:The text was updated successfully, but these errors were encountered: