ofw-mcp 2.0.19 → 2.2.0

package/dist/client.js

@@ -40,6 +40,19 @@ function redactHeaders(h) {
         out.Authorization = `Bearer ${out.Authorization.slice(7, 17)}…`;
     return out;
 }
+// Per-request timeout. Overridable via OFW_REQUEST_TIMEOUT_MS. The default
+// (30s) is comfortably above OFW's typical p99 but low enough that a stuck
+// upstream fails fast instead of burning the MCP client-side budget — which
+// is what produced the multi-minute hangs we've seen on ofw_list_messages
+// and ofw_save_draft. Each retry (401/429 replay) gets its own fresh window.
+const DEFAULT_REQUEST_TIMEOUT_MS = 30_000;
+function getRequestTimeoutMs() {
+    const raw = process.env.OFW_REQUEST_TIMEOUT_MS;
+    if (typeof raw !== 'string' || raw.trim().length === 0)
+        return DEFAULT_REQUEST_TIMEOUT_MS;
+    const n = Number(raw.trim());
+    return Number.isFinite(n) && n > 0 ? n : DEFAULT_REQUEST_TIMEOUT_MS;
+}
 export class OFWClient {
     token = null;
     tokenExpiry = null;
@@ -85,13 +98,40 @@ export class OFWClient {
             console.error(`[ofw-debug]   headers: ${JSON.stringify(redactHeaders(headers))}`);
             console.error(`[ofw-debug]   body: ${bodyPreview}`);
         }
-        const response = await fetch(url, {
-            method,
-            headers,
-            ...(body !== undefined ? { body: isFormData ? body : JSON.stringify(body) } : {}),
-        });
+        // AbortController + setTimeout (not AbortSignal.timeout) so vitest fake
+        // timers can drive the timeout in tests, and so we can attach a clear
+        // error message instead of a bare DOMException on the abort path.
+        const timeoutMs = getRequestTimeoutMs();
+        const ac = new AbortController();
+        const timer = setTimeout(() => ac.abort(), timeoutMs);
+        const startedAt = Date.now();
+        let response;
+        try {
+            response = await fetch(url, {
+                method,
+                headers,
+                signal: ac.signal,
+                ...(body !== undefined ? { body: isFormData ? body : JSON.stringify(body) } : {}),
+            });
+        }
+        catch (err) {
+            const elapsed = Date.now() - startedAt;
+            if (ac.signal.aborted) {
+                if (debugLogEnabled()) {
+                    console.error(`[ofw-debug] ⏱ TIMEOUT after ${elapsed}ms: ${method} ${url}`);
+                }
+                throw new Error(`OFW API request timed out after ${timeoutMs}ms: ${method} ${path}`);
+            }
+            if (debugLogEnabled()) {
+                console.error(`[ofw-debug] ✗ ${err.message} after ${elapsed}ms: ${method} ${url}`);
+            }
+            throw err;
+        }
+        finally {
+            clearTimeout(timer);
+        }
         if (debugLogEnabled()) {
-            console.error(`[ofw-debug] ← ${response.status} ${response.statusText}`);
+            console.error(`[ofw-debug] ← ${response.status} ${response.statusText} (${Date.now() - startedAt}ms)`);
         }
         if (response.status === 401 && !isRetry) {
             this.token = null;

package/dist/index.js

@@ -17,7 +17,7 @@ import { registerMessageTools } from './tools/messages.js';
 import { registerCalendarTools } from './tools/calendar.js';
 import { registerExpenseTools } from './tools/expenses.js';
 import { registerJournalTools } from './tools/journal.js';
-const server = new McpServer({ name: 'ofw', version: '2.0.19' }); // x-release-please-version
+const server = new McpServer({ name: 'ofw', version: '2.2.0' }); // x-release-please-version
 registerUserTools(server, client);
 registerMessageTools(server, client);
 registerCalendarTools(server, client);

package/dist/tools/messages.js

@@ -175,18 +175,59 @@ export function registerMessageTools(server, client) {
         return jsonResponse({ ...row, attachments });
     });
     server.registerTool('ofw_send_message', {
-        description: 'Send a message via OurFamilyWizard. If sending from a draft, pass draftId to delete the draft after sending. If replyToId is provided, the cache may rewrite it to the latest reply in the same thread (a note is included in the response when this happens). Attach files by passing their fileIds (from ofw_upload_attachment) in myFileIDs. After sending, the tool re-fetches the message from OFW to populate the local cache and link attachments to the new message id.',
+        description: 'Send a message via OurFamilyWizard. To send an existing draft, pass messageId — subject/body/recipientIds become optional overrides (missing fields default to the draft\'s cached values) and the draft is deleted after sending. To send a fresh message, supply subject/body/recipientIds directly. draftId is the legacy spelling of messageId and works the same way. If replyToId is provided, the cache may rewrite it to the latest reply in the same thread (a note is included in the response when this happens). Attach files by passing their fileIds (from ofw_upload_attachment) in myFileIDs. After sending, the tool re-fetches the message from OFW to populate the local cache and link attachments to the new message id.',
         annotations: { destructiveHint: true },
         inputSchema: {
-            subject: z.string().describe('Message subject'),
-            body: z.string().describe('Message body text'),
-            recipientIds: z.array(z.number()).describe('Array of recipient user IDs (get from ofw_get_profile)'),
+            subject: z.string().describe('Message subject. Required unless messageId/draftId references a cached draft.').optional(),
+            body: z.string().describe('Message body text. Required unless messageId/draftId references a cached draft.').optional(),
+            recipientIds: z.array(z.number()).describe('Array of recipient user IDs (get from ofw_get_profile). Required unless messageId/draftId references a cached draft.').optional(),
             replyToId: z.number().describe('ID of the message being replied to').optional(),
-            draftId: z.number().describe('ID of the draft to delete after sending (omit if not sending from a draft)').optional(),
+            messageId: z.number().describe('ID of an existing draft to send. When set, missing subject/body/recipientIds default to the draft\'s cached values, and the draft is deleted after sending.').optional(),
+            draftId: z.number().describe('Legacy synonym for messageId. If both are passed they must be equal.').optional(),
             myFileIDs: z.array(z.number()).describe('Attachment file ids (from ofw_upload_attachment) to attach to the message').optional(),
         },
     }, async (args) => {
-        const requestedReplyTo = args.replyToId ?? null;
+        if (args.messageId !== undefined && args.draftId !== undefined && args.messageId !== args.draftId) {
+            throw new Error(`messageId (${args.messageId}) and draftId (${args.draftId}) refer to different drafts; pass only one.`);
+        }
+        const draftRef = args.messageId ?? args.draftId;
+        // Best-effort draft lookup: when draftRef points at a cached draft, use
+        // its stored fields (including replyToId) as defaults for anything the
+        // caller didn't supply. The "missing draft" case only matters when we
+        // actually NEED the defaults — a caller passing all fields explicitly
+        // can use draftId as a pure delete-target even on an empty cache.
+        let subject = args.subject;
+        let body = args.body;
+        let recipientIds = args.recipientIds;
+        let draftReplyToId = null;
+        let draftLookupAttempted = false;
+        let draftFound = false;
+        if (draftRef !== undefined) {
+            draftLookupAttempted = true;
+            const draft = getDraft(draftRef);
+            if (draft !== null) {
+                draftFound = true;
+                subject = subject ?? draft.subject;
+                body = body ?? draft.body;
+                recipientIds = recipientIds ?? draft.recipients.map((r) => r.userId);
+                draftReplyToId = draft.replyToId;
+            }
+        }
+        if (subject === undefined || body === undefined || recipientIds === undefined) {
+            if (draftLookupAttempted && !draftFound) {
+                throw new Error(`draft ${draftRef} not found in local cache. Call ofw_sync_messages first, or supply subject/body/recipientIds explicitly.`);
+            }
+            const missing = [
+                subject === undefined ? 'subject' : null,
+                body === undefined ? 'body' : null,
+                recipientIds === undefined ? 'recipientIds' : null,
+            ].filter((n) => n !== null).join(', ');
+            throw new Error(`ofw_send_message requires ${missing}. Pass it directly, or pass messageId to default missing fields from a cached draft.`);
+        }
+        // Inherit the draft's replyToId when the caller didn't supply one. A
+        // reply-draft saved with replyToId would otherwise be sent as a
+        // top-level message — silently losing the thread.
+        const requestedReplyTo = args.replyToId ?? draftReplyToId ?? null;
         let resolvedReplyTo = requestedReplyTo;
         let chainRootId = null;
         let rewriteNote = null;
@@ -200,9 +241,9 @@ export function registerMessageTools(server, client) {
         }
         const myFileIDs = args.myFileIDs ?? [];
         const { id: newId, detail, raw } = await postMessageAndRefetch(client, {
-            subject: args.subject,
-            body: args.body,
-            recipientIds: args.recipientIds,
+            subject,
+            body,
+            recipientIds,
             attachments: { myFileIDs },
             draft: false,
             includeOriginal: resolvedReplyTo !== null,
@@ -213,11 +254,11 @@ export function registerMessageTools(server, client) {
             persisted = {
                 id: newId,
                 folder: 'sent',
-                subject: detail.subject ?? args.subject,
+                subject: detail.subject ?? subject,
                 fromUser: detail.from?.name ?? '',
                 sentAt: detail.date?.dateTime ?? new Date().toISOString(),
                 recipients: mapRecipients(detail.recipients),
-                body: detail.body ?? args.body,
+                body: detail.body ?? body,
                 fetchedBodyAt: new Date().toISOString(),
                 replyToId: resolvedReplyTo,
                 chainRootId,
@@ -240,9 +281,9 @@ export function registerMessageTools(server, client) {
                 });
             }
         }
-        if (args.draftId !== undefined) {
-            await deleteOFWMessages(client, [args.draftId]);
-            deleteDraft(args.draftId);
+        if (draftRef !== undefined) {
+            await deleteOFWMessages(client, [draftRef]);
+            deleteDraft(draftRef);
         }
         const responseObj = persisted ?? raw;
         const text = responseObj ? JSON.stringify(responseObj, null, 2) : 'Message sent successfully.';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ofw-mcp",
-  "version": "2.0.19",
+  "version": "2.2.0",
   "mcpName": "io.github.chrischall/ofw-mcp",
   "description": "OurFamilyWizard MCP server for Claude — developed and maintained by AI (Claude Code)",
   "author": "Claude Code (AI) <https://www.anthropic.com/claude>",
@@ -27,16 +27,17 @@
     "test:watch": "vitest"
   },
   "dependencies": {
-    "@fetchproxy/bootstrap": "^0.5.1",
+    "@fetchproxy/bootstrap": "^0.8.0",
+    "@fetchproxy/server": "^0.8.0",
     "@modelcontextprotocol/sdk": "^1.29.0",
-    "dotenv": "^17.4.0",
-    "zod": "^4.4.2"
+    "dotenv": "^17.4.2",
+    "zod": "^4.4.3"
   },
   "devDependencies": {
-    "@types/node": "^25.8.0",
-    "@vitest/coverage-v8": "^4.1.6",
+    "@types/node": "^25.9.1",
+    "@vitest/coverage-v8": "^4.1.7",
     "esbuild": "^0.28.0",
-    "typescript": "^6.0.2",
-    "vitest": "^4.1.6"
+    "typescript": "^6.0.3",
+    "vitest": "^4.1.7"
   }
 }

package/server.json

@@ -6,12 +6,12 @@
     "url": "https://github.com/chrischall/ofw-mcp",
     "source": "github"
   },
-  "version": "2.0.19",
+  "version": "2.2.0",
   "packages": [
     {
       "registryType": "npm",
       "identifier": "ofw-mcp",
-      "version": "2.0.19",
+      "version": "2.2.0",
       "transport": {
         "type": "stdio"
       },