good_migrations 0.0.2 → 0.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 05eb20b7481ddae6720347b0c84275997ff53e72
-  data.tar.gz: b7c85f764f8f2c22a3988517f2c272cdb59d0016
+SHA256:
+  metadata.gz: 957989ed1010171743f14af193d62b5bfbb820e59ad79b42ca16d6b8b51ca948
+  data.tar.gz: 89dc256beca58029d4de2c142834a3069b7b165959055bc4f7ab1721ffea6c82
 SHA512:
-  metadata.gz: b79f08edebb75e97c2358064c31d8c5d2de7e4cb483b5e4f7e61a4fe04b6550156c271995249af7327a66b1f5767269830ec0fe01b7144a4fcaa6a257d244689
-  data.tar.gz: 285bfb5be43c16860c0e6bff42c5e024d24199d4a9a84e9de1f946d04a979bbac87156c0603124c87aac80aecef81f0cd47690daf699bf27c73f2770ad0465fc
+  metadata.gz: 5a2e09ff0b2bf6473ececfa8ca0905eae9a974cd81ff5ea76626e7b5e56693169040472a577551aa44976e2d412e957043a31d47e0ca12b405c8b30a16f506cc
+  data.tar.gz: f2b50907f8826e6c76c03b933a1571b5b523ebb7087ab4854fddb300c27d5ba64d5f75e53f2d5639234bd8de07adff2702ceb0ae61d37744fcd7737ccb0c3323

data/.github/workflows/ruby.yml

@@ -0,0 +1,32 @@
+# This workflow uses actions that are not certified by GitHub.
+# They are provided by a third-party and are governed by
+# separate terms of service, privacy policy, and support
+# documentation.
+# This workflow will download a prebuilt Ruby version, install dependencies and run tests with Rake
+# For more information see: https://github.com/marketplace/actions/setup-ruby-jruby-and-truffleruby
+
+name: Ruby
+
+on:
+  push:
+    branches: [ main ]
+  pull_request:
+    branches: [ main ]
+
+jobs:
+  test:
+
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        ruby-version: ['2.7', '3.0']
+
+    steps:
+    - uses: actions/checkout@v2
+    - name: Set up Ruby
+      uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: ${{ matrix.ruby-version }}
+        bundler-cache: true # runs 'bundle install' and caches installed gems automatically
+    - name: Run tests
+      run: bundle exec rake

data/.gitignore

@@ -1,6 +1,5 @@
 /.bundle/
 /.yardoc
-/Gemfile.lock
 /_yardoc/
 /coverage/
 /doc/

data/CHANGELOG.md

@@ -0,0 +1,10 @@
+# CHANGELOG
+
+## 0.1.0
+
+* Add support for zeitwerk@2.5 & higher
+* Disable the autoloader patch after migrations finish
+
+## 0.0.2
+
+* Support classic autoloader

data/Gemfile

@@ -1,4 +1,4 @@
-source 'https://rubygems.org'
+source "https://rubygems.org"
 
 # Specify your gem's dependencies in good_migrations.gemspec
 gemspec

data/Gemfile.lock

@@ -0,0 +1,113 @@
+PATH
+  remote: .
+  specs:
+    good_migrations (0.1.0)
+      activerecord (>= 3.1)
+      railties (>= 3.1)
+
+GEM
+  remote: https://rubygems.org/
+  specs:
+    actionpack (6.1.4)
+      actionview (= 6.1.4)
+      activesupport (= 6.1.4)
+      rack (~> 2.0, >= 2.0.9)
+      rack-test (>= 0.6.3)
+      rails-dom-testing (~> 2.0)
+      rails-html-sanitizer (~> 1.0, >= 1.2.0)
+    actionview (6.1.4)
+      activesupport (= 6.1.4)
+      builder (~> 3.1)
+      erubi (~> 1.4)
+      rails-dom-testing (~> 2.0)
+      rails-html-sanitizer (~> 1.1, >= 1.2.0)
+    activemodel (6.1.4)
+      activesupport (= 6.1.4)
+    activerecord (6.1.4)
+      activemodel (= 6.1.4)
+      activesupport (= 6.1.4)
+    activesupport (6.1.4)
+      concurrent-ruby (~> 1.0, >= 1.0.2)
+      i18n (>= 1.6, < 2)
+      minitest (>= 5.1)
+      tzinfo (~> 2.0)
+      zeitwerk (~> 2.3)
+    ast (2.4.2)
+    builder (3.2.4)
+    coderay (1.1.3)
+    concurrent-ruby (1.1.9)
+    crass (1.0.6)
+    erubi (1.10.0)
+    i18n (1.8.10)
+      concurrent-ruby (~> 1.0)
+    loofah (2.10.0)
+      crass (~> 1.0.2)
+      nokogiri (>= 1.5.9)
+    method_source (1.0.0)
+    mini_portile2 (2.5.3)
+    minitest (5.14.4)
+    nokogiri (1.11.7)
+      mini_portile2 (~> 2.5.0)
+      racc (~> 1.4)
+    parallel (1.20.1)
+    parser (3.0.1.1)
+      ast (~> 2.4.1)
+    pry (0.14.1)
+      coderay (~> 1.1)
+      method_source (~> 1.0)
+    racc (1.5.2)
+    rack (2.2.3)
+    rack-test (1.1.0)
+      rack (>= 1.0, < 3)
+    rails-dom-testing (2.0.3)
+      activesupport (>= 4.2.0)
+      nokogiri (>= 1.6)
+    rails-html-sanitizer (1.3.0)
+      loofah (~> 2.3)
+    railties (6.1.4)
+      actionpack (= 6.1.4)
+      activesupport (= 6.1.4)
+      method_source
+      rake (>= 0.13)
+      thor (~> 1.0)
+    rainbow (3.0.0)
+    rake (13.0.3)
+    regexp_parser (2.1.1)
+    rexml (3.2.5)
+    rubocop (1.17.0)
+      parallel (~> 1.10)
+      parser (>= 3.0.0.0)
+      rainbow (>= 2.2.2, < 4.0)
+      regexp_parser (>= 1.8, < 3.0)
+      rexml
+      rubocop-ast (>= 1.7.0, < 2.0)
+      ruby-progressbar (~> 1.7)
+      unicode-display_width (>= 1.4.0, < 3.0)
+    rubocop-ast (1.7.0)
+      parser (>= 3.0.1.1)
+    rubocop-performance (1.11.2)
+      rubocop (>= 1.7.0, < 2.0)
+      rubocop-ast (>= 0.4.0)
+    ruby-progressbar (1.11.0)
+    standard (1.1.2)
+      rubocop (= 1.17.0)
+      rubocop-performance (= 1.11.2)
+    thor (1.1.0)
+    tzinfo (2.0.4)
+      concurrent-ruby (~> 1.0)
+    unicode-display_width (2.0.0)
+    zeitwerk (2.4.2)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  bundler
+  good_migrations!
+  minitest
+  pry
+  rake
+  standard
+
+BUNDLED WITH
+   2.2.15

data/LICENSE.txt

@@ -1,6 +1,6 @@
 The MIT License (MIT)
 
-Copyright (c) 2016 Test Double, LLC
+Copyright (c) 2016-2021 Test Double, Inc.
 
 Permission is hereby granted, free of charge, to any person obtaining a copy of
 this software and associated documentation files (the "Software"), to deal in

data/README.md

@@ -1,14 +1,16 @@
 # good_migrations
 
-[![Build Status](https://travis-ci.org/testdouble/good-migrations.svg?branch=master)](https://travis-ci.org/testdouble/good-migrations)
-
 This gem prevents Rails from auto-loading app code while it's running migrations,
 preventing the common mistake of referencing ActiveRecord models from migration
 code.
 
 ## Usage
 
-Add good_migrations to your gemfile:
+**HEADS UP: zeitwerk 2.5.0 is not yet released, so if you're using zeitwerk at
+all, the gem won't be able to ensure your migrations are safe and you'll see a
+warning to that effect**
+
+Add good_migrations to your Gemfile:
 
 ``` ruby
 gem 'good_migrations'
@@ -16,6 +18,15 @@ gem 'good_migrations'
 
 And you're done! That's it.
 
+## Prerequisites
+
+This gem requires that your app uses either of these autoloader strategies:
+
+* The classic `ActiveSupport::Dependencies` autoloader (e.g. `config.autoloader
+  = :classic`), which is going away with Rails 7
+* Version 2.5 or higher of the zeitwerk autoloader (e.g. `config.autoloader =
+  :zeitwerk`) If your app uses an earlier version of zeitwerk, you'll see a
+  warning every time `db:migrate` is run
 
 ## Background
 
@@ -63,3 +74,10 @@ have a few options:
 Credit for figuring out where to hook into the ActiveSupport autoloader goes
 to [@tenderlove](https://github.com/tenderlove) for [this
 gist](https://gist.github.com/tenderlove/44447d1b1e466a28eb3f).
+
+## Caveats
+
+Because this gem works by monkey-patching the ActiveSupport auto-loader, it will
+not work if your Rails environment (development, by default) is configured to
+eager load your application's classes (see:
+[config.eager_load](http://edgeguides.rubyonrails.org/configuring.html#rails-general-configuration)).

data/Rakefile

@@ -1,10 +1,11 @@
 require "bundler/gem_tasks"
 require "rake/testtask"
+require "standard/rake"
 
 Rake::TestTask.new(:test) do |t|
   t.libs << "test"
   t.libs << "lib"
-  t.test_files = FileList['test/**/*_test.rb']
+  t.test_files = FileList["test/**/*_test.rb"]
 end
 
-task :default => :test
+task default: ["standard:fix", :test]

data/example/Gemfile

@@ -1,9 +1,9 @@
-source 'http://rubygems.org'
+source "https://rubygems.org"
 
-rails_version = ENV['RAILS_VERSION'] || '3.1.0'
-gem 'rails', rails_version
+gem "rails", "~> 6.1"
 
-gem 'good_migrations', :path => '..'
+gem "good_migrations", path: ".."
 
-gem 'sqlite3-ruby', :require => 'sqlite3'
+gem "sqlite3"
 
+gem "zeitwerk", github: "fxn/zeitwerk"

data/example/Rakefile

@@ -1,7 +1,7 @@
 # Add your own tasks in files placed in lib/tasks ending in .rake,
 # for example lib/tasks/capistrano.rake, and they will automatically be available to Rake.
 
-require File.expand_path('../config/application', __FILE__)
-require 'rake'
+require File.expand_path("../config/application", __FILE__)
+require "rake"
 
 Example::Application.load_tasks

data/example/app/models/pant.rb

@@ -1,3 +1,2 @@
 class Pant < ActiveRecord::Base
 end
-

data/example/bin/rails

@@ -0,0 +1,4 @@
+#!/usr/bin/env ruby
+APP_PATH = File.expand_path("../config/application", __dir__)
+require_relative "../config/boot"
+require "rails/commands"

data/example/bin/rake

@@ -0,0 +1,4 @@
+#!/usr/bin/env ruby
+require_relative "../config/boot"
+require "rake"
+Rake.application.run

data/example/config.ru

@@ -1,4 +1,4 @@
 # This file is used by Rack-based servers to start the application.
 
-require ::File.expand_path('../config/environment',  __FILE__)
+require ::File.expand_path("../config/environment", __FILE__)
 run Example::Application

data/example/config/application.rb

@@ -1,6 +1,6 @@
-require File.expand_path('../boot', __FILE__)
+require File.expand_path("../boot", __FILE__)
 
-require 'rails/all'
+require "rails/all"
 
 # If you have a Gemfile, require the gems listed there, including any gems
 # you've limited to :test, :development, or :production.
@@ -8,6 +8,9 @@ Bundler.require(:default, Rails.env) if defined?(Bundler)
 
 module Example
   class Application < Rails::Application
+    if config.respond_to?("eager_load=")
+      config.eager_load = false
+    end
     # Settings in config/environments/* take precedence over those specified here.
     # Application configuration should go into files in config/initializers
     # -- all .rb files in that directory are automatically loaded.
@@ -38,5 +41,7 @@ module Example
 
     # Configure sensitive parameters which will be filtered from the log file.
     config.filter_parameters += [:password]
+
+    config.autoloader = ENV["AUTOLOADER"] == "zeitwerk" ? :zeitwerk : :classic
   end
 end

data/example/config/boot.rb

@@ -1,13 +1,15 @@
-require 'rubygems'
+require "rubygems"
 
 # Set up gems listed in the Gemfile.
-gemfile = File.expand_path('../../Gemfile', __FILE__)
-begin
-  ENV['BUNDLE_GEMFILE'] = gemfile
-  require 'bundler'
-  Bundler.setup
-rescue Bundler::GemNotFound => e
-  STDERR.puts e.message
-  STDERR.puts "Try running `bundle install`."
-  exit!
-end if File.exist?(gemfile)
+gemfile = File.expand_path("../../Gemfile", __FILE__)
+if File.exist?(gemfile)
+  begin
+    ENV["BUNDLE_GEMFILE"] = gemfile
+    require "bundler"
+    Bundler.setup
+  rescue Bundler::GemNotFound => e
+    warn e.message
+    warn "Try running `bundle install`."
+    exit!
+  end
+end

data/example/config/environment.rb

@@ -1,5 +1,5 @@
 # Load the rails application
-require File.expand_path('../application', __FILE__)
+require File.expand_path("../application", __FILE__)
 
 # Initialize the rails application
 Example::Application.initialize!

data/example/config/environments/development.rb

@@ -10,8 +10,7 @@ Example::Application.configure do
   config.whiny_nils = true
 
   # Show full error reports and disable caching
-  config.consider_all_requests_local       = true
-  config.action_view.debug_rjs             = true
+  config.consider_all_requests_local = true
   config.action_controller.perform_caching = false
 
   # Don't care if the mailer can't send
@@ -22,5 +21,6 @@ Example::Application.configure do
 
   # Only use best-standards-support built into browsers
   config.action_dispatch.best_standards_support = :builtin
-end
 
+  config.active_record.migration_error = false
+end

data/example/config/environments/production.rb

@@ -6,7 +6,7 @@ Example::Application.configure do
   config.cache_classes = true
 
   # Full error reports are disabled and caching is turned on
-  config.consider_all_requests_local       = false
+  config.consider_all_requests_local = false
   config.action_controller.perform_caching = true
 
   # Specifies the header that your server uses for sending files

data/example/config/environments/test.rb

@@ -11,14 +11,14 @@ Example::Application.configure do
   config.whiny_nils = true
 
   # Show full error reports and disable caching
-  config.consider_all_requests_local       = true
+  config.consider_all_requests_local = true
   config.action_controller.perform_caching = false
 
   # Raise exceptions instead of rendering exception templates
   config.action_dispatch.show_exceptions = false
 
   # Disable request forgery protection in test environment
-  config.action_controller.allow_forgery_protection    = false
+  config.action_controller.allow_forgery_protection = false
 
   # Tell Action Mailer not to deliver emails to the real world.
   # The :test delivery method accumulates sent emails in the

data/example/config/initializers/secret_token.rb

@@ -4,4 +4,4 @@
 # If you change this key, all old signed cookies will become invalid!
 # Make sure the secret is at least 30 characters and all random,
 # no regular words or you'll be exposed to dictionary attacks.
-Example::Application.config.secret_token = '5b687ab969ef4f33a41125ae95e73c368d1c391045639df9a1690221b13f8b8cbeb515db9cf5990548af5c309b8d3ab206648837001866cdf854278030285e3c'
+Example::Application.config.secret_token = "5b687ab969ef4f33a41125ae95e73c368d1c391045639df9a1690221b13f8b8cbeb515db9cf5990548af5c309b8d3ab206648837001866cdf854278030285e3c"

data/example/config/initializers/session_store.rb

@@ -1,6 +1,6 @@
 # Be sure to restart your server when you modify this file.
 
-Example::Application.config.session_store :cookie_store, :key => '_example_session'
+Example::Application.config.session_store :cookie_store, key: "_example_session"
 
 # Use the database for sessions instead of the cookie-based default,
 # which shouldn't be used to store highly confidential information

data/example/db/migrate/20160202162849_create_pants.rb

@@ -1,4 +1,4 @@
-class CreatePants < ActiveRecord::Migration
+class CreatePants < ActiveRecord::Migration[4.2]
   def change
     create_table :pants do |t|
       t.integer :length

data/example/db/migrate/20160202163803_change_pants.rb

@@ -1,6 +1,7 @@
-class ChangePants < ActiveRecord::Migration
+class ChangePants < ActiveRecord::Migration[4.2]
   class Pant < ActiveRecord::Base
   end
+
   def change
     Pant.all.each do |pant|
       # do a migration depending on the redefined Pant model

data/example/db/migrate/20160202182520_change_pants_dangerously.rb

@@ -1,4 +1,4 @@
-class ChangePantsDangerously < ActiveRecord::Migration
+class ChangePantsDangerously < ActiveRecord::Migration[4.2]
   def up
     Pant.find_each do |pant|
       # uh oh!

data/example/lib/tasks/load_pants.rake

@@ -0,0 +1,5 @@
+# Imagine this being run immediately after a migrate, and SHOULD be allowed
+# to load from app/ because it's not a migration
+task load_pants: :environment do
+  puts "This many pants: #{Pant.count} pants"
+end

data/example/script/rails

@@ -1,6 +1,6 @@
 #!/usr/bin/env ruby
 # This command will automatically be run when you run "rails" with Rails 3 gems installed from the root of your application.
 
-APP_PATH = File.expand_path('../../config/application',  __FILE__)
-require File.expand_path('../../config/boot',  __FILE__)
-require 'rails/commands'
+APP_PATH = File.expand_path("../../config/application", __FILE__)
+require File.expand_path("../../config/boot", __FILE__)
+require "rails/commands"

data/example/test/performance/browsing_test.rb

@@ -1,9 +1,9 @@
-require 'test_helper'
-require 'rails/performance_test_help'
+require "test_helper"
+require "rails/performance_test_help"
 
 # Profiling results for each test method are written to tmp/performance.
 class BrowsingTest < ActionDispatch::PerformanceTest
   def test_homepage
-    get '/'
+    get "/"
   end
 end

data/example/test/test_helper.rb

@@ -1,6 +1,6 @@
 ENV["RAILS_ENV"] = "test"
-require File.expand_path('../../config/environment', __FILE__)
-require 'rails/test_help'
+require File.expand_path("../../config/environment", __FILE__)
+require "rails/test_help"
 
 class ActiveSupport::TestCase
   # Setup all fixtures in test/fixtures/*.(yml|csv) for all tests in alphabetical order.

data/good_migrations.gemspec

@@ -1,28 +1,29 @@
-# coding: utf-8
-lib = File.expand_path('../lib', __FILE__)
+lib = File.expand_path("../lib", __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
-require 'good_migrations/version'
+require "good_migrations/version"
 
 Gem::Specification.new do |spec|
-  spec.name          = "good_migrations"
-  spec.version       = GoodMigrations::VERSION
-  spec.authors       = ["Justin Searls", "Kevin Baribeau"]
-  spec.email         = ["searls@gmail.com", "kevin.baribeau@gmail.com"]
+  spec.name = "good_migrations"
+  spec.version = GoodMigrations::VERSION
+  spec.authors = ["Justin Searls", "Kevin Baribeau"]
+  spec.email = ["searls@gmail.com", "kevin.baribeau@gmail.com"]
 
-  spec.summary       = %q{Prevents Rails from auto-loading app code in database migrations}
-  spec.description   = %q{Referencing code in app/ from a database migration risks breaking the migration when your app code changes; this gem prevents that mistake}
-  spec.homepage      = "https://github.com/testdouble/good-migrations"
+  spec.summary = "Prevents Rails from auto-loading app code in database migrations"
+  spec.description = "Referencing code in app/ from a database migration risks breaking the migration when your app code changes; this gem prevents that mistake"
+  spec.homepage = "https://github.com/testdouble/good-migrations"
+  spec.license = "MIT"
 
-  spec.files         = `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
-  spec.bindir        = "exe"
-  spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
+  spec.files = `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
+  spec.bindir = "exe"
+  spec.executables = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
   spec.add_dependency "railties", ">= 3.1"
   spec.add_dependency "activerecord", ">= 3.1"
 
-  spec.add_development_dependency "bundler", "~> 1.10"
-  spec.add_development_dependency "rake", "~> 10.0"
+  spec.add_development_dependency "bundler"
+  spec.add_development_dependency "rake"
   spec.add_development_dependency "minitest"
   spec.add_development_dependency "pry"
+  spec.add_development_dependency "standard"
 end

data/lib/good_migrations.rb

@@ -1,3 +1,5 @@
 require "good_migrations/version"
 require "good_migrations/load_error"
+require "good_migrations/patches_autoloader"
+require "good_migrations/prevents_app_load"
 require "good_migrations/railtie" if defined?(Rails)

data/lib/good_migrations/load_error.rb

@@ -1,4 +1,4 @@
 module GoodMigrations
-  class LoadError < ::LoadError
+  class LoadError < RuntimeError
   end
 end

data/lib/good_migrations/patches_autoloader.rb

@@ -0,0 +1,63 @@
+module GoodMigrations
+  class PatchesAutoloader
+    def self.instance
+      @instance ||= new
+    end
+
+    def initialize
+      @disabled = false
+    end
+
+    def disabled?
+      @disabled
+    end
+
+    def patch!
+      if Rails.singleton_class.method_defined?(:autoloaders) &&
+          Rails.autoloaders.zeitwerk_enabled?
+        if Rails.autoloaders.main.respond_to?(:on_load) &&
+            Rails.autoloaders.main.method(:on_load).arity <= 0
+          @disabled = false
+          Rails.autoloaders.each do |loader|
+            loader.on_load do |_, _, path|
+              if GoodMigrations::PreventsAppLoad.app_path?(path) &&
+                  !GoodMigrations::PatchesAutoloader.instance.disabled?
+                GoodMigrations::PreventsAppLoad.prevent_load!(path)
+              end
+            end
+          end
+        else
+          warn <<~UNSUPPORTED
+            WARNING: good_migrations is unable to ensure that your migrations are
+              not inadvertently loading application code, because your application
+              uses the zeitwerk autoloader (`config.autoloader = :zeitwerk`), but
+              is using a version prior to zeitwerk 2.5.0, which adds an on_load
+              hook that good_migrations can latch onto.
+
+            Solution: Ensure that zeitwerk isn't pinned below 2.5.0 in your
+              Gemfile and try running `bundle update zeitwerk`.
+
+          UNSUPPORTED
+        end
+      else
+        @disabled = false
+        ActiveSupport::Dependencies.class_eval do
+          extend Module.new {
+            def load_file(path, const_paths = loadable_constants_for_path(path))
+              if GoodMigrations::PreventsAppLoad.app_path?(path) &&
+                  !GoodMigrations::PatchesAutoloader.instance.disabled?
+                GoodMigrations::PreventsAppLoad.prevent_load!(path)
+              else
+                super
+              end
+            end
+          }
+        end
+      end
+    end
+
+    def unpatch!
+      @disabled = true
+    end
+  end
+end

data/lib/good_migrations/prevents_app_load.rb

@@ -0,0 +1,48 @@
+module GoodMigrations
+  class PreventsAppLoad
+    def self.app_path?(path)
+      path.starts_with? File.join(Rails.application.root, "app")
+    end
+
+    def self.prevent_load!(path)
+      raise GoodMigrations::LoadError, <<~ERROR
+        Rails attempted to auto-load:
+
+        #{path}
+
+        Which is in your project's `app/` directory. The good_migrations
+        gem was designed to prevent this, because migrations are intended
+        to be immutable and safe-to-run for the life of your project, but
+        code in `app/` is liable to change at any time.
+
+        The most common reason for this error is that you may be referencing an
+        ActiveRecord model inside the migration in order to use the ActiveRecord API
+        to implement a data migration by querying and updating objects.
+
+        For instance, if you want to access a model "User" in your migration, it's safer
+        to redefine the class inside the migration instead, like this:
+
+        class MakeUsersOlder < ActiveRecord::Migration
+          class User < ActiveRecord::Base
+            # Define whatever you need on the User beyond what AR adds automatically
+          end
+
+          def up
+            User.find_each do |user|
+              user.update!(:age => user.age + 1)
+            end
+          end
+
+          def down
+            #...
+          end
+        end
+
+        For more information, visit:
+
+        https://github.com/testdouble/good-migrations
+
+      ERROR
+    end
+  end
+end

data/lib/good_migrations/railtie.rb

@@ -3,7 +3,7 @@ require "active_record/railtie"
 module GoodMigrations
   class Railtie < Rails::Railtie
     rake_tasks do
-      Dir[File.join(File.dirname(__FILE__), '../../tasks/*.rake')].each do |file|
+      Dir[File.join(File.dirname(__FILE__), "../../tasks/*.rake")].each do |file|
         load file
       end
     end

data/lib/good_migrations/version.rb

@@ -1,3 +1,3 @@
 module GoodMigrations
-  VERSION = "0.0.2"
+  VERSION = "0.1.0"
 end

data/tasks/good_migrations.rake

@@ -1,63 +1,18 @@
-require 'active_support/dependencies'
-require 'good_migrations'
+require "active_support/dependencies"
+require "good_migrations"
 
 namespace :good_migrations do
   task :disable_autoload do
-    next if ENV['GOOD_MIGRATIONS'] == "skip"
-    ActiveSupport::Dependencies.class_eval do
-      extend Module.new {
-        def load_file(path, const_paths = loadable_constants_for_path(path))
-          if path.starts_with? File.join(Rails.application.root, 'app')
-            raise GoodMigrations::LoadError, <<-ERROR
-Rails attempted to auto-load:
-
-#{path}
-
-Which is in your project's `app/` directory. The good_migrations
-gem was designed to prevent this, because migrations are intended
-to be immutable and safe-to-run for the life of your project, but
-code in `app/` is liable to change at any time.
-
-The most common reason for this error is that you may be referencing an
-ActiveRecord model inside the migration in order to use the ActiveRecord API
-to implement a data migration by querying and updating objects.
-
-For instance, if you want to access a model "User" in your migration, it's safer
-to redefine the class inside the migration instead, like this:
-
-class MakeUsersOlder < ActiveRecord::Migration
-  class User < ActiveRecord::Base
-    # Define whatever you need on the User beyond what AR adds automatically
-  end
-
-  def up
-    User.find_each do |user|
-      user.update!(:age => user.age + 1)
-    end
+    next if ENV["GOOD_MIGRATIONS"] == "skip"
+    GoodMigrations::PatchesAutoloader.instance.patch!
   end
 
-  def down
-    #...
+  task :reenable_autoload do
+    next if ENV["GOOD_MIGRATIONS"] == "skip"
+    GoodMigrations::PatchesAutoloader.instance.unpatch!
   end
 end
 
-For more information, visit:
-
-https://github.com/testdouble/good-migrations
-
-ERROR
-          else
-            super
-          end
-        end
-      }
-    end
-  end
+Rake::Task["db:migrate"].enhance(["good_migrations:disable_autoload"]) do
+  Rake::Task["good_migrations:reenable_autoload"].invoke
 end
-
-Rake.application.in_namespace('db:migrate') do |namespace|
-  ([Rake::Task['db:migrate']] + namespace.tasks).each do |task|
-    task.prerequisites << "good_migrations:disable_autoload"
-  end
-end
-

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: good_migrations
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.1.0
 platform: ruby
 authors:
 - Justin Searls
 - Kevin Baribeau
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2016-02-02 00:00:00.000000000 Z
+date: 2021-06-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: railties
@@ -43,30 +43,30 @@ dependencies:
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.10'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.10'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: minitest
   requirement: !ruby/object:Gem::Requirement
@@ -95,6 +95,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: standard
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Referencing code in app/ from a database migration risks breaking the
   migration when your app code changes; this gem prevents that mistake
 email:
@@ -104,9 +118,11 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/ruby.yml"
 - ".gitignore"
-- ".travis.yml"
+- CHANGELOG.md
 - Gemfile
+- Gemfile.lock
 - LICENSE.txt
 - README.md
 - Rakefile
@@ -116,10 +132,13 @@ files:
 - example/Gemfile
 - example/README
 - example/Rakefile
+- example/app/assets/config/manifest.js
 - example/app/controllers/application_controller.rb
 - example/app/helpers/application_helper.rb
 - example/app/models/pant.rb
 - example/app/views/layouts/application.html.erb
+- example/bin/rails
+- example/bin/rake
 - example/config.ru
 - example/config/application.rb
 - example/config/boot.rb
@@ -141,6 +160,7 @@ files:
 - example/db/seeds.rb
 - example/doc/README_FOR_APP
 - example/lib/tasks/.gitkeep
+- example/lib/tasks/load_pants.rake
 - example/public/404.html
 - example/public/422.html
 - example/public/500.html
@@ -162,13 +182,16 @@ files:
 - good_migrations.gemspec
 - lib/good_migrations.rb
 - lib/good_migrations/load_error.rb
+- lib/good_migrations/patches_autoloader.rb
+- lib/good_migrations/prevents_app_load.rb
 - lib/good_migrations/railtie.rb
 - lib/good_migrations/version.rb
 - tasks/good_migrations.rake
 homepage: https://github.com/testdouble/good-migrations
-licenses: []
+licenses:
+- MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -183,9 +206,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.4.5.1
-signing_key: 
+rubygems_version: 3.2.15
+signing_key:
 specification_version: 4
 summary: Prevents Rails from auto-loading app code in database migrations
 test_files: []

data/.travis.yml

@@ -1,11 +0,0 @@
-language: ruby
-sudo: false
-rvm:
-  - 2.2.3
-before_script:
-  - cd example && bundle install
-env:
-  - RAILS_ENV=3.1.0
-  - RAILS_ENV=4.0.0
-  - RAILS_ENV=4.2.0
-  - RAILS_ENV=5.0.0.beta2