spree_cm_commissioner 2.8.1.pre.3 → 2.8.1.pre.pre.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8250d9a96dcceb058991c416ad4b91380fa42f5d43e301c366e74bfe9eb0e27b
-  data.tar.gz: a8c9301047c15f6ef760bdda1cc9087ac4f7a3539cd6aa86b43a99ab97fc6f16
+  metadata.gz: ed6326825a04e2ef186e72511e895b9fbf8598858d7d8f8e4358c0dbc0bc6f7e
+  data.tar.gz: ee5cdc0fd119a84c4619596f5db5a6a2aef89b39e0be982f71612a68bd8bbd16
 SHA512:
-  metadata.gz: 87bba18fe41a801abf62fc7a83c444452f5b3780b23edaa44bbf86b98271b06877bc8c52bddfc5761b5fdee5953226201c52556cfba30405c1e1f95af2067861
-  data.tar.gz: 2a7fd81827d6be81291146d17f46ff54a61a59aa4194a552f89ccc59463dc93a4ccc742c0534fd5c18354136314b2e9db571e97b6e8822fff3bbc6c116f3114d
+  metadata.gz: 7975ae665f017e022b41297314d72e25ac872241e5e224f5581cea5a736ef7e4e8abee4a8fa71916b038661da894a116c15ceeb71c03e4259627e44fff11222f
+  data.tar.gz: 6fa46d5991c849243fdac985bed14ad86d88404adf5880d0e8000c48867f27e6f20c08973bbe41c293e200650ac7b5416a0e4fc0b628a5d1a9b722062101f8c9

data/.env.example

@@ -34,6 +34,7 @@ DISTANCE_SIGNING_KEY=hei********************VY
 
 # Cache durations (in seconds) for different content types
 # See: app/controllers/concerns/spree_cm_commissioner/content_cachable.rb
+CACHE_AUTO_INVALIDATION_DISABLED="" # Set to yes to disable CDN cache invalidation entirely (kill switch)
 CACHE_CDN_MAX_AGE=86400 # CDN/server-side cache duration for all responses
 CACHE_STATIC_MAX_AGE=86400 # Static content (countries, provinces, seat layouts, CMS pages) - 1 day
 CACHE_SEMI_STATIC_MAX_AGE=3600 # Semi-static content (menus, homepage backgrounds, routes) - 1 hour

data/Gemfile.lock

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

data/app/controllers/spree/api/v2/storefront/trip_search_controller.rb

@@ -39,7 +39,6 @@ module Spree
               params[:date],
               params[:route_type],
               params[:vendor_id],
-              params[:featured],
               params[:number_of_guests],
               resource_includes&.sort&.join(','),
               sparse_fields&.sort&.join(','),

data/app/controllers/spree/api/v2/tenant/eliminated_contestants_controller.rb

@@ -0,0 +1,19 @@
+module Spree
+  module Api
+    module V2
+      module Tenant
+        class EliminatedContestantsController < ShowContestantsController
+          private
+
+          def scope
+            super.eliminated
+          end
+
+          def collection_serializer
+            Spree::V2::Tenant::ShowContestantSerializer
+          end
+        end
+      end
+    end
+  end
+end

data/app/controllers/spree/api/v2/tenant/show_contestants_controller.rb

@@ -18,6 +18,7 @@ module Spree
               .where(show_id: current_show.id)
               .includes(
                 :show_contestant_images,
+                :show_contestant_videos,
                 :current_voting_session,
                 :voting_contestants,
                 assigned_show_person: :show_person_images,

data/app/factory/spree_cm_commissioner/event_telegram_message_factory.rb

@@ -0,0 +1,68 @@
+# 🎉 ---[Event]---
+#
+# 📋 Event: {{event.name}}
+#
+# 👤 Organizer: {{vendor.name}}
+#
+# 🗓️ Start Date: {{from_date}}
+# 🗓️ End Date: {{to_date}}
+# 🔗 URL: ......
+#
+module SpreeCmCommissioner
+  class EventTelegramMessageFactory < TelegramMessageFactory
+    attr_reader :event
+
+    def initialize(event:)
+      @event = event
+
+      title = "✨ #{event.name} ✨"
+
+      super(title: title)
+    end
+
+    def body
+      [
+        event_name_line,
+        organizer_line,
+        [start_date_line, end_date_line].compact.join("\n"),
+        event_url_line
+      ].compact_blank.join("\n\n")
+    end
+
+    private
+
+    def event_name_line
+      "📋 Event: #{event.name}" if event.name.present?
+    end
+
+    def organizer_line
+      return if event.vendor&.name.blank?
+
+      "👤 Organizer: #{bold(event.vendor.name)}"
+    end
+
+    def start_date_line
+      return if event.from_date.blank?
+
+      "🗓️ Start Date: #{pretty_datetime(event.from_date)}"
+    end
+
+    def end_date_line
+      return if event.to_date.blank?
+
+      "🗓️ End Date: #{pretty_datetime(event.to_date)}"
+    end
+
+    def pretty_datetime(date)
+      return '' if date.blank?
+
+      date.strftime('%b %d, %Y · %H:%M')
+    end
+
+    def event_url_line
+      return if event.event_url.blank?
+
+      "🔗 URL: #{event.event_url}"
+    end
+  end
+end

data/app/finders/spree_cm_commissioner/accommodations/find_variant.rb

@@ -15,8 +15,8 @@ module SpreeCmCommissioner
           .joins(:inventory_items)
           .where(vendor_id: vendor_id)
           .where(inventory_items: { inventory_date: stay_dates })
-          .where('CAST(spree_variants.public_metadata->\'cm_options\'->>\'number-of-adults\' AS INTEGER) +
-                  CAST(spree_variants.public_metadata->\'cm_options\'->>\'number-of-kids\' AS INTEGER) >= ?', number_of_guests
+          .where('CAST(public_metadata->\'cm_options\'->>\'number-of-adults\' AS INTEGER) +
+                  CAST(public_metadata->\'cm_options\'->>\'number-of-kids\' AS INTEGER) >= ?', number_of_guests
           )
           .where('inventory_items.quantity_available > 0')
           .distinct

data/app/jobs/spree_cm_commissioner/event_creation_telegram_alert_sender_job.rb

@@ -0,0 +1,10 @@
+module SpreeCmCommissioner
+  class EventCreationTelegramAlertSenderJob < ApplicationUniqueJob
+    queue_as :telegram_bot
+
+    def perform(options = {})
+      event = Spree::Taxon.find(options[:event_id])
+      SpreeCmCommissioner::TelegramAlerts::EventCreation.call(event: event)
+    end
+  end
+end

data/app/jobs/spree_cm_commissioner/oauth_access_tokens/cleanup_expired_job.rb

@@ -0,0 +1,11 @@
+module SpreeCmCommissioner
+  module OauthAccessTokens
+    class CleanupExpiredJob < SpreeCmCommissioner::ApplicationJob
+      queue_as :default
+
+      def perform
+        SpreeCmCommissioner::OauthAccessTokens::CleanupExpired.new.call
+      end
+    end
+  end
+end

data/app/models/concerns/spree_cm_commissioner/store_preference.rb

@@ -7,6 +7,7 @@ module SpreeCmCommissioner
       preference :telegram_order_alert_chat_id, :string
       preference :telegram_order_request_alert_chat_id, :string
       preference :telegram_new_vendor_alert_chat_id, :string
+      preference :telegram_new_event_alert_chat_id, :string
       preference :assetlinks, :string, default: ''
       preference :apple_app_site_association, :string, default: ''
     end

data/app/models/concerns/spree_cm_commissioner/vehicle_kind.rb

@@ -2,7 +2,7 @@ module SpreeCmCommissioner
   module VehicleKind
     extend ActiveSupport::Concern
 
-    VEHICLE_KINDS = %i[suv sedan minivan van sleeping_bus luxury_van air_bus bus ferry boat].freeze
+    VEHICLE_KINDS = %i[suv sedan minivan van sleeping_bus luxury_van air_bus bus ferry].freeze
 
     included do
       enum kind: VEHICLE_KINDS if table_exists? && column_names.include?('kind')

data/app/models/spree_cm_commissioner/integration.rb

@@ -25,36 +25,5 @@ module SpreeCmCommissioner
     def unstock_external_inventory!(_order, _line_items)
       raise NotImplementedError, 'Subclasses must implement the unstock_external_inventory! method'
     end
-
-    def sync_item_availability!(_options = {})
-      raise NotImplementedError, 'Subclasses must implement the sync_item_availability! method'
-    end
-
-    def hold_inventory!(_order, _line_items)
-      raise NotImplementedError, 'Subclasses must implement the hold_inventory! method'
-    end
-
-    # Integrations that manage external inventory reservations should override this to return true
-    def supports_external_inventory_holding?
-      false
-    end
-
-    # Integrations that support automatically releasing external inventory
-    # after reservation expiration or order cancellation should override this to return true
-    def supports_auto_release_inventory?
-      false
-    end
-
-    def release_inventory!(_order, _line_items)
-      raise NotImplementedError, 'Subclasses must implement the release_inventory! method'
-    end
-
-    def external_inventory_items(_order)
-      raise NotImplementedError, 'Subclasses must implement the external_inventory_items method'
-    end
-
-    def external_inventory_item_status(_order, _options)
-      raise NotImplementedError, 'Subclasses must implement the external_inventory_item_status method'
-    end
   end
 end

data/app/models/spree_cm_commissioner/integrations/stadium_x_v1.rb

@@ -5,7 +5,10 @@ class SpreeCmCommissioner::Integrations::StadiumXV1 < SpreeCmCommissioner::Integ
 
   # override
   def sync_manager
-    SpreeCmCommissioner::Integrations::StadiumXV1::SyncManager.new(integration: self)
+    SpreeCmCommissioner::Integrations::StadiumXV1::SyncManager.new(
+      integration: self,
+      client: client
+    )
   end
 
   # override
@@ -15,6 +18,7 @@ class SpreeCmCommissioner::Integrations::StadiumXV1 < SpreeCmCommissioner::Integ
       order: order,
       line_items: line_items
     )
+
     raise SpreeCmCommissioner::Integrations::SyncError, result.error unless result.success?
   end
 
@@ -23,9 +27,11 @@ class SpreeCmCommissioner::Integrations::StadiumXV1 < SpreeCmCommissioner::Integ
     raise SpreeCmCommissioner::Integrations::SyncError, 'Ticket cannot be cancelled'
   end
 
-  def sync_item_availability!(_options = {}); end
-
   def client
-    @client ||= SpreeCmCommissioner::Integrations::StadiumXV1::ExternalClient::Client.new(integration: self)
+    SpreeCmCommissioner::Integrations::StadiumXV1::ExternalClient::Client.new(
+      public_key: public_key,
+      private_key: private_key,
+      base_url: base_url
+    )
   end
 end

data/app/models/spree_cm_commissioner/inventory_item.rb

@@ -1,15 +1,9 @@
 module SpreeCmCommissioner
   class InventoryItem < Base
     include SpreeCmCommissioner::ProductType
-    include SpreeCmCommissioner::StoreMetadata
-    include SpreeCmCommissioner::Integrations::IntegrationMappable
 
     MAX_DISPLAY_STOCK = 20
 
-    store_public_metadata :last_synced_at, :datetime
-    store_public_metadata :original_price, :integer
-    store_public_metadata :currency, :string
-
     # Association
     belongs_to :variant, class_name: 'Spree::Variant', optional: false
 
@@ -29,8 +23,23 @@ module SpreeCmCommissioner
 
     before_save -> { self.product_type = variant.product_type }, if: -> { product_type.nil? }
 
-    # Create maintaining task to purge product related caches
-    after_commit { SpreeCmCommissioner::MaintenanceTasks::CacheInvalidation.pending.create_or_find_by(maintainable: variant.product) }
+    after_commit :schedule_product_cache_invalidation
+
+    # Only for ecommerce: the app uses product.in_stock? (derived from quantity_available) for product cards.
+    # Other types (e.g. accommodation, bus) have many inventory items (including daily-generated ones),
+    # so invalidating per item would flood the task queue.
+    #
+    # This is temporary until we have a more robust cache invalidation strategy in place.
+    def schedule_product_cache_invalidation
+      return unless ecommerce?
+      return unless saved_change_to_quantity_available?
+
+      # Only fires on in_stock status transition (0 ↔ N); non-zero to non-zero changes are skipped.
+      old_val, new_val = saved_change_to_quantity_available
+      return if old_val.to_i.positive? && new_val.to_i.positive?
+
+      SpreeCmCommissioner::MaintenanceTasks::CacheInvalidation.pending.create_or_find_by(maintainable: variant.product)
+    end
 
     def public_quantity_available
       [quantity_available, MAX_DISPLAY_STOCK].min

data/app/models/spree_cm_commissioner/maintenance_tasks/cache_invalidation.rb

@@ -3,6 +3,8 @@ module SpreeCmCommissioner
   module MaintenanceTasks
     class CacheInvalidation < MaintenanceTask
       def maintain
+        return if ENV['CACHE_AUTO_INVALIDATION_DISABLED'] == 'yes'
+
         if maintainable.is_a?(SpreeCmCommissioner::HomepageBackground) || maintainable.is_a?(Spree::Menu)
           SpreeCmCommissioner::HomepageDataLoader.clear_cache
         end

data/app/models/spree_cm_commissioner/maintenance_tasks/voting_session.rb

@@ -7,6 +7,7 @@
 # `maintain` runs three steps in order:
 #   1. RebuildFromDb  — overwrites Redis counters with DB totals (source of truth)
 #   2. AuditCounters  — logs any remaining mismatches between Redis and DB
+#   3. SnapshotToDb   — writes final aggregates to cm_voting_contestants and cm_voting_session_stats
 #
 # maintainable: SpreeCmCommissioner::VotingSession
 #
@@ -19,6 +20,7 @@ module SpreeCmCommissioner
         ttl = rebuild_ttl
         SpreeCmCommissioner::VoteCounters::RebuildFromDb.call(voting_session_id: maintainable.id, ttl: ttl)
         SpreeCmCommissioner::VoteCounters::AuditCounters.call(voting_session_id: maintainable.id)
+        SpreeCmCommissioner::VoteCounters::SnapshotToDb.call(voting_session_id: maintainable.id)
       end
 
       # For closed sessions use the fixed 24h constant — closes_at is in the past

data/app/models/spree_cm_commissioner/order_decorator.rb

@@ -42,7 +42,6 @@ module SpreeCmCommissioner
       base.has_many :reserved_blocks, through: :guests, class_name: 'SpreeCmCommissioner::ReservedBlock'
       base.has_many :guest_card_classes, class_name: 'SpreeCmCommissioner::GuestCardClass', through: :variants
       base.has_many :product_completion_steps, class_name: 'SpreeCmCommissioner::ProductCompletionStep', through: :line_items
-      base.has_many :audit_events, class_name: 'SpreeCmCommissioner::AuditEvent', as: :auditable
 
       base.delegate :customer, to: :user, allow_nil: true
 

data/app/models/spree_cm_commissioner/place.rb

@@ -1,7 +1,5 @@
 module SpreeCmCommissioner
   class Place < Base
-    include SpreeCmCommissioner::Integrations::IntegrationMappable
-
     acts_as_nested_set
 
     validates :reference, presence: true, if: :validate_reference?

data/app/models/spree_cm_commissioner/route.rb

@@ -1,7 +1,6 @@
 module SpreeCmCommissioner
   class Route < Base
     include SpreeCmCommissioner::RouteType
-    include SpreeCmCommissioner::Integrations::IntegrationMappable
     include SpreeCmCommissioner::StoreMetadata
 
     belongs_to :vendor, class_name: 'Spree::Vendor', optional: false

data/app/models/spree_cm_commissioner/seat_layout.rb

@@ -1,7 +1,5 @@
 module SpreeCmCommissioner
   class SeatLayout < Base
-    include SpreeCmCommissioner::Integrations::IntegrationMappable
-
     has_many :seat_sections, class_name: 'SpreeCmCommissioner::SeatSection', dependent: :destroy
 
     has_many :blocks, class_name: 'SpreeCmCommissioner::Block'

data/app/models/spree_cm_commissioner/show.rb

@@ -28,7 +28,9 @@ module SpreeCmCommissioner
                    :max_paid_votes_per_session,
                    :sms_vote_enabled,
                    :sms_vote_cost_usd,
-                   :operator_vote_enabled
+                   :operator_vote_enabled,
+                   :votes_weight,
+                   :voters_weight
 
     # voting_credit_scope determines how purchased vote credits are grouped (scoped).
     # Changing this after credits have been allocated will cause inconsistencies:

data/app/models/spree_cm_commissioner/show_contestant.rb

@@ -27,8 +27,20 @@ module SpreeCmCommissioner
     validates :name, presence: true
     validates :contestant_number, uniqueness: { scope: :show_id }, allow_nil: true
 
+    before_save :set_eliminated_at, if: :status_changed_to_eliminated?
+
     scope :ordered, -> { order(:position, :id) }
 
+    private
+
+    def status_changed_to_eliminated?
+      status_changed? && eliminated?
+    end
+
+    def set_eliminated_at
+      self.eliminated_at = Time.current
+    end
+
     self.whitelisted_ransackable_attributes = %w[name contestant_number status gender category origin]
   end
 end

data/app/models/spree_cm_commissioner/stock_item_decorator.rb

@@ -6,9 +6,6 @@ module SpreeCmCommissioner
 
       base.after_commit :create_inventory_items, on: :create
       base.after_commit :adjust_inventory_items_async, on: :destroy
-
-      # Create maintaining task to purge product related caches
-      base.after_commit { SpreeCmCommissioner::MaintenanceTasks::CacheInvalidation.pending.create_or_find_by(maintainable: variant.product) }
     end
 
     def update_vendor_total_inventory
@@ -18,10 +15,6 @@ module SpreeCmCommissioner
     private
 
     def create_inventory_items
-      # Skip automatic inventory generation when variant has manual_generate_inventory_items set to true
-      # Use case: For variants where inventory items need to be manually created/managed (e.g., external integrations)
-      return if variant.manual_generate_inventory_items?
-
       SpreeCmCommissioner::InventoryItems::GenerateInventoryItemsJob.perform_later(variant_id: variant.id)
     end
 

data/app/models/spree_cm_commissioner/taxon_decorator.rb

@@ -74,6 +74,7 @@ module SpreeCmCommissioner
       base.has_many :taxon_option_values, class_name: 'SpreeCmCommissioner::TaxonOptionValue'
 
       base.after_commit :send_transaction_email_to_organizers, on: :create
+      base.after_commit :send_event_creation_telegram_alert, on: :create
 
       base.has_many :taxon_places, class_name: 'SpreeCmCommissioner::TaxonPlace', dependent: :destroy
       base.has_many :places, through: :taxon_places
@@ -154,6 +155,12 @@ module SpreeCmCommissioner
       SpreeCmCommissioner::OrganizersTransactionalEmailNotifier.call(event_id: id)
     end
 
+    def send_event_creation_telegram_alert
+      return unless kind == 'event' && depth == 1
+
+      SpreeCmCommissioner::EventCreationTelegramAlertSenderJob.perform_later(event_id: id)
+    end
+
     def sync_event_dates_to_line_items
       return unless event? && depth == 1
 

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::Integrations::IntegrationMappable
 
     attr_accessor :hours, :minutes, :seconds
 
@@ -26,7 +25,7 @@ module SpreeCmCommissioner
     after_commit :update_route_price_range, on: %i[create update]
 
     belongs_to :vendor, class_name: 'Spree::Vendor', inverse_of: :trips, optional: false
-    belongs_to :product, class_name: 'Spree::Product', inverse_of: :trip, optional: true
+    belongs_to :product, class_name: 'Spree::Product', inverse_of: :trip, optional: false
 
     belongs_to :service_origin, class_name: 'Spree::Taxon', optional: true
 

data/app/models/spree_cm_commissioner/trip_stop.rb

@@ -1,7 +1,5 @@
 module SpreeCmCommissioner
   class TripStop < Base
-    include SpreeCmCommissioner::Integrations::IntegrationMappable
-
     acts_as_list column: :sequence, scope: :trip_id
 
     belongs_to :trip, class_name: 'SpreeCmCommissioner::Trip', optional: false

data/app/models/spree_cm_commissioner/variant_decorator.rb

@@ -6,7 +6,6 @@ module SpreeCmCommissioner
       base.include SpreeCmCommissioner::VariantOptionsConcern
       base.include SpreeCmCommissioner::KycBitwise
       base.include SpreeCmCommissioner::Integrations::IntegrationMappable
-      base.include SpreeCmCommissioner::VariantMetadata
 
       base.after_commit :update_vendor_price
       base.validate     :validate_option_types

data/app/models/spree_cm_commissioner/vehicle.rb

@@ -1,7 +1,6 @@
 module SpreeCmCommissioner
   class Vehicle < Base
     include SpreeCmCommissioner::VehicleKind
-    include SpreeCmCommissioner::Integrations::IntegrationMappable
 
     belongs_to :vendor, class_name: 'Spree::Vendor'
     belongs_to :vehicle_type, class_name: 'SpreeCmCommissioner::VehicleType'
@@ -14,8 +13,6 @@ module SpreeCmCommissioner
     validates :name, presence: true
     validates :license_plate, uniqueness: true, allow_blank: true
 
-    has_one :seat_layout, through: :vehicle_type
-
     self.whitelisted_ransackable_attributes = %w[name license_plate kind]
 
     def display_name

data/app/models/spree_cm_commissioner/vehicle_type.rb

@@ -2,7 +2,6 @@ module SpreeCmCommissioner
   class VehicleType < Base
     include SpreeCmCommissioner::StoreMetadata
     include SpreeCmCommissioner::VehicleKind
-    include SpreeCmCommissioner::Integrations::IntegrationMappable
 
     # This model has no seat_layout column (polymorphic association), so we store the preload_seat_layout_id ID in public_metadata.
     # This lets us check if a seat layout exists without triggering a database query.

data/app/models/spree_cm_commissioner/vendor_decorator.rb

@@ -30,7 +30,6 @@ module SpreeCmCommissioner
       base.has_many :nearby_places, -> { order(position: :asc) }, class_name: 'SpreeCmCommissioner::VendorPlace', dependent: :destroy
 
       base.has_many :stock_items, through: :variants, class_name: 'Spree::StockItem'
-      base.has_many :currency_rates, class_name: 'SpreeCmCommissioner::CurrencyRate', dependent: :destroy
 
       base.has_many :taxon_vendors, class_name: 'SpreeCmCommissioner::TaxonVendor'
       base.has_many :taxons, through: :taxon_vendors
@@ -102,7 +101,6 @@ module SpreeCmCommissioner
       base.has_many :boarding_trip_stops, through: :trips, class_name: 'SpreeCmCommissioner::TripStop', source: :boarding_trip_stops
       base.has_many :drop_off_trip_stops, through: :trips, class_name: 'SpreeCmCommissioner::TripStop', source: :drop_off_trip_stops
       base.has_many :pricing_models, class_name: 'SpreeCmCommissioner::PricingModel', foreign_key: :vendor_id, dependent: :destroy
-      base.has_many :orders, -> { distinct }, through: :variants, source: :orders
 
       base.has_many :agency_categories, -> { where(kind: :agency_category) }, class_name: 'Spree::Taxon', foreign_key: :vendor_id
 
@@ -245,21 +243,6 @@ module SpreeCmCommissioner
     end
 
     delegate :present?, to: :tenant, prefix: true
-
-    # Returns the exchange rate between two currencies
-    # If no direct rate exists, calculates the inverse rate from the reverse pair
-    # Returns 1.0 for identical currencies, nil if no rate is found
-    def exchange_rate(from_currency, to_currency)
-      return 1.0 if from_currency == to_currency
-
-      pair = currency_rates.by_pair(from_currency, to_currency).first
-      return pair.rate if pair.present?
-
-      reverse_pair = currency_rates.by_pair(to_currency, from_currency).first
-      return (1.0 / reverse_pair.rate).round(6) if reverse_pair.present?
-
-      nil
-    end
   end
 end
 

data/app/models/spree_cm_commissioner/vendor_place.rb

@@ -1,7 +1,5 @@
 module SpreeCmCommissioner
   class VendorPlace < Base
-    include SpreeCmCommissioner::Integrations::IntegrationMappable
-
     enum place_type: { venue: 0, branch: 1, stop: 2, location: 3 }
     acts_as_list scope: :vendor
 

data/app/models/spree_cm_commissioner/voting_contestant.rb

@@ -20,6 +20,18 @@ module SpreeCmCommissioner
 
     validates :eliminated_via, presence: true, if: :eliminated?
 
+    # Uniqueness validation: vote_number must be unique per contestant per voting_session
+    # Scope: [show_contestant_id, voting_session_id]
+    #
+    # | contestant_id | session_id | vote_number | Status | Reason                    |
+    # |---------------|-----------|-------------|--------|---------------------------|
+    # | 1             | 1         | 1           | ✓ OK   | First record              |
+    # | 1             | 1         | 1           | ✗ FAIL | Duplicate in same scope   |
+    # | 1             | 1         | 2           | ✓ OK   | Can update/change number  |
+    # | 2             | 1         | 1           | ✓ OK   | Different contestant      |
+    # | 1             | 2         | 1           | ✓ OK   | Different session         |
+    validates :vote_number, uniqueness: { scope: %i[show_contestant_id voting_session_id], allow_nil: true }
+
     delegate :name, :contestant_number, :category, :gender, to: :show_contestant, allow_nil: true
     delegate :can_vote?, to: :voting_session, allow_nil: true