[Local catalog] Animate updates to the item list

Modules/Sources/Fakes/Yosemite.generated.swift

@@ -35,6 +35,23 @@ extension Yosemite.JustInTimeMessageTemplate {
         .banner
     }
 }
+extension Yosemite.POSItemIdentifier {
+    /// Returns a "ready to use" type filled with fake values.
+    ///
+    public static func fake() -> Yosemite.POSItemIdentifier {
+        .init(
+            underlyingType: .fake(),
+            itemID: .fake()
+        )
+    }
+}
+extension Yosemite.POSItemIdentifier.UnderlyingType {
+    /// Returns a "ready to use" type filled with fake values.
+    ///
+    public static func fake() -> Yosemite.POSItemIdentifier.UnderlyingType {
+        .product
+    }
+}
 extension Yosemite.POSSimpleProduct {
     /// Returns a "ready to use" type filled with fake values.
     ///

Modules/Sources/PointOfSale/Controllers/PointOfSaleOrderController.swift

@@ -291,7 +291,7 @@ private extension POSCart {
             guard case let .loaded(item) = purchasableItem.state else { return nil }
             return POSCartItem(item: item, quantity: Decimal(purchasableItem.quantity))
         }
-        let coupons = cart.coupons.map { POSCoupon(id: $0.id, code: $0.code, summary: $0.summary) }
+        let coupons = cart.coupons.map { POSCoupon(id: $0.posItemIdentifier, code: $0.code, summary: $0.summary) }
         self.init(items: items, coupons: coupons)
     }
 }

Modules/Sources/PointOfSale/Models/Cart.swift

@@ -2,6 +2,7 @@ import Foundation
 import protocol Yosemite.POSOrderableItem
 import enum Yosemite.POSItem
 import enum Yosemite.PointOfSaleBarcodeScanError
+import struct Yosemite.POSItemIdentifier
 
 struct Cart {
     var purchasableItems: [Cart.PurchasableItem] = []
@@ -12,6 +13,7 @@ struct Cart {
 
 protocol CartItem {
     var id: UUID { get }
+    var posItemIdentifier: POSItemIdentifier { get }
     var type: CartItemType { get }
 }
 
@@ -36,6 +38,17 @@ extension Cart {
             case error
         }
 
+        var posItemIdentifier: POSItemIdentifier {
+            switch state {
+            case .loaded(let item):
+                return item.id
+            case .loading:
+                return POSItemIdentifier(underlyingType: .loading, itemID: 0)
+            case .error:
+                return POSItemIdentifier(underlyingType: .error, itemID: 0)
+            }
+        }
+
         var formattedPrice: String? {
             switch state {
             case .loaded(let item):
@@ -76,6 +89,7 @@ extension Cart {
 
     struct CouponItem: CartItem {
         let id: UUID
+        let posItemIdentifier: POSItemIdentifier
         let code: String
         let summary: String
         let type: CartItemType = .coupon
@@ -89,7 +103,7 @@ extension Cart {
         if let purchasableItem = createPurchasableItem(from: posItem) {
             purchasableItems.insert(purchasableItem, at: purchasableItems.startIndex)
         } else if case .coupon(let coupon) = posItem {
-            let couponItem = Cart.CouponItem(id: coupon.id, code: coupon.code, summary: coupon.summary)
+            let couponItem = Cart.CouponItem(id: UUID(), posItemIdentifier: coupon.id, code: coupon.code, summary: coupon.summary)
             coupons.insert(couponItem, at: coupons.startIndex)
         }
     }

Modules/Sources/PointOfSale/Presentation/CartView.swift

@@ -479,7 +479,7 @@ private extension CartView {
 
 #Preview("Cart with one item") {
     let posModel = POSPreviewHelpers.makePreviewAggregateModel()
-    posModel.addToCart(.simpleProduct(.init(id: UUID(),
+    posModel.addToCart(.simpleProduct(.init(id: .init(underlyingType: .product, itemID: 6),
                                             name: "Sample Product",
                                             formattedPrice: "$10.00",
                                             productID: 6,

Modules/Sources/PointOfSale/Presentation/CouponRowView.swift

@@ -109,6 +109,9 @@ private extension CouponRowView {
 
 #if DEBUG
 #Preview(traits: .sizeThatFitsLayout) {
-    CouponRowView(couponItem: Cart.CouponItem(id: UUID(), code: "10-Discount", summary: "$10 Off · All products"), couponRowState: .idle) {}
+    CouponRowView(couponItem: Cart.CouponItem(id: UUID(),
+                                              posItemIdentifier: .init(underlyingType: .coupon, itemID: 123),
+                                              code: "10-Discount",
+                                              summary: "$10 Off · All products"), couponRowState: .idle) {}
 }
 #endif

Modules/Sources/PointOfSale/Presentation/Coupons/POSCouponCreationSheet.swift

@@ -2,6 +2,7 @@ import SwiftUI
 import class WooFoundation.CurrencySettings
 import struct Yosemite.Coupon
 import enum Yosemite.POSItem
+import struct Yosemite.POSItemIdentifier
 import struct Yosemite.POSCoupon
 
 extension View {
@@ -31,7 +32,8 @@ private struct POSCouponCreationSheetModifier: ViewModifier {
                     discountType: posDiscountType.discountType,
                     showTypeSelection: $showCouponSelectionSheet,
                     onSuccess: { coupon in
-                        addedCouponItem = .coupon(.init(id: UUID(), code: coupon.code, summary: coupon.summary(currencySettings: currencySettings)))
+                        let id = POSItemIdentifier(underlyingType: .coupon, itemID: coupon.couponID)
+                        addedCouponItem = .coupon(.init(id: id, code: coupon.code, summary: coupon.summary(currencySettings: currencySettings)))
                     },
                     dismissHandler: {
                         selectedType = nil

Modules/Sources/PointOfSale/Presentation/Item Selector/ChildItemList.swift

@@ -120,7 +120,7 @@ private extension ChildItemList {
 
 #Preview("Variable product child items") {
     let parentProduct = POSVariableParentProduct(
-        id: .init(),
+        id: .init(underlyingType: .product, itemID: 1),
         name: "Variable latte",
         productImageSource: nil,
         productID: 1
@@ -134,7 +134,7 @@ private extension ChildItemList {
                 [
                     .variation(
                         POSVariation(
-                            id: .init(),
+                            id: .init(underlyingType: .variation, itemID: 256),
                             name: "Cinamon chestnut latte",
                             formattedPrice: "$5.75",
                             price: "5.75",
@@ -145,12 +145,12 @@ private extension ChildItemList {
                     ),
                     .variation(
                         POSVariation(
-                            id: .init(),
+                            id: .init(underlyingType: .variation, itemID: 2567),
                             name: "Choco latte",
                             formattedPrice: "$6.5",
                             price: "6.5",
                             productID: 134,
-                            variationID: 256,
+                            variationID: 2567,
                             parentProductName: parentProduct.name
                         )
                     )
@@ -170,7 +170,7 @@ private extension ChildItemList {
 
 #Preview("Variable items load error") {
     let parentProduct = POSVariableParentProduct(
-        id: .init(),
+        id: .init(underlyingType: .product, itemID: 1),
         name: "Variable latte",
         productImageSource: nil,
         productID: 1

Modules/Sources/PointOfSale/Presentation/Item Selector/CouponCardView.swift

@@ -90,7 +90,7 @@ private extension CouponCardView {
                 .font(.caption)
                 .foregroundStyle(.secondary)
             CouponCardView(coupon: .init(
-                id: .init(),
+                id: .init(underlyingType: .coupon, itemID: 1),
                 code: "Coupon-123",
                 summary: "10% off - All Products"
             ))
@@ -101,7 +101,7 @@ private extension CouponCardView {
                 .font(.caption)
                 .foregroundStyle(.secondary)
             CouponCardView(coupon: .init(
-                id: .init(),
+                id: .init(underlyingType: .coupon, itemID: 2),
                 code: "Old-Coupon-123",
                 summary: "10% off - All Products",
                 dateExpires: Calendar.current.date(byAdding: .month, value: -1, to: Date())

Modules/Sources/PointOfSale/Presentation/Item Selector/ItemList.swift

@@ -68,6 +68,7 @@ struct ItemList<HeaderView: View>: View {
                     .frame(maxWidth: .infinity)
                     .padding(.horizontal, Constants.itemListPadding)
                     .padding(.bottom, keyboardObserver.isFullSizeKeyboardVisible ? Constants.itemListPadding : floatingControlAreaSize.height)
+                    .animation(.default, value: state?.items)
                 }
             )
 

Modules/Sources/PointOfSale/Presentation/Item Selector/POSItemActionHandler.swift

@@ -58,7 +58,7 @@ extension POSItemActionHandler {
     func shouldSkipDuplicate(_ item: POSItem, posModel: PointOfSaleAggregateModelProtocol) -> Bool {
         switch item {
         case .coupon:
-            return posModel.cart.coupons.contains(where: { $0.id == item.id })
+            return posModel.cart.coupons.contains(where: { $0.posItemIdentifier == item.id })
         default:
             return false
         }

Modules/Sources/PointOfSale/Presentation/Item Selector/SimpleProductCardView.swift

@@ -60,7 +60,7 @@ private extension SimpleProductCardView {
 
 #if DEBUG
 #Preview {
-    SimpleProductCardView(product: POSSimpleProduct(id: UUID(),
+    SimpleProductCardView(product: POSSimpleProduct(id: .init(underlyingType: .product, itemID: 123),
                                                     name: "Product name",
                                                     formattedPrice: "$3.00",
                                                     productID: 123,

Modules/Sources/PointOfSale/Presentation/Item Selector/VariationCardView.swift

@@ -48,7 +48,7 @@ private extension VariationCardView {
 }
 
 #Preview("Variation without image") {
-    let variation = POSVariation(id: .init(),
+    let variation = POSVariation(id: .init(underlyingType: .variation, itemID: 256),
                                  name: "500ml, double shot",
                                  formattedPrice: "$5.00",
                                  price: "5.00",
@@ -59,13 +59,13 @@ private extension VariationCardView {
 }
 
 #Preview("Variation with image") {
-    let variation = POSVariation(id: .init(),
+    let variation = POSVariation(id: .init(underlyingType: .variation, itemID: 257),
                                  name: "500ml, double shot",
                                  formattedPrice: "$5.00",
                                  price: "5.00",
                                  productImageSource: "https://pd.w.org/2024/12/986762d0d4d4cf17.82435881-scaled.jpeg",
                                  productID: 134,
-                                 variationID: 256,
+                                 variationID: 257,
                                  parentProductName: "Coffee")
     VariationCardView(variation: variation)
 }

Modules/Sources/PointOfSale/Utils/PreviewHelpers.swift

@@ -48,11 +48,12 @@ import protocol Yosemite.POSItemFetchAnalyticsTracking
 import protocol Yosemite.POSOrderListFetchStrategyFactoryProtocol
 import protocol Yosemite.POSOrderListFetchStrategy
 import protocol Yosemite.PointOfSaleCouponFetchStrategyFactoryProtocol
+import struct Yosemite.POSItemIdentifier
 
 // MARK: - PreviewProvider helpers
 //
 struct POSProductPreview: POSOrderableItem, Equatable {
-    let id: UUID
+    let id: POSItemIdentifier
     let name: String
     let formattedPrice: String
     var productImageSource: String?
@@ -86,7 +87,7 @@ final class PointOfSalePreviewItemService: PointOfSaleItemServiceProtocol {
     }
 
     func providePointOfSaleItem() -> POSOrderableItem {
-        POSProductPreview(id: UUID(),
+        POSProductPreview(id: POSItemIdentifier(underlyingType: .product, itemID: 1),
                           name: "Product 1",
                           formattedPrice: "$1.00")
     }
@@ -168,7 +169,7 @@ private var mockItems: [POSItem] {
         mockSimpleProductItem(id: 3, price: "3.00"),
         .variableParentProduct(
             .init(
-                id: .init(),
+                id: POSItemIdentifier(underlyingType: .product, itemID: 5),
                 name: "Variable product 1",
                 productImageSource: nil,
                 productID: 5
@@ -179,7 +180,7 @@ private var mockItems: [POSItem] {
 }
 
 private func mockSimpleProductItem(id: Int, price: String) -> POSItem {
-    .simpleProduct(POSSimpleProduct(id: UUID(),
+    .simpleProduct(POSSimpleProduct(id: POSItemIdentifier(underlyingType: .product, itemID: Int64(id)),
                                     name: "Product \(id)",
                                     formattedPrice: "$\(price)",
                                     productID: Int64(id),
@@ -191,19 +192,19 @@ private func mockSimpleProductItem(id: Int, price: String) -> POSItem {
 
 private var mockVariationItems: [POSItem] {
     [
-        .variation(.init(id: UUID(),
+        .variation(.init(id: POSItemIdentifier(underlyingType: .variation, itemID: 256),
                          name: "Variation 1",
                          formattedPrice: "$1.00",
                          price: "1.00",
                          productID: 134,
                          variationID: 256,
                          parentProductName: "Variable product")),
-        .variation(.init(id: UUID(),
+        .variation(.init(id: POSItemIdentifier(underlyingType: .variation, itemID: 257),
                          name: "Variation 2",
                          formattedPrice: "$2.00",
                          price: "2.00",
                          productID: 134,
-                         variationID: 256,
+                         variationID: 257,
                          parentProductName: "Variable product")),
     ]
 }

Modules/Sources/Yosemite/Model/Copiable/Models+Copiable.generated.swift

@@ -5,6 +5,7 @@ import Foundation
 import Networking
 import WooFoundation
 import enum NetworkingCore.OrderStatusEnum
+import struct Networking.PagedItems
 import struct NetworkingCore.Address
 import struct NetworkingCore.MetaData
 import struct NetworkingCore.Order
@@ -58,6 +59,21 @@ extension Yosemite.JustInTimeMessage {
     }
 }
 
+extension Yosemite.POSItemIdentifier {
+    public func copy(
+        underlyingType: CopiableProp<POSItemIdentifier.UnderlyingType> = .copy,
+        itemID: CopiableProp<Int64> = .copy
+    ) -> Yosemite.POSItemIdentifier {
+        let underlyingType = underlyingType ?? self.underlyingType
+        let itemID = itemID ?? self.itemID
+
+        return Yosemite.POSItemIdentifier(
+            underlyingType: underlyingType,
+            itemID: itemID
+        )
+    }
+}
+
 extension Yosemite.POSOrder {
     public func copy(
         id: CopiableProp<Int64> = .copy,
@@ -111,7 +127,7 @@ extension Yosemite.POSOrder {
 
 extension Yosemite.POSSimpleProduct {
     public func copy(
-        id: CopiableProp<UUID> = .copy,
+        id: CopiableProp<POSItemIdentifier> = .copy,
         name: CopiableProp<String> = .copy,
         formattedPrice: CopiableProp<String> = .copy,
         productImageSource: NullableCopiableProp<String> = .copy,

Modules/Sources/Yosemite/PointOfSale/Coupons/POSCoupon.swift

@@ -1,12 +1,12 @@
 import Foundation
 
 public struct POSCoupon: Equatable, Hashable {
-    public let id: UUID
+    public let id: POSItemIdentifier
     public let code: String
     public let summary: String
     public var dateExpires: Date?
 
-    public init(id: UUID, code: String, summary: String = "", dateExpires: Date? = nil) {
+    public init(id: POSItemIdentifier, code: String, summary: String = "", dateExpires: Date? = nil) {
         self.id = id
         self.code = code
         self.summary = summary

Modules/Sources/Yosemite/PointOfSale/Coupons/PointOfSaleCouponFetchStrategy.swift

@@ -169,7 +169,7 @@ private struct CouponResultsControllerAdapter {
     private func mapCouponsToPOSItems(coupons: [Coupon]) -> [POSItem] {
         coupons.compactMap { coupon in
                 .coupon(POSCoupon(
-                    id: UUID(),
+                    id: POSItemIdentifier(underlyingType: .coupon, itemID: coupon.couponID),
                     code: coupon.code,
                     summary: coupon.summary(currencySettings: currencySettings),
                     dateExpires: coupon.dateExpires

Modules/Sources/Yosemite/PointOfSale/Items/POSSimpleProduct.swift

@@ -5,7 +5,7 @@ import Networking
 
 public struct POSSimpleProduct: POSOrderableItem, OrderSyncProductTypeProtocol {
     // POSOrderableItem
-    public let id: UUID
+    public let id: POSItemIdentifier
     public let name: String
     public let formattedPrice: String
     public var productImageSource: String?
@@ -24,7 +24,7 @@ public struct POSSimpleProduct: POSOrderableItem, OrderSyncProductTypeProtocol {
         return ProductStockStatus(rawValue: stockStatusKey)
     }
 
-    public init(id: UUID,
+    public init(id: POSItemIdentifier,
                 name: String,
                 formattedPrice: String,
                 productImageSource: String? = nil,