Correct way to add configs

[parthshahp]

What is the suggested way to add personal configs to plugins that already exist from NvChad?

/* plugins/init.lua */
  {
    "hrsh7th/nvim-cmp",
    config = function()
      require "configs.cmp"
    end,
  },
  {
    "nvim-tree/nvim-tree.lua",
    config = function()
      local opts = { view = { side = "right" } }
      require("nvim-tree").setup(opts)
    end,
  },

/* configs/cmp.lua */
local cmp = require "cmp"

cmp.setup {
  sources = {
    { name = "nvim_lsp" },
  },
  mapping = cmp.mapping.preset.insert {
    ["<C-k>"] = cmp.mapping.select_prev_item(),
    ["<C-j>"] = cmp.mapping.select_next_item(),
    ["<CR>"] = cmp.mapping.confirm { select = true },
    ["<C-Space>"] = cmp.mapping.complete(),
  },
}

Here is a sample of my config files. Is this correct? Am I accidentally overwriting the entire table from NvChad?

[siduck]

@parthshahp look at the plugin examples here https://nvchad.com/docs/config/plugins/

so for small changes or changes which dont have any function call , use the opts as table, else use opts as a function ( for cmp's case )

or you could do within the config function itself

     { 
           "blabla cmp", 
            config = function(_, opts)
                     opts.blablabla -- change the opts table by LUA!!
                     
                    require("cmp").setup(opts)
              end
       }

[parthshahp]

I ended up trying to implement it based on what you said, but I can't seem to get the syntax right. Do you notice anything incorrect here? I tried to base it off of the telescope example in that link.

  {
    "hrsh7th/nvim-cmp",
    opts = function()
      local conf = require "cmp"
      conf.mapping.preset.insert {
        ["<C-j>"] = conf.mapping.select_next_item(),
        ["<C-k>"] = conf.mapping.select_prev_item(),
        ["<CR>"] = conf.mapping.confirm { select = true },
        ["<C-Space>"] = conf.mapping.complete(),
      }
      return conf
    end,
  },

[siduck]

idk about preset.insert, i'd just modify the mapping table by adding new key/values to it

  conf.mapping["<C-j>"] = conf.mapping.select_next_item()
  conf.mapping["<C-Space>"] = conf.mapping.complete()

[parthshahp]

I followed the instructions and did it this way:

  {
    "hrsh7th/nvim-cmp",
    config = function(_, opts)
      local cmp = require "cmp"
      opts.sources = {
        { name = "nvim_lsp" },
      }
      opts.mapping = cmp.mapping.preset.insert {
        ["<CR>"] = cmp.mapping.confirm { select = true },
        ["<C-Space>"] = cmp.mapping.complete(),
      }
      cmp.setup(opts)
    end,
  },

However, I'm still getting this in checkhealth:

lazy: require("lazy.health").check()

lazy.nvim ~
- OK Git installed
- OK no existing packages found by other package managers
- OK packer_compiled.lua not found
- WARNING {nvim-cmp}: overriding <config>
- WARNING {nvim-tree.lua}: overriding <config>
- WARNING {conform.nvim}: overriding <config>
- WARNING {nvim-lspconfig}: overriding <config>

Is this a problem? I don't want to be accidentally overriding the config from NvChad.

[siduck]

@parthshahp > I don't want to be accidentally overriding the config from NvChad.

You are doing that, but overriding doesnt mean erasing all of default nvchad config lol! you are overriding some parts of nvchad's default cmp which is ok!