feature: clear cache when stopping lsp client (#199)

Co-authored-by: Zephyr Lykos <[email protected]>

Author: jfly

lua/null-ls/helpers/cache.lua

@@ -2,19 +2,27 @@ local next_key = 0
 
 local M = {}
 
-M.cache = {}
+M._reset = function()
+    M.cache = {}
+end
+
+M._reset()
 
 --- creates a function that caches the output of a callback, indexed by bufnr
 ---@param cb function
 ---@return fun(params: NullLsParams): any
 M.by_bufnr = function(cb)
     -- assign next available key, since we just want to avoid collisions
     local key = next_key
-    M.cache[key] = {}
     next_key = next_key + 1
 
     return function(params)
         local bufnr = params.bufnr
+
+        if M.cache[key] == nil then
+            M.cache[key] = {}
+        end
+
         -- if we haven't cached a value yet, get it from cb
         if M.cache[key][bufnr] == nil then
             -- make sure we always store a value so we know we've already called cb
@@ -25,8 +33,4 @@ M.by_bufnr = function(cb)
     end
 end
 
-M._reset = function()
-    M.cache = {}
-end
-
 return M

lua/null-ls/rpc.lua

@@ -1,6 +1,7 @@
 local c = require("null-ls.config")
 local log = require("null-ls.logger")
 local methods = require("null-ls.methods")
+local cache = require("null-ls.helpers").cache
 
 local notification_cache = {}
 
@@ -151,6 +152,7 @@ M.start = function(dispatchers)
             return stopped
         end,
         terminate = function()
+            cache._reset()
             stopped = true
         end,
     }