annotaterb 4.1.0 → 4.2.0

data/lib/annotate_rb/model_annotator/index_annotation/annotation_builder.rb

@@ -0,0 +1,113 @@
+# frozen_string_literal: true
+
+module AnnotateRb
+  module ModelAnnotator
+    module IndexAnnotation
+      class AnnotationBuilder
+        INDEX_CLAUSES = {
+          unique: {
+            default: "UNIQUE",
+            markdown: "_unique_"
+          },
+          where: {
+            default: "WHERE",
+            markdown: "_where_"
+          },
+          using: {
+            default: "USING",
+            markdown: "_using_"
+          }
+        }.freeze
+
+        def initialize(model, options)
+          @model = model
+          @options = options
+        end
+
+        def build
+          index_info = if @options[:format_markdown]
+            "#\n# ### Indexes\n#\n"
+          else
+            "#\n# Indexes\n#\n"
+          end
+
+          indexes = @model.retrieve_indexes_from_table
+          return "" if indexes.empty?
+
+          max_size = indexes.collect { |index| index.name.size }.max + 1
+          indexes.sort_by(&:name).each do |index|
+            index_info += if @options[:format_markdown]
+              final_index_string_in_markdown(index)
+            else
+              final_index_string(index, max_size)
+            end
+          end
+
+          index_info
+        end
+
+        private
+
+        def index_using_info(index, format = :default)
+          value = index.try(:using) && index.using.try(:to_sym)
+          if !value.blank? && value != :btree
+            " #{INDEX_CLAUSES[:using][format]} #{value}"
+          else
+            ""
+          end
+        end
+
+        def index_where_info(index, format = :default)
+          value = index.try(:where).try(:to_s)
+          if value.blank?
+            ""
+          else
+            " #{INDEX_CLAUSES[:where][format]} #{value}"
+          end
+        end
+
+        def index_unique_info(index, format = :default)
+          index.unique ? " #{INDEX_CLAUSES[:unique][format]}" : ""
+        end
+
+        def final_index_string_in_markdown(index)
+          details = format(
+            "%s%s%s",
+            index_unique_info(index, :markdown),
+            index_where_info(index, :markdown),
+            index_using_info(index, :markdown)
+          ).strip
+          details = " (#{details})" unless details.blank?
+
+          format(
+            "# * `%s`%s:\n#     * **`%s`**\n",
+            index.name,
+            details,
+            index_columns_info(index).join("`**\n#     * **`")
+          )
+        end
+
+        def final_index_string(index, max_size)
+          format(
+            "#  %-#{max_size}.#{max_size}s %s%s%s%s",
+            index.name,
+            "(#{index_columns_info(index).join(",")})",
+            index_unique_info(index),
+            index_where_info(index),
+            index_using_info(index)
+          ).rstrip + "\n"
+        end
+
+        def index_columns_info(index)
+          Array(index.columns).map do |col|
+            if index.try(:orders) && index.orders[col.to_s]
+              "#{col} #{index.orders[col.to_s].upcase}"
+            else
+              col.to_s.gsub("\r", '\r').gsub("\n", '\n')
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/annotate_rb/model_annotator/index_annotation.rb

@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+module AnnotateRb
+  module ModelAnnotator
+    module IndexAnnotation
+      autoload :AnnotationBuilder, "annotate_rb/model_annotator/index_annotation/annotation_builder"
+    end
+  end
+end

data/lib/annotate_rb/model_annotator/magic_comment_parser.rb

@@ -0,0 +1,32 @@
+# frozen_string_literal: true
+
+module AnnotateRb
+  module ModelAnnotator
+    # Extracts magic comments strings and returns them
+    class MagicCommentParser
+      MAGIC_COMMENTS = [
+        HASH_ENCODING = /(^#\s*encoding:.*(?:\n|r\n))/,
+        HASH_CODING = /(^# coding:.*(?:\n|\r\n))/,
+        HASH_FROZEN_STRING = /(^#\s*frozen_string_literal:.+(?:\n|\r\n))/,
+        STAR_ENCODING = /(^# -\*- encoding\s?:.*(?:\n|\r\n))/,
+        STAR_CODING = /(^# -\*- coding:.*(?:\n|\r\n))/,
+        STAR_FROZEN_STRING = /(^# -\*- frozen_string_literal\s*:.+-\*-(?:\n|\r\n))/,
+        SORBET_TYPED_STRING = /(^#\s*typed:.*(?:\n|r\n))/.freeze
+      ].freeze
+
+      MAGIC_COMMENTS_REGEX = Regexp.union(*MAGIC_COMMENTS).freeze
+
+      class << self
+        def call(content)
+          magic_comments = content.scan(MAGIC_COMMENTS_REGEX).flatten.compact
+
+          if magic_comments.any?
+            magic_comments.join
+          else
+            ""
+          end
+        end
+      end
+    end
+  end
+end

data/lib/annotate_rb/model_annotator/model_class_getter.rb

@@ -8,8 +8,8 @@ module AnnotateRb
         # Check for namespaced models in subdirectories as well as models
         # in subdirectories without namespacing.
         def call(file, options)
-          model_path = file.gsub(/\.rb$/, '')
-          options[:model_dir].each { |dir| model_path = model_path.gsub(/^#{dir}/, '').gsub(/^\//, '') }
+          model_path = file.gsub(/\.rb$/, "")
+          options[:model_dir].each { |dir| model_path = model_path.gsub(/^#{dir}/, "").gsub(/^\//, "") }
 
           begin
             get_loaded_model(model_path, file) || raise(BadModelFileError.new)
@@ -18,8 +18,8 @@ module AnnotateRb
             file_path = File.expand_path(file)
             if File.file?(file_path) && Kernel.require(file_path)
               retry
-            elsif model_path =~ /\//
-              model_path = model_path.split('/')[1..-1].join('/').to_s
+            elsif /\//.match?(model_path)
+              model_path = model_path.split("/")[1..-1].join("/").to_s
               retry
             else
               raise
@@ -39,7 +39,7 @@ module AnnotateRb
           absolute_file = File.expand_path(file)
           model_paths =
             $LOAD_PATH.select { |path| absolute_file.include?(path) }
-                      .map { |path| absolute_file.sub(path, '').sub(/\.rb$/, '').sub(/^\//, '') }
+              .map { |path| absolute_file.sub(path, "").sub(/\.rb$/, "").sub(/^\//, "") }
           model_paths
             .map { |path| get_loaded_model_by_path(path) }
             .find { |loaded_model| !loaded_model.nil? }
@@ -51,7 +51,7 @@ module AnnotateRb
         rescue StandardError, LoadError
           # Revert to the old way but it is not really robust
           ObjectSpace.each_object(::Class)
-                     .select do |c|
+            .select do |c|
             Class === c && # note: we use === to avoid a bug in activesupport 2.3.14 OptionMerger vs. is_a?
               c.ancestors.respond_to?(:include?) && # to fix FactoryGirl bug, see https://github.com/ctran/annotate_models/pull/82
               c.ancestors.include?(::ActiveRecord::Base)

data/lib/annotate_rb/model_annotator/model_files_getter.rb

@@ -20,19 +20,21 @@ module AnnotateRb
           options[:model_dir].each do |dir|
             Dir.chdir(dir) do
               list = if options[:ignore_model_sub_dir]
-                       Dir["*.rb"].map { |f| [dir, f] }
-                     else
-                       Dir["**/*.rb"].reject { |f| f["concerns/"] }.map { |f| [dir, f] }
-                     end
+                Dir["*.rb"].map { |f| [dir, f] }
+              else
+                Dir["**/*.rb"]
+                  .reject { |f| f["concerns/"] }
+                  .map { |f| [dir, f] }
+              end
               model_files.concat(list)
             end
           end
 
           model_files
         rescue SystemCallError
-          $stderr.puts "No models found in directory '#{options[:model_dir].join("', '")}'."
-          $stderr.puts "Either specify models on the command line, or use the --model-dir option."
-          $stderr.puts "Call 'annotaterb --help' for more info."
+          warn "No models found in directory '#{options[:model_dir].join("', '")}'."
+          warn "Either specify models on the command line, or use the --model-dir option."
+          warn "Call 'annotaterb --help' for more info."
           # exit 1 # TODO: Return exit code back to caller. Right now it messes up RSpec being able to run
         end
 
@@ -47,12 +49,12 @@ module AnnotateRb
             absolute_dir_path = File.expand_path(dir)
             specified_files
               .find_all { |file| file.start_with?(absolute_dir_path) }
-              .map { |file| [dir, file.sub("#{absolute_dir_path}/", '')] }
+              .map { |file| [dir, file.sub("#{absolute_dir_path}/", "")] }
           end
 
           if model_files.size != specified_files.size
-            $stderr.puts "The specified file could not be found in directory '#{options[:model_dir].join("', '")}'."
-            $stderr.puts "Call 'annotaterb --help' for more info."
+            warn "The specified file could not be found in directory '#{options[:model_dir].join("', '")}'."
+            warn "Call 'annotaterb --help' for more info."
             # exit 1 # TODO: Return exit code back to caller. Right now it messes up RSpec being able to run
           end
 

data/lib/annotate_rb/model_annotator/model_wrapper.rb

@@ -13,22 +13,23 @@ module AnnotateRb
 
       # Gets the columns of the ActiveRecord model, processes them, and then returns them.
       def columns
-        @columns ||= begin
-                       cols = raw_columns
-                       cols += translated_columns
-
-                       ignore_columns = @options[:ignore_columns]
-                       if ignore_columns
-                         cols = cols.reject do |col|
-                           col.name.match(/#{ignore_columns}/)
-                         end
-                       end
-
-                       cols = cols.sort_by(&:name) if @options[:sort]
-                       cols = classified_sort(cols) if @options[:classified_sort]
-
-                       cols
-                     end
+        @columns ||=
+          begin
+            cols = raw_columns
+            cols += translated_columns
+
+            ignore_columns = @options[:ignore_columns]
+            if ignore_columns
+              cols = cols.reject do |col|
+                col.name.match(/#{ignore_columns}/)
+              end
+            end
+
+            cols = cols.sort_by(&:name) if @options[:sort]
+            cols = classified_sort(cols) if @options[:classified_sort]
+
+            cols
+          end
       end
 
       def connection
@@ -75,19 +76,25 @@ module AnnotateRb
       # Calculates the max width of the schema for the model by looking at the columns, schema comments, with respect
       # to the options.
       def max_schema_info_width
-        cols = columns
-
-        if with_comments?
-          max_size = cols.map do |column|
-            column.name.size + (column.comment ? Helper.width(column.comment) : 0)
-          end.max || 0
-          max_size += 2
-        else
-          max_size = cols.map(&:name).map(&:size).max
-        end
-        max_size += @options[:format_rdoc] ? 5 : 1
+        @max_schema_info_width ||=
+          begin
+            cols = columns
+
+            if with_comments?
+              column_widths = cols.map do |column|
+                column.name.size + (column.comment ? Helper.width(column.comment) : 0)
+              end
 
-        max_size
+              max_size = column_widths.max || 0
+              max_size += 2
+            else
+              max_size = cols.map(&:name).map(&:size).max
+            end
+
+            max_size += @options[:format_rdoc] ? 5 : 1
+
+            max_size
+          end
       end
 
       def retrieve_indexes_from_table
@@ -98,12 +105,12 @@ module AnnotateRb
         return indexes if indexes.any? || !@klass.table_name_prefix
 
         # Try to search the table without prefix
-        table_name_without_prefix = table_name.to_s.sub(@klass.table_name_prefix, '')
+        table_name_without_prefix = table_name.to_s.sub(@klass.table_name_prefix, "")
         @klass.connection.indexes(table_name_without_prefix)
       end
 
       def with_comments?
-        @options[:with_comment] &&
+        @with_comments ||= @options[:with_comment] &&
           raw_columns.first.respond_to?(:comment) &&
           raw_columns.map(&:comment).any? { |comment| !comment.nil? }
       end
@@ -115,11 +122,11 @@ module AnnotateRb
         id = nil
 
         cols.each do |c|
-          if c.name.eql?('id')
+          if c.name.eql?("id")
             id = c
-          elsif c.name.eql?('created_at') || c.name.eql?('updated_at')
+          elsif c.name.eql?("created_at") || c.name.eql?("updated_at")
             timestamps << c
-          elsif c.name[-3, 3].eql?('_id')
+          elsif c.name[-3, 3].eql?("_id")
             associations << c
           else
             rest_cols << c
@@ -137,9 +144,9 @@ module AnnotateRb
         # eg. Model: Car, foreign column name: car_id
         foreign_column_name = [
           @klass.translation_class.to_s
-               .gsub('::Translation', '').gsub('::', '_')
-               .downcase,
-          '_id'
+            .gsub("::Translation", "").gsub("::", "_")
+            .downcase,
+          "_id"
         ].join.to_sym
 
         [
@@ -152,4 +159,4 @@ module AnnotateRb
       end
     end
   end
-end
+end

data/lib/annotate_rb/model_annotator/pattern_getter.rb

@@ -3,25 +3,157 @@
 module AnnotateRb
   module ModelAnnotator
     class PatternGetter
+      module FilePatterns
+        # Controller files
+        CONTROLLER_DIR = File.join("app", "controllers")
+
+        # Active admin registry files
+        ACTIVEADMIN_DIR = File.join("app", "admin")
+
+        # Helper files
+        HELPER_DIR = File.join("app", "helpers")
+
+        # File.join for windows reverse bar compat?
+        # I dont use windows, can`t test
+        UNIT_TEST_DIR = File.join("test", "unit")
+        MODEL_TEST_DIR = File.join("test", "models") # since rails 4.0
+        SPEC_MODEL_DIR = File.join("spec", "models")
+
+        FIXTURE_TEST_DIR = File.join("test", "fixtures")
+        FIXTURE_SPEC_DIR = File.join("spec", "fixtures")
+
+        # Other test files
+        CONTROLLER_TEST_DIR = File.join("test", "controllers")
+        CONTROLLER_SPEC_DIR = File.join("spec", "controllers")
+        REQUEST_SPEC_DIR = File.join("spec", "requests")
+        ROUTING_SPEC_DIR = File.join("spec", "routing")
+
+        # Object Daddy http://github.com/flogic/object_daddy/tree/master
+        EXEMPLARS_TEST_DIR = File.join("test", "exemplars")
+        EXEMPLARS_SPEC_DIR = File.join("spec", "exemplars")
+
+        # Machinist http://github.com/notahat/machinist
+        BLUEPRINTS_TEST_DIR = File.join("test", "blueprints")
+        BLUEPRINTS_SPEC_DIR = File.join("spec", "blueprints")
+
+        # Factory Bot https://github.com/thoughtbot/factory_bot
+        FACTORY_BOT_TEST_DIR = File.join("test", "factories")
+        FACTORY_BOT_SPEC_DIR = File.join("spec", "factories")
+
+        # Fabrication https://github.com/paulelliott/fabrication.git
+        FABRICATORS_TEST_DIR = File.join("test", "fabricators")
+        FABRICATORS_SPEC_DIR = File.join("spec", "fabricators")
+
+        # Serializers https://github.com/rails-api/active_model_serializers
+        SERIALIZERS_DIR = File.join("app", "serializers")
+        SERIALIZERS_TEST_DIR = File.join("test", "serializers")
+        SERIALIZERS_SPEC_DIR = File.join("spec", "serializers")
+      end
+
       class << self
         def call(options, pattern_types = [])
-          current_patterns = []
+          new(options, pattern_types).get
+        end
+      end
+
+      def initialize(options, pattern_types = [])
+        @options = options
+        @pattern_types = pattern_types
+      end
 
-          options[:root_dir].each do |root_directory|
-            Array(pattern_types).each do |pattern_type|
-              patterns = FilePatterns.generate(root_directory, pattern_type, options)
+      def get
+        current_patterns = []
 
-              current_patterns += if pattern_type.to_sym == :additional_file_patterns
-                                    patterns
-                                  else
-                                    patterns.map { |p| p.sub(/^[\/]*/, '') }
-                                  end
+        @options[:root_dir].each do |root_directory|
+          Array(@pattern_types).each do |pattern_type|
+            patterns = generate(root_directory, pattern_type)
+
+            current_patterns += if pattern_type.to_sym == :additional_file_patterns
+              patterns
+            else
+              patterns.map { |p| p.sub(/^\/*/, "") }
             end
           end
+        end
+
+        current_patterns
+      end
 
-          current_patterns
+      private
+
+      def generate(root_directory, pattern_type)
+        case pattern_type
+        when "test" then test_files(root_directory)
+        when "fixture" then fixture_files(root_directory)
+        when "scaffold" then scaffold_files(root_directory)
+        when "factory" then factory_files(root_directory)
+        when "serializer" then serialize_files(root_directory)
+        when "additional_file_patterns"
+          [@options[:additional_file_patterns] || []].flatten
+        when "controller"
+          [File.join(root_directory, FilePatterns::CONTROLLER_DIR, "%PLURALIZED_MODEL_NAME%_controller.rb")]
+        when "admin"
+          [
+            File.join(root_directory, FilePatterns::ACTIVEADMIN_DIR, "%MODEL_NAME%.rb"),
+            File.join(root_directory, FilePatterns::ACTIVEADMIN_DIR, "%PLURALIZED_MODEL_NAME%.rb")
+          ]
+        when "helper"
+          [File.join(root_directory, FilePatterns::HELPER_DIR, "%PLURALIZED_MODEL_NAME%_helper.rb")]
+        else
+          []
         end
       end
+
+      def test_files(root_directory)
+        [
+          File.join(root_directory, FilePatterns::UNIT_TEST_DIR, "%MODEL_NAME%_test.rb"),
+          File.join(root_directory, FilePatterns::MODEL_TEST_DIR, "%MODEL_NAME%_test.rb"),
+          File.join(root_directory, FilePatterns::SPEC_MODEL_DIR, "%MODEL_NAME%_spec.rb")
+        ]
+      end
+
+      def fixture_files(root_directory)
+        [
+          File.join(root_directory, FilePatterns::FIXTURE_TEST_DIR, "%TABLE_NAME%.yml"),
+          File.join(root_directory, FilePatterns::FIXTURE_SPEC_DIR, "%TABLE_NAME%.yml"),
+          File.join(root_directory, FilePatterns::FIXTURE_TEST_DIR, "%PLURALIZED_MODEL_NAME%.yml"),
+          File.join(root_directory, FilePatterns::FIXTURE_SPEC_DIR, "%PLURALIZED_MODEL_NAME%.yml")
+        ]
+      end
+
+      def scaffold_files(root_directory)
+        [
+          File.join(root_directory, FilePatterns::CONTROLLER_TEST_DIR, "%PLURALIZED_MODEL_NAME%_controller_test.rb"),
+          File.join(root_directory, FilePatterns::CONTROLLER_SPEC_DIR, "%PLURALIZED_MODEL_NAME%_controller_spec.rb"),
+          File.join(root_directory, FilePatterns::REQUEST_SPEC_DIR, "%PLURALIZED_MODEL_NAME%_spec.rb"),
+          File.join(root_directory, FilePatterns::ROUTING_SPEC_DIR, "%PLURALIZED_MODEL_NAME%_routing_spec.rb")
+        ]
+      end
+
+      def factory_files(root_directory)
+        [
+          File.join(root_directory, FilePatterns::EXEMPLARS_TEST_DIR, "%MODEL_NAME%_exemplar.rb"),
+          File.join(root_directory, FilePatterns::EXEMPLARS_SPEC_DIR, "%MODEL_NAME%_exemplar.rb"),
+          File.join(root_directory, FilePatterns::BLUEPRINTS_TEST_DIR, "%MODEL_NAME%_blueprint.rb"),
+          File.join(root_directory, FilePatterns::BLUEPRINTS_SPEC_DIR, "%MODEL_NAME%_blueprint.rb"),
+          File.join(root_directory, FilePatterns::FACTORY_BOT_TEST_DIR, "%MODEL_NAME%_factory.rb"), # (old style)
+          File.join(root_directory, FilePatterns::FACTORY_BOT_SPEC_DIR, "%MODEL_NAME%_factory.rb"), # (old style)
+          File.join(root_directory, FilePatterns::FACTORY_BOT_TEST_DIR, "%TABLE_NAME%.rb"), # (new style)
+          File.join(root_directory, FilePatterns::FACTORY_BOT_SPEC_DIR, "%TABLE_NAME%.rb"), # (new style)
+          File.join(root_directory, FilePatterns::FACTORY_BOT_TEST_DIR, "%PLURALIZED_MODEL_NAME%.rb"), # (new style)
+          File.join(root_directory, FilePatterns::FACTORY_BOT_SPEC_DIR, "%PLURALIZED_MODEL_NAME%.rb"), # (new style)
+          File.join(root_directory, FilePatterns::FABRICATORS_TEST_DIR, "%MODEL_NAME%_fabricator.rb"),
+          File.join(root_directory, FilePatterns::FABRICATORS_SPEC_DIR, "%MODEL_NAME%_fabricator.rb")
+        ]
+      end
+
+      def serialize_files(root_directory)
+        [
+          File.join(root_directory, FilePatterns::SERIALIZERS_DIR, "%MODEL_NAME%_serializer.rb"),
+          File.join(root_directory, FilePatterns::SERIALIZERS_TEST_DIR, "%MODEL_NAME%_serializer_test.rb"),
+          File.join(root_directory, FilePatterns::SERIALIZERS_SPEC_DIR, "%MODEL_NAME%_serializer_spec.rb")
+        ]
+      end
     end
   end
 end

data/lib/annotate_rb/model_annotator/project_annotation_remover.rb

@@ -0,0 +1,65 @@
+# frozen_string_literal: true
+
+module AnnotateRb
+  module ModelAnnotator
+    class ProjectAnnotationRemover
+      def initialize(options)
+        @options = options
+      end
+
+      def remove_annotations
+        project_model_files = model_files
+
+        removal_instructions = project_model_files.map do |path, filename|
+          file = File.join(path, filename)
+
+          if AnnotationDecider.new(file, @options).annotate?
+            _instructions = build_instructions_for_file(file)
+          end
+        end.flatten.compact
+
+        deannotated = removal_instructions.map do |instruction|
+          if SingleFileAnnotationRemover.call_with_instructions(instruction)
+            instruction.file
+          end
+        rescue => e
+          warn "Unable to process #{File.join(instruction.file)}: #{e.message}"
+          warn "\t" + e.backtrace.join("\n\t") if @options[:trace]
+        end.flatten.compact
+
+        if deannotated.empty?
+          puts "Model files unchanged."
+        else
+          puts "Removed annotations (#{deannotated.length}) from: #{deannotated.join(", ")}"
+        end
+      end
+
+      private
+
+      def build_instructions_for_file(file)
+        klass = ModelClassGetter.call(file, @options)
+
+        instructions = []
+
+        klass.reset_column_information
+        model_name = klass.name.underscore
+        table_name = klass.table_name
+
+        model_instruction = SingleFileRemoveAnnotationInstruction.new(file, @options)
+        instructions << model_instruction
+
+        related_files = RelatedFilesListBuilder.new(file, model_name, table_name, @options).build
+        related_file_instructions = related_files.map do |f, _position_key|
+          _instruction = SingleFileRemoveAnnotationInstruction.new(f, @options)
+        end
+        instructions.concat(related_file_instructions)
+
+        instructions
+      end
+
+      def model_files
+        @model_files ||= ModelFilesGetter.call(@options)
+      end
+    end
+  end
+end

data/lib/annotate_rb/model_annotator/project_annotator.rb

@@ -0,0 +1,63 @@
+# frozen_string_literal: true
+
+module AnnotateRb
+  module ModelAnnotator
+    class ProjectAnnotator
+      def initialize(options)
+        @options = options
+      end
+
+      def annotate
+        project_model_files = model_files
+
+        annotation_instructions = project_model_files.map do |path, filename|
+          file = File.join(path, filename)
+
+          if AnnotationDecider.new(file, @options).annotate?
+            _instructions = build_instructions_for_file(file)
+          end
+        end.flatten.compact
+
+        annotated = annotation_instructions.map do |instruction|
+          if SingleFileAnnotator.call_with_instructions(instruction)
+            instruction.file
+          end
+        end.compact
+
+        if annotated.empty?
+          puts "Model files unchanged."
+        else
+          puts "Annotated (#{annotated.length}): #{annotated.join(", ")}"
+        end
+      end
+
+      private
+
+      def build_instructions_for_file(file)
+        klass = ModelClassGetter.call(file, @options)
+
+        instructions = []
+
+        klass.reset_column_information
+        annotation = AnnotationBuilder.new(klass, @options).build
+        model_name = klass.name.underscore
+        table_name = klass.table_name
+
+        model_instruction = SingleFileAnnotatorInstruction.new(file, annotation, :position_in_class, @options)
+        instructions << model_instruction
+
+        related_files = RelatedFilesListBuilder.new(file, model_name, table_name, @options).build
+        related_file_instructions = related_files.map do |f, position_key|
+          _instruction = SingleFileAnnotatorInstruction.new(f, annotation, position_key, @options)
+        end
+        instructions.concat(related_file_instructions)
+
+        instructions
+      end
+
+      def model_files
+        @model_files ||= ModelFilesGetter.call(@options)
+      end
+    end
+  end
+end