Add --editor-mode flag when invoking rubocop

Since RuboCop 1.61.0 (released in February 2024), RuboCop accepts an
`--editor-mode` flag which improves editor integrations like ale.

Some of RuboCop's auto-corrections can be surprising or annoying to run
on save. When RuboCop is running via an LSP or when the `--editor-mode`
flag is passed, it will understand that it is running in an editor, and
it will hold off on making changes that might be surprising or annoying.

For example, if I write

```ruby
def call
  results = some_process
end
```

This has an unused variable, and RuboCop will remove the unused variable
when you run it. However, if you're in the middle of editing, you may
not want it to remove that unused variable, because you may be about to
add a usage of it.

More context:

- PR which introduced it: rubocop/rubocop#12682
- Release notes for 1.61: https://github.com/rubocop/rubocop/releases/tag/v1.61.0
- Docs: https://docs.rubocop.org/rubocop/1.80/configuration.html#contextual

This will be a breaking change for anyone who is running an old version
of RuboCop, because the flag will not exist for them. If they would like
to opt out of this change, they can set an option to omit the flag. I
think this ought to be enabled by default so that people will get this
benefit out of the box.

In the meantime, I am opting into this behavior by setting this option:

```vim
let g:ale_ruby_rubocop_options = "--editor-mode"
```

So I appreciate that this seam was already introduced.

Author: maxjacobson

autoload/ale/fixers/rubocop.vim

@@ -1,6 +1,7 @@
 call ale#Set('ruby_rubocop_options', '')
 call ale#Set('ruby_rubocop_auto_correct_all', 0)
 call ale#Set('ruby_rubocop_executable', 'rubocop')
+call ale#Set('ruby_rubocop_editor_mode', '1')
 
 " Rubocop fixer outputs diagnostics first and then the fixed
 " output. These are delimited by a "=======" string that we
@@ -23,10 +24,12 @@ function! ale#fixers#rubocop#GetCommand(buffer) abort
     let l:executable = ale#Var(a:buffer, 'ruby_rubocop_executable')
     let l:options = ale#Var(a:buffer, 'ruby_rubocop_options')
     let l:auto_correct_all = ale#Var(a:buffer, 'ruby_rubocop_auto_correct_all')
+    let l:editor_mode= ale#Var(a:buffer, 'ruby_rubocop_editor_mode')
 
     return ale#ruby#EscapeExecutable(l:executable, 'rubocop')
     \   . (!empty(l:options) ? ' ' . l:options : '')
     \   . (l:auto_correct_all ? ' --auto-correct-all' : ' --auto-correct')
+    \   . (l:editor_mode ? ' --editor-mode' : '')
     \   . ' --force-exclusion --stdin %s'
 endfunction
 

doc/ale-ruby.txt

@@ -187,6 +187,18 @@ g:ale_ruby_rubocop_auto_correct_all
   This variable can be changed to make rubocop to correct all offenses (unsafe).
 
 
+                                    *ale-options.ruby_rubocop_editor_mode*
+                                          *g:ale_ruby_rubocop_editor_mode*
+                                          *b:ale_ruby_rubocop_editor_mode*
+ruby_rubocop_editor_mode
+g:ale_ruby_rubocop_editor_mode
+  Type: |Number|
+  Default: `1`
+
+  This variable can be changed to 0 to run rubocop without the `--editor-mode`
+  flag (if using RuboCop < v1.61.0).
+
+
 ===============================================================================
 ruby                                                            *ale-ruby-ruby*
 

test/fixers/test_rubocop_fixer_callback.vader

@@ -20,7 +20,7 @@ Execute(The rubocop callback should return the correct default values):
   \ {
   \   'process_with': 'ale#fixers#rubocop#PostProcess',
   \   'command': ale#Escape(g:ale_ruby_rubocop_executable)
-  \     . ' --auto-correct --force-exclusion --stdin %s',
+  \     . ' --auto-correct --editor-mode --force-exclusion --stdin %s',
   \ },
   \ ale#fixers#rubocop#Fix(bufnr(''))
 
@@ -33,7 +33,7 @@ Execute(The rubocop callback should include custom rubocop options):
   \   'process_with': 'ale#fixers#rubocop#PostProcess',
   \   'command': ale#Escape(g:ale_ruby_rubocop_executable)
   \     . ' --except Lint/Debugger'
-  \     . ' --auto-correct --force-exclusion --stdin %s',
+  \     . ' --auto-correct --editor-mode --force-exclusion --stdin %s',
   \ },
   \ ale#fixers#rubocop#Fix(bufnr(''))
 
@@ -45,7 +45,7 @@ Execute(The rubocop callback should use auto-correct-all option when set):
   \ {
   \   'process_with': 'ale#fixers#rubocop#PostProcess',
   \   'command': ale#Escape(g:ale_ruby_rubocop_executable)
-  \     . ' --auto-correct-all --force-exclusion --stdin %s'
+  \     . ' --auto-correct-all --editor-mode --force-exclusion --stdin %s'
   \ },
   \ ale#fixers#rubocop#Fix(bufnr(''))
 
@@ -87,3 +87,15 @@ Execute(The rubocop post-processor should remove diagnostics content):
   \   '         ''forrest'',',
   \   '         ''run'']',
   \ ])
+
+Execute(The rubocop callback should not use editor-mode option when configured not to):
+  let g:ale_ruby_rubocop_editor_mode = 0
+  call ale#test#SetFilename('../test-files/ruby/with_config/dummy.rb')
+
+  AssertEqual
+  \ {
+  \   'process_with': 'ale#fixers#rubocop#PostProcess',
+  \   'command': ale#Escape(g:ale_ruby_rubocop_executable)
+  \     . ' --auto-correct-all --force-exclusion --stdin %s'
+  \ },
+  \ ale#fixers#rubocop#Fix(bufnr(''))