@bobfrankston/rmfmail 1.1.245 → 1.1.247

package/packages/mailx-service/index.ts

@@ -2379,18 +2379,20 @@ export class MailxService implements MailxApi {
         try {
             await client.createmailbox(fullPath);
             // Local-first: insert the new folder into the DB immediately so
-            // the UI shows it on the next tree refresh — no waiting for a
-            // round-trip LIST in syncFolders. The next syncFolders pass will
-            // re-upsert with canonical special-use / delimiter (no-op here,
-            // since CREATE folders never carry special-use flags).
+            // the UI shows it on the next tree refresh.
             this.db.upsertFolder(accountId, fullPath, name, null, delimiter);
             this.imapManager.emit("folderCountsChanged", accountId, {});
-            // Best-effort full resync — verifies the server accepted CREATE
-            // and picks up anything else changed since the last pass. Async
-            // so the IPC returns quickly; the optimistic row is already in
-            // place if this lags.
-            this.imapManager.syncFolders(accountId, client).catch((e: any) =>
-                console.error(`  [createFolder] post-create syncFolders failed: ${e?.message || e}`));
+            // VERIFY the server actually created it, and AWAIT it. createmailbox
+            // on a wedged/congested connection has returned without error while
+            // the CREATE never reached the server — so the IPC reported success,
+            // the optimistic row got dropped by the next reconcile, and the
+            // folder silently never appeared (Bob 2026-06-12, Archive/dbgtrash).
+            // A folder create is infrequent; correctness beats the fast return.
+            await this.imapManager.syncFolders(accountId, client);
+            const created = this.db.getFolders(accountId).some(f => f.path === fullPath);
+            if (!created) {
+                throw new Error(`Folder "${name}" was not created on the server — please try again (the connection may have been busy).`);
+            }
         } finally { try { await client.logout(); } catch { /* */ } }
     }