thecore_backend_commons 3.5.1 → 3.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4d8ce64d0b510f8ec2a1325904b7a235102fafe9001818e0f98854a74c866f25
-  data.tar.gz: 6b052edaf1c26baf67e499aad0457f518c4769d5e64e1cc7529296f92501bf18
+  metadata.gz: 82db8e6c6b82cca8191867896dff88204e7fee3d3c9630a2479ba01d217da73f
+  data.tar.gz: 1f86cbae0d9a88ea2d9c11547a241523b428d561ce0284779511b6cb7259fa1a
 SHA512:
-  metadata.gz: 3e8847f846bb3f3948ef1d776b67dcde77b4d0c38b910471ec98b4e12d37a288a9015b9b2accae8479812daec7154ffd38d280e403176b26c8d92c51554d4c1f
-  data.tar.gz: a1f2b2c47b0646634e9dc58b4aa3725a797d100561fdd4dc1ca8846ba8624b9db6826a3bc543ddd5add954f9a1fa62caeb3402de872da345b382ced5eabf2671
+  metadata.gz: d2e8cfc24547a64c22ece5deb69850716462ff57354563a2ad5356dbd3ef7cfc062eeb5221363560dda8bbb7f7c6d620715dc464474ba92a7d52536ee01384c0
+  data.tar.gz: aaf01c35547236f4fce81905966d44d70a6eea017bace4336e1a47fcdd4be5063a83a929354996434eee945c8c25224aee28b8a779ed89b86643e5c9afb42182

data/app/models/concerns/time_zone_aware.rb

@@ -0,0 +1,56 @@
+module TimeZoneAware
+  extend ActiveSupport::Concern
+
+  class_methods do
+    # Declares that +field+ (an existing UTC datetime column) is time-zone aware:
+    # adds a nullable, client-writable +<field>_time_zone+ column reader/validation,
+    # and exposes +<field>_server_tz+/+<field>_record_tz+ computed JSON attributes.
+    # The +<field>_time_zone+ column itself must already exist (added via migration).
+    def time_zone_aware(*fields)
+      fields.each do |field|
+        time_zone_attr = :"#{field}_time_zone"
+
+        validate do
+          TimeZoneAware.validate_zone(self, time_zone_attr)
+        end
+
+        define_method(:"#{field}_server_tz") do
+          TimeZoneAware.localize(read_attribute(field), TimeZoneAware.server_time_zone)
+        end
+
+        define_method(:"#{field}_record_tz") do
+          zone = read_attribute(time_zone_attr).presence || TimeZoneAware.server_time_zone
+          TimeZoneAware.localize(read_attribute(field), zone)
+        end
+
+        cattr_accessor :json_attrs unless respond_to?(:json_attrs)
+        self.json_attrs = ::ModelDrivenApi.smart_merge(json_attrs || {}, {
+          methods: [:"#{field}_server_tz", :"#{field}_record_tz"],
+        })
+      end
+    end
+  end
+
+  # The deployment-wide IANA time zone identifier (ThecoreSettings ns: :main, key: :time_zone),
+  # or nil when unset — callers treat a nil zone as "leave the UTC value unshifted".
+  def self.server_time_zone
+    ::Settings.ns(:main).time_zone.presence
+  end
+
+  # NULL-safe: localizes +value+ (a UTC datetime) to +zone_name+, or returns it
+  # unshifted when there is no zone to localize to (nil value, blank/invalid zone name).
+  def self.localize(value, zone_name)
+    return nil if value.nil?
+
+    zone = zone_name.present? ? ActiveSupport::TimeZone[zone_name] : nil
+    zone ? value.in_time_zone(zone) : value
+  end
+
+  def self.valid_zone?(value)
+    value.blank? || ActiveSupport::TimeZone[value].present?
+  end
+
+  def self.validate_zone(record, attr)
+    record.errors.add(attr, :invalid) unless valid_zone?(record.read_attribute(attr))
+  end
+end

data/app/models/push_message.rb

@@ -1,4 +1,8 @@
 class PushMessage < ApplicationRecord
+  include TimeZoneAware
+
+  time_zone_aware :sent_at, :received_at, :read_at
+
   belongs_to :push_subscriber
   belongs_to :sender, class_name: "User", foreign_key: :sender_user_id, optional: true
   validates :title, presence: true

data/config/initializers/application_config.rb

@@ -7,3 +7,11 @@ Rails.application.config.active_storage.routes_prefix = ENV.fetch("RAILS_RELATIV
 Rails.application.config.action_mailer.delivery_method = :smtp
 
 Rails.application.config.action_cable.allowed_request_origins = [/http:\/\/*/, /https:\/\/*/, /file:\/\/*/, 'file://', nil]
+
+# Every Thecore app ships a dedicated Sidekiq worker service alongside the web backend (see
+# thecore_devcontainer's docker-compose.yml) — this gem already hard-requires the sidekiq gems
+# and mounts Sidekiq::Web unconditionally (thecore_background_jobs), so ActiveJob should actually
+# route through it rather than defaulting to Rails' in-process :async, which silently drops any
+# job in flight when its process restarts. Production only: :async remains the default in
+# development/test so neither requires a reachable Redis. See docs/adr/0001, CONTEXT.md.
+Rails.application.config.active_job.queue_adapter = :sidekiq if Rails.env.production?

data/db/migrate/20260908000010_add_time_zone_columns_to_push_messages.rb

@@ -0,0 +1,7 @@
+class AddTimeZoneColumnsToPushMessages < ActiveRecord::Migration[7.2]
+  def change
+    add_column :push_messages, :sent_at_time_zone, :string
+    add_column :push_messages, :received_at_time_zone, :string
+    add_column :push_messages, :read_at_time_zone, :string
+  end
+end

data/lib/thecore_backend_commons/version.rb

@@ -1,3 +1,3 @@
 module ThecoreBackendCommons
-  VERSION = "3.5.1".freeze
+  VERSION = "3.7.0".freeze
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: thecore_backend_commons
 version: !ruby/object:Gem::Version
-  version: 3.5.1
+  version: 3.7.0
 platform: ruby
 authors:
 - Gabriele Tassoni
@@ -205,6 +205,7 @@ files:
 - app/channels/push_notification_channel.rb
 - app/jobs/push_dispatch_job.rb
 - app/mailers/concerns/smtp_deliverable.rb
+- app/models/concerns/time_zone_aware.rb
 - app/models/push_message.rb
 - app/models/push_subscriber.rb
 - config/initializers/abilities.rb
@@ -227,6 +228,7 @@ files:
 - db/migrate/20260616000002_create_push_messages.rb
 - db/migrate/20260625000001_add_sender_user_id_to_push_messages.rb
 - db/migrate/20260629000001_add_message_type_to_push_messages.rb
+- db/migrate/20260908000010_add_time_zone_columns_to_push_messages.rb
 - db/seeds.rb
 - lib/tasks/thecore_backend_commons_tasks.rake
 - lib/thecore_backend_commons.rb