slim_lint 0.35.0 → 0.36.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e9478570d94e93349a03f9f7a67789f3c0ff356bd8bd95ee7fe5628248eaaab9
-  data.tar.gz: 23bc11a5e84b251523397d12bcf0c776d8755373fffbfa7ce183353fe8a86b73
+  metadata.gz: dd046afc687f2cd47b2766c90af0d4cee87282f818f60954530dc9e579a3a797
+  data.tar.gz: a6378966b0a438ba9cdf86f5771262f7e7eebb4a40b1b915ec95a68a7d51eb05
 SHA512:
-  metadata.gz: f40d2b1af0278fd5f2ea727f0a7ee39322df8b7c1ddf155ece528ac7f99c8dd8b0284af8d622e05eec9ef229544fd953b1f6a51cea9e11d35c23bba1d7d7c7c3
-  data.tar.gz: 6c96c5b859896bf1a3d04fe1ca336cfae1e7c15bbcb89f5a6143d1b952ec3508febab94dd4fa618fbbed98405bbaed941cd6a94bb3c2e2bad3a1bdc04ddb5574
+  metadata.gz: 13c3f8d0aacd071c62fdd1ddea37c690a48a53df0170a32575cced14ad6e828ca6a2a1bccbcc6cdd915cbf54a4b917d02099a75f51c08417395861a53b6933d4
+  data.tar.gz: d063b8fe3e5620ecc68a1bb6b2a38ceb0b5431971ec94c6a3cb4059be77d8b87e122c2ef7fae4cc1c4b1325bc9c5ffbb59d237b19caf22974977086410cf614d

data/lib/slim_lint/corrector.rb

@@ -0,0 +1,44 @@
+# frozen_string_literal: true
+
+module SlimLint
+  # Applies the corrections attached to a set of lints to a document's
+  # source, producing the corrected source text.
+  class Corrector
+    # @param document [SlimLint::Document] document the lints were reported against
+    def initialize(document)
+      @document = document
+    end
+
+    # Applies every correctable lint's correction to the document's source
+    # and marks each applied lint as corrected.
+    #
+    # @param lints [Array<SlimLint::Lint>]
+    # @return [String] the corrected source
+    def correct(lints)
+      lines = @document.source_lines.dup
+
+      lints.select(&:correctable?).group_by(&:line).each do |line_number, line_lints|
+        index = line_number - 1
+        next unless lines[index]
+
+        line_lints.each do |lint|
+          lines[index] = lint.correction.call(lines[index])
+          lint.corrected = true
+        end
+      end
+
+      corrected_source(lines)
+    end
+
+    private
+
+    # @param lines [Array<String>]
+    # @return [String]
+    def corrected_source(lines)
+      source = lines.join("\n")
+      trailing_newlines = @document.source[/\n+\z/]
+      source += trailing_newlines if trailing_newlines
+      @document.source_prefix + source
+    end
+  end
+end

data/lib/slim_lint/document.rb

@@ -15,6 +15,12 @@ module SlimLint
     # @return [String] original source code
     attr_reader :source
 
+    # @return [String] source code before optional frontmatter removal
+    attr_reader :original_source
+
+    # @return [String] source removed before linting, such as frontmatter
+    attr_reader :source_prefix
+
     # @return [Array<String>] original source code as an array of lines
     attr_reader :source_lines
 
@@ -36,8 +42,9 @@ module SlimLint
     # @param source [String] Slim code to parse
     # @raise [SlimLint::Exceptions::ParseError] if there was a problem parsing the document
     def process_source(source)
-      @source = process_encoding(source)
-      @source = strip_frontmatter(source)
+      @original_source = process_encoding(source)
+      @source = strip_frontmatter(@original_source)
+      @source_prefix = @original_source[0, @original_source.length - @source.length] || ''
       @source_lines = @source.split("\n")
 
       engine = SlimLint::Engine.new(file: @file)

data/lib/slim_lint/lint.rb

@@ -18,6 +18,13 @@ module SlimLint
     # @return [Symbol] whether this lint is a warning or an error
     attr_reader :severity
 
+    # @return [Proc, nil] maps a line of source to its corrected version, or
+    #   `nil` if this lint has no known automatic correction
+    attr_reader :correction
+
+    # @return [Boolean] whether this lint's correction has been applied
+    attr_accessor :corrected
+
     # Creates a new lint.
     #
     # @param linter [SlimLint::Linter]
@@ -25,12 +32,16 @@ module SlimLint
     # @param line [Fixnum]
     # @param message [String]
     # @param severity [Symbol]
-    def initialize(linter, filename, line, message, severity = :warning)
-      @linter   = linter
-      @filename = filename
-      @line     = line || 0
-      @message  = message
-      @severity = severity
+    # @param correction [Proc, nil] maps a line of source to its corrected version
+    def initialize(linter, filename, line, message, severity = :warning, # rubocop:disable Metrics/ParameterLists
+                   correction: nil)
+      @linter     = linter
+      @filename   = filename
+      @line       = line || 0
+      @message    = message
+      @severity   = severity
+      @correction = correction
+      @corrected  = false
     end
 
     # Return whether this lint has a severity of error.
@@ -39,5 +50,19 @@ module SlimLint
     def error?
       @severity == :error
     end
+
+    # Return whether this lint has a known automatic correction.
+    #
+    # @return [Boolean]
+    def correctable?
+      !@correction.nil?
+    end
+
+    # Return whether this lint's correction has been applied.
+    #
+    # @return [Boolean]
+    def corrected?
+      @corrected
+    end
   end
 end

data/lib/slim_lint/linter/quote_consistency.rb

@@ -4,6 +4,7 @@ module SlimLint
   # Checks for consistent quote usage in HTML attributes
   class Linter::QuoteConsistency < Linter
     include LinterRegistry
+    support_autocorrect
 
     MSG = 'Inconsistent quote style. %s'
 
@@ -19,15 +20,42 @@ module SlimLint
 
       if enforced_style == :single_quotes && double_quotes.any?
         report_lint(node,
-                    format(MSG, "Use single quotes for attribute values (')"))
+                    format(MSG, "Use single quotes for attribute values (')"),
+                    correction: ->(source_line) { correct_quotes(source_line, '"', "'") })
       elsif enforced_style == :double_quotes && single_quotes.any?
         report_lint(node,
-                    format(MSG, 'Use double quotes for attribute values (")'))
+                    format(MSG, 'Use double quotes for attribute values (")'),
+                    correction: ->(source_line) { correct_quotes(source_line, "'", '"') })
       end
     end
 
     private
 
+    def correct_quotes(source_line, from, to)
+      state = { quote: nil, corrected: [] }
+      source_line.scan(/\\.|['"]|[^\\'"]+/).each do |token|
+        correct_quote_token(token, state, from, to)
+      end
+      state[:corrected].join
+    end
+
+    def correct_quote_token(token, state, from, to)
+      return state[:corrected] << token if token.length > 1 || token.start_with?('\\')
+
+      if state[:quote]
+        close_quote_token(token, state, from, to)
+      else
+        state[:quote] = token
+        state[:corrected] << (token == from ? to : token)
+      end
+    end
+
+    def close_quote_token(token, state, from, to)
+      replacement = token == state[:quote] && token == from ? to : token
+      state[:corrected] << replacement
+      state[:quote] = nil if token == state[:quote]
+    end
+
     def enforced_style
       config['enforced_style']&.to_sym || :single_quotes
     end

data/lib/slim_lint/linter/redundant_div.rb

@@ -5,6 +5,7 @@ module SlimLint
   # already implies a div.
   class Linter::RedundantDiv < Linter
     include LinterRegistry
+    support_autocorrect
 
     MESSAGE = '`div` is redundant when %s attribute shortcut is present'
 
@@ -16,7 +17,8 @@ module SlimLint
       attr = captures[:attr_name]
       next unless %w[class id].include?(attr)
 
-      report_lint(sexp, MESSAGE % attr)
+      report_lint(sexp, MESSAGE % attr,
+                  correction: ->(line) { line.sub(/\bdiv(?=[.#])/, '') })
     end
   end
 end

data/lib/slim_lint/linter/tag_case.rb

@@ -4,12 +4,16 @@ module SlimLint
   # Searches for tags with uppercase characters.
   class Linter::TagCase < Linter
     include LinterRegistry
+    support_autocorrect
 
     on [:html, :tag] do |sexp|
       _, _, name = sexp
       next unless name[/[A-Z]/]
 
-      report_lint(sexp, "Tag `#{name}` should be written as `#{name.downcase}`")
+      report_lint(sexp, "Tag `#{name}` should be written as `#{name.downcase}`",
+                  correction: ->(line) {
+                    line.sub(/\b#{Regexp.escape(name.to_s)}\b/, name.downcase)
+                  })
     end
   end
 end

data/lib/slim_lint/linter/trailing_whitespace.rb

@@ -4,6 +4,7 @@ module SlimLint
   # Checks for trailing whitespace.
   class Linter::TrailingWhitespace < Linter
     include LinterRegistry
+    support_autocorrect
 
     on_start do |_sexp|
       dummy_node = Struct.new(:line)
@@ -12,7 +13,8 @@ module SlimLint
         next unless line =~ /\s+$/
 
         report_lint(dummy_node.new(index + 1),
-                    'Line contains trailing whitespace')
+                    'Line contains trailing whitespace',
+                    correction: lambda(&:rstrip))
       end
     end
   end

data/lib/slim_lint/linter.rb

@@ -15,6 +15,21 @@ module SlimLint
     #   lints for the subject instead of the linter itself.
     attr_reader :lints
 
+    class << self
+      # Declares that this linter can automatically correct (some of) the
+      # offenses it reports.
+      def support_autocorrect
+        @supports_autocorrect = true
+      end
+
+      # Returns whether this linter declared autocorrect support.
+      #
+      # @return [Boolean]
+      def supports_autocorrect?
+        @supports_autocorrect || false
+      end
+    end
+
     # Initializes a linter with the specified configuration.
     #
     # @param config [Hash] configuration for this linter
@@ -49,10 +64,13 @@ module SlimLint
     #
     # @param node [#line] node to extract the line number from
     # @param message [String] error/warning to display to the user
-    def report_lint(node, message)
+    # @param correction [Proc, nil] maps the offending line of source to its
+    #   corrected version, for linters that declared {support_autocorrect}
+    def report_lint(node, message, correction: nil)
       return if disabled_for_line?(node.line)
 
-      @lints << SlimLint::Lint.new(self, @document.file, node.line, message)
+      @lints << SlimLint::Lint.new(self, @document.file, node.line, message,
+                                   correction: correction)
     end
 
     # Parse Ruby code into an abstract syntax tree.

data/lib/slim_lint/options.rb

@@ -48,6 +48,11 @@ module SlimLint
                 'Specify which reporter you want to use to generate the output') do |reporter|
         @options[:reporter] = load_reporter_class(reporter.capitalize)
       end
+
+      parser.on('-a', '--autocorrect',
+                'Automatically correct offenses that support it') do
+        @options[:autocorrect] = true
+      end
     end
 
     # Returns the class of the specified Reporter.

data/lib/slim_lint/report.rb

@@ -19,7 +19,7 @@ module SlimLint
     end
 
     def failed?
-      @lints.any?
+      @lints.any? { |lint| !lint.corrected? }
     end
   end
 end

data/lib/slim_lint/reporter/default_reporter.rb

@@ -35,7 +35,8 @@ module SlimLint
         log.success("#{lint.linter.name}: ", false)
       end
 
-      log.log lint.message
+      log.log lint.message, false
+      lint.corrected? ? log.success(' [Corrected]') : log.newline
     end
   end
 end

data/lib/slim_lint/runner.rb

@@ -20,7 +20,8 @@ module SlimLint
       if options[:stdin_file_path].nil?
         files = extract_applicable_files(config, options)
         lints = files.map do |file|
-          collect_lints(File.read(file), file, linter_selector, config)
+          collect_lints(File.read(file), file, linter_selector, config,
+                        autocorrect: options[:autocorrect])
         end.flatten
       else
         files = [options[:stdin_file_path]]
@@ -53,16 +54,34 @@ module SlimLint
     # @param file [String] path to file to lint
     # @param linter_selector [SlimLint::LinterSelector]
     # @param config [SlimLint::Configuration]
-    def collect_lints(file_content, file_name, linter_selector, config)
+    # @param autocorrect [Boolean] whether to write corrections back to the file
+    def collect_lints(file_content, file_name, linter_selector, config, autocorrect: false)
       begin
         document = SlimLint::Document.new(file_content, file: file_name, config: config)
       rescue SlimLint::Exceptions::ParseError => e
         return [SlimLint::Lint.new(nil, file_name, e.lineno, e.error, :error)]
       end
 
-      linter_selector.linters_for_file(file_name).map do |linter|
+      lints = linter_selector.linters_for_file(file_name).map do |linter|
         linter.run(document)
       end.flatten
+
+      correct_lints(document, file_name, lints) if autocorrect
+
+      lints
+    end
+
+    # Applies any available corrections to the given file's lints and writes
+    # the result back to disk if anything changed.
+    #
+    # @param document [SlimLint::Document]
+    # @param file_name [String] path to file to correct
+    # @param lints [Array<SlimLint::Lint>]
+    def correct_lints(document, file_name, lints)
+      corrected_source = SlimLint::Corrector.new(document).correct(lints)
+      return if corrected_source == document.original_source
+
+      File.write(file_name, corrected_source)
     end
 
     # Returns the list of files that should be linted given the specified

data/lib/slim_lint/version.rb

@@ -2,5 +2,5 @@
 
 # Defines the gem version.
 module SlimLint
-  VERSION = '0.35.0'
+  VERSION = '0.36.0'
 end

data/lib/slim_lint.rb

@@ -30,6 +30,7 @@ require_relative 'slim_lint/sexp_visitor'
 require_relative 'slim_lint/lint'
 require_relative 'slim_lint/ruby_parser'
 require_relative 'slim_lint/linter'
+require_relative 'slim_lint/corrector'
 require_relative 'slim_lint/reporter'
 require_relative 'slim_lint/report'
 require_relative 'slim_lint/linter_selector'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: slim_lint
 version: !ruby/object:Gem::Version
-  version: 0.35.0
+  version: 0.36.0
 platform: ruby
 authors:
 - Shane da Silva
@@ -109,6 +109,7 @@ files:
 - lib/slim_lint/configuration.rb
 - lib/slim_lint/configuration_loader.rb
 - lib/slim_lint/constants.rb
+- lib/slim_lint/corrector.rb
 - lib/slim_lint/document.rb
 - lib/slim_lint/engine.rb
 - lib/slim_lint/exceptions.rb