ruby-lsp-rails 0.2.4 → 0.2.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 79a5ef8571e1fd77b8bbd08a782ef68f1116ce4b71e206eee890d573dd2b90dd
-  data.tar.gz: de8459cb09768c703b0f864f008238a40b78210acdef0e9eba8f4d4fd37c65d7
+  metadata.gz: 70557479de3e98dce68fda3bf76ee6ee6a547ee2d123b7e26bf687306bdd1879
+  data.tar.gz: 1c4560bc452b582ef4c40f793977ef4854cb4e8877bbd3850dbc1aa34b9e4f4a
 SHA512:
-  metadata.gz: f0160d705fba4c081925b29efce3b2d8cc9a8243e8b9e53fa8a7820ffe62b5f0ecf12c73d5546b38f3289d807018f7bc7cbd82acbc296a2151c342ffcbf709be
-  data.tar.gz: 7d033dea267d94c5068bbbf4698e2c5b8bab63708021f78c67a90eed6b2e43c7a48e731a68fa5bd7c8d3a0c49a55eb600b14f1d40001f70fb050cbb5a538a7d4
+  metadata.gz: 01c4c4e000f1d0fb7996ed31a9474bc36a0ddabebdfe3d71dc276572172f0421be818037fb5d001bb77a8bd48cfd58466d1c3431595b8aad071466011f4e4d02
+  data.tar.gz: 937340398445803a2413513a739c771bc9e21ae41813bd5529c3c68c654a2714803a0a9cf3d6d850df279188d8ef2d565767f1f157481091e5c8be891e3c0af5

data/README.md

@@ -1,6 +1,6 @@
 # Ruby LSP Rails
 
-Ruby LSP Rails is a [Ruby LSP](https://github.com/Shopify/ruby-lsp) extension for extra Rails editor features, such as:
+Ruby LSP Rails is a [Ruby LSP](https://github.com/Shopify/ruby-lsp) addon for extra Rails editor features, such as:
 
 - Displaying an ActiveRecord model's database columns and types when hovering over it
 - Running tests and debugging tests through the terminal or the editor's UI
@@ -56,7 +56,7 @@ cause the test runner to hang.
 This gem consists of two components that enable enhanced Rails functionality in the editor:
 
 1. A Rack app that automatically exposes APIs when Rails server is running
-1. A Ruby LSP extension that connects to the exposed APIs to fetch runtime information from the Rails server
+1. A Ruby LSP addon that connects to the exposed APIs to fetch runtime information from the Rails server
 
 This is why the Rails server needs to be running for some features to work.
 

data/lib/ruby_lsp/ruby_lsp_rails/{extension.rb → addon.rb}

@@ -1,7 +1,7 @@
 # typed: strict
 # frozen_string_literal: true
 
-require "ruby_lsp/extension"
+require "ruby_lsp/addon"
 
 require_relative "rails_client"
 require_relative "hover"
@@ -9,7 +9,7 @@ require_relative "code_lens"
 
 module RubyLsp
   module Rails
-    class Extension < ::RubyLsp::Extension
+    class Addon < ::RubyLsp::Addon
       extend T::Sig
 
       sig { returns(RailsClient) }
@@ -39,11 +39,13 @@ module RubyLsp
 
       sig do
         override.params(
+          nesting: T::Array[String],
+          index: RubyIndexer::Index,
           emitter: EventEmitter,
           message_queue: Thread::Queue,
         ).returns(T.nilable(Listener[T.nilable(Interface::Hover)]))
       end
-      def create_hover_listener(emitter, message_queue)
+      def create_hover_listener(nesting, index, emitter, message_queue)
         Hover.new(client, emitter, message_queue)
       end
 

data/lib/ruby_lsp/ruby_lsp_rails/code_lens.rb

@@ -40,53 +40,51 @@ module RubyLsp
       BASE_COMMAND = "bin/rails test"
 
       sig { override.returns(ResponseType) }
-      attr_reader :response
+      attr_reader :_response
 
       sig { params(uri: URI::Generic, emitter: EventEmitter, message_queue: Thread::Queue).void }
       def initialize(uri, emitter, message_queue)
-        @response = T.let([], ResponseType)
+        @_response = T.let([], ResponseType)
         @path = T.let(uri.to_standardized_path, T.nilable(String))
-        emitter.register(self, :on_command, :on_class, :on_def)
+        emitter.register(self, :on_call, :on_class, :on_def)
 
         super(emitter, message_queue)
       end
 
-      sig { params(node: SyntaxTree::Command).void }
-      def on_command(node)
-        message_value = node.message.value
-        return unless message_value == "test" && node.arguments.parts.any?
+      sig { params(node: YARP::CallNode).void }
+      def on_call(node)
+        message_value = node.message
+        return unless message_value == "test"
 
-        first_argument = node.arguments.parts.first
+        arguments = node.arguments&.arguments
+        return unless arguments&.any?
 
-        parts = case first_argument
-        when SyntaxTree::StringConcat
+        first_argument = arguments.first
+
+        content = case first_argument
+        when YARP::StringConcatNode
+          left = first_argument.left
+          right = first_argument.right
           # We only support two lines of concatenation on test names
-          if first_argument.left.is_a?(SyntaxTree::StringLiteral) &&
-              first_argument.right.is_a?(SyntaxTree::StringLiteral)
-            [*first_argument.left.parts, *first_argument.right.parts]
+          if left.is_a?(YARP::StringNode) &&
+              right.is_a?(YARP::StringNode)
+            left.content + right.content
           end
-        when SyntaxTree::StringLiteral
-          first_argument.parts
+        when YARP::StringNode
+          first_argument.content
         end
 
-        # The test name may be a blank string while the code is being typed
-        return if parts.nil? || parts.empty?
-
-        # We can't handle interpolation yet
-        return unless parts.all? { |part| part.is_a?(SyntaxTree::TStringContent) }
-
-        test_name = parts.map(&:value).join
-        return if test_name.empty?
+        return unless content && !content.empty?
 
         line_number = node.location.start_line
         command = "#{BASE_COMMAND} #{@path}:#{line_number}"
-        add_test_code_lens(node, name: test_name, command: command, kind: :example)
+        add_test_code_lens(node, name: content, command: command, kind: :example)
       end
 
       # Although uncommon, Rails tests can be written with the classic "def test_name" syntax.
-      sig { params(node: SyntaxTree::DefNode).void }
+      sig { params(node: YARP::DefNode).void }
       def on_def(node)
-        method_name = node.name.value
+        method_name = node.name.to_s
         if method_name.start_with?("test_")
           line_number = node.location.start_line
           command = "#{BASE_COMMAND} #{@path}:#{line_number}"
@@ -94,9 +92,9 @@ module RubyLsp
         end
       end
 
-      sig { params(node: SyntaxTree::ClassDeclaration).void }
+      sig { params(node: YARP::ClassNode).void }
       def on_class(node)
-        class_name = node.constant.constant.value
+        class_name = node.constant_path.slice
         if class_name.end_with?("Test")
           command = "#{BASE_COMMAND} #{@path}"
           add_test_code_lens(node, name: class_name, command: command, kind: :group)
@@ -105,7 +103,7 @@ module RubyLsp
 
       private
 
-      sig { params(node: SyntaxTree::Node, name: String, command: String, kind: Symbol).void }
+      sig { params(node: YARP::Node, name: String, command: String, kind: Symbol).void }
       def add_test_code_lens(node, name:, command:, kind:)
         return unless @path
 
@@ -121,7 +119,7 @@ module RubyLsp
           },
         ]
 
-        @response << create_code_lens(
+        @_response << create_code_lens(
           node,
           title: "Run",
           command_name: "rubyLsp.runTest",
@@ -129,7 +127,7 @@ module RubyLsp
           data: { type: "test", kind: kind },
         )
 
-        @response << create_code_lens(
+        @_response << create_code_lens(
           node,
           title: "Run In Terminal",
           command_name: "rubyLsp.runTestInTerminal",
@@ -137,7 +135,7 @@ module RubyLsp
           data: { type: "test_in_terminal", kind: kind },
         )
 
-        @response << create_code_lens(
+        @_response << create_code_lens(
           node,
           title: "Debug",
           command_name: "rubyLsp.debugTest",

data/lib/ruby_lsp/ruby_lsp_rails/hover.rb

@@ -23,20 +23,25 @@ module RubyLsp
       ResponseType = type_member { { fixed: T.nilable(::RubyLsp::Interface::Hover) } }
 
       sig { override.returns(ResponseType) }
-      attr_reader :response
+      attr_reader :_response
 
       sig { params(client: RailsClient, emitter: RubyLsp::EventEmitter, message_queue: Thread::Queue).void }
       def initialize(client, emitter, message_queue)
         super(emitter, message_queue)
 
-        @response = T.let(nil, ResponseType)
+        @_response = T.let(nil, ResponseType)
         @client = client
-        emitter.register(self, :on_const, :on_command, :on_const_path_ref, :on_call)
+        emitter.register(self, :on_constant_path, :on_constant_read, :on_call)
       end
 
-      sig { params(node: SyntaxTree::Const).void }
-      def on_const(node)
-        model = @client.model(node.value)
+      sig { params(node: YARP::ConstantPathNode).void }
+      def on_constant_path(node)
+        @_response = generate_rails_document_link_hover(node.slice, node.location)
+      end
+
+      sig { params(node: YARP::ConstantReadNode).void }
+      def on_constant_read(node)
+        model = @client.model(node.name.to_s)
         return if model.nil?
 
         schema_file = model[:schema_file]
@@ -46,37 +51,28 @@ module RubyLsp
         end
         content << model[:columns].map { |name, type| "**#{name}**: #{type}\n" }.join("\n")
         contents = RubyLsp::Interface::MarkupContent.new(kind: "markdown", value: content)
-        @response = RubyLsp::Interface::Hover.new(range: range_from_syntax_tree_node(node), contents: contents)
-      end
-
-      sig { params(node: SyntaxTree::Command).void }
-      def on_command(node)
-        message = node.message
-        @response = generate_rails_document_link_hover(message.value, message)
+        @_response = RubyLsp::Interface::Hover.new(range: range_from_node(node), contents: contents)
       end
 
-      sig { params(node: SyntaxTree::ConstPathRef).void }
-      def on_const_path_ref(node)
-        @response = generate_rails_document_link_hover(full_constant_name(node), node)
-      end
-
-      sig { params(node: SyntaxTree::CallNode).void }
+      sig { params(node: YARP::CallNode).void }
       def on_call(node)
-        message = node.message
-        return if message.is_a?(Symbol)
+        message_value = node.message
+        message_loc = node.message_loc
+
+        return unless message_value && message_loc
 
-        @response = generate_rails_document_link_hover(message.value, message)
+        @_response = generate_rails_document_link_hover(message_value, message_loc)
       end
 
       private
 
-      sig { params(name: String, node: SyntaxTree::Node).returns(T.nilable(Interface::Hover)) }
-      def generate_rails_document_link_hover(name, node)
+      sig { params(name: String, location: YARP::Location).returns(T.nilable(Interface::Hover)) }
+      def generate_rails_document_link_hover(name, location)
         urls = Support::RailsDocumentClient.generate_rails_document_urls(name)
         return if urls.empty?
 
         contents = RubyLsp::Interface::MarkupContent.new(kind: "markdown", value: urls.join("\n\n"))
-        RubyLsp::Interface::Hover.new(range: range_from_syntax_tree_node(node), contents: contents)
+        RubyLsp::Interface::Hover.new(range: range_from_location(location), contents: contents)
       end
     end
   end

data/lib/ruby_lsp_rails/railtie.rb

@@ -1,7 +1,7 @@
 # typed: strict
 # frozen_string_literal: true
 
-require "rails/railtie"
+require "rails"
 require "ruby_lsp_rails/rack_app"
 
 module RubyLsp
@@ -27,8 +27,8 @@ module RubyLsp
             app_uri_path.write(app_uri)
 
             at_exit do
-              # The app_uri.txt file should only exist when the server is running. The extension uses its presence to
-              # report if the server is running or not. If the server is not running, some of the extension features
+              # The app_uri.txt file should only exist when the server is running. The addon uses its presence to
+              # report if the server is running or not. If the server is not running, some of the addon features
               # will not be available.
               File.delete(app_uri_path) if File.exist?(app_uri_path)
             end

data/lib/ruby_lsp_rails/version.rb

@@ -3,6 +3,6 @@
 
 module RubyLsp
   module Rails
-    VERSION = "0.2.4"
+    VERSION = "0.2.6"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ruby-lsp-rails
 version: !ruby/object:Gem::Version
-  version: 0.2.4
+  version: 0.2.6
 platform: ruby
 authors:
 - Shopify
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-08-30 00:00:00.000000000 Z
+date: 2023-10-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -30,20 +30,20 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.9.1
+        version: 0.11.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: 0.10.0
+        version: 0.12.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.9.1
+        version: 0.11.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: 0.10.0
+        version: 0.12.0
 - !ruby/object:Gem::Dependency
   name: sorbet-runtime
   requirement: !ruby/object:Gem::Requirement
@@ -58,7 +58,7 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 0.5.9897
-description: A Ruby LSP extension that adds extra editor functionality for Rails applications
+description: A Ruby LSP addon that adds extra editor functionality for Rails applications
 email:
 - ruby@shopify.com
 executables: []
@@ -69,8 +69,8 @@ files:
 - README.md
 - Rakefile
 - lib/ruby-lsp-rails.rb
+- lib/ruby_lsp/ruby_lsp_rails/addon.rb
 - lib/ruby_lsp/ruby_lsp_rails/code_lens.rb
-- lib/ruby_lsp/ruby_lsp_rails/extension.rb
 - lib/ruby_lsp/ruby_lsp_rails/hover.rb
 - lib/ruby_lsp/ruby_lsp_rails/rails_client.rb
 - lib/ruby_lsp/ruby_lsp_rails/support/rails_document_client.rb
@@ -101,8 +101,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.18
+rubygems_version: 3.4.20
 signing_key: 
 specification_version: 4
-summary: A Ruby LSP extension for Rails
+summary: A Ruby LSP addon for Rails
 test_files: []