fix: stagger IMAP startup connections; remove in-CI version bump that mutated main

Author: maathimself

.github/workflows/publish.yml

@@ -6,7 +6,7 @@ on:
       - 'v*.*.*'
 
 permissions:
-  contents: write
+  contents: read
   packages: write
 
 jobs:
@@ -23,18 +23,6 @@ jobs:
       - name: Set up Docker Buildx
         uses: docker/setup-buildx-action@v3
 
-      - name: Bump package.json versions
-        run: |
-          VERSION="${{ github.ref_name }}"
-          VERSION="${VERSION#v}"
-          jq --arg v "$VERSION" '.version = $v' backend/package.json > /tmp/pkg.json && mv /tmp/pkg.json backend/package.json
-          jq --arg v "$VERSION" '.version = $v' frontend/package.json > /tmp/pkg.json && mv /tmp/pkg.json frontend/package.json
-          git config user.name "github-actions[bot]"
-          git config user.email "github-actions[bot]@users.noreply.github.com"
-          git add backend/package.json frontend/package.json
-          git diff --cached --quiet || git commit -m "chore: bump version to ${VERSION}"
-          git push origin HEAD:main
-
       - name: Log in to GHCR
         uses: docker/login-action@v3
         with:

backend/src/index.js

@@ -155,17 +155,20 @@ await loadIntegrationConfigs();
 // Start background snooze watcher — polls every 60 seconds to restore snoozed messages
 imapManager.startSnoozeWatcher();
 
-// Re-connect all enabled IMAP accounts on startup so mail syncs immediately
-// even before any WebSocket client connects (covers cold-start and container restarts)
+// Re-connect all enabled IMAP accounts on startup, staggered 500 ms apart so a
+// large user base doesn't hammer IMAP servers and the DB connection pool at once.
 try {
   const startupResult = await query(
     "SELECT DISTINCT user_id FROM email_accounts WHERE enabled = true AND protocol = 'imap'"
   );
-  for (const row of startupResult.rows) {
-    imapManager.connectAllForUser(row.user_id).catch(err =>
-      console.error(`Startup connect failed for user ${row.user_id}:`, err.message)
+  startupResult.rows.forEach(({ user_id }, i) => {
+    setTimeout(
+      () => imapManager.connectAllForUser(user_id).catch(err =>
+        console.error(`Startup connect failed for user ${user_id}:`, err.message)
+      ),
+      i * 500,
     );
-  }
+  });
   if (startupResult.rows.length) {
     console.log(`Reconnecting accounts for ${startupResult.rows.length} user(s) on startup`);
   }