avoid qualify code action on shadowed modules

Author: scristobal

CHANGELOG.md

@@ -411,3 +411,8 @@
 - Only print the user-friendly LSP message when stdin is a terminal to avoid
   emitting redundant logs.
   ([Ariel Parker](https://github.com/arielherself))
+
+- Fixed a bug where the language server would wrongly override local variables
+  if they were shadowing an unqualified module function.
+  ([Samuel Cristobal](https://github.com/scristobal))
+

compiler-core/src/language_server/code_action.rs

@@ -2221,7 +2221,11 @@ impl<'ast> ast::visit::Visit<'ast> for UnqualifiedToQualifiedImportSecondPass<'a
             layer,
             ..
         } = &self.unqualified_constructor;
-        if layer.is_value() && wanted_constructor.used_name() == name {
+
+        if layer.is_value()
+            && wanted_constructor.used_name() == name
+            && !constructor.is_local_variable()
+        {
             self.add_module_qualifier(*location);
         }
         ast::visit::visit_typed_expr_var(self, location, constructor, name);

compiler-core/src/language_server/tests/action.rs

@@ -3582,6 +3582,37 @@ pub fn main() {
             .select_until(find_position_of("(1)")),
     );
 }
+
+#[test]
+fn test_unqualified_to_qualified_import_variable_shadowing() {
+    let src = r#"
+
+import wibble.{wobble}
+
+pub fn example() {
+  echo wobble
+
+  let wobble = 1
+
+  echo wobble
+
+  let _ = fn(wobble) {
+    echo wobble
+  }
+
+  todo
+}
+"#;
+
+    assert_code_action!(
+        "Qualify wobble as wibble.wobble",
+        TestProject::for_source(src).add_hex_module("wibble", "pub fn wobble() { todo }"),
+        find_position_of("wob")
+            .nth_occurrence(2)
+            .select_until(find_position_of("ble").nth_occurrence(3))
+    );
+}
+
 /* TODO: implement qualified unused location
 #[test]
 fn test_remove_unused_qualified_action() {

compiler-core/src/language_server/tests/snapshots/gleam_core__language_server__tests__action__unqualified_to_qualified_import_variable_shadowing.snap

@@ -0,0 +1,43 @@
+---
+source: compiler-core/src/language_server/tests/action.rs
+expression: "\n\nimport wibble.{wobble}\n\npub fn example() {\n  echo wobble\n\n  let wobble = 1\n\n  echo wobble\n\n  let _ = fn(wobble) {\n    echo wobble\n  }\n\n  todo\n}\n"
+---
+----- BEFORE ACTION
+
+
+import wibble.{wobble}
+
+pub fn example() {
+  echo wobble
+       ▔▔▔↑  
+
+  let wobble = 1
+
+  echo wobble
+
+  let _ = fn(wobble) {
+    echo wobble
+  }
+
+  todo
+}
+
+
+----- AFTER ACTION
+
+
+import wibble.{}
+
+pub fn example() {
+  echo wibble.wobble
+
+  let wobble = 1
+
+  echo wobble
+
+  let _ = fn(wobble) {
+    echo wobble
+  }
+
+  todo
+}