Enhance filters.exclude For More Granularity

[sarmong]

Description

I have the following line

  filters = {
    dotfiles = true,
    exclude = { ".config", ".local" },
  },

And now gitignore files are always shown, and using I to toggle them doesn't work.

If I remove the line with exclude, all works fine.

Neovim version

NVIM v0.6.1
Build type: Release
LuaJIT 2.1.0-beta3
Compiled by builduser

Operating system and version

Arch Linux , kernel - 5.17.1-arch1-1

nvim-tree version

commit 618ea25

Steps to reproduce

1. Create a folder with .gitignore and a couple of files
2. Gitignore file1
3. nvim -nu /tmp/nvt-min.lua
4. :NvimTreeToggle
5. Observe the gitignored file being show
6. Press I - gitignored file still shown

Expected behavior

1. Gitignored files are not shown by default
2. Gitignored files are toggled with I

Actual behavior

No response

Minimal config

vim.cmd([[set runtimepath=$VIMRUNTIME]])
vim.cmd([[set packpath=/tmp/nvt-min/site]])
local package_root = "/tmp/nvt-min/site/pack"
local install_path = package_root .. "/packer/start/packer.nvim"
local function load_plugins()
	require("packer").startup({
		{
			"wbthomason/packer.nvim",
			"kyazdani42/nvim-tree.lua",
			"kyazdani42/nvim-web-devicons",
		},
		config = {
			package_root = package_root,
			compile_path = install_path .. "/plugin/packer_compiled.lua",
			display = { non_interactive = true },
		},
	})
end
if vim.fn.isdirectory(install_path) == 0 then
	print("Installing nvim-tree and dependencies.")
	vim.fn.system({ "git", "clone", "--depth=1", "https://github.com/wbthomason/packer.nvim", install_path })
end
load_plugins()
require("packer").sync()
vim.cmd([[autocmd User PackerComplete ++once echo "Ready!" | lua setup()]])
vim.opt.termguicolors = true
vim.opt.cursorline = true

-- MODIFY NVIM-TREE SETTINGS THAT ARE _NECESSARY_ FOR REPRODUCING THE ISSUE
_G.setup = function()
	require("nvim-tree").setup({
		filters = {
			dotfiles = true,
			exclude = { ".config", ".local" },
		},
	})
end

[sarmong]

P.S.
This used to work on commit 6eebc10

[kyazdani42]

works just fine on my end. Would you mind changing .config and .local with .config$ and .local$ ?

[kyazdani42]

maybe this was broken for you in 00fd8ae ?

[alex-courtis]

maybe this was broken for you in 00fd8ae ?

I probably should have added filters.custom_regex instead of filters.custom.

We could do that now... @kyazdani42 ?

Edit: that regex change only affects filters.custom, not filters.exclude.

Cannot replicate at 618ea25: file1 is hidden until I

[sarmong]

I've looked deeper and here's where the problem is.
If you add do echo "folder/.config/file3" >>.gitignore and create this directories and file, the file will never be toggled.

I might be a bit confused by how exclude works. I though tit should only affect the toggle_dotfiles action, but it also affects the toggle_git_ignored aciton.

I expected the gitignored files inside the excluded directory to still be togglable with I

[alex-courtis]

I see. I can replicate with config/file3 and .config/file3

filters.exclude overrides each of the filtering mechanisms: I, U, H: git.ignore, filters.custom and filters.dotfiles.

That is working as intended:

  - |filters.exclude|: list of directories or files to exclude from filtering
    (will always be shown)
      type: `{string}`
      default: `{}`

There is ambiguity in that it does not explicitly state that git.ignore is excluded. I think we should clarify the documentation, along with adding the default key bindings/action names for each filter option.

[alex-courtis]

Raised #1168

A quick review would be great @sarmong

[sarmong]

@alex-courtis . thanks, the explanation makes it clearer.

However, I think it might be worth it to change the behaviour to account for the situation I have.

I can implement it in the future, so maybe we can keep the issue open