Product List: Replace full product objects with simplified objects

Modules/Sources/Yosemite/Tools/ResultsController.swift

@@ -51,7 +51,7 @@ public class GenericResultsController<T: ResultsControllerMutableType, Output> {
 
     /// Internal NSFetchedResultsController Instance.
     ///
-    private lazy var controller: NSFetchedResultsController<T> = {
+    public private(set) lazy var controller: NSFetchedResultsController<T> = {
         viewStorage.createFetchedResultsController(
                 fetchRequest: fetchRequest,
                 sectionNameKeyPath: sectionNameKeyPath,

Modules/Tests/YosemiteTests/Tools/ResultsControllerTests.swift

@@ -56,7 +56,6 @@ final class ResultsControllerTests: XCTestCase {
     ///
     func testResultsControllerPicksUpEntitiesAvailablePriorToInstantiation() {
         storageManager.insertSampleAccount()
-        viewStorage.saveIfNeeded()
 
         let resultsController = ResultsController<StorageAccount>(viewStorage: viewStorage, sortedBy: [sampleSortDescriptor])
         try? resultsController.performFetch()
@@ -70,12 +69,11 @@ final class ResultsControllerTests: XCTestCase {
     /// Verifies that ResultsController does pick up entities inserted after being instantiated.
     ///
     func testResultsControllerPicksUpEntitiesInsertedAfterInstantiation() {
+        storageManager.insertSampleAccount()
+
         let resultsController = ResultsController<StorageAccount>(viewStorage: viewStorage, sortedBy: [sampleSortDescriptor])
         try? resultsController.performFetch()
 
-        storageManager.insertSampleAccount()
-        viewStorage.saveIfNeeded()
-
         XCTAssertEqual(resultsController.sections.count, 1)
         XCTAssertEqual(resultsController.sections.first?.objects.count, 1)
         XCTAssertEqual(resultsController.sections.first?.numberOfObjects, 1)
@@ -85,18 +83,16 @@ final class ResultsControllerTests: XCTestCase {
     /// Verifies that `sectionNameKeyPath` effectively causes the ResultsController to produce multiple sections, based on the grouping parameter.
     ///
     func testResultsControllerGroupSectionsBySectionNameKeypath() {
-        let sectionNameKeyPath = "userID"
-        let resultsController = ResultsController<StorageAccount>(viewStorage: viewStorage,
-                                                                   sectionNameKeyPath: sectionNameKeyPath,
-                                                                   sortedBy: [sampleSortDescriptor])
-        try? resultsController.performFetch()
-
         let numberOfAccounts = 100
         for _ in 0 ..< numberOfAccounts {
             storageManager.insertSampleAccount()
         }
 
-        viewStorage.saveIfNeeded()
+        let sectionNameKeyPath = "userID"
+        let resultsController = ResultsController<StorageAccount>(viewStorage: viewStorage,
+                                                                   sectionNameKeyPath: sectionNameKeyPath,
+                                                                   sortedBy: [sampleSortDescriptor])
+        try? resultsController.performFetch()
 
         XCTAssertEqual(resultsController.sections.count, numberOfAccounts)
 
@@ -109,15 +105,13 @@ final class ResultsControllerTests: XCTestCase {
     /// Verifies that `object(at indexPath:)` effectively returns the expected (ReadOnly) Entity.
     ///
     func testObjectAtIndexPathReturnsExpectedEntity() {
+        let mutableAccount = storageManager.insertSampleAccount()
         let sectionNameKeyPath = "userID"
         let resultsController = ResultsController<StorageAccount>(viewStorage: viewStorage,
                                                                    sectionNameKeyPath: sectionNameKeyPath,
                                                                    sortedBy: [sampleSortDescriptor])
         try? resultsController.performFetch()
 
-        let mutableAccount = storageManager.insertSampleAccount()
-        viewStorage.saveIfNeeded()
-
         let indexPath = IndexPath(row: 0, section: 0)
         let readOnlyAccount = resultsController.object(at: indexPath)
 
@@ -144,7 +138,6 @@ final class ResultsControllerTests: XCTestCase {
         }
 
         storageManager.insertSampleAccount()
-        viewStorage.saveIfNeeded()
 
         waitForExpectations(timeout: Constants.expectationTimeout, handler: nil)
     }
@@ -167,7 +160,6 @@ final class ResultsControllerTests: XCTestCase {
         }
 
         storageManager.insertSampleAccount()
-        viewStorage.saveIfNeeded()
 
         waitForExpectations(timeout: Constants.expectationTimeout, handler: nil)
     }
@@ -189,7 +181,6 @@ final class ResultsControllerTests: XCTestCase {
         }
 
         storageManager.insertSampleAccount()
-        viewStorage.saveIfNeeded()
 
         waitForExpectations(timeout: Constants.expectationTimeout, handler: nil)
     }
@@ -211,7 +202,6 @@ final class ResultsControllerTests: XCTestCase {
         }
 
         storageManager.insertSampleAccount()
-        viewStorage.saveIfNeeded()
 
         waitForExpectations(timeout: Constants.expectationTimeout, handler: nil)
     }
@@ -228,7 +218,6 @@ final class ResultsControllerTests: XCTestCase {
         let second = storageManager.insertSampleAccount().toReadOnly()
         let expected = [first.userID: first, second.userID: second]
 
-        viewStorage.saveIfNeeded()
 
         for retrieved in resultsController.fetchedObjects {
             XCTAssertEqual(retrieved.username, expected[retrieved.userID]?.username)
@@ -239,14 +228,13 @@ final class ResultsControllerTests: XCTestCase {
     /// Verifies that `fetchedObjects` effectively  returns all of the (readOnly) objects that are expected to be available.
     ///
     func testResettingStorageIsMappedIntoOnResetClosure() {
+        storageManager.insertSampleAccount()
+        storageManager.insertSampleAccount()
+
         let sortDescriptor = NSSortDescriptor(key: #selector(getter: StorageAccount.userID).description, ascending: true)
         let resultsController = ResultsController<StorageAccount>(viewStorage: viewStorage, sortedBy: [sortDescriptor])
         try? resultsController.performFetch()
 
-        storageManager.insertSampleAccount()
-        storageManager.insertSampleAccount()
-
-        viewStorage.saveIfNeeded()
         XCTAssertEqual(resultsController.fetchedObjects.count, 2)
 
         let expectation = self.expectation(description: "OnDidReset")
@@ -296,8 +284,6 @@ final class ResultsControllerTests: XCTestCase {
             }
         }
 
-        viewStorage.saveIfNeeded()
-
         for (sectionNumber, sectionObject) in resultsController.sections.enumerated() {
             for (row, object) in sectionObject.objects.enumerated() {
                 let indexPath = IndexPath(row: row, section: sectionNumber)

RELEASE-NOTES.txt

@@ -7,6 +7,7 @@
 - [*] Fix initialization of authenticator to avoid crashes during login [https://github.com/woocommerce/woocommerce-ios/pull/15953]
 - [*] Shipping Labels: Made HS tariff number field required in customs form for EU destinations [https://github.com/woocommerce/woocommerce-ios/pull/15946]
 - [*] Order Details: Attempt to improve performance by using a simplified version of product objects. [https://github.com/woocommerce/woocommerce-ios/pull/15959]
+- [*] Product List: Load list with simplified product objects to improve performance. [https://teamkiwip2.wordpress.com/2025/08/01/hack-week-improving-performance-when-loading-cached-products/]
 - [*] Order Creation: Prevent subscription products to be added to an order [https://github.com/woocommerce/woocommerce-ios/pull/15960]
 - [internal] Replace COTS_DEVICE reader model name with TAP_TO_PAY_DEVICE. [https://github.com/woocommerce/woocommerce-ios/pull/15961]
 

WooCommerce/Classes/Copiable/Models+Copiable.generated.swift

@@ -50,6 +50,66 @@ extension WooCommerce.AggregateOrderItem {
     }
 }
 
+extension WooCommerce.ProductListItem {
+    func copy(
+        siteID: CopiableProp<Int64> = .copy,
+        productID: CopiableProp<Int64> = .copy,
+        name: CopiableProp<String> = .copy,
+        permalink: CopiableProp<String> = .copy,
+        fullDescription: NullableCopiableProp<String> = .copy,
+        shortDescription: NullableCopiableProp<String> = .copy,
+        productTypeKey: CopiableProp<String> = .copy,
+        statusKey: CopiableProp<String> = .copy,
+        sku: NullableCopiableProp<String> = .copy,
+        price: CopiableProp<String> = .copy,
+        manageStock: CopiableProp<Bool> = .copy,
+        stockQuantity: NullableCopiableProp<Decimal> = .copy,
+        stockStatusKey: CopiableProp<String> = .copy,
+        imageURL: NullableCopiableProp<URL> = .copy,
+        variations: CopiableProp<[Int64]> = .copy,
+        bundleStockStatus: NullableCopiableProp<ProductStockStatus> = .copy,
+        bundleStockQuantity: NullableCopiableProp<Int64> = .copy
+    ) -> WooCommerce.ProductListItem {
+        let siteID = siteID ?? self.siteID
+        let productID = productID ?? self.productID
+        let name = name ?? self.name
+        let permalink = permalink ?? self.permalink
+        let fullDescription = fullDescription ?? self.fullDescription
+        let shortDescription = shortDescription ?? self.shortDescription
+        let productTypeKey = productTypeKey ?? self.productTypeKey
+        let statusKey = statusKey ?? self.statusKey
+        let sku = sku ?? self.sku
+        let price = price ?? self.price
+        let manageStock = manageStock ?? self.manageStock
+        let stockQuantity = stockQuantity ?? self.stockQuantity
+        let stockStatusKey = stockStatusKey ?? self.stockStatusKey
+        let imageURL = imageURL ?? self.imageURL
+        let variations = variations ?? self.variations
+        let bundleStockStatus = bundleStockStatus ?? self.bundleStockStatus
+        let bundleStockQuantity = bundleStockQuantity ?? self.bundleStockQuantity
+
+        return WooCommerce.ProductListItem(
+            siteID: siteID,
+            productID: productID,
+            name: name,
+            permalink: permalink,
+            fullDescription: fullDescription,
+            shortDescription: shortDescription,
+            productTypeKey: productTypeKey,
+            statusKey: statusKey,
+            sku: sku,
+            price: price,
+            manageStock: manageStock,
+            stockQuantity: stockQuantity,
+            stockStatusKey: stockStatusKey,
+            imageURL: imageURL,
+            variations: variations,
+            bundleStockStatus: bundleStockStatus,
+            bundleStockQuantity: bundleStockQuantity
+        )
+    }
+}
+
 extension WooCommerce.ShippingLabelSelectedRate {
     func copy(
         packageID: CopiableProp<String> = .copy,

WooCommerce/Classes/Extensions/Product+ListItem.swift

@@ -0,0 +1,24 @@
+import Foundation
+import Yosemite
+
+extension Product {
+    func toListItem() -> ProductListItem {
+        ProductListItem(siteID: siteID,
+                        productID: productID,
+                        name: name,
+                        permalink: permalink,
+                        fullDescription: fullDescription,
+                        shortDescription: shortDescription,
+                        productTypeKey: productTypeKey,
+                        statusKey: statusKey,
+                        sku: sku,
+                        price: price,
+                        manageStock: manageStock,
+                        stockQuantity: stockQuantity,
+                        stockStatusKey: stockStatusKey,
+                        imageURL: imageURL,
+                        variations: variations,
+                        bundleStockStatus: bundleStockStatus,
+                        bundleStockQuantity: bundleStockQuantity)
+    }
+}

WooCommerce/Classes/ViewRelated/Products/Cells/ProductsTabProductTableViewCell.swift

@@ -305,9 +305,16 @@ private struct ProductsTabProductTableViewCellRepresentable: UIViewRepresentable
 }
 
 struct ProductsTabProductTableViewCell_Previews: PreviewProvider {
-    private static var nonSelectedViewModel = ProductsTabProductViewModel(product: Product.swiftUIPreviewSample(), isSelected: false)
-    private static var selectedViewModel = ProductsTabProductViewModel(product: Product.swiftUIPreviewSample().copy(statusKey: ProductStatus.pending.rawValue),
-                                                                       isSelected: true)
+    private static var nonSelectedViewModel = ProductsTabProductViewModel(
+        product: Product.swiftUIPreviewSample().toListItem(),
+        isSelected: false
+    )
+    private static var selectedViewModel = ProductsTabProductViewModel(
+        product: Product.swiftUIPreviewSample()
+            .toListItem()
+            .copy(statusKey: ProductStatus.pending.rawValue),
+        isSelected: true
+    )
 
     private static func makeStack() -> some View {
         VStack {

WooCommerce/Classes/ViewRelated/Products/Edit Product/Linked Products List Selector/LinkedProductListSelectorDataSource.swift

@@ -74,7 +74,7 @@ final class LinkedProductListSelectorDataSource: PaginatedListSelectorDataSource
     func configureCell(cell: ProductsTabProductTableViewCell, model: Product) {
         cell.selectionStyle = .none
 
-        let viewModel = ProductsTabProductViewModel(product: model, isDraggable: true)
+        let viewModel = ProductsTabProductViewModel(product: model.toListItem(), isDraggable: true)
         cell.update(viewModel: viewModel, imageService: imageService)
 
         cell.configureAccessoryDeleteButton { [weak self] in

WooCommerce/Classes/ViewRelated/Products/Edit Product/Linked Products List Selector/ProductListSelector/ProductListMultiSelectorDataSource.swift

@@ -55,7 +55,7 @@ final class ProductListMultiSelectorDataSource: PaginatedListSelectorDataSource
     func configureCell(cell: ProductsTabProductTableViewCell, model: Product) {
         cell.selectionStyle = .default
 
-        let viewModel = ProductsTabProductViewModel(product: model, isSelected: isSelected(model: model))
+        let viewModel = ProductsTabProductViewModel(product: model.toListItem(), isSelected: isSelected(model: model))
         cell.update(viewModel: viewModel, imageService: imageService)
     }
 

WooCommerce/Classes/ViewRelated/Products/Edit Product/Linked Products List Selector/ProductListSelector/ProductListMultiSelectorSearchUICommand.swift

@@ -71,7 +71,7 @@ final class ProductListMultiSelectorSearchUICommand: NSObject, SearchUICommand {
     }
 
     func createCellViewModel(model: Product) -> ProductsTabProductViewModel {
-        return ProductsTabProductViewModel(product: model, isSelected: isProductSelected(model))
+        return ProductsTabProductViewModel(product: model.toListItem(), isSelected: isProductSelected(model))
     }
 
     /// Synchronizes the Products matching a given Keyword