Address PR feedback

DESIGN_AND_ROADMAP.md

@@ -221,7 +221,7 @@ Interested in contributing? Check out the issues tagged with [help-wanted] or [g
 
 ## Guessed types
 
-Guessed types is an experimental features where the Ruby LSP attempts to identify the type of a receiver based on its
+Guessed types is an experimental feature where the Ruby LSP attempts to identify the type of a receiver based on its
 identifier name. For example:
 
 ```ruby

lib/ruby_lsp/listeners/completion.rb

@@ -295,10 +295,11 @@ def complete_methods(node, name)
           )
         end
 
-        guessed_type = type.is_a?(TypeInferrer::GuessedType)
+        guessed_type = type.name
 
         @index.method_completion_candidates(method_name, type.name).each do |entry|
           entry_name = entry.name
+          owner_name = entry.owner&.name
 
           label_details = Interface::CompletionItemLabelDetails.new(
             description: entry.file_name,
@@ -311,7 +312,7 @@ def complete_methods(node, name)
             text_edit: Interface::TextEdit.new(range: range, new_text: entry_name),
             kind: Constant::CompletionItemKind::METHOD,
             data: {
-              owner_name: entry.owner&.name,
+              owner_name: owner_name,
               guessed_type: guessed_type,
             },
           )

lib/ruby_lsp/listeners/hover.rb

@@ -115,9 +115,8 @@ def on_call_node_enter(node)
         title = +"#{message}#{T.must(methods.first).decorated_parameters}"
 
         if type.is_a?(TypeInferrer::GuessedType)
-          title << " | guessed receiver: #{type.name}"
-          link = "https://github.com/Shopify/ruby-lsp/blob/main/DESIGN_AND_ROADMAP.md#guessed-types"
-          @response_builder.push("[Learn more about guessed types](#{link})\n", category: :links)
+          title << "\n\nGuessed receiver: #{type.name}"
+          @response_builder.push("[Learn more about guessed types](#{GUESSED_TYPES_URL})\n", category: :links)
         end
 
         categorized_markdown_from_index_entries(title, methods).each do |category, content|

lib/ruby_lsp/listeners/signature_help.rb

@@ -64,9 +64,8 @@ def on_call_node_enter(node)
         title = +""
 
         extra_links = if type.is_a?(TypeInferrer::GuessedType)
-          title << "guessed receiver: #{type.name}"
-          link = "https://github.com/Shopify/ruby-lsp/blob/main/DESIGN_AND_ROADMAP.md#guessed-types"
-          "[Learn more about guessed types](#{link})"
+          title << "\n\nGuessed receiver: #{type.name}"
+          "[Learn more about guessed types](#{GUESSED_TYPES_URL})"
         end
 
         signature_help = Interface::SignatureHelp.new(

lib/ruby_lsp/requests/completion_resolve.rb

@@ -63,10 +63,11 @@ def perform
           label = +"#{label}#{first_entry.decorated_parameters}"
         end
 
-        extra_links = if @item.dig(:data, :guessed_type)
-          label << " | guessed receiver: #{owner_name}"
-          link = "https://github.com/Shopify/ruby-lsp/blob/main/DESIGN_AND_ROADMAP.md#guessed-types"
-          "[Learn more about guessed types](#{link})"
+        guessed_type = @item.dig(:data, :guessed_type)
+
+        extra_links = if guessed_type
+          label << "\n\nGuessed receiver: #{guessed_type}"
+          "[Learn more about guessed types](#{GUESSED_TYPES_URL})"
         end
 
         @item[:documentation] = Interface::MarkupContent.new(

lib/ruby_lsp/utils.rb

@@ -25,6 +25,7 @@ module RubyLsp
     end,
     String,
   )
+  GUESSED_TYPES_URL = "https://github.com/Shopify/ruby-lsp/blob/main/DESIGN_AND_ROADMAP.md#guessed-types"
 
   # A notification to be sent to the client
   class Message

test/requests/completion_resolve_test.rb

@@ -122,13 +122,13 @@ def foo(a, b, c)
       existing_item = {
         label: "foo",
         kind: RubyLsp::Constant::CompletionItemKind::METHOD,
-        data: { owner_name: "User", guessed_type: true },
+        data: { owner_name: "User", guessed_type: "User" },
       }
 
       server.process_message(id: 1, method: "completionItem/resolve", params: existing_item)
 
       result = server.pop_response.response
-      assert_match("guessed receiver: User", result[:documentation].value)
+      assert_match("Guessed receiver: User", result[:documentation].value)
       assert_match("Learn more about guessed types", result[:documentation].value)
     end
   end

test/requests/hover_expectations_test.rb

@@ -574,7 +574,7 @@ def name; end
       )
 
       contents = server.pop_response.response.contents.value
-      assert_match("guessed receiver: User", contents)
+      assert_match("Guessed receiver: User", contents)
       assert_match("Learn more about guessed types", contents)
     end
   end

test/requests/signature_help_test.rb

@@ -366,7 +366,7 @@ def subscribe!(news_letter)
       signature = result.signatures.first
 
       assert_equal("subscribe!(news_letter)", signature.label)
-      assert_match("guessed receiver: User", signature.documentation.value)
+      assert_match("Guessed receiver: User", signature.documentation.value)
     end
   end
 end