hypermail-mcp 0.6.0 → 0.6.1

package/README.md

@@ -3,6 +3,10 @@
 A **Model Context Protocol** server that lets an agent operate any of the user's
 inboxes through a single, unified tool surface.
 
+> **v0.6.1** — Docker deployment (standalone Dockerfile with HEALTHCHECK),
+> email notification bug fixes (ID-based dedup, pagination cap, dynamic
+> re-scan), Node 22 base image, dropped docker-compose.
+>
 > **v0.6.0** — Email watch notifications (polling-based), `signaturePath`
 > support in `set_account_settings` for loading signatures from files,
 > and a `check_notifications` tool for draining pending alerts.
@@ -72,6 +76,27 @@ hypermail-mcp --http --port 3000 --host 0.0.0.0
 When hosted you **must** set `HYPERMAIL_MCP_KEY` so the account file is
 reproducibly decryptable.
 
+### Docker
+
+```bash
+# Build
+docker build -t hypermail-mcp .
+
+# Run
+docker run -d \
+  --name hypermail-mcp \
+  -p 3000:3000 \
+  -e HYPERMAIL_MCP_KEY=<32-byte-key> \
+  -e MS_CLIENT_ID=<your-client-id> \
+  -e MS_TENANT_ID=<your-tenant-id> \
+  -v hypermail-data:/data \
+  hypermail-mcp
+```
+
+The image runs the server in HTTP mode on port 3000 with a 30-second
+HEALTHCHECK against `/mcp`. Data is persisted via a Docker volume at `/data`.
+Pass `HYPERMAIL_AGENTS_CONFIG` and mount a config file for agent multi-tenancy.
+
 ### Development (HTTP mode + email watch)
 
 To test the email watch feature locally:

package/dist/cli.js

@@ -3970,12 +3970,29 @@ var WatcherManager = class {
   running = false;
   /** Per-account inflight guards to prevent overlapping polls. */
   inflight = /* @__PURE__ */ new Map();
+  /** Accounts with active polling timers (lowercased email). */
+  tracked = /* @__PURE__ */ new Set();
   constructor(opts) {
     this.opts = opts;
   }
   start() {
     if (this.running) return;
     this.running = true;
+    this.scanAccounts();
+    const rescanTimer = setInterval(() => {
+      if (!this.running) return;
+      this.scanAccounts();
+    }, this.opts.pollIntervalSeconds * 1e3);
+    this.timers.push(rescanTimer);
+  }
+  stop() {
+    this.running = false;
+    for (const t of this.timers) clearInterval(t);
+    this.timers = [];
+    this.tracked.clear();
+  }
+  // ── internals ──
+  scanAccounts() {
     let accounts = this.opts.store.listAccounts();
     if (this.opts.accountFilter) {
       const filter = new Set(this.opts.accountFilter.map((e) => e.toLowerCase()));
@@ -3985,13 +4002,10 @@ var WatcherManager = class {
       this.schedulePoll(account);
     }
   }
-  stop() {
-    this.running = false;
-    for (const t of this.timers) clearInterval(t);
-    this.timers = [];
-  }
-  // ── internals ──
   schedulePoll(account) {
+    const key = account.email.toLowerCase();
+    if (this.tracked.has(key)) return;
+    this.tracked.add(key);
     this.pollAccount(account).catch(() => {
     });
     const timer = setInterval(() => {
@@ -4007,21 +4021,25 @@ var WatcherManager = class {
     this.inflight.set(key, true);
     try {
       const { provider } = this.opts.registry.resolveByEmail(account.email);
-      const lastSeen = account.lastSeenAt;
+      const seenIds = new Set(account.lastSeenIds ?? []);
+      const isFirstPoll = !account.lastSeenAt && !account.lastSeenIds?.length;
       const limit = 25;
+      const MAX_PAGES = 5;
       let skip = 0;
+      let pageCount = 0;
       const newEmails = [];
-      let newestTimestamp = lastSeen ?? "";
+      let newestTimestamp = account.lastSeenAt ?? "";
       let hitBoundary = false;
-      while (true) {
+      while (pageCount < MAX_PAGES) {
         const { items, hasMore } = await provider.listEmails(account, {
           folder: "inbox",
           limit,
           skip
         });
+        pageCount++;
         for (const item of items) {
           if (!item.receivedAt) continue;
-          if (lastSeen && item.receivedAt <= lastSeen) {
+          if (seenIds.has(item.id)) {
             hitBoundary = true;
             break;
           }
@@ -4033,11 +4051,7 @@ var WatcherManager = class {
         if (hitBoundary || !hasMore) break;
         skip += limit;
       }
-      if (!lastSeen) {
-        if (newEmails.length > 0) {
-          newestTimestamp = newEmails[0].receivedAt;
-        }
-      } else if (newEmails.length > 0) {
+      if (!isFirstPoll && newEmails.length > 0) {
         this.enqueue({
           type: "new_emails",
           account: account.email,
@@ -4045,11 +4059,27 @@ var WatcherManager = class {
           timestamp: (/* @__PURE__ */ new Date()).toISOString()
         });
       }
-      if (newestTimestamp !== (lastSeen ?? "")) {
+      if (isFirstPoll && newEmails.length === 0 && !newestTimestamp) {
+        newestTimestamp = (/* @__PURE__ */ new Date()).toISOString();
+      }
+      const newIds = newEmails.map((e) => e.id);
+      const updatedLastSeenIds = [
+        ...newIds,
+        ...account.lastSeenIds ?? []
+      ].slice(0, 200);
+      try {
         await this.opts.store.upsertAccount({
           ...account,
-          lastSeenAt: newestTimestamp || void 0
+          lastSeenAt: newestTimestamp || void 0,
+          lastSeenIds: updatedLastSeenIds
         });
+      } catch (storeErr) {
+        console.error(
+          "[hypermail-mcp] failed to persist poll state for",
+          account.email,
+          ":",
+          storeErr instanceof Error ? storeErr.message : String(storeErr)
+        );
       }
     } catch (err) {
       this.enqueue({
@@ -4186,7 +4216,8 @@ async function startServer(opts) {
   const store = await AccountStore.open({ dataDir: config.dataDir });
   const registry = buildRegistry({ store, providers: config.providers });
   const tools = resolveTools(config);
-  const notificationBuffer = config.http.enabled ? [] : void 0;
+  const watchEnabled = config.http.enabled && config.watch?.enabled !== false;
+  const notificationBuffer = watchEnabled ? [] : void 0;
   let agentStoreForFactory;
   const createServer = (agentContext = null) => {
     const s = new McpServer(
@@ -4211,18 +4242,30 @@ async function startServer(opts) {
       );
     }
     const notifyTargets = /* @__PURE__ */ new Set();
-    const watcher = new WatcherManager({
-      registry,
-      store,
-      pollIntervalSeconds: config.watch?.pollIntervalSeconds ?? 60,
-      onNotification: (notification) => {
-        for (const fn of notifyTargets) {
-          fn(notification);
+    const accountFilter = agentStoreForFactory ? (() => {
+      const all = /* @__PURE__ */ new Set();
+      for (const agent of agentStoreForFactory.listAgents()) {
+        for (const email of agent.accounts) {
+          all.add(email.toLowerCase());
         }
-      },
-      buffer: notificationBuffer
-    });
-    watcher.start();
+      }
+      return all.size > 0 ? [...all] : void 0;
+    })() : void 0;
+    if (watchEnabled) {
+      const watcher = new WatcherManager({
+        registry,
+        store,
+        pollIntervalSeconds: config.watch?.pollIntervalSeconds ?? 60,
+        accountFilter,
+        onNotification: (notification) => {
+          for (const fn of notifyTargets) {
+            fn(notification);
+          }
+        },
+        buffer: notificationBuffer
+      });
+      watcher.start();
+    }
     await startHttp(
       createServer,
       config.http.host,