activerecord-temporal 0.2.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: efddd21db7e9259c1a6314f94384cb7d4da7e06e0553df5817fafea120bf23d9
-  data.tar.gz: ac3c593aaedb772f5837c117a13d8eb391d3aa414eb68eaef16b863da1cffd11
+  metadata.gz: 87a493283c2b5a1746266fdedb4978b7ad768be307b73b40b2aa404b460a198c
+  data.tar.gz: ea10fd65c8c61878d1a92ae098a558094e7090e4ae09692ac15c1bd61b94a406
 SHA512:
-  metadata.gz: 3decd8140332021022bcd6a265d2ef12ea16407665ceec52946ed79d14c9fc72b5848ff998ada1f297b17f1e76d563d8a8fd0e3c8c05dba0fcd87e4ea2cf8b30
-  data.tar.gz: 9cfde96a0d807112c0486d755616d81cfa3d05d00571e40853c6bcb32e7a31e935b48bf1cc17263d958908375ec0545b58109925ef802d9a46e540ec44920438
+  metadata.gz: 1ffaf0ec3fdc39fb9c4239dcde8f10f27921726be199a4969125c6873d75f29c30f62c60627d229edd45bd4db532eabbbca3e64c760878599f2bd7b4bec0f6e8
+  data.tar.gz: 4cecd795b83076371ad73ac26014a77a7791202269c6fb6903bfef358d9226e445d1e40a0dc8fe433a668562d82c57463809dedab4892dfa9303a12d9154bd44

data/CHANGELOG.md

@@ -1,5 +1,11 @@
 ## [Unreleased]
 
+## [0.3.0] - 2025-11-20
+
+- Application versioning: Raise `ClosedRevisionError` when trying to revise or inactive closed versions.
+- System versioning: Store gem version in versioning hooks
+- System versioning: Add assertion for matching primary key on versioning hook creation
+
 ## [0.2.0] - 2025-11-19
 
 - Added ambient/global time scopes

data/README.md

@@ -8,16 +8,17 @@ It provides both system versioning and application versioning. They can be used
 
 As applications mature, changing business requirements become increasingly complicated by the need to handle historical data. You might need to:
 
-- Update subscription plans, but retain existing subscribers' original payment schedules
-- Allow users to see information as it was before their view permission was revoked
-- Understand why generated financial reports have changed recently
-- Restore erroneously updated data
+- Know the price of a product at the time it was added to a cart
+- Allow users to see content as it was before their subscription ended
+- Track the lifetime of long-lived, slowly change entities like projects or customers 
+- Generate reports based on the data as it was known at some time in the past
 
 Many Rails applications use a patchwork of approaches:
 
 - **Soft deletes** with a `deleted_at` column, but updates that still permanently overwrite data.
-- **Audit gems or JSON columns** that serialize changes. Their data doesn't evolve with schema changes and cannot be easily integrated into Active Record queries, scopes, and associations.
+- **Audit gems or JSON fields** that serialize rows. Their data doesn't evolve with schema changes and can't be easily integrated into Active Record queries, scopes, and associations.
 - **Event systems** that are used to fill gaps in the data model and gradually take on responsibilities that are implementation details with no business relevance.
+- **Ad-hoc snapshot columns** that result in important business entities having their historical data duplicated across many different and incohesive tables.
 
 Temporal databases solve these problems by providing a simple and coherent data model to reach for whenever historical data is needed.
 
@@ -38,6 +39,8 @@ gem "activerecord-temporal"
 
 ### Create a System Versioned Table
 
+Make sure you're using the `:sql` schema dumper.
+
 Create your regular `employees` table. For the `employees_history` table, add the `system_period` column and include it in the table's primary key. `#create_versioning_hook` is what enables system versioning.
 
 ```ruby
@@ -412,7 +415,7 @@ class LineItem < ApplicationRecord
 end
 
 module History
-  include Temporal::SystemVersioningNamespace
+  include ActiveRecord::Temporal::HistoryModelNamespace
 end
 
 History::Product                   # => History::Product(id: integer, system_period: tstzrange, name: string)
@@ -442,7 +445,7 @@ By default, calling `system_versioning` will look for a namespace called `Histor
 
 ```ruby
 module Versions
-  include Temporal::SystemVersioningNamespace
+  include ActiveRecord::Temporal::HistoryModelNamespace
 end
 
 class ApplicationRecord < ActiveRecord::Base
@@ -462,7 +465,7 @@ By default, the namespace will only provide history models for models in the roo
 
 ```ruby
 module History
-  include Temporal::SystemVersioningNamespace
+  include ActiveRecord::Temporal::HistoryModelNamespace
 
   namespace "Tenant"
 

data/lib/activerecord/temporal/application_versioning/application_versioned.rb

@@ -92,7 +92,7 @@ module ActiveRecord::Temporal
       end
 
       def revise_at(time)
-        raise "not head revision" unless head_revision?
+        raise ClosedRevisionError, "Cannot revise closed version" unless head_revision?
 
         Revision.new(self, time, save: true)
       end
@@ -102,7 +102,7 @@ module ActiveRecord::Temporal
       end
 
       def revision_at(time)
-        raise "not head revision" unless head_revision?
+        raise ClosedRevisionError, "Cannot revise closed version" unless head_revision?
 
         Revision.new(self, time, save: false)
       end
@@ -112,7 +112,7 @@ module ActiveRecord::Temporal
       end
 
       def inactivate_at(time)
-        raise "not head revision" unless head_revision?
+        raise ClosedRevisionError, "Cannot inactivate closed version" unless head_revision?
 
         set_time_dimension_end(time)
         save

data/lib/activerecord/temporal/application_versioning.rb

@@ -2,6 +2,8 @@ module ActiveRecord::Temporal
   module ApplicationVersioning
     extend ActiveSupport::Concern
 
+    class ClosedRevisionError < StandardError; end
+
     class_methods do
       def application_versioned
         include ApplicationVersioned

data/lib/activerecord/temporal/patches/join_dependency.rb

@@ -1,8 +1,9 @@
 module ActiveRecord::Temporal
   module Patches
     # This is a copy of a fix from https://github.com/rails/rails/pull/56088 that
-    # impacts this gem. I has been backported to supported stable versions of
-    # Active Record, but until those patches are released it's included here.
+    # impacts this gem. I has been merged and backported to supported stable
+    # versions of Active Record, but until those patches are released it's included
+    # here.
     module JoinDependency
       def instantiate(result_set, strict_loading_value, &block)
         primary_key = Array(join_root.primary_key).map { |column| aliases.column_alias(join_root, column) }

data/lib/activerecord/temporal/system_versioning/command_recorder.rb

@@ -1,26 +1,10 @@
 module ActiveRecord::Temporal
   module SystemVersioning
     module CommandRecorder
-      module ArrayExtractOptions
-        refine Array do
-          def extract_options
-            if last.is_a?(Hash) && last.extractable_options?
-              last
-            else
-              {}
-            end
-          end
-        end
-      end
-
-      using ArrayExtractOptions
-
       [
         :create_versioning_hook,
         :drop_versioning_hook,
-        :change_versioning_hook,
-        :create_table_with_system_versioning,
-        :drop_table_with_system_versioning
+        :change_versioning_hook
       ].each do |method|
         class_eval <<-EOV, __FILE__, __LINE__ + 1
           def #{method}(*args)
@@ -58,16 +42,6 @@ module ActiveRecord::Temporal
           ]
         ]
       end
-
-      def invert_create_table_with_system_versioning(args)
-        [:drop_table_with_system_versioning, args]
-      end
-
-      def invert_drop_table_with_system_versioning(args)
-        # TODO make this reversible
-
-        raise ActiveRecord::IrreversibleMigration, "drop_table_with_system_versioning is not reversible"
-      end
     end
   end
 end

data/lib/activerecord/temporal/system_versioning/schema_creation.rb

@@ -30,7 +30,8 @@ module ActiveRecord::Temporal
           verb: :insert,
           source_table: o.source_table,
           history_table: o.history_table,
-          columns: o.columns
+          columns: o.columns,
+          gem_version: o.gem_version
         }
 
         <<~SQL
@@ -64,7 +65,8 @@ module ActiveRecord::Temporal
           source_table: o.source_table,
           history_table: o.history_table,
           columns: o.columns,
-          primary_key: o.primary_key
+          primary_key: o.primary_key,
+          gem_version: o.gem_version
         }
 
         <<~SQL
@@ -101,7 +103,8 @@ module ActiveRecord::Temporal
           verb: :delete,
           source_table: o.source_table,
           history_table: o.history_table,
-          primary_key: o.primary_key
+          primary_key: o.primary_key,
+          gem_version: o.gem_version
         }
 
         <<~SQL

data/lib/activerecord/temporal/system_versioning/schema_definitions.rb

@@ -1,37 +1,39 @@
 module ActiveRecord::Temporal
   module SystemVersioning
     class VersioningHookDefinition
-      attr_accessor :source_table, :history_table, :columns, :primary_key
+      attr_accessor :source_table, :history_table, :columns, :primary_key, :gem_version
 
       def initialize(
         source_table,
         history_table,
         columns:,
-        primary_key:
+        primary_key:,
+        gem_version:
       )
         @source_table = source_table
         @history_table = history_table
         @columns = columns
         @primary_key = primary_key
+        @gem_version = gem_version
       end
 
       def insert_hook
-        InsertHookDefinition.new(@source_table, @history_table, @columns)
+        InsertHookDefinition.new(@source_table, @history_table, @columns, @gem_version)
       end
 
       def update_hook
-        UpdateHookDefinition.new(@source_table, @history_table, @columns, @primary_key)
+        UpdateHookDefinition.new(@source_table, @history_table, @columns, @primary_key, @gem_version)
       end
 
       def delete_hook
-        DeleteHookDefinition.new(@source_table, @history_table, @primary_key)
+        DeleteHookDefinition.new(@source_table, @history_table, @primary_key, @gem_version)
       end
     end
 
-    InsertHookDefinition = Struct.new(:source_table, :history_table, :columns)
+    InsertHookDefinition = Struct.new(:source_table, :history_table, :columns, :gem_version)
 
-    UpdateHookDefinition = Struct.new(:source_table, :history_table, :columns, :primary_key)
+    UpdateHookDefinition = Struct.new(:source_table, :history_table, :columns, :primary_key, :gem_version)
 
-    DeleteHookDefinition = Struct.new(:source_table, :history_table, :primary_key)
+    DeleteHookDefinition = Struct.new(:source_table, :history_table, :primary_key, :gem_version)
   end
 end

data/lib/activerecord/temporal/system_versioning/schema_statements.rb

@@ -1,72 +1,29 @@
 module ActiveRecord::Temporal
   module SystemVersioning
     module SchemaStatements
-      def create_table_with_system_versioning(table_name, **options, &block)
-        create_table(table_name, **options, &block)
-
-        source_pk = Array(primary_key(table_name))
-        history_options = options.merge(primary_key: source_pk + ["system_period"])
-
-        exclusion_constraint_expression = source_pk.map do |col|
-          "#{col} WITH ="
-        end.join(", ") + ", system_period WITH &&"
-
-        create_table("#{table_name}_history", **history_options) do |t|
-          columns(table_name).each do |column|
-            t.send(
-              column.type,
-              column.name,
-              comment: column.comment,
-              collation: column.collation,
-              default: nil,
-              limit: column.limit,
-              null: column.null,
-              precision: column.precision,
-              scale: column.scale
-            )
-          end
-
-          t.tstzrange :system_period, null: false
-          t.exclusion_constraint exclusion_constraint_expression, using: :gist
-        end
-
-        create_versioning_hook table_name,
-          "#{table_name}_history",
-          columns: :all,
-          primary_key: source_pk
-      end
-
-      def drop_table_with_system_versioning(*table_names, **options)
-        table_names.each do |table_name|
-          history_table_name = "#{table_name}_history"
-
-          drop_table(table_name, **options)
-          drop_table(history_table_name, **options)
-          drop_versioning_hook(table_name, history_table_name, **options.slice(:columns, :primary_key, :if_exists))
-        end
-      end
-
       def create_versioning_hook(source_table, history_table, **options)
         options.assert_valid_keys(:columns, :primary_key)
 
-        column_names = if (columns = options.fetch(:columns)) == :all
+        columns = options.fetch(:columns, :all)
+        primary_key = options.fetch(:primary_key, :id)
+
+        column_names = if columns == :all
           columns(source_table).map(&:name)
         else
           Array(columns).map(&:to_s)
         end
 
-        primary_key = options.fetch(:primary_key, :id)
-
         primary_key = if primary_key.is_a?(Array) && primary_key.length == 1
           primary_key.first
         else
           primary_key
         end
 
-        ensure_table_exists!(source_table)
-        ensure_table_exists!(history_table)
-        ensure_columns_match!(source_table, history_table, column_names)
-        ensure_columns_exists!(source_table, Array(primary_key))
+        assert_table_exists!(source_table)
+        assert_table_exists!(history_table)
+        assert_columns_match!(source_table, history_table, column_names)
+        assert_columns_exists!(source_table, Array(primary_key))
+        assert_primary_key_matches!(source_table, Array(primary_key))
 
         schema_creation = SchemaCreation.new(self)
 
@@ -74,7 +31,8 @@ module ActiveRecord::Temporal
           source_table,
           history_table,
           columns: column_names,
-          primary_key: primary_key
+          primary_key: primary_key,
+          gem_version: VERSION
         )
 
         execute schema_creation.accept(hook_definition)
@@ -97,14 +55,14 @@ module ActiveRecord::Temporal
       def versioning_hook(source_table)
         update_function_name = versioning_function_name(source_table, :update)
 
-        row = execute(<<~SQL.squish).first
+        row = exec_query(<<~SQL.squish, "SQL", [update_function_name]).first
           SELECT
             pg_proc.proname as function_name,
             obj_description(pg_proc.oid, 'pg_proc') as comment
           FROM pg_proc
           JOIN pg_namespace ON pg_proc.pronamespace = pg_namespace.oid
           WHERE pg_namespace.nspname NOT IN ('pg_catalog', 'information_schema')
-            AND pg_proc.proname = '#{update_function_name}'
+            AND pg_proc.proname = $1
         SQL
 
         return unless row
@@ -115,7 +73,8 @@ module ActiveRecord::Temporal
           metadata["source_table"],
           metadata["history_table"],
           columns: metadata["columns"],
-          primary_key: metadata["primary_key"]
+          primary_key: metadata["primary_key"],
+          gem_version: metadata["gem_version"]
         )
       end
 
@@ -125,13 +84,13 @@ module ActiveRecord::Temporal
         add_columns = (options[:add_columns] || []).map(&:to_s)
         remove_columns = (options[:remove_columns] || []).map(&:to_s)
 
-        ensure_table_exists!(source_table)
-        ensure_table_exists!(history_table)
-        ensure_columns_match!(source_table, history_table, add_columns)
+        assert_table_exists!(source_table)
+        assert_table_exists!(history_table)
+        assert_columns_match!(source_table, history_table, add_columns)
 
         hook_definition = versioning_hook(source_table)
 
-        ensure_hook_has_columns!(hook_definition, remove_columns)
+        assert_hook_has_columns!(hook_definition, remove_columns)
 
         drop_versioning_hook(source_table, history_table)
 
@@ -161,15 +120,15 @@ module ActiveRecord::Temporal
       def validate_create_versioning_hook_options!(options)
       end
 
-      def ensure_table_exists!(table_name)
+      def assert_table_exists!(table_name)
         return if table_exists?(table_name)
 
         raise ArgumentError, "table '#{table_name}' does not exist"
       end
 
-      def ensure_columns_match!(source_table, history_table, column_names)
-        ensure_columns_exists!(source_table, column_names)
-        ensure_columns_exists!(history_table, column_names)
+      def assert_columns_match!(source_table, history_table, column_names)
+        assert_columns_exists!(source_table, column_names)
+        assert_columns_exists!(history_table, column_names)
 
         column_names.each do |column|
           source_column = columns(source_table).find { _1.name == column }
@@ -181,7 +140,7 @@ module ActiveRecord::Temporal
         end
       end
 
-      def ensure_columns_exists!(table_name, column_names)
+      def assert_columns_exists!(table_name, column_names)
         column_names.each do |column|
           next if column_exists?(table_name, column)
 
@@ -189,7 +148,14 @@ module ActiveRecord::Temporal
         end
       end
 
-      def ensure_hook_has_columns!(hook, column_names)
+      def assert_primary_key_matches!(source_table, primary_key)
+        primary_key = primary_key&.map(&:to_s)
+        unless Array(primary_key(source_table)) == primary_key
+          raise ArgumentError, "table '#{source_table}' does not have primary key #{primary_key}"
+        end
+      end
+
+      def assert_hook_has_columns!(hook, column_names)
         column_names.each do |column_name|
           next if hook.columns.include?(column_name)
 

data/lib/activerecord/temporal/version.rb

@@ -1,3 +1,3 @@
 module ActiveRecord::Temporal
-  VERSION = "0.2.0"
+  VERSION = "0.3.0"
 end

data/lib/activerecord/temporal.rb

@@ -1,9 +1,6 @@
 require "active_support"
 
 require_relative "temporal/application_versioning/application_versioned"
-require_relative "temporal/application_versioning/command_recorder"
-require_relative "temporal/application_versioning/migration"
-require_relative "temporal/application_versioning/schema_statements"
 require_relative "temporal/querying/association_macros"
 require_relative "temporal/querying/association_scope"
 require_relative "temporal/querying/association_walker"
@@ -13,7 +10,6 @@ require_relative "temporal/querying/scope_registry"
 require_relative "temporal/querying/scoping"
 require_relative "temporal/querying/time_dimensions"
 require_relative "temporal/patches/association_reflection"
-require_relative "temporal/patches/command_recorder"
 require_relative "temporal/patches/join_dependency"
 require_relative "temporal/patches/merger"
 require_relative "temporal/patches/relation"
@@ -22,7 +18,6 @@ require_relative "temporal/system_versioning/command_recorder"
 require_relative "temporal/system_versioning/history_model_namespace"
 require_relative "temporal/system_versioning/history_model"
 require_relative "temporal/system_versioning/history_models"
-require_relative "temporal/system_versioning/migration"
 require_relative "temporal/system_versioning/schema_creation"
 require_relative "temporal/system_versioning/schema_definitions"
 require_relative "temporal/system_versioning/schema_statements"
@@ -31,6 +26,7 @@ require_relative "temporal/application_versioning"
 require_relative "temporal/querying"
 require_relative "temporal/scoping"
 require_relative "temporal/system_versioning"
+require_relative "temporal/version"
 
 module ActiveRecord::Temporal
   def system_versioning
@@ -46,13 +42,7 @@ module ActiveRecord::Temporal
 end
 
 ActiveSupport.on_load(:active_record) do
-  require "active_record/connection_adapters/postgresql_adapter" # TODO: add test
-
-  ActiveRecord::ConnectionAdapters::PostgreSQLAdapter
-    .include ActiveRecord::Temporal::ApplicationVersioning::SchemaStatements
-
-  ActiveRecord::Migration::CommandRecorder
-    .include ActiveRecord::Temporal::ApplicationVersioning::CommandRecorder
+  require "active_record/connection_adapters/postgresql_adapter"
 
   ActiveRecord::ConnectionAdapters::PostgreSQLAdapter
     .include ActiveRecord::Temporal::SystemVersioning::SchemaStatements
@@ -65,11 +55,6 @@ ActiveSupport.on_load(:active_record) do
 
   # Patches
 
-  # Patches `#invert_drop` to remove the `system_versioning` option. The original
-  # method determines reversibility by looking for the presence of arguments
-  ActiveRecord::Migration::CommandRecorder
-    .prepend ActiveRecord::Temporal::Patches::CommandRecorder
-
   # Patches `#build_arel` to  wrap itself in the as-of query scope registry.
   # This is what allows temporal association scopes to be aware of the time-scope
   # value of the relation that included them.
@@ -101,8 +86,9 @@ ActiveSupport.on_load(:active_record) do
     .prepend ActiveRecord::Temporal::Patches::AssociationReflection
 
   # This is a copy of a fix from https://github.com/rails/rails/pull/56088 that
-  # impacts this gem. I has been backported to supported stable versions of
-  # Active Record, but until those patches are released it's included here.
+  # impacts this gem. I has been merged and backported to supported stable
+  # versions of Active Record, but until those patches are released it's included
+  # here.
   ActiveRecord::Associations::JoinDependency
     .prepend ActiveRecord::Temporal::Patches::JoinDependency
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: activerecord-temporal
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Martin-Alexander
@@ -78,11 +78,7 @@ files:
 - lib/activerecord/temporal.rb
 - lib/activerecord/temporal/application_versioning.rb
 - lib/activerecord/temporal/application_versioning/application_versioned.rb
-- lib/activerecord/temporal/application_versioning/command_recorder.rb
-- lib/activerecord/temporal/application_versioning/migration.rb
-- lib/activerecord/temporal/application_versioning/schema_statements.rb
 - lib/activerecord/temporal/patches/association_reflection.rb
-- lib/activerecord/temporal/patches/command_recorder.rb
 - lib/activerecord/temporal/patches/join_dependency.rb
 - lib/activerecord/temporal/patches/merger.rb
 - lib/activerecord/temporal/patches/relation.rb
@@ -104,7 +100,6 @@ files:
 - lib/activerecord/temporal/system_versioning/history_model.rb
 - lib/activerecord/temporal/system_versioning/history_model_namespace.rb
 - lib/activerecord/temporal/system_versioning/history_models.rb
-- lib/activerecord/temporal/system_versioning/migration.rb
 - lib/activerecord/temporal/system_versioning/schema_creation.rb
 - lib/activerecord/temporal/system_versioning/schema_definitions.rb
 - lib/activerecord/temporal/system_versioning/schema_statements.rb
@@ -115,7 +110,7 @@ licenses:
 - MIT
 metadata:
   bug_tracker_uri: https://github.com/Martin-Alexander/activerecord-temporal/issues
-  changelog_uri: https://github.com/Martin-Alexander/activerecord-temporal/CHANGELOG.md
+  changelog_uri: https://github.com/Martin-Alexander/activerecord-temporal/blob/master/CHANGELOG.md
   homepage_uri: https://github.com/Martin-Alexander/activerecord-temporal
   source_code_uri: https://github.com/Martin-Alexander/activerecord-temporal
 rdoc_options: []

data/lib/activerecord/temporal/application_versioning/command_recorder.rb

@@ -1,14 +0,0 @@
-module ActiveRecord::Temporal
-  module ApplicationVersioning
-    module CommandRecorder
-      def create_application_versioned_table(*args)
-        record(:create_application_versioned_table, args)
-      end
-      ruby2_keywords(:create_application_versioned_table)
-
-      def invert_create_application_versioned_table(args)
-        [:drop_table, args]
-      end
-    end
-  end
-end

data/lib/activerecord/temporal/application_versioning/migration.rb

@@ -1,25 +0,0 @@
-module ActiveRecord::Temporal
-  module ApplicationVersioning
-    module Migration
-      extend ActiveSupport::Concern
-
-      included do
-        prepend Patches
-      end
-
-      module Patches
-        def create_table(table_name, id: :primary_key, primary_key: nil, force: nil, **options, &block)
-          application_versioning = options.delete(:application_versioning)
-
-          if application_versioning
-            create_application_versioned_table(
-              table_name, id:, primary_key:, force:, **options, &block
-            )
-          else
-            super
-          end
-        end
-      end
-    end
-  end
-end

data/lib/activerecord/temporal/application_versioning/schema_statements.rb

@@ -1,33 +0,0 @@
-module ActiveRecord::Temporal
-  module ApplicationVersioning
-    module SchemaStatements
-      def create_application_versioned_table(table_name, **options, &block)
-        pk_option = options[:primary_key]
-
-        primary_key = if options[:primary_key]
-          Array(options[:primary_key]) | [:version]
-        else
-          [:id, :version]
-        end
-
-        exclusion_constraint_expression = (primary_key - [:version]).map do |col|
-          "#{col} WITH ="
-        end.join(", ") + ", validity WITH &&"
-
-        options = options.merge(primary_key: primary_key)
-
-        create_table(table_name, **options) do |t|
-          unless pk_option.is_a?(Array)
-            t.bigserial pk_option || :id, null: false
-          end
-
-          t.bigint :version, null: false, default: 1
-          t.tstzrange :validity, null: false
-          t.exclusion_constraint exclusion_constraint_expression, using: :gist
-
-          instance_exec(t, &block)
-        end
-      end
-    end
-  end
-end

data/lib/activerecord/temporal/patches/command_recorder.rb

@@ -1,23 +0,0 @@
-module ActiveRecord::Temporal
-  module Patches
-    module CommandRecorder
-      def invert_drop_table(args, &block)
-        if extract_options(args).delete(:system_versioning)
-          raise ActiveRecord::IrreversibleMigration, "drop_table with system versioning is not supported"
-        end
-
-        super
-      end
-
-      private
-
-      def extract_options(array)
-        if array.last.is_a?(Hash) && array.last.extractable_options?
-          array.last
-        else
-          {}
-        end
-      end
-    end
-  end
-end

data/lib/activerecord/temporal/system_versioning/migration.rb

@@ -1,35 +0,0 @@
-module ActiveRecord::Temporal
-  module SystemVersioning
-    module Migration
-      extend ActiveSupport::Concern
-
-      included do
-        prepend Patches
-      end
-
-      module Patches
-        def create_table(table_name, id: :primary_key, primary_key: nil, force: nil, **options, &block)
-          system_versioning = options.delete(:system_versioning)
-
-          if system_versioning
-            create_table_with_system_versioning(
-              table_name, id:, primary_key:, force:, **options, &block
-            )
-          else
-            super
-          end
-        end
-
-        def drop_table(*table_names, **options)
-          system_versioning = options.delete(:system_versioning)
-
-          if system_versioning
-            drop_table_with_system_versioning(*table_names, **options)
-          else
-            super
-          end
-        end
-      end
-    end
-  end
-end