@adeu/mcp-server 1.18.4 → 1.19.0

package/dist/templates/email_ui.html

@@ -348,6 +348,51 @@
                 background-position: -200% 0;
             }
         }
+
+        /* --- ASYNC / PENDING STATE --- */
+        .pending-state {
+            display: flex;
+            align-items: center;
+            gap: 16px;
+            padding: 24px;
+            border: 1px solid var(--border-card);
+            border-radius: 8px;
+            background: var(--bg-card);
+        }
+
+        .pending-spinner {
+            width: 28px;
+            height: 28px;
+            flex-shrink: 0;
+            border-radius: 50%;
+            border: 3px solid var(--border-card);
+            border-top-color: var(--brand-sand);
+            animation: pending-spin 0.9s linear infinite;
+        }
+
+        @keyframes pending-spin {
+            to {
+                transform: rotate(360deg);
+            }
+        }
+
+        .pending-title {
+            font-weight: 600;
+            color: var(--text-heading);
+            margin-bottom: 4px;
+        }
+
+        .pending-message {
+            font-size: 0.9rem;
+            opacity: 0.85;
+        }
+
+        .pending-task {
+            margin-top: 6px;
+            font-size: 0.8rem;
+            opacity: 0.7;
+            font-family: ui-monospace, Consolas, monospace;
+        }
     </style>
 </head>
 
@@ -559,6 +604,45 @@
             wrapper.innerHTML = threadHtml;
             return wrapper;
         }
+        function renderPending(data) {
+            const message = data.message || 'The mailbox search is still running on the server. The assistant will check progress again shortly.';
+            const taskLine = data.task_id
+                ? `<div class="pending-task">Task: ${escapeHtml(data.task_id)}</div>`
+                : '';
+            return `
+                <div class="pending-state">
+                    <div class="pending-spinner"></div>
+                    <div class="pending-info">
+                        <div class="pending-title">Still processing&hellip;</div>
+                        <div class="pending-message">${escapeHtml(message)}</div>
+                        ${taskLine}
+                    </div>
+                </div>
+            `;
+        }
+
+        // Large mailboxes can keep the backend polling for a minute or more before
+        // the first tool result arrives. Tell the human the search is still running
+        // rather than leaving a bare skeleton with no sign of progress.
+        let slowLoadStage2 = null;
+        const slowLoadNotice = setTimeout(() => {
+            if (appDiv.querySelector('.skeleton')) {
+                const notice = document.createElement('div');
+                notice.className = 'empty-state';
+                notice.textContent = 'Still working — large mailbox searches can take a minute or two…';
+                appDiv.appendChild(notice);
+                triggerResize();
+                // If no result ever arrives (e.g. the tool call errored and the host
+                // never forwards a tool-result), stop implying progress.
+                slowLoadStage2 = setTimeout(() => {
+                    if (appDiv.querySelector('.skeleton')) {
+                        notice.textContent = 'This is taking longer than expected — the search may have failed. Check the chat for details.';
+                        triggerResize();
+                    }
+                }, 150000);
+            }
+        }, 20000);
+
         window.addEventListener("message", (event) => {
             try {
                 if (!event.data || event.data.jsonrpc !== "2.0") return;
@@ -570,13 +654,32 @@
                 }
 
                 if (msg.method === "ui/notifications/tool-result") {
+                    clearTimeout(slowLoadNotice);
+                    if (slowLoadStage2) clearTimeout(slowLoadStage2);
                     const result = msg.params;
-                    if (!result || !result.structuredContent) return;
+                    // The MCP wire format is camelCase `structuredContent`, but tolerate
+                    // snake_case in case a host forwards the Python server's payload unnormalized.
+                    const data = result ? (result.structuredContent || result.structured_content) : null;
+
+                    if (!data) {
+                        // Error results and text-only results carry no structured payload.
+                        // Never leave the skeleton spinning — surface whatever text we have.
+                        const textParts = (result && Array.isArray(result.content))
+                            ? result.content.filter(c => c && c.type === "text" && c.text).map(c => c.text)
+                            : [];
+                        const fallbackText = textParts.join("\n").trim();
+                        appDiv.innerHTML = `<div class="empty-state" style="white-space: pre-wrap;">${escapeHtml(fallbackText || 'The email tool returned no displayable data. Check the chat for details.')}</div>`;
+                        triggerResize();
+                        return;
+                    }
 
-                    const data = result.structuredContent;
                     appDiv.innerHTML = ''; // Clear skeleton
 
-                    if (data.type === "previews") {
+                    if (data.status === "pending") {
+                        appDiv.innerHTML = renderPending(data);
+                        setTimeout(triggerResize, 50);
+                    }
+                    else if (data.type === "previews") {
                         appDiv.innerHTML = renderPreviews(data.previews);
                         setTimeout(triggerResize, 50);
                     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adeu/mcp-server",
-  "version": "1.18.4",
+  "version": "1.19.0",
   "description": "",
   "mcpName": "ai.adeu/adeu",
   "main": "./dist/index.js",
@@ -31,7 +31,7 @@
   "license": "MIT",
   "type": "module",
   "dependencies": {
-    "@adeu/core": "^1.18.4",
+    "@adeu/core": "^1.19.0",
     "@modelcontextprotocol/sdk": "^1.0.0",
     "@modelcontextprotocol/ext-apps": "^1.0.0",
     "zod": "^3.23.8"

package/src/index.ts

@@ -847,11 +847,11 @@ if (!isDocxOnly) {
         "2. Fetch mode (with `email_id`): returns the full email body, thread history, and downloads attachments under `max_attachment_size_mb` to the local disk.\n\n" +
         "AUTO-ESCALATION: If a search returns exactly one preview, the backend automatically fetches the full email in the same call. Plan around the response shape — check the `type` field (`previews` vs `full_email`) before assuming.\n\n" +
         "EMAIL ID FORMATS (`email_id` parameter accepts any of):\n" +
-        "- `msg_<6 chars>` — short ID returned by previews on THIS machine. NOT portable across machines or sessions; the local cache holds the most recent 1000. If you reference one that's been evicted, the tool returns a StaleShortIdError telling you to re-search.\n" +
+        "- `msg_<6 chars>` — short ID returned by previews on THIS machine. NOT portable across machines or sessions; the local cache holds the most recent 1000. If you reference one that's been evicted, the tool returns a StaleShortIdError telling you to re-search. The mailbox used in the original search is remembered with the short ID and re-applied automatically when you fetch or reply without specifying `mailbox_address`.\n" +
         "- `adeu_<numeric>` — server-side reference for emails Adeu has previously processed. Portable across machines and sessions for the same authenticated user.\n" +
         "- Raw provider ID (Gmail/Outlook native ID) — works if you have it, but you usually won't.\n\n" +
         "FOLDER DEFAULT: omitting `folder` searches the Inbox only (matching what the user sees in their mail client). Use `folder='sent'` for sent items, `folder='all'` to include Deleted Items, Drafts, and other folders.\n\n" +
-        "ATTACHMENTS: attachments larger than `max_attachment_size_mb` (default 10) are listed in the response but NOT downloaded — raise the cap if you need them. Always set `working_directory` when calling from a project so attachments land alongside the user's other files.",
+        "ATTACHMENTS: attachments larger than `max_attachment_size_mb` (default 10) are listed in the response but NOT downloaded — raise the cap if you need them. Always set `working_directory` when calling from a project so attachments land alongside the user's other files; the directory is created automatically if it does not exist. This directory path refers to the user's native operating system, not the LLM's sandbox environment.",
       inputSchema: z.object({
         reasoning: z
           .string()
@@ -868,7 +868,15 @@ if (!isDocxOnly) {
         limit: z.coerce.number().default(10),
         offset: z.coerce.number().default(0),
         email_id: z.string().optional(),
-        working_directory: z.string().optional(),
+        working_directory: z
+          .string()
+          .optional()
+          .describe(
+            "Optional. The current working directory of the project or task. " +
+              "If provided, attachments will be saved here under an 'adeu_attachments' subfolder; " +
+              "the directory is created automatically if it does not exist. " +
+              "If omitted, attachments are saved to the system temp directory.",
+          ),
         mailbox_address: z
           .string()
           .optional()

package/src/templates/email_ui.html

@@ -348,6 +348,51 @@
                 background-position: -200% 0;
             }
         }
+
+        /* --- ASYNC / PENDING STATE --- */
+        .pending-state {
+            display: flex;
+            align-items: center;
+            gap: 16px;
+            padding: 24px;
+            border: 1px solid var(--border-card);
+            border-radius: 8px;
+            background: var(--bg-card);
+        }
+
+        .pending-spinner {
+            width: 28px;
+            height: 28px;
+            flex-shrink: 0;
+            border-radius: 50%;
+            border: 3px solid var(--border-card);
+            border-top-color: var(--brand-sand);
+            animation: pending-spin 0.9s linear infinite;
+        }
+
+        @keyframes pending-spin {
+            to {
+                transform: rotate(360deg);
+            }
+        }
+
+        .pending-title {
+            font-weight: 600;
+            color: var(--text-heading);
+            margin-bottom: 4px;
+        }
+
+        .pending-message {
+            font-size: 0.9rem;
+            opacity: 0.85;
+        }
+
+        .pending-task {
+            margin-top: 6px;
+            font-size: 0.8rem;
+            opacity: 0.7;
+            font-family: ui-monospace, Consolas, monospace;
+        }
     </style>
 </head>
 
@@ -559,6 +604,45 @@
             wrapper.innerHTML = threadHtml;
             return wrapper;
         }
+        function renderPending(data) {
+            const message = data.message || 'The mailbox search is still running on the server. The assistant will check progress again shortly.';
+            const taskLine = data.task_id
+                ? `<div class="pending-task">Task: ${escapeHtml(data.task_id)}</div>`
+                : '';
+            return `
+                <div class="pending-state">
+                    <div class="pending-spinner"></div>
+                    <div class="pending-info">
+                        <div class="pending-title">Still processing&hellip;</div>
+                        <div class="pending-message">${escapeHtml(message)}</div>
+                        ${taskLine}
+                    </div>
+                </div>
+            `;
+        }
+
+        // Large mailboxes can keep the backend polling for a minute or more before
+        // the first tool result arrives. Tell the human the search is still running
+        // rather than leaving a bare skeleton with no sign of progress.
+        let slowLoadStage2 = null;
+        const slowLoadNotice = setTimeout(() => {
+            if (appDiv.querySelector('.skeleton')) {
+                const notice = document.createElement('div');
+                notice.className = 'empty-state';
+                notice.textContent = 'Still working — large mailbox searches can take a minute or two…';
+                appDiv.appendChild(notice);
+                triggerResize();
+                // If no result ever arrives (e.g. the tool call errored and the host
+                // never forwards a tool-result), stop implying progress.
+                slowLoadStage2 = setTimeout(() => {
+                    if (appDiv.querySelector('.skeleton')) {
+                        notice.textContent = 'This is taking longer than expected — the search may have failed. Check the chat for details.';
+                        triggerResize();
+                    }
+                }, 150000);
+            }
+        }, 20000);
+
         window.addEventListener("message", (event) => {
             try {
                 if (!event.data || event.data.jsonrpc !== "2.0") return;
@@ -570,13 +654,32 @@
                 }
 
                 if (msg.method === "ui/notifications/tool-result") {
+                    clearTimeout(slowLoadNotice);
+                    if (slowLoadStage2) clearTimeout(slowLoadStage2);
                     const result = msg.params;
-                    if (!result || !result.structuredContent) return;
+                    // The MCP wire format is camelCase `structuredContent`, but tolerate
+                    // snake_case in case a host forwards the Python server's payload unnormalized.
+                    const data = result ? (result.structuredContent || result.structured_content) : null;
+
+                    if (!data) {
+                        // Error results and text-only results carry no structured payload.
+                        // Never leave the skeleton spinning — surface whatever text we have.
+                        const textParts = (result && Array.isArray(result.content))
+                            ? result.content.filter(c => c && c.type === "text" && c.text).map(c => c.text)
+                            : [];
+                        const fallbackText = textParts.join("\n").trim();
+                        appDiv.innerHTML = `<div class="empty-state" style="white-space: pre-wrap;">${escapeHtml(fallbackText || 'The email tool returned no displayable data. Check the chat for details.')}</div>`;
+                        triggerResize();
+                        return;
+                    }
 
-                    const data = result.structuredContent;
                     appDiv.innerHTML = ''; // Clear skeleton
 
-                    if (data.type === "previews") {
+                    if (data.status === "pending") {
+                        appDiv.innerHTML = renderPending(data);
+                        setTimeout(triggerResize, 50);
+                    }
+                    else if (data.type === "previews") {
                         appDiv.innerHTML = renderPreviews(data.previews);
                         setTimeout(triggerResize, 50);
                     }

package/src/tools/email.test.ts

@@ -1,5 +1,19 @@
 import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
-import { search_and_fetch_emails, list_available_mailboxes } from "./email.js";
+import {
+  mkdtempSync,
+  mkdirSync,
+  rmSync,
+  writeFileSync,
+  existsSync,
+  readFileSync,
+} from "node:fs";
+import { dirname, join } from "node:path";
+import { homedir, tmpdir } from "node:os";
+import {
+  search_and_fetch_emails,
+  list_available_mailboxes,
+  create_email_draft,
+} from "./email.js";
 
 // Mock the Auth module so tests bypass active browser logins
 vi.mock("../desktop-auth.js", () => {
@@ -415,3 +429,269 @@ describe("Node Email Tools Finding #2 and Finding #6 tests", () => {
     });
   });
 });
+
+describe("Working directory resolution (silent /tmp fallback fix)", () => {
+  const originalFetch = global.fetch;
+
+  afterEach(() => {
+    global.fetch = originalFetch;
+  });
+
+  function mockFullEmailFetch(emailId: string) {
+    global.fetch = vi.fn().mockResolvedValue({
+      ok: true,
+      status: 200,
+      json: async () => ({
+        type: "full_email",
+        full_email: {
+          id: emailId,
+          subject: "Attachment Delivery",
+          sender_name: "Legal",
+          sender_email: "legal@adeu.ai",
+          received_datetime: "2026-01-01T12:00:00Z",
+          body_html: "<p>See attachment.</p>",
+          is_thread: false,
+          attachments: [
+            {
+              filename: "questionnaire.docx",
+              size_bytes: 128,
+              base64_data: Buffer.from("questionnaire contents").toString(
+                "base64",
+              ),
+            },
+          ],
+        },
+      }),
+    } as Response);
+  }
+
+  it("creates a missing working_directory recursively and saves attachments inside it", async () => {
+    const root = mkdtempSync(join(tmpdir(), "adeu-wd-test-"));
+    const requestedDir = join(root, "questionnaires", "nested");
+    try {
+      mockFullEmailFetch("adeu_777");
+
+      const result = await search_and_fetch_emails({
+        email_id: "adeu_777",
+        working_directory: requestedDir,
+      });
+
+      const text = result.content[0].text;
+      expect(existsSync(requestedDir)).toBe(true);
+      expect(text).not.toContain("Attachment location notice");
+
+      const savedPathMatch = text.match(/📎 `([^`]+)`/);
+      expect(savedPathMatch).not.toBeNull();
+      const savedPath = savedPathMatch![1];
+      expect(savedPath.startsWith(join(requestedDir, "adeu_attachments"))).toBe(
+        true,
+      );
+      expect(readFileSync(savedPath, "utf-8")).toBe("questionnaire contents");
+    } finally {
+      rmSync(root, { recursive: true, force: true });
+    }
+  });
+
+  it("falls back to the temp dir WITH an explicit notice when working_directory cannot be created", async () => {
+    const root = mkdtempSync(join(tmpdir(), "adeu-wd-test-"));
+    const blockerFile = join(root, "blocker.txt");
+    writeFileSync(blockerFile, "not a directory");
+    try {
+      mockFullEmailFetch("adeu_778");
+
+      const result = await search_and_fetch_emails({
+        email_id: "adeu_778",
+        working_directory: join(blockerFile, "sub"),
+      });
+
+      const text = result.content[0].text;
+      expect(text).toContain("Attachment location notice");
+      expect(text).toContain("do NOT re-run the search");
+
+      const savedPathMatch = text.match(/📎 `([^`]+)`/);
+      expect(savedPathMatch).not.toBeNull();
+      const savedPath = savedPathMatch![1];
+      expect(savedPath.startsWith(join(tmpdir(), "adeu_downloads"))).toBe(true);
+      expect(existsSync(savedPath)).toBe(true);
+
+      rmSync(dirname(savedPath), { recursive: true, force: true });
+    } finally {
+      rmSync(root, { recursive: true, force: true });
+    }
+  });
+
+  it("returns structuredContent for empty preview results so the UI can dismiss its skeleton", async () => {
+    global.fetch = vi.fn().mockResolvedValue({
+      ok: true,
+      status: 200,
+      json: async () => ({ type: "previews", previews: [] }),
+    } as Response);
+
+    const result = await search_and_fetch_emails({ subject: "nothing matches" });
+    expect(result.content[0].text).toContain(
+      "No emails found matching your search criteria.",
+    );
+    expect(result.structuredContent).toEqual({ type: "previews", previews: [] });
+  });
+});
+
+describe("Mailbox-aware short ID cache", () => {
+  const originalFetch = global.fetch;
+
+  afterEach(() => {
+    global.fetch = originalFetch;
+  });
+
+  it("re-applies the search's mailbox_address when fetching by short ID without one", async () => {
+    const fetchMock = vi.fn();
+    global.fetch = fetchMock;
+
+    fetchMock.mockResolvedValueOnce({
+      ok: true,
+      status: 200,
+      json: async () => ({
+        type: "previews",
+        previews: [
+          {
+            id: "AAMkAD_shared_item_1",
+            subject: "Questionnaire",
+            sender_name: "Abo Shoten",
+            sender_email: "ops@aboshoten.example",
+            received_datetime: "2026-07-07T09:00:00Z",
+            preview_text: "Please fill in",
+            has_attachments: true,
+            is_read: false,
+          },
+          {
+            id: "AAMkAD_shared_item_2",
+            subject: "Other mail",
+            sender_name: "Abo Shoten",
+            sender_email: "ops@aboshoten.example",
+            received_datetime: "2026-07-07T09:01:00Z",
+            preview_text: "Something else",
+            has_attachments: false,
+            is_read: true,
+          },
+        ],
+      }),
+    } as Response);
+
+    const searchRes = await search_and_fetch_emails({
+      subject: "Questionnaire",
+      mailbox_address: "risto.kariranta@ahti.io",
+    });
+    const shortIdMatch = searchRes.content[0].text.match(/msg_[0-9a-f]{6}/);
+    expect(shortIdMatch).not.toBeNull();
+    const shortId = shortIdMatch![0];
+
+    fetchMock.mockResolvedValueOnce({
+      ok: true,
+      status: 200,
+      json: async () => ({
+        type: "full_email",
+        full_email: {
+          id: "AAMkAD_shared_item_1",
+          subject: "Questionnaire",
+          sender_name: "Abo Shoten",
+          sender_email: "ops@aboshoten.example",
+          received_datetime: "2026-07-07T09:00:00Z",
+          body_html: "<p>Please fill in.</p>",
+          is_thread: false,
+          attachments: [],
+        },
+      }),
+    } as Response);
+
+    await search_and_fetch_emails({ email_id: shortId });
+
+    const fetchBody = JSON.parse(fetchMock.mock.calls[1][1].body);
+    expect(fetchBody.email_id).toBe("AAMkAD_shared_item_1");
+    expect(fetchBody.mailbox_address).toBe("risto.kariranta@ahti.io");
+  });
+
+  it("still resolves legacy plain-string cache entries without injecting a mailbox", async () => {
+    // Seed a legacy-format entry directly into the cache file (merge, don't clobber).
+    const cachePath = join(homedir(), ".adeu", "mcp_id_cache.json");
+    mkdirSync(join(homedir(), ".adeu"), { recursive: true });
+    let existing: Record<string, unknown> = {};
+    try {
+      existing = JSON.parse(readFileSync(cachePath, "utf-8"));
+    } catch {
+      /* no cache yet */
+    }
+    existing["msg_leg01"] = "raw_provider_id_123";
+    writeFileSync(cachePath, JSON.stringify(existing));
+
+    const fetchMock = vi.fn().mockResolvedValue({
+      ok: true,
+      status: 200,
+      json: async () => ({
+        type: "full_email",
+        full_email: {
+          id: "raw_provider_id_123",
+          subject: "Legacy",
+          sender_name: "Old Cache",
+          sender_email: "old@cache.example",
+          received_datetime: "2026-07-07T09:00:00Z",
+          body_html: "<p>hi</p>",
+          is_thread: false,
+          attachments: [],
+        },
+      }),
+    } as Response);
+    global.fetch = fetchMock;
+
+    await search_and_fetch_emails({ email_id: "msg_leg01" });
+
+    const fetchBody = JSON.parse(fetchMock.mock.calls[0][1].body);
+    expect(fetchBody.email_id).toBe("raw_provider_id_123");
+    expect(fetchBody.mailbox_address).toBeUndefined();
+  });
+
+  it("maps known errors to recovery hints when an async task fails", async () => {
+    global.fetch = vi.fn().mockResolvedValue({
+      ok: true,
+      status: 200,
+      json: async () => ({ status: "FAILED", error: "Email not found." }),
+    } as Response);
+
+    await expect(
+      search_and_fetch_emails({ task_id: "email_task_777" }),
+    ).rejects.toThrowError(
+      /re-run search_and_fetch_emails with filters[\s\S]*mailbox_address/,
+    );
+  });
+
+  it("re-applies the cached mailbox when replying to a short ID without one", async () => {
+    // Seed a new-format entry directly into the cache file (merge, don't clobber).
+    const cachePath = join(homedir(), ".adeu", "mcp_id_cache.json");
+    mkdirSync(join(homedir(), ".adeu"), { recursive: true });
+    let existing: Record<string, unknown> = {};
+    try {
+      existing = JSON.parse(readFileSync(cachePath, "utf-8"));
+    } catch {
+      /* no cache yet */
+    }
+    existing["msg_rep01"] = {
+      id: "AAMkAD_reply_item_1",
+      mailbox: "sales@ahti.io",
+    };
+    writeFileSync(cachePath, JSON.stringify(existing));
+
+    const fetchMock = vi.fn().mockResolvedValue({
+      ok: true,
+      status: 200,
+      json: async () => ({ id: "draft_1" }),
+    } as Response);
+    global.fetch = fetchMock;
+
+    await create_email_draft({
+      reply_to_email_id: "msg_rep01",
+      body_markdown: "Thanks, will do!",
+    });
+
+    const formData = fetchMock.mock.calls[0][1].body as FormData;
+    expect(formData.get("reply_to_email_id")).toBe("AAMkAD_reply_item_1");
+    expect(formData.get("mailbox_address")).toBe("sales@ahti.io");
+  });
+});