spree_cm_commissioner 2.5.0.pre.pre4 → 2.5.0.pre.pre7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ba91dacd894816a0b4105c032a0550371aeab884b3f344ebe554758c3556a0a0
-  data.tar.gz: 077d13272adb0ced0f246dc9d48d5d3b9ae56a95beaa74a6d2cd6fc7f134f7e1
+  metadata.gz: cd65c0e3f9c574eac1886f0dd7872516ee4b06ce9de12c91cd56e9101ea37914
+  data.tar.gz: 223b2e1bc19876a282cf764cbfd45da3aeb3e13b671a9bafeaf697bed5938380
 SHA512:
-  metadata.gz: 8a990d42d6cf09c9304971a41efb74525761de29ee6cefe04e1a7b0dbe2831e9ad40f52f229ca463ee905ed1e039b1562daa2eace5307c4dd3eb6f725488e671
-  data.tar.gz: 309280aeb9987456b2788be11cdad59095da58b2b258e294477cfb5edb63323b4ee6d3fd8ae03f845fd79db0d0186ff54d64f6cb406567c2a284dc26454446f8
+  metadata.gz: 959b4855a961019439474b8dc5fb987650e65ecb96c6e3b9a12355d3487d6d067c5824317cc825c882c858cee64c23e21695448999c3caf6b852e7c7d73fbfdc
+  data.tar.gz: c3fe124a224836d629669bf723d80162253e1e732512870d1ddfbaf8bd2c115bb081eb058e7f9dc2acc1bdbe23b425bf9ad19ad2134d31c523c11163016eee0f

data/Gemfile.lock

@@ -34,7 +34,7 @@ GIT
 PATH
   remote: .
   specs:
-    spree_cm_commissioner (2.5.0.pre.pre4)
+    spree_cm_commissioner (2.5.0.pre.pre7)
       activerecord-multi-tenant
       activerecord_json_validator (~> 2.1, >= 2.1.3)
       aws-sdk-cloudfront

data/app/interactors/spree_cm_commissioner/stock/stock_movement_creator.rb

@@ -23,9 +23,8 @@ module SpreeCmCommissioner
       private
 
       def adjust_inventory_items_async(variant_id, quantity)
-        args = { variant_id: variant_id, quantity: quantity }
-        CmAppLogger.log(label: "#{self.class.name}#adjust_inventory_items_async", data: args) do
-          SpreeCmCommissioner::Stock::InventoryItemsAdjusterJob.perform_later(**args)
+        CmAppLogger.log(label: "#{self.class.name}#adjust_inventory_items_async", data: { variant_id: variant_id, quantity: quantity }) do
+          SpreeCmCommissioner::Stock::InventoryItemsAdjusterJob.perform_later(variant_id: variant_id, quantity: quantity)
         end
       end
     end

data/app/jobs/spree_cm_commissioner/stock/inventory_items_adjuster_job.rb

@@ -2,10 +2,7 @@ module SpreeCmCommissioner
   module Stock
     class InventoryItemsAdjusterJob < ApplicationUniqueJob
       def perform(options = {})
-        variant = Spree::Variant.find_by(id: options[:variant_id])
-
-        # potentially the variant was deleted during the job wait time, we just skip in that case.
-        return if variant.blank?
+        variant = Spree::Variant.find(options[:variant_id])
 
         SpreeCmCommissioner::Stock::InventoryItemsAdjuster.call(variant: variant, quantity: options[:quantity])
       end

data/lib/cm_app_logger.rb

@@ -1,18 +1,18 @@
 # lib/cm_app_logger.rb
 module CmAppLogger
   def self.log(label:, data: nil)
-    message = { label: label, data: data }
+    message = { label: label, data: safe_serialize(data) }
     start_time = Time.current
     Rails.logger.info(message.to_json)
 
     return unless block_given?
 
-    # Capture the block’s return value and return it to preserve existing behavior for callers expecting that value.
+    # Capture the block's return value and return it to preserve existing behavior for callers expecting that value.
     block_result = yield
 
     message[:start_time] = start_time
     message[:duration_ms] = (Time.current - start_time) * 1000
-    message[:result] = block_result
+    message[:result] = safe_serialize(block_result)
     Rails.logger.info(message.to_json)
     block_result
   end
@@ -20,9 +20,26 @@ module CmAppLogger
   def self.error(label:, data: nil)
     message = {
       label: label,
-      data: data
+      data: safe_serialize(data)
     }
 
     Rails.logger.error(message.to_json)
   end
+
+  def self.safe_serialize(obj)
+    case obj
+    when Hash
+      obj.transform_values { |v| safe_serialize(v) }
+    when Array
+      obj.map { |item| safe_serialize(item) }
+    when NilClass, TrueClass, FalseClass, Numeric, String, Symbol
+      obj
+    when ActiveJob::Base
+      # Don't serialize the entire job object - just its class and ID
+      { job_class: obj.class.name, job_id: obj.job_id }
+    else
+      # For other objects, try to_s as fallback
+      obj.respond_to?(:id) ? { class: obj.class.name, id: obj.id } : obj.to_s
+    end
+  end
 end

data/lib/spree_cm_commissioner/version.rb

@@ -1,5 +1,5 @@
 module SpreeCmCommissioner
-  VERSION = '2.5.0-pre4'.freeze
+  VERSION = '2.5.0-pre7'.freeze
 
   module_function
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: spree_cm_commissioner
 version: !ruby/object:Gem::Version
-  version: 2.5.0.pre.pre4
+  version: 2.5.0.pre.pre7
 platform: ruby
 authors:
 - You