sorbet-runtime 0.5.6105 → 0.5.6130

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 299b0b4dce288341e7bdb362e014bf4224a0d5e40c0a13cd7b015b6ed3f64712
-  data.tar.gz: 0a068a678ed882f1ab7ff1fd4e632b6d54f2337e4331b68f9d84ae3a42720ef2
+  metadata.gz: b2c6cd9c25bf625e725848fb66c990529af90f07966dbe94929b80ac62431de8
+  data.tar.gz: c20105ed1aa4b3aca71609d04c23f68776d6c6e6a1f63cb31475dfe81ea71434
 SHA512:
-  metadata.gz: ccdd31cee488beb7f6e2cabac2e1979c62c6a7df113a0389fba0c735532426e437147152cf346616946313f1d0af0fb55ee81ab0064fcbe7002061ecfda7d870
-  data.tar.gz: 77caad3f157f448215cfe6942d5d77cd43d03c9bc4d132eabfaaefc8feee430f2df380a5badba21e8deedbbbc986a629af556a1af7585877b454397f1a9d697c
+  metadata.gz: 0c5226fcd69dff48bf84fd1856b2368d8d222386da268d85b9bb8150cfd25f33301430bfa30b81f1f12e09b9298f6225ae807e4878746d2bf435aa01bb756743
+  data.tar.gz: a03c025907fe8e304504b78f064de0bc1acd9e8a3b500f5200c7af7e66a33ce3bcb489dc7868b5cf0fd873549c3d7ee0d30093191356c3d545c3baffdef67438

data/lib/types/configuration.rb

@@ -338,7 +338,7 @@ module T::Configuration
     if values.nil?
       @scalar_types = values
     else
-      bad_values = values.select {|v| v.class != String}
+      bad_values = values.reject {|v| v.class == String}
       unless bad_values.empty?
         raise ArgumentError.new("Provided values must all be class name strings.")
       end
@@ -347,7 +347,7 @@ module T::Configuration
     end
   end
 
-  @default_scalar_types = Set.new(%w{
+  @default_scalar_types = Set.new(%w[
     NilClass
     TrueClass
     FalseClass
@@ -357,7 +357,7 @@ module T::Configuration
     Symbol
     Time
     T::Enum
-  }).freeze
+  ]).freeze
 
   def self.scalar_types
     @scalar_types || @default_scalar_types
@@ -398,7 +398,7 @@ module T::Configuration
   #   should be allowed. Useful to whitelist benign violations, like shim files
   #   generated for an autoloader.
   def self.sealed_violation_whitelist=(sealed_violation_whitelist)
-    if @sealed_violation_whitelist != nil
+    if !@sealed_violation_whitelist.nil?
       raise ArgumentError.new("Cannot overwrite sealed_violation_whitelist after setting it")
     end
 

data/lib/types/non_forcing_constants.rb

@@ -56,6 +56,6 @@ module T::NonForcingConstants
       end
     end
 
-    return current_klass.===(val)
+    current_klass.===(val)
   end
 end

data/lib/types/private/abstract/validate.rb

@@ -111,16 +111,16 @@ module T::Private::Abstract::Validate
   end
 
   private_class_method def self.describe_method(method, show_owner: true)
-    if method.source_location
-      loc = method.source_location.join(':')
+    loc = if method.source_location
+      method.source_location.join(':')
     else
-      loc = "<unknown location>"
+      "<unknown location>"
     end
 
-    if show_owner
-      owner = " declared in #{method.owner}"
+    owner = if show_owner
+      " declared in #{method.owner}"
     else
-      owner = ""
+      ""
     end
 
     "    * `#{method.name}`#{owner} at #{loc}"

data/lib/types/private/methods/_methods.rb

@@ -145,7 +145,7 @@ module T::Private::Methods
         if (ancestor.method_defined?(method_name) ||
             ancestor.private_method_defined?(method_name) ||
             ancestor.protected_method_defined?(method_name)) &&
-              final_method?(method_owner_and_name_to_key(ancestor, method_name))
+           final_method?(method_owner_and_name_to_key(ancestor, method_name))
           raise(
             "The method `#{method_name}` on #{ancestor} was declared as final and cannot be " +
             (target == ancestor ? "redefined" : "overridden in #{target}")
@@ -193,7 +193,7 @@ module T::Private::Methods
 
     if method_name == :method_added || method_name == :singleton_method_added
       raise(
-        "Putting a `sig` on `#{method_name}` is not supported" +
+        "Putting a `sig` on `#{method_name}` is not supported" \
         " (sorbet-runtime uses this method internally to perform `sig` validation logic)"
       )
     end
@@ -316,10 +316,10 @@ module T::Private::Methods
     begin
       # We allow `sig` in the current module's context (normal case) and
       if hook_mod != current_declaration.mod &&
-          # inside `class << self`, and
-          hook_mod.singleton_class != current_declaration.mod &&
-          # on `self` at the top level of a file
-          current_declaration.mod != TOP_SELF
+         # inside `class << self`, and
+         hook_mod.singleton_class != current_declaration.mod &&
+         # on `self` at the top level of a file
+         current_declaration.mod != TOP_SELF
         raise "A method (#{method_name}) is being added on a different class/module (#{hook_mod}) than the " \
               "last call to `sig` (#{current_declaration.mod}). Make sure each call " \
               "to `sig` is immediately followed by a method definition on the same " \
@@ -410,7 +410,7 @@ module T::Private::Methods
   def self.run_all_sig_blocks
     loop do
       break if @sig_wrappers.empty?
-      key, _ = @sig_wrappers.first
+      key, = @sig_wrappers.first
       run_sig_block_for_key(key)
     end
   end
@@ -427,7 +427,7 @@ module T::Private::Methods
   end
 
   def self.set_final_checks_on_hooks(enable)
-    is_enabled = @old_hooks != nil
+    is_enabled = !@old_hooks.nil?
     if enable == is_enabled
       return
     end

data/lib/types/private/methods/call_validation.rb

@@ -195,7 +195,7 @@ module T::Private::Methods::CallValidation
       raise "Should have used create_validator_procedure_fast"
     end
     # trampoline to reduce stack frame size
-    if method_sig.arg_types.length == 0
+    if method_sig.arg_types.empty?
       create_validator_method_fast0(mod, original_method, method_sig, method_sig.return_type.raw_type)
     elsif method_sig.arg_types.length == 1
       create_validator_method_fast1(mod, original_method, method_sig, method_sig.return_type.raw_type,
@@ -708,7 +708,7 @@ module T::Private::Methods::CallValidation
 
   def self.create_validator_procedure_fast(mod, original_method, method_sig)
     # trampoline to reduce stack frame size
-    if method_sig.arg_types.length == 0
+    if method_sig.arg_types.empty?
       create_validator_procedure_fast0(mod, original_method, method_sig)
     elsif method_sig.arg_types.length == 1
       create_validator_procedure_fast1(mod, original_method, method_sig,

data/lib/types/private/methods/modes.rb

@@ -8,9 +8,9 @@ module T::Private::Methods::Modes
   def self.override; 'override'; end
   def self.overridable_override; 'overridable_override'; end
   def self.untyped; 'untyped'; end
-  MODES = [self.standard, self.abstract, self.overridable, self.override, self.overridable_override, self.untyped]
+  MODES = [self.standard, self.abstract, self.overridable, self.override, self.overridable_override, self.untyped].freeze
 
-  OVERRIDABLE_MODES = [self.override, self.overridable, self.overridable_override, self.untyped, self.abstract]
-  OVERRIDE_MODES = [self.override, self.overridable_override]
+  OVERRIDABLE_MODES = [self.override, self.overridable, self.overridable_override, self.untyped, self.abstract].freeze
+  OVERRIDE_MODES = [self.override, self.overridable_override].freeze
   NON_OVERRIDE_MODES = MODES - OVERRIDE_MODES
 end

data/lib/types/private/methods/signature.rb

@@ -28,7 +28,7 @@ class T::Private::Methods::Signature
     )
   end
 
-  def initialize(method:, method_name:, raw_arg_types:, raw_return_type:, bind:, mode:, check_level:, parameters: method.parameters, on_failure:, override_allow_incompatible: false)
+  def initialize(method:, method_name:, raw_arg_types:, raw_return_type:, bind:, mode:, check_level:, on_failure:, parameters: method.parameters, override_allow_incompatible: false)
     @method = method
     @method_name = method_name
     @arg_types = []
@@ -67,7 +67,7 @@ class T::Private::Methods::Signature
     if !extra_names.empty?
       raise "The declaration for `#{method.name}` has extra parameter(s): #{extra_names.join(', ')}"
     end
-    
+
     if parameters.size != raw_arg_types.size
       raise "The declaration for `#{method.name}` has arguments with duplicate names"
     end
@@ -198,10 +198,10 @@ class T::Private::Methods::Signature
   end
 
   def method_desc
-    if @method.source_location
-      loc = @method.source_location.join(':')
+    loc = if @method.source_location
+      @method.source_location.join(':')
     else
-      loc = "<unknown location>"
+      "<unknown location>"
     end
     "#{@method} at #{loc}"
   end

data/lib/types/private/methods/signature_validation.rb

@@ -100,19 +100,16 @@ module T::Private::Methods::SignatureValidation
       raise "Unexpected mode: #{signature.mode}. Please report to #dev-productivity."
     end
 
+    # Given a singleton class, we can check if it belongs to a
+    # module by looking at its superclass; given `module M`,
+    # `M.singleton_class.superclass == Module`, which is not true
+    # for any class.
     owner = signature.method.owner
     if (signature.mode == Modes.abstract || Modes::OVERRIDABLE_MODES.include?(signature.mode)) &&
-        owner.singleton_class?
-      # Given a singleton class, we can check if it belongs to a
-      # module by looking at its superclass; given `module M`,
-      # `M.singleton_class.superclass == Module`, which is not true
-      # for any class.
-      if owner.superclass == Module
-        raise "Defining an overridable class method (via #{pretty_mode(signature)}) " \
-              "on a module is not allowed. Class methods on " \
-              "modules do not get inherited and thus cannot be overridden. For help, ask in " \
-              "#dev-productivity."
-      end
+        owner.singleton_class? && owner.superclass == Module
+      raise "Defining an overridable class method (via #{pretty_mode(signature)}) " \
+            "on a module is not allowed. Class methods on " \
+            "modules do not get inherited and thus cannot be overridden."
     end
   end
 
@@ -219,10 +216,10 @@ module T::Private::Methods::SignatureValidation
   end
 
   private_class_method def self.method_loc_str(method)
-    if method.source_location
-      loc = method.source_location.join(':')
+    loc = if method.source_location
+      method.source_location.join(':')
     else
-      loc = "<unknown location>"
+      "<unknown location>"
     end
     "#{method.owner} at #{loc}"
   end

data/lib/types/private/sealed.rb

@@ -70,7 +70,7 @@ module T::Private::Sealed
 
     if !this_file.start_with?(decl_file)
       whitelist = T::Configuration.sealed_violation_whitelist
-      if whitelist != nil && whitelist.any? {|pattern| this_file =~ pattern}
+      if !whitelist.nil? && whitelist.any? {|pattern| this_file =~ pattern}
         return
       end
 

data/lib/types/props/decorator.rb

@@ -54,7 +54,7 @@ class T::Props::Decorator
     @props = @props.merge(prop => rules.freeze).freeze
   end
 
-  VALID_RULE_KEYS = T.let(%i{
+  VALID_RULE_KEYS = T.let(%i[
     enum
     foreign
     foreign_hint_only
@@ -68,7 +68,7 @@ class T::Props::Decorator
     extra
     setter_validate
     _tnilable
-  }.map {|k| [k, true]}.to_h.freeze, T::Hash[Symbol, T::Boolean])
+  ].map {|k| [k, true]}.to_h.freeze, T::Hash[Symbol, T::Boolean])
   private_constant :VALID_RULE_KEYS
 
   sig {params(key: Symbol).returns(T::Boolean).checked(:never)}
@@ -153,12 +153,10 @@ class T::Props::Decorator
     val = instance.instance_variable_get(rules[:accessor_key])
     if !val.nil?
       val
+    elsif (d = rules[:ifunset])
+      T::Props::Utils.deep_clone_object(d)
     else
-      if (d = rules[:ifunset])
-        T::Props::Utils.deep_clone_object(d)
-      else
-        nil
-      end
+      nil
     end
   end
 
@@ -219,15 +217,13 @@ class T::Props::Decorator
       raise ArgumentError.new("At least one invalid prop arg supplied in #{self}: #{rules.keys.inspect}")
     end
 
-    if !(rules[:clobber_existing_method!]) && !(rules[:without_accessors])
-      if BANNED_METHOD_NAMES.include?(name.to_sym)
-        raise ArgumentError.new(
-          "#{name} can't be used as a prop in #{@class} because a method with " \
-          "that name already exists (defined by #{@class.instance_method(name).owner} " \
-          "at #{@class.instance_method(name).source_location || '<unknown>'}). " \
-          "(If using this name is unavoidable, try `without_accessors: true`.)"
-        )
-      end
+    if !rules[:clobber_existing_method!] && !rules[:without_accessors] && BANNED_METHOD_NAMES.include?(name.to_sym)
+      raise ArgumentError.new(
+        "#{name} can't be used as a prop in #{@class} because a method with " \
+        "that name already exists (defined by #{@class.instance_method(name).owner} " \
+        "at #{@class.instance_method(name).source_location || '<unknown>'}). " \
+        "(If using this name is unavoidable, try `without_accessors: true`.)"
+      )
     end
 
     extra = rules[:extra]
@@ -238,7 +234,7 @@ class T::Props::Decorator
     nil
   end
 
-  SAFE_NAME = /\A[A-Za-z_][A-Za-z0-9_-]*\z/
+  SAFE_NAME = T.let(/\A[A-Za-z_][A-Za-z0-9_-]*\z/.freeze, Regexp)
 
   # Used to validate both prop names and serialized forms
   sig {params(name: T.any(Symbol, String)).void}
@@ -312,13 +308,12 @@ class T::Props::Decorator
     sensitivity_and_pii = {sensitivity: rules[:sensitivity]}
     if defined?(Opus) && defined?(Opus::Sensitivity) && defined?(Opus::Sensitivity::Utils)
       sensitivity_and_pii = Opus::Sensitivity::Utils.normalize_sensitivity_and_pii_annotation(sensitivity_and_pii)
-    end
-    # We check for Class so this is only applied on concrete
-    # documents/models; We allow mixins containing props to not
-    # specify their PII nature, as long as every class into which they
-    # are ultimately included does.
-    #
-    if defined?(Opus) && defined?(Opus::Sensitivity) && defined?(Opus::Sensitivity::PIIable)
+
+      # We check for Class so this is only applied on concrete
+      # documents/models; We allow mixins containing props to not
+      # specify their PII nature, as long as every class into which they
+      # are ultimately included does.
+      #
       if sensitivity_and_pii[:pii] && @class.is_a?(Class) && !T.unsafe(@class).contains_pii?
         raise ArgumentError.new(
           'Cannot include a pii prop in a class that declares `contains_no_pii`'
@@ -488,7 +483,7 @@ class T::Props::Decorator
     .void
   end
   private def handle_foreign_hint_only_option(prop_name, prop_cls, foreign_hint_only)
-    if ![String, Array].include?(prop_cls) && !(prop_cls.is_a?(T::Props::CustomType))
+    if ![String, Array].include?(prop_cls) && !prop_cls.is_a?(T::Props::CustomType)
       raise ArgumentError.new(
         "`foreign_hint_only` can only be used with String or Array prop types"
       )
@@ -546,10 +541,10 @@ class T::Props::Decorator
         # of the method, optimizing it so this only runs on the first invocation.
         foreign = resolved_foreign
       end
-      if allow_direct_mutation.nil?
-        opts = {}
+      opts = if allow_direct_mutation.nil?
+        {}
       else
-        opts = {allow_direct_mutation: allow_direct_mutation}
+        {allow_direct_mutation: allow_direct_mutation}
       end
 
       T.unsafe(self.class).decorator.foreign_prop_get(self, prop_name, foreign, rules, opts)
@@ -644,25 +639,33 @@ class T::Props::Decorator
       # (b) it's safe because the getter was defined by this file.
       #
       unless rules[:without_accessors]
-        if child.decorator.method(:prop_get).owner != method(:prop_get).owner &&
-            child.instance_method(name).source_location&.first == __FILE__
+        if clobber_getter?(child, name)
           child.send(:define_method, name) do
             T.unsafe(self.class).decorator.prop_get(self, name, rules)
           end
         end
 
-        unless rules[:immutable]
-          if child.decorator.method(:prop_set).owner != method(:prop_set).owner &&
-              child.instance_method("#{name}=").source_location&.first == __FILE__
-            child.send(:define_method, "#{name}=") do |val|
-              T.unsafe(self.class).decorator.prop_set(self, name, val, rules)
-            end
+        if !rules[:immutable] && clobber_setter?(child, name)
+          child.send(:define_method, "#{name}=") do |val|
+            T.unsafe(self.class).decorator.prop_set(self, name, val, rules)
           end
         end
       end
     end
   end
 
+  sig {params(child: T.all(Module, T::Props::ClassMethods), prop: Symbol).returns(T::Boolean).checked(:never)}
+  private def clobber_getter?(child, prop)
+    !!(child.decorator.method(:prop_get).owner != method(:prop_get).owner &&
+       child.instance_method(prop).source_location&.first == __FILE__)
+  end
+
+  sig {params(child: T.all(Module, T::Props::ClassMethods), prop: Symbol).returns(T::Boolean).checked(:never)}
+  private def clobber_setter?(child, prop)
+    !!(child.decorator.method(:prop_set).owner != method(:prop_set).owner &&
+       child.instance_method("#{prop}=").source_location&.first == __FILE__)
+  end
+
   sig {params(mod: Module).void.checked(:never)}
   def plugin(mod)
     decorated_class.plugins << mod

data/lib/types/props/generated_code_validation.rb

@@ -38,7 +38,7 @@ module T::Props
       assert_equal(s(:lvar, :found), ret)
 
       prop_clauses.each_with_index do |clause, i|
-        if i % 2 == 0
+        if i.even?
           validate_deserialize_hash_read(clause)
         else
           validate_deserialize_ivar_set(clause)
@@ -251,17 +251,17 @@ module T::Props
     # Method calls generated by SerdeTransform
     private_class_method def self.whitelisted_methods_for_serialize
       @whitelisted_methods_for_serialize ||= {
-        :lvar => %i{dup map transform_values transform_keys each_with_object nil? []= serialize},
-        :ivar => %i{dup map transform_values transform_keys each_with_object serialize},
-        :const => %i{checked_serialize deep_clone_object},
+        lvar: %i{dup map transform_values transform_keys each_with_object nil? []= serialize},
+        ivar: %i[dup map transform_values transform_keys each_with_object serialize],
+        const: %i[checked_serialize deep_clone_object],
       }
     end
 
     # Method calls generated by SerdeTransform
     private_class_method def self.whitelisted_methods_for_deserialize
       @whitelisted_methods_for_deserialize ||= {
-        :lvar => %i{dup map transform_values transform_keys each_with_object nil? []= to_f},
-        :const => %i{deserialize from_hash deep_clone_object soft_assert_handler},
+        lvar: %i{dup map transform_values transform_keys each_with_object nil? []= to_f},
+        const: %i[deserialize from_hash deep_clone_object soft_assert_handler],
       }
     end
   end

data/lib/types/props/has_lazily_specialized_methods.rb

@@ -84,7 +84,7 @@ module T::Props
 
         cls = decorated_class
         cls.class_eval(source)
-        lazily_defined_methods.keys.each {|name| cls.send(:private, name)}
+        lazily_defined_methods.each_key {|name| cls.send(:private, name)}
         lazily_defined_methods.clear
       end
     end

data/lib/types/props/private/deserializer_generator.rb

@@ -48,12 +48,12 @@ module T::Props
             SerdeTransform::Mode::DESERIALIZE,
             'val'
           )
-          if transformation
+          transformed_val = if transformation
             # Rescuing exactly NoMethodError is intended as a temporary hack
             # to preserve the semantics from before codegen. More generally
             # we are inconsistent about typechecking on deser and need to decide
             # our strategy here.
-            transformed_val = <<~RUBY
+            <<~RUBY
               begin
                 #{transformation}
               rescue NoMethodError => e
@@ -71,7 +71,7 @@ module T::Props
               end
             RUBY
           else
-            transformed_val = 'val'
+            'val'
           end
 
           nil_handler = generate_nil_handler(

data/lib/types/props/private/serde_transform.rb

@@ -99,20 +99,18 @@ module T::Props
             else
               "#{varname}.nil? ? nil : #{inner}"
             end
-          else
+          elsif type.types.all? {|t| generate(t, mode, varname).nil?}
             # Handle, e.g., T::Boolean
-            if type.types.all? {|t| generate(t, mode, varname).nil?}
-              nil
-            else
-              # We currently deep_clone_object if the type was T.any(Integer, Float).
-              # When we get better support for union types (maybe this specific
-              # union type, because it would be a replacement for
-              # Chalk::ODM::DeprecatedNumemric), we could opt to special case
-              # this union to have no specific serde transform (the only reason
-              # why Float has a special case is because round tripping through
-              # JSON might normalize Floats to Integers)
-              "T::Props::Utils.deep_clone_object(#{varname})"
-            end
+            nil
+          else
+            # We currently deep_clone_object if the type was T.any(Integer, Float).
+            # When we get better support for union types (maybe this specific
+            # union type, because it would be a replacement for
+            # Chalk::ODM::DeprecatedNumemric), we could opt to special case
+            # this union to have no specific serde transform (the only reason
+            # why Float has a special case is because round tripping through
+            # JSON might normalize Floats to Integers)
+            "T::Props::Utils.deep_clone_object(#{varname})"
           end
         when T::Types::Intersection
           dynamic_fallback = "T::Props::Utils.deep_clone_object(#{varname})"

data/lib/types/props/private/setter_factory.rb

@@ -58,10 +58,7 @@ module T::Props
           # from disk) are correct, so we use more thorough runtime
           # checks there
           if non_nil_type.recursively_valid?(val)
-            if validate
-              validate.call(prop, val)
-            end
-            instance_variable_set(accessor_key, val)
+            validate&.call(prop, val)
           else
             T::Props::Private::SetterFactory.raise_pretty_error(
               klass,
@@ -69,8 +66,8 @@ module T::Props
               non_nil_type,
               val,
             )
-            instance_variable_set(accessor_key, val)
           end
+          instance_variable_set(accessor_key, val)
         end
       end
 
@@ -94,9 +91,7 @@ module T::Props
           # from disk) are correct, so we use more thorough runtime
           # checks there
           elsif non_nil_type.recursively_valid?(val)
-            if validate
-              validate.call(prop, val)
-            end
+            validate&.call(prop, val)
             instance_variable_set(accessor_key, val)
           else
             T::Props::Private::SetterFactory.raise_pretty_error(

data/lib/types/props/serializable.rb

@@ -15,7 +15,7 @@ module T::Props::Serializable
   #   exception if this object has mandatory props with missing
   #   values.
   # @return [Hash] A serialization of this object.
-  def serialize(strict=true)
+  def serialize(strict=true) # rubocop:disable Style/OptionalBooleanParameter (changing this API is unfortunately not feasible)
     begin
       h = __t_props_generated_serialize(strict)
     rescue => e
@@ -56,7 +56,7 @@ module T::Props::Serializable
   #  the hash contains keys that do not correspond to any known
   #  props on this instance.
   # @return [void]
-  def deserialize(hash, strict=false)
+  def deserialize(hash, strict=false) # rubocop:disable Style/OptionalBooleanParameter (changing this API is unfortunately not feasible)
     begin
       hash_keys_matching_props = __t_props_generated_deserialize(hash)
     rescue => e
@@ -180,7 +180,7 @@ module T::Props::Serializable::DecoratorMethods
   def prop_dont_store?(prop); prop_rules(prop)[:dont_store]; end
   def prop_by_serialized_forms; @class.prop_by_serialized_forms; end
 
-  def from_hash(hash, strict=false)
+  def from_hash(hash, strict=false) # rubocop:disable Style/OptionalBooleanParameter (changing this API is unfortunately not feasible)
     raise ArgumentError.new("#{hash.inspect} provided to from_hash") if !(hash && hash.is_a?(Hash))
 
     i = @class.allocate
@@ -227,7 +227,7 @@ module T::Props::Serializable::DecoratorMethods
     source_lines = self.send(generate_source_method).split("\n")
     previous_blank = source_lines[0...line_num].rindex(&:empty?) || 0
     next_blank = line_num + (source_lines[line_num..-1]&.find_index(&:empty?) || 0)
-    context = "  " + source_lines[(previous_blank + 1)...(next_blank)].join("\n  ")
+    context = "  #{source_lines[(previous_blank + 1)...next_blank].join("\n  ")}"
     <<~MSG
       Error in #{decorated_class.name}##{generated_method}: #{error.message}
       at line #{line_num-previous_blank-1} in:
@@ -318,7 +318,7 @@ module T::Props::Serializable::ClassMethods
   # Allocate a new instance and call {#deserialize} to load a new
   # object from a hash.
   # @return [Serializable]
-  def from_hash(hash, strict=false)
+  def from_hash(hash, strict=false) # rubocop:disable Style/OptionalBooleanParameter (changing this API is unfortunately not feasible)
     self.decorator.from_hash(hash, strict)
   end
 

data/lib/types/props/type_validation.rb

@@ -4,7 +4,7 @@
 module T::Props::TypeValidation
   include T::Props::Plugin
 
-  BANNED_TYPES = [Object, BasicObject, Kernel]
+  BANNED_TYPES = [Object, BasicObject, Kernel].freeze
 
   class UnderspecifiedType < ArgumentError; end
 
@@ -13,7 +13,7 @@ module T::Props::TypeValidation
 
     sig {params(key: Symbol).returns(T::Boolean).checked(:never)}
     def valid_rule_key?(key)
-      super || :DEPRECATED_underspecified_type == key
+      super || key == :DEPRECATED_underspecified_type
     end
 
     sig do

data/lib/types/types/fixed_hash.rb

@@ -8,7 +8,7 @@ module T::Types
     attr_reader :types
 
     def initialize(types)
-      @types = types.each_with_object({}) {|(k, v), h| h[k] = T::Utils.coerce(v)}
+      @types = types.transform_values {|v| T::Utils.coerce(v)}
     end
 
     # @override Base

data/lib/types/types/type_variable.rb

@@ -7,7 +7,7 @@ module T::Types
   class TypeVariable < Base
     attr_reader :variance
 
-    VALID_VARIANCES = [:in, :out, :invariant]
+    VALID_VARIANCES = %i[in out invariant].freeze
 
     def initialize(variance)
       if !VALID_VARIANCES.include?(variance)

data/lib/types/types/typed_enumerable.rb

@@ -37,7 +37,7 @@ module T::Types
             return false unless @type.recursively_valid?(obj[it])
             it += 1
           end
-          return true
+          true
         end
       when Hash
         return false unless @type.is_a?(FixedArray)
@@ -50,10 +50,10 @@ module T::Types
           # iterate over a [key, value] array, so we can't juse use the @type.recursively_valid?(v)
           return false if !key_type.recursively_valid?(key) || !value_type.recursively_valid?(val)
         end
-        return true
+        true
       when Enumerator
         # Enumerators can be unbounded: see `[:foo, :bar].cycle`
-        return true
+        true
       when Range
         # A nil beginning or a nil end does not provide any type information. That is, nil in a range represents
         # boundlessness, it does not express a type. For example `(nil...nil)` is not a T::Range[NilClass], its a range
@@ -65,7 +65,7 @@ module T::Types
           return false unless @type.recursively_valid?(item)
         end
 
-        return true
+        true
       else
         # We don't check the enumerable since it isn't guaranteed to be
         # rewindable (e.g. STDIN) and it may be expensive to enumerate

data/lib/types/utils.rb

@@ -176,7 +176,7 @@ module T::Utils
     def self.get_type_info(prop_type)
       if prop_type.is_a?(T::Types::Union)
         non_nilable_type = T::Utils.unwrap_nilable(prop_type)
-        if non_nilable_type && non_nilable_type.is_a?(T::Types::Simple)
+        if non_nilable_type&.is_a?(T::Types::Simple)
           non_nilable_type = non_nilable_type.raw_type
         end
         TypeInfo.new(true, non_nilable_type)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sorbet-runtime
 version: !ruby/object:Gem::Version
-  version: 0.5.6105
+  version: 0.5.6130
 platform: ruby
 authors:
 - Stripe
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-11-24 00:00:00.000000000 Z
+date: 2020-12-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: minitest