Minor type safety improvements

Library/Homebrew/cask/cask_loader.rb

@@ -56,24 +56,21 @@ module Cask
           .returns(T.nilable(T.attached_class))
       }
       def self.try_new(ref, warn: false)
-        case ref
+        return if ref.is_a?(Cask)
-        when Cask, URI::Generic
-          # do nothing
-        else
-          content = ref.to_str
 
-          # Cache compiled regex
+        content = ref.to_str
-          @regex ||= begin
-            token  = /(?:"[^"]*"|'[^']*')/
-            curly  = /\(\s*#{token.source}\s*\)\s*\{.*\}/
-            do_end = /\s+#{token.source}\s+do(?:\s*;\s*|\s+).*end/
-            /\A\s*cask(?:#{curly.source}|#{do_end.source})\s*\Z/m
-          end
 
-          return unless content.match?(@regex)
+        # Cache compiled regex
-
+        @regex ||= begin
-          new(content)
+          token  = /(?:"[^"]*"|'[^']*')/
+          curly  = /\(\s*#{token.source}\s*\)\s*\{.*\}/
+          do_end = /\s+#{token.source}\s+do(?:\s*;\s*|\s+).*end/
+          /\A\s*cask(?:#{curly.source}|#{do_end.source})\s*\Z/m
         end
+
+        return unless content.match?(@regex)
+
+        new(content)
       end
 
       sig { params(content: String, tap: Tap).void }

Library/Homebrew/cask/url.rb

@@ -155,7 +155,7 @@ module Cask
       # @api public
       def method_missing(method, *args, &block)
         if @dsl.respond_to?(method)
-          T.unsafe(@dsl).public_send(method, *args, &block)
+          @dsl.public_send(method, *args, &block)
         else
           super
         end

Library/Homebrew/cmd/home.rb

@@ -36,7 +36,7 @@ module Homebrew
           formula_or_cask.homepage
         end
 
-        exec_browser(*T.unsafe(homepages))
+        exec_browser(*homepages)
       end
 
       private

Library/Homebrew/dev-cmd/tests.rb

@@ -50,9 +50,9 @@ module Homebrew
 
           parallel = true
 
-          files = if args.only
+          only = args.only
-            # FIXME: This is safe once args are namespaced by command
+          files = if only
-            test_name, line = T.unsafe(args.only).split(":", 2)
+            test_name, line = only.split(":", 2)
 
             if line.nil?
               Dir.glob("test/{#{test_name},#{test_name}/**/*}_spec.rb")
@@ -67,7 +67,7 @@ module Homebrew
           end
 
           if files.blank?
-            raise UsageError, "The `--only` argument requires a valid file or folder name!" if args.only
+            raise UsageError, "The `--only` argument requires a valid file or folder name!" if only
 
             if args.changed?
               opoo "No tests are directly associated with the changed files!"

Library/Homebrew/formula.rb

@@ -3124,7 +3124,7 @@ class Formula
     removed = ENV.remove_cc_etc
 
     begin
-      T.unsafe(self).system("xcodebuild", *args)
+      self.system("xcodebuild", *args)
     ensure
       ENV.update(removed)
     end

Library/Homebrew/source_location.rb

@@ -11,7 +11,7 @@ module Homebrew
     attr_reader :column
 
     sig { params(line: Integer, column: T.nilable(Integer)).void }
-    def initialize(line, column = T.unsafe(nil))
+    def initialize(line, column = nil)
       @line = line
       @column = column
     end

Library/Homebrew/version.rb

@@ -731,12 +731,10 @@ class Version
   def to_s = version.to_s
 
   sig { params(options: T.untyped).returns(String) }
-  def to_json(*options)
+  def to_json(*options) = version.to_json(*options)
-    T.unsafe(version).to_json(*options)
-  end
 
   sig { params(method: T.any(Symbol, String), include_all: T::Boolean).returns(T::Boolean) }
-  def respond_to?(method, include_all = T.unsafe(nil))
+  def respond_to?(method, include_all = false) # rubocop:disable Style/OptionalBooleanParameter (`Object` override)
     return !null? if ["to_str", :to_str].include?(method)
 
     super