migrate: from efm to null-ls ( #500 #504 )

lua/modules/configs/completion/formatting.lua

@@ -103,8 +103,8 @@ function M.format_filter(clients)
 		local status_ok, formatting_supported = pcall(function()
 			return client.supports_method("textDocument/formatting")
 		end)
-		if status_ok and formatting_supported and client.name == "efm" then
-			return "efm"
+		if status_ok and formatting_supported and client.name == "null-ls" then
+			return "null-ls"
 		elseif client.name ~= "lua_ls" and client.name ~= "tsserver" and client.name ~= "clangd" then
 			return status_ok and formatting_supported and client.name
 		end
@@ -140,7 +140,7 @@ function M.format(opts)
 
 	if #clients == 0 then
 		vim.notify(
-			"[LSP] Format request failed, no matching language servers.",
+			"[LSP]Format request failed, no matching language servers.",
 			vim.log.levels.WARN,
 			{ title = "Formatting Failed!" }
 		)
@@ -165,7 +165,7 @@ function M.format(opts)
 		if result and result.result then
 			vim.lsp.util.apply_text_edits(result.result, bufnr, client.offset_encoding)
 			vim.notify(
-				string.format("Format successfully with %s!", client.name),
+				string.format("[LSP]Format successfully with [%s]!", client.name),
 				vim.log.levels.INFO,
 				{ title = "LSP Format Success!" }
 			)

lua/modules/configs/completion/lsp.lua

@@ -1,6 +1,4 @@
 return function()
-	local formatting = require("completion.formatting")
-
 	local nvim_lsp = require("lspconfig")
 	local mason = require("mason")
 	local mason_lspconfig = require("mason-lspconfig")
@@ -33,13 +31,15 @@ return function()
 		},
 	})
 	mason_lspconfig.setup({
+		-- NOTE: use the lsp names in nvim-lspconfig
+		-- https://github.com/williamboman/mason-lspconfig.nvim/blob/main/lua/mason-lspconfig/mappings/server.lua
 		ensure_installed = {
 			"bashls",
 			"clangd",
-			"efm",
 			"gopls",
-			"pyright",
+			"html",
 			"lua_ls",
+			"pyright",
 		},
 	})
 
@@ -85,16 +85,18 @@ return function()
 			nvim_lsp.clangd.setup(final_opts)
 		end,
 
-		efm = function()
-			-- Do not setup efm
-		end,
-
 		gopls = function()
 			local _opts = require("completion.servers.gopls")
 			local final_opts = vim.tbl_deep_extend("keep", _opts, opts)
 			nvim_lsp.gopls.setup(final_opts)
 		end,
 
+		html = function()
+			local _opts = require("completion.servers.html")
+			local final_opts = vim.tbl_deep_extend("keep", _opts, opts)
+			nvim_lsp.html.setup(final_opts)
+		end,
+
 		jsonls = function()
 			local _opts = require("completion.servers.jsonls")
 			local final_opts = vim.tbl_deep_extend("keep", _opts, opts)
@@ -107,70 +109,4 @@ return function()
 			nvim_lsp.lua_ls.setup(final_opts)
 		end,
 	})
-
-	if vim.fn.executable("html-languageserver") then
-		local _opts = require("completion.servers.html")
-		local final_opts = vim.tbl_deep_extend("keep", _opts, opts)
-		nvim_lsp.html.setup(final_opts)
-	end
-
-	local efmls = require("efmls-configs")
-
-	-- Init `efm-langserver` here.
-
-	efmls.init({
-		on_attach = opts.on_attach,
-		capabilities = capabilities,
-		init_options = { documentFormatting = true, codeAction = true },
-	})
-
-	-- Require `efmls-configs-nvim`'s config here
-
-	local vint = require("efmls-configs.linters.vint")
-	local eslint = require("efmls-configs.linters.eslint")
-	local flake8 = require("efmls-configs.linters.flake8")
-
-	local black = require("efmls-configs.formatters.black")
-	local stylua = require("efmls-configs.formatters.stylua")
-	local prettier = require("efmls-configs.formatters.prettier")
-	local shfmt = require("efmls-configs.formatters.shfmt")
-
-	-- Add your own config for formatter and linter here
-
-	-- local rustfmt = require("completion.efm.formatters.rustfmt")
-	local clangfmt = require("completion.efm.formatters.clangfmt")
-
-	-- Override default config here
-
-	flake8 = vim.tbl_extend("force", flake8, {
-		prefix = "flake8: max-line-length=160, ignore F403 and F405",
-		lintStdin = true,
-		lintIgnoreExitCode = true,
-		lintFormats = { "%f:%l:%c: %t%n%n%n %m" },
-		lintCommand = "flake8 --max-line-length 160 --extend-ignore F403,F405 --format '%(path)s:%(row)d:%(col)d: %(code)s %(code)s %(text)s' --stdin-display-name ${INPUT} -",
-	})
-
-	-- Setup formatter and linter for efmls here
-
-	efmls.setup({
-		vim = { formatter = vint },
-		lua = { formatter = stylua },
-		c = { formatter = clangfmt },
-		cpp = { formatter = clangfmt },
-		python = { formatter = black },
-		vue = { formatter = prettier },
-		typescript = { formatter = prettier, linter = eslint },
-		javascript = { formatter = prettier, linter = eslint },
-		typescriptreact = { formatter = prettier, linter = eslint },
-		javascriptreact = { formatter = prettier, linter = eslint },
-		yaml = { formatter = prettier },
-		html = { formatter = prettier },
-		css = { formatter = prettier },
-		scss = { formatter = prettier },
-		sh = { formatter = shfmt },
-		markdown = { formatter = prettier },
-		-- rust = {formatter = rustfmt},
-	})
-
-	formatting.configure_format_on_save()
 end

lua/modules/configs/completion/null-ls.lua

@@ -0,0 +1,38 @@
+return function()
+	local formatting = require("completion.formatting")
+
+	local null_ls = require("null-ls")
+	local btn = null_ls.builtins -- https://github.com/jose-elias-alvarez/null-ls.nvim/tree/main/lua/null-ls/builtins
+
+	local sources = {
+		-- formatting
+		btn.formatting.black.with({ extra_args = { "--fast" } }),
+		btn.formatting.clang_format,
+		btn.formatting.eslint_d,
+		btn.formatting.jq,
+		btn.formatting.markdownlint,
+		btn.formatting.prettierd,
+		btn.formatting.rustfmt,
+		btn.formatting.shfmt,
+		btn.formatting.stylua,
+
+		-- diagnostics
+		btn.diagnostics.shellcheck.with({ diagnostics_format = "#{m} [#{c}]" }),
+		btn.diagnostics.markdownlint.with({ extra_args = { "--disable MD033" } }),
+	}
+
+	null_ls.setup({
+		debug = false,
+		update_in_insert = false,
+		diagnostics_format = "[#{c}] #{m} (#{s})",
+		sources = sources,
+	})
+
+	require("mason-null-ls").setup({
+		ensure_installed = nil,
+		automatic_installation = true,
+		automatic_setup = false,
+	})
+
+	formatting.configure_format_on_save()
+end

lua/modules/plugins/completion.lua

@@ -5,18 +5,21 @@ completion["neovim/nvim-lspconfig"] = {
 	event = { "BufReadPost", "BufAdd", "BufNewFile" },
 	config = require("completion.lsp"),
 	dependencies = {
-		{ "creativenull/efmls-configs-nvim" },
+		{ "ray-x/lsp_signature.nvim" },
 		{ "williamboman/mason.nvim" },
 		{ "williamboman/mason-lspconfig.nvim" },
-		{
-			"WhoIsSethDaniel/mason-tool-installer.nvim",
-			config = require("completion.mason-tool-installer"),
-		},
 		{
 			"glepnir/lspsaga.nvim",
 			config = require("completion.lspsaga"),
 		},
-		{ "ray-x/lsp_signature.nvim" },
+		{
+			"jose-elias-alvarez/null-ls.nvim",
+			dependencies = {
+				"nvim-lua/plenary.nvim",
+				"jay-babu/mason-null-ls.nvim",
+			},
+			config = require("completion.null-ls"),
+		},
 	},
 }
 completion["hrsh7th/nvim-cmp"] = {

snips/package.json

@@ -6,7 +6,13 @@
   "contributes": {
     "snippets": [
       {
-        "language": ["plaintext", "markdown", "tex", "html", "global"],
+        "language": [
+          "plaintext",
+          "markdown",
+          "tex",
+          "html",
+          "global"
+        ],
         "path": "./snippets/global.json"
       },
       {
@@ -18,11 +24,17 @@
         "path": "./snippets/cpp.json"
       },
       {
-        "language": ["csharp", "cs"],
+        "language": [
+          "csharp",
+          "cs"
+        ],
         "path": "./snippets/csharp.json"
       },
       {
-        "language": ["gitcommit", "NeogitCommitMessage"],
+        "language": [
+          "gitcommit",
+          "NeogitCommitMessage"
+        ],
         "path": "./snippets/gitcommit.json"
       },
       {
@@ -106,35 +118,63 @@
         "path": "./snippets/sql.json"
       },
       {
-        "language": ["shellscript", "shell", "sh"],
+        "language": [
+          "shellscript",
+          "shell",
+          "sh"
+        ],
         "path": "./snippets/shell.json"
       },
       {
-        "language": ["markdown", "rmd"],
+        "language": [
+          "markdown",
+          "rmd"
+        ],
         "path": "./snippets/markdown.json"
       },
       {
-        "language": ["plaintex", "tex"],
+        "language": [
+          "plaintex",
+          "tex"
+        ],
         "path": "./snippets/latex.json"
       },
       {
         "language": "java",
         "path": "./snippets/java.json"
       },
       {
-        "language": ["html", "jade", "pug", "eruby"],
+        "language": [
+          "html",
+          "jade",
+          "pug",
+          "eruby"
+        ],
         "path": "./snippets/html.json"
       },
       {
-        "language": ["css", "scss", "sass", "less", "stylus"],
+        "language": [
+          "css",
+          "scss",
+          "sass",
+          "less",
+          "stylus"
+        ],
         "path": "./snippets/css.json"
       },
       {
-        "language": ["javascript", "javascriptreact", "vue"],
+        "language": [
+          "javascript",
+          "javascriptreact",
+          "vue"
+        ],
         "path": "./snippets/javascript/javascript.json"
       },
       {
-        "language": ["typescript", "typescriptreact"],
+        "language": [
+          "typescript",
+          "typescriptreact"
+        ],
         "path": "./snippets/javascript/typescript.json"
       },
       {
@@ -230,19 +270,28 @@
         "path": "./snippets/kotlin.json"
       },
       {
-        "language": ["plaintex", "tex"],
+        "language": [
+          "plaintex",
+          "tex"
+        ],
         "path": "./snippets/latex/latex-snippets.json"
       },
       {
-        "language": ["plaintex", "tex"],
+        "language": [
+          "plaintex",
+          "tex"
+        ],
         "path": "./snippets/latex/vscode-latex-snippets.json"
       },
       {
         "language": "twig",
         "path": "./snippets/frameworks/twig.json"
       },
       {
-        "language": ["r", "rmd"],
+        "language": [
+          "r",
+          "rmd"
+        ],
         "path": "./snippets/r.json"
       },
       {