Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions .jules/bolt.md
Original file line number Diff line number Diff line change
Expand Up @@ -28,3 +28,6 @@
## 2025-10-24 - Bulk Disk I/O Parallelization and Thread Pool Starvation
**Learning:** Both `withTaskGroup` and `Task.detached` schedule their work on Swift's cooperative thread pool, which has only as many threads as the CPU has cores. Running synchronous blocking I/O (like `FileManager.removeItem`) directly inside such tasks ties up cooperative threads β€” when every thread is parked in a syscall there is nothing left to advance other Swift Concurrency work, which manifests as starvation and (with self-referential `await` chains) outright deadlock. `Task.detached` does not help here: "detached" means unstructured/independent, not "off the cooperative pool."
**Action:** To parallelize bulk blocking I/O, combine a sliding-window `withThrowingTaskGroup` (e.g., `maxConcurrency` of 8) with a per-item handoff to a GCD queue: wrap the blocking call in `withCheckedThrowingContinuation` and dispatch it via `DispatchQueue.global(qos: .userInitiated).async { ... continuation.resume(...) }`. The cooperative-pool task only `await`s the continuation, so it never holds a thread while the syscall runs.
## 2024-05-30 - Short-circuit evaluations on arrays
**Learning:** Using `.isEmpty` on a filtered array forces eager evaluation of the entire filter criteria. For checking if any elements match a condition, it's heavily inefficient.
**Action:** Use `.contains(where:)` instead of checking `.isEmpty` on a filtered array to short-circuit evaluation, stopping at the first match.
6 changes: 4 additions & 2 deletions Sources/Cacheout/ViewModels/CacheoutViewModel.swift
Original file line number Diff line number Diff line change
Expand Up @@ -105,8 +105,9 @@ class CacheoutViewModel: ObservableObject {
scanResults.filter { $0.isSelected }
}

// ⚑ Bolt Optimization: Use .lazy chaining for reduce to prevent intermediate array allocation
var selectedSize: Int64 {
selectedResults.reduce(0) { $0 + $1.sizeBytes }
scanResults.lazy.filter { $0.isSelected }.reduce(0) { $0 + $1.sizeBytes }
}

var formattedSelectedSize: String {
Expand All @@ -118,7 +119,8 @@ class CacheoutViewModel: ObservableObject {
}

var hasResults: Bool { !scanResults.isEmpty || !nodeModulesItems.isEmpty }
var hasSelection: Bool { !selectedResults.isEmpty || selectedNodeModulesSize > 0 }
// ⚑ Bolt Optimization: Use contains(where:) to short-circuit rather than filtering entire array to check emptiness
var hasSelection: Bool { scanResults.contains(where: { $0.isSelected }) || selectedNodeModulesSize > 0 }

// MARK: - Node Modules computed properties

Expand Down
Loading