run rubocop lint via stdin

Author: joe-re

lib/language_server.rb

@@ -91,11 +91,10 @@ def on(method, &callback)
 
   on :"textDocument/didChange" do |request:, notifier:, file_store:, project:|
     uri = request[:params][:textDocument][:uri]
-    filePath = uri.match(/^file:\/\/(.*\.rb)/)[1]
     text = request[:params][:contentChanges][0][:text]
     file_store.cache(uri, text)
     project.recalculate_result(uri)
-    diagnostics = (Linter::Rubocop.new(filePath).call + Linter::RubyWC.new(text).call).flatten
+    diagnostics = (Linter::Rubocop.new(text).call + Linter::RubyWC.new(text).call).flatten
 
     diagnostics = diagnostics.map do |error|
       Protocol::Interface::Diagnostic.new(

lib/language_server/linter/rubocop.rb

@@ -6,26 +6,29 @@
 module LanguageServer
   module Linter
     class Rubocop
-      def initialize(path, config_path="")
-        @path = path
+      def initialize(source, config_path="")
+        @source = source
         @config_path = config_path
       end
 
 
       def call
         return [] unless defined? ::RuboCop
-        args = ["--format", "json", @path]
+        args = []
         args += ["--config", @config_path] if @config_path != ''
-        options, paths = ::RuboCop::Options.new.parse(args)
-        config_store = ::RuboCop::ConfigStore.new
-        config_store.options_config = options[:config] if options[:config]
+        args += ["--format", "json","--stdin", "lsp_buffer.rb"]
         o = nil
         begin
+          $stdin = StringIO.new(@source)
           $stdout = StringIO.new
+          config_store = ::RuboCop::ConfigStore.new
+          options, paths = ::RuboCop::Options.new.parse(args)
+          config_store.options_config = options[:config] if options[:config]
           runner = ::RuboCop::Runner.new(options, config_store)
           runner.run(paths)
           o = $stdout.string
         ensure
+          $stdin = STDIN
           $stdout = STDOUT
         end
         return [] unless o

test/language_server/linter/rubocop_test.rb

@@ -7,13 +7,8 @@ def setup
     end
 
     def test_fatal_error
-      file = Tempfile.open(['bar_', '.rb']) do |v|
-        v.puts 'require "foo'
-        v
-      end
-
-      actual = Rubocop.new(file.path, @config_path).call
-
+      source = 'require "foo'
+      actual = Rubocop.new(source, @config_path).call
       assert do
         actual == [
           Error.new(
@@ -26,26 +21,16 @@ def test_fatal_error
     end
 
     def test_warning_error
-      file = Tempfile.open(['bar_', '.rb']) do |v|
-        v.puts "a = 'a'"
-        v
-      end
-
-      actual = Rubocop.new(file.path, @config_path).call
-
+      source = "a = 'a'"
+      actual = Rubocop.new(source, @config_path).call
       assert do
         actual.first == Error.new(line_num: 0, message: "Useless assignment to variable - `a`.", type: 'warning')
       end
     end
 
     def test_convention_error
-      file = Tempfile.open(['bar_', '.rb']) do |v|
-        v.puts 'require "foo"'
-        v
-      end
-
-      actual = Rubocop.new(file.path, @config_path).call
-
+      source = 'require "foo"'
+      actual = Rubocop.new(source, @config_path).call
       assert do
         actual.first == Error.new(line_num: 0, message: "Prefer single-quoted strings when you don't need string interpolation or special symbols.", type: 'warning')
       end