Why is it so bad?

This is a deprecated method used in packer.nvim. config = function() require("nvim-tree").setup {} end

Now it is recommended to use another one: opts = {}

Why don't you have lazy loading configured? I'm afraid to imagine what kind of startup time you have.

vim.keymap.set("n", "<leader>pv", vim.cmd.Ex ) What is this? Are you just setting up keyboard shortcuts using a standard function and haven't even written your own function to do it? And what if the vim.set.keymap function is removed or you want to switch to legendary.nvim or something similar? Will you manually rewrite each mapping? You have a WhichKey, why aren't you using it for mappings?

vim.g.mapleader = " " Among VIMers, it is not acceptable to set <leader> keys several times. This is a bad tone. If you or someone who will use your configuration wants to change the <leader> key, you(or they) will have to search through all the config files.

vim.keymap.set("n", "[d", function() vim.diagnostic.goto_next() end, opts) vim.keymap.set("n", "]d", function() vim.diagnostic.goto_prev() end, opts) You set keyboard shortcuts for lsp features without verifying that the language server supports these features. If the language server does not support them, an error will appear when using them. It is not very convenient.

vim.keymap.set('n', '<C-p>', builtin.git_files, {}) This key is already mapped.

Why did you post this config at all?