[#1310] Add a configuration option for advertised LSP providers. (#1341)

* add a configuration option for advertised LSP providers

* clean up `is_enabled/0` callbacks

apps/els_core/src/els_config.erl

@@ -59,7 +59,8 @@
     | compiler_telemetry_enabled
     | refactorerl
     | wrangler
-    | edoc_custom_tags.
+    | edoc_custom_tags
+    | providers.
 
 -type path() :: file:filename().
 -type state() :: #{
@@ -81,7 +82,8 @@
     indexing_enabled => boolean(),
     compiler_telemetry_enabled => boolean(),
     wrangler => map() | 'notconfigured',
-    refactorerl => map() | 'notconfigured'
+    refactorerl => map() | 'notconfigured',
+    providers => map()
 }.
 
 %%==============================================================================
@@ -149,6 +151,7 @@ do_initialize(RootUri, Capabilities, InitOptions, {ConfigPath, Config}) ->
     IndexingEnabled = maps:get(<<"indexingEnabled">>, InitOptions, true),
 
     RefactorErl = maps:get("refactorerl", Config, notconfigured),
+    Providers = maps:get("providers", Config, #{}),
 
     %% Initialize and start Wrangler
     case maps:get("wrangler", Config, notconfigured) of
@@ -201,6 +204,7 @@ do_initialize(RootUri, Capabilities, InitOptions, {ConfigPath, Config}) ->
     ok = set(macros, Macros),
     ok = set(plt_path, DialyzerPltPath),
     ok = set(code_reload, CodeReload),
+    ok = set(providers, Providers),
     ?LOG_INFO("Config=~p", [Config]),
     ok = set(
         runtime,

apps/els_lsp/src/els_call_hierarchy_provider.erl

@@ -3,7 +3,6 @@
 -behaviour(els_provider).
 
 -export([
-    is_enabled/0,
     handle_request/1
 ]).
 
@@ -16,9 +15,6 @@
 %%==============================================================================
 %% els_provider functions
 %%==============================================================================
--spec is_enabled() -> boolean().
-is_enabled() -> true.
-
 -spec handle_request(any()) -> {response, any()}.
 handle_request({prepare, Params}) ->
     {Uri, Line, Char} =

apps/els_lsp/src/els_code_action_provider.erl

@@ -3,7 +3,6 @@
 -behaviour(els_provider).
 
 -export([
-    is_enabled/0,
     handle_request/1
 ]).
 
@@ -12,9 +11,6 @@
 %%==============================================================================
 %% els_provider functions
 %%==============================================================================
--spec is_enabled() -> boolean().
-is_enabled() -> true.
-
 -spec handle_request(any()) -> {response, any()}.
 handle_request({document_codeaction, Params}) ->
     #{

apps/els_lsp/src/els_code_lens_provider.erl

@@ -2,7 +2,6 @@
 
 -behaviour(els_provider).
 -export([
-    is_enabled/0,
     options/0,
     handle_request/1
 ]).
@@ -13,9 +12,6 @@
 %%==============================================================================
 %% els_provider functions
 %%==============================================================================
--spec is_enabled() -> boolean().
-is_enabled() -> true.
-
 -spec options() -> map().
 options() ->
     #{resolveProvider => false}.

apps/els_lsp/src/els_definition_provider.erl

@@ -3,7 +3,6 @@
 -behaviour(els_provider).
 
 -export([
-    is_enabled/0,
     handle_request/1
 ]).
 
@@ -12,9 +11,6 @@
 %%==============================================================================
 %% els_provider functions
 %%==============================================================================
--spec is_enabled() -> boolean().
-is_enabled() -> true.
-
 -spec handle_request(any()) -> {response, any()}.
 handle_request({definition, Params}) ->
     #{

apps/els_lsp/src/els_diagnostics_provider.erl

@@ -3,7 +3,6 @@
 -behaviour(els_provider).
 
 -export([
-    is_enabled/0,
     options/0,
     handle_request/1
 ]).
@@ -22,9 +21,6 @@
 %%==============================================================================
 %% els_provider functions
 %%==============================================================================
--spec is_enabled() -> boolean().
-is_enabled() -> true.
-
 -spec options() -> map().
 options() ->
     #{}.

apps/els_lsp/src/els_document_highlight_provider.erl

@@ -3,7 +3,6 @@
 -behaviour(els_provider).
 
 -export([
-    is_enabled/0,
     handle_request/1
 ]).
 
@@ -15,9 +14,6 @@
 %%==============================================================================
 %% els_provider functions
 %%==============================================================================
--spec is_enabled() -> boolean().
-is_enabled() -> true.
-
 -spec handle_request(any()) -> {response, any()}.
 handle_request({document_highlight, Params}) ->
     #{

apps/els_lsp/src/els_document_symbol_provider.erl

@@ -3,7 +3,6 @@
 -behaviour(els_provider).
 
 -export([
-    is_enabled/0,
     handle_request/1
 ]).
 
@@ -12,9 +11,6 @@
 %%==============================================================================
 %% els_provider functions
 %%==============================================================================
--spec is_enabled() -> boolean().
-is_enabled() -> true.
-
 -spec handle_request(any()) -> {response, any()}.
 handle_request({document_symbol, Params}) ->
     #{<<"textDocument">> := #{<<"uri">> := Uri}} = Params,

apps/els_lsp/src/els_execute_command_provider.erl

@@ -3,7 +3,6 @@
 -behaviour(els_provider).
 
 -export([
-    is_enabled/0,
     options/0,
     handle_request/1
 ]).
@@ -17,9 +16,6 @@
 %%==============================================================================
 %% els_provider functions
 %%==============================================================================
--spec is_enabled() -> boolean().
-is_enabled() -> true.
-
 -spec options() -> map().
 options() ->
     Commands = [

apps/els_lsp/src/els_folding_range_provider.erl

@@ -5,7 +5,6 @@
 -include("els_lsp.hrl").
 
 -export([
-    is_enabled/0,
     handle_request/1
 ]).
 
@@ -17,9 +16,6 @@
 %%==============================================================================
 %% els_provider functions
 %%==============================================================================
--spec is_enabled() -> boolean().
-is_enabled() -> true.
-
 -spec handle_request(tuple()) -> {response, folding_range_result()}.
 handle_request({document_foldingrange, Params}) ->
     #{<<"textDocument">> := #{<<"uri">> := Uri}} = Params,

apps/els_lsp/src/els_formatting_provider.erl

@@ -3,11 +3,7 @@
 -behaviour(els_provider).
 
 -export([
-    handle_request/1,
-    is_enabled/0,
-    is_enabled_document/0,
-    is_enabled_range/0,
-    is_enabled_on_type/0
+    handle_request/1
 ]).
 
 %%==============================================================================
@@ -23,23 +19,6 @@
 %%==============================================================================
 %% els_provider functions
 %%==============================================================================
-%% Keep the behaviour happy
--spec is_enabled() -> boolean().
-is_enabled() -> is_enabled_document().
-
--spec is_enabled_document() -> boolean().
-is_enabled_document() -> true.
-
--spec is_enabled_range() -> boolean().
-is_enabled_range() ->
-    false.
-
-%% NOTE: because erlang_ls does not send incremental document changes
-%%       via `textDocument/didChange`, this kind of formatting does not
-%%       make sense.
--spec is_enabled_on_type() -> document_ontypeformatting_options().
-is_enabled_on_type() -> false.
-
 -spec handle_request(any()) -> {response, any()}.
 handle_request({document_formatting, Params}) ->
     #{
@@ -66,6 +45,9 @@ handle_request({document_rangeformatting, Params}) ->
     {ok, Document} = els_utils:lookup_document(Uri),
     {ok, TextEdit} = rangeformat_document(Uri, Document, Range, Options),
     {response, TextEdit};
+%% NOTE: because erlang_ls does not send incremental document changes
+%%       via `textDocument/didChange`, this kind of formatting does not
+%%       make sense.
 handle_request({document_ontypeformatting, Params}) ->
     #{
         <<"position">> := #{