fennel stdio client in conjure cause neovim to block forever?

[glyh]

Steps to reproduce

Here's the minimal init.lua to reproduce:

require('packer').startup( {function()
  -- Packer can manage itself
  use 'wbthomason/packer.nvim'
  use {'Olical/conjure', commit="8dedf17ad815ce78122c22d25e93777e3f2d8cfe"} 
end, config = {
  git = {
    default_url_format = "https://hub.fastgit.org/%s"}}})

vim.g["conjure#filetype#fennel"] = "conjure.client.fennel.stdio"

Then start neovim and install all plugins, and open some fennel file, then the neovim get stucks and nothing will happen, except I manually stop neovim with <C-c>, even the UI rendered by neovim is left in the terminal...

Some information

• neovim version:

NVIM v0.6.0-dev+98-ga5c25e4f3
Build type: RelWithDebInfo
LuaJIT 2.1.0-beta3
Compilation: /usr/bin/gcc-11 -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -DNVIM_TS_HAS_SET_MATCH_LIMIT -O2 -g -Og -g -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wmissing-prototypes -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fno-common -fdiagnostics-color=always -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -DMIN_LOG_LEVEL=3 -I/home/runner/work/neovim/neovim/build/config -I/home/runner/work/neovim/neovim/src -I/home/runner/work/neovim/neovim/.deps/usr/include -I/usr/include -I/home/runner/work/neovim/neovim/build/src/nvim/auto -I/home/runner/work/neovim/neovim/build/include
Compiled by runner@fv-az87-30

Features: +acl +iconv +tui
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/share/nvim"

Run :checkhealth for more info

• fennel: Fennel 0.10.0-dev on Lua 5.4
• OS : Linux arch 5.13.6-arch1-1 #1 SMP PREEMPT Thu, 29 Jul 2021 00:21:06 +0000 x86_64 GNU/Linux

[Olical]

Hmm this is working absolutely fine for me 🤔 do you have fennel installed through luarocks, added it to your path and have a working lua instance?

[glyh]

I've installed it through AUR, which can be required from lua REPL.

I just tired out install fennel via luarocks

sudo luarocks install fennel

However, this results in the same bug ...

[Olical]

So if you run fennel in your terminal it launches the REPL fine?

[glyh]

Yep

[glyh]

I've found that it's not blocking all the way, but make every key stroke lags, and most of the keystrokes are not responded...
Here's a video:

[Olical]

Hmm, could you share the output from your Fennel REPL? Mine is:

Welcome to Fennel 0.10.0 on Lua 5.2!
Use ,help to see available commands.
Try installing readline via luarocks for a better repl experience.

I'm wondering if that readline part is an important difference 🤔 I'm also wondering if this is related to Olical/aniseed#53

I can't reproduce either, so maybe if I can find a similarity between the two I can spot the issue. Or at least have some sort of clue.

[thiru]

I'm having the same issue. It seems to only happen when the stdio REPL is used (which I just tried for the first time today):

vim.g["conjure#filetype#fennel"] = "conjure.client.fennel.stdio"

When run through Aniseed it seems to work for the most part. I also have readline installed @Olical but that doesn't seem to help. Here's my welcome message:

; Sponsored by @martinklepsch ❤
; --------------------------------------------------------------------------------
; "fennel" (started)
Welcome to Fennel 0.10.0 on Lua 5.4!
Use ,help to see available commands.

The reason I tried using the stdio REPL is because require wasn't working for me when attempting to load another .fnl file in the same directory. I'll open another ticket on it.

[thiru]

Ahh actually readline is the issue. I tried uninstalling it and it works great. The relative .fnl require also works for me now.

I wonder if there's an easy way to start fennel without loading readline?

[thiru]

I just asked about this on their Matrix channel and the main developer of Fennel (technomancy) about this and he said:

conjure should be setting TERM=dumb if it's running in a context that can't handle readline

[Olical]

Fantastic! Thank you for finding this! Will do.

…

On Thu, 12 Aug 2021, 18:50 Thiru, ***@***.***> wrote: I just asked about this on their Matrix channel and the main developer of Fennel (technomancy) about this and he said: conjure should be setting TERM=dumb if it's running in a context that can't handle readline — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#232 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AACM6XLX4V3BC6QJCZPBKX3T4QCVPANCNFSM5BLNEQCA> .

[Olical]

I've pushed a change to develop that hopefully addresses this in a fairly blanket way. Let me know if it looks better for you on the develop branch!

[glyh]

A lot better! Thanks a lot!

[Olical]

Excellent! Thank you!