solargraph 0.38.6 → 0.39.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 061561cbbc7728685d8db8a482d16004254b6abcefde23ad1c8666f1dc8d37fb
-  data.tar.gz: 5c2a8991b9b2a7264a63337fb2d427b0c62c96843c6014202b2247d6c727558b
+  metadata.gz: 71e5d6fa6baec1130d64df74ed6fcf49c6c41df2f1386cb5cd2760a7605a9e48
+  data.tar.gz: 91d9d6c695a63a0defab2c31967b2828907c800a94eb220c9210db9b672f2f27
 SHA512:
-  metadata.gz: fc5d80cf77ad1c3b505ea3ecfbaaf514d2c49c7b93654b1e1fbd3b692266e045f4578e7bb15f5594842f677709f0a91b568379950648141012c694597b969686
-  data.tar.gz: '09e24fd600309f0b4f76391d33ba929bc6d153224acef25c384639f1072b5f9cd999102bdcd63b46e4238ee4089423068ff25d5b93af57c64d40d425cf884008'
+  metadata.gz: 2b37f7a917d8c865063f14c62e185e919cb88f221219fdc611ef003aa0a17af99bdd762fd4d4fc36105f37c36b0bdd4017500abe17b3aa1506423df8b16ee55a
+  data.tar.gz: 60110bec4c847690cecdfa0d73d0495d09340a56ddf650dfbfc737e1f8b62682e1cce3de568edec7e52f4d4ff052e4c4746a2ec907c20320b6b133de610b6f3a

data/SPONSORS.md

@@ -0,0 +1,9 @@
+# Sponsors
+
+Solargraph is developed and maintained by [Castwide Technologies](https://castwide.com).
+
+The following people and organizations provide funding or other resources. [Become a sponsor](https://patreon.com/castwide)
+
+## Named Sponsors
+
+- Emily Strickland

data/lib/solargraph.rb

@@ -38,6 +38,7 @@ module Solargraph
   autoload :Environ,          'solargraph/environ'
   autoload :Convention,       'solargraph/convention'
   autoload :Documentor,       'solargraph/documentor'
+  autoload :Parser,           'solargraph/parser'
 
   dir = File.dirname(__FILE__)
   YARDOC_PATH = File.realpath(File.join(dir, '..', 'yardoc'))

data/lib/solargraph/api_map.rb

@@ -149,7 +149,7 @@ module Solargraph
     # @return [Source::Cursor]
     def cursor_at filename, position
       position = Position.normalize(position)
-      raise "File not found: #{filename}" unless source_map_hash.has_key?(filename)
+      raise FileNotFoundError, "File not found: #{filename}" unless source_map_hash.has_key?(filename)
       source_map_hash[filename].cursor_at(position)
     end
 
@@ -215,7 +215,7 @@ module Solargraph
       contexts.push '' if contexts.empty?
       cached = cache.get_constants(namespace, contexts)
       return cached.clone unless cached.nil?
-      skip = []
+      skip = Set.new
       result = []
       contexts.each do |context|
         fqns = qualify(namespace, context)
@@ -238,9 +238,9 @@ module Solargraph
       cached = cache.get_qualified_namespace(namespace, context)
       return cached.clone unless cached.nil?
       result = if namespace.start_with?('::')
-                 inner_qualify(namespace[2..-1], '', [])
+                 inner_qualify(namespace[2..-1], '', Set.new)
                else
-                 inner_qualify(namespace, context, [])
+                 inner_qualify(namespace, context, Set.new)
                end
       cache.set_qualified_namespace(namespace, context, result)
       result
@@ -295,7 +295,7 @@ module Solargraph
       cached = cache.get_methods(fqns, scope, visibility, deep)
       return cached.clone unless cached.nil?
       result = []
-      skip = []
+      skip = Set.new
       if fqns == ''
         # @todo Implement domains
         implicit.domains.each do |domain|
@@ -308,6 +308,7 @@ module Solargraph
         result.concat inner_get_methods('Kernel', :instance, visibility, deep, skip)
       else
         result.concat inner_get_methods(fqns, scope, visibility, deep, skip)
+        result.concat inner_get_methods('Kernel', :instance, [:public], deep, skip) if visibility.include?(:private)
       end
       resolved = resolve_method_aliases(result, visibility)
       cache.set_methods(fqns, scope, visibility, deep, resolved)
@@ -551,15 +552,21 @@ module Solargraph
     # @param scope [Symbol] :class or :instance
     # @param visibility [Array<Symbol>] :public, :protected, and/or :private
     # @param deep [Boolean]
-    # @param skip [Array<String>]
+    # @param skip [Set<String>]
     # @param no_core [Boolean] Skip core classes if true
     # @return [Array<Pin::Base>]
     def inner_get_methods fqns, scope, visibility, deep, skip, no_core = false
       return [] if no_core && fqns =~ /^(Object|BasicObject|Class|Module|Kernel)$/
       reqstr = "#{fqns}|#{scope}|#{visibility.sort}|#{deep}"
       return [] if skip.include?(reqstr)
-      skip.push reqstr
+      skip.add reqstr
       result = []
+      if deep && scope == :instance
+        store.get_prepends(fqns).reverse.each do |im|
+          fqim = qualify(im, fqns)
+          result.concat inner_get_methods(fqim, scope, visibility, deep, skip, true) unless fqim.nil?
+        end
+      end
       result.concat store.get_methods(fqns, scope: scope, visibility: visibility).sort{ |a, b| a.name <=> b.name }
       if deep
         if scope == :instance
@@ -596,12 +603,16 @@ module Solargraph
 
     # @param fqns [String]
     # @param visibility [Array<Symbol>]
-    # @param skip [Array<String>]
+    # @param skip [Set<String>]
     # @return [Array<Pin::Base>]
     def inner_get_constants fqns, visibility, skip
       return [] if fqns.nil? || skip.include?(fqns)
-      skip.push fqns
-      result = store.get_constants(fqns, visibility)
+      skip.add fqns
+      result = []
+      store.get_prepends(fqns).each do |is|
+        result.concat inner_get_constants(qualify(is, fqns), [:public], skip)
+      end
+      result.concat store.get_constants(fqns, visibility)
                     .sort { |a, b| a.name <=> b.name }
       store.get_includes(fqns).each do |is|
         result.concat inner_get_constants(qualify(is, fqns), [:public], skip)
@@ -636,12 +647,12 @@ module Solargraph
 
     # @param name [String]
     # @param root [String]
-    # @param skip [Array<String>]
+    # @param skip [Set<String>]
     # @return [String, nil]
     def inner_qualify name, root, skip
       return nil if name.nil?
       return nil if skip.include?(root)
-      skip.push root
+      skip.add root
       if name == ''
         if root == ''
           return ''
@@ -727,7 +738,7 @@ module Solargraph
         comments: origin.comments,
         scope: origin.scope,
         visibility: origin.visibility,
-        args: origin.parameters
+        parameters: origin.parameters
       )
     end
   end

data/lib/solargraph/api_map/bundler_methods.rb

@@ -6,6 +6,8 @@ module Solargraph
     module BundlerMethods
       module_function
 
+      # @param directory [String]
+      # @return [Hash]
       def require_from_bundle directory
         @require_from_bundle ||= begin
           Solargraph.logger.info "Loading gems for bundler/require"
@@ -16,6 +18,7 @@ module Solargraph
         end
       end
 
+      # @return [void]
       def reset_require_from_bundle
         @require_from_bundle = nil
       end

data/lib/solargraph/api_map/source_to_yard.rb

@@ -60,12 +60,7 @@ module Solargraph
           code_object_map[pin.path].docstring = pin.docstring
           code_object_map[pin.path].visibility = pin.visibility || :public
           code_object_map[pin.path].parameters = pin.parameters.map do |p|
-            n = p.match(/^[a-z0-9_]*:?/i)[0]
-            v = nil
-            if p.length > n.length
-              v = p[n.length..-1].gsub(/^ = /, '')
-            end
-            [n, v]
+            [p.name, p.asgn_code]
           end
         end
       end

data/lib/solargraph/api_map/store.rb

@@ -19,7 +19,7 @@ module Solargraph
       # @return [Array<Solargraph::Pin::Base>]
       def get_constants fqns, visibility = [:public]
         namespace_children(fqns).select { |pin|
-          !pin.name.empty? and (pin.is_a?(Pin::Namespace) || pin.is_a?(Pin::Constant)) && visibility.include?(pin.visibility)
+          !pin.name.empty? && (pin.is_a?(Pin::Namespace) || pin.is_a?(Pin::Constant)) && visibility.include?(pin.visibility)
         }
       end
 
@@ -34,7 +34,7 @@ module Solargraph
       end
 
       # @param fqns [String]
-      # @return [String]
+      # @return [String, nil]
       def get_superclass fqns
         return superclass_references[fqns].first if superclass_references.key?(fqns)
         return 'Object' if fqns != 'BasicObject' && namespace_exists?(fqns)
@@ -48,6 +48,12 @@ module Solargraph
         include_references[fqns] || []
       end
 
+      # @param fqns [String]
+      # @return [Array<String>]
+      def get_prepends fqns
+        prepend_references[fqns] || []
+      end
+
       # @param fqns [String]
       # @return [Array<String>]
       def get_extends fqns
@@ -172,6 +178,10 @@ module Solargraph
         @include_references ||= {}
       end
 
+      def prepend_references
+        @prepend_references ||= {}
+      end
+
       def extend_references
         @extend_references ||= {}
       end
@@ -210,13 +220,16 @@ module Solargraph
         pins.each do |pin|
           namespace_map[pin.namespace] ||= []
           namespace_map[pin.namespace].push pin
-          namespaces.add pin.path if pin.is_a?(Pin::Namespace) and !pin.path.empty?
+          namespaces.add pin.path if pin.is_a?(Pin::Namespace) && !pin.path.empty?
           namespace_pins.push pin if pin.is_a?(Pin::Namespace)
           method_pins.push pin if pin.is_a?(Pin::BaseMethod)
           symbols.push pin if pin.is_a?(Pin::Symbol)
           if pin.is_a?(Pin::Reference::Include)
             include_references[pin.namespace] ||= []
             include_references[pin.namespace].push pin.name
+          elsif pin.is_a?(Pin::Reference::Prepend)
+            prepend_references[pin.namespace] ||= []
+            prepend_references[pin.namespace].push pin.name
           elsif pin.is_a?(Pin::Reference::Extend)
             extend_references[pin.namespace] ||= []
             extend_references[pin.namespace].push pin.name
@@ -245,6 +258,9 @@ module Solargraph
             pin.docstring.add_tag(tag)
           end
         end
+        # @todo This is probably not the best place for these overrides
+        superclass_references['Integer'] = ['Numeric']
+        superclass_references['Float'] = ['Numeric']
       end
     end
   end

data/lib/solargraph/complex_type/type_methods.rb

@@ -113,13 +113,13 @@ module Solargraph
           t.qualify api_map, context
         end
         if list_parameters?
-          Solargraph::ComplexType.parse("#{fqns}<#{rtypes.map(&:tag).join(', ')}>").first
+          Solargraph::ComplexType.parse("#{fqns}<#{rtypes.map(&:tag).join(', ')}>")
         elsif fixed_parameters?
-          Solargraph::ComplexType.parse("#{fqns}(#{rtypes.map(&:tag).join(', ')})").first
+          Solargraph::ComplexType.parse("#{fqns}(#{rtypes.map(&:tag).join(', ')})")
         elsif hash_parameters?
-          Solargraph::ComplexType.parse("#{fqns}{#{ltypes.map(&:tag).join(', ')} => #{rtypes.map(&:tag).join(', ')}}").first
+          Solargraph::ComplexType.parse("#{fqns}{#{ltypes.map(&:tag).join(', ')} => #{rtypes.map(&:tag).join(', ')}}")
         else
-          Solargraph::ComplexType.parse(fqns).first
+          Solargraph::ComplexType.parse(fqns)
         end
       end
     end

data/lib/solargraph/convention/base.rb

@@ -13,11 +13,6 @@ module Solargraph
         false
       end
 
-      # @return [Environ]
-      def process
-        match? ? EMPTY_ENVIRON : environ
-      end
-
       # The Environ for this convention.
       # Subclasses should override this method.
       #

data/lib/solargraph/core_fills.rb

@@ -65,11 +65,13 @@ module Solargraph
   @return [self]
 @return_single_parameter
       )),
+      Override.method_return('Array#uniq', 'self'),
 
       Override.from_comment('BasicObject#==', %(
 @param other [BasicObject]
 @return [Boolean]
       )),
+      Override.method_return('BasicObject#initialize', 'void'),
 
       Override.method_return('Class#new', 'self'),
       Override.method_return('Class.new', 'Class<Object>'),
@@ -84,6 +86,11 @@ module Solargraph
       Override.method_return('File.extname', 'String'),
       Override.method_return('File.join', 'String'),
 
+      Override.from_comment('Float#+', %(
+@param y [Numeric]
+@return [Numeric]
+      )),
+
       Override.from_comment('Hash#[]', %(
 @return_value_parameter
       )),
@@ -94,9 +101,25 @@ module Solargraph
 @param_tuple
       )),
 
+      Override.from_comment('Integer#+', %(
+@param y [Numeric]
+@return [Numeric]
+      )),
+
+      Override.method_return('Kernel#puts', 'nil'),
+
+      # Override.method_return('Module#attr_reader', 'void'),
+      # Override.method_return('Module#attr_writer', 'void'),
+      # Override.method_return('Module#attr_accessor', 'void'),
+
+      Override.from_comment('Numeric#+', %(
+@param y [Numeric]
+@return [Numeric]
+      )),
+
       Override.method_return('Object#!', 'Boolean'),
       Override.method_return('Object#clone', 'self', delete: [:overload]),
-      Override.method_return('Object#dup', 'self'),
+      Override.method_return('Object#dup', 'self', delete: [:overload]),
       Override.method_return('Object#freeze', 'self', delete: [:overload]),
       Override.method_return('Object#inspect', 'String'),
       Override.method_return('Object#taint', 'self'),
@@ -121,7 +144,10 @@ module Solargraph
 
       Override.method_return('String#freeze', 'self'),
       Override.method_return('String#split', 'Array<String>'),
-      Override.method_return('String#lines', 'Array<String>')
+      Override.method_return('String#lines', 'Array<String>'),
+      Override.from_comment('String#each_line', %(
+@yieldparam [String]
+      ))
     ].concat(
       methods_with_yieldparam_subtypes.map do |path|
         Override.from_comment(path, %(

data/lib/solargraph/diagnostics/base.rb

@@ -5,6 +5,7 @@ module Solargraph
     # The base class for diagnostics reporters.
     #
     class Base
+      # @return [Array<String>]
       attr_reader :args
 
       def initialize *args

data/lib/solargraph/diagnostics/type_check.rb

@@ -8,19 +8,19 @@ module Solargraph
     class TypeCheck < Base
       def diagnose source, api_map
         return [] unless args.include?('always') || api_map.workspaced?(source.filename)
-        severity = (args.include?('strict') ? Diagnostics::Severities::ERROR : Diagnostics::Severities::WARNING)
-        checker = Solargraph::TypeChecker.new(source.filename, api_map: api_map)
-        result = checker.return_type_problems + checker.param_type_problems
-        result.concat checker.strict_type_problems if args.include?('strict')
-        result.sort! { |a, b| a.location.range.start.line <=> b.location.range.start.line }
-        result.map do |problem|
-          {
-            range: extract_first_line(problem.location, source),
-            severity: severity,
-            source: 'Typecheck',
-            message: problem.message
-          }
-        end
+        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)
+        checker.problems
+          .sort { |a, b| a.location.range.start.line <=> b.location.range.start.line }
+          .map do |problem|
+            {
+              range: extract_first_line(problem.location, source),
+              severity: severity,
+              source: 'Typecheck',
+              message: problem.message
+            }
+          end
       end
 
       private

data/lib/solargraph/language_server/host.rb

@@ -207,6 +207,15 @@ module Solargraph
                 type: LanguageServer::MessageTypes::ERROR,
                 message: "Error in diagnostics: #{e.message}"
               }
+            rescue FileNotFoundError => e
+              # @todo This appears to happen when an external file is open and
+              #   scheduled for diagnosis, but the file was closed (i.e., the
+              #   editor moved to a different file) before diagnosis started
+              logger.warn "Unable to diagnose #{uri} : #{e.message}"
+              send_notification 'textDocument/publishDiagnostics', {
+                uri: uri,
+                diagnostics: []
+              }
             end
           else
             logger.info "Deferring diagnosis of #{uri}"
@@ -245,7 +254,7 @@ module Solargraph
       #
       # @return [String] The most recent data or an empty string.
       def flush
-        tmp = nil
+        tmp = ''
         @buffer_semaphore.synchronize do
           tmp = @buffer.clone
           @buffer.clear
@@ -539,7 +548,7 @@ module Solargraph
       end
 
       # @param query [String]
-      # @return [String]
+      # @return [Array]
       def document query
         result = []
         libraries.each { |lib| result.concat lib.document(query) }

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

@@ -12,7 +12,9 @@ module Solargraph
         include UriHelpers
 
         def initialize
+          @mutex = Mutex.new
           @stopped = true
+          @has_uri = ConditionVariable.new
         end
 
         def stopped?
@@ -24,17 +26,14 @@ module Solargraph
           return unless @stopped
           @stopped = false
           Thread.new do
-            until stopped?
-              tick
-              sleep 0.25 if queue.empty?
-            end
+            tick until stopped?
           end
         end
 
         # @return [void]
         def tick
-          return if queue.empty?
-          uri = mutex.synchronize { queue.shift }
+          uri = mutex.synchronize { next_uri }
+
           return if queue.include?(uri)
           mutex.synchronize do
             nxt = open_source_hash[uri].finish_synchronize
@@ -44,6 +43,18 @@ module Solargraph
           end
         end
 
+        # @return [void]
+        def add_uri(uri)
+          queue.push(uri)
+          @has_uri.signal
+        end
+
+        # @return [String]
+        def next_uri
+          @has_uri.wait(mutex) if queue.empty?
+          queue.shift
+        end
+
         # @return [void]
         def stop
           @stopped = true
@@ -87,7 +98,7 @@ module Solargraph
           src = find(uri)
           mutex.synchronize do
             open_source_hash[uri] = src.start_synchronize(updater)
-            queue.push uri
+            add_uri(uri)
           end
           changed
           notify_observers uri
@@ -125,15 +136,13 @@ module Solargraph
 
         private
 
-        # @return [Array<Source>]
+        # @return [Hash]
         def open_source_hash
           @open_source_hash ||= {}
         end
 
         # @return [Mutex]
-        def mutex
-          @mutex ||= Mutex.new
-        end
+        attr_reader :mutex
 
         # An array of source URIs that are waiting to finish synchronizing.
         #