refactor: rename code action

Author: ankddev

CHANGELOG.md

@@ -307,13 +307,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

@@ -1363,15 +1363,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,
@@ -1398,7 +1398,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)
@@ -1407,7 +1407,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

@@ -45,7 +45,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,
@@ -467,9 +467,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";
@@ -11171,7 +11171,7 @@ fn merge_case_branch_does_not_merge_branches_with_variables_with_same_name_and_d
 
 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
 
@@ -11189,7 +11189,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