telemetry improvements

Author: lukaszsamson

apps/language_server/lib/language_server/build.ex

@@ -66,7 +66,7 @@ defmodule ElixirLS.LanguageServer.Build do
           Tracer.save()
           Logger.info("Compile took #{div(us, 1000)} milliseconds")
 
-          JsonRpc.telemetry("build", %{"elixir_ls.result" => result}, %{
+          JsonRpc.telemetry("build", %{"elixir_ls.build_result" => result}, %{
             "elixir_ls.build_time" => div(us, 1000)
           })
         end)

apps/language_server/lib/language_server/server.ex

@@ -1440,10 +1440,10 @@ defmodule ElixirLS.LanguageServer.Server do
         Logger.error("client/registerCapability returned: #{inspect(other)}")
 
         JsonRpc.telemetry(
-          "reverse_request_error",
+          "lsp_reverse_request_error",
           %{
-            "elixir_ls.reverse_request_error" => inspect(other),
-            "elixir_ls.reverse_request" => "client/registerCapability"
+            "elixir_ls.lsp_reverse_request_error" => inspect(other),
+            "elixir_ls.lsp_reverse_request" => "client/registerCapability"
           },
           %{}
         )
@@ -1467,10 +1467,10 @@ defmodule ElixirLS.LanguageServer.Server do
         Logger.error("client/registerCapability returned: #{inspect(other)}")
 
         JsonRpc.telemetry(
-          "reverse_request_error",
+          "lsp_reverse_request_error",
           %{
-            "elixir_ls.reverse_request_error" => inspect(other),
-            "elixir_ls.reverse_request" => "client/registerCapability"
+            "elixir_ls.lsp_reverse_request_error" => inspect(other),
+            "elixir_ls.lsp_reverse_request" => "client/registerCapability"
           },
           %{}
         )
@@ -1720,10 +1720,10 @@ defmodule ElixirLS.LanguageServer.Server do
           Logger.error("Cannot get client configuration: #{inspect(other)}")
 
           JsonRpc.telemetry(
-            "reverse_request_error",
+            "lsp_reverse_request_error",
             %{
-              "elixir_ls.reverse_request_error" => inspect(other),
-              "elixir_ls.reverse_request" => "workspace/configuration"
+              "elixir_ls.lsp_reverse_request_error" => inspect(other),
+              "elixir_ls.lsp_reverse_request" => "workspace/configuration"
             },
             %{}
           )