@bobfrankston/mailx-imap 0.1.113 → 0.1.115

package/index.d.ts

@@ -701,7 +701,10 @@ export declare class ImapManager extends EventEmitter {
      *  contacts added or removed in this run. */
     syncGoogleContacts(accountId: string): Promise<number>;
     private syncGoogleContactsImpl;
-    /** Sync contacts for all OAuth accounts */
+    /** Sync contacts for all GOOGLE OAuth accounts. The People API only
+     *  accepts Google tokens — feeding it an Outlook/AOL OAuth token just
+     *  logged `API error for outlook: 401` on every periodic tick. Outlook
+     *  contacts belong to the Graph provider (blocked on Azure app reg). */
     syncAllContacts(): Promise<void>;
     /** Shut down all watchers and timers */
     shutdown(): Promise<void>;

package/index.js

@@ -906,7 +906,54 @@ export class ImapManager extends EventEmitter {
             // to re-enable for a debugging session. Default is quiet.
             const imapTrace = process.env.RMFMAIL_IMAP_TRACE === "1";
             const cfgWithVerbose = (imapTrace && (purpose === "ops" || purpose === "fast")) ? { ...config, verbose: true } : config;
-            client = new CompatImapClient(cfgWithVerbose, this.transportFactory);
+            // C121: connect-phase progress. Wrap the injected transport so the
+            // socket phase (DNS+TCP+TLS inside transport.connect) is visibly
+            // distinct from the IMAP handshake (greeting + LOGIN, which run
+            // inside client.connect after the socket is up). The client shows
+            // "Connecting to <host> (<phase>, N.Ns)" from these events, so
+            // "DNS/TCP/TLS hung" vs "greeting missing" vs "LOGIN slow" are
+            // distinguishable at a glance instead of one opaque long wait.
+            let connT0 = Date.now();
+            const phaseT = {};
+            const emitPhase = (phase, detail) => {
+                phaseT[phase] = Date.now();
+                this.emit("connectProgress", accountId, {
+                    host, purpose, phase, detail,
+                    elapsedMs: Date.now() - connT0,
+                });
+            };
+            const phaseTimings = () => Object.entries(phaseT).map(([p, t]) => `${p}@${t - connT0}ms`).join(" ");
+            const wrappedFactory = () => {
+                const t = this.transportFactory();
+                const origTransportConnect = t.connect.bind(t);
+                t.connect = async (h, p, tls, sni) => {
+                    emitPhase("socket");
+                    await origTransportConnect(h, p, tls, sni);
+                    emitPhase("handshake");
+                };
+                return t;
+            };
+            client = new CompatImapClient(cfgWithVerbose, wrappedFactory);
+            const origClientConnect = client.connect?.bind(client);
+            if (origClientConnect) {
+                client.connect = async (...a) => {
+                    // Clock starts when the connect actually begins — clients
+                    // are constructed eagerly but connect lazily on first use.
+                    connT0 = Date.now();
+                    try {
+                        const r = await origClientConnect(...a);
+                        emitPhase("done");
+                        const total = Date.now() - connT0;
+                        if (total > 3000)
+                            console.log(`  [conn-slow] ${accountId} (${purpose}): connect took ${total}ms — ${phaseTimings()}`);
+                        return r;
+                    }
+                    catch (e) {
+                        emitPhase("failed", e?.message || String(e));
+                        throw e;
+                    }
+                };
+            }
         }
         catch (e) {
             releaseHostSlot();
@@ -2172,7 +2219,7 @@ export class ImapManager extends EventEmitter {
                     for (const uid of toDelete) {
                         this.scheduleDeferredReconcileDelete(accountId, folderId, uid, folder.path);
                     }
-                    console.log(`  [reconcile-defer] ${accountId}/${folder.path}: scheduled ${toDelete.length} deletes (60s grace for move-detect)`);
+                    console.log(`  [reconcile-defer] ${accountId}/${folder.path}: scheduled ${toDelete.length} deletes (${ImapManager.RECONCILE_DELETE_GRACE_MS / 60_000}min grace for move-detect)`);
                 }
             }
             catch (e) {
@@ -5996,13 +6043,20 @@ export class ImapManager extends EventEmitter {
         }
         return changed;
     }
-    /** Sync contacts for all OAuth accounts */
+    /** Sync contacts for all GOOGLE OAuth accounts. The People API only
+     *  accepts Google tokens — feeding it an Outlook/AOL OAuth token just
+     *  logged `API error for outlook: 401` on every periodic tick. Outlook
+     *  contacts belong to the Graph provider (blocked on Azure app reg). */
     async syncAllContacts() {
         const settings = loadSettings();
         for (const account of settings.accounts) {
-            if (account.imap.auth === "oauth2" && (account.enabled || account.syncContacts)) {
-                await this.syncGoogleContacts(account.id);
-            }
+            if (account.imap.auth !== "oauth2")
+                continue;
+            if (!(account.enabled || account.syncContacts))
+                continue;
+            if (!this.isGmailAccount(account.id))
+                continue;
+            await this.syncGoogleContacts(account.id);
         }
     }
     /** Shut down all watchers and timers */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx-imap",
-  "version": "0.1.113",
+  "version": "0.1.115",
   "type": "module",
   "main": "index.js",
   "types": "index.d.ts",
@@ -9,14 +9,14 @@
   },
   "license": "ISC",
   "dependencies": {
-    "@bobfrankston/mailx-types": "^0.1.22",
-    "@bobfrankston/mailx-settings": "^0.1.33",
+    "@bobfrankston/mailx-types": "^0.1.23",
+    "@bobfrankston/mailx-settings": "^0.1.34",
     "@bobfrankston/mailx-store": "^0.1.58",
     "@bobfrankston/iflow-direct": "^0.1.56",
-    "@bobfrankston/tcp-transport": "^0.1.7",
+    "@bobfrankston/tcp-transport": "^0.1.8",
     "@bobfrankston/smtp-direct": "^0.1.9",
     "@bobfrankston/mailx-sync": "^0.1.27",
-    "@bobfrankston/oauthsupport": "^1.0.33"
+    "@bobfrankston/oauthsupport": "^1.0.34"
   },
   "repository": {
     "type": "git",
@@ -37,14 +37,14 @@
   },
   ".transformedSnapshot": {
     "dependencies": {
-      "@bobfrankston/mailx-types": "^0.1.22",
-      "@bobfrankston/mailx-settings": "^0.1.33",
+      "@bobfrankston/mailx-types": "^0.1.23",
+      "@bobfrankston/mailx-settings": "^0.1.34",
       "@bobfrankston/mailx-store": "^0.1.58",
       "@bobfrankston/iflow-direct": "^0.1.56",
-      "@bobfrankston/tcp-transport": "^0.1.7",
+      "@bobfrankston/tcp-transport": "^0.1.8",
       "@bobfrankston/smtp-direct": "^0.1.9",
       "@bobfrankston/mailx-sync": "^0.1.27",
-      "@bobfrankston/oauthsupport": "^1.0.33"
+      "@bobfrankston/oauthsupport": "^1.0.34"
     }
   }
 }