ofw-mcp 2.0.14 → 2.0.16

package/.claude-plugin/marketplace.json

@@ -6,7 +6,7 @@
   },
   "metadata": {
     "description": "OurFamilyWizard tools for Claude Code",
-    "version": "2.0.14"
+    "version": "2.0.16"
   },
   "plugins": [
     {
@@ -14,7 +14,7 @@
       "displayName": "OurFamilyWizard",
       "source": "./",
       "description": "OurFamilyWizard co-parenting tools for Claude — messages, calendar, expenses, and journal via MCP",
-      "version": "2.0.14",
+      "version": "2.0.16",
       "author": {
         "name": "Chris Chall"
       },

package/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "ofw",
   "displayName": "OurFamilyWizard",
-  "version": "2.0.14",
+  "version": "2.0.16",
   "description": "OurFamilyWizard co-parenting tools for Claude — messages, calendar, expenses, and journal via MCP",
   "author": {
     "name": "Chris Chall"

package/dist/bundle.js

@@ -36616,7 +36616,7 @@ async function loginWithPassword(username, password) {
 // package.json
 var package_default = {
   name: "ofw-mcp",
-  version: "2.0.14",
+  version: "2.0.16",
   mcpName: "io.github.chrischall/ofw-mcp",
   description: "OurFamilyWizard MCP server for Claude \u2014 developed and maintained by AI (Claude Code)",
   author: "Claude Code (AI) <https://www.anthropic.com/claude>",
@@ -36748,6 +36748,15 @@ function parseContentDispositionFilename(cd) {
   const m = /filename="?([^";]+)"?/i.exec(cd);
   return m ? m[1] : null;
 }
+function debugLogEnabled() {
+  const v = process.env.OFW_DEBUG_LOG;
+  return v === "1" || v === "true" || v === "yes" || v === "on";
+}
+function redactHeaders(h) {
+  const out = { ...h };
+  if (out.Authorization) out.Authorization = `Bearer ${out.Authorization.slice(7, 17)}\u2026`;
+  return out;
+}
 var OFWClient = class {
   token = null;
   tokenExpiry = null;
@@ -36755,6 +36764,9 @@ var OFWClient = class {
     await this.ensureAuthenticated();
     const response = await this.fetchWithRetry(method, path, body, "application/json", false);
     const text = await response.text();
+    if (debugLogEnabled()) {
+      console.error(`[ofw-debug] response body: ${text || "<empty>"}`);
+    }
     return text ? JSON.parse(text) : null;
   }
   /** Like `request`, but returns the raw bytes plus Content-Type/-Disposition metadata. */
@@ -36778,11 +36790,21 @@ var OFWClient = class {
       Authorization: `Bearer ${this.token}`
     };
     if (body !== void 0 && !isFormData) headers["Content-Type"] = "application/json";
-    const response = await fetch(`${BASE_URL2}${path}`, {
+    const url2 = `${BASE_URL2}${path}`;
+    if (debugLogEnabled()) {
+      const bodyPreview = body === void 0 ? "<none>" : isFormData ? `<FormData entries=${Array.from(body.keys()).join(",")}>` : JSON.stringify(body);
+      console.error(`[ofw-debug] \u2192 ${method} ${url2}${isRetry ? " (retry)" : ""}`);
+      console.error(`[ofw-debug]   headers: ${JSON.stringify(redactHeaders(headers))}`);
+      console.error(`[ofw-debug]   body: ${bodyPreview}`);
+    }
+    const response = await fetch(url2, {
       method,
       headers,
       ...body !== void 0 ? { body: isFormData ? body : JSON.stringify(body) } : {}
     });
+    if (debugLogEnabled()) {
+      console.error(`[ofw-debug] \u2190 ${response.status} ${response.statusText}`);
+    }
     if (response.status === 401 && !isRetry) {
       this.token = null;
       this.tokenExpiry = null;
@@ -37344,9 +37366,6 @@ async function syncDrafts(client2, draftsFolderId) {
     seenIds.add(item.id);
     const modifiedAt = item.date?.dateTime ?? (/* @__PURE__ */ new Date()).toISOString();
     const existing = getDraft(item.id);
-    if (existing && existing.modifiedAt === modifiedAt) {
-      continue;
-    }
     const detail = await client2.request("GET", `/pub/v3/messages/${item.id}`);
     const row = {
       id: item.id,
@@ -37358,7 +37377,9 @@ async function syncDrafts(client2, draftsFolderId) {
       listData: item
     };
     upsertDraft(row);
-    synced++;
+    if (!existing || existing.body !== row.body || existing.subject !== row.subject || existing.replyToId !== row.replyToId) {
+      synced++;
+    }
   }
   for (const id of listDraftIds()) {
     if (!seenIds.has(id)) deleteDraft(id);
@@ -37520,7 +37541,7 @@ function registerMessageTools(server2, client2) {
     return jsonResponse({ ...row, attachments });
   });
   server2.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.",
+    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.",
     annotations: { destructiveHint: true },
     inputSchema: {
       subject: external_exports.string().describe("Message subject"),
@@ -37553,21 +37574,24 @@ function registerMessageTools(server2, client2) {
       includeOriginal: resolvedReplyTo !== null,
       replyToId: resolvedReplyTo
     });
-    if (data && typeof data.id === "number") {
-      const row = {
-        id: data.id,
+    const newId = typeof data?.id === "number" ? data.id : typeof data?.entityId === "number" ? data.entityId : null;
+    let persisted = null;
+    if (newId !== null) {
+      const detail = await client2.request("GET", `/pub/v3/messages/${newId}`);
+      persisted = {
+        id: newId,
         folder: "sent",
-        subject: data.subject ?? args.subject,
-        fromUser: data.from?.name ?? "",
-        sentAt: data.date?.dateTime ?? (/* @__PURE__ */ new Date()).toISOString(),
-        recipients: mapRecipients(data.recipients),
-        body: data.body ?? args.body,
+        subject: detail.subject ?? args.subject,
+        fromUser: detail.from?.name ?? "",
+        sentAt: detail.date?.dateTime ?? (/* @__PURE__ */ new Date()).toISOString(),
+        recipients: mapRecipients(detail.recipients),
+        body: detail.body ?? args.body,
         fetchedBodyAt: (/* @__PURE__ */ new Date()).toISOString(),
         replyToId: resolvedReplyTo,
         chainRootId,
-        listData: data
+        listData: detail
       };
-      upsertMessage(row);
+      upsertMessage(persisted);
       for (const fileId of myFileIDs) {
         const existing = getAttachment(fileId);
         upsertAttachmentForMessage({
@@ -37577,7 +37601,7 @@ function registerMessageTools(server2, client2) {
           mimeType: existing?.mimeType ?? "application/octet-stream",
           sizeBytes: existing?.sizeBytes ?? null,
           metadata: existing?.metadata ?? {},
-          messageId: data.id
+          messageId: newId
         });
       }
     }
@@ -37585,7 +37609,8 @@ function registerMessageTools(server2, client2) {
       await deleteOFWMessages(client2, [args.draftId]);
       deleteDraft(args.draftId);
     }
-    const text = data ? JSON.stringify(data, null, 2) : "Message sent successfully.";
+    const responseObj = persisted ?? data;
+    const text = responseObj ? JSON.stringify(responseObj, null, 2) : "Message sent successfully.";
     return textResponse(rewriteNote ? `${rewriteNote}
 
 ${text}` : text);
@@ -37605,7 +37630,7 @@ ${text}` : text);
     return jsonResponse(payload);
   });
   server2.registerTool("ofw_save_draft", {
-    description: "Save a message as a draft in OurFamilyWizard. Recipients are optional. To update an existing draft, provide its messageId. If replyToId is provided, the cache may rewrite it to the latest reply in the thread (note included in response). Attach files by passing their fileIds (from ofw_upload_attachment) in myFileIDs.",
+    description: "Save a message as a draft in OurFamilyWizard. Recipients are optional. To update an existing draft, provide its messageId. If replyToId is provided, the cache may rewrite it to the latest reply in the thread (note included in response). Attach files by passing their fileIds (from ofw_upload_attachment) in myFileIDs. After saving, the tool re-fetches the draft from OFW to populate the local cache and verify what was actually persisted; if OFW silently no-ops an update (a known issue with repeated updates to the same draft), the response includes a WARNING note with a workaround.",
     annotations: { readOnlyHint: false },
     inputSchema: {
       subject: external_exports.string().describe("Message subject"),
@@ -37637,20 +37662,29 @@ ${text}` : text);
     };
     if (args.messageId !== void 0) payload.messageId = args.messageId;
     const data = await client2.request("POST", "/pub/v3/messages", payload);
-    if (data && typeof data.id === "number") {
-      const draft = {
-        id: data.id,
-        subject: data.subject ?? args.subject,
-        body: data.body ?? args.body,
-        recipients: mapRecipients(data.recipients),
-        replyToId: data.replyToId ?? resolvedReplyTo,
-        modifiedAt: data.date?.dateTime ?? (/* @__PURE__ */ new Date()).toISOString(),
-        listData: data
+    const newId = typeof data?.id === "number" ? data.id : typeof data?.entityId === "number" ? data.entityId : null;
+    let persisted = null;
+    let noOpWarning = null;
+    if (newId !== null) {
+      const detail = await client2.request("GET", `/pub/v3/messages/${newId}`);
+      persisted = {
+        id: newId,
+        subject: detail.subject ?? args.subject,
+        body: detail.body ?? "",
+        recipients: mapRecipients(detail.recipients),
+        replyToId: detail.replyToId ?? resolvedReplyTo,
+        modifiedAt: detail.date?.dateTime ?? (/* @__PURE__ */ new Date()).toISOString(),
+        listData: detail
       };
-      upsertDraft(draft);
+      upsertDraft(persisted);
+      if (args.messageId !== void 0 && persisted.body !== args.body) {
+        noOpWarning = "WARNING: OFW reported success but the draft body it returned does not match the requested update. The OFW POST /pub/v3/messages endpoint can silently no-op on subsequent updates to the same draft. Workaround: delete this draft (ofw_delete_draft) and create a new one (ofw_save_draft without messageId).";
+      }
     }
-    const text = data ? JSON.stringify(data, null, 2) : "Draft saved.";
-    return textResponse(rewriteNote ? `${rewriteNote}
+    const responseObj = persisted ?? data;
+    const text = responseObj ? JSON.stringify(responseObj, null, 2) : "Draft saved.";
+    const notes = [rewriteNote, noOpWarning].filter((n) => n !== null).join("\n\n");
+    return textResponse(notes ? `${notes}
 
 ${text}` : text);
   });
@@ -37979,7 +38013,7 @@ process.emit = function(event, ...args) {
   }
   return originalEmit(event, ...args);
 };
-var server = new McpServer({ name: "ofw", version: "2.0.14" });
+var server = new McpServer({ name: "ofw", version: "2.0.16" });
 registerUserTools(server, client);
 registerMessageTools(server, client);
 registerCalendarTools(server, client);

package/dist/client.js

@@ -31,6 +31,19 @@ function parseContentDispositionFilename(cd) {
     const m = /filename="?([^";]+)"?/i.exec(cd);
     return m ? m[1] : null;
 }
+// Set OFW_DEBUG_LOG=1 (or true/yes/on) to log every OFW request/response to
+// stderr. Authorization is redacted. Bodies are logged in full — set this
+// only when debugging, never in normal use.
+function debugLogEnabled() {
+    const v = process.env.OFW_DEBUG_LOG;
+    return v === '1' || v === 'true' || v === 'yes' || v === 'on';
+}
+function redactHeaders(h) {
+    const out = { ...h };
+    if (out.Authorization)
+        out.Authorization = `Bearer ${out.Authorization.slice(7, 17)}…`;
+    return out;
+}
 export class OFWClient {
     token = null;
     tokenExpiry = null;
@@ -38,6 +51,9 @@ export class OFWClient {
         await this.ensureAuthenticated();
         const response = await this.fetchWithRetry(method, path, body, 'application/json', false);
         const text = await response.text();
+        if (debugLogEnabled()) {
+            console.error(`[ofw-debug] response body: ${text || '<empty>'}`);
+        }
         return (text ? JSON.parse(text) : null);
     }
     /** Like `request`, but returns the raw bytes plus Content-Type/-Disposition metadata. */
@@ -62,11 +78,25 @@ export class OFWClient {
         };
         if (body !== undefined && !isFormData)
             headers['Content-Type'] = 'application/json';
-        const response = await fetch(`${BASE_URL}${path}`, {
+        const url = `${BASE_URL}${path}`;
+        if (debugLogEnabled()) {
+            const bodyPreview = body === undefined
+                ? '<none>'
+                : isFormData
+                    ? `<FormData entries=${Array.from(body.keys()).join(',')}>`
+                    : JSON.stringify(body);
+            console.error(`[ofw-debug] → ${method} ${url}${isRetry ? ' (retry)' : ''}`);
+            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) } : {}),
         });
+        if (debugLogEnabled()) {
+            console.error(`[ofw-debug] ← ${response.status} ${response.statusText}`);
+        }
         if (response.status === 401 && !isRetry) {
             this.token = null;
             this.tokenExpiry = 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.14' });
+const server = new McpServer({ name: 'ofw', version: '2.0.16' });
 registerUserTools(server, client);
 registerMessageTools(server, client);
 registerCalendarTools(server, client);

package/dist/sync.js

@@ -127,10 +127,10 @@ export async function syncDrafts(client, draftsFolderId) {
     for (const item of items) {
         seenIds.add(item.id);
         const modifiedAt = item.date?.dateTime ?? new Date().toISOString();
+        // OFW's list endpoint's `date.dateTime` is NOT a reliable modification
+        // timestamp for drafts — direct UI edits don't bump it — so we can't
+        // use it to skip the detail fetch. Always re-fetch; drafts are few.
         const existing = getDraft(item.id);
-        if (existing && existing.modifiedAt === modifiedAt) {
-            continue;
-        }
         const detail = await client.request('GET', `/pub/v3/messages/${item.id}`);
         const row = {
             id: item.id,
@@ -142,7 +142,12 @@ export async function syncDrafts(client, draftsFolderId) {
             listData: item,
         };
         upsertDraft(row);
-        synced++;
+        if (!existing
+            || existing.body !== row.body
+            || existing.subject !== row.subject
+            || existing.replyToId !== row.replyToId) {
+            synced++;
+        }
     }
     for (const id of listDraftIds()) {
         if (!seenIds.has(id))

package/dist/tools/messages.js

@@ -149,7 +149,7 @@ 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.',
+        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.',
         annotations: { destructiveHint: true },
         inputSchema: {
             subject: z.string().describe('Message subject'),
@@ -173,6 +173,9 @@ export function registerMessageTools(server, client) {
             chainRootId = parent?.chainRootId ?? parent?.id ?? requestedReplyTo;
         }
         const myFileIDs = args.myFileIDs ?? [];
+        // OFW's POST /pub/v3/messages response is minimal — typically just
+        // `{entityId: <id>}` — so the cache write needs to fetch detail
+        // afterwards (same shape as ofw_save_draft).
         const data = await client.request('POST', '/pub/v3/messages', {
             subject: args.subject,
             body: args.body,
@@ -182,21 +185,26 @@ export function registerMessageTools(server, client) {
             includeOriginal: resolvedReplyTo !== null,
             replyToId: resolvedReplyTo,
         });
-        if (data && typeof data.id === 'number') {
-            const row = {
-                id: data.id,
+        const newId = typeof data?.id === 'number' ? data.id
+            : typeof data?.entityId === 'number' ? data.entityId
+                : null;
+        let persisted = null;
+        if (newId !== null) {
+            const detail = await client.request('GET', `/pub/v3/messages/${newId}`);
+            persisted = {
+                id: newId,
                 folder: 'sent',
-                subject: data.subject ?? args.subject,
-                fromUser: data.from?.name ?? '',
-                sentAt: data.date?.dateTime ?? new Date().toISOString(),
-                recipients: mapRecipients(data.recipients),
-                body: data.body ?? args.body,
+                subject: detail.subject ?? args.subject,
+                fromUser: detail.from?.name ?? '',
+                sentAt: detail.date?.dateTime ?? new Date().toISOString(),
+                recipients: mapRecipients(detail.recipients),
+                body: detail.body ?? args.body,
                 fetchedBodyAt: new Date().toISOString(),
                 replyToId: resolvedReplyTo,
                 chainRootId,
-                listData: data,
+                listData: detail,
             };
-            upsertMessage(row);
+            upsertMessage(persisted);
             // Link attached files to the new message in the attachments cache.
             // We may not have full metadata if the upload happened in a prior
             // session — fall back to what we know.
@@ -209,7 +217,7 @@ export function registerMessageTools(server, client) {
                     mimeType: existing?.mimeType ?? 'application/octet-stream',
                     sizeBytes: existing?.sizeBytes ?? null,
                     metadata: existing?.metadata ?? {},
-                    messageId: data.id,
+                    messageId: newId,
                 });
             }
         }
@@ -217,7 +225,8 @@ export function registerMessageTools(server, client) {
             await deleteOFWMessages(client, [args.draftId]);
             deleteDraft(args.draftId);
         }
-        const text = data ? JSON.stringify(data, null, 2) : 'Message sent successfully.';
+        const responseObj = persisted ?? data;
+        const text = responseObj ? JSON.stringify(responseObj, null, 2) : 'Message sent successfully.';
         return textResponse(rewriteNote ? `${rewriteNote}\n\n${text}` : text);
     });
     server.registerTool('ofw_list_drafts', {
@@ -237,7 +246,7 @@ export function registerMessageTools(server, client) {
         return jsonResponse(payload);
     });
     server.registerTool('ofw_save_draft', {
-        description: 'Save a message as a draft in OurFamilyWizard. Recipients are optional. To update an existing draft, provide its messageId. If replyToId is provided, the cache may rewrite it to the latest reply in the thread (note included in response). Attach files by passing their fileIds (from ofw_upload_attachment) in myFileIDs.',
+        description: 'Save a message as a draft in OurFamilyWizard. Recipients are optional. To update an existing draft, provide its messageId. If replyToId is provided, the cache may rewrite it to the latest reply in the thread (note included in response). Attach files by passing their fileIds (from ofw_upload_attachment) in myFileIDs. After saving, the tool re-fetches the draft from OFW to populate the local cache and verify what was actually persisted; if OFW silently no-ops an update (a known issue with repeated updates to the same draft), the response includes a WARNING note with a workaround.',
         annotations: { readOnlyHint: false },
         inputSchema: {
             subject: z.string().describe('Message subject'),
@@ -269,21 +278,42 @@ export function registerMessageTools(server, client) {
         };
         if (args.messageId !== undefined)
             payload.messageId = args.messageId;
+        // OFW's POST /pub/v3/messages response for drafts is minimal — typically
+        // just `{entityId: <id>}` — and worse, it returns the same success shape
+        // even when the server silently no-ops on a subsequent update to the
+        // same draft. Don't trust the POST response: extract the id from it,
+        // then GET the detail endpoint to repopulate the cache from
+        // authoritative server state.
         const data = await client.request('POST', '/pub/v3/messages', payload);
-        if (data && typeof data.id === 'number') {
-            const draft = {
-                id: data.id,
-                subject: data.subject ?? args.subject,
-                body: data.body ?? args.body,
-                recipients: mapRecipients(data.recipients),
-                replyToId: data.replyToId ?? resolvedReplyTo,
-                modifiedAt: data.date?.dateTime ?? new Date().toISOString(),
-                listData: data,
+        const newId = typeof data?.id === 'number' ? data.id
+            : typeof data?.entityId === 'number' ? data.entityId
+                : null;
+        let persisted = null;
+        let noOpWarning = null;
+        if (newId !== null) {
+            const detail = await client.request('GET', `/pub/v3/messages/${newId}`);
+            persisted = {
+                id: newId,
+                subject: detail.subject ?? args.subject,
+                body: detail.body ?? '',
+                recipients: mapRecipients(detail.recipients),
+                replyToId: detail.replyToId ?? resolvedReplyTo,
+                modifiedAt: detail.date?.dateTime ?? new Date().toISOString(),
+                listData: detail,
             };
-            upsertDraft(draft);
+            upsertDraft(persisted);
+            // If this was an update (messageId provided) and OFW's reported body
+            // doesn't match what we asked it to save, the server silently
+            // dropped the change. Warn the caller so the model can take the
+            // create-then-delete fallback.
+            if (args.messageId !== undefined && persisted.body !== args.body) {
+                noOpWarning = 'WARNING: OFW reported success but the draft body it returned does not match the requested update. The OFW POST /pub/v3/messages endpoint can silently no-op on subsequent updates to the same draft. Workaround: delete this draft (ofw_delete_draft) and create a new one (ofw_save_draft without messageId).';
+            }
         }
-        const text = data ? JSON.stringify(data, null, 2) : 'Draft saved.';
-        return textResponse(rewriteNote ? `${rewriteNote}\n\n${text}` : text);
+        const responseObj = persisted ?? data;
+        const text = responseObj ? JSON.stringify(responseObj, null, 2) : 'Draft saved.';
+        const notes = [rewriteNote, noOpWarning].filter((n) => n !== null).join('\n\n');
+        return textResponse(notes ? `${notes}\n\n${text}` : text);
     });
     server.registerTool('ofw_delete_draft', {
         description: 'Delete a draft message from OurFamilyWizard. Also removes the draft from the local cache.',

package/package.json

@@ -1,8 +1,8 @@
 {
   "name": "ofw-mcp",
-  "version": "2.0.14",
+  "version": "2.0.16",
   "mcpName": "io.github.chrischall/ofw-mcp",
-  "description": "OurFamilyWizard MCP server for Claude \u2014 developed and maintained by AI (Claude Code)",
+  "description": "OurFamilyWizard MCP server for Claude — developed and maintained by AI (Claude Code)",
   "author": "Claude Code (AI) <https://www.anthropic.com/claude>",
   "repository": {
     "type": "git",

package/server.json

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