erb_lint 0.4.0 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9b3cff2cda3a196df2b19ff70f2999cea11a467b57bfa9a95fc031354752c6f1
-  data.tar.gz: 7e85499590c4d58c8f2cd4af0d12d49ffe7fb27f219fdb610c777c2a5b3e2100
+  metadata.gz: ba9ef5f8b8a251d649dd1cffba1094d770471a630d0fa66b402621883ea3b050
+  data.tar.gz: db969c9e8bacc58b20fa9ad568c98263225c8bbf41f2d32f1b5d9f44a4c58aab
 SHA512:
-  metadata.gz: 23c04a0063e05a08c9d3b376542c6cc1f97659d700a4ba7fa5a50997231a485bb8971ba240b25cc998235c1ff5b138e3e4a5fa648351603f06061339c71ec75a
-  data.tar.gz: 538c54359e677e2a56ae558b3d2f5527138c9f82485a943c9f4c5e06d6a3ebd6d6107f2285913920c7d3d1213e1353162f55b257bf1b7eb6da238af8487d142d
+  metadata.gz: a1d9e86c1862a41bb8cc2d1ae520af38ced2390e3d06c3d9bc9550a434993de502167c87adf93a780fec7f87d4921e77367e8bf8c546bf59c6f88908d0208039
+  data.tar.gz: 468ef87d28e39aa2fb0166cdfc98bee1f1f7349d21a9a5e2c08a81f5aab47c067d1d440d6466cf28077f5d72984b9d0018bdc23af2e3b91a6127bfc46f850f5a

data/lib/erb_lint/cache.rb

@@ -16,7 +16,7 @@ module ERBLint
       file_checksum = checksum(filename, file_content)
       begin
         cache_file_contents_as_offenses = JSON.parse(
-          File.read(File.join(@cache_dir, file_checksum))
+          File.read(File.join(@cache_dir, file_checksum)),
         ).map do |offense_hash|
           ERBLint::CachedOffense.new(offense_hash)
         end
@@ -76,7 +76,7 @@ module ERBLint
       mode = File.stat(filename).mode
 
       digester.update(
-        "#{mode}#{config.to_hash}#{ERBLint::VERSION}#{file_content}"
+        "#{mode}#{config.to_hash}#{ERBLint::VERSION}#{file_content}",
       )
       digester.hexdigest
     rescue Errno::ENOENT

data/lib/erb_lint/cached_offense.rb

@@ -38,7 +38,7 @@ module ERBLint
           last_line: offense.last_line,
           last_column: offense.last_column,
           length: offense.length,
-        }
+        },
       )
     end
 

data/lib/erb_lint/cli.rb

@@ -74,7 +74,7 @@ module ERBLint
       @stats.linters = enabled_linter_classes.size
       @stats.autocorrectable_linters = enabled_linter_classes.count(&:support_autocorrect?)
 
-      reporter = Reporter.create_reporter(@options[:format], @stats, autocorrect?)
+      reporter = Reporter.create_reporter(@options[:format], @stats, autocorrect?, @options[:show_linter_names])
       reporter.preview
 
       runner = ERBLint::Runner.new(file_loader, @config, @options[:disable_inline_configs])
@@ -87,7 +87,7 @@ module ERBLint
         rescue => e
           @stats.exceptions += 1
           puts "Exception occurred when processing: #{relative_filename(filename)}"
-          puts "If this file cannot be processed by erb-lint, "\
+          puts "If this file cannot be processed by erb-lint, " \
             "you can exclude it in your configuration file."
           puts e.message
           puts Rainbow(e.backtrace.join("\n")).red
@@ -303,7 +303,7 @@ module ERBLint
           ERBLint::LinterRegistry.linters.map do |klass|
             linters[klass.simple_name] = { "enabled" => enabled_linter_classes.include?(klass) }
           end
-        end
+        end,
       )
     end
 
@@ -348,8 +348,12 @@ module ERBLint
           @options[:clear_cache] = config
         end
 
-        opts.on("--enable-linters LINTER[,LINTER,...]", Array,
-          "Only use specified linter", "Known linters are: #{known_linter_names.join(", ")}") do |linters|
+        opts.on(
+          "--enable-linters LINTER[,LINTER,...]",
+          Array,
+          "Only use specified linter",
+          "Known linters are: #{known_linter_names.join(", ")}",
+        ) do |linters|
           linters.each do |linter|
             unless known_linter_names.include?(linter)
               failure!("#{linter}: not a valid linter name (#{known_linter_names.join(", ")})")
@@ -371,6 +375,10 @@ module ERBLint
           @options[:autocorrect] = config
         end
 
+        opts.on("--show-linter-names", "Show linter names") do
+          @options[:show_linter_names] = true
+        end
+
         opts.on("--allow-no-files", "When no matching files found, exit successfully (default: false)") do |config|
           @options[:allow_no_files] = config
         end
@@ -382,7 +390,7 @@ module ERBLint
         opts.on(
           "-sFILE",
           "--stdin FILE",
-          "Pipe source from STDIN. Takes the path to be used to check which rules to apply."
+          "Pipe source from STDIN. Takes the path to be used to check which rules to apply.",
         ) do |file|
           @options[:stdin] = [file]
         end
@@ -398,7 +406,7 @@ module ERBLint
     end
 
     def format_options_help
-      "Report offenses in the given format: "\
+      "Report offenses in the given format: " \
         "(#{Reporter.available_formats.join(", ")}) (default: multiline)"
     end
 

data/lib/erb_lint/linter.rb

@@ -38,7 +38,7 @@ module ERBLint
     def initialize(file_loader, config)
       @file_loader = file_loader
       @config = config
-      raise ArgumentError, "expect `config` to be #{self.class.config_schema} instance, "\
+      raise ArgumentError, "expect `config` to be #{self.class.config_schema} instance, " \
         "not #{config.class}" unless config.is_a?(self.class.config_schema)
       @offenses = []
     end

data/lib/erb_lint/linters/allowed_script_type.rb

@@ -12,7 +12,8 @@ module ERBLint
       include LinterRegistry
 
       class ConfigSchema < LinterConfig
-        property :allowed_types, accepts: array_of?(String),
+        property :allowed_types,
+          accepts: array_of?(String),
           default: -> { ["text/javascript"] }
         property :allow_blank, accepts: [true, false], default: true, reader: :allow_blank?
         property :disallow_inline_scripts, accepts: [true, false], default: false, reader: :disallow_inline_scripts?
@@ -30,8 +31,8 @@ module ERBLint
             name_node = tag_node.to_a[1]
             add_offense(
               name_node.loc,
-              "Avoid using inline `<script>` tags altogether. "\
-                "Instead, move javascript code into a static file."
+              "Avoid using inline `<script>` tags altogether. " \
+                "Instead, move javascript code into a static file.",
             )
             next
           end
@@ -44,14 +45,14 @@ module ERBLint
             add_offense(
               name_node.loc,
               "Missing a `type=\"text/javascript\"` attribute to `<script>` tag.",
-              [type_attribute]
+              [type_attribute],
             )
           elsif type_present && !@config.allowed_types.include?(type_attribute.value)
             add_offense(
               type_attribute.loc,
-              "Avoid using #{type_attribute.value.inspect} as type for `<script>` tag. "\
-                "Must be one of: #{@config.allowed_types.join(", ")}"\
-                "#{" (or no type attribute)" if @config.allow_blank?}."
+              "Avoid using #{type_attribute.value.inspect} as type for `<script>` tag. " \
+                "Must be one of: #{@config.allowed_types.join(", ")}" \
+                "#{" (or no type attribute)" if @config.allow_blank?}.",
             )
           end
         end

data/lib/erb_lint/linters/closing_erb_tag_indent.rb

@@ -25,13 +25,13 @@ module ERBLint
             add_offense(
               code_node.loc.end.adjust(begin_pos: -end_spaces.size),
               "Remove newline before `%>` to match start of tag.",
-              " "
+              " ",
             )
           elsif start_with_newline && !end_with_newline
             add_offense(
               code_node.loc.end.adjust(begin_pos: -end_spaces.size),
               "Insert newline before `%>` to match start of tag.",
-              "\n"
+              "\n",
             )
           elsif start_with_newline && end_with_newline
             current_indent = end_spaces.split("\n", -1).last
@@ -39,7 +39,7 @@ module ERBLint
               add_offense(
                 code_node.loc.end.adjust(begin_pos: -current_indent.size),
                 "Indent `%>` on column #{erb_node.loc.column} to match start of tag.",
-                " " * erb_node.loc.column
+                " " * erb_node.loc.column,
               )
             end
           end

data/lib/erb_lint/linters/comment_syntax.rb

@@ -31,7 +31,7 @@ module ERBLint
 
           add_offense(
             source_range,
-            <<~EOF.chomp
+            <<~EOF.chomp,
               Bad ERB comment syntax. Should be #{correct_erb_tag} without a space between.
               Leaving a space between ERB tags and the Ruby comment character can cause parser errors.
             EOF

data/lib/erb_lint/linters/deprecated_classes.rb

@@ -49,7 +49,7 @@ module ERBLint
           process_nested_offenses(
             source: sub_source,
             offset: offset + content_node.loc.begin_pos,
-            parent_source: parent_source
+            parent_source: parent_source,
           )
         end
       end
@@ -99,7 +99,7 @@ module ERBLint
 
           add_offense(
             range,
-            format(message, class_name, violated_rule[:class_expr], suggestion)
+            format(message, class_name, violated_rule[:class_expr], suggestion),
           )
         end
       end

data/lib/erb_lint/linters/erb_safety.rb

@@ -26,7 +26,7 @@ module ERBLint
           tester.errors.each do |error|
             add_offense(
               error.location,
-              error.message
+              error.message,
             )
           end
         end

data/lib/erb_lint/linters/extra_newline.rb

@@ -15,7 +15,7 @@ module ERBLint
           add_offense(
             processed_source
               .to_source_range((matches.begin(index) + 2)...matches.end(index)),
-            "Extra blank line detected."
+            "Extra blank line detected.",
           )
         end
       end

data/lib/erb_lint/linters/final_newline.rb

@@ -29,22 +29,22 @@ module ERBLint
             add_offense(
               processed_source.to_source_range(file_content.size...file_content.size),
               "Missing a trailing newline at the end of the file.",
-              :insert
+              :insert,
             )
           else
             add_offense(
               processed_source.to_source_range(
-                (file_content.size - final_newline.size + 1)...file_content.size
+                (file_content.size - final_newline.size + 1)...file_content.size,
               ),
               "Remove multiple trailing newline at the end of the file.",
-              :remove
+              :remove,
             )
           end
         elsif !@new_lines_should_be_present && !final_newline.empty?
           add_offense(
             processed_source.to_source_range(match.begin(0)...match.end(0)),
             "Remove #{final_newline.size} trailing newline at the end of the file.",
-            :remove
+            :remove,
           )
         end
       end

data/lib/erb_lint/linters/hard_coded_string.rb

@@ -17,7 +17,7 @@ module ERBLint
       ALLOWED_CORRECTORS = ["I18nCorrector", "RuboCop::Corrector::I18n::HardCodedString"]
 
       NON_TEXT_TAGS = Set.new(["script", "style", "xmp", "iframe", "noembed", "noframes", "listing"])
-      TEXT_NOT_ALLOWED = Set.new([
+      NO_TRANSLATION_NEEDED = Set.new([
         " ",
         "&",
         "<",
@@ -40,6 +40,7 @@ module ERBLint
         " ",
         " ",
         " ",
+        "×",
       ])
 
       class ConfigSchema < LinterConfig
@@ -66,7 +67,7 @@ module ERBLint
 
           add_offense(
             source_range,
-            message(source_range.source)
+            message(source_range.source),
           )
         end
       end
@@ -83,7 +84,7 @@ module ERBLint
       def autocorrect(processed_source, offense)
         string = offense.source_range.source
         return unless (klass = load_corrector)
-        return unless string.strip.length > 1
+        return if string.strip.length <= 1
 
         node = ::RuboCop::AST::StrNode.new(:str, [string])
         corrector = klass.new(node, processed_source.filename, corrector_i18n_load_path, offense.source_range)
@@ -96,7 +97,7 @@ module ERBLint
 
       def check_string?(str)
         string = str.gsub(/\s*/, "")
-        string.length > 1 && !TEXT_NOT_ALLOWED.include?(string)
+        string.length > 1 && !NO_TRANSLATION_NEEDED.include?(string)
       end
 
       def load_corrector

data/lib/erb_lint/linters/no_javascript_tag_helper.rb

@@ -38,9 +38,9 @@ module ERBLint
 
           add_offense(
             erb_node.loc,
-            "Avoid using 'javascript_tag do' as it confuses tests "\
+            "Avoid using 'javascript_tag do' as it confuses tests " \
               "that validate html, use inline <script> instead",
-            [erb_node, send_node]
+            [erb_node, send_node],
           )
         end
       end
@@ -82,8 +82,10 @@ module ERBLint
           corrector.replace(end_node.loc, end_content)
         elsif script_content
           script_content = "\n//<![CDATA[\n#{script_content}\n//]]>\n" if @config.correction_style == :cdata
-          corrector.replace(begin_node.loc,
-            "<script#{arguments}>#{script_content}</script>")
+          corrector.replace(
+            begin_node.loc,
+            "<script#{arguments}>#{script_content}</script>",
+          )
         end
       rescue Utils::RubyToERB::Error, Utils::BlockMap::ParseError
         nil

data/lib/erb_lint/linters/no_unused_disable.rb

@@ -35,8 +35,10 @@ module ERBLint
 
         disabled_rules_and_line_number.each do |rule, line_numbers|
           line_numbers.each do |line_number|
-            add_offense(processed_source.source_buffer.line_range(line_number),
-              "Unused erblint:disable comment for #{rule}")
+            add_offense(
+              processed_source.source_buffer.line_range(line_number),
+              "Unused erblint:disable comment for #{rule}",
+            )
           end
         end
       end

data/lib/erb_lint/linters/parser_errors.rb

@@ -9,7 +9,7 @@ module ERBLint
         processed_source.parser.parser_errors.each do |error|
           add_offense(
             error.loc,
-            "#{error.message} (at #{error.loc.source})"
+            "#{error.message} (at #{error.loc.source})",
           )
         end
       end

data/lib/erb_lint/linters/partial_instance_variable.rb

@@ -13,9 +13,9 @@ module ERBLint
 
         add_offense(
           processed_source.to_source_range(
-            processed_source.file_content =~ instance_variable_regex..processed_source.file_content.size
+            processed_source.file_content =~ instance_variable_regex..processed_source.file_content.size,
           ),
-          "Instance variable detected in partial."
+          "Instance variable detected in partial.",
         )
       end
     end

data/lib/erb_lint/linters/require_input_autocomplete.rb

@@ -62,9 +62,9 @@ module ERBLint
 
           add_offense(
             tag_node.to_a[1].loc,
-            "Input tag is missing an autocomplete attribute. If no "\
+            "Input tag is missing an autocomplete attribute. If no " \
               "autocomplete behaviour is desired, use the value `off` or `nope`.",
-            [autocomplete_attribute]
+            [autocomplete_attribute],
           )
         end
       end
@@ -96,9 +96,9 @@ module ERBLint
 
           add_offense(
             erb_node.loc,
-            "Input field helper is missing an autocomplete attribute. If no "\
+            "Input field helper is missing an autocomplete attribute. If no " \
               "autocomplete behaviour is desired, use the value `off` or `nope`.",
-            [erb_node, send_node]
+            [erb_node, send_node],
           )
         end
       end

data/lib/erb_lint/linters/require_script_nonce.rb

@@ -29,7 +29,7 @@ module ERBLint
           add_offense(
             tag_node.to_a[1].loc,
             "Missing a nonce attribute. Use request.content_security_policy_nonce",
-            [nonce_attribute]
+            [nonce_attribute],
           )
         end
       end
@@ -67,7 +67,7 @@ module ERBLint
           add_offense(
             erb_node.loc,
             "Missing a nonce attribute. Use nonce: true",
-            [erb_node, send_node]
+            [erb_node, send_node],
           )
         end
       end

data/lib/erb_lint/linters/right_trim.rb

@@ -19,7 +19,7 @@ module ERBLint
 
           add_offense(
             trim_node.loc,
-            "Prefer #{@config.enforced_style}%> instead of #{trim_node.loc.source}%> for trimming on the right."
+            "Prefer #{@config.enforced_style}%> instead of #{trim_node.loc.source}%> for trimming on the right.",
           )
         end
       end

data/lib/erb_lint/linters/rubocop.rb

@@ -134,7 +134,7 @@ module ERBLint
         source = ::RuboCop::ProcessedSource.new(
           content,
           @rubocop_config.target_ruby_version,
-          filename
+          filename,
         )
         if ::RuboCop::Version::STRING.to_f >= 1.38
           registry = RuboCop::Cop::Registry.global
@@ -146,10 +146,10 @@ module ERBLint
 
       def cop_classes
         if @only_cops.present?
-          selected_cops = ::RuboCop::Cop::Cop.all.select { |cop| cop.match?(@only_cops) }
+          selected_cops = ::RuboCop::Cop::Registry.all.select { |cop| cop.match?(@only_cops) }
           ::RuboCop::Cop::Registry.new(selected_cops)
         else
-          ::RuboCop::Cop::Registry.new(::RuboCop::Cop::Cop.all)
+          ::RuboCop::Cop::Registry.new(::RuboCop::Cop::Registry.all)
         end
       end
 

data/lib/erb_lint/linters/rubocop_text.rb

@@ -29,7 +29,7 @@ module ERBLint
       end
 
       def cop_classes
-        selected_cops = ::RuboCop::Cop::Cop.all.select { |cop| cop.match?(@only_cops) }
+        selected_cops = ::RuboCop::Cop::Registry.all.select { |cop| cop.match?(@only_cops) }
 
         ::RuboCop::Cop::Registry.new(selected_cops)
       end

data/lib/erb_lint/linters/self_closing_tag.rb

@@ -11,8 +11,25 @@ module ERBLint
       end
       self.config_schema = ConfigSchema
 
-      SELF_CLOSING_TAGS = ["area", "base", "br", "col", "command", "embed", "hr", "input", "keygen", "link",
-                           "menuitem", "meta", "param", "source", "track", "wbr", "img",]
+      SELF_CLOSING_TAGS = [
+        "area",
+        "base",
+        "br",
+        "col",
+        "command",
+        "embed",
+        "hr",
+        "input",
+        "keygen",
+        "link",
+        "menuitem",
+        "meta",
+        "param",
+        "source",
+        "track",
+        "wbr",
+        "img",
+      ]
 
       def run(processed_source)
         processed_source.ast.descendants(:tag).each do |tag_node|
@@ -24,7 +41,7 @@ module ERBLint
             add_offense(
               start_solidus.loc,
               "Tag `#{tag.name}` is a void element, it must not start with `</`.",
-              ""
+              "",
             )
           end
 
@@ -32,7 +49,7 @@ module ERBLint
             add_offense(
               tag_node.loc.end.offset(-1),
               "Tag `#{tag.name}` is self-closing, it must end with `/>`.",
-              "/"
+              "/",
             )
           end
 
@@ -42,7 +59,7 @@ module ERBLint
           add_offense(
             end_solidus.loc,
             "Tag `#{tag.name}` is a void element, it must end with `>` and not `/>`.",
-            ""
+            "",
           )
         end
       end

data/lib/erb_lint/linters/space_around_erb_tag.rb

@@ -23,17 +23,17 @@ module ERBLint
           if start_spaces.size != 1 && !start_spaces.include?("\n")
             add_offense(
               code_node.loc.resize(start_spaces.size),
-              "Use 1 space after `<%#{indicator}#{ltrim&.loc&.source}` "\
+              "Use 1 space after `<%#{indicator}#{ltrim&.loc&.source}` " \
                 "instead of #{start_spaces.size} space#{"s" if start_spaces.size > 1}.",
-              " "
+              " ",
             )
           elsif start_spaces.count("\n") > 1
             lines = start_spaces.split("\n", -1)
             add_offense(
               code_node.loc.resize(start_spaces.size),
-              "Use 1 newline after `<%#{indicator&.loc&.source}#{ltrim&.loc&.source}` "\
+              "Use 1 newline after `<%#{indicator&.loc&.source}#{ltrim&.loc&.source}` " \
                 "instead of #{start_spaces.count("\n")}.",
-              "#{lines.first}\n#{lines.last}"
+              "#{lines.first}\n#{lines.last}",
             )
           end
 
@@ -41,17 +41,17 @@ module ERBLint
           if end_spaces.size != 1 && !end_spaces.include?("\n")
             add_offense(
               code_node.loc.end.adjust(begin_pos: -end_spaces.size),
-              "Use 1 space before `#{rtrim&.loc&.source}%>` "\
+              "Use 1 space before `#{rtrim&.loc&.source}%>` " \
                 "instead of #{end_spaces.size} space#{"s" if start_spaces.size > 1}.",
-              " "
+              " ",
             )
           elsif end_spaces.count("\n") > 1
             lines = end_spaces.split("\n", -1)
             add_offense(
               code_node.loc.end.adjust(begin_pos: -end_spaces.size),
-              "Use 1 newline before `#{rtrim&.loc&.source}%>` "\
+              "Use 1 newline before `#{rtrim&.loc&.source}%>` " \
                 "instead of #{end_spaces.count("\n")}.",
-              "#{lines.first}\n#{lines.last}"
+              "#{lines.first}\n#{lines.last}",
             )
           end
         end

data/lib/erb_lint/linters/space_in_html_tag.rb

@@ -50,7 +50,7 @@ module ERBLint
         add_offense(
           processed_source.to_source_range(range),
           "Extra space detected where there should be no space.",
-          ""
+          "",
         )
       end
 
@@ -69,24 +69,24 @@ module ERBLint
         if non_space && !non_space.captures.empty?
           add_offense(
             processed_source.to_source_range(range),
-            "Non-whitespace character(s) detected: "\
+            "Non-whitespace character(s) detected: " \
               "#{non_space.captures.map(&:inspect).join(", ")}.",
-            expected
+            expected,
           )
         elsif newlines && accept_newline
           if expected != chars
             add_offense(
               processed_source.to_source_range(range),
-              "#{chars.empty? ? "No" : "Extra"} space detected where there should be "\
+              "#{chars.empty? ? "No" : "Extra"} space detected where there should be " \
                 "a single space or a single line break.",
-              expected
+              expected,
             )
           end
         else
           add_offense(
             processed_source.to_source_range(range),
             "#{chars.empty? ? "No" : "Extra"} space detected where there should be a single space.",
-            expected
+            expected,
           )
         end
       end
@@ -103,7 +103,7 @@ module ERBLint
 
           single_space_or_newline(
             processed_source,
-            attribute.loc.end_pos...next_attribute.loc.begin_pos
+            attribute.loc.end_pos...next_attribute.loc.begin_pos,
           )
         end
       end

data/lib/erb_lint/linters/space_indentation.rb

@@ -23,7 +23,7 @@ module ERBLint
             add_offense(
               processed_source.to_source_range(document_pos...(document_pos + spaces.length)),
               "Indent with spaces instead of tabs.",
-              spaces.gsub("\t", " " * @config.tab_width)
+              spaces.gsub("\t", " " * @config.tab_width),
             )
           end
 

data/lib/erb_lint/linters/trailing_whitespace.rb

@@ -18,7 +18,7 @@ module ERBLint
 
           add_offense(
             processed_source.to_source_range((document_pos - whitespace.length - 1)...(document_pos - 1)),
-            "Extra whitespace detected at end of line."
+            "Extra whitespace detected at end of line.",
           )
         end
       end

data/lib/erb_lint/offense.rb

@@ -23,9 +23,9 @@ module ERBLint
     end
 
     def inspect
-      "#<#{self.class.name} linter=#{linter.class.name} "\
-        "source_range=#{source_range.begin_pos}...#{source_range.end_pos} "\
-        "message=#{message}> "\
+      "#<#{self.class.name} linter=#{linter.class.name} " \
+        "source_range=#{source_range.begin_pos}...#{source_range.end_pos} " \
+        "message=#{message}> " \
         "severity=#{severity}"
     end
 

data/lib/erb_lint/processed_source.rb

@@ -27,7 +27,7 @@ module ERBLint
       BetterHtml::Tokenizer::Location.new(
         source_buffer,
         range.begin,
-        range.exclude_end? ? range.end : range.end + 1
+        range.exclude_end? ? range.end : range.end + 1,
       )
     end
   end

data/lib/erb_lint/reporter.rb

@@ -23,9 +23,10 @@ module ERBLint
         .sort
     end
 
-    def initialize(stats, autocorrect)
+    def initialize(stats, autocorrect, show_linter_names = false)
       @stats = stats
       @autocorrect = autocorrect
+      @show_linter_names = show_linter_names
     end
 
     def preview; end
@@ -34,7 +35,7 @@ module ERBLint
 
     private
 
-    attr_reader :stats, :autocorrect
+    attr_reader :stats, :autocorrect, :show_linter_names
 
     delegate :processed_files, to: :stats
   end

data/lib/erb_lint/reporters/compact_reporter.rb

@@ -33,8 +33,9 @@ module ERBLint
           "#{filename}:",
           "#{offense.line_number}:",
           "#{offense.column}: ",
+          ("[#{offense.simple_name}] " if show_linter_names),
           offense.message.to_s,
-        ].join
+        ].compact.join
       end
 
       def footer; end
@@ -60,7 +61,7 @@ module ERBLint
 
         if corrected_found_diff > 0
           message = Rainbow(
-            "#{stats.corrected} error(s) corrected and #{corrected_found_diff} error(s) remaining in ERB files"
+            "#{stats.corrected} error(s) corrected and #{corrected_found_diff} error(s) remaining in ERB files",
           ).red
 
           warn(message)

data/lib/erb_lint/reporters/gitlab_reporter.rb

@@ -0,0 +1,55 @@
+# frozen_string_literal: true
+
+require "json"
+
+module ERBLint
+  module Reporters
+    class GitlabReporter < Reporter
+      def preview; end
+
+      def show
+        puts formatted_data
+      end
+
+      private
+
+      def formatted_data
+        formatted_files.to_json
+      end
+
+      def formatted_files
+        processed_files.flat_map do |filename, offenses|
+          formatted_offenses(filename, offenses)
+        end
+      end
+
+      def formatted_offenses(filename, offenses)
+        offenses.map do |offense|
+          format_offense(filename, offense)
+        end
+      end
+
+      def format_offense(filename, offense)
+        {
+          description: offense.message,
+          check_name: offense.simple_name,
+          fingerprint: generate_fingerprint(filename, offense),
+          severity: offense.severity,
+          location: {
+            path: filename,
+            lines: {
+              begin: offense.line_number,
+              end: offense.last_line,
+            },
+          },
+        }
+      end
+
+      def generate_fingerprint(filename, offense)
+        Digest::MD5.hexdigest(
+          "#{offense.simple_name}@#{filename}:#{offense.line_number}:#{offense.last_line}",
+        )
+      end
+    end
+  end
+end

data/lib/erb_lint/reporters/junit_reporter.rb

@@ -0,0 +1,112 @@
+# frozen_string_literal: true
+
+require "rexml/document"
+require "rexml/formatters/pretty"
+
+module ERBLint
+  module Reporters
+    class JunitReporter < Reporter
+      def preview; end
+
+      def show
+        xml = create_junit_xml
+        formatted_xml_string = StringIO.new
+        REXML::Formatters::Pretty.new.write(xml, formatted_xml_string)
+        puts formatted_xml_string.string
+      end
+
+      private
+
+      CONTEXT = {
+        prologue_quote: :quote,
+        attribute_quote: :quote,
+      }
+
+      def create_junit_xml
+        # create prologue
+        xml = REXML::Document.new(nil, CONTEXT)
+        xml << REXML::XMLDecl.new("1.0", "UTF-8")
+
+        xml.add_element(create_testsuite_element)
+
+        xml
+      end
+
+      def create_testsuite_element
+        tests = stats.processed_files.size
+        failures = stats.found
+        testsuite_element = REXML::Element.new("testsuite", nil, CONTEXT)
+        testsuite_element.add_attribute("name", "erblint")
+        testsuite_element.add_attribute("tests", tests.to_s)
+        testsuite_element.add_attribute("failures", failures.to_s)
+
+        testsuite_element.add_element(create_properties)
+
+        processed_files.each do |filename, offenses|
+          if offenses.empty?
+            testcase_element = REXML::Element.new("testcase", nil, CONTEXT)
+            testcase_element.add_attribute("name", filename.to_s)
+            testcase_element.add_attribute("file", filename.to_s)
+
+            testsuite_element.add_element(testcase_element)
+          end
+
+          offenses.each do |offense|
+            testsuite_element.add_element(create_testcase(filename, offense))
+          end
+        end
+
+        testsuite_element
+      end
+
+      def create_properties
+        properties_element = REXML::Element.new("properties", nil, CONTEXT)
+
+        [
+          ["erb_lint_version", ERBLint::VERSION],
+          ["ruby_engine", RUBY_ENGINE],
+          ["ruby_version", RUBY_VERSION],
+          ["ruby_patchlevel", RUBY_PATCHLEVEL.to_s],
+          ["ruby_platform", RUBY_PLATFORM],
+        ].each do |property_attribute|
+          properties_element.add_element(create_property(*property_attribute))
+        end
+
+        properties_element
+      end
+
+      def create_property(name, value)
+        property_element = REXML::Element.new("property")
+        property_element.add_attribute("name", name)
+        property_element.add_attribute("value", value)
+
+        property_element
+      end
+
+      def create_testcase(filename, offense)
+        testcase_element = REXML::Element.new("testcase", nil, CONTEXT)
+        testcase_element.add_attribute("name", filename.to_s)
+        testcase_element.add_attribute("file", filename.to_s)
+        testcase_element.add_attribute("lineno", offense.line_number.to_s)
+
+        testcase_element.add_element(create_failure(filename, offense))
+
+        testcase_element
+      end
+
+      def create_failure(filename, offense)
+        message = offense.message
+        type = offense.simple_name
+
+        failure_element = REXML::Element.new("failure", nil, CONTEXT)
+        failure_element.add_attribute("message", "#{type}: #{message}")
+        failure_element.add_attribute("type", type.to_s)
+
+        cdata_element = REXML::CData.new("#{type}: #{message} at #{filename}:#{offense.line_number}:#{offense.column}")
+        failure_element.add_text(cdata_element)
+
+        failure_element
+      end
+    end
+  end
+end

data/lib/erb_lint/reporters/multiline_reporter.rb

@@ -8,9 +8,14 @@ module ERBLint
       private
 
       def format_offense(filename, offense)
+        details = "#{offense.message}#{Rainbow(" (not autocorrected)").red if autocorrect}"
+        if show_linter_names
+          details = "[#{offense.simple_name}] " + details
+        end
+
         <<~EOF
 
-          #{offense.message}#{Rainbow(" (not autocorrected)").red if autocorrect}
+          #{details}
           In file: #{filename}:#{offense.line_number}
         EOF
       end

data/lib/erb_lint/runner.rb

@@ -57,7 +57,7 @@ module ERBLint
       if no_unused_disable_enabled? && enable_inline_configs?
         @no_unused_disable = ERBLint::Linters::NoUnusedDisable.new(
           @file_loader,
-          @config.for_linter(ERBLint::Linters::NoUnusedDisable)
+          @config.for_linter(ERBLint::Linters::NoUnusedDisable),
         )
         @no_unused_disable.run(processed_source, @offenses)
         @offenses.concat(@no_unused_disable.offenses)

data/lib/erb_lint/runner_config.rb

@@ -83,7 +83,7 @@ module ERBLint
     def config_hash_for_linter(klass_name)
       config_hash = linters_config[klass_name] || {}
       config_hash["exclude"] ||= []
-      config_hash["exclude"].concat(global_exclude) if config_hash["exclude"].is_a?(Array)
+      config_hash["exclude"].concat(global_exclude).uniq! if config_hash["exclude"].is_a?(Array)
       config_hash
     end
 

data/lib/erb_lint/utils/block_map.rb

@@ -60,7 +60,7 @@ module ERBLint
         end
 
         def inspect
-          "\#<#{self.class.name} type=#{type.inspect} nodes=#{nodes.inspect}>"
+          "#<#{self.class.name} type=#{type.inspect} nodes=#{nodes.inspect}>"
         end
 
         def &(other)
@@ -102,7 +102,7 @@ module ERBLint
             node.type,
             extract_map_locations(node)
               .map { |loc| find_entry(loc) }
-              .compact.map(&:node)
+              .compact.map(&:node),
           )
         end
 
@@ -111,7 +111,7 @@ module ERBLint
             :begin,
             (extract_map_locations(node) + rescue_locations(node))
               .map { |loc| find_entry(loc) }
-              .compact.map(&:node)
+              .compact.map(&:node),
           )
         end
 
@@ -120,7 +120,7 @@ module ERBLint
             node.type,
             (extract_map_locations(node) + when_locations(node))
               .map { |loc| find_entry(loc) }
-              .compact.map(&:node)
+              .compact.map(&:node),
           )
         end
 

data/lib/erb_lint/utils/offset_corrector.rb

@@ -42,15 +42,12 @@ module ERBLint
         range = to_range(node_or_range)
 
         @processed_source.to_source_range(
-          bound(@offset + range.begin_pos)..bound(@offset + (range.end_pos - 1))
+          bound(@offset + range.begin_pos)..bound(@offset + (range.end_pos - 1)),
         )
       end
 
       def bound(pos)
-        [
-          [pos, @bound_range.min].max,
-          @bound_range.max,
-        ].min
+        pos.clamp(@bound_range.min, @bound_range.max)
       end
 
       private

data/lib/erb_lint/utils/severity_levels.rb

@@ -5,8 +5,14 @@ module ERBLint
     module SeverityLevels
       SEVERITY_NAMES = [:info, :refactor, :convention, :warning, :error, :fatal].freeze
 
-      SEVERITY_CODE_TABLE = { I: :info, R: :refactor, C: :convention,
-                              W: :warning, E: :error, F: :fatal, }.freeze
+      SEVERITY_CODE_TABLE = {
+        I: :info,
+        R: :refactor,
+        C: :convention,
+        W: :warning,
+        E: :error,
+        F: :fatal,
+      }.freeze
 
       def severity_level_for_name(name)
         SEVERITY_NAMES.index(name || :error) + 1

data/lib/erb_lint/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ERBLint
-  VERSION = "0.4.0"
+  VERSION = "0.6.0"
 end

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: erb_lint
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.6.0
 platform: ruby
 authors:
 - Justin Chan
 - Shopify Developers
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2023-03-27 00:00:00.000000000 Z
+date: 2024-08-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -73,14 +73,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1'
 - !ruby/object:Gem::Dependency
   name: smart_properties
   requirement: !ruby/object:Gem::Requirement
@@ -182,7 +182,9 @@ files:
 - lib/erb_lint/processed_source.rb
 - lib/erb_lint/reporter.rb
 - lib/erb_lint/reporters/compact_reporter.rb
+- lib/erb_lint/reporters/gitlab_reporter.rb
 - lib/erb_lint/reporters/json_reporter.rb
+- lib/erb_lint/reporters/junit_reporter.rb
 - lib/erb_lint/reporters/multiline_reporter.rb
 - lib/erb_lint/runner.rb
 - lib/erb_lint/runner_config.rb
@@ -199,7 +201,7 @@ licenses:
 - MIT
 metadata:
   allowed_push_host: https://rubygems.org
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -214,8 +216,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.9
-signing_key: 
+rubygems_version: 3.5.16
+signing_key:
 specification_version: 4
 summary: ERB lint tool
 test_files: []