Suggest an appropriate module name with the 'defprotocol' snippet

apps/language_server/lib/language_server/providers/completion.ex

@@ -587,6 +587,11 @@ defmodule ElixirLS.LanguageServer.Providers.Completion do
     "defmodule #{suggest_module_name(file_path)}$1 do\n\t$0\nend"
   end
 
+  defp snippet_for({"Kernel", "defprotocol"}, %{file_path: file_path})
+       when is_binary(file_path) do
+    "defprotocol #{suggest_module_name(file_path)}$1 do\n\t$0\nend"
+  end
+
   defp snippet_for(key, %{pipe_before?: true}) do
     # Get pipe-friendly version of snippet if available, otherwise fallback to standard
     Map.get(@pipe_func_snippets, key) || Map.get(@func_snippets, key)
@@ -981,14 +986,15 @@ defmodule ElixirLS.LanguageServer.Providers.Completion do
           insert_text_format(:snippet)
         else
           insert_text_format(:plain_text)
-        end,
+        end
     }
 
-    json = if item.preselect do
-      Map.put(json, "preselect", true)
-    else
-      json
-    end
+    json =
+      if item.preselect do
+        Map.put(json, "preselect", true)
+      else
+        json
+      end
 
     # deprecated as of Language Server Protocol Specification - 3.15
     json =

apps/language_server/test/providers/completion_test.exs

@@ -964,6 +964,62 @@ defmodule ElixirLS.LanguageServer.Providers.CompletionTest do
                "insertText" => "defmodule $1 do\n\t$0\nend"
              } = first
     end
+
+    test "will suggest defprotocol with protocol_name snippet when file path matches **/lib/**/*.ex" do
+      text = """
+      defpro
+      #     ^
+      """
+
+      {line, char} = {0, 6}
+
+      TestUtils.assert_has_cursor_char(text, line, char)
+
+      assert {:ok, %{"items" => [first | _] = _items}} =
+               Completion.completion(
+                 text,
+                 line,
+                 char,
+                 @supports
+                 |> Keyword.put(
+                   :file_path,
+                   "/some/path/my_project/lib/my_project/sub_folder/my_file.ex"
+                 )
+               )
+
+      assert %{
+               "label" => "defprotocol",
+               "insertText" => "defprotocol MyProject.SubFolder.MyFile$1 do\n\t$0\nend"
+             } = first
+    end
+
+    test "will suggest defprotocol without protocol_name snippet when file path does not match expected patterns" do
+      text = """
+      defpro
+      #     ^
+      """
+
+      {line, char} = {0, 6}
+
+      TestUtils.assert_has_cursor_char(text, line, char)
+
+      assert {:ok, %{"items" => [first | _] = _items}} =
+               Completion.completion(
+                 text,
+                 line,
+                 char,
+                 @supports
+                 |> Keyword.put(
+                   :file_path,
+                   "/some/path/my_project/lib/my_project/sub_folder/my_file.heex"
+                 )
+               )
+
+      assert %{
+               "label" => "defprotocol",
+               "insertText" => "defprotocol $1 do\n\t$0\nend"
+             } = first
+    end
   end
 
   describe "generic suggestions" do
@@ -1030,7 +1086,7 @@ defmodule ElixirLS.LanguageServer.Providers.CompletionTest do
       text = """
       defmodule MyModule do
         def hello do
-          Date.today() |> 
+          Date.today() |>
           #               ^
         end
       end