solargraph 0.41.0 → 0.42.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: eed08215481576c83b9ff5014648c0fa5fe7e458eb59eabcf86632496e4a9f32
-  data.tar.gz: 8f036f65304ecf1401504a84a49bc34ed6d9be97308be8a1e080be884af100fc
+  metadata.gz: d4cd4acd35a7ce8242976cdd2b23d583bbd01392e3d79a16e3b4e181443e4858
+  data.tar.gz: 4cb4f369d97681a43d24a8fc3de14a454721c4334426655e23fffafceeb242c7
 SHA512:
-  metadata.gz: '024920c42f1f1e37f604b616481bca07b91ec52ed9fd315e6e12a6cc0ab7c0a88ef9ecd827866ab52d66d2f43b679a97c68d3877b88adc5092fc9c2dc2dca5a1'
-  data.tar.gz: 02c6de30fdac85feedbee8e8b3cbd35ea846a9cbb3a5cb8c476053208a63ae5781a45382fcca43e760875f3f5f911675bdec3f8ba318b122ef6ea0c42f735a29
+  metadata.gz: 5918eb0b9797ed3bed1fe97a702761889b4fb5b8280de655820d8a4319eefd651f4958e420b32468aac2ab197f657b8cb1386029654dcbd9b3a30bbdd2ca521b
+  data.tar.gz: 649a461beb8b23b69ae40bb347389602e20b42d0a689752d805beb2c08000a863c24d9e45b39d00ee544d37802be0762d9933a6d4e27b2617249d1443ca582f4

data/.travis.yml

@@ -8,15 +8,10 @@ rvm:
   - jruby-head
 matrix:
   include:
-    - rvm: 2.4
-      os: osx
-    - rvm: 2.6
-      os: osx
     - rvm: 2.7
       os: osx
   allow_failures:
     - rvm: jruby-head
-    - rvm: 3.0
 before_install:
 - gem update --system
 - gem install bundler

data/CHANGELOG.md

@@ -1,3 +1,27 @@
+## 0.42.2 - June 14, 2021
+- Improve download-core command output
+- Ignore missing requests to client responses
+- Add automatically required gems to YardMap
+- Use closures to identify local variables
+
+## 0.42.1 - June 11, 2021
+- YardMap#change sets new directory (#445)
+
+## 0.42.0 - June 11, 2021
+- Improve YardMap efficiency
+- Bench includes Workspace for cataloging
+- Initialize confirms static features from options (#436)
+- Enable simple repairs without incremental sync (#416)
+- Discard unrecognized client responses
+- Notify on use of closest match for core (#390)
+
+## 0.41.2 - June 9, 2021
+- Rescue InvalidOffset in async diagnosis
+- Remove erroneous escaping from Hover
+
+## 0.41.1 - May 31, 2021
+- ApiMap handles required bundles (#443)
+
 ## 0.41.0 - May 30, 2021
 - Chain constant at last double colon with more than two nested namespaces
 - Fill Integer#times return type (#440)

data/lib/.rubocop.yml

@@ -1,5 +1,5 @@
-Layout/EndOfLine:
-  Enabled: false
+AllCops:
+  NewCops: enable
 Style/MethodDefParentheses:
   Enabled: false
 Layout/EmptyLineAfterGuardClause:
@@ -10,6 +10,8 @@ Lint/RaiseException:
   Enabled: true
 Lint/StructNewOverride:
   Enabled: true
+Metrics/MethodLength:
+  Max: 25
 Style/ExponentialNotation:
   Enabled: true
 Style/HashEachMethods:
@@ -18,4 +20,3 @@ Style/HashTransformKeys:
   Enabled: true
 Style/HashTransformValues:
   Enabled: true
-

data/lib/solargraph/api_map.rb

@@ -17,7 +17,6 @@ module Solargraph
     autoload :BundlerMethods, 'solargraph/api_map/bundler_methods'
 
     include SourceToYard
-    include BundlerMethods
 
     # @return [Array<String>]
     attr_reader :unresolved_requires
@@ -59,30 +58,14 @@ module Solargraph
       implicit.clear
       @cache.clear
       @source_map_hash = bench.source_maps.map { |s| [s.filename, s] }.to_h
-      pins = []
-      @required = Set.new
-      local_path_hash.clear
+      pins = bench.source_maps.map(&:pins).flatten
+      external_requires = bench.external_requires
       source_map_hash.each_value do |map|
-        pins.concat map.pins
-        @required.merge map.requires.map(&:name)
         implicit.merge map.environ
       end
-      @required.merge implicit.requires
-      external_requires = []
-      @required.each do |req|
-        result = bench.load_paths.find do |path|
-          full = Pathname.new(path).join("#{req}.rb").to_s
-          @source_map_hash.key?(full)
-        end
-        if result
-          local_path_hash[req] = Pathname.new(result).join("#{req}.rb").to_s
-        else
-          external_requires.push req unless result
-        end
-      end
-      br = @required.include?('bundler/require') ? bench.gemnames.to_h { |gs| [gs.name, gs] } : {}
-      @required.merge br.keys
-      yard_map.change(external_requires, br, bench.gemnames)
+      external_requires.merge implicit.requires
+      external_requires.merge bench.workspace.config.required
+      yard_map.change(external_requires, bench.workspace.directory, bench.workspace.source_gems)
       @store = Store.new(yard_map.pins + implicit.pins + pins)
       @unresolved_requires = yard_map.unresolved_requires
       @rebindable_method_names = nil
@@ -105,11 +88,6 @@ module Solargraph
       @implicit ||= Environ.new
     end
 
-    # @return [Hash{String => String}]
-    def local_path_hash
-      @local_paths ||= {}
-    end
-
     # @param filename [String]
     # @param position [Position, Array(Integer, Integer)]
     # @return [Source::Cursor]
@@ -456,27 +434,6 @@ module Solargraph
       source_map_hash.keys.include?(filename)
     end
 
-    # True if the specified file is included in the workspace.
-    #
-    # @param filename [String]
-    def workspaced? filename
-      workspace_filenames.include?(filename)
-    end
-
-    # @param location [Location]
-    # @return [Location]
-    def require_reference_at location
-      map = source_map(location.filename)
-      pin = map.requires.select { |p| p.location.range.contain?(location.range.start) }.first
-      return nil if pin.nil?
-      if local_path_hash.key?(pin.name)
-        return Location.new(local_path_hash[pin.name], Solargraph::Range.from_to(0, 0, 0, 0))
-      end
-      yard_map.require_reference(pin.name)
-    rescue FileNotFoundError
-      nil
-    end
-
     # Check if a class is a superclass of another class.
     #
     # @param sup [String] The superclass
@@ -508,11 +465,6 @@ module Solargraph
 
     private
 
-    # @return [Array<String>]
-    def workspace_filenames
-      @workspace_filenames ||= []
-    end
-
     # A hash of source maps with filename keys.
     #
     # @return [Hash{String => SourceMap}]

data/lib/solargraph/api_map/bundler_methods.rb

@@ -9,7 +9,7 @@ module Solargraph
       # @param directory [String]
       # @return [Hash]
       def require_from_bundle directory
-        @require_from_bundle ||= begin
+        begin
           Solargraph.logger.info "Loading gems for bundler/require"
           Documentor.specs_from_bundle(directory)
         rescue BundleNotFoundError => e
@@ -17,11 +17,6 @@ module Solargraph
           {}
         end
       end
-
-      # @return [void]
-      def reset_require_from_bundle
-        @require_from_bundle = nil
-      end
     end
   end
 end

data/lib/solargraph/bench.rb

@@ -3,25 +3,25 @@
 require 'set'
 
 module Solargraph
-  # A container of source maps and gem specs to be cataloged in an ApiMap.
+  # A container of source maps and workspace data to be cataloged in an ApiMap.
   #
   class Bench
     # @return [Set<SourceMap>]
     attr_reader :source_maps
 
-    # @return [Set<String>]
-    attr_reader :load_paths
+    # @return [Workspace]
+    attr_reader :workspace
 
     # @return [Set<String>]
-    attr_reader :gemnames
+    attr_reader :external_requires
 
     # @param source_maps [Array<SourceMap>, Set<SourceMap>]
-    # @param load_paths [Array<String>, Set<String>]
-    # @param gemnames [Array<String>, Set<String>]
-    def initialize source_maps: [], load_paths: [], gemnames: []
+    # @param workspace [Workspace]
+    # @param external_requires [Array<String>, Set<String>]
+    def initialize source_maps: [], workspace: Workspace.new, external_requires: []
       @source_maps = source_maps.to_set
-      @load_paths = load_paths.to_set
-      @gemnames = gemnames.to_set
+      @workspace = workspace
+      @external_requires = external_requires.to_set
     end
   end
 end

data/lib/solargraph/diagnostics/type_check.rb

@@ -7,7 +7,7 @@ module Solargraph
     #
     class TypeCheck < Base
       def diagnose source, api_map
-        return [] unless args.include?('always') || api_map.workspaced?(source.filename)
+        # return [] unless args.include?('always') || api_map.workspaced?(source.filename)
         severity = Diagnostics::Severities::ERROR
         level = (args.reverse.find { |a| ['normal', 'typed', 'strict', 'strong'].include?(a) }) || :normal
         checker = Solargraph::TypeChecker.new(source.filename, api_map: api_map, level: level.to_sym)

data/lib/solargraph/language_server/host.rb

@@ -1,8 +1,9 @@
 # frozen_string_literal: true
 
+require 'diff/lcs'
 require 'observer'
-require 'set'
 require 'securerandom'
+require 'set'
 
 module Solargraph
   module LanguageServer
@@ -30,7 +31,7 @@ module Solargraph
         @cancel = []
         @buffer = String.new
         @stopped = true
-        @next_request_id = 0
+        @next_request_id = 1
         @dynamic_capabilities = Set.new
         @registered_capabilities = Set.new
       end
@@ -107,9 +108,13 @@ module Solargraph
           end
           message
         elsif request['id']
-          # @todo What if the id is invalid?
-          requests[request['id']].process(request['result'])
-          requests.delete request['id']
+          if requests[request['id']]
+            requests[request['id']].process(request['result'])
+            requests.delete request['id']
+          else
+            logger.warn "Discarding client response to unrecognized message #{request['id']}"
+            nil
+          end
         else
           logger.warn "Invalid message received."
           logger.debug request
@@ -165,8 +170,6 @@ module Solargraph
       # @return [void]
       def open_from_disk uri
         sources.open_from_disk(uri)
-        library = library_for(uri)
-        # library.open_from_disk uri_to_file(uri)
         diagnoser.schedule uri
       end
 
@@ -672,7 +675,8 @@ module Solargraph
       # @return [Source::Updater]
       def generate_updater params
         changes = []
-        params['contentChanges'].each do |chng|
+        params['contentChanges'].each do |recvd|
+          chng = check_diff(params['textDocument']['uri'], recvd)
           changes.push Solargraph::Source::Change.new(
             (chng['range'].nil? ? 
               nil :
@@ -688,6 +692,36 @@ module Solargraph
         )
       end
 
+      # @param uri [String]
+      # @param change [Hash]
+      # @return [Hash]
+      def check_diff uri, change
+        return change if change['range']
+        source = sources.find(uri)
+        return change if source.code.length + 1 != change['text'].length
+        diffs = Diff::LCS.diff(source.code, change['text'])
+        return change if diffs.length.zero? || diffs.length > 1 || diffs.first.length > 1
+        # @type [Diff::LCS::Change]
+        diff = diffs.first.first
+        return change unless diff.adding? && ['.', ':'].include?(diff.element)
+        position = Solargraph::Position.from_offset(source.code, diff.position)
+        {
+          'range' => {
+            'start' => {
+              'line' => position.line,
+              'character' => position.character
+            },
+            'end' => {
+              'line' => position.line,
+              'character' => position.character
+            }
+          },
+          'text' => diff.element
+        }
+      rescue Solargraph::FileNotFoundError
+        change
+      end
+
       # @return [Hash]
       def dynamic_capability_options
         @dynamic_capability_options ||= {
@@ -787,7 +821,7 @@ module Solargraph
           next unless uuid
           cur = ((library.source_map_hash.keys.length.to_f / total.to_f) * 100).to_i
           if cur > pct && cur % mod == 0
-          pct = cur
+            pct = cur
             send_notification '$/progress', {
               token: uuid,
               value: {

data/lib/solargraph/language_server/host/diagnoser.rb

@@ -62,7 +62,15 @@ module Solargraph
           end
           current = mutex.synchronize { queue.shift }
           return if queue.include?(current)
-          host.diagnose current
+          begin
+            host.diagnose current
+          rescue InvalidOffsetError
+            # @todo This error can occur when the Source is out of sync with
+            #   with the ApiMap. It's probably not the best way to handle it,
+            #   but it's quick and easy.
+            Logging.logger.warn "Deferring diagnosis due to invalid offset: #{current}"
+            mutex.synchronize { queue.push current }
+          end
         end
 
         private

data/lib/solargraph/language_server/host/sources.rb

@@ -76,7 +76,7 @@ module Solargraph
           source = Solargraph::Source.load(UriHelpers.uri_to_file(uri))
           open_source_hash[uri] = source
         end
-    
+
         # Update an existing source.
         #
         # @raise [FileNotFoundError] if the URI does not match an open source.

data/lib/solargraph/language_server/message/initialize.rb

@@ -51,6 +51,7 @@ module Solargraph
         end
 
         def static_completion
+          return {} unless host.options['completion']
           {
             completionProvider: {
               resolveProvider: true,
@@ -84,18 +85,21 @@ module Solargraph
         end
 
         def static_hover
+          return {} unless host.options['hover']
           {
             hoverProvider: true
           }
         end
 
         def static_document_formatting
+          return {} unless host.options['formatting']
           {
             documentFormattingProvider: true
           }
         end
 
         def static_document_symbols
+          return {} unless host.options['symbols']
           {
             documentSymbolProvider: true
           }
@@ -108,6 +112,7 @@ module Solargraph
         end
 
         def static_definitions
+          return {} unless host.options['definitions']
           {
             definitionProvider: true
           }
@@ -120,12 +125,14 @@ module Solargraph
         end
 
         def static_references
+          return {} unless host.options['references']
           {
             referencesProvider: true
           }
         end
 
         def static_folding_range
+          return {} unless host.options['folding']
           {
             foldingRangeProvider: true
           }

data/lib/solargraph/language_server/message/text_document/hover.rb

@@ -17,7 +17,7 @@ module Solargraph
               if !this_link.nil? && this_link != last_link
                 parts.push this_link
               end
-              parts.push "`" + pin.detail.gsub(':', '\\:') + "`" unless pin.is_a?(Pin::Namespace) || pin.detail.nil?
+              parts.push "`#{pin.detail}`" unless pin.is_a?(Pin::Namespace) || pin.detail.nil?
               parts.push pin.documentation unless pin.documentation.nil? || pin.documentation.empty?
               unless parts.empty?
                 data = parts.join("\n\n")

data/lib/solargraph/language_server/transport/adapter.rb

@@ -42,9 +42,7 @@ module Solargraph
         # @return [void]
         def process request
           message = @host.receive(request)
-          message.send_response
-          # tmp = @host.flush
-          # write tmp unless tmp.empty?
+          message && message.send_response
         end
 
         def shutdown

data/lib/solargraph/library.rb

@@ -46,8 +46,10 @@ module Solargraph
     # @return [void]
     def attach source
       mutex.synchronize do
-        if @current && @current.filename != source.filename && source_map_hash.key?(@current.filename) && !workspace.has_file?(@current.filename)
+        if @current && (!source || @current.filename != source.filename) && source_map_hash.key?(@current.filename) && !workspace.has_file?(@current.filename)
           source_map_hash.delete @current.filename
+          source_map_external_require_hash.delete @current.filename
+          @external_requires = nil
           @synchronized = false
         end
         @current = source
@@ -252,7 +254,18 @@ module Solargraph
     end
 
     def locate_ref location
-      api_map.require_reference_at location
+      map = source_map_hash[location.filename]
+      return if map.nil?
+      pin = map.requires.select { |p| p.location.range.contain?(location.range.start) }.first
+      return nil if pin.nil?
+      workspace.require_paths.each do |path|
+        full = Pathname.new(path).join("#{pin.name}.rb").to_s
+        next unless source_map_hash.key?(full)
+        return Location.new(full, Solargraph::Range.from_to(0, 0, 0, 0))
+      end
+      api_map.yard_map.require_reference(pin.name)
+    rescue FileNotFoundError
+      nil
     end
 
     # Get an array of pins that match a path.
@@ -326,6 +339,7 @@ module Solargraph
       return [] unless open?(filename)
       result = []
       source = read(filename)
+      catalog
       repargs = {}
       workspace.config.reporters.each do |line|
         if line == 'all!'
@@ -361,12 +375,10 @@ module Solargraph
     end
 
     def bench
-      source_maps = @current ? [@current] : []
-      source_maps.concat source_map_hash.values
       Bench.new(
-        source_maps: source_maps,
-        load_paths: workspace.require_paths,
-        gemnames: workspace.gemnames
+        source_maps: source_map_hash.values,
+        workspace: workspace,
+        external_requires: external_requires
       )
     end
 
@@ -422,6 +434,8 @@ module Solargraph
         if src
           Logging.logger.debug "Mapping #{src.filename}"
           source_map_hash[src.filename] = Solargraph::SourceMap.map(src)
+          find_external_requires(source_map_hash[src.filename])
+          source_map_hash[src.filename]
         else
           false
         end
@@ -431,6 +445,7 @@ module Solargraph
     def map!
       workspace.sources.each do |src|
         source_map_hash[src.filename] = Solargraph::SourceMap.map(src)
+        find_external_requires(source_map_hash[src.filename])
       end
       self
     end
@@ -439,8 +454,29 @@ module Solargraph
       @pins ||= []
     end
 
+    def external_requires
+      @external_requires ||= source_map_external_require_hash.values.flatten.to_set
+    end
+
     private
 
+    def source_map_external_require_hash
+      @source_map_external_require_hash ||= {}
+    end
+
+    # @param source_map [SourceMap]
+    def find_external_requires source_map
+      new_set = source_map.requires.map(&:name).to_set
+      # return if new_set == source_map_external_require_hash[source_map.filename]
+      source_map_external_require_hash[source_map.filename] = new_set.reject do |path|
+        workspace.require_paths.any? do |base|
+          full = Pathname.new(base).join("#{path}.rb").to_s
+          workspace.filenames.include?(full)
+        end
+      end
+      @external_requires = nil
+    end
+
     # @return [Mutex]
     def mutex
       @mutex ||= Mutex.new
@@ -475,6 +511,7 @@ module Solargraph
     end
 
     def maybe_map source
+      return unless source
       if source_map_hash.key?(source.filename)
         return if source_map_hash[source.filename].code == source.code && 
           source_map_hash[source.filename].source.synchronized? &&
@@ -483,6 +520,7 @@ module Solargraph
           new_map = Solargraph::SourceMap.map(source)
           unless source_map_hash[source.filename].try_merge!(new_map)
             source_map_hash[source.filename] = new_map
+            find_external_requires(source_map_hash[source.filename])
             @synchronized = false
           end
         else
@@ -491,6 +529,7 @@ module Solargraph
         end
       else
         source_map_hash[source.filename] = Solargraph::SourceMap.map(source)
+        find_external_requires(source_map_hash[source.filename])
         @synchronized = false
       end
     end

data/lib/solargraph/pin/local_variable.rb

@@ -3,7 +3,8 @@
 module Solargraph
   module Pin
     class LocalVariable < BaseVariable
-      include Localized
+      # @return [Range]
+      attr_reader :presence
 
       def initialize assignment: nil, presence: nil, **splat
         super(**splat)
@@ -16,6 +17,51 @@ module Solargraph
         @presence = pin.presence
         true
       end
+
+      # @param other [Pin::Base] The caller's block
+      # @param position [Position, Array(Integer, Integer)] The caller's position
+      # @return [Boolean]
+      def visible_from?(other, position)
+        position = Position.normalize(position)
+        other.filename == filename &&
+          match_tags(other.full_context.tag, full_context.tag) &&
+          (other == closure ||
+            (closure.location.range.contain?(other.location.range.start) && closure.location.range.contain?(other.location.range.ending))
+          ) &&
+          presence.contain?(position)
+      end
+
+      # @param other_loc [Location]
+      def visible_at?(other_closure, other_loc)
+        return true if location.filename == other_loc.filename &&
+          presence.include?(other_loc.range.start) &&
+          match_named_closure(other_closure, closure)
+      end
+
+      private
+
+      # @param tag1 [String]
+      # @param tag2 [String]
+      # @return [Boolean]
+      def match_tags tag1, tag2
+        # @todo This is an unfortunate hack made necessary by a discrepancy in
+        #   how tags indicate the root namespace. The long-term solution is to
+        #   standardize it, whether it's `Class<>`, an empty string, or
+        #   something else.
+        tag1 == tag2 ||
+          (['', 'Class<>'].include?(tag1) && ['', 'Class<>'].include?(tag2))
+      end
+
+      def match_named_closure needle, haystack
+        return true if needle == haystack
+        cursor = haystack
+        until cursor.nil?
+          return true if needle.path == cursor.path
+          return false if cursor.path && !cursor.path.empty?
+          cursor = cursor.closure
+        end
+        false
+      end
     end
   end
 end

data/lib/solargraph/shell.rb

@@ -74,10 +74,14 @@ module Solargraph
     desc 'download-core [VERSION]', 'Download core documentation'
     def download_core version = nil
       ver = version || Solargraph::YardMap::CoreDocs.best_download
+      if RUBY_VERSION != ver
+        puts "Documentation for #{RUBY_VERSION} is not available. Reverting to closest match..."
+      end
       puts "Downloading docs for #{ver}..."
       Solargraph::YardMap::CoreDocs.download ver
       # Clear cached documentation if it exists
       FileUtils.rm_rf Dir.glob(File.join(Solargraph::YardMap::CoreDocs.cache_dir, ver, '*.ser'))
+      puts "Download complete."
     rescue ArgumentError => e
       STDERR.puts "ERROR: #{e.message}"
       STDERR.puts "Run `solargraph available-cores` for a list."

data/lib/solargraph/source/chain/call.rb

@@ -77,6 +77,7 @@ module Solargraph
               end
               if match
                 type = extra_return_type(ol, context)
+                break if type
                 type = ComplexType.try_parse(*ol.tag(:return).types).self_to(context.to_s).qualify(api_map, context.namespace) if ol.has_tag?(:return) && ol.tag(:return).types && !ol.tag(:return).types.empty? && (type.nil? || type.undefined?)
                 type ||= ComplexType::UNDEFINED
               end
@@ -170,8 +171,8 @@ module Solargraph
         # @param context [ComplexType]
         # @return [ComplexType]
         def extra_return_type docstring, context
-          if docstring.has_tag?(:return_single_parameter) && context.subtypes.one?
-            return context.subtypes.first
+          if docstring.has_tag?(:return_single_parameter) #&& context.subtypes.one?
+            return context.subtypes.first || ComplexType::UNDEFINED
           elsif docstring.has_tag?(:return_value_parameter) && context.value_types.one?
             return context.value_types.first
           end

data/lib/solargraph/source_map.rb

@@ -131,7 +131,8 @@ module Solargraph
     # @return [Array<Pin::LocalVariable>]
     def locals_at(location)
       return [] if location.filename != filename
-      locals.select { |pin| pin.visible_at?(location) }
+      closure = locate_named_path_pin(location.range.start.line, location.range.start.character)
+      locals.select { |pin| pin.visible_at?(closure, location) }
     end
 
     class << self

data/lib/solargraph/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Solargraph
-  VERSION = '0.41.0'
+  VERSION = '0.42.2'
 end

data/lib/solargraph/workspace.rb

@@ -18,6 +18,7 @@ module Solargraph
 
     # @return [Array<String>]
     attr_reader :gemnames
+    alias source_gems gemnames
 
     # @param directory [String]
     # @param config [Config, nil]

data/lib/solargraph/yard_map.rb

@@ -20,6 +20,8 @@ module Solargraph
     autoload :Helpers,     'solargraph/yard_map/helpers'
     autoload :ToMethod,    'solargraph/yard_map/to_method'
 
+    include ApiMap::BundlerMethods
+
     CoreDocs.require_minimum
 
     def stdlib_paths
@@ -37,33 +39,16 @@ module Solargraph
       end
     end
 
-    # @return [Array<String>]
-    attr_reader :required
-
     # @return [Boolean]
     attr_writer :with_dependencies
 
-    # A hash of gem names and the version numbers to include in the map.
-    #
-    # @return [Hash{String => String}]
-    attr_reader :gemset
-
-    # @param required [Array<String>]
-    # @param gemset [Hash{String => String}]
+    # @param required [Array<String>, Set<String>]
+    # @param directory [String]
+    # @param source_gems [Array<String>, Set<String>]
     # @param with_dependencies [Boolean]
-    def initialize(required: [], gemset: {}, with_dependencies: true)
-      # HACK: YardMap needs its own copy of this array
-      @required = required.clone
-      # HACK: Hardcoded YAML handling
-      @required.push 'psych' if @required.include?('yaml')
+    def initialize(required: [], directory: '', source_gems: [], with_dependencies: true)
       @with_dependencies = with_dependencies
-      @gem_paths = {}
-      @stdlib_namespaces = []
-      @gemset = gemset
-      @source_gems = []
-      process_requires
-      yardocs.uniq!
-      @pin_select_cache = {}
+      change required.to_set, directory, source_gems.to_set
     end
 
     # @return [Array<Solargraph::Pin::Base>]
@@ -76,25 +61,24 @@ module Solargraph
       @with_dependencies
     end
 
-    # @param new_requires [Array<String>]
-    # @param new_gemset [Hash{String => String}]
+    # @param new_requires [Set<String>] Required paths to use for loading gems
+    # @param new_directory [String] The workspace directory
+    # @param new_source_gems [Set<String>] Gems under local development (i.e., part of the workspace)
     # @return [Boolean]
-    def change new_requires, new_gemset, source_gems = []
+    def change new_requires, new_directory, new_source_gems
+      return false if new_requires == base_required && new_directory == @directory && new_source_gems == @source_gems
+      @gem_paths = {}
+      base_required.replace new_requires
+      required.replace new_requires
       # HACK: Hardcoded YAML handling
-      new_requires.push 'psych' if new_requires.include?('yaml')
-      if new_requires.uniq.sort == required.uniq.sort && new_gemset == gemset && @source_gems.uniq.sort == source_gems.uniq.sort
-        false
-      else
-        required.clear
-        required.concat new_requires
-        @gemset = new_gemset
-        @source_gems = source_gems
-        process_requires
-        @rebindable_method_names = nil
-        @pin_class_hash = nil
-        @pin_select_cache = {}
-        true
-      end
+      required.add 'psych' if new_requires.include?('yaml')
+      @source_gems = new_source_gems
+      @directory = new_directory
+      process_requires
+      @rebindable_method_names = nil
+      @pin_class_hash = nil
+      @pin_select_cache = {}
+      true
     end
 
     # @return [Set<String>]
@@ -111,6 +95,11 @@ module Solargraph
       @yardocs ||= []
     end
 
+    # @return [Set<String>]
+    def required
+      @required ||= Set.new
+    end
+
     # @return [Array<String>]
     def unresolved_requires
       @unresolved_requires ||= []
@@ -163,6 +152,14 @@ module Solargraph
       @stdlib_pins ||= []
     end
 
+    def base_required
+      @base_required ||= Set.new
+    end
+
+    def directory
+      @directory ||= ''
+    end
+
     private
 
     # @return [YardMap::Cache]
@@ -193,6 +190,8 @@ module Solargraph
 
     # @return [void]
     def process_requires
+      @gemset = process_gemsets
+      required.merge @gemset.keys if required.include?('bundler/require')
       pins.replace core_pins
       unresolved_requires.clear
       stdlib_pins.clear
@@ -252,6 +251,11 @@ module Solargraph
       pins.concat environ.pins
     end
 
+    def process_gemsets
+      return {} if directory.empty? || !File.file?(File.join(directory, 'Gemfile'))
+      require_from_bundle(directory)
+    end
+
     # @param spec [Gem::Specification]
     # @return [void]
     def add_gem_dependencies spec

data/lib/solargraph/yard_map/core_fills.rb

@@ -82,6 +82,18 @@ module Solargraph
         Override.method_return('Class#allocate', 'self'),
         Override.method_return('Class.allocate', 'Class<Object>'),
 
+        Override.from_comment('Enumerable#detect', %(
+@overload detect(&block)
+  @return_single_parameter
+@overload detect()
+  @return [Enumerator]
+        )),
+        Override.from_comment('Enumerable#find', %(
+@overload find(&block)
+  @return_single_parameter
+@overload find()
+  @return [Enumerator]
+        )),
         Override.method_return('Enumerable#select', 'self'),
 
         Override.method_return('File.absolute_path', 'String'),
@@ -111,7 +123,12 @@ module Solargraph
 @param y [Numeric]
 @return [Numeric]
         )),
-        Override.method_return('Integer#times', 'Enumerator', delete: [:overload]),
+        Override.from_comment('Integer#times', %(
+@overload times(&block)
+  @return [Integer]
+@overload times()
+  @return [Enumerator]
+        )),
 
         Override.method_return('Kernel#puts', 'nil'),
 

data/solargraph.gemspec

@@ -19,9 +19,10 @@ Gem::Specification.new do |s|
 
   s.required_ruby_version = '>= 2.4'
 
-  s.add_runtime_dependency 'backport', '~> 1.1'
+  s.add_runtime_dependency 'backport', '~> 1.2'
   s.add_runtime_dependency 'benchmark'
   s.add_runtime_dependency 'bundler', '>= 1.17.2'
+  s.add_runtime_dependency 'diff-lcs', '~> 1.4'
   s.add_runtime_dependency 'e2mmap'
   s.add_runtime_dependency 'jaro_winkler', '~> 1.5'
   s.add_runtime_dependency 'kramdown', '~> 2.3'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: solargraph
 version: !ruby/object:Gem::Version
-  version: 0.41.0
+  version: 0.42.2
 platform: ruby
 authors:
 - Fred Snyder
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-05-30 00:00:00.000000000 Z
+date: 2021-06-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: backport
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.1'
+        version: '1.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.1'
+        version: '1.2'
 - !ruby/object:Gem::Dependency
   name: benchmark
   requirement: !ruby/object:Gem::Requirement
@@ -52,6 +52,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 1.17.2
+- !ruby/object:Gem::Dependency
+  name: diff-lcs
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.4'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.4'
 - !ruby/object:Gem::Dependency
   name: e2mmap
   requirement: !ruby/object:Gem::Requirement
@@ -463,7 +477,6 @@ files:
 - lib/solargraph/pin/keyword.rb
 - lib/solargraph/pin/keyword_param.rb
 - lib/solargraph/pin/local_variable.rb
-- lib/solargraph/pin/localized.rb
 - lib/solargraph/pin/method.rb
 - lib/solargraph/pin/method_alias.rb
 - lib/solargraph/pin/namespace.rb

data/lib/solargraph/pin/localized.rb

@@ -1,43 +0,0 @@
-# frozen_string_literal: true
-
-module Solargraph
-  module Pin
-    module Localized
-      # @return [Range]
-      attr_reader :presence
-
-      # @param other [Pin::Base] The caller's block
-      # @param position [Position, Array(Integer, Integer)] The caller's position
-      # @return [Boolean]
-      def visible_from?(other, position)
-        position = Position.normalize(position)
-        other.filename == filename &&
-          match_tags(other.full_context.tag, full_context.tag) &&
-          (other == closure ||
-            (closure.location.range.contain?(other.location.range.start) && closure.location.range.contain?(other.location.range.ending))
-          ) &&
-          presence.contain?(position)
-      end
-
-      # @param other_loc [Location]
-      def visible_at?(other_loc)
-        return false if location.filename != other_loc.filename
-        presence.include?(other_loc.range.start)
-      end
-
-      private
-
-      # @param tag1 [String]
-      # @param tag2 [String]
-      # @return [Boolean]
-      def match_tags tag1, tag2
-        # @todo This is an unfortunate hack made necessary by a discrepancy in
-        #   how tags indicate the root namespace. The long-term solution is to
-        #   standardize it, whether it's `Class<>`, an empty string, or
-        #   something else.
-        tag1 == tag2 ||
-          (['', 'Class<>'].include?(tag1) && ['', 'Class<>'].include?(tag2))
-      end
-    end
-  end
-end