spoom 1.6.0 → 1.6.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ebcf07d1aa065b643270105e211a05e9698befb8a67cf307c87c2fb957cd69fa
-  data.tar.gz: 4de71559b8d0812d7ab2456404cdf1ec840bd7031c7138a04e3233208ab8f827
+  metadata.gz: e2f7a10c996ca4d114ec5d34f9c0a08aa1e3ab5ed8ac1853997ff5725635d61c
+  data.tar.gz: c2cb5939f9fa17f7737dc9ca65f0a232bfc04241a0eeb21ec4770616f0990da9
 SHA512:
-  metadata.gz: ffd0520b3dcce46ea2ee2e00f4f7de974cedf21eddc706bed67616696f3f23406cbee80aeb62715c7dbfa7562a88486e21b6424b84b62d9f9a13b0a2d2f287b9
-  data.tar.gz: 7a7256c1a7114e46254d0cd648306ee8b1e9565522671318c710af33a571461da14b40e0d99405e63a12481f05c29b35bc3d0e660c8e7129928f1a9c34264ca9
+  metadata.gz: f4127b9180f965024a9e8afd64789bd21a2174e63933440e6c01085d54263b4f7edbc94a3e9869432c511f65dcc0c2ea8e2d1035e6e22a5f5b809a415ec49c24
+  data.tar.gz: 0b70e5d289ac509cbd45c8a69a5f27aeefe580d934acca250ac2feaa4d6b0a4f3cb73972e27c1524f05749b8d17f03c67fd0fe89b14a8e0528e9983576683ceb

data/README.md

@@ -221,6 +221,20 @@ Count the number of type-checking errors if all files were bumped to true:
 $ spoom srb bump --count-errors --dry
 ```
 
+#### Translate sigs between RBI and RBS
+
+Translate all file sigs from RBI to RBS:
+
+```
+$ spoom srb sigs translate
+```
+
+Translate one file's sigs from RBS to RBI:
+
+```
+$ spoom srb sigs translate --from rbs --to rbi /path/to/file.rb
+```
+
 #### Interact with Sorbet LSP mode
 
 **Experimental**

data/lib/spoom/backtrace_filter/minitest.rb

@@ -6,7 +6,7 @@ require "minitest"
 module Spoom
   module BacktraceFilter
     class Minitest < ::Minitest::BacktraceFilter
-      SORBET_PATHS = T.let(Gem.loaded_specs["sorbet-runtime"].full_require_paths.freeze, T::Array[String])
+      SORBET_PATHS = Gem.loaded_specs["sorbet-runtime"].full_require_paths.freeze #: Array[String]
 
       # @override
       #: (Array[String]? bt) -> Array[String]

data/lib/spoom/cli/helper.rb

@@ -56,7 +56,7 @@ module Spoom
       # Returns the context at `--path` (by default the current working directory)
       #: -> Context
       def context
-        @context ||= T.let(Context.new(exec_path), T.nilable(Context))
+        @context ||= Context.new(exec_path) #: Context?
       end
 
       # Raise if `spoom` is not ran inside a context with a `sorbet/config` file
@@ -81,15 +81,17 @@ module Spoom
       end
 
       # Collect files from `paths`, defaulting to `exec_path`
-      #: (Array[String] paths) -> Array[String]
-      def collect_files(paths)
+      #: (Array[String] paths, ?include_rbi_files: bool) -> Array[String]
+      def collect_files(paths, include_rbi_files: false)
         paths << exec_path if paths.empty?
 
         files = paths.flat_map do |path|
           if File.file?(path)
             path
           else
-            Dir.glob("#{path}/**/*.rb")
+            exts = ["rb"]
+            exts << "rbi" if include_rbi_files
+            Dir.glob("#{path}/**/*.{#{exts.join(",")}}")
           end
         end
 
@@ -104,7 +106,7 @@ module Spoom
       # Colors
 
       # Color used to highlight expressions in backticks
-      HIGHLIGHT_COLOR = T.let(Spoom::Color::BLUE, Spoom::Color)
+      HIGHLIGHT_COLOR = Spoom::Color::BLUE #: Spoom::Color
 
       # Is the `--color` option true?
       #: -> bool
@@ -118,7 +120,7 @@ module Spoom
 
         res = StringIO.new
         word = StringIO.new
-        in_ticks = T.let(false, T::Boolean)
+        in_ticks = false #: bool
         string.chars.each do |c|
           if c == "`" && !in_ticks
             in_ticks = true

data/lib/spoom/cli/srb/coverage.rb

@@ -89,7 +89,7 @@ module Spoom
 
             context.git_checkout!(ref: commit.sha)
 
-            snapshot = T.let(nil, T.nilable(Spoom::Coverage::Snapshot))
+            snapshot = nil #: Spoom::Coverage::Snapshot?
             if options[:bundle_install]
               Bundler.with_unbundled_env do
                 next unless bundle_install(path, commit.sha)

data/lib/spoom/cli/srb/sigs.rb

@@ -12,6 +12,12 @@ module Spoom
         desc "translate", "Translate signatures from/to RBI and RBS"
         option :from, type: :string, aliases: :f, desc: "From format", enum: ["rbi", "rbs"], default: "rbi"
         option :to, type: :string, aliases: :t, desc: "To format", enum: ["rbi", "rbs"], default: "rbs"
+        option :positional_names,
+          type: :boolean,
+          aliases: :p,
+          desc: "Use positional names when translating from RBI to RBS",
+          default: true
+        option :include_rbi_files, type: :boolean, desc: "Include RBI files", default: false
         def translate(*paths)
           from = options[:from]
           to = options[:to]
@@ -21,7 +27,7 @@ module Spoom
             exit(1)
           end
 
-          files = collect_files(paths)
+          files = collect_files(paths, include_rbi_files: options[:include_rbi_files])
 
           say("Translating signatures from `#{from}` to `#{to}` " \
             "in `#{files.size}` file#{files.size == 1 ? "" : "s"}...\n\n")
@@ -29,7 +35,7 @@ module Spoom
           case from
           when "rbi"
             transformed_files = transform_files(files) do |_file, contents|
-              Spoom::Sorbet::Sigs.rbi_to_rbs(contents)
+              Spoom::Sorbet::Sigs.rbi_to_rbs(contents, positional_names: options[:positional_names])
             end
           when "rbs"
             transformed_files = transform_files(files) do |_file, contents|
@@ -119,7 +125,7 @@ module Spoom
             gem "#{spec.name}", path: "#{copy_context.absolute_path}"
           RB
           exec(tapioca_context, "bundle install")
-          exec(tapioca_context, "bundle exec tapioca gem #{spec.name} --no-loc --no-file-header")
+          exec(tapioca_context, "bundle exec tapioca gem #{spec.name} --no-doc --no-loc --no-file-header")
 
           rbi_path = tapioca_context.glob("sorbet/rbi/gems/#{spec.name}@*.rbi").first
           unless rbi_path && tapioca_context.file?(rbi_path)

data/lib/spoom/cli/srb/tc.rb

@@ -22,6 +22,7 @@ module Spoom
         option :format, type: :string, aliases: :f, desc: "Format line output"
         option :uniq, type: :boolean, aliases: :u, desc: "Remove duplicated lines"
         option :count, type: :boolean, default: true, desc: "Show errors count"
+        option :junit_output_path, type: :string, desc: "Output failures to XML file formatted for JUnit"
         option :sorbet, type: :string, desc: "Path to custom Sorbet bin"
         option :sorbet_options, type: :string, default: "", desc: "Pass options to Sorbet"
         def tc(*paths_to_select)
@@ -32,6 +33,7 @@ module Spoom
           uniq = options[:uniq]
           format = options[:format]
           count = options[:count]
+          junit_output_path = options[:junit_output_path]
           sorbet = options[:sorbet]
 
           unless limit || code || sort
@@ -55,6 +57,12 @@ module Spoom
 
           if result.status
             say_error(result.err, status: nil, nl: false)
+            if junit_output_path
+              doc = Spoom::Sorbet::Errors.to_junit_xml([])
+              file = File.open(junit_output_path, "w")
+              doc.write(output: file, indent: 2)
+              file.close
+            end
             exit(0)
           end
 
@@ -94,6 +102,13 @@ module Spoom
             say_error(line, status: nil)
           end
 
+          if junit_output_path
+            doc = Spoom::Sorbet::Errors.to_junit_xml(errors)
+            file = File.open(junit_output_path, "w")
+            doc.write(output: file, indent: 2)
+            file.close
+          end
+
           if count
             if errors_count == errors.size
               say_error("Errors: #{errors_count}", status: nil)
@@ -132,7 +147,7 @@ module Spoom
           def colorize_message(message)
             return message unless color?
 
-            cyan = T.let(false, T::Boolean)
+            cyan = false #: bool
             word = StringIO.new
             message.chars.each do |c|
               if c == "`"

data/lib/spoom/context/exec.rb

@@ -31,7 +31,7 @@ module Spoom
       #: (String command, ?capture_err: bool) -> ExecResult
       def exec(command, capture_err: true)
         Bundler.with_unbundled_env do
-          opts = T.let({ chdir: absolute_path }, T::Hash[Symbol, T.untyped])
+          opts = { chdir: absolute_path } #: Hash[Symbol, untyped]
 
           if capture_err
             out, err, status = Open3.capture3(command, opts)

data/lib/spoom/context.rb

@@ -45,7 +45,7 @@ module Spoom
     # Call `#make!` to create it.
     #: (String absolute_path) -> void
     def initialize(absolute_path)
-      @absolute_path = T.let(::File.expand_path(absolute_path), String)
+      @absolute_path = ::File.expand_path(absolute_path) #: String
     end
   end
 end

data/lib/spoom/coverage/report.rb

@@ -40,7 +40,7 @@ module Spoom
 
       abstract!
 
-      TEMPLATE = T.let("#{Spoom::SPOOM_PATH}/templates/page.erb", String)
+      TEMPLATE = "#{Spoom::SPOOM_PATH}/templates/page.erb" #: String
 
       #: String
       attr_reader :title
@@ -88,7 +88,7 @@ module Spoom
       class Card < Template
         extend T::Sig
 
-        TEMPLATE = T.let("#{Spoom::SPOOM_PATH}/templates/card.erb", String)
+        TEMPLATE = "#{Spoom::SPOOM_PATH}/templates/card.erb" #: String
 
         #: String?
         attr_reader :title, :body
@@ -120,7 +120,7 @@ module Spoom
       end
 
       class Snapshot < Card
-        TEMPLATE = T.let("#{Spoom::SPOOM_PATH}/templates/card_snapshot.erb", String)
+        TEMPLATE = "#{Spoom::SPOOM_PATH}/templates/card_snapshot.erb" #: String
 
         #: Coverage::Snapshot
         attr_reader :snapshot

data/lib/spoom/coverage/snapshot.rb

@@ -25,7 +25,7 @@ module Spoom
       prop :sigils_excluding_rbis, T::Hash[String, Integer], default: Hash.new(0)
 
       # The strictness name as found in the Sorbet metrics file
-      STRICTNESSES = T.let(["ignore", "false", "true", "strict", "strong", "stdlib"].freeze, T::Array[String])
+      STRICTNESSES = ["ignore", "false", "true", "strict", "strong", "stdlib"].freeze #: Array[String]
 
       #: (?out: (IO | StringIO), ?colors: bool, ?indent_level: Integer) -> void
       def print(out: $stdout, colors: true, indent_level: 0)

data/lib/spoom/deadcode/index.rb

@@ -24,9 +24,9 @@ module Spoom
       #: (Model model) -> void
       def initialize(model)
         @model = model
-        @definitions = T.let({}, T::Hash[String, T::Array[Definition]])
-        @references = T.let({}, T::Hash[String, T::Array[Model::Reference]])
-        @ignored = T.let(Set.new, T::Set[Model::SymbolDef])
+        @definitions = {} #: Hash[String, Array[Definition]]
+        @references = {} #: Hash[String, Array[Model::Reference]]
+        @ignored = Set.new #: Set[Model::SymbolDef]
       end
 
       # Indexing

data/lib/spoom/deadcode/plugins/actionpack.rb

@@ -7,23 +7,20 @@ module Spoom
       class ActionPack < Base
         ignore_classes_inheriting_from("ApplicationController")
 
-        CALLBACKS = T.let(
-          [
-            "after_action",
-            "append_after_action",
-            "append_around_action",
-            "append_before_action",
-            "around_action",
-            "before_action",
-            "prepend_after_action",
-            "prepend_around_action",
-            "prepend_before_action",
-            "skip_after_action",
-            "skip_around_action",
-            "skip_before_action",
-          ].freeze,
-          T::Array[String],
-        )
+        CALLBACKS = [
+          "after_action",
+          "append_after_action",
+          "append_around_action",
+          "append_before_action",
+          "around_action",
+          "before_action",
+          "prepend_after_action",
+          "prepend_around_action",
+          "prepend_before_action",
+          "skip_after_action",
+          "skip_around_action",
+          "skip_before_action",
+        ].freeze #: Array[String]
 
         # @override
         #: (Model::Method definition) -> void

data/lib/spoom/deadcode/plugins/active_record.rb

@@ -15,58 +15,49 @@ module Spoom
           "to_param",
         )
 
-        CALLBACKS = T.let(
-          [
-            "after_commit",
-            "after_create_commit",
-            "after_create",
-            "after_destroy_commit",
-            "after_destroy",
-            "after_find",
-            "after_initialize",
-            "after_rollback",
-            "after_save_commit",
-            "after_save",
-            "after_touch",
-            "after_update_commit",
-            "after_update",
-            "after_validation",
-            "around_create",
-            "around_destroy",
-            "around_save",
-            "around_update",
-            "before_create",
-            "before_destroy",
-            "before_save",
-            "before_update",
-            "before_validation",
-          ].freeze,
-          T::Array[String],
-        )
+        CALLBACKS = [
+          "after_commit",
+          "after_create_commit",
+          "after_create",
+          "after_destroy_commit",
+          "after_destroy",
+          "after_find",
+          "after_initialize",
+          "after_rollback",
+          "after_save_commit",
+          "after_save",
+          "after_touch",
+          "after_update_commit",
+          "after_update",
+          "after_validation",
+          "around_create",
+          "around_destroy",
+          "around_save",
+          "around_update",
+          "before_create",
+          "before_destroy",
+          "before_save",
+          "before_update",
+          "before_validation",
+        ].freeze #: Array[String]
 
-        CRUD_METHODS = T.let(
-          [
-            "assign_attributes",
-            "create",
-            "create!",
-            "insert",
-            "insert!",
-            "new",
-            "update",
-            "update!",
-            "upsert",
-          ].freeze,
-          T::Array[String],
-        )
+        CRUD_METHODS = [
+          "assign_attributes",
+          "create",
+          "create!",
+          "insert",
+          "insert!",
+          "new",
+          "update",
+          "update!",
+          "upsert",
+        ].freeze #: Array[String]
 
-        ARRAY_METHODS = T.let(
-          [
-            "insert_all",
-            "insert_all!",
-            "upsert_all",
-          ].freeze,
-          T::Array[String],
-        )
+        ARRAY_METHODS = [
+          "insert_all",
+          "insert_all!",
+          "upsert_all",
+        ].freeze #: Array[String]
 
         # @override
         #: (Send send) -> void

data/lib/spoom/deadcode/plugins/active_support.rb

@@ -16,7 +16,7 @@ module Spoom
           "before_teardown",
         )
 
-        SETUP_AND_TEARDOWN_METHODS = T.let(["setup", "teardown"], T::Array[String])
+        SETUP_AND_TEARDOWN_METHODS = ["setup", "teardown"] #: Array[String]
 
         # @override
         #: (Send send) -> void

data/lib/spoom/deadcode/plugins/rubocop.rb

@@ -5,7 +5,7 @@ module Spoom
   module Deadcode
     module Plugins
       class Rubocop < Base
-        RUBOCOP_CONSTANTS = T.let(["MSG", "RESTRICT_ON_SEND"].to_set.freeze, T::Set[String])
+        RUBOCOP_CONSTANTS = ["MSG", "RESTRICT_ON_SEND"].to_set.freeze #: Set[String]
 
         ignore_classes_inheriting_from(
           "RuboCop::Cop::Cop",

data/lib/spoom/deadcode/plugins.rb

@@ -25,34 +25,28 @@ module Spoom
   module Deadcode
     DEFAULT_CUSTOM_PLUGINS_PATH = ".spoom/deadcode/plugins"
 
-    DEFAULT_PLUGINS = T.let(
-      Set.new([
-        Spoom::Deadcode::Plugins::Namespaces,
-        Spoom::Deadcode::Plugins::Ruby,
-      ]).freeze,
-      T::Set[T.class_of(Plugins::Base)],
-    )
+    DEFAULT_PLUGINS = Set.new([
+      Spoom::Deadcode::Plugins::Namespaces,
+      Spoom::Deadcode::Plugins::Ruby,
+    ]).freeze #: Set[singleton(Plugins::Base)]
 
-    PLUGINS_FOR_GEM = T.let(
-      {
-        "actionmailer" => Spoom::Deadcode::Plugins::ActionMailer,
-        "actionpack" => Spoom::Deadcode::Plugins::ActionPack,
-        "activejob" => Spoom::Deadcode::Plugins::ActiveJob,
-        "activemodel" => Spoom::Deadcode::Plugins::ActiveModel,
-        "activerecord" => Spoom::Deadcode::Plugins::ActiveRecord,
-        "activesupport" => Spoom::Deadcode::Plugins::ActiveSupport,
-        "graphql" => Spoom::Deadcode::Plugins::GraphQL,
-        "minitest" => Spoom::Deadcode::Plugins::Minitest,
-        "rails" => Spoom::Deadcode::Plugins::Rails,
-        "rake" => Spoom::Deadcode::Plugins::Rake,
-        "rspec" => Spoom::Deadcode::Plugins::RSpec,
-        "rubocop" => Spoom::Deadcode::Plugins::Rubocop,
-        "sorbet-runtime" => Spoom::Deadcode::Plugins::Sorbet,
-        "sorbet-static" => Spoom::Deadcode::Plugins::Sorbet,
-        "thor" => Spoom::Deadcode::Plugins::Thor,
-      }.freeze,
-      T::Hash[String, T.class_of(Plugins::Base)],
-    )
+    PLUGINS_FOR_GEM = {
+      "actionmailer" => Spoom::Deadcode::Plugins::ActionMailer,
+      "actionpack" => Spoom::Deadcode::Plugins::ActionPack,
+      "activejob" => Spoom::Deadcode::Plugins::ActiveJob,
+      "activemodel" => Spoom::Deadcode::Plugins::ActiveModel,
+      "activerecord" => Spoom::Deadcode::Plugins::ActiveRecord,
+      "activesupport" => Spoom::Deadcode::Plugins::ActiveSupport,
+      "graphql" => Spoom::Deadcode::Plugins::GraphQL,
+      "minitest" => Spoom::Deadcode::Plugins::Minitest,
+      "rails" => Spoom::Deadcode::Plugins::Rails,
+      "rake" => Spoom::Deadcode::Plugins::Rake,
+      "rspec" => Spoom::Deadcode::Plugins::RSpec,
+      "rubocop" => Spoom::Deadcode::Plugins::Rubocop,
+      "sorbet-runtime" => Spoom::Deadcode::Plugins::Sorbet,
+      "sorbet-static" => Spoom::Deadcode::Plugins::Sorbet,
+      "thor" => Spoom::Deadcode::Plugins::Thor,
+    }.freeze #: Hash[String, singleton(Plugins::Base)]
 
     class << self
       #: (Context context) -> Set[singleton(Plugins::Base)]

data/lib/spoom/deadcode/remover.rb

@@ -31,11 +31,11 @@ module Spoom
         #: (String source, Definition::Kind? kind, Location location) -> void
         def initialize(source, kind, location)
           @old_source = source
-          @new_source = T.let(source.dup, String)
+          @new_source = source.dup #: String
           @kind = kind
           @location = location
 
-          @node_context = T.let(NodeFinder.find(source, location, kind), NodeContext)
+          @node_context = NodeFinder.find(source, location, kind) #: NodeContext
         end
 
         #: -> void
@@ -99,8 +99,10 @@ module Spoom
           prev_node = context.previous_node
           next_node = context.next_node
 
-          if (prev_node && prev_node.location.end_line != node.location.start_line) &&
-              (next_node && next_node.location.start_line != node.location.end_line)
+          has_prev_node_on_different_line = prev_node && prev_node.location.end_line != node.location.start_line
+          has_next_node_on_different_line = next_node && next_node.location.start_line != node.location.end_line
+
+          if has_prev_node_on_different_line && has_next_node_on_different_line
             # We have a node before and after, but on different lines, we need to remove the whole line
             #
             # ~~~
@@ -164,8 +166,10 @@ module Spoom
           prev_node = context.previous_node
           next_node = context.next_node
 
-          if (prev_node && prev_node.location.end_line != context.node.location.start_line) &&
-              (next_node && next_node.location.start_line != context.node.location.end_line)
+          has_prev_node_on_different_line = prev_node && prev_node.location.end_line != context.node.location.start_line
+          has_next_node_on_different_line = next_node && next_node.location.start_line != context.node.location.end_line
+
+          if has_prev_node_on_different_line && has_next_node_on_different_line
             # We have a node before and after, but on different lines, we need to remove the whole line
             #
             # ~~~
@@ -275,7 +279,7 @@ module Spoom
 
           # Adjust the lines to remove to include previous blank lines
           prev_context = NodeContext.new(@old_source, @node_context.comments, first_node, context.nesting)
-          before = T.let(prev_context.previous_node, T.nilable(T.any(Prism::Node, Prism::Comment)))
+          before = prev_context.previous_node #: (Prism::Node | Prism::Comment)?
 
           # There may be an unrelated comment between the current node and the one before
           # if there is, we only want to delete lines up to the last comment found
@@ -329,7 +333,7 @@ module Spoom
 
         #: (Prism::CallNode node, name: String, kind: Definition::Kind?) -> String
         def transform_sig(node, name:, kind:)
-          type = T.let(nil, T.nilable(String))
+          type = nil #: String?
 
           block = T.cast(node.block, Prism::BlockNode)
           statements = T.cast(block.body, Prism::StatementsNode)
@@ -428,7 +432,7 @@ module Spoom
 
         #: -> NodeContext?
         def sclass_context
-          sclass = T.let(nil, T.nilable(Prism::SingletonClassNode))
+          sclass = nil #: Prism::SingletonClassNode?
 
           nesting = @nesting.dup
           until nesting.empty? || sclass
@@ -473,7 +477,7 @@ module Spoom
 
         #: (Integer start_line, Integer end_line) -> Array[Prism::Comment]
         def comments_between_lines(start_line, end_line)
-          comments = T.let([], T::Array[Prism::Comment])
+          comments = [] #: Array[Prism::Comment]
 
           (start_line + 1).upto(end_line - 1) do |line|
             comment = @comments[line]
@@ -485,7 +489,7 @@ module Spoom
 
         #: (Prism::Node node) -> Array[Prism::Comment]
         def attached_comments(node)
-          comments = T.let([], T::Array[Prism::Comment])
+          comments = [] #: Array[Prism::Comment]
 
           start_line = node.location.start_line - 1
           start_line.downto(1) do |line|
@@ -500,7 +504,7 @@ module Spoom
 
         #: -> Array[Prism::Node]
         def attached_sigs
-          nodes = T.let([], T::Array[Prism::Node])
+          nodes = [] #: Array[Prism::Node]
 
           previous_nodes.reverse_each do |prev_node|
             break unless sorbet_signature?(prev_node)
@@ -553,12 +557,9 @@ module Spoom
               raise Error, "Can't find node at #{location}, expected #{kind} but got #{node.class}"
             end
 
-            comments_by_line = T.let(
-              result.comments.to_h do |comment|
-                [comment.location.start_line, comment]
-              end,
-              T::Hash[Integer, Prism::Comment],
-            )
+            comments_by_line = result.comments.to_h do |comment|
+              [comment.location.start_line, comment]
+            end #: Hash[Integer, Prism::Comment]
 
             NodeContext.new(source, comments_by_line, node, visitor.nodes_nesting)
           end
@@ -599,8 +600,8 @@ module Spoom
           super()
           @location = location
           @kind = kind
-          @node = T.let(nil, T.nilable(Prism::Node))
-          @nodes_nesting = T.let([], T::Array[Prism::Node])
+          @node = nil #: Prism::Node?
+          @nodes_nesting = [] #: Array[Prism::Node]
         end
 
         # @override

data/lib/spoom/file_collector.rb

@@ -16,7 +16,7 @@ module Spoom
     # the list.
     #: (?allow_extensions: Array[String], ?allow_mime_types: Array[String], ?exclude_patterns: Array[String]) -> void
     def initialize(allow_extensions: [], allow_mime_types: [], exclude_patterns: [])
-      @files = T.let([], T::Array[String])
+      @files = [] #: Array[String]
       @allow_extensions = allow_extensions
       @allow_mime_types = allow_mime_types
       @exclude_patterns = exclude_patterns