fix: buffer issues (#68)

Fixed the case where there is no telescope preview and catch possible undefined buffer behaviour.

lua/nvim-devdocs/init.lua

@@ -79,7 +79,7 @@ M.toggle = function()
   local buf = state.get("last_buf")
   local win = state.get("last_win")
 
-  if not buf then return end
+  if not buf or not vim.api.nvim_buf_is_valid(buf) then return end
 
   if win and vim.api.nvim_win_is_valid(win) then
     vim.api.nvim_win_close(win, true)

lua/nvim-devdocs/operations.lua

@@ -150,17 +150,37 @@ M.uninstall = function(alias)
   end
 end
 
+---@param entry DocEntry
+---@return string[]
+M.read_entry = function(entry)
+  local splited_path = vim.split(entry.path, ",")
+  local file = splited_path[1]
+  local file_path = DOCS_DIR:joinpath(entry.alias, file .. ".md")
+  local content = file_path:read()
+  local pattern = splited_path[2]
+  local next_pattern = nil
+
+  if entry.next_path ~= nil then next_pattern = vim.split(entry.next_path, ",")[2] end
+
+  local lines = vim.split(content, "\n")
+  local filtered_lines = M.filter_doc(lines, pattern, next_pattern)
+
+  return filtered_lines
+end
+
 ---@param entry DocEntry
 ---@param callback function
-M.read_entry = function(entry, callback)
+M.read_entry_async = function(entry, callback)
   local splited_path = vim.split(entry.path, ",")
   local file = splited_path[1]
   local file_path = DOCS_DIR:joinpath(entry.alias, file .. ".md")
 
   file_path:_read_async(vim.schedule_wrap(function(content)
     local pattern = splited_path[2]
     local next_pattern = nil
+
     if entry.next_path ~= nil then next_pattern = vim.split(entry.next_path, ",")[2] end
+
     local lines = vim.split(content, "\n")
     local filtered_lines = M.filter_doc(lines, pattern, next_pattern)
 
@@ -289,7 +309,7 @@ M.keywordprg = function(keyword)
   for _, value in pairs(entries or {}) do
     if value.name == keyword or value.link == keyword then
       entry = value
-      M.read_entry(entry, callback)
+      M.read_entry_async(entry, callback)
     end
   end
 

lua/nvim-devdocs/pickers.lua

@@ -57,7 +57,7 @@ local doc_previewer = previewers.new_buffer_previewer({
   define_preview = function(self, entry)
     local bufnr = self.state.bufnr
 
-    operations.read_entry(entry.value, function(filtered_lines)
+    operations.read_entry_async(entry.value, function(filtered_lines)
       vim.api.nvim_buf_set_lines(bufnr, 0, -1, false, filtered_lines)
 
       if plugin_config.options.previewer_cmd and plugin_config.options.picker_cmd then
@@ -71,14 +71,12 @@ local doc_previewer = previewers.new_buffer_previewer({
 })
 
 local function open_doc(selection, float)
-  local bufnr = nil
+  local bufnr = state.get_global_key("last_preview_bufnr")
 
-  if plugin_config.options.picker_cmd then
+  if plugin_config.options.picker_cmd or not bufnr or not vim.api.nvim_buf_is_valid(bufnr) then
     bufnr = vim.api.nvim_create_buf(false, true)
-    local lines = plugin_state.get("preview_lines")
+    local lines = plugin_state.get("preview_lines") or operations.read_entry(selection.value)
     vim.api.nvim_buf_set_lines(bufnr, 0, -1, false, lines)
-  else
-    bufnr = state.get_global_key("last_preview_bufnr")
   end
 
   plugin_state.set("last_mode", float and "float" or "normal")