Calling 'vim.diagnostic.goto_next()' inactivates lightline about lightline.vim

[crowdigit]

I figured out it's not related to goto_next, it's caused by opening floating window.

It has been discussed already, and there's a plugin implementing it using Neovim's laststatus feature, but it's limited to Neovim.

[itchyny]

Are you using the latest version of lightline? I recently accepted a Neovim related patch so make sure to update. If it is the latest, please create a precise steps to reproduce the issue. Just lua vim.diagnostic.goto_next() warns No more valid diagnostics to move to.

[crowdigit]

Thanks for checking out,

I'm using latest commit

commit f11645cc6d37871244f2ea0ae972d3f6af9c8f73 (HEAD -> master, origin/master, origin/HEAD)
Merge: b1e91b4 3d10b50
Author: itchyny <[email protected]>
Date:   Sun Sep 3 19:09:07 2023 +0900

    Merge pull request #659 from 4513ECHO/fix/neovim-floatwin
    
    fix: Make not to skip floatwin if it is focusable

I have checked, opening floating window actually does not inactivates lightline, I used below command to check

:call nvim_open_win(0, v:false, { "relative": "editor", "width": 40, "height": 20, "col": 0, "row": 20})

I think goto_next does some more work than just opening window.

To reproduce, we need working LSP environment which includes,

1. Neovim
2. Neovim LSP config
3. Dev environment and an language server

• I used Typescript tsserver for the example, so npm and Typescript compiler is required.

  # assume npm is already installed and $PATH is configured
  npm install --global typescript
  npm install --global typescript-language-server

• Link to tsserver configuration doc for nvim-lspconfig

5. Assume now you have working LSP config, make a directory and init Typescript project

   npm init -y
   npx tsc --init

6. Type some erroneous code, below code should suffice

   function foo() {
     throw new Error("oh no");
     console.log("bar")
   }

7. Invoke goto_next

I attached my minimal Typescript project for case if you have trouble creating project.

ts.tar.gz

[itchyny]

@4513ECHO Would you help us fixing this?

[4513ECHO]

Please try this workaround. Maybe this is bug of Neovim itself...

#659 (comment)

[itchyny]

Ah, okay, thanks. Is there any bug report to Neovim?

[crowdigit]

@4513ECHO Yup I tried the patch and it works well now.

It looks like it's default behavior, (i.e. it's not a bug) so I think it won't be changed.

@itchyny We may conclude this issue invalid and close, but new users would face this issue later on so documenting it would be helpful. If you mind cluttered README file with troubleshooting methods, Github Wiki might come in handy.