solargraph 0.59.0.dev.1 → 0.59.0

data/lib/solargraph/pin/search.rb

@@ -46,7 +46,7 @@ module Solargraph
           # @param b [self]
           # @sg-ignore https://github.com/castwide/solargraph/pull/1050
           .sort { |a, b| b.match <=> a.match }
-          .map(&:pin)
+             .map(&:pin)
       end
 
       # @param str1 [String]
@@ -54,7 +54,7 @@ module Solargraph
       #
       # @return [Float]
       def fuzzy_string_match str1, str2
-        return 1.0 + (str2.length.to_f / str1.length.to_f) if str1.downcase.include?(str2.downcase)
+        return 1.0 + (str2.length.to_f / str1.length) if str1.downcase.include?(str2.downcase)
         JaroWinkler.similarity(str1, str2, ignore_case: true)
       end
     end

data/lib/solargraph/pin/signature.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Solargraph
   module Pin
     class Signature < Callable
@@ -5,10 +7,6 @@ module Solargraph
       # to the method pin
       attr_writer :closure
 
-      def initialize **splat
-        super(**splat)
-      end
-
       def generics
         # @type [Array<::String, nil>]
         @generics ||= [].freeze
@@ -18,8 +16,6 @@ module Solargraph
         @identity ||= "signature#{object_id}"
       end
 
-      attr_writer :closure
-
       # @ sg-ignore need boolish support for ? methods
       def dodgy_return_type_source?
         super || closure&.dodgy_return_type_source?
@@ -49,16 +45,15 @@ module Solargraph
         method_stack = closure.rest_of_stack api_map
         logger.debug { "Signature#typify(self=#{self}) - method_stack: #{method_stack}" }
         method_stack.each do |pin|
-          sig = pin.signatures.find { |s| s.arity == self.arity }
+          sig = pin.signatures.find { |s| s.arity == arity }
           next unless sig
           # @sg-ignore Need to add nil check here
-          unless sig.return_type.undefined?
-            # @sg-ignore Need to add nil check here
-            qualified = sig.return_type.qualify(api_map, closure.namespace)
-            # @sg-ignore Need to add nil check here
-            logger.debug { "Signature#typify(self=#{self}) => #{qualified.rooted_tags.inspect}" }
-            return qualified
-          end
+          next if sig.return_type.undefined?
+          # @sg-ignore Need to add nil check here
+          qualified = sig.return_type.qualify(api_map, closure.namespace)
+          # @sg-ignore Need to add nil check here
+          logger.debug { "Signature#typify(self=#{self}) => #{qualified.rooted_tags.inspect}" }
+          return qualified
         end
         out = super
         logger.debug { "Signature#typify(self=#{self}) => #{out}" }

data/lib/solargraph/pin/symbol.rb

@@ -5,6 +5,7 @@ module Solargraph
     class Symbol < Base
       # @param location [Solargraph::Location, nil]
       # @param name [String]
+      # @param [Hash{Symbol => Object}] kwargs
       def initialize(location, name, **kwargs)
         # @sg-ignore "Unrecognized keyword argument kwargs to Solargraph::Pin::Base#initialize"
         super(location: location, name: name, **kwargs)

data/lib/solargraph/pin/until.rb

@@ -5,10 +5,8 @@ module Solargraph
     class Until < CompoundStatement
       include Breakable
 
-      # @param receiver [Parser::AST::Node, nil]
       # @param node [Parser::AST::Node, nil]
-      # @param context [ComplexType, nil]
-      # @param args [::Array<Parameter>]
+      # @param [Hash{Symbol => Object}] splat
       def initialize node: nil, **splat
         super(**splat)
         @node = node

data/lib/solargraph/pin/while.rb

@@ -5,10 +5,8 @@ module Solargraph
     class While < CompoundStatement
       include Breakable
 
-      # @param receiver [Parser::AST::Node, nil]
       # @param node [Parser::AST::Node, nil]
-      # @param context [ComplexType, nil]
-      # @param args [::Array<Parameter>]
+      # @param [Hash{Symbol => Object}] splat
       def initialize node: nil, **splat
         super(**splat)
         @node = node