Conflict between ufo and reactive.nvim causes weird behavior of nvim-cmp scroll_docs

[b1nhack]

Neovim version (nvim -v | head -n1)

NVIM v0.10.0

Operating system/version

macOS 14.5

How to reproduce the issue

As per the documentation, I have the following minimal configuration:

vim.o.foldcolumn = "0" -- '0' is not bad
vim.o.foldlevel = 99 -- Using ufo provider need a large value, feel free to decrease the value
vim.o.foldlevelstart = 99
vim.o.foldenable = true

require("ufo").setup({
    provider_selector = function(bufnr, filetype, buftype)
    return { "treesitter", "indent" }
    end,
})

nvim-cmp has the following keymap:

["<C-h>"] = cmp.mapping(function(fallback) 
	if cmp.visible_docs() then             
		cmp.scroll_docs(-5)                
	else                                   
		fallback()                         
	end                                    
end),                                      
["<C-b>"] = cmp.mapping(function(fallback) 
	if cmp.visible_docs() then             
		cmp.scroll_docs(5)                 
	else                                   
		fallback()                         
	end                                    
end),                                      

At this point, the scroll_docs function will cause some confusion.
vim.o.foldenable = true does not cause problems when ufo is disabled.

Expected behavior

nvim-cmp scroll_docs normal behavior.

Actual behavior

nvim-cmp scroll_docs causes confusion.

[kevinhwang91]

what's error msg?

[b1nhack]

what's error msg?

No error message, just scroll_docs causes strange behavior, cmp.scroll_docs() moves my cursor forward one character, and the autocomplete window goes back to its original state

[b1nhack]

I did more testing, and it turns out that the problem has nothing to do with vim.o.foldenable, but with ufo itself!

[kevinhwang91]

can't reproduce it.

[b1nhack]

can't reproduce it.

Okay, I'll try to make a minimal configuration. There may be other conflicts.

[b1nhack]

I'm sorry, using ufo and nvim-cmp minimal configuration did not cause the problem. It should be other plugins and ufo that caused the problem, because if I disable ufo, everything works fine.
I will continue testing.

[b1nhack]

In fact, reactive.nvim and ufo have some conflicts, which will cause this problem when they exist at the same time.
I don't know who is the source of the problem.
The minimum configuration is as follows:

-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
	vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
	vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
	{
		"rasulomaroff/reactive.nvim",

		config = function()
			require("reactive").setup({
				builtin = {
					cursorline = true,
					cursor = true,
					modemsg = true,
				},
			})
		end,
	},

	{
		"williamboman/mason.nvim",

		config = function()
			require("mason").setup()
		end,
	},
	{
		"WhoIsSethDaniel/mason-tool-installer.nvim",

		config = function()
			require("mason-tool-installer").setup({
				ensure_installed = {
					"lua-language-server",
				},
			})
		end,
	},
	{
		"neovim/nvim-lspconfig",

		config = function()
			require("lspconfig").lua_ls.setup({})
		end,
	},
	{
		"hrsh7th/nvim-cmp",
		dependencies = {
			"hrsh7th/cmp-nvim-lsp",
		},

		config = function()
			local cmp = require("cmp")

			cmp.setup({
				mapping = {
					["<C-u>"] = cmp.mapping(function(fallback)
						if cmp.visible() then
							cmp.select_prev_item()
						else
							fallback()
						end
					end),
					["<C-e>"] = cmp.mapping(function(fallback)
						if cmp.visible() then
							cmp.select_next_item()
						else
							fallback()
						end
					end),

					["<C-h>"] = cmp.mapping(function(fallback)
						if cmp.visible_docs() then
							cmp.scroll_docs(-5)
						else
							fallback()
						end
					end),
					["<C-b>"] = cmp.mapping(function(fallback)
						if cmp.visible_docs() then
							cmp.scroll_docs(5)
						else
							fallback()
						end
					end),
				},
				sources = cmp.config.sources({
					{ name = "nvim_lsp" },
				}),
			})
		end,
	},

	{
		"nvim-treesitter/nvim-treesitter",

		config = function()
			require("nvim-treesitter.configs").setup({
				ensure_installed = { "c", "lua", "vim", "vimdoc", "query" },
			})
		end,
	},
	{
		"kevinhwang91/nvim-ufo",
		dependencies = {
			"kevinhwang91/promise-async",
		},

		config = function()
			vim.o.foldcolumn = "1" -- '0' is not bad
			vim.o.foldlevel = 99 -- Using ufo provider need a large value, feel free to decrease the value
			vim.o.foldlevelstart = 99
			vim.o.foldenable = true

			-- Using ufo provider need remap `zR` and `zM`. If Neovim is 0.6.1, remap yourself
			vim.keymap.set("n", "zR", require("ufo").openAllFolds)
			vim.keymap.set("n", "zM", require("ufo").closeAllFolds)

			require("ufo").setup({
				provider_selector = function(bufnr, filetype, buftype)
					return { "treesitter", "indent" }
				end,
			})
		end,
	},
}
require("lazy").setup(plugins, {
	root = root .. "/plugins",
})

[kevinhwang91]

The issue is complex. reactive.nvim fires a redraw, and then ufo invokes vim.api.nvim_win_call to make the floating window of nvim-cmp broken.

I'm not sure if there's an issue of vim.api.nvim_set_decoration_provider used by ufo.

[b1nhack]

In which repository do you think this problem would be more simple and elegant to solve? Or should it be solved in conjunction with the reactive.nvim contributors?

[kevinhwang91]

Solved by ufo.

[b1nhack]

Ha ha ha, let me know if you need any help.