spree_cm_commissioner 2.5.1.pre.pre1 → 2.5.1.pre.pre2

data/app/models/spree_cm_commissioner/pricing_actions/create_line_item_adjustments.rb

@@ -0,0 +1,6 @@
+module SpreeCmCommissioner
+  module PricingActions
+    class CreateLineItemAdjustments < SpreeCmCommissioner::PricingAction
+    end
+  end
+end

data/app/models/spree_cm_commissioner/pricing_rule.rb

@@ -8,6 +8,10 @@ module SpreeCmCommissioner
       raise 'eligible? should be implemented in a sub-class of SpreeCmCommissioner::PricingRule'
     end
 
+    def guest_eligible?(_guest)
+      raise 'guest_eligible? should be implemented in a sub-class of SpreeCmCommissioner::PricingRule'
+    end
+
     def self.available_rule_types
       SpreeCmCommissioner::PricingRules.constants.map(&:to_s)
     end

data/app/models/spree_cm_commissioner/pricing_rules/age_group.rb

@@ -0,0 +1,45 @@
+module SpreeCmCommissioner
+  module PricingRules
+    class AgeGroup < SpreeCmCommissioner::PricingRule
+      AGE_GROUPS = {
+        child: [0, 12],
+        adult: [13, 99]
+      }.freeze
+
+      RULE_TYPES = %w[all any none].freeze
+
+      store_accessor :private_metadata, :rule_type, :age_groups
+
+      def eligible?(line_item)
+        return false if line_item.guests.blank?
+
+        case rule_type
+        when 'all'
+          line_item.guests.all? { |guest| guest_eligible?(guest) }
+        when 'any'
+          line_item.guests.any? { |guest| guest_eligible?(guest) }
+        when 'none'
+          line_item.guests.none? { |guest| guest_eligible?(guest) }
+        else
+          false
+        end
+      end
+
+      def guest_eligible?(guest)
+        guest_age_group = resolve_guest_age_group(guest)
+        return false if guest_age_group.nil?
+
+        Array.wrap(age_groups).map(&:to_sym).include?(guest_age_group.to_sym)
+      end
+
+      private
+
+      def resolve_guest_age_group(guest)
+        return guest.age_group if guest.age_group.present?
+        return guest.saved_guest.age_group if guest.saved_guest&.age_group.present?
+
+        nil
+      end
+    end
+  end
+end

data/app/models/spree_cm_commissioner/pricing_rules/nationality.rb

@@ -14,12 +14,31 @@ module SpreeCmCommissioner
         return false if guest_nationalities.empty?
 
         case rule_type
+
         when 'all'
           guest_nationalities.all? { |n| targets.include?(n) }
         when 'any'
           guest_nationalities.any? { |n| targets.include?(n) }
         when 'none'
-          guest_nationalities.none? { |n| targets.include?(n) }
+          guest_nationalities.any? { |n| targets.exclude?(n) }
+        else
+          false
+        end
+      end
+
+      def guest_eligible?(guest)
+        return false if nationalities.blank?
+
+        targets = Array.wrap(nationalities).map(&:downcase)
+        guest_nat = guest_nationality(guest)
+
+        return false if guest_nat.blank?
+
+        case rule_type
+        when 'all', 'any'
+          targets.include?(guest_nat)
+        when 'none'
+          targets.exclude?(guest_nat)
         else
           false
         end
@@ -28,7 +47,7 @@ module SpreeCmCommissioner
       private
 
       def guest_nationality(guest)
-        guest.nationality&.name.to_s.downcase
+        (guest.nationality&.name || guest.saved_guest&.nationality&.name).to_s.downcase
       end
     end
   end

data/app/models/spree_cm_commissioner/pricing_rules/nationality_group.rb

@@ -0,0 +1,41 @@
+module SpreeCmCommissioner
+  module PricingRules
+    class NationalityGroup < SpreeCmCommissioner::PricingRule
+      RULE_TYPES = %w[all any none].freeze
+
+      store_accessor :private_metadata, :rule_type, :nationality_groups
+
+      def eligible?(line_item)
+        return false if nationality_groups.blank? || line_item.guests.blank?
+
+        case rule_type
+        when 'all'
+          line_item.guests.all? { |guest| guest_eligible?(guest) }
+        when 'any'
+          line_item.guests.any? { |guest| guest_eligible?(guest) }
+        when 'none'
+          line_item.guests.none? { |guest| guest_eligible?(guest) }
+        else
+          false
+        end
+      end
+
+      def guest_eligible?(guest)
+        guest_nationality_group = resolve_guest_nationality_group(guest)
+        return false if guest_nationality_group.nil?
+        return false if nationality_groups.blank?
+
+        Array.wrap(nationality_groups).map(&:to_sym).include?(guest_nationality_group.to_sym)
+      end
+
+      private
+
+      def resolve_guest_nationality_group(guest)
+        return guest.nationality_group if guest.nationality_group.present?
+        return guest.saved_guest.nationality_group if guest.saved_guest&.nationality_group.present?
+
+        nil
+      end
+    end
+  end
+end

data/app/models/spree_cm_commissioner/product_decorator.rb

@@ -48,6 +48,7 @@ module SpreeCmCommissioner
       base.accepts_nested_attributes_for :product_places, allow_destroy: true
 
       base.has_one :trip, class_name: 'SpreeCmCommissioner::Trip', dependent: :destroy
+      base.has_one :service_calendar, as: :calendarable, class_name: 'SpreeCmCommissioner::ServiceCalendar', dependent: :destroy
 
       base.belongs_to :event, class_name: 'Spree::Taxon', optional: true
 

data/app/models/spree_cm_commissioner/route.rb

@@ -1,12 +1,53 @@
 module SpreeCmCommissioner
   class Route < Base
-    has_many :vendor_routes, class_name: 'SpreeCmCommissioner::VendorRoute', dependent: :destroy
-    has_many :vendors, through: :vendor_routes
+    include SpreeCmCommissioner::RouteType
+
+    belongs_to :vendor, class_name: 'Spree::Vendor', optional: false
+    belongs_to :tenant, class_name: 'SpreeCmCommissioner::Tenant', optional: true
+
+    belongs_to :origin_place, class_name: 'SpreeCmCommissioner::Place', optional: false
+    belongs_to :destination_place, class_name: 'SpreeCmCommissioner::Place', optional: false
+
+    has_one :primary_photo, -> { order(position: :asc) },
+            class_name: 'SpreeCmCommissioner::RoutePhoto',
+            as: :viewable,
+            dependent: :destroy
+    has_many :route_photos, class_name: 'SpreeCmCommissioner::RoutePhoto', as: :viewable, dependent: :destroy
+
     has_many :trips, inverse_of: :route
 
-    belongs_to :origin_place, class_name: 'SpreeCmCommissioner::Place'
-    belongs_to :destination_place, class_name: 'SpreeCmCommissioner::Place'
+    validates :route_name, presence: true
+
+    validate :validate_route_stops
+
+    delegate :multi_leg?, to: :route_stops
+
+    def self.ransackable_attributes(_auth_object = nil)
+      %w[route_type short_name route_name]
+    end
+
+    def route_stops
+      Transit::RouteStopCollection.from_hash(read_attribute(:route_stops))
+    end
+
+    def route_stops=(value)
+      if value.is_a?(Transit::RouteStopCollection)
+        @route_stops = value
+        self[:route_stops] = value.to_h
+      elsif value.is_a?(Array)
+        @route_stops = Transit::RouteStopCollection.from_hash(value)
+        self[:route_stops] = @route_stops.to_h
+      else
+        raise ArgumentError, 'route_stops must be an Array or RouteStopCollection'
+      end
+    end
+
+    private
+
+    def validate_route_stops
+      return if route_stops.valid?
 
-    validates :origin_place_id, uniqueness: { scope: :destination_place_id }
+      route_stops.errors.each { |error| errors.add(:route_stops, error) }
+    end
   end
 end

data/app/models/spree_cm_commissioner/route_metric.rb

@@ -0,0 +1,21 @@
+module SpreeCmCommissioner
+  class RouteMetric < Base
+    include SpreeCmCommissioner::RouteType
+
+    belongs_to :origin_place, class_name: 'SpreeCmCommissioner::Place', optional: false
+    belongs_to :destination_place, class_name: 'SpreeCmCommissioner::Place', optional: false
+
+    validates :route_type, presence: true
+    validates :route_type, uniqueness: { scope: %i[origin_place_id destination_place_id] }
+
+    validate :origin_and_destination_cannot_be_the_same
+
+    private
+
+    def origin_and_destination_cannot_be_the_same
+      return unless origin_place_id == destination_place_id
+
+      errors.add(:base, 'Origin and destination cannot be the same')
+    end
+  end
+end

data/app/models/spree_cm_commissioner/route_photo.rb

@@ -0,0 +1,12 @@
+module SpreeCmCommissioner
+  class RoutePhoto < Asset
+    # 16x9
+    def asset_styles
+      {
+        mini: '160x90>',
+        small: '480x270>',
+        medium: '960x540>'
+      }
+    end
+  end
+end

data/app/models/spree_cm_commissioner/taxon_decorator.rb

@@ -77,6 +77,7 @@ module SpreeCmCommissioner
       base.has_many :taxon_places, class_name: 'SpreeCmCommissioner::TaxonPlace', dependent: :destroy
       base.has_many :places, through: :taxon_places
       base.has_many :check_in_sessions, class_name: 'SpreeCmCommissioner::CheckInSession', foreign_key: :event_id, dependent: :destroy
+      base.has_many :active_check_in_sessions, -> { active }, class_name: 'SpreeCmCommissioner::CheckInSession', foreign_key: :event_id
 
       base.has_many :import_orders, as: :importable, class_name: 'SpreeCmCommissioner::Imports::ImportOrder', dependent: :destroy
 

data/app/models/spree_cm_commissioner/trip.rb

@@ -2,7 +2,6 @@ module SpreeCmCommissioner
   class Trip < Base
     include SpreeCmCommissioner::StoreMetadata
     include SpreeCmCommissioner::RouteType
-    include SpreeCmCommissioner::RouteTripCountCallbacks
 
     attr_accessor :hours, :minutes, :seconds
 
@@ -15,7 +14,9 @@ module SpreeCmCommissioner
 
     # before create, duplicate seat layout from vehicle_type if present
     before_validation :duplicate_seat_layout_from_vehicle, if: -> { seat_layout.nil? && vehicle_type&.seat_layout.present? }
-    before_validation :assign_route_and_vendor_route, if: :changed_route_attributes?
+
+    after_create :increment_route_metric_trip_count
+    before_destroy :decrement_route_metric_trip_count
 
     belongs_to :vendor, class_name: 'Spree::Vendor', inverse_of: :trips, optional: false
     belongs_to :product, class_name: 'Spree::Product', inverse_of: :trip, optional: false
@@ -26,7 +27,7 @@ module SpreeCmCommissioner
 
     belongs_to :origin_place, class_name: 'SpreeCmCommissioner::Place', optional: false
     belongs_to :destination_place, class_name: 'SpreeCmCommissioner::Place', optional: false
-    belongs_to :route, inverse_of: :trips
+    belongs_to :route, class_name: 'SpreeCmCommissioner::Route', inverse_of: :trips, optional: true
 
     has_many :trip_blazer_queries, as: :queryable, class_name: 'SpreeCmCommissioner::BlazerQueryable'
     has_many :blazer_queries, through: :trip_blazer_queries, source: :blazer_query, class_name: 'Blazer::Query'
@@ -48,10 +49,6 @@ module SpreeCmCommissioner
     self.whitelisted_ransackable_associations = %w[product vehicle vehicle_type]
     self.whitelisted_ransackable_attributes = %w[origin_place_id destination_place_id]
 
-    def changed_route_attributes?
-      origin_place_id_changed? || destination_place_id_changed? || vendor_id_changed?
-    end
-
     def convert_duration_to_seconds
       return if hours.blank? && minutes.blank? && seconds.blank?
 
@@ -104,34 +101,12 @@ module SpreeCmCommissioner
                 )
     end
 
-    # Check if any of the route context attributes are changing
-    def will_route_context_change?
-      # Returns true if any of the route context attributes will change on save.
-      will_save_change_to_route_id? ||
-        will_save_change_to_origin_place_id? ||
-        will_save_change_to_destination_place_id? ||
-        will_save_change_to_vendor_id?
-    end
-
-    def changed_route_context?
-      saved_change_to_route_id? ||
-        saved_change_to_origin_place_id? ||
-        saved_change_to_destination_place_id? ||
-        saved_change_to_vendor_id?
+    def increment_route_metric_trip_count
+      SpreeCmCommissioner::RouteMetrics::IncreaseTripCountJob.perform_later(trip_id: id)
     end
 
-    # Find or create the Route for this trip (by origin/destination) and
-    # assign it to the trip. Also populate a default route_name from the
-    # product when the route is new/blank.
-    def assign_route_and_vendor_route
-      route = SpreeCmCommissioner::Route.find_or_create_by!(
-        origin_place_id: origin_place_id,
-        destination_place_id: destination_place_id
-      )
-      route.route_name = product.name if route.route_name.blank?
-      route.save! if route.changed?
-
-      self.route_id = route.id
+    def decrement_route_metric_trip_count
+      SpreeCmCommissioner::RouteMetrics::DecreaseTripCountJob.perform_later(trip_id: id)
     end
   end
 end

data/app/models/spree_cm_commissioner/trip_stop.rb

@@ -1,25 +1,39 @@
 module SpreeCmCommissioner
   class TripStop < Base
     acts_as_list column: :sequence, scope: :trip_id
-    enum :stop_type, { boarding: 0, drop_off: 1 }
 
     belongs_to :trip, class_name: 'SpreeCmCommissioner::Trip', optional: false
     belongs_to :stop_place, class_name: 'SpreeCmCommissioner::Place', optional: false
     belongs_to :location_place, class_name: 'SpreeCmCommissioner::Place', optional: false
 
+    belongs_to :board_to_trip, class_name: 'SpreeCmCommissioner::Trip', optional: true
+
     before_validation :set_stop_name
+
     validates :stop_place_id, uniqueness: { scope: :trip_id }
+    validate :boarding_trip_requires_allow_boarding
+
+    scope :boarding, -> { where(allow_boarding: true) }
+    scope :drop_off, -> { where(allow_drop_off: true) }
 
     def set_stop_name
       self.stop_name = stop_place.name if stop_place.present?
     end
 
     def self.ransackable_attributes(_auth_object = nil)
-      %w[stop_name stop_type]
+      %w[stop_name allow_boarding allow_drop_off]
     end
 
     def self.ransackable_associations(_auth_object = nil)
       ['stop_place']
     end
+
+    private
+
+    def boarding_trip_requires_allow_boarding
+      return unless board_to_trip_id.present? && !allow_boarding?
+
+      errors.add(:board_to_trip, 'requires allow_boarding to be true')
+    end
   end
 end

data/app/models/spree_cm_commissioner/vendor_decorator.rb

@@ -82,7 +82,9 @@ module SpreeCmCommissioner
       base.has_many :subscriptions, through: :customers, class_name: 'SpreeCmCommissioner::Subscription'
       base.has_many :subscription_orders, through: :subscriptions, class_name: 'Spree::Order', source: :orders
       base.has_many :promotion_categories, class_name: 'Spree::PromotionCategory', foreign_key: :vendor_id, dependent: :destroy
-      base.has_many :vendor_routes, class_name: 'SpreeCmCommissioner::VendorRoute'
+      base.has_many :routes, class_name: 'SpreeCmCommissioner::Route', dependent: :destroy
+      base.has_many :popular_routes, -> { order(fulfilled_order_count: :desc, order_count: :desc) },
+                    class_name: 'SpreeCmCommissioner::Route'
 
       base.has_many :homepage_section_relatables,
                     as: :relatable,

data/app/queries/spree_cm_commissioner/trip_query.rb

@@ -77,8 +77,8 @@ module SpreeCmCommissioner
 
       scope
         .joins(<<~SQL.squish)
-          INNER JOIN cm_trip_stops AS boarding ON boarding.trip_id = cm_trips.id AND boarding.stop_type = 0
-          INNER JOIN cm_trip_stops AS drop_off ON drop_off.trip_id = cm_trips.id AND drop_off.stop_type = 1
+          INNER JOIN cm_trip_stops AS boarding ON boarding.trip_id = cm_trips.id AND boarding.allow_boarding = true
+          INNER JOIN cm_trip_stops AS drop_off ON drop_off.trip_id = cm_trips.id AND drop_off.allow_drop_off = true
           INNER JOIN cm_places AS origin_places ON origin_places.id = cm_trips.origin_place_id
           INNER JOIN cm_places AS dest_places ON dest_places.id = cm_trips.destination_place_id
           INNER JOIN spree_variants AS master ON master.product_id = cm_trips.product_id AND master.is_master = true

data/app/serializers/spree/v2/tenant/transit_cart_serializer.rb

@@ -0,0 +1,11 @@
+module Spree
+  module V2
+    module Tenant
+      class TransitCartSerializer < CartSerializer
+        has_many :line_items, serializer: SpreeCmCommissioner::V2::Storefront::TransitLineItemSerializer
+
+        cache_options store: nil
+      end
+    end
+  end
+end

data/app/serializers/spree_cm_commissioner/v2/operator/check_in_serializer.rb

@@ -2,8 +2,6 @@ module SpreeCmCommissioner
   module V2
     module Operator
       class CheckInSerializer < BaseSerializer
-        set_type :check_in
-
         attributes :guest_id,
                    :verification_state,
                    :check_in_type,
@@ -13,6 +11,7 @@ module SpreeCmCommissioner
                    :token
 
         belongs_to :check_in_by, serializer: ::Spree::V2::Storefront::UserSerializer
+        belongs_to :check_in_session, serializer: ::SpreeCmCommissioner::V2::Operator::CheckInSessionSerializer
         belongs_to :guest, serializer: ::SpreeCmCommissioner::V2::Operator::GuestSerializer
         belongs_to :line_item, serializer: ::SpreeCmCommissioner::V2::Operator::LineItemSerializer
       end

data/app/serializers/spree_cm_commissioner/v2/operator/check_in_session_serializer.rb

@@ -0,0 +1,9 @@
+module SpreeCmCommissioner
+  module V2
+    module Operator
+      class CheckInSessionSerializer < BaseSerializer
+        attributes :name, :description, :event_id, :start_time, :end_time, :is_archived, :position
+      end
+    end
+  end
+end

data/app/serializers/spree_cm_commissioner/v2/operator/guest_serializer.rb

@@ -13,7 +13,7 @@ module SpreeCmCommissioner
 
         belongs_to :occupation, serializer: Spree::V2::Storefront::TaxonSerializer
 
-        has_one :check_in, serializer: SpreeCmCommissioner::V2::Operator::CheckInSerializer
+        has_many :check_ins, serializer: SpreeCmCommissioner::V2::Operator::CheckInSerializer
         has_one :line_item, serializer: SpreeCmCommissioner::V2::Operator::LineItemSerializer
       end
     end

data/app/serializers/spree_cm_commissioner/v2/storefront/guest_serializer.rb

@@ -27,7 +27,8 @@ module SpreeCmCommissioner
         has_many :guest_dynamic_fields, serializer: SpreeCmCommissioner::V2::Storefront::GuestDynamicFieldSerializer
 
         has_one :id_card, serializer: Spree::V2::Storefront::IdCardSerializer
-        has_one :check_in
+
+        has_many :check_ins, serializer: SpreeCmCommissioner::V2::Storefront::CheckInSerializer
 
         # allowed_checkout updates frequently
         cache_options store: nil

data/app/serializers/spree_cm_commissioner/v2/storefront/route_serializer.rb

@@ -2,8 +2,10 @@ module SpreeCmCommissioner
   module V2
     module Storefront
       class RouteSerializer < BaseSerializer
-        attributes :route_name
+        attributes :route_name, :route_type, :short_name
 
+        has_many :route_photos, serializer: ::SpreeCmCommissioner::V2::Storefront::AssetSerializer,
+                                if: proc { |record, _params| record.respond_to?(:route_photos) }
         has_one :origin_place, serializer: ::Spree::V2::Storefront::PlaceSerializer
         has_one :destination_place, serializer: ::Spree::V2::Storefront::PlaceSerializer
       end

data/app/serializers/spree_cm_commissioner/v2/storefront/transit_line_item_serializer.rb

@@ -0,0 +1,17 @@
+module SpreeCmCommissioner
+  module V2
+    module Storefront
+      class TransitLineItemSerializer < BaseSerializer
+        # Nested-only API shape
+        attributes :from_date,
+                   :to_date,
+                   :trip_id,
+                   :direction,
+                   :boarding_trip_stop_id,
+                   :drop_off_trip_stop_id
+
+        cache_options store: nil
+      end
+    end
+  end
+end

data/app/serializers/spree_cm_commissioner/v2/storefront/trip_stop_serializer.rb

@@ -2,7 +2,7 @@ module SpreeCmCommissioner
   module V2
     module Storefront
       class TripStopSerializer < BaseSerializer
-        attributes :id, :stop_type, :stop_name, :arrival_time, :departure_time
+        attributes :id, :allow_boarding, :allow_drop_off, :stop_name, :arrival_time, :departure_time
 
         has_one :stop_place, serializer: ::SpreeCmCommissioner::V2::Storefront::TripPlaceSerializer
       end

data/app/services/spree_cm_commissioner/check_ins/create_bulk.rb

@@ -0,0 +1,65 @@
+module SpreeCmCommissioner
+  module CheckIns
+    class CreateBulk
+      prepend Spree::ServiceModule::Base
+
+      def call(check_ins_attributes:, check_in_by: nil, confirmed_at: nil)
+        return failure(:guest_ids_must_not_blank) if check_ins_attributes.blank?
+
+        confirmed_at ||= Time.current
+        guest_ids = check_ins_attributes.pluck(:guest_id)
+        indexed_guests = SpreeCmCommissioner::Guest.where(id: guest_ids).index_by(&:id)
+
+        check_ins = ActiveRecord::Base.transaction do
+          check_ins_attributes.map do |attrs|
+            guest = indexed_guests[attrs[:guest_id].to_i]
+            raise ActiveRecord::RecordNotFound, "Couldn't find Guest with 'id'=#{attrs[:guest_id]}" if guest.blank?
+
+            check_in = check_in!(guest, attrs, check_in_by, confirmed_at)
+            update_guest!(guest, attrs[:guest_attributes]) if attrs[:guest_attributes].present?
+
+            check_in
+          end
+        end
+
+        success(check_ins: check_ins)
+      rescue ActiveRecord::RecordInvalid => e
+        failure(:invalid_record, e.record.errors.full_messages.join(', '))
+      rescue ActiveRecord::RecordNotFound => e
+        failure(:record_not_found, e.message)
+      end
+
+      private
+
+      def check_in!(guest, check_in_attributes, check_in_by, confirmed_at)
+        check_in = SpreeCmCommissioner::CheckIn.new(
+          guest: guest,
+          check_in_session_id: check_in_attributes[:check_in_session_id],
+          check_in_by: check_in_by,
+          checkinable: guest.event,
+          confirmed_at: check_in_attributes[:confirmed_at] || confirmed_at
+        )
+
+        # save will validate attributes, uniqueness of check-in etc.
+        check_in.save!
+
+        # Track state change with session ID in the name for audit trail
+        name = check_in_attributes[:check_in_session_id].present? ? "guest_#{check_in_attributes[:check_in_session_id]}" : 'guest'
+        guest.state_changes.create!(
+          user: check_in_by,
+          previous_state: 'unchecked_in',
+          next_state: 'checked_in',
+          name: name
+        )
+
+        check_in
+      end
+
+      def update_guest!(guest, guest_attributes)
+        return if guest_attributes.blank?
+
+        guest.update!(guest_attributes)
+      end
+    end
+  end
+end

data/app/services/spree_cm_commissioner/check_ins/destroy_bulk.rb

@@ -0,0 +1,54 @@
+module SpreeCmCommissioner
+  module CheckIns
+    class DestroyBulk
+      prepend Spree::ServiceModule::Base
+
+      def call(check_ins_attributes:, destroyed_by: nil)
+        return failure(:guest_ids_must_not_blank) if check_ins_attributes.blank?
+
+        check_ins = []
+
+        ActiveRecord::Base.transaction do
+          check_ins = check_ins_attributes.map do |attrs|
+            guest = SpreeCmCommissioner::Guest.find(attrs[:guest_id])
+            destroy_check_in!(guest, attrs, destroyed_by)
+          end.compact
+
+          success(check_ins: check_ins)
+        end
+      rescue ActiveRecord::RecordNotFound => e
+        failure(:record_not_found, e.message)
+      rescue ActiveRecord::RecordInvalid => e
+        failure(:invalid_record, e.record.errors.full_messages.join(', '))
+      rescue StandardError => e
+        failure(:failed_to_destroy_check_ins, e.message)
+      end
+
+      private
+
+      def destroy_check_in!(guest, check_in_attributes, destroyed_by)
+        # Find the check-in to destroy
+        check_in = if check_in_attributes[:check_in_session_id].present?
+                     guest.check_ins.find_by(check_in_session_id: check_in_attributes[:check_in_session_id])
+                   else
+                     guest.earliest_check_in
+                   end
+
+        return nil if check_in.blank?
+
+        check_in.destroy!
+
+        # Track state change with session ID in the name for audit trail
+        name = check_in_attributes[:check_in_session_id].present? ? "guest_#{check_in_attributes[:check_in_session_id]}" : 'guest'
+        guest.state_changes.create!(
+          user: destroyed_by,
+          previous_state: 'checked_in',
+          next_state: 'unchecked_in',
+          name: name
+        )
+
+        check_in
+      end
+    end
+  end
+end

data/app/services/spree_cm_commissioner/guests/preload_check_in_session_ids.rb

@@ -0,0 +1,22 @@
+module SpreeCmCommissioner
+  module Guests
+    class PreloadCheckInSessionIds
+      prepend ::Spree::ServiceModule::Base
+
+      def call(event:)
+        return failure(nil, 'Event cannot be blank') if event.blank?
+
+        event.guests
+             .includes(event: { check_in_sessions: :check_in_rules })
+             .find_each(batch_size: 100) do |guest|
+          guest.preload_eligible_check_in_session_ids
+          guest.save(validate: false) # Skip validations for performance
+        end
+
+        success(true)
+      rescue StandardError => e
+        failure(nil, e.message)
+      end
+    end
+  end
+end