rails_git_hooks 0.7.1 → 0.7.3

data/lib/rails_git_hooks/config/defaults.yml

@@ -96,6 +96,160 @@ PreCommit:
     file_based: true
     install_hint: Add `gem "rubocop"` to your Gemfile and run bundle install
 
+  RailsBestPractices:
+    enabled: false
+    quiet: true
+    on_fail: warn
+    on_warn: warn
+    on_missing_dependency: warn
+    include: []
+    exclude: []
+    dependencies:
+      executables: [bundle]
+      libraries: [rails_best_practices]
+    command: [bundle, exec, rails_best_practices]
+    description: Warn on Rails best practices violations
+    file_based: true
+    install_hint: Add `gem "rails_best_practices"` to your Gemfile and run bundle install
+
+  ErbLint:
+    enabled: false
+    quiet: true
+    on_fail: fail
+    on_warn: warn
+    on_missing_dependency: warn
+    include: []
+    exclude: []
+    dependencies:
+      executables: [bundle]
+      libraries: [erblint]
+    command: [bundle, exec, erblint]
+    description: Run erblint on staged ERB files
+    file_based: true
+    install_hint: Add `gem "erblint"` to your Gemfile and run bundle install
+
+  Eslint:
+    enabled: false
+    quiet: true
+    on_fail: fail
+    on_warn: warn
+    on_missing_dependency: warn
+    include: []
+    exclude: []
+    dependencies:
+      executables: [eslint]
+    command: [eslint]
+    description: Run eslint on staged JavaScript files
+    file_based: true
+    install_hint: Install eslint (e.g. npm install -D eslint) or override command in config
+
+  Golint:
+    enabled: false
+    quiet: true
+    on_fail: fail
+    on_warn: warn
+    on_missing_dependency: warn
+    include: []
+    exclude: []
+    dependencies:
+      executables: [golint]
+    command: [golint]
+    description: Run golint on staged Go files
+    file_based: true
+    install_hint: Install golint (e.g. go install golang.org/x/lint/golint@latest) or override command in config
+
+  HamlLint:
+    enabled: false
+    quiet: true
+    on_fail: fail
+    on_warn: warn
+    on_missing_dependency: warn
+    include: []
+    exclude: []
+    dependencies:
+      executables: [bundle]
+      libraries: [haml_lint]
+    command: [bundle, exec, haml-lint]
+    description: Run haml-lint on staged HAML files
+    file_based: true
+    install_hint: Add `gem "haml_lint"` to your Gemfile and run bundle install
+
+  Jslint:
+    enabled: false
+    quiet: true
+    on_fail: fail
+    on_warn: warn
+    on_missing_dependency: warn
+    include: []
+    exclude: []
+    dependencies:
+      executables: [jslint]
+    command: [jslint]
+    description: Run jslint on staged JavaScript files
+    file_based: true
+    install_hint: Install jslint (e.g. npm install -D jslint or npm install -g jslint) or override command in config
+
+  PhpLint:
+    enabled: false
+    quiet: true
+    on_fail: fail
+    on_warn: warn
+    on_missing_dependency: warn
+    include: []
+    exclude: []
+    dependencies:
+      executables: [php]
+    command: [php, "-l"]
+    description: Run php -l (syntax check) on staged PHP files
+    file_based: true
+    install_hint: Ensure PHP is installed and on PATH
+
+  Pylint:
+    enabled: false
+    quiet: true
+    on_fail: fail
+    on_warn: warn
+    on_missing_dependency: warn
+    include: []
+    exclude: []
+    dependencies:
+      executables: [pylint]
+    command: [pylint]
+    description: Run pylint on staged Python files
+    file_based: true
+    install_hint: Install pylint (e.g. pip install pylint) or override command in config
+
+  ScssLint:
+    enabled: false
+    quiet: true
+    on_fail: fail
+    on_warn: warn
+    on_missing_dependency: warn
+    include: []
+    exclude: []
+    dependencies:
+      executables: [bundle]
+      libraries: [scss_lint]
+    command: [bundle, exec, scss-lint]
+    description: Run scss-lint on staged SCSS files
+    file_based: true
+    install_hint: Add `gem "scss_lint"` to your Gemfile and run bundle install
+
+  GoVet:
+    enabled: false
+    quiet: true
+    on_fail: fail
+    on_warn: warn
+    on_missing_dependency: warn
+    include: []
+    exclude: []
+    dependencies:
+      executables: [go]
+    command: [go, vet]
+    description: Run go vet on staged Go files
+    file_based: true
+    install_hint: Ensure Go toolchain is installed and on PATH
+
 CommitMsg:
   JiraPrefix:
     enabled: true
@@ -110,9 +264,22 @@ CommitMsg:
     description: Prefix commit messages with ticket id from branch
     file_based: false
 
+  NotEmpty:
+    enabled: true
+    quiet: false
+    on_fail: fail
+    on_warn: warn
+    on_missing_dependency: warn
+    include: []
+    exclude: []
+    dependencies: {}
+    command: []
+    description: Reject empty commit messages
+    file_based: false
+
 PrePush:
   RunTests:
-    enabled: true
+    enabled: false
     quiet: false
     on_fail: fail
     on_warn: warn
@@ -121,7 +288,160 @@ PrePush:
     exclude: []
     dependencies:
       executables: [bundle]
+      libraries: [rspec]
     command: [bundle, exec, rspec]
     description: Run test suite before push
     file_based: false
     install_hint: Install test dependencies and ensure `bundle exec rspec` runs successfully
+
+  RunPytest:
+    enabled: false
+    quiet: false
+    on_fail: fail
+    on_warn: warn
+    on_missing_dependency: warn
+    include: []
+    exclude: []
+    dependencies:
+      executables: [pytest]
+    command: [pytest]
+    description: Run pytest test suite before push
+    file_based: false
+    install_hint: Install pytest (e.g. pip install pytest) and ensure pytest runs successfully
+
+  RunGoTest:
+    enabled: false
+    quiet: false
+    on_fail: fail
+    on_warn: warn
+    on_missing_dependency: warn
+    include: []
+    exclude: []
+    dependencies:
+      executables: [go]
+    command: [go, test, "./..."]
+    description: Run go test suite before push
+    file_based: false
+    install_hint: Ensure Go toolchain is installed and `go test ./...` runs successfully
+
+PostCheckout:
+  BundleInstall:
+    enabled: false
+    quiet: false
+    on_fail: fail
+    on_warn: warn
+    on_missing_dependency: warn
+    include: [Gemfile, Gemfile.lock]
+    exclude: []
+    dependencies:
+      executables: [bundle]
+    command: [bundle, install]
+    description: Run bundle install when Gemfile or Gemfile.lock changed (branch checkout)
+    file_based: true
+    install_hint: Ensure bundle is available
+
+  DbMigrate:
+    enabled: false
+    quiet: false
+    on_fail: fail
+    on_warn: warn
+    on_missing_dependency: warn
+    include: [db/migrate/*.rb, db/schema.rb, db/structure.sql]
+    exclude: []
+    dependencies:
+      executables: [bundle]
+    command: [bundle, exec, rails, db:migrate]
+    description: Run db:migrate when migrations or schema changed (branch checkout)
+    file_based: true
+    install_hint: Rails app with db:migrate (or override command in config)
+
+  NpmInstall:
+    enabled: false
+    quiet: false
+    on_fail: fail
+    on_warn: warn
+    on_missing_dependency: warn
+    include: [package.json, package-lock.json]
+    exclude: []
+    dependencies:
+      executables: [npm]
+    command: [npm, install]
+    description: Run npm install when package.json or package-lock.json changed (branch checkout)
+    file_based: true
+    install_hint: Ensure npm is available
+
+  YarnInstall:
+    enabled: false
+    quiet: false
+    on_fail: fail
+    on_warn: warn
+    on_missing_dependency: warn
+    include: [package.json, yarn.lock]
+    exclude: []
+    dependencies:
+      executables: [yarn]
+    command: [yarn, install]
+    description: Run yarn install when package.json or yarn.lock changed (branch checkout)
+    file_based: true
+    install_hint: Ensure yarn is available
+
+PostMerge:
+  BundleInstall:
+    enabled: false
+    quiet: false
+    on_fail: fail
+    on_warn: warn
+    on_missing_dependency: warn
+    include: [Gemfile, Gemfile.lock]
+    exclude: []
+    dependencies:
+      executables: [bundle]
+    command: [bundle, install]
+    description: Run bundle install when Gemfile or Gemfile.lock changed (after merge)
+    file_based: true
+    install_hint: Ensure bundle is available
+
+  DbMigrate:
+    enabled: false
+    quiet: false
+    on_fail: fail
+    on_warn: warn
+    on_missing_dependency: warn
+    include: [db/migrate/*.rb, db/schema.rb, db/structure.sql]
+    exclude: []
+    dependencies:
+      executables: [bundle]
+    command: [bundle, exec, rails, db:migrate]
+    description: Run db:migrate when migrations or schema changed (after merge)
+    file_based: true
+    install_hint: Rails app with db:migrate (or override command in config)
+
+  NpmInstall:
+    enabled: false
+    quiet: false
+    on_fail: fail
+    on_warn: warn
+    on_missing_dependency: warn
+    include: [package.json, package-lock.json]
+    exclude: []
+    dependencies:
+      executables: [npm]
+    command: [npm, install]
+    description: Run npm install when package.json or package-lock.json changed (after merge)
+    file_based: true
+    install_hint: Ensure npm is available
+
+  YarnInstall:
+    enabled: false
+    quiet: false
+    on_fail: fail
+    on_warn: warn
+    on_missing_dependency: warn
+    include: [package.json, yarn.lock]
+    exclude: []
+    dependencies:
+      executables: [yarn]
+    command: [yarn, install]
+    description: Run yarn install when package.json or yarn.lock changed (after merge)
+    file_based: true
+    install_hint: Ensure yarn is available

data/lib/rails_git_hooks/install/installer.rb

@@ -5,19 +5,23 @@ require 'fileutils'
 module GitHooks
   class Installer
     def initialize(git_dir: nil)
-      @git_dir = git_dir || Repository.new.git_dir
+      if git_dir
+        @git_dir = git_dir
+        @repo = nil
+      else
+        @repo = Repository.new
+        @git_dir = @repo.git_dir
+      end
     end
 
-    def install(*hook_names)
+    def install
       target_dir = File.join(@git_dir, 'hooks')
       raise GitHooks::Error, "Not a git repository or .git/hooks not found: #{@git_dir}" unless Dir.exist?(target_dir)
 
       copy_runtime(target_dir)
 
-      hooks = hook_names.empty? ? Constants::DEFAULT_HOOKS : hook_names
+      hooks = hooks_enabled_in_config.select { |name| self.class.available_hook_names.include?(name) }
       hooks.each_with_object([]) do |name, installed|
-        next unless self.class.available_hook_names.include?(name)
-
         dest = File.join(target_dir, name)
         File.write(dest, File.read(File.join(Constants::HOOKS_DIR, name)))
         File.chmod(0o755, dest)
@@ -35,6 +39,28 @@ module GitHooks
 
     private
 
+    def repo_for_config
+      @repo_for_config ||= begin
+        root = File.dirname(@git_dir)
+        Struct.new(:root, :config_path, :local_config_path).new(
+          root,
+          File.join(root, Constants::CONFIG_FILE),
+          File.join(root, Constants::CONFIG_FILE_LOCAL)
+        )
+      end
+    end
+
+    def hooks_enabled_in_config
+      override_config = OverrideConfig.new(repo: repo_for_config)
+      effective = override_config.effective_config(CheckRegistry.all)
+      effective.each_with_object([]) do |(section_name, check_configs), out|
+        hook_name = Constants::SECTION_TO_HOOK[section_name]
+        next unless hook_name && self.class.available_hook_names.include?(hook_name)
+
+        out << hook_name if check_configs.values.any? { |cfg| cfg['enabled'] == true }
+      end.uniq
+    end
+
     def copy_runtime(target_dir)
       runtime_dir = File.join(target_dir, Constants::RUNTIME_DIR_NAME)
       FileUtils.rm_rf(runtime_dir)

data/lib/rails_git_hooks/runtime/check_registry.rb

@@ -10,8 +10,29 @@ module GitHooks
       Checks::PreCommit::MigrationsCheck,
       Checks::PreCommit::WhitespaceCheck,
       Checks::PreCommit::RuboCop,
+      Checks::PreCommit::RailsBestPractices,
+      Checks::PreCommit::ErbLint,
+      Checks::PreCommit::Eslint,
+      Checks::PreCommit::Golint,
+      Checks::PreCommit::HamlLint,
+      Checks::PreCommit::Jslint,
+      Checks::PreCommit::PhpLint,
+      Checks::PreCommit::Pylint,
+      Checks::PreCommit::ScssLint,
+      Checks::PreCommit::GoVet,
       Checks::CommitMsg::JiraPrefix,
-      Checks::PrePush::RunTests
+      Checks::CommitMsg::NotEmpty,
+      Checks::PrePush::RunTests,
+      Checks::PrePush::RunPytest,
+      Checks::PrePush::RunGoTest,
+      Checks::PostCheckout::BundleInstall,
+      Checks::PostCheckout::DbMigrate,
+      Checks::PostCheckout::NpmInstall,
+      Checks::PostCheckout::YarnInstall,
+      Checks::PostMerge::BundleInstall,
+      Checks::PostMerge::DbMigrate,
+      Checks::PostMerge::NpmInstall,
+      Checks::PostMerge::YarnInstall
     ].freeze
 
     def self.all

data/lib/rails_git_hooks/runtime/override_config.rb

@@ -15,9 +15,9 @@ module GitHooks
     end
 
     def load
-      return {} unless File.exist?(@repo.config_path)
-
-      deep_stringify(YAML.safe_load(File.read(@repo.config_path), aliases: true) || {})
+      main = load_file(@repo.config_path)
+      local = load_file(@repo.local_config_path)
+      deep_merge(main, local)
     end
 
     def config_for(definition)
@@ -35,7 +35,7 @@ module GitHooks
     end
 
     def set_option(definition, option, value)
-      data = load
+      data = load_file(@repo.config_path)
       section = data[definition.hook_section] ||= {}
       section[definition.config_name] ||= {}
 
@@ -69,6 +69,12 @@ module GitHooks
 
     private
 
+    def load_file(path)
+      return {} unless File.exist?(path)
+
+      deep_stringify(YAML.safe_load(File.read(path), aliases: true) || {})
+    end
+
     def write(data)
       if data.empty?
         FileUtils.rm_f(@repo.config_path)

data/lib/rails_git_hooks/runtime/repository.rb

@@ -14,6 +14,10 @@ module GitHooks
       File.join(root, Constants::CONFIG_FILE)
     end
 
+    def local_config_path
+      File.join(root, Constants::CONFIG_FILE_LOCAL)
+    end
+
     def hook_runtime_dir(hooks_dir)
       File.join(hooks_dir, Constants::RUNTIME_DIR_NAME)
     end
@@ -38,6 +42,10 @@ module GitHooks
       git_output('diff', '--cached', '--name-only').split("\n").map(&:strip).reject(&:empty?)
     end
 
+    def changed_files(ref1, ref2)
+      git_output('diff', '--name-only', ref1.to_s, ref2.to_s).split("\n").map(&:strip).reject(&:empty?)
+    end
+
     private
 
     def resolve_paths(start_dir)

data/lib/rails_git_hooks/runtime/runner.rb

@@ -52,6 +52,10 @@ module GitHooks
     def modified_files
       @modified_files ||= case @hook_name
                           when :pre_commit then @repo.staged_files
+                          when :post_checkout
+                            @argv[2] == '1' ? @repo.changed_files(@argv[0], @argv[1]) : []
+                          when :post_merge
+                            @repo.changed_files('ORIG_HEAD', 'HEAD')
                           else []
                           end
     end

data/lib/rails_git_hooks/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module GitHooks
-  VERSION = '0.7.1'
+  VERSION = '0.7.3'
 end

data/templates/hooks/post-checkout

@@ -0,0 +1,13 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+hooks_dir = File.dirname(File.expand_path(__FILE__))
+begin
+  require 'bundler/setup'
+rescue LoadError, StandardError
+  nil
+end
+
+require File.join(hooks_dir, 'rails_git_hooks', 'runtime')
+
+exit GitHooks::Runtime.execute(:post_checkout, argv: ARGV, stdin: $stdin.read)

data/templates/hooks/post-merge

@@ -0,0 +1,13 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+hooks_dir = File.dirname(File.expand_path(__FILE__))
+begin
+  require 'bundler/setup'
+rescue LoadError, StandardError
+  nil
+end
+
+require File.join(hooks_dir, 'rails_git_hooks', 'runtime')
+
+exit GitHooks::Runtime.execute(:post_merge, argv: ARGV, stdin: $stdin.read)

metadata

@@ -1,56 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rails_git_hooks
 version: !ruby/object:Gem::Version
-  version: 0.7.1
+  version: 0.7.3
 platform: ruby
 authors:
 - Nikita Nazarov
 bindir: bin
 cert_chain: []
 date: 1980-01-02 00:00:00.000000000 Z
-dependencies:
-- !ruby/object:Gem::Dependency
-  name: rake
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '13.0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '13.0'
-- !ruby/object:Gem::Dependency
-  name: rspec
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.0'
-- !ruby/object:Gem::Dependency
-  name: rubocop
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.0'
+dependencies: []
 description: Installs most useful git hooks for Rails and Ruby projects
 email:
 - nikenor11@gmail.com
@@ -68,16 +26,43 @@ files:
 - lib/rails_git_hooks/checks/base.rb
 - lib/rails_git_hooks/checks/commit_msg.rb
 - lib/rails_git_hooks/checks/commit_msg/jira_prefix.rb
+- lib/rails_git_hooks/checks/commit_msg/not_empty.rb
+- lib/rails_git_hooks/checks/post_checkout.rb
+- lib/rails_git_hooks/checks/post_checkout/bundle_install.rb
+- lib/rails_git_hooks/checks/post_checkout/db_migrate.rb
+- lib/rails_git_hooks/checks/post_checkout/npm_install.rb
+- lib/rails_git_hooks/checks/post_checkout/yarn_install.rb
+- lib/rails_git_hooks/checks/post_merge.rb
+- lib/rails_git_hooks/checks/post_merge/bundle_install.rb
+- lib/rails_git_hooks/checks/post_merge/db_migrate.rb
+- lib/rails_git_hooks/checks/post_merge/npm_install.rb
+- lib/rails_git_hooks/checks/post_merge/yarn_install.rb
 - lib/rails_git_hooks/checks/pre_commit.rb
 - lib/rails_git_hooks/checks/pre_commit/debugger_check.rb
 - lib/rails_git_hooks/checks/pre_commit/default_branch.rb
+- lib/rails_git_hooks/checks/pre_commit/erblint.rb
+- lib/rails_git_hooks/checks/pre_commit/eslint.rb
+- lib/rails_git_hooks/checks/pre_commit/go_vet.rb
+- lib/rails_git_hooks/checks/pre_commit/golint.rb
+- lib/rails_git_hooks/checks/pre_commit/haml_lint.rb
+- lib/rails_git_hooks/checks/pre_commit/jslint.rb
 - lib/rails_git_hooks/checks/pre_commit/json_format_check.rb
 - lib/rails_git_hooks/checks/pre_commit/migrations_check.rb
+- lib/rails_git_hooks/checks/pre_commit/php_lint.rb
+- lib/rails_git_hooks/checks/pre_commit/pylint.rb
+- lib/rails_git_hooks/checks/pre_commit/rails_best_practices.rb
 - lib/rails_git_hooks/checks/pre_commit/rubocop.rb
+- lib/rails_git_hooks/checks/pre_commit/scss_lint.rb
 - lib/rails_git_hooks/checks/pre_commit/whitespace_check.rb
 - lib/rails_git_hooks/checks/pre_commit/yaml_format_check.rb
 - lib/rails_git_hooks/checks/pre_push.rb
+- lib/rails_git_hooks/checks/pre_push/run_go_test.rb
+- lib/rails_git_hooks/checks/pre_push/run_pytest.rb
 - lib/rails_git_hooks/checks/pre_push/run_tests.rb
+- lib/rails_git_hooks/checks/shared/bundle_install_check.rb
+- lib/rails_git_hooks/checks/shared/db_migrate_check.rb
+- lib/rails_git_hooks/checks/shared/npm_install_check.rb
+- lib/rails_git_hooks/checks/shared/yarn_install_check.rb
 - lib/rails_git_hooks/cli.rb
 - lib/rails_git_hooks/config/constants.rb
 - lib/rails_git_hooks/config/defaults.yml
@@ -96,6 +81,8 @@ files:
 - lib/rails_git_hooks/runtime/runner.rb
 - lib/rails_git_hooks/version.rb
 - templates/hooks/commit-msg
+- templates/hooks/post-checkout
+- templates/hooks/post-merge
 - templates/hooks/pre-commit
 - templates/hooks/pre-push
 homepage: https://github.com/NikitaNazarov1/rails_git_hooks