archive_hook 0.1.1 → 0.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8e28890f3f0aa049ddfd3d24e63157b586c45d648c7f1fd186b27b5f285e7198
-  data.tar.gz: 52273e4374126d9ee717badf9f3e674774644447559101ac04ff20dcc50b8257
+  metadata.gz: 4952c4f26bb1327189da3753cbf44bf1fcd76f5477cc8e0160265d508b3f46db
+  data.tar.gz: 8e0930ef126fef1b8ff7e347a5ba17b95b0aad13eaabc4a1818f918c10edba3c
 SHA512:
-  metadata.gz: e194db706bc79ddbadbc41328da3956756aeecb3f99feeb313e4bf19c6fe764f27b2f23ab39e7a83a5c905bcbf113103569a261c21a23b039a559669f0462903
-  data.tar.gz: 96cdf5ff4e22b1186cac5810129a287fa576d828c8df2981e64ee050a39ce8d93b5651132c0e162700eda2939aa826924c0440ebaafde9c238eb63be6e151af3
+  metadata.gz: 9ea29d7cb8a0269f8a3bd956820d21f69bd33cff1cfc9ca3c8a3c3d7b35fabb0cb2b34362402ee4233f50dd11ea33f72f96f1d57ef3505834da837d793a158a0
+  data.tar.gz: b1dcefaf9e2b9e0b6c7caa491756a7876557f2dc0e6ce8bd95070a7f3120a7fa2717370c0dc6a8eda06c510251aeac0080976fd3be4c27e905c22513d11b52ad

data/Gemfile

@@ -2,8 +2,5 @@ source "https://rubygems.org"
 
 git_source(:github) {|repo_name| "https://github.com/#{repo_name}" }
 
-gem "rails", "~> 6.1"
-gem "pg"
-
 # Specify your gem's dependencies in archive_hook.gemspec
 gemspec

data/README.md

@@ -48,6 +48,12 @@ end
 Run the generator for the root model and for every model listed under a `children:` entry in
 your dependency mapping (see below) — each one needs its own `_archive` table.
 
+> **Postgres only (for now):** the generator's `CREATE TABLE ... (LIKE ...)` is Postgres syntax.
+> On other databases, write the `_archive` table migration by hand instead — the archiving and
+> restoring API itself (`.archive`, `.archive_scope`, `.restore_scope`) is plain SQL
+> (`INSERT INTO ... SELECT`, `DELETE ... WHERE id IN (...)`) run through the ActiveRecord
+> connection, so it isn't tied to any particular adapter.
+
 `ArchiveHook` reads the columns off the model at call time, so the archive table just needs to
 have matching column names — it doesn't need matching constraints, indexes, or an FK to the
 original table.

data/archive_hook.gemspec

@@ -32,7 +32,11 @@ Gem::Specification.new do |spec|
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
+  # Only ActiveRecord (+ Arel, bundled with it) is used at runtime; no need for full Rails.
+  spec.add_dependency "activerecord", ">= 5.2", "< 9"
+
   spec.add_development_dependency "bundler", "~> 2.3"
-  spec.add_development_dependency "rake", "~> 10.0"
+  spec.add_development_dependency "pg"
+  spec.add_development_dependency "rake", "~> 13.0"
   spec.add_development_dependency "rspec", "~> 3.0"
 end

data/lib/archive_hook/version.rb

@@ -1,3 +1,3 @@
 module ArchiveHook
-  VERSION = "0.1.1"
+  VERSION = "0.1.3"
 end

data/lib/archive_hook.rb

@@ -1,7 +1,29 @@
 require "archive_hook/version"
 
 module ArchiveHook
+  # Identifiers are interpolated straight into SQL, so anything that needs
+  # quoting -- a column called "default", "order" or "group", a table name that
+  # collides with a keyword -- has to be quoted or Postgres rejects the
+  # statement with a syntax error.
+  module Quoting
+    private
+
+    def quote_columns(column_names)
+      column_names.map { |name| connection.quote_column_name(name) }.join(",")
+    end
+
+    def quote_table(name)
+      connection.quote_table_name(name)
+    end
+
+    def connection
+      ActiveRecord::Base.connection
+    end
+  end
+
   class ScopeArchiver
+    include Quoting
+
     def initialize(dependencies: {})
       @dependencies = dependencies
     end
@@ -37,23 +59,24 @@ module ArchiveHook
     end
 
     def archive_records_sql(scope)
-      table_name = scope.table_name
-      attributes_list = scope.column_names.join(",")
+      attributes_list = quote_columns(scope.column_names)
       <<-SQL
-        INSERT INTO #{table_name}_archive (#{attributes_list})
+        INSERT INTO #{quote_table("#{scope.table_name}_archive")} (#{attributes_list})
         #{scope.select(attributes_list).to_sql}
       SQL
     end
   end
 
   class ScopeRestorer
+    include Quoting
+
     def initialize(dependencies: {})
       @dependencies = dependencies
     end
 
     def call(scope)
       parent = scope.model
-      table_name = "#{scope.table_name}_archive as #{scope.table_name}"
+      table_name = "#{quote_table("#{scope.table_name}_archive")} as #{quote_table(scope.table_name)}"
       parent_id_groups = scope.from(table_name).in_batches.map { |relation| relation.pluck(:id) }
       parent_id_groups.each do |parent_ids|
         restore_by_ids(parent, parent_ids)
@@ -78,22 +101,22 @@ module ArchiveHook
     end
 
     def restore_by_ids(model, ids)
-      table_name = model.table_name
       ActiveRecord::Base.transaction do
-        ActiveRecord::Base.connection.execute(Arel.sql(restore_records_sql(model, ids)))
-        ActiveRecord::Base.connection.execute(Arel.sql <<-SQL
-          DELETE FROM #{table_name}_archive WHERE id IN (#{ids.join(', ')})
+        connection.execute(Arel.sql(restore_records_sql(model, ids)))
+        connection.execute(Arel.sql <<-SQL
+          DELETE FROM #{quote_table("#{model.table_name}_archive")}
+          WHERE #{connection.quote_column_name(model.primary_key)} IN (#{ids.join(', ')})
         SQL
         )
       end
     end
 
     def restore_records_sql(model, ids)
-      table_name = model.table_name
-      attributes_list = model.column_names.join(",")
+      attributes_list = quote_columns(model.column_names)
       <<-SQL
-        INSERT INTO #{table_name} (#{attributes_list})
-        SELECT #{attributes_list} FROM #{table_name}_archive WHERE id IN (#{ids.join(', ')})
+        INSERT INTO #{quote_table(model.table_name)} (#{attributes_list})
+        SELECT #{attributes_list} FROM #{quote_table("#{model.table_name}_archive")}
+        WHERE #{connection.quote_column_name(model.primary_key)} IN (#{ids.join(', ')})
       SQL
     end
   end
@@ -101,7 +124,8 @@ module ArchiveHook
   class << self
     def archive(root, archive_date, dependencies)
       column = dependencies[root] && dependencies[root][:column] || :created_at
-      base_scope = root.where("#{column} < ?", archive_date)
+      quoted_column = ActiveRecord::Base.connection.quote_column_name(column)
+      base_scope = root.where("#{quoted_column} < ?", archive_date)
       ScopeArchiver.new(dependencies: dependencies).call(base_scope)
     end
 

data/lib/generators/archive_hook/templates/migration.rb

@@ -2,7 +2,13 @@
 
 class Create<%= archive_table_name.camelize %> < ActiveRecord::Migration<%= migration_version %>
   def up
-    execute("CREATE TABLE <%= archive_table_name %> (LIKE <%= table_name %>)")
+    # Quoted at run time so a mixed-case table name survives Postgres folding
+    # unquoted identifiers to lower case, and so a name that collides with a
+    # reserved word still parses.
+    execute(
+      "CREATE TABLE #{connection.quote_table_name('<%= archive_table_name %>')} " \
+      "(LIKE #{connection.quote_table_name('<%= table_name %>')})"
+    )
   end
 
   def down

metadata

@@ -1,15 +1,35 @@
 --- !ruby/object:Gem::Specification
 name: archive_hook
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.3
 platform: ruby
 authors:
 - alvir
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-08-28 00:00:00.000000000 Z
+date: 2026-09-03 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: activerecord
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '5.2'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '9'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '5.2'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '9'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -24,20 +44,34 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '2.3'
+- !ruby/object:Gem::Dependency
+  name: pg
+  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'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '13.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '13.0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -83,7 +117,7 @@ licenses:
 metadata:
   homepage_uri: https://github.com/alvir/archive_hook
   source_code_uri: https://github.com/alvir/archive_hook
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -98,8 +132,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3.1
-signing_key: 
+rubygems_version: 3.4.10
+signing_key:
 specification_version: 4
 summary: Archive ActiveRecord models (and their dependents) to parallel _archive tables
 test_files: []