refactor: rename code action

Author: ankddev

CHANGELOG.md

@@ -15,13 +15,13 @@
   }
 
   pub fn add_one(thing) {
-    //     ^ Triggering "Annotate all top level type definitions" code action here
+    //     ^ Triggering "Annotate all top level definitions" code action here
     let result = add(thing, 1)
     result
   }
   ```
 
-  Triggering the "Annotate all top level type definitions" code action over
+  Triggering the "Annotate all top level definitions" code action over
   the name of function `add_one` would result in following code:
 
   ```gleam

compiler-core/src/language_server/code_action.rs

@@ -1362,15 +1362,15 @@ impl<'a> AddAnnotations<'a> {
 
 /// Code action to add type annotations to all top level definitions
 ///
-pub struct AnnotateTopLevelTypeDefinitions<'a> {
+pub struct AnnotateTopLevelDefinitions<'a> {
     module: &'a Module,
     params: &'a CodeActionParams,
     edits: TextEdits<'a>,
     printer: Printer<'a>,
     is_hovering_definition: bool,
 }
 
-impl<'a> AnnotateTopLevelTypeDefinitions<'a> {
+impl<'a> AnnotateTopLevelDefinitions<'a> {
     pub fn new(
         module: &'a Module,
         line_numbers: &'a LineNumbers,
@@ -1397,7 +1397,7 @@ impl<'a> AnnotateTopLevelTypeDefinitions<'a> {
         };
 
         let mut action = Vec::with_capacity(1);
-        CodeActionBuilder::new("Annotate all top level type definitions")
+        CodeActionBuilder::new("Annotate all top level definitions")
             .kind(CodeActionKind::REFACTOR_REWRITE)
             .changes(self.params.text_document.uri.clone(), self.edits.edits)
             .preferred(false)
@@ -1406,7 +1406,7 @@ impl<'a> AnnotateTopLevelTypeDefinitions<'a> {
     }
 }
 
-impl<'ast> ast::visit::Visit<'ast> for AnnotateTopLevelTypeDefinitions<'_> {
+impl<'ast> ast::visit::Visit<'ast> for AnnotateTopLevelDefinitions<'_> {
     fn visit_typed_module_constant(&mut self, constant: &'ast TypedModuleConstant) {
         // Since type variable names are local to definitions, any type variables
         // in other parts of the module shouldn't affect what we print for the

compiler-core/src/language_server/engine.rs

@@ -44,7 +44,7 @@ use std::{collections::HashSet, sync::Arc};
 use super::{
     DownloadDependencies, MakeLocker,
     code_action::{
-        AddAnnotations, AnnotateTopLevelTypeDefinitions, CodeActionBuilder, ConvertFromUse,
+        AddAnnotations, AnnotateTopLevelDefinitions, CodeActionBuilder, ConvertFromUse,
         ConvertToFunctionCall, ConvertToPipe, ConvertToUse, ExpandFunctionCapture, ExtractConstant,
         ExtractVariable, FillInMissingLabelledArgs, FillUnusedFields, FixBinaryOperation,
         FixTruncatedBitArraySegment, GenerateDynamicDecoder, GenerateFunction, GenerateJsonEncoder,
@@ -461,9 +461,8 @@ where
             )
             .code_actions();
             AddAnnotations::new(module, &lines, &params).code_action(&mut actions);
-            actions.extend(
-                AnnotateTopLevelTypeDefinitions::new(module, &lines, &params).code_actions(),
-            );
+            actions
+                .extend(AnnotateTopLevelDefinitions::new(module, &lines, &params).code_actions());
             Ok(if actions.is_empty() {
                 None
             } else {

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

@@ -112,7 +112,7 @@ const ASSIGN_UNUSED_RESULT: &str = "Assign unused Result value to `_`";
 const ADD_MISSING_PATTERNS: &str = "Add missing patterns";
 const ADD_ANNOTATION: &str = "Add type annotation";
 const ADD_ANNOTATIONS: &str = "Add type annotations";
-const ANNOTATE_TOP_LEVEL_TYPE_DEFINITIONS: &str = "Annotate all top level type definitions";
+const ANNOTATE_TOP_LEVEL_DEFINITIONS: &str = "Annotate all top level definitions";
 const CONVERT_FROM_USE: &str = "Convert from `use`";
 const CONVERT_TO_USE: &str = "Convert to `use`";
 const EXTRACT_VARIABLE: &str = "Extract variable";
@@ -10642,7 +10642,7 @@ fn wibble(f: fn() -> Float) -> Float { f() }
 #[test]
 fn annotate_all_top_level_definitions_dont_affect_local_vars() {
     assert_code_action!(
-        ANNOTATE_TOP_LEVEL_TYPE_DEFINITIONS,
+        ANNOTATE_TOP_LEVEL_DEFINITIONS,
         r#"
 pub const answer = 42
 
@@ -10660,7 +10660,7 @@ pub fn add_one(thing) {
 #[test]
 fn annotate_all_top_level_definitions_constant() {
     assert_code_action!(
-        ANNOTATE_TOP_LEVEL_TYPE_DEFINITIONS,
+        ANNOTATE_TOP_LEVEL_DEFINITIONS,
         r#"
 pub const answer = 42