use root option

Author: lukaszsamson

apps/language_server/lib/language_server/providers/execute_command/apply_spec.ex

@@ -57,7 +57,7 @@ defmodule ElixirLS.LanguageServer.Providers.ExecuteCommand.ApplySpec do
         try do
           target_line_length =
             case SourceFile.formatter_for(uri, state.project_dir, true) do
-              {:ok, {_, opts, _formatter_exs_dir}} ->
+              {:ok, {_, opts}} ->
                 Keyword.get(opts, :line_length, @default_target_line_length)
 
               {:error, reason} ->

apps/language_server/lib/language_server/providers/formatting.ex

@@ -12,7 +12,8 @@ defmodule ElixirLS.LanguageServer.Providers.Formatting do
     if SourceFile.Path.path_in_dir?(file_path, project_dir) do
       # file in project_dir we find formatter and options for file
       case SourceFile.formatter_for(uri, project_dir, mix_project?) do
-        {:ok, {formatter, opts, formatter_exs_dir}} ->
+        {:ok, {formatter, opts}} ->
+          formatter_exs_dir = opts[:root]
           if should_format?(uri, formatter_exs_dir, opts[:inputs], project_dir) do
             do_format(source_file, formatter, opts)
           else

apps/language_server/lib/language_server/server.ex

@@ -1096,7 +1096,7 @@ defmodule ElixirLS.LanguageServer.Server do
 
     locals_without_parens =
       case SourceFile.formatter_for(uri, state.project_dir, state.mix_project?) do
-        {:ok, {_, opts, _formatter_exs_dir}} ->
+        {:ok, {_, opts}} ->
           locals_without_parens = Keyword.get(opts, :locals_without_parens, [])
 
           if List.improper?(locals_without_parens) do
@@ -1226,7 +1226,7 @@ defmodule ElixirLS.LanguageServer.Server do
       if String.ends_with?(uri, [".ex", ".exs"]) or source_file.language_id in ["elixir"] do
         formatter_opts =
           case SourceFile.formatter_for(uri, state.project_dir, state.mix_project?) do
-            {:ok, {_, opts, _formatter_exs_dir}} -> opts
+            {:ok, {_, opts}} -> opts
             {:error, _} -> []
           end
 

apps/language_server/lib/language_server/source_file.ex

@@ -257,7 +257,7 @@ defmodule ElixirLS.LanguageServer.SourceFile do
   end
 
   @spec formatter_for(String.t(), String.t() | nil, boolean) ::
-          {:ok, {function | nil, keyword(), String.t()}} | {:error, any}
+          {:ok, {function | nil, keyword()}} | {:error, any}
   def formatter_for(uri = "file:" <> _, project_dir, mix_project?) when is_binary(project_dir) do
     path = __MODULE__.Path.from_uri(uri)
 
@@ -271,7 +271,12 @@ defmodule ElixirLS.LanguageServer.SourceFile do
             manifest_path: MixProjectCache.manifest_path(),
             config_mtime: MixProjectCache.config_mtime(),
             mix_project: MixProjectCache.get(),
-            root: project_dir
+            root: project_dir,
+            plugin_loader: fn _plugins ->
+              # we don't do any plugin loading as this may trigger compile
+              # TODO it may be safe to compile on 1.18+
+              :ok
+            end
           ]
 
           {:ok, Mix.Tasks.ElixirLSFormat.formatter_for_file(path, opts)}