fix: reuse of LSP client if the root dir and config name has not changed

[AntoineGS]

Prevents scenarios where a second client could be created and cause and caught in store_client_id. May fix #359

[AntoineGS]

Turns out lsp.lua wants one of the workspace_folders to match the root_dir or it assumes a new workspace.

--- @param client vim.lsp.Client
--- @param config vim.lsp.ClientConfig
--- @return boolean
local function reuse_client_default(client, config)
  if client.name ~= config.name then
    return false
  end

  if config.root_dir then
    for _, dir in ipairs(client.workspace_folders or {}) do
      -- note: do not need to check client.root_dir since that should be client.workspace_folders[1]
      if config.root_dir == dir.name then
        return true
      end
    end
  end

  -- TODO(lewis6991): also check config.workspace_folders

  return false
end

I am closing this as this is fixed in the workspace implementation.