From c5c68889a3d1984e00d9637d6cf7195f6039ec81 Mon Sep 17 00:00:00 2001
From: Brad Lewis <22850972+BradLewis@users.noreply.github.com>
Date: Wed, 10 Sep 2025 18:44:35 -0400
Subject: Check if name is referring to a known symbol before adding to the
 import code action

---
 src/server/action.odin | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

(limited to 'src/server')

diff --git a/src/server/action.odin b/src/server/action.odin
index 4a337a3..0c3c74a 100644
--- a/src/server/action.odin
+++ b/src/server/action.odin
@@ -80,6 +80,10 @@ add_missing_imports :: proc(
 	actions: ^[dynamic]CodeAction,
 ) {
 	if name, ok := selector.expr.derived.(^ast.Ident); ok {
+		// If we already know what the name is referring to, don't prompt anything
+		if _, ok := resolve_type_identifier(ast_context, name^); ok {
+			return
+		}
 		for collection, pkgs in build_cache.pkg_aliases {
 			for pkg in pkgs {
 				fullpath := path.join({config.collections[collection], pkg})
@@ -97,7 +101,6 @@ add_missing_imports :: proc(
 
 				if pkg == name.name {
 					pkg_decl := ast_context.file.pkg_decl
-					log.error(pkg_decl.end.line)
 					import_edit := TextEdit {
 						range = {
 							start = {line = pkg_decl.end.line + 1, character = 0},
-- 
cgit v1.2.3