ba_spree 1.0.0

data/app/models/ba_spree/order_decorator.rb

@@ -0,0 +1,95 @@
+module BaSpree
+  module OrderDecorator
+    ASSOCIATED_USER_ATTRIBUTES = [ :user_id, :email, :created_by_id, :bill_address_id, :ship_address_id ]
+
+    # @gem-override spree_core-5.3.6/app/models/spree/order.rb#associate_user!
+    # @see https://github.com/be-agile/giga-repeat/commit/d0dbe2b4524e22c1cf974bbd4829c587ef7cd254
+    # Associates the specified user with the order.
+    def associate_user!(user, override_email = true)
+      self.user           = user
+      self.email          = user.email if override_email
+      # we need to check if user is of admin user class to avoid mismatch type error
+      # in a scenario where we have separate classes for admin and regular users
+      self.created_by   ||= user if user.is_a?(Spree.admin_user_class)
+      self.bill_address ||= user.bill_address
+      self.ship_address ||= user.ship_address
+
+      # 未保存(id=nil)の order では where(id: self) が `WHERE id IS NULL` となり、
+      # update_all が無関係なレコードを対象にしたり reload が RecordNotFound を投げる。
+      # まだ永続化されていない場合はメモリ上の代入のみで十分なので DB 反映はスキップする。
+      return self unless persisted?
+
+      changes = slice(*ASSOCIATED_USER_ATTRIBUTES)
+
+      # immediately persist the changes we just made, but don't use save
+      # since we might have an invalid address associated
+      ActiveRecord::Base.connected_to(role: :writing) do
+        self.class.unscoped.where(id: self).update_all(changes)
+      end
+
+      # Reload to ensure memory state matches DB state, then recalculate promotions
+      reload
+      updater.update
+
+      # Spree 5.3 追従: update_all はコールバックを bypass するため order.updated を手動 publish
+      publish_event('order.updated') if changes.present?
+    end
+
+    # @gem-override spree_core-5.3.6/app/models/spree/order.rb#disassociate_user!
+    # @see https://github.com/be-agile/giga-repeat/commit/d0dbe2b4524e22c1cf974bbd4829c587ef7cd254
+    def disassociate_user!
+      nullified_attributes = ASSOCIATED_USER_ATTRIBUTES.index_with(nil)
+
+      update_columns(nullified_attributes)
+
+      # Reload to ensure memory state matches DB state, then recalculate promotions
+      reload
+      updater.update
+    end
+
+    # @gem-override spree_core-5.3.6/app/models/spree/order.rb#after_cancel
+    # @see https://github.com/be-agile/giga-repeat/issues/1001
+    # Spree 5.3 追従: 本家追加の publish_order_canceled_event は super で取り込まれる
+    # 注文キャンセル時にpending状態のゲートウェイ決済（オーソリ）もキャンセルする
+    # Spreeコアは completed のみをキャンセルするが、オーソリのまま残すと以下の問題がある：
+    # - 決済プロバイダー側にオーソリが残る
+    # - 顧客のカード利用可能額が減少したまま
+    # - 後で誤って売上計上される可能性
+    def after_cancel
+      super
+
+      # pending 状態のゲートウェイ決済（オーソリ）をキャンセル
+      # store_credit は Spree コアで処理済みなので除外
+      payments.pending.not_store_credits.each do |payment|
+        # Gatewayベースの決済方法のみ対象（銀行振込や代引きは除外）
+        if payment.payment_method.is_a?(Spree::Gateway)
+          payment.cancel!
+        end
+      end
+    end
+
+    # @gem-override spree_core-5.3.6/app/models/spree/order.rb#canceled_by
+    # @see https://github.com/be-agile/giga-repeat/issues/1232
+    # Spree 5.3.6 本家のバグ修正: canceled_by 末尾の publish_event('order.canceled') は
+    # コメント（"Manually publish update event"）通り本来 'order.updated' を publish すべき箇所。
+    # 'order.canceled' のままだと、transaction 内の cancel! が after_cancel 経由で既に
+    # 'order.canceled' を publish しているのと合わせて二重 publish となり、
+    # OrderEmailSubscriber#send_cancel_email がガード無しのためキャンセルメールが2通送信される。
+    # update_columns はコールバックを bypass するため、本来意図された 'order.updated' を publish する。
+    def canceled_by(user, canceled_at = nil)
+      canceled_at ||= Time.current
+      changes = { canceler_id: user.id, canceled_at: canceled_at }
+
+      transaction do
+        update_columns(changes)
+        cancel!
+      end
+
+      # Manually publish update event since update_columns bypasses callbacks
+      # （cancel! 内の after_cancel が 'order.canceled' を publish 済みなので、ここは update のみ）
+      publish_event('order.updated')
+    end
+  end
+end
+
+Spree::Order.prepend(BaSpree::OrderDecorator)

data/app/models/ba_spree/order_updater_decorator.rb

@@ -0,0 +1,33 @@
+module BaSpree
+  module OrderUpdaterDecorator
+    # @gem-override spree_core-5.3.6/app/models/spree/order_updater.rb#update_adjustment_total
+    # @see https://github.com/be-agile/giga-repeat/issues/909
+    # super 後に order-level adjustment に対する内税計算を追加する。
+    # Spree 5.3 追従: 本家の sum→pick 単一クエリ最適化は super 経由で取り込まれるため
+    #                 override 本体は無変更(super の結果を部分的に上書きする設計のため互換)。
+    def update_adjustment_total
+      super
+
+      order.adjustments.tax.where(adjustable_type: 'Spree::Order').destroy_all
+
+      rates = Spree::TaxRate.match(order.tax_zone)
+      rates.each do |rate|
+        rate.adjust_order_level_adjustments(order)
+      end
+
+      order.included_tax_total = line_items.sum(:included_tax_total) +
+                                 shipments.sum(:included_tax_total) +
+                                 adjustments.tax.eligible.where(adjustable_type: 'Spree::Order').sum(:amount)
+      # 0より小さいとvalidateエラーになるため
+      order.included_tax_total = 0 if order.included_tax_total.negative?
+
+      order.adjustment_total = line_items.sum(:adjustment_total) +
+                              shipments.sum(:adjustment_total) +
+                              adjustments.eligible.non_tax.sum(:amount)
+
+      update_order_total
+    end
+  end
+end
+
+Spree::OrderUpdater.prepend(BaSpree::OrderUpdaterDecorator)

data/app/models/ba_spree/payment_method_decorator.rb

@@ -0,0 +1,10 @@
+module BaSpree
+  module PaymentMethodDecorator
+    def tax_category
+      return nil unless respond_to?(:preferred_tax_category_id)
+      Spree::TaxCategory.find_by(id: preferred_tax_category_id) if preferred_tax_category_id.present?
+    end
+  end
+end
+
+Spree::PaymentMethod.prepend(BaSpree::PaymentMethodDecorator)

data/app/models/ba_spree/product_decorator.rb

@@ -0,0 +1,44 @@
+module BaSpree
+  module ProductDecorator
+    def self.prepended(base)
+      # Spree 5.3 以前は、商品ごとのオプション並び順(spree_product_option_types.position)を正にするため
+      # `option_types` を reorder('spree_product_option_types.position') で上書きしていた。
+      # しかし through association の order/reorder に「他テーブルの列」を書くと、Rails が preload
+      # (WHERE id IN (...)) で読む経路で JOIN が外れ ORDER 句だけ残るため、
+      # 「Unknown column 'spree_product_option_types.position' in 'order clause'」で
+      # taxons#show(カテゴリページ)が 500 になる(5.3 で option_types を preload する経路が増え顕在化)。
+      #
+      # この罠は through の order に他テーブル列を書く限り原理的に回避できないため、上書きを削除し
+      # 5.3 core のデフォルト(`option_types, through: :product_option_types` + `product_option_types
+      # -> order(:position)`)に委ねる。これにより並び順は
+      # `spree_option_types.position`(グローバル順) → `spree_product_option_types.position`(商品ごと順)
+      # の複合になり、「商品ごと順」は第2ソートキーへ後退する。
+      # ※ org4 の複数 option_type 商品 355 件すべてで両者の並び順が一致することを確認済み(実害なし)。
+      #   将来「商品ごとにオプション表示順を変える」運用が必要になった場合は、option_types を直接
+      #   並べ替えるのではなく product_option_types.order(:position) 経由で option_type を引く形に
+      #   変更すること(preload 安全な書き方が必要)。
+      base.has_many :option_types_for_includes, -> { order(:id) },
+                    through: :product_option_types,
+                    source: :option_type,
+                    class_name: 'Spree::OptionType'
+
+      # ProductProperty -> metafields 移行用の読み書きヘルパー(Spree 5.4 の Property 削除対応)
+      base.include Spree::ProductPropertyMetafields
+    end
+
+    def set_slug
+      self.slug = generate_slug
+    end
+
+    private
+    def generate_slug
+      if new_record?
+        uuid_for_friendly_id
+      else
+        self.changes["slug"]&.first || self.slug
+      end
+    end
+  end
+end
+
+Spree::Product.prepend(BaSpree::ProductDecorator)

data/app/models/ba_spree/store_decorator.rb

@@ -0,0 +1,23 @@
+module BaSpree
+  module StoreDecorator
+    def self.prepended(base)
+      base.has_rich_text :special_instruction_message
+
+      base.class_eval do
+        # @gem-override spree_core-5.3.6/app/models/spree/store.rb#default
+        # @see https://github.com/be-agile/giga-repeat/commit/b7c74b2a1f45555616c19f3788def6aa3f67a7a1
+        # 本家は default フラグが無ければ DB の最初の1件を返すが、ba_spree では
+        # 該当が無いとき新規 default インスタンスを返す（マルチテナントで未作成テナント対策）。
+        def self.default
+          if where(default: true).any?
+            where(default: true).first
+          else
+            new(default: true)
+          end
+        end
+      end
+    end
+  end
+end
+
+Spree::Store.prepend BaSpree::StoreDecorator

data/app/models/ba_spree/variant_decorator.rb

@@ -0,0 +1,49 @@
+module BaSpree
+  module VariantDecorator
+    # @gem-override spree_core-5.3.6/app/models/spree/variant.rb#set_option_value
+    # @see https://github.com/be-agile/giga-repeat/commit/108316dd2cab88a61933080614e15663517c7291
+    def set_option_value(opt_name, opt_value, opt_type_position = nil)
+      # no option values on master
+      return if is_master
+
+      option_type = Spree::OptionType.where(name: opt_name).first_or_initialize do |o|
+        o.name = o.presentation = opt_name
+        o.save!
+      end
+
+      current_value = find_option_value(opt_name)
+
+      if current_value.nil?
+        # then we have to check to make sure that the product has the option type
+        product_option_type = if (existing_prod_ot = product.product_option_types.find { |ot| ot.option_type_id == option_type.id })
+          existing_prod_ot
+        else
+          product_option_type = product.product_option_types.new
+          product_option_type.option_type = option_type
+        end
+        product_option_type.position = opt_type_position if opt_type_position
+        product_option_type.save! if product_option_type.new_record? || product_option_type.changed?
+      else
+        return if current_value.name == opt_value
+
+        option_values.delete(current_value)
+      end
+
+      option_value = option_type.option_values.where(name: opt_value).first_or_initialize do |o|
+        o.name = o.presentation = opt_value
+        o.save!
+      end
+
+      option_values << option_value
+      save
+    end
+
+    # @gem-override spree_core-5.3.6/app/models/spree/variant.rb#find_option_value
+    # @see https://github.com/be-agile/giga-repeat/commit/108316dd2cab88a61933080614e15663517c7291
+    def find_option_value(opt_name)
+      option_values.includes(:option_type).detect { |o| o.option_type.name == opt_name }
+    end
+  end
+end
+
+Spree::Variant.prepend(BaSpree::VariantDecorator)

data/app/models/concerns/spree/adjustment_source_decorator.rb

@@ -0,0 +1,66 @@
+module Spree
+  module AdjustmentSourceDecorator
+    # @gem-override spree_core-5.3.6/app/models/concerns/spree/adjustment_source.rb#create_unique_adjustment
+    # @see https://github.com/be-agile/giga-repeat/issues/1183
+    # 並行リクエスト時の重複 adjustment 作成を防ぐため、order を排他ロックしてから
+    # 既存チェックと作成を実行する。これがないと、ブラウザ複数タブから同一 order に
+    # 同時にカート操作されたとき、`already_adjusted?` の SELECT が両リクエストで
+    # false を返し、同じ source の adjustment が二重に INSERT されてしまう。
+    def create_unique_adjustment(order, adjustable)
+      with_order_lock(order) { super }
+    end
+
+    # @gem-override spree_core-5.3.6/app/models/concerns/spree/adjustment_source.rb#create_unique_adjustments
+    # @see https://github.com/be-agile/giga-repeat/issues/1183
+    def create_unique_adjustments(order, adjustables, &block)
+      with_order_lock(order) { super }
+    end
+
+    private
+
+    # @gem-override spree_core-5.3.6/app/models/concerns/spree/adjustment_source.rb#already_adjusted?
+    # @see https://github.com/be-agile/giga-repeat/issues/1183
+    # `already_adjusted?` を locking read (FOR UPDATE) で実行することで、
+    # MySQL REPEATABLE READ 下の snapshot 問題を回避する。
+    # 通常 SELECT は transaction 開始時の snapshot を見るため、先行 transaction の
+    # adjustment INSERT が後続 transaction から見えないケースがあるが、
+    # locking read は current read で最新状態を見るため、先行の commit 後の
+    # adjustment が必ず見える。
+    def already_adjusted?(adjustable)
+      adjustments.lock.where(adjustable: adjustable).exists?
+    end
+
+    # @gem-override spree_core-5.3.6/app/models/concerns/spree/adjustment_source.rb#already_adjusted_ids
+    # @see https://github.com/be-agile/giga-repeat/issues/1183
+    # 同上の理由で locking read で実行する。
+    def already_adjusted_ids(order)
+      adjustments.lock.where(order: order).pluck(:adjustable_id)
+    end
+
+    # order インスタンス自体に dirty な属性 (例: last_ip_address) がある状態で
+    # `order.with_lock` を呼ぶと Rails 8 では RuntimeError になるため、
+    # 別インスタンスで `SELECT ... FOR UPDATE` を発行してロックを取得する。
+    # トランザクション内で行リレベルロックを取れば、後続リクエストは
+    # 先行トランザクションの commit を待つので race condition は防げる。
+    def with_order_lock(order)
+      order.class.transaction do
+        order.class.lock.where(id: order.id).pluck(:id)
+        yield
+      end
+    end
+  end
+end
+
+# Spree::AdjustmentSource を include している promotion action 系の各クラスに対して
+# decorator を prepend する。concern module 自体に prepend するのではなく include 元クラスに
+# 直接適用することで、Ruby のメソッド解決順が確実に decorator を経由するようにする。
+# TaxRate は AdjustmentSource を include しているが create_unique_adjustment を経由しないため対象外。
+[
+  Spree::Promotion::Actions::CreateAdjustment,
+  Spree::Promotion::Actions::CreateItemAdjustments,
+  Spree::Promotion::Actions::FreeShipping,
+  defined?(Spree::Promotion::Actions::CreateVolumeDiscountAdjustments) ? Spree::Promotion::Actions::CreateVolumeDiscountAdjustments : nil,
+  defined?(Spree::Promotion::Actions::CreateVolumeBasedOptionFee) ? Spree::Promotion::Actions::CreateVolumeBasedOptionFee : nil
+].compact.each do |klass|
+  klass.prepend(Spree::AdjustmentSourceDecorator)
+end

data/app/models/concerns/spree/product_property_metafields.rb

@@ -0,0 +1,52 @@
+module Spree
+  # Spree 5.4 で Spree::Property が削除されるため、ProductProperty で
+  # 保持していた商品の付加情報(gravity_forms / 地図形式 / 提供形態 / short_description 等)を
+  # Spree 5.2 の metafields に移行する。その読み書きを一箇所に集約する薄いヘルパー。
+  #
+  # 移行 rake(migrate_product_properties_to_metafields)は旧 ProductProperty を
+  #   namespace="properties", key=property.name, value=product_property.value
+  # で metafield にコピー済み。したがって本ヘルパーは namespace="properties" 規約のもとで
+  # 本家の get_metafield / set_metafield を薄くラップするだけにとどめ、本家 API を再実装しない。
+  #
+  # 日本語 key("地図形式" 等)は ba_spree/metafield_definition_decorator.rb が
+  # 本家 normalizes(parameterize) を無効化済みのため、そのまま通る(再 normalize 不要)。
+  module ProductPropertyMetafields
+    extend ActiveSupport::Concern
+
+    PROPERTY_NAMESPACE = 'properties'.freeze
+
+    # 旧 product.property(name) 相当。指定プロパティ名の metafield 値(文字列)を返す。無ければ nil。
+    def property_metafield_value(name)
+      get_metafield(property_metafield_key(name))&.value
+    end
+
+    # 旧 product_properties.create!/update 相当。指定プロパティ名の metafield に値を設定する。
+    # MetafieldDefinition は本家 set_metafield 内で find_or_create される。
+    def set_property_metafield(name, value)
+      set_metafield(property_metafield_key(name), value)
+    end
+
+    # 旧 product.has_property? 相当。
+    def property_metafield?(name)
+      has_metafield?(property_metafield_key(name))
+    end
+
+    # PDP 表示用。namespace="properties" かつフロント表示対象(available_on_front_end)の
+    # metafield を、定義の作成順で返す。旧
+    #   product_properties.joins(:property).merge(Spree::Property.available_on_front_end)
+    #     .sort_by_property_position
+    # の置き換え。各要素は Spree::Metafield で、name(=表示名) / value / id を持つ。
+    def front_end_property_metafields
+      public_metafields
+        .select { |mf| mf.metafield_definition&.namespace == PROPERTY_NAMESPACE }
+        .sort_by { |mf| mf.metafield_definition_id }
+    end
+
+    private
+
+    # "提供形態" -> "properties.提供形態"
+    def property_metafield_key(name)
+      "#{PROPERTY_NAMESPACE}.#{name}"
+    end
+  end
+end

data/app/models/spree/adjustable/adjuster/promotion_decorator.rb

@@ -0,0 +1,16 @@
+module Spree
+  module Adjustable
+    module Adjuster
+      module PromotionDecorator
+        # @gem-override spree_core-5.3.6/app/models/spree/adjustable/adjuster/promotion.rb#choose_best_promo_adjustment
+        # @see https://github.com/be-agile/giga-repeat/commit/3d5683b6c3e6875cfd808368f0ab21c020270ebb
+        # 本家は競合プロモーション adjustment のうち最良のもの以外を eligible: false にするが、
+        # ba_spree では複数プロモーションを併用させるため、この絞り込みを無効化する。
+        def choose_best_promo_adjustment
+        end
+      end
+    end
+  end
+end
+
+Spree::Adjustable::Adjuster::Promotion.prepend(Spree::Adjustable::Adjuster::PromotionDecorator)

data/app/models/spree/admin/table/filter_decorator.rb

@@ -0,0 +1,52 @@
+module Spree
+  module Admin
+    class Table
+      # 本家 Spree::Admin::Table::Filter の OPERATORS は label が英語ハードコード
+      # ("equals" 等)で、フィルタービルダーの演算子セレクトが日本語にならない
+      # 原因の一つ(Ruby 側)。operators_for_select はこの label を
+      # data-query-builder-operators-value として JS に渡すため、ここを i18n
+      # (admin.tables.operators.*) から引くよう上書きする。
+      #
+      # 注意: OPERATORS 定数を起動時に書き換えるとロケールが起動時で固定されて
+      # しまうため、リクエストごとに評価される operators_for_select 側で翻訳する。
+      module FilterDecorator
+        module ClassMethods
+          # OPERATORS の各 operator キー -> i18n キー(admin.tables.operators 配下)。
+          # キーと i18n キーの対応は本家ビュー(_filter_row 等)が使う
+          # admin.tables.operators.* に合わせる。
+          OPERATOR_I18N_KEYS = {
+            eq: :equals,
+            not_eq: :not_equals,
+            cont: :contains,
+            not_cont: :does_not_contain,
+            start: :starts_with,
+            end: :ends_with,
+            gt: :greater_than,
+            gteq: :greater_than_or_equal,
+            lt: :less_than,
+            lteq: :less_than_or_equal,
+            in: :is_any_of,
+            not_in: :is_none_of,
+            null: :is_empty,
+            not_null: :is_not_empty
+          }.freeze
+
+          def operators_for_select
+            super.map do |operator|
+              i18n_key = OPERATOR_I18N_KEYS[operator[:value].to_sym]
+              next operator unless i18n_key
+
+              operator.merge(label: Spree.t("admin.tables.operators.#{i18n_key}"))
+            end
+          end
+        end
+
+        def self.prepended(base)
+          base.singleton_class.prepend(ClassMethods)
+        end
+      end
+    end
+  end
+end
+
+Spree::Admin::Table::Filter.prepend(Spree::Admin::Table::FilterDecorator)

data/app/models/spree/admin/table/filter_group_decorator.rb

@@ -0,0 +1,64 @@
+# frozen_string_literal: true
+
+module Spree
+  module Admin
+    class Table
+      # フィルタービルダーの「または」グループが ransack に届かず、絞り込みとして機能していない。
+      #
+      # 本家 (spree_admin 5.3.6) の build_or_grouping には2つの問題がある:
+      #
+      # 1. p (述語) と v (値) を a (属性) の内側に入れている。
+      #    ransack は a と兄弟の階層に p / v を期待するため、grouping ごと黙って無視される
+      #    (= 全件が返る)。
+      # 2. 述語を operator.to_s.sub('_', '') で作っており、最初の _ を落としてしまう。
+      #    not_eq → "noteq"、not_cont → "notcont" のように不正な述語になる。
+      #
+      # @see https://github.com/be-agile/giga-repeat/issues/1262
+      module FilterGroupDecorator
+        private
+
+        # @gem-override spree_admin-5.3.6/app/models/spree/admin/table/filter_group.rb#build_or_grouping
+        # @see https://github.com/be-agile/giga-repeat/issues/1262
+        def build_or_grouping(group)
+          conditions = {}
+
+          group.filters.each_with_index do |filter, index|
+            param = filter.to_ransack_param
+            next if param.empty?
+
+            conditions[index.to_s] = {
+              a: { '0' => { name: filter.field } },
+              p: ransack_predicate_for(filter),
+              v: ransack_values_for(param.values.first)
+            }
+          end
+
+          {
+            m: 'or',
+            c: conditions
+          }
+        end
+
+        # ransack の述語名 (先頭の _ を除いた OPERATORS の predicate)
+        # @return [String]
+        def ransack_predicate_for(filter)
+          predicate = Spree::Admin::Table::Filter::OPERATORS.dig(filter.operator, :predicate)
+          predicate.to_s.delete_prefix('_')
+        end
+
+        # ransack の値。in / not_in のように配列を取る述語では、値をまとめて1つの v に入れると
+        # マッチしなくなる (0件になる) ため、要素ごとに v のエントリを作る。
+        # @return [Hash]
+        def ransack_values_for(value)
+          values = value.is_a?(Array) ? value : [ value ]
+          # 値が未入力 (空配列) の場合は空の v を作らない。nil を渡せば ransack がその条件を無視する
+          values = [ nil ] if values.empty?
+
+          values.each_with_index.to_h { |item, index| [ index.to_s, { value: item } ] }
+        end
+      end
+    end
+  end
+end
+
+Spree::Admin::Table::FilterGroup.prepend(Spree::Admin::Table::FilterGroupDecorator)

data/app/models/spree/asset_decorator.rb

@@ -0,0 +1,102 @@
+module Spree
+  # @gem-override spree_core-5.3.6/app/models/spree/asset.rb
+  # reason: Spree 5.3 で Asset#attachment に named variant の事前生成ブロック
+  #   (mini/small/medium/large/xlarge/og_image を format: webp + preprocessed: true で定義)が
+  #   追加された。preprocessed: true は画像 attach 時に 1 画像あたり 6 本の
+  #   ActiveStorage::TransformJob をキューし、さらに表示時に未生成 variant を
+  #   /rails/active_storage/representations/... でオンザフライ webp 変換する。
+  #
+  #   動産社等のローカル / stg では画像 blob の実体が本番 S3 にしか無く、変換が
+  #   元画像を取得できずに 1 リクエストあたり数百ms〜1秒の遅延・失敗となる。画像の多い
+  #   ページではこれが積み重なり、スモークテスト / ブラウザテストがタイムアウトする
+  #   (Issue #1210 のコメント参照)。
+  #
+  #   そこで「variant の定義自体は残す(ビューの image.variant(:large) 等が
+  #   ArgumentError にならないように)が、事前生成 preprocessed を環境フラグで
+  #   false に倒す」ことで、attach 時の TransformJob 大量発行を止める。
+  #   本番(フラグ未設定)では 5.3 デフォルトの preprocessed: true を維持する。
+  #
+  # 制御: ENV['DISABLE_IMAGE_VARIANT_PREPROCESSING'] が truthy のとき preprocessed を無効化。
+  #   development / test ではデフォルトで無効化する(画像実体が無い環境での遅延回避)。
+  #
+  # ⚠️ stg は RAILS_ENV=production で動作するため、env 名だけでは本番と区別できない。
+  #   stg で画像処理を止めたい場合は stg のデプロイ環境変数に
+  #   DISABLE_IMAGE_VARIANT_PREPROCESSING=true を明示設定すること。
+  #   本番環境では未設定のままにし、5.3 デフォルト(preprocessed: true)を維持する。
+  #
+  # @see https://github.com/be-agile/giga-repeat/issues/1210
+  module AssetDecorator
+    def self.prepended(base)
+      # acts_as_list の採番 (bottom_position_in_list + 1) はロックを取らないため、
+      # 管理画面の一括アップロードのように同一 viewable へ並列に INSERT すると
+      # 同じ position を持つレコードが同時に作られる。その状態になると並び替えが
+      # 効かなくなるため、採番より先に viewable の行を排他ロックして直列化する。
+      # acts_as_list の before_create :avoid_collision より先に走らせる必要がある。
+      # @see https://github.com/be-agile/giga-repeat/issues/1292
+      base.before_create :lock_position_scope, prepend: true
+
+      base.has_one_attached :attachment, service: Spree.public_storage_service_name do |attachable|
+        # 本家同様、キー順(format, resize_to_fill, saver)を維持して variation digest を一致させる。
+        Spree::Config.product_image_variant_sizes.each do |name, (width, height)|
+          attachable.variant name,
+                              format: 'webp',
+                              resize_to_fill: [ width, height ],
+                              saver: Spree::Asset::WEBP_SAVER_OPTIONS,
+                              preprocessed: !Spree::Asset.image_variant_preprocessing_disabled?
+        end
+      end
+    end
+
+    # viewable の行に排他ロックを取ったうえで position を採番し、同一 viewable への
+    # 並列 INSERT を直列化する。ロックはこの save のトランザクション終了時に解放される。
+    #
+    # 採番をここで行うのは、acts_as_list の採番が通常の SELECT (consistent read) で
+    # 最大 position を読むためである。MySQL の REPEATABLE READ では、行ロックの取得を
+    # 待っている間に他トランザクションが commit した行が consistent read から見えず、
+    # ロックを取っても同じ position を採番してしまう。ロック付き読み取り (FOR UPDATE) は
+    # 常に最新の確定値を読むため、こちらで採番する。
+    #
+    # viewable が未確定 (新規商品作成時のセッション退避アップロード等) の場合は
+    # ロック対象の行が無いため何もしない。この経路の position は
+    # Spree::Admin::ProductsControllerDecorator が付け替え後に詰め直す。
+    def lock_position_scope
+      return if viewable_id.blank? || viewable_type.blank?
+
+      lock_viewable_row
+      return if self[:position].present?
+
+      bottom = Spree::Asset.unscoped
+                           .where(viewable_type: viewable_type, viewable_id: viewable_id)
+                           .lock
+                           .maximum(:position)
+      self[:position] = bottom.to_i + 1
+    end
+
+    # 同一 viewable の position を操作する処理を直列化するための行ロック。
+    # 採番(lock_position_scope)と並び替え(Spree::Admin::AssetsControllerDecorator)の
+    # 双方が同じ行を取り合うことで、両者が混ざらないようにする。
+    def lock_viewable_row
+      return if viewable_id.blank? || viewable_type.blank?
+
+      viewable_class = viewable_type.safe_constantize
+      return if viewable_class.nil?
+
+      viewable_class.unscoped.lock.where(id: viewable_id).pick(:id)
+    end
+
+    module ClassMethods
+      # 画像 variant の事前生成(preprocessed)を無効化すべき環境かどうか。
+      # ENV['DISABLE_IMAGE_VARIANT_PREPROCESSING'] を最優先で見る(本番でも明示的に切り替え可能)。
+      # 未設定なら development / test で自動的に無効化する。
+      def image_variant_preprocessing_disabled?
+        flag = ENV['DISABLE_IMAGE_VARIANT_PREPROCESSING']
+        return ActiveModel::Type::Boolean.new.cast(flag) unless flag.nil?
+
+        Rails.env.development? || Rails.env.test?
+      end
+    end
+  end
+end
+
+Spree::Asset.singleton_class.prepend(Spree::AssetDecorator::ClassMethods)
+Spree::Asset.prepend(Spree::AssetDecorator)

data/app/models/spree/contact_decorator.rb

@@ -0,0 +1,22 @@
+module Spree
+  module ContactDecorator
+    def self.prepended(base)
+      base.attribute :mail_from_address, validate: true
+    end
+
+    # @gem-override spree_storefront-5.3.6/app/models/spree/contact.rb#headers
+    # @see https://github.com/be-agile/giga-repeat/issues/1092
+    # 本家は from を問い合わせ者本人のメールにするが、SES 未検証アドレスで 500 になるため
+    # from を mail_from_address（検証済み送信元）に変え、本人は reply_to に回す。
+    def headers
+      {
+        subject: 'Contact Us',
+        to: customer_support_email.to_s,
+        from: mail_from_address,
+        reply_to: %("#{name}" <#{email}>)
+      }
+    end
+  end
+end
+
+Spree::Contact.prepend Spree::ContactDecorator

data/app/models/spree/digital_link_decorator.rb

@@ -0,0 +1,9 @@
+module Spree
+  module DigitalLinkDecorator
+    def downloadable?
+      line_item.order.paid?
+    end
+  end
+end
+
+Spree::DigitalLink.prepend Spree::DigitalLinkDecorator