[mini.pick] Mapping delete not working on buffers picker

[Kogaraashi1]

Contributing guidelines

• I have read CODE_OF_CONDUCT.md

Module(s)

mini.pick

Question

• Neovim version: nvim --version

• Configuración mínima: [return {

  pickers = {

    files = {
  
    	find = "fd . --type f --hidden --exclude .git", -- 🛠️ Usa el comando fd para buscar archivos
  
    	-- - --type f: solo archivos
  
    	-- - --hidden: incluye archivos ocultos
  
    	-- - --exclude .git: ignora la carpeta .git
    },
  
    buffers = {
  
    	source = {
  
    		items = vim.api.nvim_list_bufs, -- 📜 Obtiene TODOS los buffers (incluyendo vacíos)
  
    		show = function(buf_id) -- 🎨 Formato de visualización
    			local name = vim.api.nvim_buf_get_name(buf_id)
  
    			return name == "" and "[Empty]" or vim.fn.fnamemodify(name, ":t") -- 🏷️ Muestra "[Empty]" o el nombre corto del archivo
    		end,
    	},
  
    	preview = function(buf_id) -- 🔍 Previsualización del contenido
    		return vim.api.nvim_buf_get_lines(buf_id, 0, -1, false) -- 📃 Muestra todas las líneas del buffer
    	end,
    },
  

  },
  }
  ]

• Comportamiento esperado: <C-d> elimina buffer y actualiza lista

• Comportamiento actual:
  [se desliza un movimiento hacia arriba en la lista de buffers en lugar de eliminar el buffer]

atajo utilizado [
vim.keymap.set("n", "b", function()
require("mini.pick").builtin.buffers()
end, { desc = "List buffers" })

]

-en mi github tambien esta el codigo por si hay que tener alguna otra consideracion,bendecida noche y muchas gracias

[krovuxdev]

@Kogaraashi1 The issue occurs because when trying to delete a buffer with <C-d>, the list of buffers isn't updated as expected. This happens due to how mini.pick is configured. According to the documentation for mini.pick, you can register a custom function to handle buffers, as explained in registry.

Additionally, as mentioned in buffers, it's possible to map keys like <C-d> to perform specific actions on buffers, such as deleting them.

Thus, the code should be modified as follows to achieve the expected behavior:

Code (spoiler)

local pick = require("mini.pick")
pick.registry.buffers = function(local_opts, opts)
    local_opts = local_opts or {}

    -- Delete the current buffer
    local wipeout_cur = function()
        vim.api.nvim_buf_delete(MiniPick.get_picker_matches().current.bufnr, {})
        MiniPick.builtin.buffers(local_opts, opts)
    end

    -- Map <C-d> to delete the buffer
    local buffer_mappings = { wipeout = { char = "<C-d>", func = wipeout_cur } }

    -- Show buffers with short names
    local show = function(buf_id, items, query)
        vim.tbl_map(function(i) i.text = vim.fn.fnamemodify(i.text, ":t") end, items)
        MiniPick.default_show(buf_id, items, query, { show_icons = true })
    end

    -- Merge options
    opts = vim.tbl_deep_extend("force", {
        source = { show = show },
        mappings = buffer_mappings,
    }, opts or {})

    return MiniPick.builtin.buffers(local_opts, opts)
end

With this modification, the buffer will be deleted correctly when <C-d> is pressed, and the buffer list will be updated, showing only the short file names.

[echasnovski]

Yes, <C-d> is not an "out of the box" key to delete a buffer and requires custom configuration. This is mentioned in the help.

Also, @Kogaraashi1, please use English to communicate here (auto-translation should be fine). This will help both people who try to answer questions and who will try to search information here in the future.

---

According to the documentation for mini.pick, you can register a custom function to handle buffers, as explained in registry.

After reading it second time, this seems correct, but a bit confusing. A "custom function to handle buffers" is meant here as "custom function that starts a buffer picker", right? I initially thought that "handle buffers" meant as "delete the buffer".

[krovuxdev]

After reading it second time, this seems correct, but a bit confusing. A "custom function to handle buffers" is meant here as "custom function that starts a buffer picker", right? I initially thought that "handle buffers" meant as "delete the buffer".

Ah I see, thanks for the correction! 😄

[krovuxdev]

... please use English to communicate here (auto-translation should be fine). This will help both people who try to answer questions and who will try to search information here in the future.

Oh I didn't realize, in my mind I read everything in English and just replied without thinking. Spanish is my native language 😅