jawshooah-overcommit 0.22.0

data/lib/overcommit/hook/pre_commit/berksfile_check.rb

@@ -0,0 +1,20 @@
+module Overcommit::Hook::PreCommit
+  # Check if local Berksfile.lock matches Berksfile when either changes, unless
+  # Berksfile.lock is ignored by git.
+  class BerksfileCheck < Base
+    LOCK_FILE = 'Berksfile.lock'
+
+    def run
+      # Ignore if Berksfile.lock is not tracked by git
+      ignored_files = execute(%w[git ls-files -o -i --exclude-standard]).stdout.split("\n")
+      return :pass if ignored_files.include?(LOCK_FILE)
+
+      result = execute(%W[#{executable} list --quiet])
+      unless result.success?
+        return :fail, result.stderr
+      end
+
+      :pass
+    end
+  end
+end

data/lib/overcommit/hook/pre_commit/brakeman.rb

@@ -0,0 +1,12 @@
+module Overcommit::Hook::PreCommit
+  # Runs `brakeman` against any modified Ruby/Rails files.
+  class Brakeman < Base
+    def run
+      result = execute(%W[#{executable} --exit-on-warn --quiet --summary --only-files] <<
+                       applicable_files.join(','))
+      return :pass if result.success?
+
+      [:fail, result.stdout]
+    end
+  end
+end

data/lib/overcommit/hook/pre_commit/broken_symlinks.rb

@@ -0,0 +1,15 @@
+module Overcommit::Hook::PreCommit
+  # Checks for broken symlinks.
+  class BrokenSymlinks < Base
+    def run
+      broken_symlinks = applicable_files.
+        select { |file| Overcommit::Utils.broken_symlink?(file) }
+
+      if broken_symlinks.any?
+        return :fail, "Broken symlinks detected:\n#{broken_symlinks.join("\n")}"
+      end
+
+      :pass
+    end
+  end
+end

data/lib/overcommit/hook/pre_commit/bundle_check.rb

@@ -0,0 +1,25 @@
+module Overcommit::Hook::PreCommit
+  # Check if local Gemfile.lock matches Gemfile when either changes, unless
+  # Gemfile.lock is ignored by git.
+  class BundleCheck < Base
+    LOCK_FILE = 'Gemfile.lock'
+
+    def run
+      # Ignore if Gemfile.lock is not tracked by git
+      ignored_files = execute(%w[git ls-files -o -i --exclude-standard]).stdout.split("\n")
+      return :pass if ignored_files.include?(LOCK_FILE)
+
+      result = execute(%W[#{executable} check])
+      unless result.success?
+        return :fail, result.stdout
+      end
+
+      result = execute(%w[git diff --quiet --] + [LOCK_FILE])
+      unless result.success?
+        return :fail, "#{LOCK_FILE} is not up-to-date -- run `#{executable} check`"
+      end
+
+      :pass
+    end
+  end
+end

data/lib/overcommit/hook/pre_commit/chamber_security.rb

@@ -0,0 +1,11 @@
+module Overcommit::Hook::PreCommit
+  # Runs `chamber secure` against any modified Chamber settings files
+  class ChamberSecurity < Base
+    def run
+      result = execute(%W[#{executable} secure --files] + applicable_files)
+
+      return :pass if result.stdout.empty?
+      [:fail, "These settings appear to need to be secured but were not: #{result.stdout}"]
+    end
+  end
+end

data/lib/overcommit/hook/pre_commit/coffee_lint.rb

@@ -0,0 +1,11 @@
+module Overcommit::Hook::PreCommit
+  # Runs `coffeelint` against any modified CoffeeScript files.
+  class CoffeeLint < Base
+    def run
+      result = execute(%W[#{executable} --quiet] + applicable_files)
+      return :pass if result.success?
+
+      [:fail, result.stdout]
+    end
+  end
+end

data/lib/overcommit/hook/pre_commit/css_lint.rb

@@ -0,0 +1,11 @@
+module Overcommit::Hook::PreCommit
+  # Runs `csslint` against any modified CSS files.
+  class CssLint < Base
+    def run
+      result = execute(%W[#{executable} --quiet --format=compact] + applicable_files)
+      return :pass if result.stdout !~ /Error - (?!Unknown @ rule)/
+
+      [:fail, result.stdout]
+    end
+  end
+end

data/lib/overcommit/hook/pre_commit/go_lint.rb

@@ -0,0 +1,12 @@
+module Overcommit::Hook::PreCommit
+  # Runs `golint` against any modified Golang files.
+  class GoLint < Base
+    def run
+      result = execute([executable] + applicable_files)
+      # Unfortunately the exit code is always 0
+      return :pass if result.stdout.empty?
+
+      [:fail, result.stdout]
+    end
+  end
+end

data/lib/overcommit/hook/pre_commit/haml_lint.rb

@@ -0,0 +1,19 @@
+module Overcommit::Hook::PreCommit
+  # Runs `haml-lint` against any modified HAML files.
+  class HamlLint < Base
+    MESSAGE_TYPE_CATEGORIZER = lambda do |type|
+      type.include?('W') ? :warning : :error
+    end
+
+    def run
+      result = execute([executable] + applicable_files)
+      return :pass if result.success?
+
+      extract_messages(
+        result.stdout.split("\n"),
+        /^(?<file>[^:]+):(?<line>\d+)[^ ]* (?<type>[^ ]+)/,
+        MESSAGE_TYPE_CATEGORIZER,
+      )
+    end
+  end
+end

data/lib/overcommit/hook/pre_commit/hard_tabs.rb

@@ -0,0 +1,14 @@
+module Overcommit::Hook::PreCommit
+  # Checks for hard tabs in files.
+  class HardTabs < Base
+    def run
+      # Catches hard tabs
+      result = execute(%w[grep -IHn] + ["\t"] + applicable_files)
+      unless result.stdout.empty?
+        return :fail, "Hard tabs detected:\n#{result.stdout}"
+      end
+
+      :pass
+    end
+  end
+end

data/lib/overcommit/hook/pre_commit/image_optim.rb

@@ -0,0 +1,41 @@
+module Overcommit::Hook::PreCommit
+  # Checks for images that can be optimized with `image_optim`.
+  class ImageOptim < Base
+    def run
+      begin
+        require 'image_optim'
+      rescue LoadError
+        return :warn, 'image_optim not installed -- run `gem install image_optim`'
+      end
+
+      optimized_images =
+        begin
+          optimize_images(applicable_files)
+        rescue ::ImageOptim::BinResolver::BinNotFound => e
+          return :fail, "#{e.message}. The image_optim gem is dependendent on this binary."
+        end
+
+      if optimized_images.any?
+        return :fail,
+          "The following images are optimizable:\n#{optimized_images.join("\n")}" \
+          "\n\nOptimize them by running:\n" \
+          "  image_optim --skip-missing-workers #{optimized_images.join(' ')}"
+      end
+
+      :pass
+    end
+
+    private
+
+    def optimize_images(image_paths)
+      image_optim = ::ImageOptim.new(skip_missing_workers: true)
+
+      optimized_images =
+        image_optim.optimize_images(image_paths) do |path, optimized|
+          path if optimized
+        end
+
+      optimized_images.compact
+    end
+  end
+end

data/lib/overcommit/hook/pre_commit/js_hint.rb

@@ -0,0 +1,13 @@
+module Overcommit::Hook::PreCommit
+  # Runs `jshint` against any modified JavaScript files.
+  class JsHint < Base
+    def run
+      result = execute([executable] + applicable_files)
+      output = result.stdout
+
+      return :pass if output.empty?
+
+      [:fail, output]
+    end
+  end
+end

data/lib/overcommit/hook/pre_commit/jscs.rb

@@ -0,0 +1,22 @@
+module Overcommit::Hook::PreCommit
+  # Runs `jscs` (JavaScript Code Style Checker) against any modified JavaScript
+  # files.
+  class Jscs < Base
+    def run
+      result = execute(%W[#{executable} --reporter=inline --verbose] + applicable_files)
+      return :pass if result.success?
+
+      if result.status == 1
+        # No configuration was found
+        return :warn, result.stderr.chomp
+      end
+
+      # example message:
+      #   path/to/file.js: line 7, col 0, ruleName: Error message
+      extract_messages(
+        result.stdout.split("\n"),
+        /^(?<file>[^:]+):[^\d]+(?<line>\d+)/,
+      )
+    end
+  end
+end

data/lib/overcommit/hook/pre_commit/json_syntax.rb

@@ -0,0 +1,22 @@
+require 'json'
+
+module Overcommit::Hook::PreCommit
+  # Checks the syntax of any modified JSON files.
+  class JsonSyntax < Base
+    def run
+      output = []
+
+      applicable_files.each do |file|
+        begin
+          JSON.parse(IO.read(file))
+        rescue JSON::ParserError => e
+          output << "#{e.message} parsing #{file}"
+        end
+      end
+
+      return :pass if output.empty?
+
+      [:fail, output]
+    end
+  end
+end

data/lib/overcommit/hook/pre_commit/jsx_hint.rb

@@ -0,0 +1,13 @@
+module Overcommit::Hook::PreCommit
+  # Runs `jsxhint` against any modified JSX files.
+  class JsxHint < Base
+    def run
+      result = execute([executable] + applicable_files)
+      output = result.stdout
+
+      return :pass if output.empty?
+
+      [:fail, output]
+    end
+  end
+end

data/lib/overcommit/hook/pre_commit/jsxcs.rb

@@ -0,0 +1,20 @@
+module Overcommit::Hook::PreCommit
+  # Runs `jsxcs` (JSCS (JavaScript Code Style Checker) wrapper for JSX files)
+  # against any modified JavaScript files.
+  class Jsxcs < Base
+    def run
+      result = execute(%W[#{executable} --reporter=inline] + applicable_files)
+      return :pass if result.success?
+
+      if result.status == 1
+        # No configuration found
+        return :warn, result.stderr.chomp
+      end
+
+      extract_messages(
+        result.stdout.split("\n"),
+        /^(?<file>[^:]+):[^\d]+(?<line>\d+)/,
+      )
+    end
+  end
+end

data/lib/overcommit/hook/pre_commit/local_paths_in_gemfile.rb

@@ -0,0 +1,14 @@
+module Overcommit::Hook::PreCommit
+  # Checks for local paths in files and issues a warning
+  class LocalPathsInGemfile < Base
+    def run
+      result = execute(%w[grep -IHnE (\s*path:\s*)|(\s*:path\s*=>)] + applicable_files)
+
+      unless result.stdout.empty?
+        return :warn, "Avoid pointing to local paths in Gemfiles:\n#{result.stdout}"
+      end
+
+      :pass
+    end
+  end
+end

data/lib/overcommit/hook/pre_commit/merge_conflicts.rb

@@ -0,0 +1,14 @@
+module Overcommit::Hook::PreCommit
+  # Checks for unresolved merge conflicts
+  class MergeConflicts < Base
+    def run
+      result = execute(%w[grep -IHn ^<<<<<<<\s] + applicable_files)
+
+      unless result.stdout.empty?
+        return :fail, "Merge conflict markers detected:\n#{result.stdout}"
+      end
+
+      :pass
+    end
+  end
+end

data/lib/overcommit/hook/pre_commit/pry_binding.rb

@@ -0,0 +1,14 @@
+module Overcommit::Hook::PreCommit
+  # Adds a check to make sure no `binding.pry`'s have been left in the code
+  class PryBinding < Base
+    def run
+      result = execute(%w[grep -IHnE ^\s*binding\.pry] + applicable_files)
+
+      unless result.stdout.empty?
+        return :fail, "Found a `binding.pry` call left in:\n#{result.stdout}"
+      end
+
+      :pass
+    end
+  end
+end

data/lib/overcommit/hook/pre_commit/python_flake8.rb

@@ -0,0 +1,11 @@
+module Overcommit::Hook::PreCommit
+  # Runs `flake8` against any modified Python files.
+  class PythonFlake8 < Base
+    def run
+      result = execute([executable] + applicable_files)
+      return :pass if result.success?
+
+      [:fail, result.stdout]
+    end
+  end
+end

data/lib/overcommit/hook/pre_commit/rails_schema_up_to_date.rb

@@ -0,0 +1,45 @@
+module Overcommit::Hook::PreCommit
+  # Check to see whether the schema file is in line with the migrations
+  class RailsSchemaUpToDate < Base
+    def run # rubocop:disable CyclomaticComplexity, PerceivedComplexity
+      if migration_files.any? && schema_files.none?
+        return :fail, "It looks like you're adding a migration, but did not update the schema file"
+      elsif migration_files.none? && schema_files.any?
+        return :fail, "You're trying to change the schema without adding a migration file"
+      elsif migration_files.any? && schema_files.any?
+        # Get the latest version from the migration filename. Use
+        # `File.basename` to prevent finding numbers that could appear in
+        # directories, such as the home directory of a user with a number in
+        # their username.
+        latest_version = migration_files.map do |file|
+          File.basename(file)[/\d+/]
+        end.sort.last
+
+        schema     = schema_files.map { |file| File.read(file) }.join
+        up_to_date = schema.include?(latest_version)
+
+        unless up_to_date
+          return :fail, "The latest migration version you're committing is " \
+                       "#{latest_version}, but your schema file " \
+                       "#{schema_files.join(' or ')} is on a different version."
+        end
+      end
+
+      :pass
+    end
+
+    private
+
+    def migration_files
+      @migration_files ||= applicable_files.select do |file|
+        file.match %r{db/migrate/.*\.rb}
+      end
+    end
+
+    def schema_files
+      @schema_files    ||= applicable_files.select do |file|
+        file.match %r{db/schema\.rb|db/structure.*\.sql}
+      end
+    end
+  end
+end

data/lib/overcommit/hook/pre_commit/reek.rb

@@ -0,0 +1,22 @@
+module Overcommit::Hook::PreCommit
+  # Runs `reek` against any modified Ruby files.
+  class Reek < Base
+    def run
+      result = execute(%W[#{executable} --single-line --no-color] + applicable_files)
+      return :pass if result.success?
+
+      output = scrub_output(result.stdout + result.stderr)
+
+      extract_messages(
+        output,
+        /^(?<file>[^:]+):(?<line>\d+):/,
+      )
+    end
+
+    private
+
+    def scrub_output(raw_output)
+      raw_output.split("\n").grep(/^(.(?!warning))*$/)
+    end
+  end
+end