Fixed errors when LSP didn't finished loading + updated README

Author: markelca

README.md

@@ -5,15 +5,6 @@ The [telescope.nvim](https://github.com/nvim-telescope/telescope.nvim) pickers t
 The plugin analyzes the entire class/module hierarchy, including parent classes, traits, interfaces, and other inherited elements, giving you a complete view of the API surface.
 ![demo.gif](https://github.com/user-attachments/assets/e9ddda56-912d-4475-b7d6-94c573939db6)
 
-## Features
-
-- **Complete API view**: Display all class/module members including inherited elements
-- **Visibility indicators**: Shows access modifiers (public, private, protected) for each member
-- **Hierarchical analysis**: Understands inheritance from parent classes, traits, and interfaces
-- **Fast navigation**: Quickly jump to any symbol definition
-- **Integration with LSP**: Works alongside your language server for accurate symbol information
-- **Telescope integration**: Familiar fuzzy-finding interface for efficient filtering
-
 ## Supported Languages
 
 - PHP
@@ -30,37 +21,37 @@ The plugin analyzes the entire class/module hierarchy, including parent classes,
 ## Installation
 Using [lazy.nvim](https://github.com/folke/lazy.nvim)
 ```lua
-{
+{ -- Add the clapi plugin with its dependencies
   'markelca/clapi.nvim',
-  -- Dev Mode (Clone the repo, update the `dir` value and uncomment the two lines below)
-  -- dir = '~/<dir-where-you-cloned>/clapi.nvim/',
-  -- name = 'clapi',
   dependencies = {
     'nvim-telescope/telescope.nvim',
     'nvim-treesitter/nvim-treesitter',
     'nvim-lua/plenary.nvim',
   },
+},
+{ -- Add the following to your telescope configuration
+  'nvim-telescope/telescope.nvim',
   config = function()
-    -- Enable the clapi extension adding the following line to your telescope configuration:
-    pcall(require('telescope').load_extension 'clapi')
-
-    -- Optionally you can set up a keymap to run the picker
-    vim.keymap.set('n', '<leader>sa', require('clapi').builtin, { desc = '[S]earch [A]pi' })
-
     -- Configurations for the clapi picker
     require('telescope').setup {
       extensions = {
         clapi = {
           -- Additional options can be configured here
-          -- show_inherited = true, -- Set to false to only show members defined in the current class
-          -- default_visibility = "public", -- Filter by default visibility (public, protected, private)
+          show_inherited = true, -- Set to false to only show members defined in the current class
+          default_visibility = "public", -- Filter by default visibility (public, protected, private)
         },
       },
     }
+    -- Enable the clapi extension adding the following line to your telescope configuration:
+    pcall(require('telescope').load_extension 'clapi')
+    -- Optionally you can set up a keymap to run the picker
+    vim.keymap.set('n', '<leader>sa', require('clapi').builtin, { desc = '[S]earch [A]pi' })
   end,
 }
 ```
-Full example in my nvim config repository: [nvim](https://github.com/MarkelCA/nvim/blob/master/lua/plugins/clapi.lua)
+Full example in my [nvim](https://github.com/markelca/nvim) config repository: 
+- [clapi](https://github.com/markelca/nvim/blob/master/lua/plugins/clapi.lua)
+- [telescope](https://github.com/markelca/nvim/blob/master/lua/plugins/telescope.lua)
 
 ## Usage
 
@@ -129,4 +120,4 @@ Contributions are welcome! Please feel free to submit a Pull Request.
 4. Push to the branch
 5. Submit a pull request
 
-For adding support for a new language, check the `lua/clapi/parser/` directory for examples.
+For adding support for a new language, check the `lua/clapi/parser/` directory for examples.

lua/clapi/lsp.lua

@@ -24,10 +24,21 @@ M.get_file_from_position = async.wrap(function(opts, callback)
 		return
 	end
 
+	local clients = vim.lsp.get_clients({ bufnr = opts.bufnr })
+	if #clients == 0 then
+		utils.notify("get_file_from_position", {
+			msg = "Lsp is not attached",
+			level = "ERROR",
+		})
+		callback(nil)
+		return
+	end
+
 	local callback_called = false
 
 	---Safe callback to prevent multiple callbacks
 	---@param result string|nil The file path or nil
+	---FIX: Accumulate results on multiple callbacks instead of ignoring them
 	local function safe_callback(result)
 		if not callback_called then
 			callback_called = true
@@ -40,27 +51,38 @@ M.get_file_from_position = async.wrap(function(opts, callback)
 		end
 	end
 
-	vim.lsp.buf_request(opts.bufnr, "textDocument/definition", {
+	vim.lsp.buf_request_all(opts.bufnr, "textDocument/definition", {
 		position = opts.position,
 		textDocument = {
 			uri = string.format("file://%s", vim.api.nvim_buf_get_name(opts.bufnr)),
 		},
-	}, function(err, result, _, _)
-		if err or not result then
-			utils.notify("get_parent_file", {
-				msg = "Couldn't get the file for the parent class",
-				level = "ERROR",
-			})
-			callback(nil)
-			return
-		end
+	}, function(results)
+		for _, result_entry in pairs(results) do
+			if result_entry.error then
+				utils.notify("get_parent_file", {
+					msg = "Couldn't get the file for the parent class: " .. result_entry.error.message,
+					level = "ERROR",
+				})
+				callback(nil)
+				return
+			else
+				if result_entry.result == nil then
+					utils.notify("get_parent_file", {
+						msg = "Couldn't get the file for the parent class",
+						level = "WARN",
+					})
+				else
+					local result = result_entry.result
 
-		local uri = result.uri or result[1].uri or result[1].targetUri
+					local uri = result.uri or result[1].uri or result[1].targetUri
 
-		if uri then
-			safe_callback(vim.uri_to_fname(uri))
-		else
-			safe_callback(nil)
+					if uri then
+						safe_callback(vim.uri_to_fname(uri))
+					else
+						safe_callback(nil)
+					end
+				end
+			end
 		end
 	end)
 end, 2)

lua/clapi/parser/init.lua

@@ -46,14 +46,6 @@ end
 local get_parent_file = async.wrap(function(opts, callback)
 	opts = opts or {}
 	opts.bufnr = opts.bufnr or 0
-	-- Use explicit check for boolean values since vim.F.if_nil doesn't handle false properly
-	if opts.show_inherited ~= nil then
-		-- Keep the provided value (could be true or false)
-		opts.show_inherited = opts.show_inherited
-	else
-		-- Default to true if not specified
-		opts.show_inherited = true
-	end
 
 	local filetype = tsparsers.get_buf_lang(opts.bufnr)
 	local parser = vim.treesitter.get_parser(opts.bufnr, filetype)
@@ -63,6 +55,7 @@ local get_parent_file = async.wrap(function(opts, callback)
 			level = "ERROR",
 		})
 		callback(nil)
+		return
 	end
 
 	-- Parse the query
@@ -74,6 +67,7 @@ local get_parent_file = async.wrap(function(opts, callback)
 			level = "ERROR",
 		})
 		callback(nil)
+		return
 	end
 
 	local query = vim.treesitter.query.parse(filetype, query_str)
@@ -84,6 +78,7 @@ local get_parent_file = async.wrap(function(opts, callback)
 			level = "ERROR",
 		})
 		callback(nil)
+		return
 	end
 
 	-- Parse the content
@@ -95,6 +90,7 @@ local get_parent_file = async.wrap(function(opts, callback)
 			level = "ERROR",
 		})
 		callback(nil)
+		return
 	end
 
 	tree = tree[1]
@@ -116,14 +112,16 @@ local get_parent_file = async.wrap(function(opts, callback)
 				if not filepath or filepath == "" then
 					-- error already printed in get_file_from_position
 					callback(nil)
+					return
 				end
+				vim.print("fp", filepath)
 				-- Pass the show_inherited option to parse_file for recursive parent parsing
 				local defs = M.parse_file({
 					filename = filepath,
 					class_name = class_name,
 					show_inherited = opts.show_inherited,
 				})
-				for _, value in pairs(defs) do
+				for _, value in pairs(defs or {}) do
 					if value["visibility"] ~= "private" then
 						table.insert(result, value)
 					end
@@ -165,6 +163,11 @@ M.parse_file = async.wrap(function(opts, callback)
 	end
 
 	if not opts.filetype then
+		if not opts.filename then
+			callback(nil)
+			return
+		end
+
 		local filetype = utils.get_file_extension(opts.filename)
 		if not filetype then
 			utils.notify("parse_file", {
@@ -297,4 +300,3 @@ M.parse_file = async.wrap(function(opts, callback)
 end, 2)
 
 return M
-