Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

reload_git and reload_node_status stack overflow #2945

Closed
jugarpeupv opened this issue Oct 7, 2024 · 4 comments · Fixed by #2949
Closed

reload_git and reload_node_status stack overflow #2945

jugarpeupv opened this issue Oct 7, 2024 · 4 comments · Fixed by #2949
Labels
bug Something isn't working

Comments

@jugarpeupv
Copy link

Description

If i run Git! pull which is a command available through vim fugitive, then i get this error

image

Error detected while processing function 26_RunClose[23]..26_RunFinished[12]..fugitive#DidChange[9]..26_DoAutocmdChanged[17]..function 26_RunClose[23]..26_RunFinished[12]..fugitive#DidChange[9]..26_DoAutocmdChanged[10]..User Autocommands for "FugitiveChanged": Error executing lua callback: stack overflow stack traceback: ...hare/nvim/lazy/nvim-tree.lua/lua/nvim-tree/node/init.lua:145: in function 'reload_node_status' ...hare/nvim/lazy/nvim-tree.lua/lua/nvim-tree/node/init.lua:150: in function 'reload_node_status' ...hare/nvim/lazy/nvim-tree.lua/lua/nvim-tree/node/init.lua:150: in function 'reload_node_status' ...hare/nvim/lazy/nvim-tree.lua/lua/nvim-tree/node/init.lua:150: in function 'reload_node_status' ...hare/nvim/lazy/nvim-tree.lua/lua/nvim-tree/node/init.lua:150: in function 'reload_node_status' ...hare/nvim/lazy/nvim-tree.lua/lua/nvim-tree/node/init.lua:150: in function 'reload_node_status' ...hare/nvim/lazy/nvim-tree.lua/lua/nvim-tree/node/init.lua:150: in function 'reload_node_status' ...hare/nvim/lazy/nvim-tree.lua/lua/nvim-tree/node/init.lua:150: in function 'reload_node_status' ...hare/nvim/lazy/nvim-tree.lua/lua/nvim-tree/node/init.lua:150: in function 'reload_node_status' ...hare/nvim/lazy/nvim-tree.lua/lua/nvim-tree/node/init.lua:150: in function 'reload_node_status' ...hare/nvim/lazy/nvim-tree.lua/lua/nvim-tree/node/init.lua:150: in function 'reload_node_status' ... ...hare/nvim/lazy/nvim-tree.lua/lua/nvim-tree/node/init.lua:150: in function 'reload_node_status' ...hare/nvim/lazy/nvim-tree.lua/lua/nvim-tree/node/init.lua:150: in function 'reload_node_status' ...hare/nvim/lazy/nvim-tree.lua/lua/nvim-tree/node/init.lua:150: in function 'reload_node_status' ...hare/nvim/lazy/nvim-tree.lua/lua/nvim-tree/node/init.lua:150: in function 'reload_node_status' ...hare/nvim/lazy/nvim-tree.lua/lua/nvim-tree/node/init.lua:150: in function 'reload_node_status' ...hare/nvim/lazy/nvim-tree.lua/lua/nvim-tree/node/init.lua:150: in function 'reload_node_status' ...hare/nvim/lazy/nvim-tree.lua/lua/nvim-tree/node/init.lua:150: in function 'reload_node_status' ...hare/nvim/lazy/nvim-tree.lua/lua/nvim-tree/node/init.lua:150: in function 'reload_node_status' .../nvim/lazy/nvim-tree.lua/lua/nvim-tree/explorer/init.lua:371: in function 'reload_git' ...a/.local/share/nvim/lazy/nvim-tree.lua/lua/nvim-tree.lua:255: in function <...a/.local/share/nvim/lazy/nvim-tree.lua/lua/nvim-tree.lua:251>

If i point nvim-tree to commit = "517e4fbb9ef3c0986da7047f44b4b91a2400f93c" it does not break

I think the issue is related to this #2930

Neovim version

NVIM v0.10.1
Build type: Release
LuaJIT 2.1.1724512491
Run "nvim -V1 -v" for more info

Operating system and version

macOs

Windows variant

No response

nvim-tree version

main

Clean room replication

**

Steps to reproduce

Git! pull

Expected behavior

Reload properly

Actual behavior

Error

@jugarpeupv jugarpeupv added the bug Something isn't working label Oct 7, 2024
@gegoune
Copy link
Collaborator

gegoune commented Oct 7, 2024

@gegoune gegoune closed this as completed Oct 7, 2024
@alex-courtis
Copy link
Member

This was refactored yesterday via #2916

Reverted.

@alex-courtis alex-courtis reopened this Oct 8, 2024
@alex-courtis
Copy link
Member

@jugarpeupv please do raise an issue as described above, so that we may fix and proceed with the reverted change.

@alex-courtis
Copy link
Member

Replicated, occurs only when watchers disabled or Api.git.reload

alex-courtis added a commit that referenced this issue Oct 11, 2024
…atchers disabled (#2949)

* Reapply "refactor(#2871, #2886): multi instance: node classes created (#2916)"

This reverts commit 50e9194.

* fix(#2945): stack overflow on api.git.reload or fugitive event
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
3 participants