remove redundant argument

Author: lukaszsamson

apps/language_server/lib/language_server/providers/execute_command.ex

@@ -3,7 +3,7 @@ defmodule ElixirLS.LanguageServer.Providers.ExecuteCommand do
   Adds a @spec annotation to the document when the user clicks on a code lens.
   """
 
-  @callback execute(String.t(), [any], %ElixirLS.LanguageServer.Server{}) ::
+  @callback execute([any], %ElixirLS.LanguageServer.Server{}) ::
               {:ok, any} | {:error, atom, String.t()}
 
   def execute(command, args, state) do
@@ -15,7 +15,7 @@ defmodule ElixirLS.LanguageServer.Providers.ExecuteCommand do
       end
 
     if handler do
-      handler.execute(command, args, state)
+      handler.execute(args, state)
     else
       {:error, :invalid_request, nil}
     end

apps/language_server/lib/language_server/providers/execute_command/apply_spec.ex

@@ -13,7 +13,7 @@ defmodule ElixirLS.LanguageServer.Providers.ExecuteCommand.ApplySpec do
   @default_target_line_length 98
 
   @impl ElixirLS.LanguageServer.Providers.ExecuteCommand
-  def execute("spec:" <> _, args, state) do
+  def execute(args, state) do
     [
       %{
         "uri" => uri,

apps/language_server/lib/language_server/providers/execute_command/expand_macro.ex

@@ -9,7 +9,7 @@ defmodule ElixirLS.LanguageServer.Providers.ExecuteCommand.ExpandMacro do
   @behaviour ElixirLS.LanguageServer.Providers.ExecuteCommand
 
   @impl ElixirLS.LanguageServer.Providers.ExecuteCommand
-  def execute("expandMacro", [uri, text, line], state)
+  def execute([uri, text, line], state)
       when is_binary(text) and is_integer(line) do
     source_file = Server.get_source_file(state, uri)
     cur_text = source_file.text

apps/language_server/test/providers/execute_command/expand_macro_test.exs

@@ -14,7 +14,7 @@ defmodule ElixirLS.LanguageServer.Providers.ExecuteCommand.ExpandMacroTest do
     """
 
     assert {:ok, res} =
-             ExpandMacro.execute("expandMacro", [uri, "", 1], %Server{
+             ExpandMacro.execute([uri, "", 1], %Server{
                source_files: %{
                  uri => %SourceFile{
                    text: text
@@ -30,7 +30,7 @@ defmodule ElixirLS.LanguageServer.Providers.ExecuteCommand.ExpandMacroTest do
            }
 
     assert {:ok, res} =
-             ExpandMacro.execute("expandMacro", [uri, "abc", 1], %Server{
+             ExpandMacro.execute([uri, "abc", 1], %Server{
                source_files: %{
                  uri => %SourceFile{
                    text: text
@@ -56,7 +56,7 @@ defmodule ElixirLS.LanguageServer.Providers.ExecuteCommand.ExpandMacroTest do
     """
 
     assert {:ok, res} =
-             ExpandMacro.execute("expandMacro", [uri, "use ElixirLS.Test.MacroA", 1], %Server{
+             ExpandMacro.execute([uri, "use ElixirLS.Test.MacroA", 1], %Server{
                source_files: %{
                  uri => %SourceFile{
                    text: text