dionysus-rb 1.7.0 → 1.7.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a103f61431f824a5d8769b0d2ed6d3ab6b9493690b3503726ed87d8790da7dd9
-  data.tar.gz: 9d87121dd840665b4bba91771b5f3e7f2a27e753d2dd374b5e461f1db172aa8d
+  metadata.gz: 4b87f084174020b048f571661e800ea514567cce186ed7701c8db43ada5f35be
+  data.tar.gz: 62f4126b20a85405bb3337eb9b0bd71d540e39943d2a2e1b5a87b86c1eabbe5a
 SHA512:
-  metadata.gz: 784b7e7894cfabc80c1146a9ac93ce7d466a872cb3d86caec664fd1f2c1fa453d125ca2528b03e1a6d55b71fc15f87fdd54eb05cebb32639365c144a7303777d
-  data.tar.gz: b22a989af8d7a2cfe867f6ad1485ebf88f312ba3eb7e0ea0df52663cd60a552cf1576c1c7fc85ee4caffb576397c2684d06f596e8bf993c6d6b3cdf610dcf837
+  metadata.gz: e0bb5902c1a0a9aaeec93ffbd16b0408fbe0288252ef36e31cca95a67ee2eb15bdb41cf120af4f6a6aec60735a2c2d9f6b571d4aad3a506ef2e214f7731cc7d7
+  data.tar.gz: ef7a256c9ca43d3987055df6c49abfe5fcdbf53bd855b7af00ba39f0f4a394dd3923559d69d1b8abc4e8639bc0f7a6c0398b55c2b86c40188a9ee5512ee053e3

data/CHANGELOG.md

@@ -1,5 +1,8 @@
 ## [Unreleased]
 
+## [1.7.1]
+- Parse an ISO8601 string timestamp in the payload instead of ignoring it. 1.7.0 required every timestamp to answer `acts_like_time?`, so a serializer that renders the parent's `updated_at` as a string made the whole class a silent no-op: the parent never looked time-like, no payload was ever examined, no row was ever corrected and `dionysus.publish.timestamp_repair` never fired.
+
 ## [1.7.0]
 - Correct a row whose own `updated_at` predates a record embedded in its payload, then serialize again, behind `config.touch_records_behind_their_embedded_records` (default `false`). A serializer reads the parent's columns before the associations, so a child committed in between arrives under a timestamp older than itself, and the consumer discards the payload together with that child.
 

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    dionysus-rb (1.7.0)
+    dionysus-rb (1.7.1)
       activerecord (>= 5)
       activesupport (>= 3.2)
       concurrent-ruby

data/lib/dionysus/producer/embedded_timestamp_repair.rb

@@ -3,6 +3,7 @@
 class Dionysus::Producer::EmbeddedTimestampRepair
   TIMESTAMP_ATTRIBUTE = "updated_at"
   PRIMARY_KEY_ATTRIBUTE = "id"
+  ISO8601_WITH_OFFSET = %r{\A\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}(\.\d+)?(Z|[+-]\d{2}:?\d{2})\z}
 
   # Takes the block that serializes, so a repaired record can be serialized again in one place.
   def self.call(records, config, &)
@@ -43,8 +44,9 @@ class Dionysus::Producer::EmbeddedTimestampRepair
   def repaired?(record, record_payload)
     return false unless repairable?(record, record_payload)
 
+    published_at = coerce_time(record_payload[TIMESTAMP_ATTRIBUTE])
     latest = embedded_timestamps(record_payload).max
-    return false unless latest && latest > record_payload[TIMESTAMP_ATTRIBUTE]
+    return false unless published_at && latest && latest > published_at
 
     # WHERE guards the write: a stale read must never move the row backwards. When the row is already
     # ahead nothing is written, and the reload alone repairs the payload.
@@ -61,8 +63,7 @@ class Dionysus::Producer::EmbeddedTimestampRepair
 
   def repairable?(record, record_payload)
     record.is_a?(ActiveRecord::Base) && record.persisted? &&
-      record_payload.is_a?(Hash) && record_payload[PRIMARY_KEY_ATTRIBUTE] == record.id &&
-      time_like?(record_payload[TIMESTAMP_ATTRIBUTE])
+      record_payload.is_a?(Hash) && record_payload[PRIMARY_KEY_ATTRIBUTE] == record.id
   end
 
   def embedded_timestamps(record_payload)
@@ -70,8 +71,7 @@ class Dionysus::Producer::EmbeddedTimestampRepair
       Array.wrap(value).filter_map do |embedded|
         next unless embedded.is_a?(Hash)
 
-        timestamp = embedded[TIMESTAMP_ATTRIBUTE]
-        timestamp if time_like?(timestamp)
+        coerce_time(embedded[TIMESTAMP_ATTRIBUTE])
       end
     end
   end
@@ -81,7 +81,23 @@ class Dionysus::Producer::EmbeddedTimestampRepair
       tags: ["model:#{record.class}", "outcome:#{updated.zero? ? "stale_read" : "row_behind"}"])
   end
 
-  def time_like?(value)
-    value.respond_to?(:acts_like_time?) && value.acts_like_time?
+  # A serializer may render the timestamp it is about to publish, so one payload carries both shapes.
+  def coerce_time(value)
+    case value
+    when ActiveSupport::TimeWithZone, Time then value
+    when String then parse_iso8601(value)
+    end
+  end
+
+  def parse_iso8601(value)
+    return nil unless ISO8601_WITH_OFFSET.match?(value)
+
+    time = Time.iso8601(value)
+    # Time.iso8601 normalizes an impossible date: 2026-09-31 becomes October 1.
+    return nil unless value.start_with?(time.strftime("%Y-%m-%d"))
+
+    time.in_time_zone
+  rescue ArgumentError
+    nil
   end
 end

data/lib/dionysus/version.rb

@@ -3,5 +3,5 @@
 module Dionysus
   module Version
   end
-  VERSION = "1.7.0"
+  VERSION = "1.7.1"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: dionysus-rb
 version: !ruby/object:Gem::Version
-  version: 1.7.0
+  version: 1.7.1
 platform: ruby
 authors:
 - Karol Galanciak
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-09-07 00:00:00.000000000 Z
+date: 2026-09-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord