Pick up changes from .rubocop-todo.yml

Author: AlexVPopov

lib/ruby_lsp/server.rb

@@ -310,7 +310,7 @@ def run_initialize(message)
           @current_request_id,
           Interface::RelativePattern.new(
             base_uri: @global_state.workspace_uri.to_s,
-            pattern: "{.rubocop.yml,.rubocop}",
+            pattern: "{.rubocop.yml,.rubocop,.rubocop_todo.yml}",
           ),
           registration_id: "rubocop-watcher",
         ))
@@ -1058,7 +1058,7 @@ def workspace_did_change_watched_files(message)
 
         file_name = File.basename(file_path)
 
-        if file_name == ".rubocop.yml" || file_name == ".rubocop"
+        if file_name == ".rubocop.yml" || file_name == ".rubocop" || file_name == ".rubocop_todo.yml"
           handle_rubocop_config_change(uri)
         end
       end

test/server_test.rb

@@ -502,6 +502,10 @@ def test_changed_file_only_indexes_ruby
               uri: URI("file:///.rubocop.yml"),
               type: RubyLsp::Constant::FileChangeType::CREATED,
             },
+            {
+              uri: URI("file:///.rubocop_todo.yml"),
+              type: RubyLsp::Constant::FileChangeType::CREATED,
+            },
           ],
         },
       })
@@ -619,23 +623,11 @@ def version
   end
 
   def test_did_change_watched_files_processes_unique_change_entries
-    @server.global_state.index.index_all(uris: [])
-    @server.expects(:handle_rubocop_config_change).once
-    @server.process_message({
-      method: "workspace/didChangeWatchedFiles",
-      params: {
-        changes: [
-          {
-            uri: URI::Generic.from_path(path: File.join(Dir.pwd, ".rubocop.yml")).to_s,
-            type: RubyLsp::Constant::FileChangeType::CHANGED,
-          },
-          {
-            uri: URI::Generic.from_path(path: File.join(Dir.pwd, ".rubocop.yml")).to_s,
-            type: RubyLsp::Constant::FileChangeType::CHANGED,
-          },
-        ],
-      },
-    })
+    assert_rubocop_config_handles_unique_change_entries(".rubocop.yml")
+  end
+
+  def test_did_change_watched_files_processes_unique_change_entries_for_rubocop_todo
+    assert_rubocop_config_handles_unique_change_entries(".rubocop_todo.yml")
   end
 
   def test_workspace_addons
@@ -1123,37 +1115,11 @@ def test_edits_outside_of_declarations_do_not_trigger_indexing
   end
 
   def test_rubocop_config_changes_trigger_workspace_diagnostic_refresh
-    uri = URI::Generic.from_path(path: File.join(Dir.pwd, ".rubocop.yml"))
-
-    @server.process_message({
-      id: 1,
-      method: "initialize",
-      params: {
-        initializationOptions: {},
-        capabilities: {
-          general: {
-            positionEncodings: ["utf-8"],
-          },
-          workspace: { diagnostics: { refreshSupport: true } },
-        },
-      },
-    })
-
-    @server.global_state.index.index_all(uris: [])
-    @server.process_message({
-      method: "workspace/didChangeWatchedFiles",
-      params: {
-        changes: [
-          {
-            uri: uri,
-            type: RubyLsp::Constant::FileChangeType::CHANGED,
-          },
-        ],
-      },
-    })
+    assert_rubocop_config_triggers_diagnostic_refresh(".rubocop.yml")
+  end
 
-    request = find_message(RubyLsp::Request)
-    assert_equal("workspace/diagnostic/refresh", request.method)
+  def test_rubocop_todo_config_changes_trigger_workspace_diagnostic_refresh
+    assert_rubocop_config_triggers_diagnostic_refresh(".rubocop_todo.yml")
   end
 
   def test_compose_bundle_creates_file_to_skip_next_compose
@@ -1712,6 +1678,58 @@ def deactivate; end
     end
   end
 
+  def assert_rubocop_config_handles_unique_change_entries(config_file)
+    @server.global_state.index.index_all(uris: [])
+    @server.expects(:handle_rubocop_config_change).once
+    @server.process_message({
+      method: "workspace/didChangeWatchedFiles",
+      params: {
+        changes: [
+          {
+            uri: URI::Generic.from_path(path: File.join(Dir.pwd, config_file)).to_s,
+            type: RubyLsp::Constant::FileChangeType::CHANGED,
+          },
+          {
+            uri: URI::Generic.from_path(path: File.join(Dir.pwd, config_file)).to_s,
+            type: RubyLsp::Constant::FileChangeType::CHANGED,
+          },
+        ],
+      },
+    })
+  end
+
+  def assert_rubocop_config_triggers_diagnostic_refresh(config_file)
+    @server.process_message({
+      id: 1,
+      method: "initialize",
+      params: {
+        initializationOptions: {},
+        capabilities: {
+          general: {
+            positionEncodings: ["utf-8"],
+          },
+          workspace: { diagnostics: { refreshSupport: true } },
+        },
+      },
+    })
+
+    @server.global_state.index.index_all(uris: [])
+    @server.process_message({
+      method: "workspace/didChangeWatchedFiles",
+      params: {
+        changes: [
+          {
+            uri: URI::Generic.from_path(path: File.join(Dir.pwd, config_file)).to_s,
+            type: RubyLsp::Constant::FileChangeType::CHANGED,
+          },
+        ],
+      },
+    })
+
+    request = find_message(RubyLsp::Request)
+    assert_equal("workspace/diagnostic/refresh", request.method)
+  end
+
   #: (Class desired_class, ?String? desired_method, ?id: Integer?) -> untyped
   def find_message(desired_class, desired_method = nil, id: nil)
     message = @server.pop_response