rails_git_hooks 0.7.2 → 0.7.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8f679908a499fe6aed2f61cf433d27bf93042bf0dcf1ecfe2e4854f05b0e7b9e
-  data.tar.gz: 481df290f21058504a14d0b6995854861ed7d15099b40a34ec9ccdf6f18fcf3c
+  metadata.gz: 3e545765ea7576f3a3f3f20cb93e3137d24b9286d28d1fbf1a9a2750d8992b42
+  data.tar.gz: 7b2873f1099d4e1dbfdfc180bce56862fc782a21ff9131f9403d86f164cc1773
 SHA512:
-  metadata.gz: 63518e1bcbca51fc17881473b90a0eb510eb2a6e62b26e6471e8890f8fd4e702392b6401f92124d4cd9cc1d398ad0aa03b3c9306a4ce34dc40313d6ab028da52
-  data.tar.gz: c549c30ca8f646f65bdf10a7281e0f2412bb8a796e0665a80e68c05cdac6ed0a39aed58f48011e23b3eba4622db43b6764bae103e434170a601938f4e5f8291e
+  metadata.gz: dbb863f5890def3ef2cee7e3f762b428f81d9aaf3236325956adb302db92145e5b6eb1f0eeb500a2e996825439101df1e2e949ef1345f30670289a6cbd536c3c
+  data.tar.gz: 17b2b1e96f64cd8f1d59494ab40193e7f065bd1f450ea9e55365699f6b58ec3c79a08e49c392de0476cdbb06bba90b977fecbd8bb04d4a6dcd889969cae7d296

data/README.md

@@ -15,21 +15,50 @@ Git hooks for Rails and Ruby projects: sensible defaults out of the box, optiona
 
 ## Included hooks and checks
 
+### When hooks trigger
+
+| Hook | Triggers at |
+|------|-------------|
+| **commit-msg** | After the user finishes editing the commit message, before the commit is created. |
+| **pre-commit** | Before the commit is created (when running `git commit`). |
+| **pre-push** | Before pushing to the remote (when running `git push`). |
+| **post-checkout** | After switching branches or restoring files (e.g. `git checkout`). |
+| **post-merge** | After a merge completes (when running `git merge`). |
+
+### Checks by hook
+
 | Hook         | Check key            | Enabled By Default | Description |
 |-------------|----------------------|---------|-------------|
 | **commit-msg** | `jira-prefix`        | ✅      | Prefix commit messages with Jira-style ticket IDs from the branch name (e.g. `[TICKET-123]`). |
-| **pre-commit** | `default-branch`     | ✅      | Block commits on `master` / `main`; prompt to use a feature branch. |
+| **commit-msg** | `not-empty`          | ✅      | Reject empty commit messages. |
+| **pre-commit** | `default-branch`     | ✅      | Block commits on `master` / `main`, prompt to use a feature branch. |
 | **pre-commit** | `debugger-check`     | ✅      | Warn (or fail) on debugger statements in Ruby, JavaScript/TypeScript, and Python. |
 | **pre-commit** | `yaml-format-check`  | ✅      | Warn on invalid `.yml` / `.yaml` files. |
 | **pre-commit** | `json-format-check`  | ✅      | Warn on invalid `.json` files. |
 | **pre-commit** | `migrations-check`   | ✅      | Warn when migration files are staged but schema/data_schema files are not. |
 | **pre-commit** | `whitespace-check`   | Off     | Fail on trailing whitespace and merge conflict markers. |
 | **pre-commit** | `rubocop-check`      | Off     | Run RuboCop on staged Ruby files (requires `rubocop` in the project). |
+| **pre-commit** | `rails-best-practices` | Off   | Warn on Rails best practices violations (requires `rails_best_practices` gem). |
+| **pre-commit** | `erblint-check`       | Off   | Run `erblint` on staged ERB files (requires `erblint` gem). |
+| **pre-commit** | `eslint-check`        | Off   | Run `eslint` on staged JavaScript files (.js, .jsx, .mjs, .cjs). You can set `command` in config to use an npm script (e.g. `npm run lint`). |
+| **pre-commit** | `golint-check`        | Off   | Run `golint` on staged Go files (requires `golint`). |
+| **pre-commit** | `haml-lint-check`     | Off   | Run `haml-lint` on staged HAML files (requires `haml_lint` gem). |
+| **pre-commit** | `jslint-check`        | Off   | Run `jslint` on staged JavaScript files (.js, .jsx, .mjs, .cjs). |
+| **pre-commit** | `php-lint-check`      | Off   | Run `php -l` (syntax check) on staged PHP files. |
+| **pre-commit** | `pylint-check`        | Off   | Run `pylint` on staged Python files. |
+| **pre-commit** | `scss-lint-check`     | Off   | Run `scss-lint` on staged SCSS files (requires `scss_lint` gem). |
+| **pre-commit** | `go-vet-check`        | Off   | Run `go vet` on staged Go files. |
 | **pre-push**   | `run-tests`          | Off     | Run test suite before push (default: `bundle exec rspec`). Enable in config to install pre-push. |
-| **post-checkout** | `bundle-install`   | ✅      | Run `bundle install` when Gemfile or Gemfile.lock changed after a branch checkout. |
-| **post-checkout** | `db-migrate`       | ✅      | Run `rails db:migrate` when migrations or schema changed after a branch checkout. |
-| **post-merge** | `bundle-install`     | ✅      | Run `bundle install` when Gemfile or Gemfile.lock changed after a merge. |
-| **post-merge** | `db-migrate`        | ✅      | Run `rails db:migrate` when migrations or schema changed after a merge. |
+| **pre-push**   | `run-pytest`         | Off     | Run pytest test suite before push. Enable in config to install pre-push. |
+| **pre-push**   | `run-go-test`        | Off     | Run `go test` suite before push. Enable in config to install pre-push. |
+| **post-checkout** | `bundle-install`   | Off     | Run `bundle install` when Gemfile or Gemfile.lock changed after a branch checkout. |
+| **post-checkout** | `db-migrate`       | Off     | Run `rails db:migrate` when migrations or schema changed after a branch checkout. |
+| **post-checkout** | `npm-install`      | Off     | Run `npm install` when package.json or package-lock.json changed after a branch checkout. |
+| **post-checkout** | `yarn-install`     | Off     | Run `yarn install` when package.json or yarn.lock changed after a branch checkout. |
+| **post-merge** | `bundle-install`     | Off     | Run `bundle install` when Gemfile or Gemfile.lock changed after a merge. |
+| **post-merge** | `db-migrate`        | Off     | Run `rails db:migrate` when migrations or schema changed after a merge. |
+| **post-merge** | `npm-install`       | Off     | Run `npm install` when package.json or package-lock.json changed after a merge. |
+| **post-merge** | `yarn-install`      | Off     | Run `yarn install` when package.json or yarn.lock changed after a merge. |
 
 ## Quick start
 
@@ -90,6 +119,11 @@ PreCommit:
       - "lib/**/*.rb"
     exclude:
       - "db/schema.rb"
+
+  # Run eslint via your npm script (useful if the script has its own config):
+  Eslint:
+    enabled: true
+    command: [npm, run, lint]
 ```
 
 ### Per-check options
@@ -105,13 +139,6 @@ PreCommit:
 | `exclude` | Glob patterns to exclude from `include`. |
 | `command` | Override the command for checks that run external commands. |
 
-## CLI reference
-
-| Command | Description |
-|---------|--------------|
-| `rails_git_hooks install` | Install hooks that have at least one enabled check in the merged config (defaults + .rails_git_hooks.yml + .rails_git_hooks.local.yml). |
-| `rails_git_hooks init` | Create an empty `.rails_git_hooks.yml`. |
-
 ## Contributing
 
 Contributions are welcome. Open an [issue](https://github.com/NikitaNazarov1/rails_git_hooks/issues) for bugs or ideas, or submit a pull request.

data/lib/rails_git_hooks/checks/commit_msg/not_empty.rb

@@ -0,0 +1,25 @@
+# frozen_string_literal: true
+
+module GitHooks
+  module Checks
+    module CommitMsg
+      class NotEmpty < Base
+        check_definition key: 'not-empty',
+                         hook: :commit_msg,
+                         description: 'Reject empty commit messages'
+
+        def run
+          message_file = argv.first
+          return CheckResult.pass unless message_file && File.file?(message_file)
+
+          message = File.read(message_file)
+          # Strip comment lines (git uses # in the message template) and blank lines
+          content = message.lines.reject { |line| line.strip.start_with?('#') || line.strip.empty? }.join.strip
+          return CheckResult.pass unless content.empty?
+
+          CheckResult.fail(messages: ['Commit message must not be empty.'])
+        end
+      end
+    end
+  end
+end

data/lib/rails_git_hooks/checks/commit_msg.rb

@@ -8,3 +8,4 @@ module GitHooks
 end
 
 require_relative 'commit_msg/jira_prefix'
+require_relative 'commit_msg/not_empty'

data/lib/rails_git_hooks/checks/post_checkout/npm_install.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+module GitHooks
+  module Checks
+    module PostCheckout
+      class NpmInstall < Base
+        include NpmInstallCheck
+
+        check_definition key: 'npm-install',
+                         hook: :post_checkout,
+                         description: 'Run npm install when package.json or package-lock.json changed (branch checkout)',
+                         **NpmInstallCheck::DEFINITION_OPTIONS
+      end
+    end
+  end
+end

data/lib/rails_git_hooks/checks/post_checkout/yarn_install.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+module GitHooks
+  module Checks
+    module PostCheckout
+      class YarnInstall < Base
+        include YarnInstallCheck
+
+        check_definition key: 'yarn-install',
+                         hook: :post_checkout,
+                         description: 'Run yarn install when package.json or yarn.lock changed (branch checkout)',
+                         **YarnInstallCheck::DEFINITION_OPTIONS
+      end
+    end
+  end
+end

data/lib/rails_git_hooks/checks/post_checkout.rb

@@ -9,3 +9,5 @@ end
 
 require_relative 'post_checkout/bundle_install'
 require_relative 'post_checkout/db_migrate'
+require_relative 'post_checkout/npm_install'
+require_relative 'post_checkout/yarn_install'

data/lib/rails_git_hooks/checks/post_merge/npm_install.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+module GitHooks
+  module Checks
+    module PostMerge
+      class NpmInstall < Base
+        include NpmInstallCheck
+
+        check_definition key: 'npm-install',
+                         hook: :post_merge,
+                         description: 'Run npm install when package.json or package-lock.json changed (after merge)',
+                         **NpmInstallCheck::DEFINITION_OPTIONS
+      end
+    end
+  end
+end

data/lib/rails_git_hooks/checks/post_merge/yarn_install.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+module GitHooks
+  module Checks
+    module PostMerge
+      class YarnInstall < Base
+        include YarnInstallCheck
+
+        check_definition key: 'yarn-install',
+                         hook: :post_merge,
+                         description: 'Run yarn install when package.json or yarn.lock changed (after merge)',
+                         **YarnInstallCheck::DEFINITION_OPTIONS
+      end
+    end
+  end
+end

data/lib/rails_git_hooks/checks/post_merge.rb

@@ -9,3 +9,5 @@ end
 
 require_relative 'post_merge/bundle_install'
 require_relative 'post_merge/db_migrate'
+require_relative 'post_merge/npm_install'
+require_relative 'post_merge/yarn_install'

data/lib/rails_git_hooks/checks/pre_commit/erblint.rb

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+module GitHooks
+  module Checks
+    module PreCommit
+      class ErbLint < Base
+        check_definition key: 'erblint-check',
+                         hook: :pre_commit,
+                         description: 'Run erblint on staged ERB files',
+                         file_based: true,
+                         enabled: false,
+                         quiet: true,
+                         dependencies: { 'executables' => ['bundle'], 'libraries' => ['erblint'] },
+                         command: %w[bundle exec erblint],
+                         install_hint: 'Add `gem "erblint"` to your Gemfile and run bundle install'
+
+        def run
+          erb_files = applicable_files.select { |path| path.end_with?('.erb') && File.file?(path) }
+          return CheckResult.pass if erb_files.empty?
+
+          output, status = capture(*Array(config['command']), *erb_files)
+          return CheckResult.pass if status.success?
+
+          messages = output.split("\n").map(&:rstrip).reject(&:empty?)
+          CheckResult.fail(messages: messages)
+        end
+      end
+    end
+  end
+end

data/lib/rails_git_hooks/checks/pre_commit/eslint.rb

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+module GitHooks
+  module Checks
+    module PreCommit
+      class Eslint < Base
+        JS_EXTENSIONS = %w[.js .jsx .mjs .cjs].freeze
+
+        check_definition key: 'eslint-check',
+                         hook: :pre_commit,
+                         description: 'Run eslint on staged JavaScript files',
+                         file_based: true,
+                         enabled: false,
+                         quiet: true,
+                         dependencies: { 'executables' => ['eslint'] },
+                         command: %w[eslint],
+                         install_hint: 'Install eslint (e.g. npm install -D eslint) or override command in config'
+
+        def run
+          js_files = applicable_files.select do |path|
+            File.file?(path) && JS_EXTENSIONS.include?(File.extname(path))
+          end
+          return CheckResult.pass if js_files.empty?
+
+          output, status = capture(*Array(config['command']), *js_files)
+          return CheckResult.pass if status.success?
+
+          messages = output.split("\n").map(&:rstrip).reject(&:empty?)
+          CheckResult.fail(messages: messages)
+        end
+      end
+    end
+  end
+end

data/lib/rails_git_hooks/checks/pre_commit/go_vet.rb

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+module GitHooks
+  module Checks
+    module PreCommit
+      class GoVet < Base
+        check_definition key: 'go-vet-check',
+                         hook: :pre_commit,
+                         description: 'Run go vet on staged Go files',
+                         file_based: true,
+                         enabled: false,
+                         quiet: true,
+                         dependencies: { 'executables' => ['go'] },
+                         command: %w[go vet],
+                         install_hint: 'Ensure Go toolchain is installed and on PATH'
+
+        def run
+          go_files = applicable_files.select { |path| File.file?(path) && File.extname(path) == '.go' }
+          return CheckResult.pass if go_files.empty?
+
+          output, status = capture(*Array(config['command']), *go_files)
+          return CheckResult.pass if status.success?
+
+          messages = output.split("\n").map(&:rstrip).reject(&:empty?)
+          CheckResult.fail(messages: messages)
+        end
+      end
+    end
+  end
+end

data/lib/rails_git_hooks/checks/pre_commit/golint.rb

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+module GitHooks
+  module Checks
+    module PreCommit
+      class Golint < Base
+        check_definition key: 'golint-check',
+                         hook: :pre_commit,
+                         description: 'Run golint on staged Go files',
+                         file_based: true,
+                         enabled: false,
+                         quiet: true,
+                         dependencies: { 'executables' => ['golint'] },
+                         command: %w[golint],
+                         install_hint: 'Install golint or override command in config'
+
+        def run
+          go_files = applicable_files.select { |path| File.file?(path) && File.extname(path) == '.go' }
+          return CheckResult.pass if go_files.empty?
+
+          output, status = capture(*Array(config['command']), *go_files)
+          return CheckResult.pass if status.success?
+
+          messages = output.split("\n").map(&:rstrip).reject(&:empty?)
+          CheckResult.fail(messages: messages)
+        end
+      end
+    end
+  end
+end

data/lib/rails_git_hooks/checks/pre_commit/haml_lint.rb

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+module GitHooks
+  module Checks
+    module PreCommit
+      class HamlLint < Base
+        check_definition key: 'haml-lint-check',
+                         hook: :pre_commit,
+                         description: 'Run haml-lint on staged HAML files',
+                         file_based: true,
+                         enabled: false,
+                         quiet: true,
+                         dependencies: { 'executables' => ['bundle'], 'libraries' => ['haml_lint'] },
+                         command: %w[bundle exec haml-lint],
+                         install_hint: 'Add `gem "haml_lint"` to your Gemfile and run bundle install'
+
+        def run
+          haml_files = applicable_files.select { |path| path.end_with?('.haml') && File.file?(path) }
+          return CheckResult.pass if haml_files.empty?
+
+          output, status = capture(*Array(config['command']), *haml_files)
+          return CheckResult.pass if status.success?
+
+          messages = output.split("\n").map(&:rstrip).reject(&:empty?)
+          CheckResult.fail(messages: messages)
+        end
+      end
+    end
+  end
+end

data/lib/rails_git_hooks/checks/pre_commit/jslint.rb

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+module GitHooks
+  module Checks
+    module PreCommit
+      class Jslint < Base
+        JS_EXTENSIONS = %w[.js .jsx .mjs .cjs].freeze
+
+        check_definition key: 'jslint-check',
+                         hook: :pre_commit,
+                         description: 'Run jslint on staged JavaScript files',
+                         file_based: true,
+                         enabled: false,
+                         quiet: true,
+                         dependencies: { 'executables' => ['jslint'] },
+                         command: %w[jslint],
+                         install_hint: 'Install jslint or override command in config'
+
+        def run
+          js_files = applicable_files.select do |path|
+            File.file?(path) && JS_EXTENSIONS.include?(File.extname(path))
+          end
+          return CheckResult.pass if js_files.empty?
+
+          output, status = capture(*Array(config['command']), *js_files)
+          return CheckResult.pass if status.success?
+
+          messages = output.split("\n").map(&:rstrip).reject(&:empty?)
+          CheckResult.fail(messages: messages)
+        end
+      end
+    end
+  end
+end

data/lib/rails_git_hooks/checks/pre_commit/php_lint.rb

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+module GitHooks
+  module Checks
+    module PreCommit
+      class PhpLint < Base
+        check_definition key: 'php-lint-check',
+                         hook: :pre_commit,
+                         description: 'Run php -l (syntax check) on staged PHP files',
+                         file_based: true,
+                         enabled: false,
+                         quiet: true,
+                         dependencies: { 'executables' => ['php'] },
+                         command: %w[php -l],
+                         install_hint: 'Ensure PHP is installed and on PATH'
+
+        def run
+          php_files = applicable_files.select { |path| path.end_with?('.php') && File.file?(path) }
+          return CheckResult.pass if php_files.empty?
+
+          output, status = capture(*Array(config['command']), *php_files)
+          return CheckResult.pass if status.success?
+
+          messages = output.split("\n").map(&:rstrip).reject(&:empty?)
+          CheckResult.fail(messages: messages)
+        end
+      end
+    end
+  end
+end

data/lib/rails_git_hooks/checks/pre_commit/pylint.rb

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+module GitHooks
+  module Checks
+    module PreCommit
+      class Pylint < Base
+        check_definition key: 'pylint-check',
+                         hook: :pre_commit,
+                         description: 'Run pylint on staged Python files',
+                         file_based: true,
+                         enabled: false,
+                         quiet: true,
+                         dependencies: { 'executables' => ['pylint'] },
+                         command: %w[pylint],
+                         install_hint: 'Install pylint (e.g. pip install pylint) or override command in config'
+
+        def run
+          py_files = applicable_files.select { |path| path.end_with?('.py') && File.file?(path) }
+          return CheckResult.pass if py_files.empty?
+
+          output, status = capture(*Array(config['command']), *py_files)
+          return CheckResult.pass if status.success?
+
+          messages = output.split("\n").map(&:rstrip).reject(&:empty?)
+          CheckResult.fail(messages: messages)
+        end
+      end
+    end
+  end
+end

data/lib/rails_git_hooks/checks/pre_commit/rails_best_practices.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+module GitHooks
+  module Checks
+    module PreCommit
+      class RailsBestPractices < Base
+        check_definition key: 'rails-best-practices',
+                         hook: :pre_commit,
+                         description: 'Warn on Rails best practices violations',
+                         file_based: true,
+                         enabled: false,
+                         on_fail: :warn,
+                         quiet: true,
+                         dependencies: { 'executables' => ['bundle'], 'libraries' => ['rails_best_practices'] },
+                         command: %w[bundle exec rails_best_practices],
+                         install_hint: 'Add `gem "rails_best_practices"` to your Gemfile and run bundle install'
+
+        def run
+          ruby_files = applicable_files.select { |path| File.extname(path) == '.rb' && File.file?(path) }
+          return CheckResult.pass if ruby_files.empty?
+
+          output, status = capture(*Array(config['command']), *ruby_files)
+          return CheckResult.pass if status.success?
+
+          messages = output.split("\n").map(&:rstrip).reject(&:empty?)
+          CheckResult.fail(messages: messages)
+        end
+      end
+    end
+  end
+end

data/lib/rails_git_hooks/checks/pre_commit/scss_lint.rb

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+module GitHooks
+  module Checks
+    module PreCommit
+      class ScssLint < Base
+        check_definition key: 'scss-lint-check',
+                         hook: :pre_commit,
+                         description: 'Run scss-lint on staged SCSS files',
+                         file_based: true,
+                         enabled: false,
+                         quiet: true,
+                         dependencies: { 'executables' => ['bundle'], 'libraries' => ['scss_lint'] },
+                         command: %w[bundle exec scss-lint],
+                         install_hint: 'Add `gem "scss_lint"` to your Gemfile and run bundle install'
+
+        def run
+          scss_files = applicable_files.select { |path| path.end_with?('.scss') && File.file?(path) }
+          return CheckResult.pass if scss_files.empty?
+
+          output, status = capture(*Array(config['command']), *scss_files)
+          return CheckResult.pass if status.success?
+
+          messages = output.split("\n").map(&:rstrip).reject(&:empty?)
+          CheckResult.fail(messages: messages)
+        end
+      end
+    end
+  end
+end

data/lib/rails_git_hooks/checks/pre_commit.rb

@@ -14,3 +14,13 @@ require_relative 'pre_commit/json_format_check'
 require_relative 'pre_commit/migrations_check'
 require_relative 'pre_commit/whitespace_check'
 require_relative 'pre_commit/rubocop'
+require_relative 'pre_commit/rails_best_practices'
+require_relative 'pre_commit/erblint'
+require_relative 'pre_commit/eslint'
+require_relative 'pre_commit/golint'
+require_relative 'pre_commit/haml_lint'
+require_relative 'pre_commit/jslint'
+require_relative 'pre_commit/php_lint'
+require_relative 'pre_commit/pylint'
+require_relative 'pre_commit/scss_lint'
+require_relative 'pre_commit/go_vet'

data/lib/rails_git_hooks/checks/pre_push/run_go_test.rb

@@ -0,0 +1,24 @@
+# frozen_string_literal: true
+
+module GitHooks
+  module Checks
+    module PrePush
+      class RunGoTest < Base
+        check_definition key: 'run-go-test',
+                         hook: :pre_push,
+                         description: 'Run go test suite before push',
+                         dependencies: { 'executables' => ['go'] },
+                         command: %w[go test ./...],
+                         install_hint: 'Ensure Go toolchain is installed and `go test ./...` runs successfully'
+
+        def run
+          output, status = capture(*Array(config['command']))
+          return CheckResult.pass if status.success?
+
+          messages = output.split("\n").map(&:rstrip).reject(&:empty?)
+          CheckResult.fail(messages: messages)
+        end
+      end
+    end
+  end
+end

data/lib/rails_git_hooks/checks/pre_push/run_pytest.rb

@@ -0,0 +1,24 @@
+# frozen_string_literal: true
+
+module GitHooks
+  module Checks
+    module PrePush
+      class RunPytest < Base
+        check_definition key: 'run-pytest',
+                         hook: :pre_push,
+                         description: 'Run pytest test suite before push',
+                         dependencies: { 'executables' => ['pytest'] },
+                         command: %w[pytest],
+                         install_hint: 'Install pytest (e.g. pip install pytest) and ensure pytest runs successfully'
+
+        def run
+          output, status = capture(*Array(config['command']))
+          return CheckResult.pass if status.success?
+
+          messages = output.split("\n").map(&:rstrip).reject(&:empty?)
+          CheckResult.fail(messages: messages)
+        end
+      end
+    end
+  end
+end

data/lib/rails_git_hooks/checks/pre_push.rb

@@ -8,3 +8,5 @@ module GitHooks
 end
 
 require_relative 'pre_push/run_tests'
+require_relative 'pre_push/run_pytest'
+require_relative 'pre_push/run_go_test'

data/lib/rails_git_hooks/checks/shared/npm_install_check.rb

@@ -0,0 +1,28 @@
+# frozen_string_literal: true
+
+module GitHooks
+  module Checks
+    # Shared logic and options for "run npm install when package.json/package-lock.json changed"
+    # (post-checkout and post-merge). Each hook has its own class that passes hook + description.
+    module NpmInstallCheck
+      DEFINITION_OPTIONS = {
+        file_based: true,
+        enabled: false,
+        include: %w[package.json package-lock.json],
+        dependencies: { executables: ['npm'] },
+        command: %w[npm install],
+        install_hint: 'Ensure npm is available'
+      }.freeze
+
+      def run
+        return CheckResult.pass if applicable_files.empty?
+
+        output, status = capture(*Array(config['command']))
+        return CheckResult.pass if status.success?
+
+        messages = output.split("\n").map(&:rstrip).reject(&:empty?)
+        CheckResult.fail(messages: messages)
+      end
+    end
+  end
+end

data/lib/rails_git_hooks/checks/shared/yarn_install_check.rb

@@ -0,0 +1,28 @@
+# frozen_string_literal: true
+
+module GitHooks
+  module Checks
+    # Shared logic and options for "run yarn install when package.json/yarn.lock changed"
+    # (post-checkout and post-merge). Each hook has its own class that passes hook + description.
+    module YarnInstallCheck
+      DEFINITION_OPTIONS = {
+        file_based: true,
+        enabled: false,
+        include: %w[package.json yarn.lock],
+        dependencies: { executables: ['yarn'] },
+        command: %w[yarn install],
+        install_hint: 'Ensure yarn is available'
+      }.freeze
+
+      def run
+        return CheckResult.pass if applicable_files.empty?
+
+        output, status = capture(*Array(config['command']))
+        return CheckResult.pass if status.success?
+
+        messages = output.split("\n").map(&:rstrip).reject(&:empty?)
+        CheckResult.fail(messages: messages)
+      end
+    end
+  end
+end

data/lib/rails_git_hooks/checks.rb

@@ -6,5 +6,7 @@ require_relative 'checks/commit_msg'
 require_relative 'checks/pre_push'
 require_relative 'checks/shared/bundle_install_check'
 require_relative 'checks/shared/db_migrate_check'
+require_relative 'checks/shared/npm_install_check'
+require_relative 'checks/shared/yarn_install_check'
 require_relative 'checks/post_checkout'
 require_relative 'checks/post_merge'

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,6 +264,19 @@ 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: false
@@ -127,9 +294,39 @@ PrePush:
     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: true
+    enabled: false
     quiet: false
     on_fail: fail
     on_warn: warn
@@ -144,7 +341,7 @@ PostCheckout:
     install_hint: Ensure bundle is available
 
   DbMigrate:
-    enabled: true
+    enabled: false
     quiet: false
     on_fail: fail
     on_warn: warn
@@ -158,9 +355,39 @@ PostCheckout:
     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: true
+    enabled: false
     quiet: false
     on_fail: fail
     on_warn: warn
@@ -175,7 +402,7 @@ PostMerge:
     install_hint: Ensure bundle is available
 
   DbMigrate:
-    enabled: true
+    enabled: false
     quiet: false
     on_fail: fail
     on_warn: warn
@@ -188,3 +415,33 @@ PostMerge:
     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/runtime/check_registry.rb

@@ -10,12 +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::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::DbMigrate,
+      Checks::PostMerge::NpmInstall,
+      Checks::PostMerge::YarnInstall
     ].freeze
 
     def self.all

data/lib/rails_git_hooks/runtime/runner.rb

@@ -53,7 +53,7 @@ module GitHooks
       @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]) : []
+                            @argv[2] == '1' ? @repo.changed_files(@argv[0], @argv[1]) : []
                           when :post_merge
                             @repo.changed_files('ORIG_HEAD', 'HEAD')
                           else []

data/lib/rails_git_hooks/version.rb

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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rails_git_hooks
 version: !ruby/object:Gem::Version
-  version: 0.7.2
+  version: 0.7.3
 platform: ruby
 authors:
 - Nikita Nazarov
@@ -26,24 +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