feat(lsp): only notify on server status error by default

Author: Marc Jakobi

doc/rustaceanvim.txt

@@ -300,17 +300,15 @@ rustaceanvim.lsp.ClientOpts                        *rustaceanvim.lsp.ClientOpts*
                                                                                                                    Whether to search (upward from the buffer) for rust-analyzer settings in .vscode/settings json.
                                                                                                                    If found, loaded settings will override configured options.
                                                                                                                    Default: true
-
-    See: ~
-        |vim.lsp.ClientConfig|
+        {status_notify_level?}   (rustaceanvim.server.status_notify_level)
+                                                                                                                   Server status warning level to notify at.
+                                                                                                                   Default: 'error'
 
 
-rustaceanvim.dap.Opts                                    *rustaceanvim.dap.Opts*
+rustaceanvim.server.status_notify_level*rustaceanvim.server.status_notify_level*
 
-    Fields: ~
-        {autoload_configurations}  (boolean)
-                                              Whether to autoload nvim-dap configurations when rust-analyzer has attached?
-                                              Default: `true`.
+    Type: ~
+        "error"|"warning"|rustaceanvim.disable
 
 
 rustaceanvim.disable                                      *rustaceanvim.disable*
@@ -319,6 +317,14 @@ rustaceanvim.disable                                      *rustaceanvim.disable*
         false
 
 
+rustaceanvim.dap.Opts                                    *rustaceanvim.dap.Opts*
+
+    Fields: ~
+        {autoload_configurations}  (boolean)
+                                              Whether to autoload nvim-dap configurations when rust-analyzer has attached?
+                                              Default: `true`.
+
+
 rustaceanvim.dap.Command                              *rustaceanvim.dap.Command*
 
     Type: ~

lua/rustaceanvim/config/init.lua

@@ -191,8 +191,17 @@ vim.g.rustaceanvim = vim.g.rustaceanvim
 ---If found, loaded settings will override configured options.
 ---Default: true
 ---@field load_vscode_settings? boolean
+---
+---Server status warning level to notify at.
+---Default: 'error'
+---@field status_notify_level? rustaceanvim.server.status_notify_level
+---
 ---@see vim.lsp.ClientConfig
 
+---@alias rustaceanvim.server.status_notify_level 'error' | 'warning' | rustaceanvim.disable
+
+---@alias rustaceanvim.disable false
+
 ---@class rustaceanvim.dap.Opts
 ---
 ---Whether to autoload nvim-dap configurations when rust-analyzer has attached?
@@ -220,8 +229,6 @@ vim.g.rustaceanvim = vim.g.rustaceanvim
 ---Default: `true`.
 ---@field load_rust_types? fun():boolean | boolean
 
----@alias rustaceanvim.disable false
-
 ---@alias rustaceanvim.dap.Command string
 
 ---@class rustaceanvim.dap.executable.Config

lua/rustaceanvim/config/internal.lua

@@ -295,6 +295,8 @@ local RustaceanDefaultConfig = {
     },
     ---@type boolean Whether to search (upward from the buffer) for rust-analyzer settings in .vscode/settings json.
     load_vscode_settings = true,
+    ---@type rustaceanvim.server.status_notify_level
+    status_notify_level = 'error',
   },
 
   --- debugging stuff

lua/rustaceanvim/server_status.lua

@@ -5,14 +5,30 @@ local M = {}
 ---@type { [integer]: boolean }
 local _ran_once = {}
 
+---@param health rustaceanvim.lsp_server_health_status
+---@return boolean
+local function is_notify_enabled_for(health)
+  if health and health == 'ok' then
+    return false
+  end
+  local notify_level = config.server.status_notify_level
+  if not notify_level then
+    return false
+  end
+  if notify_level == 'error' then
+    return health == 'error'
+  end
+  return true
+end
+
 ---@param result rustaceanvim.internal.RAInitializedStatus
 function M.handler(_, result, ctx, _)
   -- quiescent means the full set of results is ready.
   if not result or not result.quiescent then
     return
   end
   -- notify of LSP errors/warnings
-  if result.health and result.health ~= 'ok' then
+  if is_notify_enabled_for(result.health) then
     local message = ([[
 rust-analyzer health status is [%s]:
 %s