fix: use GenLSP loggin utility

Author: doorgan

apps/expert/lib/expert/state.ex

@@ -129,14 +129,9 @@ defmodule Expert.State do
 
         Expert.Project.Supervisor.stop(project)
 
-        GenLSP.notify(
+        GenLSP.log(
           Expert.get_lsp(),
-          %GenLSP.Notifications.WindowLogMessage{
-            params: %GenLSP.Structures.LogMessageParams{
-              type: GenLSP.Enumerations.MessageType.info(),
-              message: "Stopping project node for #{Project.name(project)}"
-            }
-          }
+          "Stopping project node for #{Project.name(project)}"
         )
 
         project
@@ -278,15 +273,7 @@ defmodule Expert.State do
       {:ok, _pid} ->
         Logger.info("Project node started for #{Project.name(project)}")
 
-        GenLSP.notify(
-          Expert.get_lsp(),
-          %GenLSP.Notifications.WindowLogMessage{
-            params: %GenLSP.Structures.LogMessageParams{
-              type: GenLSP.Enumerations.MessageType.info(),
-              message: "Started project node for #{Project.name(project)}"
-            }
-          }
-        )
+        GenLSP.log(Expert.get_lsp(), "Started project node for #{Project.name(project)}")
 
       {:error, {reason, pid}} when reason in [:already_started, :already_present] ->
         {:ok, pid}
@@ -296,14 +283,9 @@ defmodule Expert.State do
           "Failed to start project node for #{Project.name(project)}: #{inspect(reason, pretty: true)}"
         )
 
-        GenLSP.notify(
+        GenLSP.log(
           Expert.get_lsp(),
-          %GenLSP.Notifications.WindowLogMessage{
-            params: %GenLSP.Structures.LogMessageParams{
-              type: GenLSP.Enumerations.MessageType.error(),
-              message: "Failed to start project node for #{Project.name(project)}"
-            }
-          }
+          "Failed to start project node for #{Project.name(project)}"
         )
 
         {:error, reason}

apps/expert/test/expert/expert_test.exs

@@ -94,10 +94,7 @@ defmodule ExpertTest do
 
       assert_notification(
         "window/logMessage",
-        %{
-          "type" => 3,
-          "message" => ^expected_message
-        }
+        %{"message" => ^expected_message}
       )
 
       assert [project] = Expert.ActiveProjects.projects()
@@ -126,10 +123,7 @@ defmodule ExpertTest do
 
       assert_notification(
         "window/logMessage",
-        %{
-          "type" => 3,
-          "message" => ^expected_message
-        }
+        %{"message" => ^expected_message}
       )
 
       assert [_project_1] = Expert.ActiveProjects.projects()
@@ -155,10 +149,7 @@ defmodule ExpertTest do
 
       assert_notification(
         "window/logMessage",
-        %{
-          "type" => 3,
-          "message" => ^expected_message
-        }
+        %{"message" => ^expected_message}
       )
 
       assert [project_1, project_2] = Expert.ActiveProjects.projects()
@@ -185,10 +176,7 @@ defmodule ExpertTest do
 
       assert_notification(
         "window/logMessage",
-        %{
-          "type" => 3,
-          "message" => ^expected_message
-        }
+        %{"message" => ^expected_message}
       )
 
       assert [project] = Expert.ActiveProjects.projects()
@@ -216,10 +204,7 @@ defmodule ExpertTest do
 
       assert_notification(
         "window/logMessage",
-        %{
-          "type" => 3,
-          "message" => ^expected_message
-        }
+        %{"message" => ^expected_message}
       )
 
       assert [] = Expert.ActiveProjects.projects()
@@ -245,10 +230,7 @@ defmodule ExpertTest do
 
       assert_notification(
         "window/logMessage",
-        %{
-          "type" => 3,
-          "message" => ^expected_message
-        }
+        %{"message" => ^expected_message}
       )
 
       file_uri = Path.join([secondary_project.root_uri, "lib", "secondary.ex"])
@@ -274,10 +256,7 @@ defmodule ExpertTest do
 
       assert_notification(
         "window/logMessage",
-        %{
-          "type" => 3,
-          "message" => ^expected_message
-        }
+        %{"message" => ^expected_message}
       )
 
       assert [_main, project] = Expert.ActiveProjects.projects()