erb_lint 0.0.37 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 20ce12719e17ebfc24ca522965e85985fe665c0cbec1d9751320b1a779ec8c9c
-  data.tar.gz: 536222d32134c71f72083c5083f575e8422024f8f5e26e0daf9b9eadd181d3e9
+  metadata.gz: b98507494f8af2a33e0e6fcbf28c48a2d91c78f42273a3cc6c1206578f3a55b0
+  data.tar.gz: c090a633a9d041a6d4d71bc292b1525b3a74dccd61311e0224bc1cbb0d8fea33
 SHA512:
-  metadata.gz: 44f138dfdfff77b04d03f293f3b48f90744bbf3fcbcfeadc1cf3b5f0d47cae5c933b18b0e7273d64cdf549170e2a2aa33c5110cbc0aa62c84855e6e29a5f67ae
-  data.tar.gz: 1fe70e0ddb97e0930b2645b0942d5ed08e74c5b40a41af1b6b9c7b9219b4be136495a5f3a0991363e8877aaef7bab5da2eb1f4162dc85f64b2f17d6ae2ca1184
+  metadata.gz: b2ac77d88b7be36010d69c072e3ee72c370edefa1fc3b68135593b5b494da8b0f9eac79e9b5408d10155a007d94948ae6a4b3b279de2541be66f0576abc964a8
+  data.tar.gz: f0be6a75c5060b49a8b7c5e5a5d19adb3222fb580a11c2e2e488e5b83ada00db0c4daee25e99ba5538057c749f5937b80d25a96bda1ed967e6d1baeddf787965

data/exe/erblint

@@ -3,7 +3,7 @@
 
 $LOAD_PATH.unshift("#{__dir__}/../lib")
 
-require 'erb_lint/cli'
+require "erb_lint/cli"
 
 cli = ERBLint::CLI.new
 exit(cli.run)

data/lib/erb_lint/all.rb

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+require "rubocop"
+
+require "erb_lint"
+require "erb_lint/corrector"
+require "erb_lint/file_loader"
+require "erb_lint/linter_config"
+require "erb_lint/linter_registry"
+require "erb_lint/linter"
+require "erb_lint/offense"
+require "erb_lint/processed_source"
+require "erb_lint/runner_config"
+require "erb_lint/runner"
+require "erb_lint/stats"
+require "erb_lint/reporter"
+
+# Load linters
+Dir[File.expand_path("linters/**/*.rb", __dir__)].each do |file|
+  require file
+end
+
+# Load reporters
+Dir[File.expand_path("reporters/**/*.rb", __dir__)].each do |file|
+  require file
+end

data/lib/erb_lint/cli.rb

@@ -1,16 +1,19 @@
 # frozen_string_literal: true
 
-require 'erb_lint'
-require 'active_support'
-require 'active_support/inflector'
-require 'optparse'
-require 'psych'
-require 'yaml'
-require 'rainbow'
+require "erb_lint/all"
+require "active_support"
+require "active_support/inflector"
+require "optparse"
+require "psych"
+require "yaml"
+require "rainbow"
+require "erb_lint/utils/severity_levels"
 
 module ERBLint
   class CLI
-    DEFAULT_CONFIG_FILENAME = '.erb-lint.yml'
+    include Utils::SeverityLevels
+
+    DEFAULT_CONFIG_FILENAME = ".erb-lint.yml"
     DEFAULT_LINT_ALL_GLOB = "**/*.html{+*,}.erb"
 
     class ExitWithFailure < RuntimeError; end
@@ -27,7 +30,7 @@ module ERBLint
     def run(args = ARGV)
       dupped_args = args.dup
       load_options(dupped_args)
-      @files = dupped_args
+      @files = @options[:stdin] || dupped_args
 
       load_config
 
@@ -40,10 +43,11 @@ module ERBLint
       ensure_files_exist(lint_files)
 
       if enabled_linter_classes.empty?
-        failure!('no linter available with current configuration')
+        failure!("no linter available with current configuration")
       end
 
       @options[:format] ||= :multiline
+      @options[:fail_level] ||= severity_level_for_name(:refactor)
       @stats.files = lint_files.size
       @stats.linters = enabled_linter_classes.size
 
@@ -51,14 +55,15 @@ module ERBLint
       reporter.preview
 
       runner = ERBLint::Runner.new(file_loader, @config)
+      file_content = nil
 
       lint_files.each do |filename|
         runner.clear_offenses
         begin
-          run_with_corrections(runner, filename)
+          file_content = run_with_corrections(runner, filename)
         rescue => e
           @stats.exceptions += 1
-          puts "Exception occured when processing: #{relative_filename(filename)}"
+          puts "Exception occurred when processing: #{relative_filename(filename)}"
           puts "If this file cannot be processed by erb-lint, "\
             "you can exclude it in your configuration file."
           puts e.message
@@ -69,6 +74,12 @@ module ERBLint
 
       reporter.show
 
+      if stdin? && autocorrect?
+        # When running from stdin, we only lint a single file
+        puts "================ #{lint_files.first} ==================\n"
+        puts file_content
+      end
+
       @stats.found == 0 && @stats.exceptions == 0
     rescue OptionParser::InvalidOption, OptionParser::InvalidArgument, ExitWithFailure => e
       warn(Rainbow(e.message).red)
@@ -88,7 +99,7 @@ module ERBLint
     end
 
     def run_with_corrections(runner, filename)
-      file_content = File.read(filename, encoding: Encoding::UTF_8)
+      file_content = read_content(filename)
 
       7.times do
         processed_source = ERBLint::ProcessedSource.new(filename, file_content)
@@ -101,8 +112,11 @@ module ERBLint
 
         @stats.corrected += corrector.corrections.size
 
-        File.open(filename, "wb") do |file|
-          file.write(corrector.corrected_content)
+        # Don't overwrite the file if the input comes from stdin
+        unless stdin?
+          File.open(filename, "wb") do |file|
+            file.write(corrector.corrected_content)
+          end
         end
 
         file_content = corrector.corrected_content
@@ -111,14 +125,27 @@ module ERBLint
       offenses_filename = relative_filename(filename)
       offenses = runner.offenses || []
 
-      @stats.found += offenses.size
+      @stats.ignored, @stats.found = offenses.partition do |offense|
+        severity_level_for_name(offense.severity) < @options[:fail_level]
+      end.map(&:size)
+        .zip([@stats.ignored, @stats.found])
+        .map(&:sum)
+
       @stats.processed_files[offenses_filename] ||= []
       @stats.processed_files[offenses_filename] |= offenses
+
+      file_content
+    end
+
+    def read_content(filename)
+      return File.read(filename, encoding: Encoding::UTF_8) unless stdin?
+
+      $stdin.binmode.read.force_encoding(Encoding::UTF_8)
     end
 
     def correct(processed_source, offenses)
       corrector = ERBLint::Corrector.new(processed_source, offenses)
-      failure!(corrector.diagnostics.join(', ')) if corrector.diagnostics.any?
+      failure!(corrector.diagnostics.join(", ")) if corrector.diagnostics.any?
       corrector
     end
 
@@ -156,7 +183,7 @@ module ERBLint
         else
           @files
             .map { |f| Dir.exist?(f) ? Dir[File.join(f, glob)] : f }
-            .map { |f| f.include?('*') ? Dir[f] : f }
+            .map { |f| f.include?("*") ? Dir[f] : f }
             .flatten
             .map { |f| File.expand_path(f, Dir.pwd) }
             .select { |filename| !excluded?(filename) }
@@ -169,7 +196,8 @@ module ERBLint
 
     def excluded?(filename)
       @config.global_exclude.any? do |path|
-        File.fnmatch?(path, filename)
+        expanded_path = File.expand_path(path, Dir.pwd)
+        File.fnmatch?(expanded_path, filename)
       end
     end
 
@@ -212,14 +240,14 @@ module ERBLint
     end
 
     def relative_filename(filename)
-      filename.sub("#{File.expand_path('.', Dir.pwd)}/", '')
+      filename.sub("#{File.expand_path(".", Dir.pwd)}/", "")
     end
 
     def runner_config_override
       RunnerConfig.new(
         linters: {}.tap do |linters|
           ERBLint::LinterRegistry.linters.map do |klass|
-            linters[klass.simple_name] = { 'enabled' => enabled_linter_classes.include?(klass) }
+            linters[klass.simple_name] = { "enabled" => enabled_linter_classes.include?(klass) }
           end
         end
       )
@@ -255,19 +283,36 @@ module ERBLint
         end
 
         opts.on("--enable-linters LINTER[,LINTER,...]", Array,
-          "Only use specified linter", "Known linters are: #{known_linter_names.join(', ')}") do |linters|
+          "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(', ')})")
+              failure!("#{linter}: not a valid linter name (#{known_linter_names.join(", ")})")
             end
           end
           @options[:enabled_linters] = linters
         end
 
+        opts.on("--fail-level SEVERITY", "Minimum severity for exit with error code") do |level|
+          parsed_severity = SEVERITY_CODE_TABLE[level.upcase.to_sym] || (SEVERITY_NAMES & [level.downcase]).first
+
+          if parsed_severity.nil?
+            failure!("#{level}: not a valid failure level (#{SEVERITY_NAMES.join(", ")})")
+          end
+          @options[:fail_level] = severity_level_for_name(parsed_severity)
+        end
+
         opts.on("-a", "--autocorrect", "Correct offenses automatically if possible (default: false)") do |config|
           @options[:autocorrect] = config
         end
 
+        opts.on(
+          "-sFILE",
+          "--stdin FILE",
+          "Pipe source from STDIN. Takes the path to be used to check which rules to apply."
+        ) do |file|
+          @options[:stdin] = [file]
+        end
+
         opts.on_tail("-h", "--help", "Show this message") do
           success!(opts)
         end
@@ -280,12 +325,16 @@ module ERBLint
 
     def format_options_help
       "Report offenses in the given format: "\
-      "(#{Reporter.available_formats.join(', ')}) (default: multiline)"
+      "(#{Reporter.available_formats.join(", ")}) (default: multiline)"
     end
 
     def invalid_format_error_message(given_format)
       formats = Reporter.available_formats.map { |format| "  - #{format}\n" }
       "#{given_format}: is not a valid format. Available formats:\n#{formats.join}"
     end
+
+    def stdin?
+      @options[:stdin].present?
+    end
   end
 end

data/lib/erb_lint/corrector.rb

@@ -21,7 +21,7 @@ module ERBLint
     end
 
     if ::RuboCop::Version::STRING.to_f >= 0.87
-      require 'rubocop/cop/legacy/corrector'
+      require "rubocop/cop/legacy/corrector"
       BASE = ::RuboCop::Cop::Legacy::Corrector
 
       def diagnostics

data/lib/erb_lint/linter.rb

@@ -15,9 +15,9 @@ module ERBLint
       # `ERBLint::Linters::Compass::Bar.simple_name` #=> "Compass::Bar"
       def inherited(linter)
         super
-        linter.simple_name = if linter.name.start_with?('ERBLint::Linters::')
-          name_parts = linter.name.split('::')
-          name_parts[2..-1].join('::')
+        linter.simple_name = if linter.name.start_with?("ERBLint::Linters::")
+          name_parts = linter.name.split("::")
+          name_parts[2..-1].join("::")
         else
           linter.name
         end
@@ -53,8 +53,8 @@ module ERBLint
       raise NotImplementedError, "must implement ##{__method__}"
     end
 
-    def add_offense(source_range, message, context = nil)
-      @offenses << Offense.new(self, source_range, message, context)
+    def add_offense(source_range, message, context = nil, severity = nil)
+      @offenses << Offense.new(self, source_range, message, context, severity)
     end
 
     def clear_offenses

data/lib/erb_lint/linter_config.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'active_support'
-require 'smart_properties'
+require "active_support"
+require "smart_properties"
 
 module ERBLint
   class LinterConfig
@@ -27,7 +27,7 @@ module ERBLint
       allowed_keys = self.class.properties.keys.map(&:to_s)
       given_keys = config.keys
       if (extra_keys = given_keys - allowed_keys).any?
-        raise Error, "Given key is not allowed: #{extra_keys.join(', ')}"
+        raise Error, "Given key is not allowed: #{extra_keys.join(", ")}"
       end
       super(config)
     rescue SmartProperties::InitializationError => e

data/lib/erb_lint/linter_registry.rb

@@ -3,7 +3,7 @@
 module ERBLint
   # Stores all linters available to the application.
   module LinterRegistry
-    CUSTOM_LINTERS_DIR = '.erb-linters'
+    CUSTOM_LINTERS_DIR = ".erb-linters"
     @loaded_linters = []
 
     class << self
@@ -27,7 +27,7 @@ module ERBLint
       end
 
       def load_custom_linters(directory = CUSTOM_LINTERS_DIR)
-        ruby_files = Dir.glob(File.expand_path(File.join(directory, '**', '*.rb')))
+        ruby_files = Dir.glob(File.expand_path(File.join(directory, "**", "*.rb")))
         ruby_files.each { |file| require file }
       end
     end

data/lib/erb_lint/linters/allowed_script_type.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'better_html'
-require 'better_html/tree/tag'
+require "better_html"
+require "better_html/tree/tag"
 
 module ERBLint
   module Linters
@@ -13,7 +13,7 @@ module ERBLint
 
       class ConfigSchema < LinterConfig
         property :allowed_types, accepts: array_of?(String),
-          default: -> { ['text/javascript'] }
+          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?
       end
@@ -24,7 +24,7 @@ module ERBLint
         parser.nodes_with_type(:tag).each do |tag_node|
           tag = BetterHtml::Tree::Tag.from_node(tag_node)
           next if tag.closing?
-          next unless tag.name == 'script'
+          next unless tag.name == "script"
 
           if @config.disallow_inline_scripts?
             name_node = tag_node.to_a[1]
@@ -36,7 +36,7 @@ module ERBLint
             next
           end
 
-          type_attribute = tag.attributes['type']
+          type_attribute = tag.attributes["type"]
           type_present = type_attribute.present? && type_attribute.value_node.present?
 
           if !type_present && !@config.allow_blank?
@@ -50,8 +50,8 @@ module ERBLint
             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?}."
+              "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,7 +25,7 @@ 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(
@@ -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/deprecated_classes.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'better_html'
-require 'better_html/parser'
+require "better_html"
+require "better_html/parser"
 
 module ERBLint
   module Linters
@@ -11,7 +11,7 @@ module ERBLint
 
       class RuleSet
         include SmartProperties
-        property :suggestion, accepts: String, default: ''
+        property :suggestion, accepts: String, default: ""
         property :deprecated, accepts: LinterConfig.array_of?(String), default: -> { [] }
       end
 
@@ -57,9 +57,9 @@ module ERBLint
       def class_name_with_loc(processed_source)
         Enumerator.new do |yielder|
           tags(processed_source).each do |tag|
-            class_value = tag.attributes['class']&.value
+            class_value = tag.attributes["class"]&.value
             next unless class_value
-            class_value.split(' ').each do |class_name|
+            class_value.split(" ").each do |class_name|
               yielder.yield(class_name, tag.loc)
             end
           end
@@ -69,7 +69,7 @@ module ERBLint
       def text_tags_content(processed_source)
         Enumerator.new do |yielder|
           script_tags(processed_source)
-            .select { |tag| tag.attributes['type']&.value == 'text/html' }
+            .select { |tag| tag.attributes["type"]&.value == "text/html" }
             .each do |tag|
               index = processed_source.ast.to_a.find_index(tag.node)
               next_node = processed_source.ast.to_a[index + 1]
@@ -80,7 +80,7 @@ module ERBLint
       end
 
       def script_tags(processed_source)
-        tags(processed_source).select { |tag| tag.name == 'script' }
+        tags(processed_source).select { |tag| tag.name == "script" }
       end
 
       def tags(processed_source)