fix crash when hover header cannot be formatted

elixir-lsp · Oct 24, 2023 · 2fb3108 · 2fb3108
1 parent cbdec4b
commit 2fb3108
Showing 1 changed file with 13 additions and 4 deletions.
diff --git a/apps/language_server/lib/language_server/providers/hover.ex b/apps/language_server/lib/language_server/providers/hover.ex
@@ -130,8 +130,7 @@ defmodule ElixirLS.LanguageServer.Providers.Hover do
 
     function_name =
       "#{mod_str}.#{fun_str}(#{Enum.join(info.args, ", ")})"
-      |> Code.format_string!(line_length: 40)
-      |> to_string
+      |> format_header
 
     """
     ```elixir
@@ -159,8 +158,7 @@ defmodule ElixirLS.LanguageServer.Providers.Hover do
 
     type_name =
       "#{mod_formatted}#{info.type}(#{Enum.join(info.args, ", ")})"
-      |> Code.format_string!(line_length: 40)
-      |> to_string
+      |> format_header
 
     """
     ```elixir
@@ -307,4 +305,15 @@ defmodule ElixirLS.LanguageServer.Providers.Hover do
   defp get_metadata_entry_md({key, value}) do
     "**#{key}** #{value}"
   end
+
+  defp format_header(text) do
+    text
+    |> Code.format_string!(line_length: 40)
+    |> to_string
+  rescue
+    _ ->
+      # Code.format_string! can raise SyntaxError e.g.
+      # for Kernel...(first, last)
+      text
+  end
 end