Broken with neovim python-client 0.3.0: request from non-main thread

[numirias]

Broken with neovim python-client 0.3.0: request from non-main thread

[bfredl]

Calls from other threads must use nvim.async_call(closure), this has been documented for quite some time https://pynvim.readthedocs.io/en/latest/usage/python-plugin-api.html#async-calls.

It might have been working by accident for some use patterns, but is inherently racy and could trigger issues on any code or environmental change. An API should only be considered thread-safe it it explicitly claims to be. Changing unpredictable behavior to a consistent error message is not a breaking change, it only makes already present issues to be discovered faster.

[bfredl]

@numirias Indeed, I will improve the message to next version.

[numirias]

Thanks for reporting! It seems with 0.3.0, Neovim is now raising an error when attempting to perform an API request from a different thread. (Although, technically it works.)

Since I'm currently traveling, I'll need some time to look into this and properly fix it. However, I'll try to come up with a quickfix that bypasses the error message until then. Sorry for the trouble.

(If someone feels like supplying a patch, please go ahead. I'm happy for a PR.)

[wookayin]

Thanks for your response!

For other's reference, this is the commit that started to prevent requests from a different thread: neovim/pynvim#255

I will also look into this after I get into a free cycle.

[numirias]

@bfredl Thanks for clarifying! Not sure how I missed that. This will probably resolve other bugs here too.

Mentioning in the error message or the source that async_call() should be used from threads might have made it even more obvious.