feat: use null-ls for goimports and format instead of gopls

cli/vim/neovim/lua/plugins/lsp.lua

@@ -3,8 +3,44 @@ if not lspconfig_ok then
     return
 end
 
+-- Use an on_attach function to only map the following keys
+-- after the language server attaches to the current buffer
+local on_attach = function(client, bufnr)
+    -- Enable completion triggered by <c-x><c-o>
+    vim.api.nvim_buf_set_option(bufnr, 'omnifunc', 'v:lua.vim.lsp.omnifunc')
+
+    -- Mappings.
+    -- See `:help vim.lsp.*` for documentation on any of the below functions
+    local bufopts = { noremap = true, silent = true, buffer = bufnr }
+    vim.keymap.set('n', 'gD', vim.lsp.buf.declaration, bufopts)
+    vim.keymap.set('n', 'gd', vim.lsp.buf.definition, bufopts)
+    vim.keymap.set('n', 'od', vim.lsp.buf.hover, bufopts)
+    vim.keymap.set('n', 'gi', vim.lsp.buf.implementation, bufopts)
+    vim.keymap.set('n', '<C-k>', vim.lsp.buf.signature_help, bufopts)
+    vim.keymap.set('n', '<space>wa', vim.lsp.buf.add_workspace_folder, bufopts)
+    vim.keymap.set('n', '<space>wr', vim.lsp.buf.remove_workspace_folder, bufopts)
+    vim.keymap.set('n', '<space>wl', function()
+        print(vim.inspect(vim.lsp.buf.list_workspace_folders()))
+    end, bufopts)
+    vim.keymap.set('n', '<space>D', vim.lsp.buf.type_definition, bufopts)
+    vim.keymap.set('n', '<leader>rn', vim.lsp.buf.rename, bufopts)
+    vim.keymap.set('n', '<leader>ca', vim.lsp.buf.code_action, bufopts)
+    vim.keymap.set('n', 'gr', vim.lsp.buf.references, bufopts)
+    vim.keymap.set('n', '<leader>fd', vim.lsp.buf.formatting, bufopts)
+end
+
 local language_servers = {
-    gopls = {},
+    gopls = {
+        -- capabilities = {
+        --     document_formatting = false,
+        --     document_range_formatting = false,
+        -- }
+        on_attach = function(client, bufnr)
+            on_attach(client, bufnr)
+            client.resolved_capabilities.document_formatting = false
+            client.resolved_capabilities.document_range_formatting = false
+        end
+    },
     -- grammarly = {},
     -- marksman = {},
     jsonls = {},
@@ -33,32 +69,6 @@ local language_servers = {
     }
 }
 
--- Use an on_attach function to only map the following keys
--- after the language server attaches to the current buffer
-local on_attach = function(client, bufnr)
-    -- Enable completion triggered by <c-x><c-o>
-    vim.api.nvim_buf_set_option(bufnr, 'omnifunc', 'v:lua.vim.lsp.omnifunc')
-
-    -- Mappings.
-    -- See `:help vim.lsp.*` for documentation on any of the below functions
-    local bufopts = { noremap = true, silent = true, buffer = bufnr }
-    vim.keymap.set('n', 'gD', vim.lsp.buf.declaration, bufopts)
-    vim.keymap.set('n', 'gd', vim.lsp.buf.definition, bufopts)
-    vim.keymap.set('n', 'od', vim.lsp.buf.hover, bufopts)
-    vim.keymap.set('n', 'gi', vim.lsp.buf.implementation, bufopts)
-    vim.keymap.set('n', '<C-k>', vim.lsp.buf.signature_help, bufopts)
-    vim.keymap.set('n', '<space>wa', vim.lsp.buf.add_workspace_folder, bufopts)
-    vim.keymap.set('n', '<space>wr', vim.lsp.buf.remove_workspace_folder, bufopts)
-    vim.keymap.set('n', '<space>wl', function()
-        print(vim.inspect(vim.lsp.buf.list_workspace_folders()))
-    end, bufopts)
-    vim.keymap.set('n', '<space>D', vim.lsp.buf.type_definition, bufopts)
-    vim.keymap.set('n', '<leader>rn', vim.lsp.buf.rename, bufopts)
-    vim.keymap.set('n', '<leader>ca', vim.lsp.buf.code_action, bufopts)
-    vim.keymap.set('n', 'gr', vim.lsp.buf.references, bufopts)
-    vim.keymap.set('n', '<leader>fd', vim.lsp.buf.formatting, bufopts)
-end
-
 local capabilities = vim.lsp.protocol.make_client_capabilities()
 
 
@@ -71,18 +81,16 @@ if cmp_status_ok then
 end
 
 for langsvr, settings in pairs(language_servers) do
-    settings["on_attach"] = on_attach
-    settings["capabilities"] = capabilities
+    settings = vim.tbl_deep_extend("keep", settings, {
+        on_attach = on_attach,
+        capabilities = capabilities,
+    })
     lspconfig[langsvr].setup(settings)
 end
 
-if not cmp_status_ok then
-    return
-end
-
 local luasnip_status_ok, luasnip = pcall(require, "luasnip")
-
-cmp.setup {
+if cmp_status_ok and luasnip_status_ok then
+    cmp.setup {
         snippet = {
             expand = function(args)
                 if luasnip_status_ok then
@@ -122,14 +130,13 @@ cmp.setup {
             { name = 'buffer' },
             { name = 'path' },
         },
-}
-
-local null_ls_ok, null_ls = pcall(require, "null_ls")
-if not null_ls_ok then
-    return
+    }
 end
 
-null_ls.setup({
+
+local null_ls_ok, null_ls = pcall(require, "null-ls")
+if null_ls_ok then
+    null_ls.setup({
         sources = {
             null_ls.builtins.formatting.stylua,
             null_ls.builtins.diagnostics.eslint,
@@ -137,4 +144,5 @@ null_ls.setup({
             null_ls.builtins.formatting.goimports,
             null_ls.builtins.formatting.gofumpt,
         },
-})
+    })
+end