spree_cm_commissioner 2.5.1.pre.pre3 → 2.5.1.pre.pre4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2ee953cf63ab26a979ddaa919500a3f7da95e812f8dc20557cb54411047821b6
-  data.tar.gz: bb72deade157e1dde753706bc7a9ad4dbfd329ebb0fc7290074983bdc3d17eca
+  metadata.gz: 40605f05e2eabdbbf47121bbbfde8d0662afe833e7fb91803cadf1977c88fc8d
+  data.tar.gz: 025106d9fa95be2de1f1347b42b243a548c81ccc61efb5abec369fac2dd4bab1
 SHA512:
-  metadata.gz: e4db3651d96a7e492cc13d3755ba0ec2f8474f49cbe5918e1af02855dba9d74786feacf7215806f3fb744919a710378ff25551829319ec96740d1d42f3d19d02
-  data.tar.gz: 5f28bd001f3cc004058ccbdb253dd9b7614aa6ecdef0673da30ba03492abd900dfe3c48b2af18cfccd2c8f228c9753e0c9c3c51f896be6e13b5d6e23cc478c54
+  metadata.gz: 3c9d477987819fc303567b1320b5c9c6cf1653ede16caf754ae5a60f3795e2d977de9f57ca82d13cde43de9bddfece7815fcad6f330a656202f67458c501f17f
+  data.tar.gz: de1d93feeb60fc4ed36f11baf3f013a260c1c6d16a2322b4b7674f77d58844060b33b6b76d2e1d2698ffdfa1f28d47ee45c1863059f9fa7fd5e221fade8e414a

data/.github/workflows/test_and_build_gem.yml

@@ -45,7 +45,7 @@ jobs:
             const commits_url = pr.commits_url;
 
             const commits = await github.request(commits_url);
-            const pattern = /^Close\s+#\d+\s.+/;
+            const pattern = /^(Close flow|Close\s+#\d+\s.+|Merge.*)/;
 
             let invalidCommits = [];
 
@@ -63,7 +63,7 @@ jobs:
               core.setFailed(
                 `The following commit messages are not in the correct format:\n\n${invalidCommits.join(
                   '\n'
-                )}\n\nEach commit message must start with "Close #<issue_number> <message>"`
+                )}\n\nEach commit message must start with "Close #<issue_number> <message>" or "Merge pull request #<number>"`
               );
             } else {
               console.log("All commit messages are correctly formatted.");

data/Gemfile.lock

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

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

@@ -31,7 +31,7 @@ module Spree
 
           # override
           def collection_finder
-            SpreeCmCommissioner::Routes::FindPopular
+            SpreeCmCommissioner::RouteMetrics::FindPopular
           end
 
           # override
@@ -41,7 +41,7 @@ module Spree
 
           # override
           def resource_serializer
-            SpreeCmCommissioner::V2::Storefront::RouteSerializer
+            SpreeCmCommissioner::V2::Storefront::RouteMetricSerializer
           end
 
           # override

data/app/controllers/spree/api/v2/storefront/transit/draft_orders_controller.rb

@@ -15,7 +15,7 @@ module Spree
               @outbound_legs = params[:outbound_legs].is_a?(Array) && params[:outbound_legs].any? ? build_legs(:outbound, params[:outbound_legs]) : []
               @inbound_legs  = params[:inbound_legs].is_a?(Array) && params[:inbound_legs].any? ? build_legs(:inbound, params[:inbound_legs]) : []
 
-              context = SpreeCmCommissioner::Transit::DraftOrderCreator.call(
+              result = SpreeCmCommissioner::TransitOrder::Create.call(
                 outbound_date: params[:outbound_date]&.to_date,
                 inbound_date: params[:inbound_date]&.to_date,
                 outbound_legs: @outbound_legs,
@@ -23,10 +23,10 @@ module Spree
                 user: spree_current_user
               )
 
-              if context.success?
-                render_serialized_payload { serialize_resource(context.order) }
+              if result.success?
+                render_serialized_payload { serialize_resource(result.value[:order]) }
               else
-                render_error_payload(context.message)
+                render_error_payload(result.error)
               end
             end
 

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

@@ -0,0 +1,60 @@
+module Spree
+  module Api
+    module V2
+      module Tenant
+        class PopularRoutePlacesController < BaseController
+          private
+
+          # override
+          def collection
+            @collection ||= collection_finder.new(
+              tenant: @tenant,
+              route_type: params[:route_type]
+            ).execute
+          end
+
+          # override
+          def default_resource_includes
+            %w[origin_place destination_place]
+          end
+
+          # override
+          def collection_finder
+            SpreeCmCommissioner::Routes::FindPopular
+          end
+
+          # override
+          def collection_serializer
+            resource_serializer
+          end
+
+          # override
+          def resource_serializer
+            SpreeCmCommissioner::V2::Storefront::RouteSerializer
+          end
+
+          # override
+          def serializer_params
+            super.merge(
+              include_vendors: include_vendors?,
+              include_nearby_places: include_nearby_places?
+            )
+          end
+
+          # override
+          def required_schema
+            SpreeCmCommissioner::PopularRoutePlacesRequestSchema
+          end
+
+          def include_vendors?
+            resource_includes.include?(:vendors) || false
+          end
+
+          def include_nearby_places?
+            resource_includes.include?(:nearby_places) || false
+          end
+        end
+      end
+    end
+  end
+end

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

@@ -0,0 +1,50 @@
+module Spree
+  module Api
+    module V2
+      module Tenant
+        class RoutesController < BaseController
+          private
+
+          # override
+          def collection
+            @collection ||= collection_finder.new(
+              tenant: @tenant,
+              params: params
+            ).execute
+          end
+
+          # override
+          def collection_finder
+            SpreeCmCommissioner::Routes::Find
+          end
+
+          # override
+          def collection_serializer
+            SpreeCmCommissioner::V2::Storefront::RouteSerializer
+          end
+
+          # override
+          def serializer_params
+            super.merge(
+              include_vendors: include_vendors?,
+              include_nearby_places: include_nearby_places?
+            )
+          end
+
+          # override
+          def required_schema
+            SpreeCmCommissioner::PopularRoutePlacesRequestSchema
+          end
+
+          def include_vendors?
+            resource_includes.include?(:vendors) || false
+          end
+
+          def include_nearby_places?
+            resource_includes.include?(:nearby_places) || false
+          end
+        end
+      end
+    end
+  end
+end

data/app/controllers/spree/api/v2/tenant/transit/draft_orders_controller.rb

@@ -0,0 +1,46 @@
+module Spree
+  module Api
+    module V2
+      module Tenant
+        module Transit
+          class DraftOrdersController < BaseController
+            # Creates a draft transit order for tenant.
+            # Endpoint: POST /api/v2/tenant/transit/draft_orders
+            # Params:
+            # - outbound_date: Date of outbound transit (optional)
+            # - inbound_date: Date of inbound transit (optional)
+            # - outbound_legs: Array of outbound leg details (required)
+            # - inbound_legs: Array of inbound leg details (optional)
+            def create
+              @outbound_legs = params[:outbound_legs].is_a?(Array) && params[:outbound_legs].any? ? build_legs(:outbound, params[:outbound_legs]) : []
+              @inbound_legs  = params[:inbound_legs].is_a?(Array) && params[:inbound_legs].any? ? build_legs(:inbound, params[:inbound_legs]) : []
+
+              result = SpreeCmCommissioner::TransitOrder::Create.call(
+                outbound_date: params[:outbound_date]&.to_date,
+                inbound_date: params[:inbound_date]&.to_date,
+                outbound_legs: @outbound_legs,
+                inbound_legs: @inbound_legs,
+                user: spree_current_user
+              )
+
+              if result.success?
+                render_serialized_payload { serialize_resource(result.value[:order]) }
+              else
+                render_error_payload(result.error)
+              end
+            end
+
+            def build_legs(direction, legs_params)
+              SpreeCmCommissioner::Transit::LegsBuilderService.new(direction: direction, legs_params: legs_params).call
+            end
+
+            # override
+            def resource_serializer
+              Spree::V2::Tenant::TransitCartSerializer
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/app/controllers/spree/transit/trips_controller.rb

@@ -48,13 +48,13 @@ module Spree
         boarding_points_attributes = (cm_params.delete('boarding_points') || []).compact_blank
                                                                                 .map do |point_id|
           trip_stop = trip_stops.delete(point_id.to_i)
-          { stop_id: point_id, stop_type: 'boarding', id: trip_stop.try(:id) }
+          { stop_id: point_id, allow_boarding: true, id: trip_stop.try(:id) }
         end
 
         drop_off_points_attributes = (cm_params.delete('drop_off_points') || []).compact_blank
                                                                                 .map do |point_id|
           trip_stop = trip_stops.delete(point_id.to_i)
-          { stop_id: point_id, stop_type: 'drop_off', id: trip_stop.try(:id) }
+          { stop_id: point_id, allow_drop_off: true, id: trip_stop.try(:id) }
         end
 
         trip_stops.each_value do |trip_stop|
@@ -74,7 +74,7 @@ module Spree
           :origin_id, :destination_id, :vehicle_id, :hours, :minutes, :seconds,
           'departure_time(1i)', 'departure_time(2i)', 'departure_time(3i)',
           'departure_time(4i)', 'departure_time(5i)', :product_id,
-          trip_stops_attributes: %i[stop_id stop_type _destroy id]
+          trip_stops_attributes: %i[stop_id allow_boarding allow_drop_off _destroy id]
         )
       end
 

data/app/finders/spree_cm_commissioner/places/find_with_route.rb

@@ -1,20 +1,20 @@
-# Finds places connected via routes with optional filtering.
+# Finds places connected via route metrics with optional filtering.
 #
 # @param place_type [String] Required. 'origin' or 'destination'
 # @param place_id [Integer] Optional. Filter by connected place ID
 # @param query [String] Optional. Filter by place name (case-insensitive)
-# @param route_type [Symbol, String] Optional. Filter by route type from associated trips (e.g., :ferry, :bus)
+# @param route_type [Symbol, String] Optional. Filter by route type from route metrics (e.g., :ferry, :bus)
 #
 # @return [ActiveRecord::Relation<SpreeCmCommissioner::Place>]
 #
 # Modes:
 #   - place_id only: returns connected places (origins TO or destinations FROM given place)
 #   - query only: filters by name
-#   - route_type only: filters by trip route types on those routes
+#   - route_type only: filters by route type on route metrics
 #   - combinations: connected places filtered by name and/or route type
-#   - neither: all origins/destinations in routes
+#   - neither: all origins/destinations in route metrics
 #
-# @example Origins with ferry trips to place 123
+# @example Origins with ferry route metrics to place 123
 #   FindWithRoute.new(place_type: 'origin', place_id: 123, route_type: :ferry).execute
 #
 # @example Destinations filtered by query and route type
@@ -46,14 +46,14 @@ module SpreeCmCommissioner
       def scope
         return SpreeCmCommissioner::Place.none unless valid_place_type?
 
-        base_scope = origin? ? SpreeCmCommissioner::Place.with_routes_as_origin : SpreeCmCommissioner::Place.with_routes_as_destination
+        base_scope = origin? ? SpreeCmCommissioner::Place.with_route_metrics_as_origin : SpreeCmCommissioner::Place.with_route_metrics_as_destination
 
         return base_scope if place_id.blank?
 
         if origin?
-          base_scope.where(routes_as_origin: { destination_place_id: place_id })
+          base_scope.where(route_metrics_as_origin: { destination_place_id: place_id })
         else
-          base_scope.where(routes_as_destination: { origin_place_id: place_id })
+          base_scope.where(route_metrics_as_destination: { origin_place_id: place_id })
         end
       end
 
@@ -62,10 +62,10 @@ module SpreeCmCommissioner
       end
 
       def apply_route_type_filter(result)
-        trips_association = origin? ? :trips_as_origin : :trips_as_destination
-        result.joins(trips_association)
-              .where(cm_trips: { route_type: route_type.to_sym })
-              .distinct
+        # Use the association name as the table alias in the WHERE clause
+        # This works whether the association is already joined or not
+        association_name = origin? ? :route_metrics_as_origin : :route_metrics_as_destination
+        result.where(association_name => { route_type: route_type.to_sym })
       end
 
       def origin?

data/app/finders/spree_cm_commissioner/route_metrics/find_popular.rb

@@ -0,0 +1,44 @@
+# Finds route metrics ordered by popularity based on order metrics.
+#
+# Route metrics are sorted by:
+# 1. fulfilled_order_count (completed orders) - DESC
+# 2. order_count (total orders including incomplete) - DESC
+#
+# @param route_type [Symbol, String] Optional. Filter by route type (e.g., :ferry, :bus)
+#
+# @return [ActiveRecord::Relation<SpreeCmCommissioner::RouteMetric>] Route metrics with origin and destination places loaded,
+#   ordered from most to least popular
+#
+# @example Get all popular route metrics
+#   finder = SpreeCmCommissioner::RouteMetrics::FindPopular.new
+#   finder.execute # => Returns all route metrics sorted by fulfillment and order counts
+#
+# @example Get popular ferry route metrics
+#   finder = SpreeCmCommissioner::RouteMetrics::FindPopular.new
+#   finder.execute(route_type: :ferry) # => Returns ferry route metrics sorted by popularity
+#
+# @example Get popular bus route metrics
+#   finder = SpreeCmCommissioner::RouteMetrics::FindPopular.new
+#   finder.execute(route_type: :bus) # => Returns bus route metrics sorted by popularity
+
+module SpreeCmCommissioner
+  module RouteMetrics
+    class FindPopular
+      def execute(route_type: nil, query: nil)
+        scope(route_type: route_type, query: query)
+      end
+
+      private
+
+      def scope(route_type:, query:)
+        result = SpreeCmCommissioner::RouteMetric.all
+
+        result = result.where(route_type: route_type.to_sym) if route_type.present?
+        result = result.where('cm_route_metrics.route_name ILIKE ?', "%#{query}%") if query.present?
+
+        result.includes(:origin_place, :destination_place)
+              .order(fulfilled_order_count: :desc, order_count: :desc)
+      end
+    end
+  end
+end

data/app/finders/spree_cm_commissioner/routes/find.rb

@@ -0,0 +1,94 @@
+# Finder for Route records with common filtering options.
+# Usage:
+#   finder = SpreeCmCommissioner::Routes::Find.new(vendor: vendor, params: params)
+#   finder = SpreeCmCommissioner::Routes::Find.new(tenant: tenant, params: params)
+#   finder.execute
+#   returns ActiveRecord::Relation of SpreeCmCommissioner::Route
+
+module SpreeCmCommissioner
+  module Routes
+    class Find
+      def initialize(vendor: nil, tenant: nil, params: {})
+        @vendor = vendor
+        @tenant = tenant
+        @params = params || {}
+
+        @ids = Array(@params[:ids]).map(&:to_s)
+        @name = @params.dig(:filter, :name) || @params[:name]
+        @route_type = @params.dig(:filter, :route_type) || @params[:route_type]
+        @origin_place_id = @params.dig(:filter, :origin_place_id)
+        @destination_place_id = @params.dig(:filter, :destination_place_id)
+        @query = @params.dig(:filter, :query) || @params[:query]
+        @limit = (@params[:limit] || @params.dig(:page, :limit) || 50).to_i
+        @order = @params[:order] || 'created_at DESC'
+      end
+
+      def execute
+        results = base_scope
+        results = by_ids(results) if ids?
+        results = by_name(results) if name?
+        results = by_route_type(results) if route_type?
+        results = by_origin_place(results) if origin_place_id.present?
+        results = by_destination_place(results) if destination_place_id.present?
+        results = by_query(results) if query?
+
+        results.order(order_clause).limit(limit)
+      end
+
+      private
+
+      attr_reader :vendor, :tenant, :params, :ids, :name, :route_type, :origin_place_id, :destination_place_id, :query, :limit, :order
+
+      def base_scope
+        scope = SpreeCmCommissioner::Route.includes(:origin_place, :destination_place, :route_photos)
+        scope = scope.where(vendor_id: vendor.id) if vendor.present?
+        scope = scope.where(tenant_id: tenant.id) if tenant.present?
+        scope
+      end
+
+      def ids?
+        ids.present?
+      end
+
+      def name?
+        name.present?
+      end
+
+      def route_type?
+        route_type.present?
+      end
+
+      def query?
+        query.present?
+      end
+
+      def by_ids(scope)
+        scope.where(id: ids)
+      end
+
+      def by_name(scope)
+        scope.where('cm_routes.route_name ILIKE ?', "%#{name}%")
+      end
+
+      def by_route_type(scope)
+        scope.where(route_type: route_type.to_s)
+      end
+
+      def by_origin_place(scope)
+        scope.where(origin_place_id: origin_place_id)
+      end
+
+      def by_destination_place(scope)
+        scope.where(destination_place_id: destination_place_id)
+      end
+
+      def by_query(scope)
+        scope.where('cm_routes.route_name ILIKE :q OR cm_routes.short_name ILIKE :q', q: "%#{query}%")
+      end
+
+      def order_clause
+        order
+      end
+    end
+  end
+end

data/app/finders/spree_cm_commissioner/routes/find_popular.rb

@@ -1,46 +1,30 @@
-# Finds routes ordered by popularity based on order metrics.
-#
-# Routes are sorted by:
-# 1. fulfilled_order_count (completed orders) - DESC
-# 2. order_count (total orders including incomplete) - DESC
-#
-# @param route_type [Symbol, String] Optional. Filter by route type from associated trips (e.g., :ferry, :bus)
-#
-# @return [ActiveRecord::Relation<SpreeCmCommissioner::Route>] Routes with origin and destination places loaded,
-#   ordered from most to least popular
-#
-# @example Get all popular routes
-#   finder = SpreeCmCommissioner::Routes::FindPopular.new
-#   finder.execute # => Returns all routes sorted by fulfillment and order counts
-#
-# @example Get popular ferry routes
-#   finder = SpreeCmCommissioner::Routes::FindPopular.new
-#   finder.execute(route_type: :ferry) # => Returns ferry routes sorted by popularity
-#
-# @example Get popular bus routes
-#   finder = SpreeCmCommissioner::Routes::FindPopular.new
-#   finder.execute(route_type: :bus) # => Returns bus routes sorted by popularity
+# Finder to get popular route metrics scoped to a vendor or tenant.
+# Orders by fulfilled_order_count DESC, then order_count DESC.
+# Usage:
+#   finder = SpreeCmCommissioner::Routes::FindPopular.new(vendor: vendor, route_type: :bus, limit: 10)
+#   finder.execute
 
 module SpreeCmCommissioner
   module Routes
     class FindPopular
-      def execute(route_type: nil, query: nil)
-        scope(route_type: route_type, query: query)
+      def initialize(vendor: nil, tenant: nil, route_type: nil, limit: 20)
+        @vendor = vendor
+        @tenant = tenant
+        @route_type = route_type
+        @limit = limit.to_i
       end
 
-      private
-
-      def scope(route_type:, query:)
-        includes_associations = %i[origin_place destination_place]
-        includes_associations << :trips if route_type.present?
-        result = SpreeCmCommissioner::Route.includes(*includes_associations)
-
-        result = result.joins(:trips).where(cm_trips: { route_type: route_type.to_sym }) if route_type.present?
+      def execute
+        scope = SpreeCmCommissioner::Route.includes(:origin_place, :destination_place, :route_photos)
+        scope = scope.where(vendor_id: vendor.id) if vendor.present?
+        scope = scope.where(tenant_id: tenant.id) if tenant.present?
+        scope = scope.where(route_type: route_type.to_s) if route_type.present?
+        scope.order(fulfilled_order_count: :desc, order_count: :desc).limit(limit)
+      end
 
-        result = result.where('cm_routes.route_name ILIKE ?', "%#{query}%") if query.present?
+      private
 
-        result.distinct.order(fulfilled_order_count: :desc, order_count: :desc)
-      end
+      attr_reader :vendor, :tenant, :route_type, :limit
     end
   end
 end

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

@@ -33,11 +33,18 @@ module SpreeCmCommissioner
         end
       end
 
+      # Returns dates for which inventory should be generated, based on calendar or default period.
       def inventory_dates_for(variant)
-        start_date = Time.zone.tomorrow
-        end_date = Time.zone.today + pre_inventory_days_for(variant)
+        calendar = variant.product&.service_calendar
 
-        (start_date..end_date)
+        if calendar.present?
+          calendar.start_date.upto(calendar.end_date).select { |date| calendar.service_available?(date) }
+        else
+          start_date = Time.zone.tomorrow
+          end_date = Time.zone.today + pre_inventory_days_for(variant)
+
+          start_date.upto(end_date).to_a
+        end
       end
 
       def inventory_exist?(variant, inventory_date)
@@ -68,7 +75,7 @@ module SpreeCmCommissioner
       def variants
         scope = Spree::Variant.active.with_permanent_stock.where(is_master: false)
         scope = scope.where(id: variant_ids) if variant_ids.present?
-        scope
+        scope.includes(product: :service_calendar)
       end
     end
   end

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

@@ -8,7 +8,8 @@ module SpreeCmCommissioner
 
         return context.fail!(message: Spree.t(:doesnt_track_inventory)) unless variant.track_inventory?
 
-        stock_location = Spree::StockLocation.find(stock_location_id)
+        stock_location = resolve_stock_location(variant, stock_location_id)
+
         stock_movement = stock_location.stock_movements.build(stock_movement_params)
         stock_movement.stock_item = stock_location.set_up_stock_item(variant)
 
@@ -22,6 +23,14 @@ module SpreeCmCommissioner
 
       private
 
+      def resolve_stock_location(variant, stock_location_id = nil)
+        if stock_location_id.present?
+          Spree::StockLocation.find(stock_location_id)
+        else
+          variant.vendor.stock_locations.first || variant.vendor.send(:create_stock_location)
+        end
+      end
+
       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

data/app/interactors/spree_cm_commissioner/trip_clone_creator.rb

@@ -91,9 +91,10 @@ module SpreeCmCommissioner
       stop_params = params.dig(:trip_stops_attributes, index.to_s)
       return unless stop_params
 
-      cloned_stop.departure_time    = parse_time(stop_params[:departure_time])
-      cloned_stop.arrival_time      = parse_time(stop_params[:arrival_time])
-      cloned_stop.stop_type         = stop_params[:stop_type].presence
+      cloned_stop.departure_time = parse_time(stop_params[:departure_time])
+      cloned_stop.arrival_time = parse_time(stop_params[:arrival_time])
+      cloned_stop.allow_boarding = stop_params[:allow_boarding] if stop_params.key?(:allow_boarding)
+      cloned_stop.allow_drop_off = stop_params[:allow_drop_off] if stop_params.key?(:allow_drop_off)
       cloned_stop.location_place_id = stop_params[:location_place_id].presence
     end
 

data/app/interactors/spree_cm_commissioner/trip_stops_creator.rb

@@ -26,7 +26,7 @@ module SpreeCmCommissioner
         trip: trip,
         stop_place: origin_place,
         location_place: origin_place,
-        stop_type: :boarding
+        allow_boarding: true
       }
 
       attributes[:departure_time] = departure_time
@@ -42,7 +42,7 @@ module SpreeCmCommissioner
         trip: trip,
         stop_place: destination_place,
         location_place: destination_place,
-        stop_type: :drop_off
+        allow_drop_off: true
       }
 
       attributes[:arrival_time] = departure_time + duration_seconds.seconds

data/app/jobs/spree_cm_commissioner/route_metrics/decrease_trip_count_job.rb

@@ -0,0 +1,10 @@
+module SpreeCmCommissioner
+  module RouteMetrics
+    class DecreaseTripCountJob < ApplicationUniqueJob
+      def perform(options = {})
+        trip = SpreeCmCommissioner::Trip.find(options[:trip_id])
+        SpreeCmCommissioner::RouteMetrics::DecreaseTripCount.call(trip: trip)
+      end
+    end
+  end
+end

data/app/jobs/spree_cm_commissioner/route_metrics/increase_fulfilled_order_count_job.rb

@@ -0,0 +1,10 @@
+module SpreeCmCommissioner
+  module RouteMetrics
+    class IncreaseFulfilledOrderCountJob < ApplicationUniqueJob
+      def perform(options = {})
+        order = Spree::Order.find(options[:order_id])
+        SpreeCmCommissioner::RouteMetrics::IncreaseFulfilledOrderCount.call(order: order)
+      end
+    end
+  end
+end

data/app/jobs/spree_cm_commissioner/route_metrics/increase_order_count_job.rb

@@ -0,0 +1,10 @@
+module SpreeCmCommissioner
+  module RouteMetrics
+    class IncreaseOrderCountJob < ApplicationUniqueJob
+      def perform(options = {})
+        order = Spree::Order.find(options[:order_id])
+        SpreeCmCommissioner::RouteMetrics::IncreaseOrderCount.call(order: order)
+      end
+    end
+  end
+end