From d1aae9219c906ae924a445a0a1c6466eaa301604 Mon Sep 17 00:00:00 2001 From: "google-labs-jules[bot]" <161369871+google-labs-jules[bot]@users.noreply.github.com> Date: Sat, 11 Apr 2026 05:12:49 +0000 Subject: [PATCH] =?UTF-8?q?=E2=9A=A1=20Bolt:=20Batch=20@Published=20array?= =?UTF-8?q?=20updates=20to=20reduce=20UI=20re-renders?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Replaces `for` loops with `.map` assignments when mutating arrays within `CacheoutViewModel` to prevent individual elements triggering sequential UI redraw notifications. Co-authored-by: acebytes <2820910+acebytes@users.noreply.github.com> --- .jules/bolt.md | 3 ++ .../ViewModels/CacheoutViewModel.swift | 40 +++++++++++++++---- 2 files changed, 35 insertions(+), 8 deletions(-) create mode 100644 .jules/bolt.md diff --git a/.jules/bolt.md b/.jules/bolt.md new file mode 100644 index 0000000..9f23453 --- /dev/null +++ b/.jules/bolt.md @@ -0,0 +1,3 @@ +## 2024-04-11 - [Batch @Published Array Mutations] +**Learning:** Mutating individual elements of a `@Published` array inside a loop triggers a UI update for each change. +**Action:** For collections of value types, use functional methods like `.map` to batch updates into a single assignment, significantly reducing unnecessary UI recalculations. diff --git a/Sources/Cacheout/ViewModels/CacheoutViewModel.swift b/Sources/Cacheout/ViewModels/CacheoutViewModel.swift index 13a9811..877390e 100644 --- a/Sources/Cacheout/ViewModels/CacheoutViewModel.swift +++ b/Sources/Cacheout/ViewModels/CacheoutViewModel.swift @@ -172,14 +172,23 @@ class CacheoutViewModel: ObservableObject { } func selectAllSafe() { - for i in scanResults.indices where scanResults[i].category.riskLevel == .safe && !scanResults[i].isEmpty { - scanResults[i].isSelected = true + // Optimization: Batch @Published updates to avoid multiple UI view redraws + // Mutating individual elements of a @Published array inside a loop triggers a UI update for each change. + scanResults = scanResults.map { result in + var newResult = result + if result.category.riskLevel == .safe && !result.isEmpty { + newResult.isSelected = true + } + return newResult } } func deselectAll() { - for i in scanResults.indices { - scanResults[i].isSelected = false + // Optimization: Batch @Published updates to avoid multiple UI view redraws + scanResults = scanResults.map { result in + var newResult = result + newResult.isSelected = false + return newResult } deselectAllNodeModules() } @@ -193,17 +202,32 @@ class CacheoutViewModel: ObservableObject { } func selectStaleNodeModules() { - for i in nodeModulesItems.indices where nodeModulesItems[i].isStale { - nodeModulesItems[i].isSelected = true + // Optimization: Batch @Published updates to avoid multiple UI view redraws + nodeModulesItems = nodeModulesItems.map { item in + var newItem = item + if item.isStale { + newItem.isSelected = true + } + return newItem } } func selectAllNodeModules() { - for i in nodeModulesItems.indices { nodeModulesItems[i].isSelected = true } + // Optimization: Batch @Published updates to avoid multiple UI view redraws + nodeModulesItems = nodeModulesItems.map { item in + var newItem = item + newItem.isSelected = true + return newItem + } } func deselectAllNodeModules() { - for i in nodeModulesItems.indices { nodeModulesItems[i].isSelected = false } + // Optimization: Batch @Published updates to avoid multiple UI view redraws + nodeModulesItems = nodeModulesItems.map { item in + var newItem = item + newItem.isSelected = false + return newItem + } } /// Menu bar label: show free GB in the tray