Downloading Server causes Neovim to Stuck #21

Activity

1. joaopluigi commented on Aug 6, 2025

   

   joaopluigi

   on Aug 6, 2025

   ContributorAuthor

   More actions

   The piece of code to change:

   eca-nvim/lua/eca/init.lua

   Line 270 in 60c6afa

   M.server:start()

   I'm unsure if other server start occurrences, such as EcaStartServer command, should also have the async behavior.

   I'm also unsure about using vim.defer_fn there, not clear what are the benefits.
2. joaopluigi

   added

   good first issueGood for newcomersGood for newcomers

   on Aug 6, 2025
3. tomgeorge commented on Aug 6, 2025

   

   tomgeorge

   on Aug 6, 2025

   Member

   More actions

   I'm a fan of using vim.system to manage the server process. It is available since vim > 0.10 but you can easily add a compatibility layer to support older neovims.

   --- copy system.lua from neovim runtime
   vim.system = vim.system or require("eca.system")
   
   vim.system("eca server", { 
     stdout = function(data, err)
       print(data)
     end,
   stderr = function(data, err)
       print(data)
     end
   }, some_on_exit_function())
4. joaopluigi commented on Aug 7, 2025

   

   joaopluigi

   on Aug 7, 2025

   ContributorAuthor

   More actions

   @tomgeorge, I don't know vim.system, so I'll take a look, thanks for sharing.

   My initial idea was to do a minimal change like:

   Changing this:

     -- Start server automatically in background
     vim.defer_fn(function()
       M.server:start()
     end, 100) -- Small delay to ensure everything is loaded

   To something like this

   -- Start server automatically in background
   local ok, async = pcall(require, "plenary.async")
   if ok then
     async.run(function()
       M.server:start()
     end)
   else
     vim.defer_fn(function()
       M.server:start()
     end, 100) -- Small delay to ensure everything is loaded
   end
   
5. joaopluigi

   added

   enhancementNew feature or requestNew feature or request

   on Aug 29, 2025
6. joaopluigi commented on Oct 8, 2025

   

   joaopluigi

   on Oct 8, 2025

   ContributorAuthor

   More actions

   Solved by #48
7. joaopluigi

   closed this as completedon Oct 8, 2025