From 1307ebd3e69bc19e289c053ed3b147611fdf51a6 Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Thu, 23 Jul 2026 21:32:51 +0000
Subject: [PATCH 1/2] =?UTF-8?q?=E2=9A=A1=20Bolt:=20Optimize=20ArtifactStor?=
 =?UTF-8?q?e.gc=20with=20batched=20operations?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .jules/bolt.md                        |  3 ++
 src/core/sub-agent/artifacts/store.ts | 48 +++++++++++++++++----------
 2 files changed, 34 insertions(+), 17 deletions(-)
 create mode 100644 .jules/bolt.md

diff --git a/.jules/bolt.md b/.jules/bolt.md
new file mode 100644
index 00000000..06f5c2f2
--- /dev/null
+++ b/.jules/bolt.md
@@ -0,0 +1,3 @@
+## 2026-07-23 - Batched FS Operations in GC
+**Learning:** Sequential `fs.stat` and `fs.rm` operations in file garbage collection (like `ArtifactStore.gc`) cause performance bottlenecks and can hit EMFILE limits when iterating over many files. Additionally, doing file deletion within the loop that calculates total size leads to race conditions.
+**Action:** Always process files sequentially to build a safe removal queue first, then execute file deletions (e.g. `fs.rm`) concurrently in chunks (e.g. `Promise.all` with chunk size 10) to prevent performance bottlenecks and race conditions during state calculation.
diff --git a/src/core/sub-agent/artifacts/store.ts b/src/core/sub-agent/artifacts/store.ts
index 8a7aff5c..b4a3f5ad 100644
--- a/src/core/sub-agent/artifacts/store.ts
+++ b/src/core/sub-agent/artifacts/store.ts
@@ -92,14 +92,24 @@ export class ArtifactStore {
     const root = getArtifactsRoot();
     const entries = await fs.readdir(root, { withFileTypes: true }).catch(() => []);
 
+    const fileEntries = entries.filter((e) => e.isFile());
     const files: Array<{ name: string; path: string; mtimeMs: number; size: number }> = [];
-    for (const entry of entries) {
-      if (!entry.isFile()) continue;
-      const filePath = path.join(root, entry.name);
-      if (!isWithinDir(root, filePath)) continue;
-      const stat = await fs.stat(filePath).catch(() => null);
-      if (!stat) continue;
-      files.push({ name: entry.name, path: filePath, mtimeMs: stat.mtimeMs, size: stat.size });
+
+    // Process stats in chunks of 10 to avoid EMFILE and improve performance
+    for (let i = 0; i < fileEntries.length; i += 10) {
+      const chunk = fileEntries.slice(i, i + 10);
+      const statPromises = chunk.map(async (entry) => {
+        const filePath = path.join(root, entry.name);
+        if (!isWithinDir(root, filePath)) return null;
+        const stat = await fs.stat(filePath).catch(() => null);
+        if (!stat) return null;
+        return { name: entry.name, path: filePath, mtimeMs: stat.mtimeMs, size: stat.size };
+      });
+
+      const chunkResults = await Promise.all(statPromises);
+      for (const result of chunkResults) {
+        if (result) files.push(result);
+      }
     }
 
     const maxAgeMs = options?.maxAgeMs ?? LIMITS.artifactTtlMs;
@@ -112,15 +122,7 @@ export class ArtifactStore {
     let removedFiles = 0;
     let removedBytes = 0;
 
-    const removeFile = async (file: { path: string; size: number }) => {
-      await fs.rm(file.path, { force: true }).catch(() => null);
-      removedFiles += 1;
-      removedBytes += file.size;
-    };
-
-    for (const file of expired) {
-      await removeFile(file);
-    }
+    const filesToRemove: Array<{ path: string; size: number }> = [...expired];
 
     // Recompute remaining after TTL removal (newest first).
     const remaining = files
@@ -136,11 +138,23 @@ export class ArtifactStore {
       if (!tooManyFiles && !tooManyBytes) break;
 
       const oldest = remaining[i];
-      await removeFile(oldest);
+      filesToRemove.push(oldest);
       currentFiles -= 1;
       currentBytes -= oldest.size;
     }
 
+    // Process removals in chunks of 10
+    for (let i = 0; i < filesToRemove.length; i += 10) {
+      const chunk = filesToRemove.slice(i, i + 10);
+      await Promise.all(
+        chunk.map(async (file) => {
+          await fs.rm(file.path, { force: true }).catch(() => null);
+          removedFiles += 1;
+          removedBytes += file.size;
+        }),
+      );
+    }
+
     return { removedFiles, removedBytes };
   }
 

From 4fd7c96d15078324a2c97c001c4e14d0aaafad83 Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Thu, 23 Jul 2026 22:04:05 +0000
Subject: [PATCH 2/2] =?UTF-8?q?=E2=9A=A1=20Bolt:=20Optimize=20ArtifactStor?=
 =?UTF-8?q?e.gc=20with=20batched=20operations?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit