still confussion about config on spec

[mnabila]

i'm alredy install nvim-cmp from hrsh7th and now i want loaad my nvim-cmp configuration

i have question about value from config in spec from readme.md say config = string or function

if fill like this, my configuration not loaded

    use({
        "hrsh7th/nvim-cmp",
        requires = {
            "hrsh7th/cmp-buffer",
            "hrsh7th/cmp-nvim-lsp",
            "hrsh7th/cmp-nvim-lua",
            "hrsh7th/cmp-path",
            "hrsh7th/cmp-vsnip",
            "hrsh7th/vim-vsnip",
        },
        config = function()
            require("plugins.completion").config()
        end,
    })

if I directly use it as below, the config can be loaded but before loading the config I want to check first whether the plugins are successfully loaded or not

    use({
        "hrsh7th/nvim-cmp",
        requires = {
            "hrsh7th/cmp-buffer",
            "hrsh7th/cmp-nvim-lsp",
            "hrsh7th/cmp-nvim-lua",
            "hrsh7th/cmp-path",
            "hrsh7th/cmp-vsnip",
            "hrsh7th/vim-vsnip",
        },
        config = require("plugins.completion").config(),
    })

configurtion

local M = {}

function M.config()
    local cmp = require("cmp")
    vim.g.vsnip_snippet_dir = vim.fn.stdpath("config") .. "/snippets"

    cmp.setup({
        snippet = {
            expand = function(args)
                vim.fn["vsnip#anonymous"](args.body)
            end,
        },

        completion = {
            autocomplete = { cmp.TriggerEvent.TextChanged },
        },

        documentation = {
            border = "single",
            winhighlight = "NormalFloat:CmpDocumentation,FloatBorder:CmpDocumentationBorder",
        },

        sources = {
            { name = "nvim_lsp" },
            { name = "path" },
            { name = "buffer" },
            { name = "nvim_lua" },
            { name = "vsnip" },
        },

        mapping = {
            ["<S-TAB>"] = cmp.mapping.select_prev_item(),
            ["<TAB>"] = cmp.mapping.select_next_item(),
            ["<C-Space>"] = cmp.mapping.complete(),
            ["<C-e>"] = cmp.mapping.close(),
            ["<CR>"] = cmp.mapping.confirm({
                behavior = cmp.ConfirmBehavior.Replace,
                select = true,
            }),
        },
    })
end

return M

sorry if my english is bad

[wbthomason]

Thanks for your report, and sorry for my delayed response (#562). Your first version looks correct - did you run PackerCompile or PackerSync after saving it?

[mnabila]

yes, always execute PackerSync after edit packer configurations

[wbthomason]

Hmm. To be honest, I'm not able to reproduce this on my end. Do you see the same error with the form:

use {
        "hrsh7th/nvim-cmp",
        requires = {
            "hrsh7th/cmp-buffer",
            "hrsh7th/cmp-nvim-lsp",
            "hrsh7th/cmp-nvim-lua",
            "hrsh7th/cmp-path",
            "hrsh7th/cmp-vsnip",
            "hrsh7th/vim-vsnip",
        },
        config = require("plugins.completion").config
    }

[mnabila]

error message not showing up, but configuration from this plugins not loaded and make this plugins not working

[wbthomason]

You're sure the function is not running? Can you please send the relevant portion of your compiled file?

[mnabila]

this my compiled file https://gist.github.com/mnabila/84c720b7d9152437e42fe582606403ee

but idk whats happen after try again with this config ⬇️, this config now is working on my vimrc 😕

config = function()
            require("plugins.completion").config()
        end,
and 

and
config = require("plugins.completion").config

ps: thanks for your help @wbthomason

[zanglg]

I have meet the same issue here. I think the requires has no guarantee of load "required plugins" before the declaring one.
And I force the load sequence by after keywords, not work still:

-- language server protocol and auto completion
use({ "neovim/nvim-lspconfig", ft = { "c", "cpp", "rust", "python" } })
use({ "hrsh7th/cmp-nvim-lsp", after = "neovim/nvim-lspconfig" })
use({ "L3MON4D3/LuaSnip", after = "hrsh7th/cmp-nvim-lsp" })
use({ "saadparwaiz1/cmp_luasnip", after = "L3MON4D3/LuaSnip" })
use({
    "hrsh7th/nvim-cmp",
    config = require("configs.completion").setup,
    after = "saadparwaiz1/cmp_luasnip",
})