steep 1.5.1 → 1.5.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d7775c9db83acf70b19f27b4983923ee40e8d1a39043c981726573b0c06eb1c8
-  data.tar.gz: 1d9f2753a19a577cce778f6ec1ee924534523812764c0350c21280d82a0e1c69
+  metadata.gz: d2971f6bf7e70b227ac38de9682f640c8b274d8c27db2e02ec6920163d9e26df
+  data.tar.gz: 6f756247c85e018f5b2feaac36238480438495d242994f9dd23ccacd52e195d2
 SHA512:
-  metadata.gz: 6bd6340dfecc114df47a959d25a4201a87ccccbc0b1e59238962de51c28218b01aa25499ae8a760e678481192131dfdd3c99b6c89412ce4b6e0b3fad2bf34ca5
-  data.tar.gz: 87437181a053c65f297192eb9e339c0b58ad074d4c6716eef2a1019116d3ed06d0525eebcc49855ae99a8393bffac82640d77f608479b98f67e158b42c54d327
+  metadata.gz: f4ae7293b7ba2f8a33c96e470346f853f83afff373f5724a094c788841602cff54ad911ea74d73cc63471383f8fe043b217fef6528d4d6a6d91cb3351dc358e7
+  data.tar.gz: 720040f41d6a7c0fa11facd1bc0b8e0860cd54d3572fd698807894ff6c4fe33beea6c9fff8a4a1bd115e3b494021590ef33bfcd7035556a46cbcc00455943292

data/.github/dependabot.yml

@@ -8,6 +8,13 @@ updates:
       time: "20:00"
     open-pull-requests-limit: 10
 
+  - package-ecosystem: bundler
+    directory: "/gemfile_steep"
+    schedule:
+      interval: daily
+      time: "20:00"
+    open-pull-requests-limit: 10
+
   - package-ecosystem: "github-actions"
     directory: "/"
     schedule:

data/CHANGELOG.md

@@ -2,6 +2,17 @@
 
 ## master
 
+## 1.5.2 (2023-07-27)
+
+### Type checker core
+
+* Avoid inifinite loop in `#shape` ([#884](https://github.com/soutaro/steep/pull/884))
+* Fix `nil?` typing with `untyped` receiver ([#882](https://github.com/soutaro/steep/pull/882))
+
+### Language server
+
+* Avoid breaking the original source code through `CompletionProvider` ([#883](https://github.com/soutaro/steep/pull/883))
+
 ## 1.5.1 (2023-07-20)
 
 ### Type checker core

data/Gemfile

@@ -4,7 +4,7 @@ source 'https://rubygems.org'
 gemspec
 
 gem "rake"
-gem "minitest", "~> 5.18"
+gem "minitest", "~> 5.19"
 gem "minitest-hooks"
 group :stackprof, optional: true do
   gem "stackprof"

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    steep (1.5.1)
+    steep (1.5.2)
       activesupport (>= 5.1)
       concurrent-ruby (>= 1.1.10)
       csv (>= 3.0.9)
@@ -38,7 +38,7 @@ GEM
       rb-fsevent (~> 0.10, >= 0.10.3)
       rb-inotify (~> 0.9, >= 0.9.10)
     logger (1.5.3)
-    minitest (5.18.1)
+    minitest (5.19.0)
     minitest-hooks (1.5.0)
       minitest (> 5.3)
     minitest-slow_test (0.2.0)
@@ -74,7 +74,7 @@ PLATFORMS
   ruby
 
 DEPENDENCIES
-  minitest (~> 5.18)
+  minitest (~> 5.19)
   minitest-hooks
   minitest-slow_test
   rake

data/Gemfile.steep

@@ -1,3 +0,0 @@
-source "https://rubygems.org"
-
-gem 'steep', require: false

data/bin/rbs

@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
 
 BINSTUB_DIR=$(cd $(dirname $0); pwd)
-GEMFILE=$(readlink -f ${BINSTUB_DIR}/../Gemfile.steep)
+GEMFILE=$(readlink -f ${BINSTUB_DIR}/../gemfile_steep/Gemfile)
 COLLECTION=$(readlink -f ${BINSTUB_DIR}/../rbs_collection.steep.yaml)
 ROOT_DIR=$(readlink -f ${BINSTUB_DIR}/..)
 

data/bin/setup

@@ -5,5 +5,5 @@ set -vx
 
 bundle install
 
-bundle install --gemfile=Gemfile.steep
+bundle install --gemfile=gemfile_steep/Gemfile
 bin/rbs collection install

data/bin/steep

@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
 
 BINSTUB_DIR=$(cd $(dirname $0); pwd)
-GEMFILE=$(readlink -f ${BINSTUB_DIR}/../Gemfile.steep)
+GEMFILE=$(readlink -f ${BINSTUB_DIR}/../gemfile_steep/Gemfile)
 ROOT_DIR=$(readlink -f ${BINSTUB_DIR}/..)
 
 STEEP="bundle exec --gemfile=${GEMFILE} steep"

data/gemfile_steep/Gemfile

@@ -0,0 +1,3 @@
+source "https://rubygems.org"
+
+gem 'steep', require: false

data/{Gemfile.steep.lock → gemfile_steep/Gemfile.lock}

@@ -30,7 +30,7 @@ GEM
       ffi (~> 1.0)
     rbs (3.1.1)
     securerandom (0.2.2)
-    steep (1.5.0)
+    steep (1.5.1)
       activesupport (>= 5.1)
       concurrent-ruby (>= 1.1.10)
       csv (>= 3.0.9)
@@ -55,6 +55,7 @@ GEM
 PLATFORMS
   arm64-darwin-21
   arm64-darwin-22
+  x86_64-linux
 
 DEPENDENCIES
   steep

data/lib/steep/interface/builder.rb

@@ -68,6 +68,24 @@ module Steep
             )
           end
         end
+
+        def self_type?
+          unless self_type.is_a?(AST::Types::Self)
+            self_type
+          end
+        end
+
+        def instance_type?
+          unless instance_type.is_a?(AST::Types::Instance)
+            instance_type
+          end
+        end
+
+        def class_type?
+          unless class_type.is_a?(AST::Types::Class)
+            class_type
+          end
+        end
       end
 
       attr_reader :factory, :cache, :raw_object_cache
@@ -122,16 +140,18 @@ module Steep
         fetch_cache(type, public_only, config) do
           case type
           when AST::Types::Self
-            self_type = config.self_type.subst(config.subst)
-            shape(self_type, public_only: public_only, config: config.update(resolve_self: false))
+            if self_type = config.self_type?
+              self_type = self_type.subst(config.subst)
+              shape(self_type, public_only: public_only, config: config.update(resolve_self: false))
+            end
           when AST::Types::Instance
-            if config.instance_type
-              instance_type = config.instance_type.subst(config.subst)
+            if instance_type = config.instance_type?
+              instance_type = instance_type.subst(config.subst)
               shape(instance_type, public_only: public_only, config: config.update(resolve_instance: false))
             end
           when AST::Types::Class
-            if config.class_type
-              class_type = config.class_type.subst(config.subst)
+            if class_type = config.class_type?
+              class_type = class_type.subst(config.subst)
               shape(class_type, public_only: public_only, config: config.update(resolve_class: false))
             end
           when AST::Types::Name::Instance, AST::Types::Name::Interface, AST::Types::Name::Singleton
@@ -159,7 +179,7 @@ module Steep
               instance_var = AST::Types::Var.fresh(:INSTANCE)
 
               bounds = config.variable_bounds.merge({ self_var.name => config.self_type, instance_var.name => config.instance_type, class_var.name => config.class_type })
-              type_ = type.subst(Substitution.build([], [], self_type: self_var, instance_type: instance_var, module_type: class_var))
+              type_ = type.subst(Substitution.build([], [], self_type: self_var, instance_type: instance_var, module_type: class_var)) #: AST::Types::Union
 
               config_ = config.update(resolve_self: false, resolve_class: true, resolve_instance: true, variable_bounds: bounds)
 
@@ -203,7 +223,7 @@ module Steep
             instance_var = AST::Types::Var.fresh(:INSTANCE)
 
             bounds = config.variable_bounds.merge({ self_var.name => config.self_type, instance_var.name => config.instance_type, class_var.name => config.class_type })
-            type_ = type.subst(Substitution.build([], [], self_type: self_var, instance_type: instance_var, module_type: class_var))
+            type_ = type.subst(Substitution.build([], [], self_type: self_var, instance_type: instance_var, module_type: class_var)) #: AST::Types::Intersection
 
             config_ = config.update(resolve_self: false, resolve_class: true, resolve_instance: true, variable_bounds: bounds)
 

data/lib/steep/services/completion_provider.rb

@@ -244,6 +244,9 @@ module Steep
 
         rescue Parser::SyntaxError => exn
           Steep.logger.info "recovering syntax error: #{exn.inspect}"
+
+          @source_text = source_text.dup
+
           case possible_trigger
           when "."
             if source_text[index-2] == "&"

data/lib/steep/type_inference/logic_type_interpreter.rb

@@ -292,7 +292,7 @@ module Steep
             )
 
             truthy_result = Result.new(type: TRUE, env: truthy_env, unreachable: false)
-            truthy_result.unreachable! if unwrap == receiver_type
+            truthy_result.unreachable! if no_subtyping?(sub_type: AST::Builtin.nil_type, super_type: receiver_type)
 
             falsy_result = Result.new(type: FALSE, env: falsy_env, unreachable: false)
             falsy_result.unreachable! unless unwrap
@@ -529,15 +529,14 @@ module Steep
           # 3. K <: T           (example: T = Numeric, K = Integer)
           # 4. none of the above (example: T = String, K = Integer)
 
-          relation = Subtyping::Relation.new(sub_type: type, super_type: instance_type)
-          if subtyping.check(relation, constraints: Subtyping::Constraints.empty, self_type: AST::Types::Self.instance, instance_type: AST::Types::Instance.instance, class_type: AST::Types::Class.instance).success?
+          if subtyping?(sub_type: type, super_type: instance_type)
             # 1 or 2. Satisfies the condition, no narrowing because `type` is already more specific than/equals to `instance_type`
             [
               [type],
               []
             ]
           else
-            if subtyping.check(relation.flip, constraints: Subtyping::Constraints.empty, self_type: AST::Types::Self.instance, instance_type: AST::Types::Instance.instance, class_type: AST::Types::Class.instance).success?
+            if subtyping?(sub_type: instance_type, super_type: type)
               # 3. Satisfied the condition, narrows to `instance_type`, but cannot remove it from *falsy* list
               [
                 [instance_type],
@@ -554,6 +553,19 @@ module Steep
         end
       end
 
+      def no_subtyping?(sub_type:, super_type:)
+        relation = Subtyping::Relation.new(sub_type: sub_type, super_type: super_type)
+        result = subtyping.check(relation, constraints: Subtyping::Constraints.empty, self_type: AST::Types::Self.instance, instance_type: AST::Types::Instance.instance, class_type: AST::Types::Class.instance)
+
+        if result.failure?
+          result
+        end
+      end
+
+      def subtyping?(sub_type:, super_type:)
+        !no_subtyping?(sub_type: sub_type, super_type: super_type)
+      end
+
       def try_convert(type, method)
         if shape = subtyping.builder.shape(type, public_only: true, config: config)
           if entry = shape.methods[method]

data/lib/steep/version.rb

@@ -1,3 +1,3 @@
 module Steep
-  VERSION = "1.5.1"
+  VERSION = "1.5.2"
 end

data/rbs_collection.steep.lock.yaml

@@ -103,4 +103,4 @@ gems:
   version: '0'
   source:
     type: stdlib
-gemfile_lock_path: Gemfile.steep.lock
+gemfile_lock_path: gemfile_steep/Gemfile.lock

data/sig/steep/interface/builder.rbs

@@ -71,6 +71,18 @@ module Steep
 
         # Substitution for immediate type expressions
         def subst: () -> Substitution
+
+        # Returns `self_type`, or `nil` when it is `Types::Self`
+        #
+        def self_type?: () -> AST::Types::t?
+
+        # Returns `class_type`, or `nil` when it is `Types::Class`
+        #
+        def class_type?: () -> AST::Types::t?
+
+        # Returns `instanc_type`, or `nil` when it is `Types::Instance`
+        #
+        def instance_type?: () -> AST::Types::t?
       end
 
       attr_reader factory: AST::Types::Factory

data/sig/steep/type_inference/logic_type_interpreter.rbs

@@ -99,6 +99,10 @@ module Steep
       def type_case_select0: (AST::Types::t `type`, RBS::TypeName klass) -> [Array[AST::Types::t], Array[AST::Types::t]]
 
       def try_convert: (AST::Types::t, Symbol) -> AST::Types::t?
+
+      def no_subtyping?: (sub_type: AST::Types::t, super_type: AST::Types::t) -> Subtyping::Result::t?
+
+      def subtyping?: (sub_type: AST::Types::t, super_type: AST::Types::t) -> bool
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: steep
 version: !ruby/object:Gem::Version
-  version: 1.5.1
+  version: 1.5.2
 platform: ruby
 authors:
 - Soutaro Matsumoto
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2023-07-20 00:00:00.000000000 Z
+date: 2023-07-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: parser
@@ -242,7 +242,6 @@ files:
 - Gemfile
 - Gemfile.lock
 - Gemfile.steep
-- Gemfile.steep.lock
 - LICENSE
 - README.md
 - Rakefile
@@ -255,6 +254,8 @@ files:
 - bin/steep
 - bin/steep-prof
 - exe/steep
+- gemfile_steep/Gemfile
+- gemfile_steep/Gemfile.lock
 - guides/README.md
 - guides/src/gem-rbs-collection/gem-rbs-collection.md
 - guides/src/getting-started/getting-started.md