Validate only used providers?

[stevalkr]

Make sure you have done the following

• Updated to the latest version of blink.cmp
• Searched for existing issues and documentation (try <C-k> on https://cmp.saghen.dev)

Bug Description

While sources can be dynamically provided, providers are always determined and validated, regardless of whether they’re used or not. This could be improved by accepting a function or merging all the sources and validating only the ones that are actually used.

blink.cmp/lua/blink/cmp/config/sources.lua

Lines 120 to 122 in a5402a1

	for id, provider in pairs(config.providers) do
	sources.validate_provider(id, provider)
	end

Relevant configuration

sources = {
  default = { 'lsp', 'path', 'snippets', 'buffer' },
  providers = { }
},

neovim version

NVIM v0.11.0-dev-1417+g487c48ec86-Homebrew Build type: Release LuaJIT 2.1.1734355927 Run "nvim -V1 -v" for more info

blink.cmp version

1cc3b1a

[Saghen]

I don't see the use case for this. You should always provide valid configurations, and we should fail early, rather than only failing when you try to load the source. Open to your thoughts on this but closing for now

[stevalkr]

When I disable a plugin that provides a source (for example, render-markdown.nvim), the blink.cmp plugin completely breaks down. Even typing :wq would result in an error. However, I don’t want all the plugins to be loaded at all times, especially when I open Nvim to navigate terminal output, use it as a file explorer, and so on. Minimizing plugins is somehow important when working on a remote machine or container. In these cases only certain plugins are enabled, but I still want command-line and path completion to be functional.

[Saghen]

Ah it sounds like #973 would fix your case, since we'd only show that error once. Fyi with lazy.nvim, you can isolate the enabled source via dependencies. With the recommended opts_extend from the installation docs, you can extend the sources.default array via opts = { sources = { default = { 'your-source' } } }

{
  enabled = your_condition,
  'your-plugin',
  dependencies = {
    {
      'saghen/blink.cmp',
      opts = { ... }
    }
  }
}