rails_git_hooks 0.7.0 → 0.7.2

data/templates/hooks/pre-commit

@@ -1,27 +1,13 @@
 #!/usr/bin/env ruby
 # frozen_string_literal: true
 
-#
-# Blocks commits on default branch (master/main). Warns on debugger statements (Ruby/JS/TS/Python).
-# Optionally runs RuboCop and whitespace/conflict checks.
-# Logic is in templates/shared/pre_commit/ for easier maintenance and adding new checks.
-
-# Bootstrap: resolve .git dir and skip if hook is disabled
-repo_root = Dir.pwd
-git_dir = File.join(repo_root, '.git')
-git_dir = File.expand_path(File.read(git_dir).strip.sub(/\Agitdir: \s*/, ''), repo_root) if File.file?(git_dir)
-disabled_file = File.join(git_dir, 'rails_git_hooks_disabled')
-if File.exist?(disabled_file)
-  disabled = File.read(disabled_file).split("\n").map(&:strip)
-  exit 0 if disabled.include?('*') || disabled.include?('pre-commit')
+hooks_dir = File.dirname(File.expand_path(__FILE__))
+begin
+  require 'bundler/setup'
+rescue LoadError, StandardError
+  nil
 end
 
-module RailsGitHooks
-end
-RailsGitHooks.const_set(:GIT_DIR, git_dir.freeze)
-hooks_dir = File.dirname(File.expand_path(__FILE__))
+require File.join(hooks_dir, 'rails_git_hooks', 'runtime')
 
-load File.join(hooks_dir, 'pre_commit', 'whitespace_check.rb')
-load File.join(hooks_dir, 'pre_commit', 'default_branch.rb')
-load File.join(hooks_dir, 'pre_commit', 'debugger_check.rb')
-load File.join(hooks_dir, 'pre_commit', 'rubocop_check.rb')
+exit GitHooks::Runtime.execute(:pre_commit, argv: ARGV, stdin: $stdin.read)

data/templates/hooks/pre-push

@@ -1,23 +1,13 @@
 #!/usr/bin/env ruby
 # frozen_string_literal: true
 
-#
-# Runs the full test suite before push. Push is aborted if tests fail.
-# Logic in templates/shared/pre_push/.
-
-# Bootstrap: resolve .git dir and skip if hook is disabled
-repo_root = Dir.pwd
-git_dir = File.join(repo_root, '.git')
-git_dir = File.expand_path(File.read(git_dir).strip.sub(/\Agitdir: \s*/, ''), repo_root) if File.file?(git_dir)
-disabled_file = File.join(git_dir, 'rails_git_hooks_disabled')
-if File.exist?(disabled_file)
-  disabled = File.read(disabled_file).split("\n").map(&:strip)
-  exit 0 if disabled.include?('*') || disabled.include?('pre-push')
+hooks_dir = File.dirname(File.expand_path(__FILE__))
+begin
+  require 'bundler/setup'
+rescue LoadError, StandardError
+  nil
 end
 
-module RailsGitHooks
-end
-RailsGitHooks.const_set(:GIT_DIR, git_dir.freeze)
-hooks_dir = File.dirname(File.expand_path(__FILE__))
+require File.join(hooks_dir, 'rails_git_hooks', 'runtime')
 
-load File.join(hooks_dir, 'pre_push', 'run_tests.rb')
+exit GitHooks::Runtime.execute(:pre_push, 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.0
+  version: 0.7.2
 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
@@ -64,19 +22,50 @@ files:
 - README.md
 - bin/rails_git_hooks
 - lib/rails_git_hooks.rb
+- lib/rails_git_hooks/checks.rb
+- 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/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_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/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/json_format_check.rb
+- lib/rails_git_hooks/checks/pre_commit/migrations_check.rb
+- lib/rails_git_hooks/checks/pre_commit/rubocop.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_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/cli.rb
-- lib/rails_git_hooks/constants.rb
-- lib/rails_git_hooks/installer.rb
+- lib/rails_git_hooks/config/constants.rb
+- lib/rails_git_hooks/config/defaults.yml
+- lib/rails_git_hooks/config/defaults_loader.rb
+- lib/rails_git_hooks/core/check_definition.rb
+- lib/rails_git_hooks/core/check_result.rb
+- lib/rails_git_hooks/core/error.rb
+- lib/rails_git_hooks/install/installer.rb
+- lib/rails_git_hooks/runtime.rb
+- lib/rails_git_hooks/runtime/check_registry.rb
+- lib/rails_git_hooks/runtime/dependency_checker.rb
+- lib/rails_git_hooks/runtime/file_matcher.rb
+- lib/rails_git_hooks/runtime/override_config.rb
+- lib/rails_git_hooks/runtime/policy_resolver.rb
+- lib/rails_git_hooks/runtime/repository.rb
+- 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
-- templates/shared/commit_msg/jira_prefix.rb
-- templates/shared/pre_commit/debugger_check.rb
-- templates/shared/pre_commit/default_branch.rb
-- templates/shared/pre_commit/rubocop_check.rb
-- templates/shared/pre_commit/whitespace_check.rb
-- templates/shared/pre_push/run_tests.rb
 homepage: https://github.com/NikitaNazarov1/rails_git_hooks
 licenses:
 - MIT

data/lib/rails_git_hooks/constants.rb

@@ -1,21 +0,0 @@
-# frozen_string_literal: true
-
-module GitHooks
-  # Paths and default config (single source of truth).
-  module Constants
-    GEM_ROOT = File.expand_path('../..', __dir__)
-    HOOKS_DIR = File.expand_path('templates/hooks', GEM_ROOT).freeze
-    SHARED_DIR = File.expand_path('templates/shared', GEM_ROOT).freeze
-
-    DISABLED_FILE = 'rails_git_hooks_disabled'
-
-    # Default hooks when install is run with no arguments.
-    DEFAULT_HOOKS = %w[commit-msg pre-commit].freeze
-
-    # Pre-commit feature flag file names (keys = CLI tokens).
-    FEATURE_FLAG_FILES = {
-      'whitespace-check' => 'rails_git_hooks_whitespace_check',
-      'rubocop-check' => 'rails_git_hooks_rubocop'
-    }.freeze
-  end
-end

data/lib/rails_git_hooks/installer.rb

@@ -1,156 +0,0 @@
-# frozen_string_literal: true
-
-require 'fileutils'
-
-module GitHooks
-  class Installer
-    def initialize(git_dir: nil)
-      @git_dir = git_dir || find_git_dir
-    end
-
-    def install(*hook_names)
-      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_shared_files(target_dir)
-
-      hooks = hook_names.empty? ? Constants::DEFAULT_HOOKS : hook_names
-      installed = []
-
-      hooks.each do |name|
-        src = File.join(Constants::HOOKS_DIR, name)
-        next unless File.file?(src)
-
-        dest = File.join(target_dir, name)
-        write_hook(dest, read_template(name))
-        make_executable(dest)
-        installed << name
-      end
-
-      installed
-    end
-
-    def self.available_hook_names
-      Dir.children(Constants::HOOKS_DIR).select { |f| File.file?(File.join(Constants::HOOKS_DIR, f)) }
-    end
-
-    def available_hooks
-      self.class.available_hook_names
-    end
-
-    def disabled_file_path
-      File.join(@git_dir, Constants::DISABLED_FILE)
-    end
-
-    def disabled_hooks
-      path = disabled_file_path
-      return [] unless File.exist?(path)
-
-      File.read(path).split("\n").map(&:strip).reject(&:empty?)
-    end
-
-    def disable(*hook_names)
-      path = disabled_file_path
-      current = (disabled_hooks + hook_names).uniq
-      File.write(path, "#{current.join("\n")}\n")
-      hook_names
-    end
-
-    def enable(*hook_names)
-      path = disabled_file_path
-      return [] unless File.exist?(path)
-
-      current = disabled_hooks - hook_names
-      if current.empty?
-        File.delete(path)
-      else
-        File.write(path, "#{current.join("\n")}\n")
-      end
-      hook_names
-    end
-
-    def enable_whitespace_check
-      enable_feature_flag('whitespace-check')
-    end
-
-    def disable_whitespace_check
-      disable_feature_flag('whitespace-check')
-    end
-
-    def whitespace_check_enabled?
-      feature_flag_enabled?('whitespace-check')
-    end
-
-    def enable_rubocop_check
-      enable_feature_flag('rubocop-check')
-    end
-
-    def disable_rubocop_check
-      disable_feature_flag('rubocop-check')
-    end
-
-    def rubocop_check_enabled?
-      feature_flag_enabled?('rubocop-check')
-    end
-
-    private
-
-    def enable_feature_flag(name)
-      file = Constants::FEATURE_FLAG_FILES[name]
-      return unless file
-
-      File.write(File.join(@git_dir, file), '')
-    end
-
-    def disable_feature_flag(name)
-      file = Constants::FEATURE_FLAG_FILES[name]
-      return unless file
-
-      FileUtils.rm_f(File.join(@git_dir, file))
-    end
-
-    def feature_flag_enabled?(name)
-      file = Constants::FEATURE_FLAG_FILES[name]
-      file && File.exist?(File.join(@git_dir, file))
-    end
-
-    def find_git_dir
-      dir = Dir.pwd
-      loop do
-        git = File.join(dir, '.git')
-        return git if File.directory?(git)
-
-        parent = File.dirname(dir)
-        raise GitHooks::Error, 'Not inside a git repository' if parent == dir
-
-        dir = parent
-      end
-    end
-
-    def copy_shared_files(target_dir)
-      return unless Dir.exist?(Constants::SHARED_DIR)
-
-      Dir.glob(File.join(Constants::SHARED_DIR, '**', '*')).each do |src|
-        next unless File.file?(src)
-
-        rel = src.sub(%r{\A#{Regexp.escape(Constants::SHARED_DIR)}/}, '')
-        dest = File.join(target_dir, rel)
-        FileUtils.mkdir_p(File.dirname(dest))
-        File.write(dest, File.read(src))
-      end
-    end
-
-    def read_template(name)
-      path = File.join(Constants::HOOKS_DIR, name)
-      File.read(path)
-    end
-
-    def write_hook(path, content)
-      File.write(path, content)
-    end
-
-    def make_executable(path)
-      File.chmod(0o755, path)
-    end
-  end
-end

data/templates/shared/commit_msg/jira_prefix.rb

@@ -1,20 +0,0 @@
-# frozen_string_literal: true
-
-# Prepends [TICKET] to commit message when branch name contains a Jira-style ticket.
-# No config needed; works with any project key (2–5 letters + digits).
-
-commit_message_file = ARGV[0]
-branch_name = `git branch --no-color 2> /dev/null`[/^\* (.+)/, 1].to_s
-original_commit_message = File.read(commit_message_file).strip
-
-branch_ticket_pattern = /([A-Z]{2,5}-\d+)/i
-skip_if_already_prefixed = /\A\[[A-Z]{2,5}-\d+\]/i
-
-if (m = branch_name.match(branch_ticket_pattern))
-  jira_ticket = m.captures.first
-
-  unless original_commit_message.match?(skip_if_already_prefixed) || original_commit_message.include?(jira_ticket)
-    message = "[#{jira_ticket}] #{original_commit_message}"
-    File.open(commit_message_file, 'w') { |f| f.write message }
-  end
-end

data/templates/shared/pre_commit/debugger_check.rb

@@ -1,48 +0,0 @@
-# frozen_string_literal: true
-
-# Warns when staged files contain debugger statements (Ruby, JavaScript/TypeScript, Python).
-# Does not block the commit. Expects RailsGitHooks::GIT_DIR to be set by the loader.
-
-# [ extension => [ [ regex, label ], ... ] ]
-DEBUGGER_PATTERNS = {
-  '.rb' => [
-    [/\bbinding\.pry\b/, 'binding.pry'],
-    [/\bbinding\.irb\b/, 'binding.irb'],
-    [/\bdebugger\b/, 'debugger'],
-    [/\bbyebug\b/, 'byebug']
-  ],
-  '.js' => [[/\bdebugger\s*;?/, 'debugger']],
-  '.jsx' => [[/\bdebugger\s*;?/, 'debugger']],
-  '.ts' => [[/\bdebugger\s*;?/, 'debugger']],
-  '.tsx' => [[/\bdebugger\s*;?/, 'debugger']],
-  '.mjs' => [[/\bdebugger\s*;?/, 'debugger']],
-  '.cjs' => [[/\bdebugger\s*;?/, 'debugger']],
-  '.py' => [
-    [/\bbreakpoint\s*\(\s*\)/, 'breakpoint()'],
-    [/\bpdb\.set_trace\s*\(\s*\)/, 'pdb.set_trace()'],
-    [/\bipdb\.set_trace\s*\(\s*\)/, 'ipdb.set_trace()']
-  ]
-}.freeze
-
-staged = `git diff --cached --name-only`.split("\n").map(&:strip).reject(&:empty?)
-warnings = []
-
-staged.each do |path|
-  next unless File.file?(path)
-
-  ext = File.extname(path)
-  patterns = DEBUGGER_PATTERNS[ext]
-  next unless patterns
-
-  File.read(path).lines.each_with_index do |line, i|
-    patterns.each do |regex, label|
-      warnings << "#{path}:#{i + 1}: #{label}" if line.match?(regex)
-    end
-  end
-end
-
-unless warnings.empty?
-  warn 'Warning (debugger check):'
-  warnings.uniq.each { |e| warn "  #{e}" }
-end
-# Does not exit 1 — commit is never blocked by this check

data/templates/shared/pre_commit/default_branch.rb

@@ -1,9 +0,0 @@
-# frozen_string_literal: true
-
-# Prevent commits on default branch (master/main).
-
-branch = `git rev-parse --abbrev-ref HEAD`.strip
-if %w[master main].include?(branch)
-  warn "Commits on '#{branch}' are not allowed. Create a feature branch."
-  exit 1
-end

data/templates/shared/pre_commit/rubocop_check.rb

@@ -1,24 +0,0 @@
-# frozen_string_literal: true
-
-# RuboCop on staged .rb files (off by default; enable with rails_git_hooks enable rubocop-check).
-# Expects RailsGitHooks::GIT_DIR to be set by the loader.
-
-git_dir = RailsGitHooks::GIT_DIR
-rubocop_check_file = File.join(git_dir, 'rails_git_hooks_rubocop')
-if File.exist?(rubocop_check_file)
-  require 'english'
-  require 'rubocop'
-
-  ADDED_OR_MODIFIED = /A|AM|^M/.freeze
-
-  changed_files = `git status --porcelain`.split(/\n/)
-                                          .select { |file_name_with_status| file_name_with_status =~ ADDED_OR_MODIFIED }
-                                          .map { |file_name_with_status| file_name_with_status.split(' ')[1] }
-                                          .select { |file_name| File.extname(file_name) == '.rb' }
-                                          .join(' ')
-
-  unless changed_files.empty?
-    system("rubocop #{changed_files}")
-    exit $CHILD_STATUS.to_s[-1].to_i
-  end
-end

data/templates/shared/pre_commit/whitespace_check.rb

@@ -1,25 +0,0 @@
-# frozen_string_literal: true
-
-# Trailing whitespace / conflict markers (off by default; enable with rails_git_hooks enable whitespace-check).
-# Expects RailsGitHooks::GIT_DIR to be set by the loader.
-
-git_dir = RailsGitHooks::GIT_DIR
-whitespace_check_file = File.join(git_dir, 'rails_git_hooks_whitespace_check')
-if File.exist?(whitespace_check_file)
-  staged = `git diff --cached --name-only`.split("\n").map(&:strip).reject(&:empty?)
-  errors = []
-  staged.each do |path|
-    next unless File.file?(path)
-
-    File.read(path).lines.each_with_index do |line, i|
-      errors << "#{path}:#{i + 1}: trailing whitespace" if line.match?(/[ \t]\z/)
-      stripped = line.strip
-      errors << "#{path}:#{i + 1}: conflict marker" if stripped.start_with?('<<<<<<<', '=======', '>>>>>>>')
-    end
-  end
-  unless errors.empty?
-    warn 'Commit rejected (whitespace/conflict check):'
-    errors.uniq.each { |e| warn "  #{e}" }
-    exit 1
-  end
-end

data/templates/shared/pre_push/run_tests.rb

@@ -1,9 +0,0 @@
-# frozen_string_literal: true
-
-# Runs the full test suite before push; aborts push if tests fail.
-# Uses bundle exec rspec. For Minitest, edit to use bundle exec rake test.
-
-require 'english'
-
-system('bundle exec rspec')
-exit $CHILD_STATUS.to_s[-1].to_i