deimos-ruby 2.5.0 → 2.5.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7a0b8bc495e0ef1c93a1f3a7d4b721ecd0ec149df866c0281e572c032ba25de5
-  data.tar.gz: 9426813c28bb569ee16e1f453b9f636baf309197ed59a25a337146852aec029a
+  metadata.gz: 705269764ba0122cc1eff515eef044d8855b2b050c0e4ad29627c65055da8e66
+  data.tar.gz: 6184e80be2db0d657b47fd2d4a54957b95ce52f3c40eeee0edafa59d7de99c96
 SHA512:
-  metadata.gz: bc7752b7345504f327c2c415dc90c58b31a3d6a92175eef71c31dcb66ecc358d7580bc79d7cd6b78c2df71bc8a45fab554d89018d6d85c2d23f91d652569f55c
-  data.tar.gz: b21aead2768acf0045a91175405a38352763a6ed85b7adc78fc70bdf1ea6dbf4cd993a94ac038c1275e7be6821bd623ab2e5b665f4509d7ce299dfba465e964d
+  metadata.gz: 608942a3b72c7f5da730349d19da526582a564f9ffdb66e3c2a3b9bfe63821ce575b87e663aeaf341879fb2887a48927dbae496b8068b61ff6e32e23ca650cdd
+  data.tar.gz: 393aa82894d78399130fd3aed4ae46794dd3879cb6c52f992f4f8e11ff2e1febde69a68f3d9e18b7360c6fd6ab298b823ed69e32ae41231e299e6b96e28eecfc

data/CHANGELOG.md

@@ -7,6 +7,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## Unreleased
 
+## 2.5.2 - 2026-04-24
+
+- Fix: `BatchRecordList#fill_primary_keys!` no longer short-circuits on stale in-memory primary keys left over from a rolled-back `DeadlockRetry` attempt, preventing `ActiveRecord::InvalidForeignKey` during association imports.
+
+## 2.5.1 - 2026-03-18
+
+- Fix: Outbox producer would swallow ActiveRecord errors except for uniqueness.
+
 ## 2.5.0 - 2026-03-04
 
 - Breaking: `process_message?` in batch consumption now receives the full Kafka message object instead of just the payload, allowing tombstones to be identified via the message key and metadata.

data/lib/deimos/active_record_consume/batch_record.rb

@@ -16,6 +16,8 @@ module Deimos
       attr_accessor :bulk_import_id
       # @return [String] The column name to use for bulk IDs - defaults to `bulk_import_id`.
       attr_accessor :bulk_import_column
+      # @return [Boolean] true if the primary key was supplied in the input attributes,
+      attr_accessor :primary_key_preset
 
       delegate :valid?, :errors, :send, :attributes, to: :record
 
@@ -32,6 +34,7 @@ module Deimos
         end
         attributes = attributes.with_indifferent_access
         self.record = klass.new(attributes.slice(*klass.column_names))
+        self.primary_key_preset = !self.record[klass.primary_key].nil?
         assoc_keys = attributes.keys.select { |k| klass.reflect_on_association(k) }
         # a hash with just the association keys, removing all actual column information.
         self.associations = attributes.slice(*assoc_keys)

data/lib/deimos/active_record_consume/batch_record_list.rb

@@ -53,7 +53,8 @@ module Deimos
       def fill_primary_keys!
         primary_col = self.klass.primary_key
 
-        return if self.batch_records.empty? || self.batch_records.first.send(primary_col).present?
+        # Skip if nothing to backfill or caller already supplied the PK.
+        return if self.batch_records.empty? || self.batch_records.first.primary_key_preset
 
         bulk_import_map = self.klass.
           where(self.bulk_import_column => self.batch_records.map(&:bulk_import_id)).

data/lib/deimos/kafka_topic_info.rb

@@ -18,7 +18,7 @@ module Deimos
       def lock(topic, lock_id) # rubocop:disable Naming/PredicateMethod
         # Try to create it - it's fine if it already exists
         begin
-          self.create(topic: topic, last_processed_at: Time.zone.now)
+          self.create!(topic: topic, last_processed_at: Time.zone.now)
         rescue ActiveRecord::RecordNotUnique
           # continue on
         end

data/lib/deimos/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Deimos
-  VERSION = '2.5.0'
+  VERSION = '2.5.2'
 end

data/spec/active_record_consume/mass_updater_spec.rb

@@ -112,6 +112,34 @@ RSpec.describe Deimos::ActiveRecordConsume::MassUpdater do
           expect(Detail.count).to eq(0)
         end
 
+        it 'should refetch parent primary keys when retrying child import' do
+          allow(Widget).to receive(:import!).and_call_original
+          allow(Widget).to receive(:where).and_call_original
+
+          # Deadlock the first child import to trigger DeadlockRetry; let the retry pass through.
+          detail_call_count = 0
+          allow(Detail).to receive(:import!).and_wrap_original do |m, *args|
+            detail_call_count += 1
+            raise ActiveRecord::Deadlocked, 'Lock wait timeout exceeded' if detail_call_count == 1
+
+            m.call(*args)
+          end
+
+          described_class.new(Widget, bulk_import_id_generator: bulk_id_generator).mass_update(batch)
+
+          # Parent ids written by `fill_primary_keys!` during attempt 1 stay in
+          # memory after the DB rollback, so they're stale on retry. Assert the
+          # retry re-queries. Outcome-level checks are unreliable: SQLite reuses
+          # rolled-back ids so stale memory matches coincidentally; MySQL and
+          # Postgres burn auto-increment and surface ActiveRecord::InvalidForeignKey.
+          expect(Widget).to have_received(:where).
+            with(bulk_import_id: [instance_of(String), instance_of(String)]).twice
+          expect(Widget.count).to eq(2)
+          expect(Detail.count).to eq(2)
+          expect(Widget.first.detail).not_to be_nil
+          expect(Widget.last.detail).not_to be_nil
+        end
+
       end
 
       context 'with save_associations_first' do

data/spec/kafka_topic_info_spec.rb

@@ -11,6 +11,11 @@ each_db_config(Deimos::KafkaTopicInfo) do
     expect(described_class.last.locked_by).to eq('def')
   end
 
+  it 'should raise an error if the create fails for a non-unique reason' do
+    allow(described_class).to receive(:create!).and_raise(ActiveRecord::ActiveRecordError, 'some other error')
+    expect { described_class.lock('my-topic', 'abc') }.to raise_error(ActiveRecord::ActiveRecordError)
+  end
+
   it "should lock the topic if it's old" do
     described_class.create!(topic: 'my-topic', locked_by: 'abc', error: true,
                             locked_at: 2.minutes.ago)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: deimos-ruby
 version: !ruby/object:Gem::Version
-  version: 2.5.0
+  version: 2.5.2
 platform: ruby
 authors:
 - Daniel Orner
@@ -711,7 +711,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.3
+rubygems_version: 4.0.6
 specification_version: 4
 summary: Kafka libraries for Ruby.
 test_files: []