sorbet-runtime 0.5.12214 → 0.5.12219

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 16a0c060e90e301e158fb992f5e30dd165de9d924e09a84afee967472738d2e7
-  data.tar.gz: b8647ade48a60151f782da8191652b5b091e5a01e9a96fae14986aaa203846e8
+  metadata.gz: 1f6086643f7a3f4f05ef2d0dbf300335b0944ffd4c216d7dd2e24c83851887ce
+  data.tar.gz: 8f327ece9b13165973052513a3c8c8d9c90e9769f06d2ddc31344d6830bc503d
 SHA512:
-  metadata.gz: c88768de97137ffca384ed5d658270c5cc3b502c7b16c5d320479265cf2885d3064852157bdd6589caff1904e1cee20681bf96a64e381fe2bb44aaa438b882b1
-  data.tar.gz: d5aa9e02d8d2fa3d0948663ef053075a61b237a27ffd82e64f93a08ec6544f4aee1e64040e02f032a33227ec0880037fbbf40a78b390c5e8864e06a9217e8da0
+  metadata.gz: 2992e679ddef4c96ad7916e088d76125ce9d50d727ef44c0476f5ddd628110644506c3497b2c16dddc3ff30b73d6344ea41fe02c6617b0ffaf793400de3d2073
+  data.tar.gz: '0648ef93b75218013bf5347e5bd2abeaac496f1290803cd97f4e33fd59a67ab76186f359b7842c9387bd84055ee93acba131877d4e291d65147fa042df569ef7'

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

@@ -589,6 +589,22 @@ module T::Private::Methods
     mod.extend(SingletonMethodHooks)
   end
 
+  # `name` must be an instance method (for class methods, pass in mod.singleton_class)
+  def self.visibility_method_name(mod, name)
+    if mod.public_method_defined?(name)
+      :public
+    elsif mod.protected_method_defined?(name)
+      :protected
+    elsif mod.private_method_defined?(name)
+      :private
+    else
+      # Raises a NameError formatted like the Ruby VM would (the exact text formatting
+      # of these errors changed across Ruby VM versions, in ways that would sometimes
+      # cause tests to fail if they were dependent on hard coding errors).
+      mod.method(name)
+    end
+  end
+
   # use this directly if you don't want/need to box up the method into an object to pass to method_to_key.
   private_class_method def self.method_owner_and_name_to_key(owner, name)
     "#{owner.object_id}##{name}"

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

@@ -12,7 +12,7 @@ module T::Private::Methods::CallValidation
   # @param method_sig [T::Private::Methods::Signature]
   # @return [UnboundMethod] the new wrapper method (or the original one if we didn't wrap it)
   def self.wrap_method_if_needed(mod, method_sig, original_method)
-    original_visibility = visibility_method_name(mod, method_sig.method_name)
+    original_visibility = T::Private::Methods.visibility_method_name(mod, method_sig.method_name)
     if method_sig.mode == T::Private::Methods::Modes.abstract
       create_abstract_wrapper(mod, method_sig, original_method, original_visibility)
     # Do nothing in this case; this method was not wrapped in _on_method_added.
@@ -330,19 +330,6 @@ module T::Private::Methods::CallValidation
       location: caller_loc
     )
   end
-
-  # `name` must be an instance method (for class methods, pass in mod.singleton_class)
-  private_class_method def self.visibility_method_name(mod, name)
-    if mod.public_method_defined?(name)
-      :public
-    elsif mod.protected_method_defined?(name)
-      :protected
-    elsif mod.private_method_defined?(name)
-      :private
-    else
-      mod.method(name) # Raises
-    end
-  end
 end
 
 if T::Configuration::AT_LEAST_RUBY_2_7

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

@@ -89,6 +89,7 @@ module T::Private::Methods::SignatureValidation
         validate_override_mode(signature, super_signature)
         validate_override_shape(signature, super_signature)
         validate_override_types(signature, super_signature)
+        validate_override_visibility(signature, super_signature)
       end
     else
       validate_non_override_mode(signature)
@@ -276,6 +277,38 @@ module T::Private::Methods::SignatureValidation
     end
   end
 
+  def self.validate_override_visibility(signature, super_signature)
+    return if super_signature.mode == Modes.untyped
+    # This departs from the behavior of other `validate_override_whatever` functions in that it
+    # only comes into effect when the child signature explicitly says the word `override`. This was
+    # done because the primary method for silencing these errors (`allow_incompatible: :visibility`)
+    # requires an `override` node to attach to. Once we have static override checking for implicitly
+    # overridden methods, we can remove this.
+    return unless [Modes.override, Modes.overridable_override].include?(signature.mode)
+    return if [:visibility, true].include?(signature.override_allow_incompatible)
+    method = signature.method
+    super_method = super_signature.method
+    mode_noun = super_signature.mode == Modes.abstract ? 'implementation' : 'override'
+    vis = method_visibility(method)
+    super_vis = method_visibility(super_method)
+
+    if visibility_strength(vis) > visibility_strength(super_vis)
+      raise "Incompatible visibility for #{mode_noun} of method #{method.name}\n" \
+            "* Base: #{super_vis} (in #{method_loc_str(super_method)})\n" \
+            "* #{mode_noun.capitalize}: #{vis} (in #{method_loc_str(method)})\n" \
+            "(The override must be at least as permissive as the supermethod)" \
+    end
+  end
+
+  private_class_method def self.method_visibility(method)
+    T::Private::Methods.visibility_method_name(method.owner, method.name)
+  end
+
+  # Higher = more restrictive.
+  private_class_method def self.visibility_strength(vis)
+    %i[public protected private].find_index(vis)
+  end
+
   private_class_method def self.base_override_loc_str(signature, super_signature)
     mode_noun = super_signature.mode == Modes.abstract ? 'Implementation' : 'Override'
     "\n * Base definition: in #{method_loc_str(super_signature.method)}" \

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sorbet-runtime
 version: !ruby/object:Gem::Version
-  version: 0.5.12214
+  version: 0.5.12219
 platform: ruby
 authors:
 - Stripe
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2025-07-01 00:00:00.000000000 Z
+date: 2025-07-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: minitest