update changelog and names

Author: giacomocavalieri

CHANGELOG.md

@@ -154,18 +154,20 @@
   both within a module and across multiple modules.
   ([Surya Rose](https://github.com/GearsDatapacks))
 
-- The language server now offers a code action to remove `echo`. For example:
+- The language server now offers a code action to remove all `echo`s in a
+  module. For example:
 
   ```gleam
   pub fn main() {
     [1, 2, 3]
     |> echo
     // ^^^^ If you put your cursor over here
     |> list.filter(int.is_even)
+    |> echo
   }
   ```
 
-  Triggering the code action would remove the `echo` pipeline step:
+  Triggering the code action would remove all the `echo` pipeline steps:
 
   ```gleam
   pub fn main() {
@@ -174,7 +176,7 @@
   }
   ```
 
-  This would also work with `echo` used before an expression:
+  This also works with all the `echo`s used before an expression:
 
   ```gleam
   pub fn main() {

compiler-core/src/language_server/code_action.rs

@@ -5463,7 +5463,7 @@ impl<'ast> ast::visit::Visit<'ast> for FillUnusedFields<'ast> {
 
 /// Code action to remove an echo.
 ///
-pub struct RemoveEcho<'a> {
+pub struct RemoveEchos<'a> {
     module: &'a Module,
     params: &'a CodeActionParams,
     edits: TextEdits<'a>,
@@ -5475,7 +5475,7 @@ pub struct RemoveEcho<'a> {
     second_to_latest_pipe_step: Option<SrcSpan>,
 }
 
-impl<'a> RemoveEcho<'a> {
+impl<'a> RemoveEchos<'a> {
     pub fn new(
         module: &'a Module,
         line_numbers: &'a LineNumbers,
@@ -5515,7 +5515,7 @@ impl<'a> RemoveEcho<'a> {
     }
 }
 
-impl<'ast> ast::visit::Visit<'ast> for RemoveEcho<'ast> {
+impl<'ast> ast::visit::Visit<'ast> for RemoveEchos<'ast> {
     fn visit_typed_expr_echo(
         &mut self,
         location: &'ast SrcSpan,

compiler-core/src/language_server/engine.rs

@@ -38,7 +38,7 @@ use super::{
         ConvertToUse, ExpandFunctionCapture, ExtractVariable, FillInMissingLabelledArgs,
         FillUnusedFields, GenerateDynamicDecoder, GenerateFunction, GenerateJsonEncoder,
         InlineVariable, InterpolateString, LetAssertToCase, PatternMatchOnValue,
-        RedundantTupleInCaseSubject, RemoveEcho, UseLabelShorthandSyntax,
+        RedundantTupleInCaseSubject, RemoveEchos, UseLabelShorthandSyntax,
         code_action_add_missing_patterns, code_action_convert_qualified_constructor_to_unqualified,
         code_action_convert_unqualified_constructor_to_qualified, code_action_import_module,
         code_action_inexhaustive_let_to_case,
@@ -385,7 +385,7 @@ where
             actions.extend(UseLabelShorthandSyntax::new(module, &lines, &params).code_actions());
             actions.extend(FillInMissingLabelledArgs::new(module, &lines, &params).code_actions());
             actions.extend(ConvertFromUse::new(module, &lines, &params).code_actions());
-            actions.extend(RemoveEcho::new(module, &lines, &params).code_actions());
+            actions.extend(RemoveEchos::new(module, &lines, &params).code_actions());
             actions.extend(ConvertToUse::new(module, &lines, &params).code_actions());
             actions.extend(ExpandFunctionCapture::new(module, &lines, &params).code_actions());
             actions.extend(FillUnusedFields::new(module, &lines, &params).code_actions());