fix(lsp): disable php formatting workaround

README.md

@@ -63,7 +63,11 @@ when needed.
 
 ### Language servers
 Language servers are installed automatically to the nvim data directory
-(`:echo stdpath('data') .. '/mason'`). The following are some noted requirements
+(`:echo stdpath('data') .. '/mason'`) upon entering a buffer of related
+filetype. Automatic installation can be turned off, see the end of this section
+for instructions.
+
+The following are some noted requirements
 for the installations themselves:
 
 - **diagnostic-languageserver**: npm
@@ -78,9 +82,11 @@ Some servers have additional runtime dependencies:
 
 - **bash-language-server**: shellcheck (optional, used for linting)
 
-If you don't need some specific language server, and want to get rid of any
+If you don't need some specific language server, you may either remove them from
-warning messages, you may either remove them from the top of `lua/lsp.lua`
+the top of `lua/lsp.lua` or disable them in `lua/lsp/<server>.lua`.
-or disable them in `lua/lsp/<server>.lua`.
+
+To disable automatic installation of a selected language server, remove or
+comment out the mason part of the configuration at `lua/lsp/<server>.lua`.
 
 ### Nerd Font
 It's recommended to use a [Nerd Font](https://www.nerdfonts.com/),

lua/lsp.lua

@@ -1,14 +1,14 @@
-local package_name = "lsp"
+local module_name = "lsp"
 local utils = require("utils")
 
-local P = {}
+local M = {}
 
-P._filetypes = nil
+local _filetypes = nil
-P._language_servers = nil
+-- local auto_installed_servers = nil
 
-P.capabilities = {}
+local capabilities = {}
 
-P.config = {
+local config = {
     bashls = {},
     clangd = {},
     cmake = {},
@@ -23,9 +23,9 @@ P.config = {
     gopls = {},
 }
 
-for server, _ in pairs(P.config) do
+for server, _ in pairs(config) do
-    utils.try_require("lsp." .. server, package_name, function (mod)
+    utils.try_require("lsp." .. server, module_name, function (mod)
-        P.config[server] = mod
+        config[server] = mod
     end)
 end
 
@@ -43,7 +43,7 @@ local function ca_rename()
     end
 end
 
-function P._setup_diagnostics()
+local function setup_diagnostics()
     -- https://github.com/neovim/nvim-lspconfig/wiki/UI-Customization#customizing-how-diagnostics-are-displayed
     vim.diagnostic.config({
         underline = true,
@@ -73,7 +73,7 @@ function P._setup_diagnostics()
     end
 end
 
-function P.on_attach(client, bufnr)
+local function on_attach(client, bufnr)
     -- Mappings.
     -- See `:help vim.lsp.*` for documentation on any of the below functions
     local opts = { buffer = bufnr, }
@@ -93,27 +93,7 @@ function P.on_attach(client, bufnr)
     vim.keymap.set(
         { "n", "x", },
         "<leader>lf",
-        function ()
+        vim.lsp.buf.format,
-            if vim.bo.filetype ~= "php" then
-                return vim.lsp.buf.format()
-            end
-
-            local dls = require("lsp.diagnosticls")
-            local formatters = dls.lspconfig.init_options.formatFiletypes.php
-            for _, fmt in ipairs(formatters) do
-                if fmt == "php_cs_fixer" then
-                    ---@type table
-                    local winview = vim.fn.winsaveview()
-                    vim.cmd.write({ bang = true, })
-                    vim.lsp.buf.format()
-                    vim.cmd.write({ bang = true, })
-                    vim.fn.winrestview(winview)
-                    return
-                end
-            end
-
-            return vim.lsp.buf.format()
-        end,
         opts
     )
 
@@ -146,20 +126,20 @@ function P.on_attach(client, bufnr)
 
     require("lsp-inlayhints").on_attach(client, bufnr, false)
 
-    vim.lsp.handlers['textDocument/hover'] = vim.lsp.with(
+    vim.lsp.handlers["textDocument/hover"] = vim.lsp.with(
         vim.lsp.handlers.hover, {
-            border = "single"
+            border = "single",
         }
     )
-    vim.lsp.handlers['textDocument/signatureHelp'] = vim.lsp.with(
+    vim.lsp.handlers["textDocument/signatureHelp"] = vim.lsp.with(
         vim.lsp.handlers.signature_help, {
-            border = "single"
+            border = "single",
         }
     )
 end
 
-function P.reload_server_buf(name)
+local function reload_server_buf(name)
-    local server = P.config[name]
+    local server = config[name]
     local ft_map = {}
     for _, ft in ipairs(server.lspconfig.filetypes) do
         ft_map[ft] = true
@@ -180,101 +160,15 @@ function P.reload_server_buf(name)
     end
 end
 
-function P.filetypes()
-    if not P._filetypes then
-        P._filetypes = {}
-        local unique = {}
-        for _, server in pairs(P.config) do
-            for _, ft in ipairs(server.lspconfig.filetypes) do
-                if not unique[ft] then
-                    table.insert(P._filetypes, ft)
-                    unique[ft] = true
-                end
-            end
-        end
-    end
 
-    return P._filetypes
+local function configure_server(name, server)
-end
+    local ok, ret = pcall(require, "lspconfig")
-
-function P.language_servers()
-    if not P._language_servers then
-        P._language_servers = {}
-        for server, opts in pairs(P.config) do
-            if opts.enabled ~= true then
-                goto next_server
-            end
-            if opts.dependencies ~= nil then
-                local not_installed = {}
-                for _, dep in ipairs(opts.dependencies) do
-                    if not utils.is_installed(dep) then
-                        table.insert(not_installed, dep)
-                    end
-                end
-
-                if #not_installed > 0 then
-                    utils.warn(
-                        ("Disabling %s "
-                            .. "because the following required package(s) "
-                            .. "are not installed: %s")
-                        :format(
-                            server,
-                            table.concat(not_installed, ", ")
-                        ),
-                        package_name
-                    )
-                    opts.enabled = false
-                    goto next_server
-                end
-            end
-
-            if opts.py_module_deps ~= nil then
-                local not_installed = {}
-                for _, mod in ipairs(opts.py_module_deps) do
-                    if not utils.python3_module_is_installed(mod) then
-                        table.insert(not_installed, mod)
-                    end
-                end
-
-                if #not_installed > 0 then
-                    utils.warn(
-                        ("Disabling %s "
-                            + "because the following required python3 "
-                            + "module(s) are not installed: %s")
-                        :format(
-                            server,
-                            table.concat(not_installed, ", ")
-                        ),
-                        package_name
-                    )
-                    opts.enabled = false
-                    goto next_server
-                end
-            end
-
-            table.insert(P._language_servers, server)
-
-            ::next_server::
-        end
-    end
-
-    return P._language_servers
-end
-
-function P.setup_server(name)
-    local server = P.config[name]
-
-    if not server or server.enabled ~= true then
-        return
-    end
-
-    local ok, lspconfig = pcall(require, "lspconfig")
     if not ok then
-        utils.err("Missing required plugin lspconfig", package_name)
+        utils.err("Missing required plugin lspconfig", module_name)
         return
     end
+    local lspconfig = ret
 
-    -- server.lspconfig.root_dir = function () return vim.fn.getcwd() end
     if server.root_pattern then
         server.lspconfig.root_dir = lspconfig.util.root_pattern(
             unpack(server.root_pattern)
@@ -282,39 +176,113 @@ function P.setup_server(name)
     else
         server.lspconfig.root_dir = lspconfig.util.find_git_ancestor
     end
-    server.lspconfig.capabilities = P.capabilities
+    server.lspconfig.capabilities = capabilities
     server.lspconfig.on_attach = function (...)
-        local resp
+        ok, ret = pcall(on_attach, ...)
-        ok, resp = pcall(P.on_attach, ...)
         if not ok then
             utils.err(
-                ("Failed to load on_attach for %s:\n%s"):format(name, resp)
+                ("Failed to load on_attach for %s:\n%s"):format(name, ret),
+                module_name
             )
         end
     end
 
-    if not pcall(lspconfig[name].setup, server.lspconfig) then
+    ok, ret = pcall(lspconfig[name].setup, server.lspconfig)
-        utils.err("Unknown LSP server for lspconfig: " .. name, package_name)
+    if not ok then
+        utils.err(
+            ("Failed to setup LSP server %s with lspconfig: %s"):format(
+                name,
+                ret
+            ),
+            module_name
+        )
         return
     end
 
-    P.reload_server_buf(name)
+    reload_server_buf(name)
 end
 
-function P.setup()
+local function setup_server(name, server)
-    P._setup_diagnostics()
+    local registry = require("mason-registry")
+    local pkg_name
 
-    utils.try_require("cmp_nvim_lsp", package_name, function (mod)
+    if server.mason then
-        P.capabilities = mod.default_capabilities()
+        pkg_name = server.mason.name
-    end)
+    end
 
-    utils.try_require("mason-lspconfig", package_name, function (mod)
+    if (pkg_name and not registry.is_installed(pkg_name)) then
-        mod.setup_handlers({
+        local pkg = registry.get_package(pkg_name)
-            function (name)
+        local handle = pkg:install({ version = server.mason.version, })
-                P.setup_server(name)
+        utils.info("Installing " .. pkg_name)
-            end,
+        local err
+        handle:on("stderr", vim.schedule_wrap(function (msg)
+            err = (err or "") .. msg
+        end))
+        handle:once("closed", vim.schedule_wrap(function ()
+            if err then
+                utils.err(err, module_name)
+            end
+
+            if pkg:is_installed() then
+                utils.info("Installation finished for " .. pkg_name)
+                configure_server(name, server)
+            else
+                utils.err("Installation failed for " .. pkg_name)
+                server.enable = false
+            end
+        end))
+    else
+        if vim.fn.executable(server.lspconfig.cmd[1]) == 1 then
+            configure_server(name, server)
+        else
+            utils.info(name .. " not installed, disabling", module_name)
+            server.enable = false
+        end
+    end
+end
+
+local function register_server(name, server)
+    local augroup = vim.api.nvim_create_augroup("LSP-" .. name, {})
+    vim.api.nvim_create_autocmd("FileType", {
+        once = true,
+        pattern = table.concat(server.lspconfig.filetypes, ","),
+        callback = vim.schedule_wrap(function ()
+            setup_server(name, server)
+            vim.api.nvim_del_augroup_by_id(augroup)
+        end),
+        group = augroup,
     })
-    end)
 end
 
-return P
+function M.filetypes()
+    if not _filetypes then
+        _filetypes = {}
+        local unique = {}
+        for _, server in pairs(config) do
+            for _, ft in ipairs(server.lspconfig.filetypes) do
+                if not unique[ft] then
+                    table.insert(_filetypes, ft)
+                    unique[ft] = true
+                end
+            end
+        end
+    end
+
+    return _filetypes
+end
+
+function M.setup()
+    setup_diagnostics()
+
+    utils.try_require("cmp_nvim_lsp", module_name, function (mod)
+        capabilities = mod.default_capabilities()
+    end)
+
+    for name, server in pairs(config) do
+        if server.enable then
+            register_server(name, server)
+        end
+    end
+end
+
+return M

lua/lsp/bashls.lua

@@ -1,8 +1,12 @@
 return {
-    enabled = true,
+    enable = true,
     dependencies = {
         "npm",
     },
+    mason = {
+        name = "bash-language-server",
+        -- version = "",
+    },
     lspconfig = {
         filetypes = {
             "sh",

lua/lsp/clangd.lua

@@ -1,5 +1,9 @@
 return {
-    enabled = true,
+    enable = true,
+    mason = {
+        name = "clangd",
+        -- version = "",
+    },
     lspconfig = {
         filetypes = {
             "c",

lua/lsp/cmake.lua

@@ -1,11 +1,15 @@
 return {
-    enabled = true,
+    enable = true,
     dependencies = {
         "python3",
     },
     py_module_deps = {
         "venv",
     },
+    mason = {
+        name = "cmake-language-server",
+        -- version = "",
+    },
     lspconfig = {
         filetypes = {
             "cmake",

lua/lsp/diagnosticls.lua

@@ -5,10 +5,14 @@
 -- https://github.com/iamcco/coc-diagnostic/blob/master/src/config.ts
 
 return {
-    enabled = true,
+    enable = true,
     dependencies = {
         "npm",
     },
+    mason = {
+        name = "diagnostic-languageserver",
+        -- version = "",
+    },
     lspconfig = {
         filetypes = {
             "python",
@@ -110,16 +114,16 @@ return {
                 sh = { "shfmt", },
                 bash = { "shfmt", },
                 zsh = { "shfmt", },
-                php = { "php_cs_fixer", },
+                -- php = { "php_cs_fixer", },
             },
             formatters = {
                 autopep8 = {
                     command = "autopep8",
                     args = {
                         "--aggressive",
-                        "-"
+                        "-",
                     },
-                    rootPatterns = {"Pipfile", "tox.ini", ".git"},
+                    rootPatterns = { "Pipfile", "tox.ini", ".git", },
                     isStdout = true,
                     isStderr = false,
                     ignoreExitCode = false,

lua/lsp/gopls.lua

@@ -1,7 +1,11 @@
 -- spec: https://rust-analyzer.github.io/manual.html#configuration
 
 return {
-    enabled = true,
+    enable = true,
+    mason = {
+        name = "gopls",
+        -- version = "",
+    },
     lspconfig = {
         filetypes = {
             "go",

lua/lsp/intelephense.lua

@@ -1,7 +1,7 @@
 -- spec: https://github.com/bmewburn/intelephense-docs/blob/master/installation.md
 
 return {
-    enabled = true,
+    enable = true,
     dependencies = {
         "npm",
     },
@@ -10,6 +10,10 @@ return {
         "composer.lock",
         "vendor",
     },
+    mason = {
+        name = "intelephense",
+        -- version = "",
+    },
     lspconfig = {
         filetypes = {
             "php",

lua/lsp/jedi_language_server.lua

@@ -1,11 +1,15 @@
 return {
-    enabled = true,
+    enable = true,
     dependencies = {
         "python3",
     },
     py_module_deps = {
         "venv",
     },
+    mason = {
+        name = "jedi-language-server",
+        -- version = "",
+    },
     lspconfig = {
         filetypes = {
             "python",

lua/lsp/lemminx.lua

@@ -2,6 +2,10 @@
 
 return {
     enabled = true,
+    mason = {
+        name = "lemminx",
+        -- version = "",
+    },
     lspconfig = {
         filetypes = {
             "xml",

lua/lsp/lua_ls.lua

@@ -1,7 +1,11 @@
 -- spec: https://luals.github.io/wiki/settings/
 
 return {
-    enabled = true,
+    enable = true,
+    mason = {
+        name = "lua-language-server",
+        -- version = "",
+    },
     lspconfig = {
         filetypes = {
             "lua",

lua/lsp/rust_analyzer.lua

@@ -1,7 +1,11 @@
 -- spec: https://rust-analyzer.github.io/manual.html#configuration
 
 return {
-    enabled = true,
+    enable = true,
+    mason = {
+        name = "rust-analyzer",
+        -- version = "",
+    },
     lspconfig = {
         filetypes = {
             "rust",

lua/lsp/zls.lua

@@ -1,8 +1,14 @@
+-- spec: https://github.com/zigtools/zls#configuration-options
+
 return {
-    enabled = true,
+    enable = true,
     dependencies = {
         "zig",
     },
+    -- mason = {
+    --     name = "zls",
+    --     -- version = "",
+    -- },
     lspconfig = {
         filetypes = {
             "zig",

lua/plugins.lua

@@ -8,6 +8,20 @@ local plugins = {
         name = "moonfly",
         config = require("plugins.moonfly"),
     },
+    --[[ {
+        "catppuccin/nvim",
+        name = "catppuccin",
+        priority = 100,
+        lazy = false,
+        config = require("plugins.catppuccin"),
+    }, ]]
+    --[[ {
+        "navarasu/onedark.nvim",
+        priority = 1000,
+        lazy = false,
+        -- name = "moonfly",
+        config = require("plugins.onedark"),
+    }, ]]
     {
         "rcarriga/nvim-notify",
         priority = 900,
@@ -60,12 +74,6 @@ local plugins = {
         lazy = true,
         event = "VimEnter",
     },
-    {
-        "williamboman/mason-lspconfig.nvim",
-        config = require("plugins.mason_lspconfig"),
-        lazy = true,
-        event = "VimEnter",
-    },
     {
         "neovim/nvim-lspconfig",
         config = require("lsp").setup,
@@ -156,10 +164,6 @@ local plugins = {
         lazy = true,
         event = "VimEnter",
     },
-    {
-        "RubixDev/mason-update-all",
-        config = require("plugins.mason_update_all"),
-    },
     {
         "famiu/bufdelete.nvim",
         config = require("plugins.bufdelete"),

lua/plugins/mason_lspconfig.lua

@@ -1,12 +0,0 @@
--- https://github.com/williamboman/mason-lspconfig.nvim
-
-local function setup()
-    require("mason-lspconfig").setup({
-        -- A list of servers to automatically install if they're not already installed. Example: { "rust_analyzer@nightly", "lua_ls" }
-        -- This setting has no relation with the `automatic_installation` setting.
-        ---@type string[]
-        ensure_installed = require("lsp").language_servers(),
-    })
-end
-
-return setup

lua/plugins/mason_update_all.lua

@@ -1,7 +0,0 @@
--- https://github.com/RubixDev/mason-update-all
-
-local function setup()
-    require("mason-update-all").setup()
-end
-
-return setup