ofw-mcp 2.0.10 → 2.0.11

package/.claude-plugin/marketplace.json

@@ -6,7 +6,7 @@
   },
   "metadata": {
     "description": "OurFamilyWizard tools for Claude Code",
-    "version": "2.0.10"
+    "version": "2.0.11"
   },
   "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.10",
+      "version": "2.0.11",
       "author": {
         "name": "Chris Chall"
       },

package/.claude-plugin/plugin.json

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

package/dist/bundle.js

@@ -31379,12 +31379,14 @@ function listAttachmentsForMessage(messageId) {
 function upsertAttachmentForMessage(input) {
   const { db } = openCache();
   const existing = db.prepare("SELECT message_ids_json FROM attachments WHERE file_id = ?").get(input.fileId);
+  const prior = existing ? JSON.parse(existing.message_ids_json) : [];
   let messageIds;
-  if (existing) {
-    const arr = JSON.parse(existing.message_ids_json);
-    messageIds = arr.includes(input.messageId) ? arr : [...arr, input.messageId];
+  if (input.messageId === 0) {
+    messageIds = prior;
+  } else if (prior.includes(input.messageId)) {
+    messageIds = prior;
   } else {
-    messageIds = [input.messageId];
+    messageIds = [...prior, input.messageId];
   }
   db.prepare(
     `INSERT INTO attachments (
@@ -31621,6 +31623,13 @@ var MIME_BY_EXT = {
 function mimeFromName(name) {
   return MIME_BY_EXT[extname(name).toLowerCase()] ?? "application/octet-stream";
 }
+function listDataHintsAtFiles(listData) {
+  if (typeof listData !== "object" || listData === null) return false;
+  const ld = listData;
+  if (typeof ld.files === "number") return ld.files > 0;
+  if (Array.isArray(ld.files)) return ld.files.length > 0;
+  return false;
+}
 function registerMessageTools(server2, client2) {
   server2.registerTool("ofw_list_message_folders", {
     description: "List OurFamilyWizard message folders (inbox, sent, etc.) and their unread counts. Returns folder IDs needed to call ofw_list_messages. Does NOT return message content.",
@@ -31680,7 +31689,17 @@ function registerMessageTools(server2, client2) {
     const id = Number(args.messageId);
     const cached2 = getMessage(id);
     if (cached2 && cached2.body !== null) {
-      const attachments2 = listAttachmentsForMessage(id);
+      let attachments2 = listAttachmentsForMessage(id);
+      if (attachments2.length === 0 && listDataHintsAtFiles(cached2.listData)) {
+        try {
+          const detail2 = await client2.request("GET", `/pub/v3/messages/${id}`);
+          if (Array.isArray(detail2.files) && detail2.files.length > 0) {
+            await fetchAttachmentMetaForMessage(client2, id, detail2.files);
+            attachments2 = listAttachmentsForMessage(id);
+          }
+        } catch {
+        }
+      }
       return { content: [{ type: "text", text: JSON.stringify({ ...cached2, attachments: attachments2 }, null, 2) }] };
     }
     const detail = await client2.request("GET", `/pub/v3/messages/${encodeURIComponent(args.messageId)}`);
@@ -32161,7 +32180,7 @@ process.emit = function(event, ...args) {
   }
   return originalEmit(event, ...args);
 };
-var server = new McpServer({ name: "ofw", version: "2.0.10" });
+var server = new McpServer({ name: "ofw", version: "2.0.11" });
 registerUserTools(server, client);
 registerMessageTools(server, client);
 registerCalendarTools(server, client);

package/dist/cache.js

@@ -295,13 +295,19 @@ export function upsertAttachmentForMessage(input) {
     const { db } = openCache();
     const existing = db.prepare('SELECT message_ids_json FROM attachments WHERE file_id = ?')
         .get(input.fileId);
+    // messageId === 0 is the "metadata-only, not yet linked to a message"
+    // sentinel used by upload-without-send and download-by-id. Don't
+    // pollute the array with it — leave the list empty / unchanged.
+    const prior = existing ? JSON.parse(existing.message_ids_json) : [];
     let messageIds;
-    if (existing) {
-        const arr = JSON.parse(existing.message_ids_json);
-        messageIds = arr.includes(input.messageId) ? arr : [...arr, input.messageId];
+    if (input.messageId === 0) {
+        messageIds = prior;
+    }
+    else if (prior.includes(input.messageId)) {
+        messageIds = prior;
     }
     else {
-        messageIds = [input.messageId];
+        messageIds = [...prior, input.messageId];
     }
     db.prepare(`INSERT INTO attachments (
        file_id, file_name, label, mime_type, size_bytes,

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.10' });
+const server = new McpServer({ name: 'ofw', version: '2.0.11' });
 registerUserTools(server, client);
 registerMessageTools(server, client);
 registerCalendarTools(server, client);

package/dist/tools/messages.js

@@ -31,6 +31,20 @@ const MIME_BY_EXT = {
 function mimeFromName(name) {
     return MIME_BY_EXT[extname(name).toLowerCase()] ?? 'application/octet-stream';
 }
+// The list endpoint payload (cached as `listData`) reports attachments via
+// `files: <count>` (a number) — the actual fileIds only appear on the detail
+// endpoint as `files: [number, ...]`. Some intermediate shapes return an
+// array on the list too. Treat any of those as "this message has files".
+function listDataHintsAtFiles(listData) {
+    if (typeof listData !== 'object' || listData === null)
+        return false;
+    const ld = listData;
+    if (typeof ld.files === 'number')
+        return ld.files > 0;
+    if (Array.isArray(ld.files))
+        return ld.files.length > 0;
+    return false;
+}
 export function registerMessageTools(server, client) {
     server.registerTool('ofw_list_message_folders', {
         description: 'List OurFamilyWizard message folders (inbox, sent, etc.) and their unread counts. Returns folder IDs needed to call ofw_list_messages. Does NOT return message content.',
@@ -94,7 +108,26 @@ export function registerMessageTools(server, client) {
         const id = Number(args.messageId);
         const cached = getMessage(id);
         if (cached && cached.body !== null) {
-            const attachments = listAttachmentsForMessage(id);
+            let attachments = listAttachmentsForMessage(id);
+            // Lazy attachment backfill. The list-endpoint payload (stored in
+            // listData) hints at attachments via `files: <count>` but doesn't
+            // expose the fileIds — those live only on /pub/v3/messages/{id}.
+            // For messages bodied before attachment caching existed, the
+            // attachments table is empty even though OFW has files. Re-hit
+            // detail to harvest fileIds (idempotent: body is already cached so
+            // OFW state isn't changing).
+            if (attachments.length === 0 && listDataHintsAtFiles(cached.listData)) {
+                try {
+                    const detail = await client.request('GET', `/pub/v3/messages/${id}`);
+                    if (Array.isArray(detail.files) && detail.files.length > 0) {
+                        await fetchAttachmentMetaForMessage(client, id, detail.files);
+                        attachments = listAttachmentsForMessage(id);
+                    }
+                }
+                catch {
+                    // Backfill is best-effort. Fall through with whatever we have.
+                }
+            }
             return { content: [{ type: 'text', text: JSON.stringify({ ...cached, attachments }, null, 2) }] };
         }
         const detail = await client.request('GET', `/pub/v3/messages/${encodeURIComponent(args.messageId)}`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ofw-mcp",
-  "version": "2.0.10",
+  "version": "2.0.11",
   "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>",

package/server.json

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