Fix requestMessage doesnt seem to match spec

apps/language_server/lib/language_server/json_rpc.ex

@@ -27,6 +27,16 @@ defmodule ElixirLS.LanguageServer.JsonRpc do
     end
   end
 
+  defmacro request(id, method) do
+    quote do
+      %{
+        "id" => unquote(id),
+        "method" => unquote(method),
+        "jsonrpc" => "2.0"
+      }
+    end
+  end
+
   defmacro request(id, method, params) do
     quote do
       %{

apps/language_server/lib/language_server/server.ex

@@ -121,22 +121,11 @@ defmodule ElixirLS.LanguageServer.Server do
 
   @impl GenServer
   def handle_cast({:receive_packet, request(id, _, _) = packet}, state) do
-    state =
-      case handle_request(packet, state) do
-        {:ok, result, state} ->
-          JsonRpc.respond(id, result)
-          state
-
-        {:error, type, msg, state} ->
-          JsonRpc.respond_with_error(id, type, msg)
-          state
-
-        {:async, fun, state} ->
-          {pid, _ref} = handle_request_async(id, fun)
-          %{state | requests: Map.put(state.requests, id, pid)}
-      end
+    {:noreply, handle_request_packet(id, packet, state)}
+  end
 
-    {:noreply, state}
+  def handle_cast({:receive_packet, request(id, method)}, state) do
+    {:noreply, handle_request_packet(id, request(id, method, nil), state)}
   end
 
   @impl GenServer
@@ -317,6 +306,22 @@ defmodule ElixirLS.LanguageServer.Server do
     state
   end
 
+  defp handle_request_packet(id, packet, state) do
+    case handle_request(packet, state) do
+      {:ok, result, state} ->
+        JsonRpc.respond(id, result)
+        state
+
+      {:error, type, msg, state} ->
+        JsonRpc.respond_with_error(id, type, msg)
+        state
+
+      {:async, fun, state} ->
+        {pid, _ref} = handle_request_async(id, fun)
+        %{state | requests: Map.put(state.requests, id, pid)}
+    end
+  end
+
   defp handle_request(initialize_req(_id, root_uri, client_capabilities), state) do
     show_version_warnings()
 
@@ -460,7 +465,15 @@ defmodule ElixirLS.LanguageServer.Server do
     {:ok, x, state}
   end
 
+  defp handle_request(request(_, _) = req, state) do
+    handle_invalid_request(req, state)
+  end
+
   defp handle_request(request(_, _, _) = req, state) do
+    handle_invalid_request(req, state)
+  end
+
+  defp handle_invalid_request(req, state) do
     IO.inspect(req, label: "Unmatched request")
     {:error, :invalid_request, nil, state}
   end

apps/language_server/test/server_test.exs

@@ -116,6 +116,16 @@ defmodule ElixirLS.LanguageServer.ServerTest do
     }
   end
 
+  test "requests shutdown without params", %{server: server} do
+    Server.receive_packet(server, request(1, "shutdown"))
+    assert %{received_shutdown?: true} = :sys.get_state(server)
+  end
+
+  test "requests shutdown with params", %{server: server} do
+    Server.receive_packet(server, request(1, "shutdown", nil))
+    assert %{received_shutdown?: true} = :sys.get_state(server)
+  end
+
   # TODO: Fix this test for the incremental formatter
   @tag :pending
   test "formatter", %{server: server} do