ruby-lsp 0.14.2 → 0.14.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 293c03761c9b7d497045546e3bf7ca88a08ed5556ad0ba52c205a9798e820ce5
-  data.tar.gz: 38157f012f3f1dc944186ac1530dc660d8662cb59ca7b12ad3b117655e0c1709
+  metadata.gz: 4fc74feb692f2c2e902e213dc61a4b3d58f14bcc4520fc7db60bdae022484e6f
+  data.tar.gz: 98f6d5b939bb945119169e2ca3ce060f275b961d421e752774ebea1c25b74c2d
 SHA512:
-  metadata.gz: c4996e277088f015004df0f82e2cc47dc2f979f6658f9dfac19fe495c8c4deced3028b021e330ac0681dbfd79048e9033037c7ce93e6cc044c8ae6d91087e0df
-  data.tar.gz: 67c5c31c97c279f192b0b3d6c729c97a19f7aac4581eca037c0e5355cc0f3d3ed69fb4007a5f32985e046980324575e2e35a92ce81e8078b41ee5b6db711ac88
+  metadata.gz: 8c33776eb483463d1052cdd5a82d1f66d5d507325b09848d0c5d78cea310c218b056bef834f6f6f03e17d87088b65c8cd77f7c214735390e8b5d244a059ea9e7
+  data.tar.gz: 7e57548a8dede5695774c27137b6ab23c8029f8ff8c0551bef8b1cf8f6dd920e9bdb423d4905c6a7f9ac360433da99cdec31efc885a120f4c1684acbd3da9582

data/LICENSE.txt

@@ -19,4 +19,3 @@ AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
 LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
 OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 THE SOFTWARE.
-

data/README.md

@@ -1,3 +1,7 @@
+<p align="center">
+  <img alt="Ruby LSP logo" width="200" src="vscode/icon.png" />
+</p>
+
 [![Build Status](https://github.com/Shopify/ruby-lsp/workflows/CI/badge.svg)](https://github.com/Shopify/ruby-lsp/actions/workflows/ci.yml)
 [![Ruby LSP extension](https://img.shields.io/badge/VS%20Code-Ruby%20LSP-success?logo=visual-studio-code)](https://marketplace.visualstudio.com/items?itemName=Shopify.ruby-lsp)
 [![Ruby DX Slack](https://img.shields.io/badge/Slack-Ruby%20DX-success?logo=slack)](https://join.slack.com/t/ruby-dx/shared_invite/zt-2c8zjlir6-uUDJl8oIwcen_FS_aA~b6Q)
@@ -11,12 +15,38 @@ experience to Ruby developers using modern standards for cross-editor features,
 Want to discuss Ruby developer experience? Consider joining the public
 [Ruby DX Slack workspace](https://join.slack.com/t/ruby-dx/shared_invite/zt-2c8zjlir6-uUDJl8oIwcen_FS_aA~b6Q).
 
+## Features
+
+![Ruby LSP demo](vscode/extras/ruby_lsp_demo.gif)
+
+The Ruby LSP features include
+
+- Semantic highlighting
+- Symbol search and code outline
+- RuboCop errors and warnings (diagnostics)
+- Format on save (with RuboCop or Syntax Tree)
+- Format on type
+- Debugging support
+- Running and debugging tests through VS Code's UI
+- Go to definition for classes, modules, constants and required files
+- Showing documentaton on hover for classes, modules and constants
+- Completion for classes, modules, constants and require paths
+- Fuzzy search classes, modules and constants anywhere in the project and its dependencies (workspace symbol)
+
+Adding method support for definition, completion, hover and workspace symbol is planned, but not yet completed.
+
+See complete information about features [here](https://shopify.github.io/ruby-lsp/RubyLsp/Requests.html).
+
+If you experience issues, please see the [troubleshooting
+guide](https://github.com/Shopify/ruby-lsp/blob/main/TROUBLESHOOTING.md).
+
 ## Usage
 
 ### With VS Code
 
-If using VS Code, all you have to do is install the [Ruby LSP extension](https://github.com/Shopify/vscode-ruby-lsp) to
-get the extra features in the editor. Do not install this gem manually.
+If using VS Code, all you have to do is install the [Ruby LSP
+extension](https://marketplace.visualstudio.com/items?itemName=Shopify.ruby-lsp) to get the extra features in the
+editor. Do not install the `ruby-lsp` gem manually.
 
 ### With other editors
 
@@ -27,16 +57,8 @@ The gem can be installed by doing
 gem install ruby-lsp
 ```
 
-**NOTE**: starting with v0.7.0, it is no longer recommended to add the `ruby-lsp` to the bundle. The gem will generate a
-custom bundle in `.ruby-lsp/Gemfile` which is used to identify the versions of dependencies that should be used for the
-application (e.g.: the correct RuboCop version).
-
-For older versions, if you decide to add the gem to the bundle, it is not necessary to require it.
-```ruby
-group :development do
-  gem "ruby-lsp", require: false
-end
-```
+and the language server can be launched running `ruby-lsp` (without bundle exec in order to properly hook into your
+project's dependencies).
 
 ### Documentation
 
@@ -54,7 +76,7 @@ default gems, except for
 - Gems that only appear under the `:development` group
 - All Ruby files under `test/**/*.rb`
 
-By creating a `.index.yml` file, these configurations can be overridden and tuned.
+By creating a `.index.yml` file, these configurations can be overridden and tuned. Note that indexing dependent behavior, such as definition, hover, completion or workspace symbol will be impacted by the configurations placed here.
 
 ```yaml
 # Exclude files based on a given pattern. Often used to exclude test files or fixtures
@@ -98,16 +120,13 @@ For instructions on how to create addons, see the [addons documentation](ADDONS.
 
 ## Contributing
 
-Bug reports and pull requests are welcome on GitHub at https://github.com/Shopify/ruby-lsp.
-This project is intended to be a safe, welcoming space for collaboration, and contributors
-are expected to adhere to the
-[Contributor Covenant](CODE_OF_CONDUCT.md)
-code of conduct.
+Bug reports and pull requests are welcome on GitHub at https://github.com/Shopify/ruby-lsp.  This project is intended to
+be a safe, welcoming space for collaboration, and contributors are expected to adhere to the [Contributor
+Covenant](CODE_OF_CONDUCT.md) code of conduct.
 
 If you wish to contribute, see [CONTRIBUTING](CONTRIBUTING.md) for development instructions and check out our pinned
 [roadmap issue](https://github.com/Shopify/ruby-lsp/issues) for a list of tasks to get started.
 
 ## License
 
-The gem is available as open source under the terms of the
-[MIT License](LICENSE.txt).
+The gem is available as open source under the terms of the [MIT License](LICENSE.txt).

data/VERSION

@@ -1 +1 @@
-0.14.2
+0.14.4

data/exe/ruby-lsp

@@ -94,13 +94,6 @@ if options[:debug]
     exit 1
   end
 
-  sockets_dir = "/tmp/ruby-lsp-debug-sockets"
-  Dir.mkdir(sockets_dir) unless Dir.exist?(sockets_dir)
-  # ruby-debug-ENV["USER"] is an implicit naming pattern in ruby/debug
-  # if it's not present, rdbg will not find the socket
-  socket_identifier = "ruby-debug-#{ENV["USER"]}-#{File.basename(Dir.pwd)}.sock"
-  ENV["RUBY_DEBUG_SOCK_PATH"] = "#{sockets_dir}/#{socket_identifier}"
-
   begin
     require "debug/open_nonstop"
   rescue LoadError

data/lib/ruby_lsp/executor.rb

@@ -121,24 +121,10 @@ module RubyLsp
         begin
           formatting(uri)
         rescue Requests::Formatting::InvalidFormatter => error
-          @message_queue << Notification.new(
-            message: "window/showMessage",
-            params: Interface::ShowMessageParams.new(
-              type: Constant::MessageType::ERROR,
-              message: "Configuration error: #{error.message}",
-            ),
-          )
-
+          @message_queue << Notification.window_show_error("Configuration error: #{error.message}")
           nil
         rescue StandardError, LoadError => error
-          @message_queue << Notification.new(
-            message: "window/showMessage",
-            params: Interface::ShowMessageParams.new(
-              type: Constant::MessageType::ERROR,
-              message: "Formatting error: #{error.message}",
-            ),
-          )
-
+          @message_queue << Notification.window_show_error("Formatting error: #{error.message}")
           nil
         end
       when "textDocument/documentHighlight"
@@ -174,14 +160,7 @@ module RubyLsp
         begin
           diagnostic(uri)
         rescue StandardError, LoadError => error
-          @message_queue << Notification.new(
-            message: "window/showMessage",
-            params: Interface::ShowMessageParams.new(
-              type: Constant::MessageType::ERROR,
-              message: "Error running diagnostics: #{error.message}",
-            ),
-          )
-
+          @message_queue << Notification.window_show_error("Error running diagnostics: #{error.message}")
           nil
         end
       when "textDocument/completion"
@@ -287,13 +266,7 @@ module RubyLsp
             false
           end
         rescue StandardError => error
-          @message_queue << Notification.new(
-            message: "window/showMessage",
-            params: Interface::ShowMessageParams.new(
-              type: Constant::MessageType::ERROR,
-              message: "Error while indexing: #{error.message}",
-            ),
-          )
+          @message_queue << Notification.window_show_error("Error while indexing: #{error.message}")
         end
 
         # Always end the progress notification even if indexing failed or else it never goes away and the user has no
@@ -402,21 +375,11 @@ module RubyLsp
 
       case result
       when Requests::CodeActionResolve::Error::EmptySelection
-        @message_queue << Notification.new(
-          message: "window/showMessage",
-          params: Interface::ShowMessageParams.new(
-            type: Constant::MessageType::ERROR,
-            message: "Invalid selection for Extract Variable refactor",
-          ),
-        )
+        @message_queue << Notification.window_show_error("Invalid selection for Extract Variable refactor")
         raise Requests::CodeActionResolve::CodeActionError
       when Requests::CodeActionResolve::Error::InvalidTargetRange
-        @message_queue << Notification.new(
-          message: "window/showMessage",
-          params: Interface::ShowMessageParams.new(
-            type: Constant::MessageType::ERROR,
-            message: "Couldn't find an appropriate location to place extracted refactor",
-          ),
+        @message_queue << Notification.window_show_error(
+          "Couldn't find an appropriate location to place extracted refactor",
         )
         raise Requests::CodeActionResolve::CodeActionError
       else
@@ -639,12 +602,8 @@ module RubyLsp
       unless defined?(RubyLsp::Requests::Support::RuboCopRunner)
         @store.formatter = "none"
 
-        @message_queue << Notification.new(
-          message: "window/showMessage",
-          params: Interface::ShowMessageParams.new(
-            type: Constant::MessageType::ERROR,
-            message: "Ruby LSP formatter is set to `rubocop` but RuboCop was not found in the Gemfile or gemspec.",
-          ),
+        @message_queue << Notification.window_show_error(
+          "Ruby LSP formatter is set to `rubocop` but RuboCop was not found in the Gemfile or gemspec.",
         )
       end
     end

data/lib/ruby_lsp/listeners/completion.rb

@@ -14,30 +14,24 @@ module RubyLsp
           nesting: T::Array[String],
           typechecker_enabled: T::Boolean,
           dispatcher: Prism::Dispatcher,
+          uri: URI::Generic,
         ).void
       end
-      def initialize(response_builder, index, nesting, typechecker_enabled, dispatcher)
+      def initialize(response_builder, index, nesting, typechecker_enabled, dispatcher, uri) # rubocop:disable Metrics/ParameterLists
         @response_builder = response_builder
         @index = index
         @nesting = nesting
         @typechecker_enabled = typechecker_enabled
+        @uri = uri
 
         dispatcher.register(
           self,
-          :on_string_node_enter,
           :on_constant_path_node_enter,
           :on_constant_read_node_enter,
           :on_call_node_enter,
         )
       end
 
-      sig { params(node: Prism::StringNode).void }
-      def on_string_node_enter(node)
-        @index.search_require_paths(node.content).map!(&:require_path).sort!.each do |path|
-          @response_builder << build_completion(T.must(path), node)
-        end
-      end
-
       # Handle completion on regular constant references (e.g. `Bar`)
       sig { params(node: Prism::ConstantReadNode).void }
       def on_constant_read_node_enter(node)
@@ -107,11 +101,64 @@ module RubyLsp
       sig { params(node: Prism::CallNode).void }
       def on_call_node_enter(node)
         return if @typechecker_enabled
-        return unless self_receiver?(node)
 
         name = node.message
         return unless name
 
+        case name
+        when "require"
+          complete_require(node)
+        when "require_relative"
+          complete_require_relative(node)
+        else
+          complete_self_receiver_method(node, name) if self_receiver?(node)
+        end
+      end
+
+      private
+
+      sig { params(node: Prism::CallNode).void }
+      def complete_require(node)
+        arguments_node = node.arguments
+        return unless arguments_node
+
+        path_node_to_complete = arguments_node.arguments.first
+
+        return unless path_node_to_complete.is_a?(Prism::StringNode)
+
+        @index.search_require_paths(path_node_to_complete.content).map!(&:require_path).sort!.each do |path|
+          @response_builder << build_completion(T.must(path), path_node_to_complete)
+        end
+      end
+
+      sig { params(node: Prism::CallNode).void }
+      def complete_require_relative(node)
+        arguments_node = node.arguments
+        return unless arguments_node
+
+        path_node_to_complete = arguments_node.arguments.first
+
+        return unless path_node_to_complete.is_a?(Prism::StringNode)
+
+        origin_dir = Pathname.new(@uri.to_standardized_path).dirname
+
+        path_query = path_node_to_complete.content
+        # if the path is not a directory, glob all possible next characters
+        # for example ../somethi| (where | is the cursor position)
+        # should find files for ../somethi*/
+        path_query += "*/" unless path_query.end_with?("/")
+        path_query += "**/*.rb"
+
+        Dir.glob(path_query, base: origin_dir).sort!.each do |path|
+          @response_builder << build_completion(
+            path.delete_suffix(".rb"),
+            path_node_to_complete,
+          )
+        end
+      end
+
+      sig { params(node: Prism::CallNode, name: String).void }
+      def complete_self_receiver_method(node, name)
         receiver_entries = @index[@nesting.join("::")]
         return unless receiver_entries
 
@@ -125,8 +172,6 @@ module RubyLsp
         end
       end
 
-      private
-
       sig do
         params(
           entry: RubyIndexer::Entry::Member,
@@ -139,7 +184,7 @@ module RubyLsp
         Interface::CompletionItem.new(
           label: name,
           filter_text: name,
-          text_edit: Interface::TextEdit.new(range: range_from_node(node), new_text: name),
+          text_edit: Interface::TextEdit.new(range: range_from_location(T.must(node.message_loc)), new_text: name),
           kind: Constant::CompletionItemKind::METHOD,
           label_details: Interface::CompletionItemLabelDetails.new(
             detail: "(#{entry.parameters.map(&:decorated_name).join(", ")})",

data/lib/ruby_lsp/requests/completion.rb

@@ -68,26 +68,13 @@ module RubyLsp
           ResponseBuilders::CollectionResponseBuilder[Interface::CompletionItem],
         )
 
-        Listeners::Completion.new(@response_builder, index, nesting, typechecker_enabled, dispatcher)
+        Listeners::Completion.new(@response_builder, index, nesting, typechecker_enabled, dispatcher, document.uri)
 
         return unless matched && parent
 
         @target = case matched
         when Prism::CallNode
-          message = matched.message
-
-          if message == "require"
-            args = matched.arguments&.arguments
-            return if args.nil? || args.is_a?(Prism::ForwardingArgumentsNode)
-
-            argument = args.first
-            return unless argument.is_a?(Prism::StringNode)
-            return unless (argument.location.start_offset..argument.location.end_offset).cover?(char_position)
-
-            argument
-          else
-            matched
-          end
+          matched
         when Prism::ConstantReadNode, Prism::ConstantPathNode
           if parent.is_a?(Prism::ConstantPathNode) && matched.is_a?(Prism::ConstantReadNode)
             parent

data/lib/ruby_lsp/requests/diagnostics.rb

@@ -42,27 +42,61 @@ module RubyLsp
 
       sig { override.returns(T.nilable(T.all(T::Array[Interface::Diagnostic], Object))) }
       def perform
+        diagnostics = []
+        diagnostics.concat(syntax_error_diagnostics, syntax_warning_diagnostics)
+
         # Running RuboCop is slow, so to avoid excessive runs we only do so if the file is syntactically valid
-        return syntax_error_diagnostics if @document.syntax_error?
-        return [] unless defined?(Support::RuboCopDiagnosticsRunner)
+        return diagnostics if @document.syntax_error?
+
+        diagnostics.concat(
+          Support::RuboCopDiagnosticsRunner.instance.run(
+            @uri,
+            @document,
+          ).map!(&:to_lsp_diagnostic),
+        ) if defined?(Support::RuboCopDiagnosticsRunner)
 
-        Support::RuboCopDiagnosticsRunner.instance.run(@uri, @document).map!(&:to_lsp_diagnostic)
+        diagnostics
       end
 
       private
 
-      sig { returns(T.nilable(T::Array[Interface::Diagnostic])) }
+      sig { returns(T::Array[Interface::Diagnostic]) }
+      def syntax_warning_diagnostics
+        @document.parse_result.warnings.map do |warning|
+          location = warning.location
+
+          Interface::Diagnostic.new(
+            source: "Prism",
+            message: warning.message,
+            severity: Constant::DiagnosticSeverity::WARNING,
+            range: Interface::Range.new(
+              start: Interface::Position.new(
+                line: location.start_line - 1,
+                character: location.start_column,
+              ),
+              end: Interface::Position.new(
+                line: location.end_line - 1,
+                character: location.end_column,
+              ),
+            ),
+          )
+        end
+      end
+
+      sig { returns(T::Array[Interface::Diagnostic]) }
       def syntax_error_diagnostics
         @document.parse_result.errors.map do |error|
+          location = error.location
+
           Interface::Diagnostic.new(
             range: Interface::Range.new(
               start: Interface::Position.new(
-                line: error.location.start_line - 1,
-                character: error.location.start_column,
+                line: location.start_line - 1,
+                character: location.start_column,
               ),
               end: Interface::Position.new(
-                line: error.location.end_line - 1,
-                character: error.location.end_column,
+                line: location.end_line - 1,
+                character: location.end_column,
               ),
             ),
             message: error.message,

data/lib/ruby_lsp/requests/on_type_formatting.rb

@@ -178,7 +178,7 @@ module RubyLsp
 
       sig { params(line: Integer, character: Integer).void }
       def move_cursor_to(line, character)
-        return if @client_name != "Visual Studio Code"
+        return unless @client_name.start_with?("Visual Studio Code")
 
         position = Interface::Position.new(
           line: line,

data/lib/ruby_lsp/utils.rb

@@ -41,7 +41,22 @@ module RubyLsp
     end
   end
 
-  class Notification < Message; end
+  class Notification < Message
+    class << self
+      extend T::Sig
+      sig { params(message: String).returns(Notification) }
+      def window_show_error(message)
+        new(
+          message: "window/showMessage",
+          params: Interface::ShowMessageParams.new(
+            type: Constant::MessageType::ERROR,
+            message: message,
+          ),
+        )
+      end
+    end
+  end
+
   class Request < Message; end
 
   # The final result of running a request before its IO is finalized

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ruby-lsp
 version: !ruby/object:Gem::Version
-  version: 0.14.2
+  version: 0.14.4
 platform: ruby
 authors:
 - Shopify
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2024-02-22 00:00:00.000000000 Z
+date: 2024-03-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: language_server-protocol