after_migrate 0.1.1 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2602aa9ee3d2ffa2c1ea85ca8cbfa1b101679aa42ed05e0a4cfa4c742f258f59
-  data.tar.gz: d581242202742eb6a62be1cd1799a49c196ca82c99e8b36428ee6390fd12c415
+  metadata.gz: a13db746f7bfcbcb7367f028493efd2b464e564b32f09be5debb62da1326f8bf
+  data.tar.gz: b1f0db683bb9b12ed407f14f3f1b1c0baba0a80cfd445c6182b9bd5ad7378821
 SHA512:
-  metadata.gz: d81dce8555f469b190b3cfd37f7eb2b919aab714e6fcfb1842271c323495b213be2c2e779c8400ad67d28832c11dbc7ad7147e90f93b6950069b6b6b9ea8d558
-  data.tar.gz: 77f23e51b264cf80b5cbf028a0e43106d13498f9f7d109b1eafe26c6380269c08bb3574946ec534d08b8ac175a09c008823e64d0c862c4e4441e66c1df145399
+  metadata.gz: '085b3be36d4031981294d5a5ddc929aae3d781c9d1c3d8446f4508f7c7c526064c8b6529b68237e3e3517885f22251b3db071839dae1bcdf8df654d1e79b2e70'
+  data.tar.gz: 212614cb1145bbe3f22d9b005e6524af133b5d50f659ca8da2efff0d6416efd7ff90fccf39575719d4237df08029a5dcab6b8ef8dfd5072027bce902bc2b3bd8

data/.ruby-version

@@ -1 +1 @@
-3.4.7
+3.4.9

data/CHANGELOG.md

@@ -2,6 +2,21 @@
 
 All notable changes to this project will be documented in this file.
 
+## [0.2.0] - 2026-04-28
+
+### Added
+- `defer` configuration option (default: `true`) — when enabled, rake task enhancements only collect touched tables; they do not run maintenance automatically
+- `AfterMigrate.run!` public API to explicitly trigger maintenance on all collected tables across all schemas, then clear the store — intended to be called once after all tenant migrations complete in multi-tenant setups
+- `after_migrate:run` rake task as a convenience wrapper around `AfterMigrate.run!`
+
+### Changed
+- **Breaking**: replaced `AfterMigrate::Current` (`ActiveSupport::CurrentAttributes`) with a module-level persistent store (`Concurrent::Map`). The store accumulates touched tables across multiple rake task invocations and only resets when `AfterMigrate.run!` is called.
+- Fixed `all_tables` analyze mode - previously iterated `.each_value` (yielding `Concurrent::Set` objects) instead of schema names, producing incorrect SQL queries.
+
+### Removed
+- `AfterMigrate::Current` - no longer needed. If you referenced this class directly, use `AfterMigrate.affected_tables` instead.
+- `Executor.call(reset:)` parameter - the store is always cleared after `run!`; pass `schema:` to scope execution to a single schema.
+
 ## [0.1.0] - 2025-04-05
 
 ### Added

data/lib/after_migrate/adapters/sql.rb

@@ -8,13 +8,13 @@ module AfterMigrate
     /x
 
     PATTERNS = {
-      update: /update\s+(?:only\s+)?(#{IDENT})(?!\s*\()/ix,
-      insert: /insert\s+into\s+(#{IDENT})(?!\s*\()/ix,
-      delete: /delete\s+from\s+(#{IDENT})(?!\s*\()/ix,
-      drop_table: /drop\s+table\s+(?:if\s+exists\s+)?(#{IDENT})(?!\s*\()/ix,
-      alter_table: /alter\s+table\s+(#{IDENT})(?!\s*\()/ix,
-      create_table: /create\s+table\s+(?:if\s+not\s+exists\s+)?(#{IDENT})(?!\s*\()/ix,
-      from_join: /(?:from|join)\s+(#{IDENT})(?!\s*\()/ix
+      update: /update\s+(?:only\s+)?(#{IDENT})(?!\()/ix,
+      insert: /insert\s+into\s+(#{IDENT})(?!\()/ix,
+      delete: /delete\s+from\s+(#{IDENT})(?!\()/ix,
+      drop_table: /drop\s+table\s+(?:if\s+exists\s+)?(#{IDENT})(?!\()/ix,
+      alter_table: /alter\s+table\s+(#{IDENT})(?!\()/ix,
+      create_table: /create\s+table\s+(?:if\s+not\s+exists\s+)?(#{IDENT})(?!\()/ix,
+      from_join: /(?:from|join)\s+(#{IDENT})(?!\()/ix
     }.freeze
 
     def parse_tables(sql)

data/lib/after_migrate/collector.rb

@@ -13,24 +13,15 @@ module AfterMigrate
       return unless sql.match?(/\A\s*(CREATE|ALTER|DROP|INSERT|UPDATE|DELETE|RENAME\s+TABLE|TRUNCATE)/i)
 
       table_names = parse_tables(sql)
-      schema ||= fetch_schema
-      # AfterMigrate.log("[#{schema}] Detected change from '#{sql}' to tables: #{table_names}") if table_names.present?
-      collect_tables(schema:, table_names:)
+      schema = fetch_schema
+      AfterMigrate.merge_tables(schema, table_names)
     end
 
     private
 
-    def collect_tables(schema:, table_names:)
-      return if table_names.blank?
-
-      AfterMigrate::Current.affected_tables ||= Hash.new { |h, k| h[k] = Concurrent::Set.new }
-      AfterMigrate::Current.affected_tables[schema].merge(table_names)
-    end
-
     def fetch_schema
       connection = ActiveRecord::Base.connection
-      adapter = connection.adapter_name
-      case adapter
+      case connection.adapter_name
       when 'PostgreSQL'
         quoted = connection.schema_search_path.split(',').first
         quoted&.delete('"')
@@ -39,8 +30,7 @@ module AfterMigrate
 
     def parse_tables(sql)
       connection = ActiveRecord::Base.connection
-      adapter = connection.adapter_name
-      case adapter
+      case connection.adapter_name
       when 'PostgreSQL'
         AfterMigrate::Postgresql.parse_tables(sql)
       when 'SQLite'
@@ -48,7 +38,7 @@ module AfterMigrate
       when 'Mysql2', 'Trilogy'
         AfterMigrate::Mysql.parse_tables(sql)
       else
-        AfterMigrate.log("No maintenance implemented for #{adapter}")
+        AfterMigrate.log("No maintenance implemented for #{connection.adapter_name}")
       end
     end
   end

data/lib/after_migrate/executor.rb

@@ -11,16 +11,17 @@ module AfterMigrate
 
     module_function
 
-    def call(reset: true, schema: nil)
-      # AfterMigrate.log("Executing schema: #{schema} -> #{target_tables}...")
-      return if target_tables.blank?
-      return run_optimize(schema:, tables: target_tables[schema]) if schema.present?
+    def call(schema: nil)
+      tables = target_tables
+      return if tables.blank?
 
-      target_tables.each do |s, tables|
-        run_optimize(schema: s, tables:)
+      if schema.present?
+        run_optimize(schema:, tables: tables[schema]) if tables[schema].present?
+      else
+        tables.each { |s, t| run_optimize(schema: s, tables: t) }
       end
     ensure
-      AfterMigrate::Current.reset if reset
+      AfterMigrate.reset!
     end
 
     public :call
@@ -31,14 +32,13 @@ module AfterMigrate
       table_names = tables.to_a.sort
       return if table_names.empty?
 
-      AfterMigrate.log("Migration touched #{table_names.size} table(s): #{table_names.join(', ')}")
+      AfterMigrate.log("Migration touched #{table_names.size} table(s) in schema #{schema.inspect}: #{table_names.join(', ')}")
       optimize_tables(schema:, table_names:)
     end
 
     def optimize_tables(schema:, table_names:)
       connection = ActiveRecord::Base.connection
-      adapter = connection.adapter_name
-      case adapter
+      case connection.adapter_name
       when 'PostgreSQL'
         AfterMigrate::Postgresql.optimize_tables(schema:, table_names:, connection:)
       when 'SQLite'
@@ -46,27 +46,26 @@ module AfterMigrate
       when 'Mysql2', 'Trilogy'
         AfterMigrate::Mysql.optimize_tables(schema:, table_names:, connection:)
       else
-        AfterMigrate.log("No maintenance implemented for #{adapter}")
+        AfterMigrate.log("No maintenance implemented for #{connection.adapter_name}")
       end
     end
 
     def target_tables
       case AfterMigrate.configuration.analyze
       when 'all_tables'
-        AfterMigrate::Current.affected_tables.each_value do |schema|
-          all_tables(schema:)
+        AfterMigrate.affected_tables.keys.each_with_object({}) do |schema, hash|
+          hash[schema] = all_tables(schema:)
         end
       when 'only_affected_tables'
-        AfterMigrate::Current.affected_tables
+        AfterMigrate.affected_tables
       else
-        []
+        {}
       end
     end
 
     def all_tables(schema:)
       connection = ActiveRecord::Base.connection
-      adapter = connection.adapter_name
-      case adapter
+      case connection.adapter_name
       when 'PostgreSQL'
         AfterMigrate::Postgresql.all_tables(schema:)
       when 'SQLite'

data/lib/after_migrate/railtie.rb

@@ -12,6 +12,8 @@ module AfterMigrate
         AfterMigrate::Collector.call(*args)
       end
 
+      # Unsubscribe when the app starts serving requests so normal web traffic
+      # is never collected.
       app.executor.to_run { ActiveSupport::Notifications.unsubscribe(subscription) }
     end
 
@@ -24,8 +26,16 @@ module AfterMigrate
         Rake::Task[task_name].enhance do
           next unless AfterMigrate.configuration.enabled
           next unless AfterMigrate.configuration.rake_tasks_enhanced
+          next if AfterMigrate.configuration.defer
 
-          AfterMigrate::Executor.call(reset: true)
+          AfterMigrate.run!
+        end
+      end
+
+      namespace :after_migrate do
+        desc 'Run database maintenance (ANALYZE/VACUUM) on all tables collected across migrations'
+        task run: :environment do
+          AfterMigrate.run!
         end
       end
     end

data/lib/after_migrate/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module AfterMigrate
-  VERSION = '0.1.1'
+  VERSION = '0.2.0'
 end

data/lib/after_migrate.rb

@@ -1,14 +1,13 @@
 # frozen_string_literal: true
 
 require 'after_migrate/version'
-require 'after_migrate/current'
 require 'after_migrate/collector'
 require 'after_migrate/executor'
 require 'after_migrate/railtie'
 
 module AfterMigrate
   class Configuration
-    attr_accessor :enabled, :verbose, :vacuum, :analyze, :rake_tasks_enhanced
+    attr_accessor :enabled, :verbose, :vacuum, :analyze, :rake_tasks_enhanced, :defer
 
     def initialize
       @enabled = true
@@ -16,6 +15,7 @@ module AfterMigrate
       @vacuum = true
       @analyze = 'only_affected_tables'
       @rake_tasks_enhanced = true
+      @defer = true
     end
   end
 
@@ -31,5 +31,29 @@ module AfterMigrate
     def log(msg)
       warn "[after_migrate] #{msg}" if AfterMigrate.configuration.verbose
     end
+
+    # Persistent cross-migration store: schema_name => Concurrent::Set<table_name>
+    def affected_tables
+      @affected_tables ||= Concurrent::Map.new
+    end
+
+    def merge_tables(schema, table_names)
+      return if table_names.blank?
+
+      set = affected_tables.compute_if_absent(schema) { Concurrent::Set.new }
+      set.merge(table_names)
+    end
+
+    # Trigger database maintenance on all collected tables, then clear the store.
+    # In multi-tenant setups call this once after all tenant migrations complete.
+    def run!(schema: nil)
+      return unless configuration.enabled
+
+      Executor.call(schema:)
+    end
+
+    def reset!
+      @affected_tables = nil
+    end
   end
 end

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: after_migrate
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.2.0
 platform: ruby
 authors:
 - Nikolay Moskvin
-autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-12-11 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: pg_query
@@ -66,7 +65,6 @@ files:
 - lib/after_migrate/adapters/sql.rb
 - lib/after_migrate/adapters/sqlite.rb
 - lib/after_migrate/collector.rb
-- lib/after_migrate/current.rb
 - lib/after_migrate/executor.rb
 - lib/after_migrate/railtie.rb
 - lib/after_migrate/version.rb
@@ -79,7 +77,6 @@ metadata:
   source_code_uri: https://github.com/moskvin/after_migrate
   changelog_uri: https://github.com/moskvin/after_migrate/blob/master/CHANGELOG.md
   rubygems_mfa_required: 'true'
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -94,8 +91,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.19
-signing_key:
+rubygems_version: 3.6.9
 specification_version: 4
 summary: Automatically ANALYZE and VACUUM tables touched during Rails migrations.
 test_files: []

data/lib/after_migrate/current.rb

@@ -1,13 +0,0 @@
-# frozen_string_literal: true
-
-require 'active_support/current_attributes'
-
-module AfterMigrate
-  class Current < ActiveSupport::CurrentAttributes
-    attribute :affected_tables
-
-    resets do
-      self.affected_tables = Hash.new { |h, k| h[k] = Concurrent::Set.new }
-    end
-  end
-end