clean up is_enabled/0 callbacks

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() ->
     #{

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">> := #{

apps/els_lsp/src/els_general_provider.erl

@@ -2,7 +2,6 @@
 
 -behaviour(els_provider).
 -export([
-    is_enabled/0,
     default_providers/0,
     enabled_providers/0,
     handle_request/1
@@ -51,9 +50,6 @@
 %%==============================================================================
 %% els_provider functions
 %%==============================================================================
--spec is_enabled() -> boolean().
-is_enabled() -> true.
-
 -spec handle_request(
     initialize_request()
     | initialized_request()
@@ -183,34 +179,22 @@ server_capabilities() ->
                     triggerCharacters =>
                         els_signature_help_provider:trigger_characters()
                 },
-            definitionProvider =>
-                els_definition_provider:is_enabled(),
-            referencesProvider =>
-                els_references_provider:is_enabled(),
-            documentHighlightProvider =>
-                els_document_highlight_provider:is_enabled(),
-            documentSymbolProvider =>
-                els_document_symbol_provider:is_enabled(),
-            workspaceSymbolProvider =>
-                els_workspace_symbol_provider:is_enabled(),
-            codeActionProvider =>
-                els_code_action_provider:is_enabled(),
-            documentFormattingProvider =>
-                els_formatting_provider:is_enabled_document(),
-            documentRangeFormattingProvider =>
-                els_formatting_provider:is_enabled_range(),
-            foldingRangeProvider =>
-                els_folding_range_provider:is_enabled(),
-            implementationProvider =>
-                els_implementation_provider:is_enabled(),
+            definitionProvider => true,
+            referencesProvider => true,
+            documentHighlightProvider => true,
+            documentSymbolProvider => true,
+            workspaceSymbolProvider => true,
+            codeActionProvider => true,
+            documentFormattingProvider => true,
+            documentRangeFormattingProvider => false,
+            foldingRangeProvider => true,
+            implementationProvider => true,
             executeCommandProvider =>
                 els_execute_command_provider:options(),
             codeLensProvider =>
                 els_code_lens_provider:options(),
-            renameProvider =>
-                els_rename_provider:is_enabled(),
-            callHierarchyProvider =>
-                els_call_hierarchy_provider:is_enabled()
+            renameProvider => true,
+            callHierarchyProvider => true
         },
     EnabledProviders = enabled_providers(),
     ConfiguredCapabilities =

apps/els_lsp/src/els_hover_provider.erl

@@ -6,7 +6,6 @@
 -behaviour(els_provider).
 
 -export([
-    is_enabled/0,
     handle_request/1
 ]).
 
@@ -20,10 +19,6 @@
 %%==============================================================================
 %% els_provider functions
 %%==============================================================================
--spec is_enabled() -> boolean().
-is_enabled() ->
-    true.
-
 -spec handle_request(any()) -> {async, uri(), pid()}.
 handle_request({hover, Params}) ->
     #{

apps/els_lsp/src/els_implementation_provider.erl

@@ -5,16 +5,12 @@
 -include("els_lsp.hrl").
 
 -export([
-    is_enabled/0,
     handle_request/1
 ]).
 
 %%==============================================================================
 %% els_provider functions
 %%==============================================================================
--spec is_enabled() -> boolean().
-is_enabled() -> true.
-
 -spec handle_request(tuple()) -> {response, [location()]}.
 handle_request({implementation, Params}) ->
     #{

apps/els_lsp/src/els_references_provider.erl

@@ -3,7 +3,6 @@
 -behaviour(els_provider).
 
 -export([
-    is_enabled/0,
     handle_request/1
 ]).
 
@@ -26,9 +25,6 @@
 %%==============================================================================
 %% els_provider functions
 %%==============================================================================
--spec is_enabled() -> boolean().
-is_enabled() -> true.
-
 -spec handle_request(any()) -> {response, [location()] | null}.
 handle_request({references, Params}) ->
     #{

apps/els_lsp/src/els_rename_provider.erl

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

apps/els_lsp/src/els_signature_help_provider.erl

@@ -6,7 +6,6 @@
 -include_lib("kernel/include/logger.hrl").
 
 -export([
-    is_enabled/0,
     handle_request/1,
     trigger_characters/0
 ]).
@@ -22,10 +21,6 @@ trigger_characters() ->
 %%==============================================================================
 %% els_provider functions
 %%==============================================================================
--spec is_enabled() -> boolean().
-is_enabled() ->
-    false.
-
 -spec handle_request(els_provider:provider_request()) ->
     {response, signature_help() | null}.
 handle_request({signature_help, Params}) ->

apps/els_lsp/src/els_workspace_symbol_provider.erl

@@ -3,7 +3,6 @@
 -behaviour(els_provider).
 
 -export([
-    is_enabled/0,
     handle_request/1
 ]).
 
@@ -14,9 +13,6 @@
 %%==============================================================================
 %% els_provider functions
 %%==============================================================================
--spec is_enabled() -> boolean().
-is_enabled() -> true.
-
 -spec handle_request(any()) -> {response, any()}.
 handle_request({symbol, Params}) ->
     %% TODO: Version 3.15 of the protocol introduces a much nicer way of