ofw-mcp 2.6.3 → 2.6.4

package/.claude-plugin/marketplace.json

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

package/.claude-plugin/plugin.json

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

package/dist/bundle.js

@@ -38410,7 +38410,7 @@ async function loginWithPassword(username, password) {
 // package.json
 var package_default = {
   name: "ofw-mcp",
-  version: "2.6.3",
+  version: "2.6.4",
   license: "MIT",
   mcpName: "io.github.chrischall/ofw-mcp",
   description: "OurFamilyWizard MCP server for Claude \u2014 developed and maintained by AI (Claude Code)",
@@ -38696,19 +38696,46 @@ var client = new OFWClient();
 var jsonResponse = textResult;
 var textResponse = rawTextResult;
 var ApiRecipientSchema = external_exports.looseObject({
-  user: external_exports.looseObject({ id: external_exports.number().optional(), name: external_exports.string().optional() }).optional(),
+  // Live OFW payloads key the recipient's id as `userId` (verified against a
+  // real /pub/v3/messages record: `recipients[].user.userId === 3039201`). An
+  // earlier guess read `id`, which is absent — so every normalized recipient
+  // came out with `userId: 0`, breaking any "find my own recipient" match. Both
+  // are accepted (userId first, id fallback) so a backend that ever returns `id`
+  // still resolves.
+  user: external_exports.looseObject({
+    userId: external_exports.number().optional(),
+    id: external_exports.number().optional(),
+    name: external_exports.string().optional()
+  }).optional(),
   viewed: external_exports.looseObject({ dateTime: external_exports.string() }).nullable().optional()
 });
 function mapRecipients(items) {
   return (items ?? []).map((r) => {
     const dt = r.viewed?.dateTime;
     const viewedAt = typeof dt === "string" && !dt.startsWith("1970-01-01") ? dt : null;
-    return { userId: r.user?.id ?? 0, name: r.user?.name ?? "", viewedAt };
+    return { userId: r.user?.userId ?? r.user?.id ?? 0, name: r.user?.name ?? "", viewedAt };
   });
 }
 function hasRealView(recipients) {
   return recipients.some((r) => r.viewedAt !== null && !r.viewedAt.startsWith("1970-01-01"));
 }
+function scrapeSaysRead(listData) {
+  if (typeof listData !== "object" || listData === null) return false;
+  const ld = listData;
+  return ld.read === true || ld.showNeverViewed === false;
+}
+function deriveRead(row, selfUserId) {
+  if (row.folder === "inbox") {
+    const viewed = selfUserId !== void 0 ? row.recipients.some((r) => r.userId === selfUserId && r.viewedAt !== null) : row.recipients.some((r) => r.viewedAt !== null);
+    return viewed || row.fetchedBodyAt !== null || scrapeSaysRead(row.listData);
+  }
+  return row.recipients.some((r) => r.viewedAt !== null) || scrapeSaysRead(row.listData);
+}
+function withReadState(row, selfUserId) {
+  const read = deriveRead(row, selfUserId);
+  const listData = typeof row.listData === "object" && row.listData !== null ? { ...row.listData, read, showNeverViewed: !read } : row.listData;
+  return { ...row, read, listData };
+}
 var expandPath2 = expandPath;
 function verifyWriteLanded(kind, sent, persisted) {
   const mismatches = [];
@@ -39239,7 +39266,7 @@ function registerMessageTools(server, client2, cacheProvider, attachmentIO) {
     const cache = cacheProvider();
     const filter = { folder, since: args.since, until: args.until, q: args.q };
     const total = await cache.countMessages(filter);
-    const messages = await cache.listMessages({ ...filter, page, size });
+    const messages = (await cache.listMessages({ ...filter, page, size })).map((m) => withReadState(m));
     const payload = { messages, total, page, size };
     if (total === 0) {
       payload.note = "No messages match these filters. If you expected results, check ofw_sync_messages was run, or relax the filters.";
@@ -39312,7 +39339,7 @@ function registerMessageTools(server, client2, cacheProvider, attachmentIO) {
         } catch {
         }
       }
-      return jsonResponse({ ...row2, attachments: attachments2 });
+      return jsonResponse({ ...withReadState(row2), attachments: attachments2 });
     }
     const detail = parseLenient(
       MessageDetailSchema,
@@ -39344,7 +39371,7 @@ function registerMessageTools(server, client2, cacheProvider, attachmentIO) {
       await fetchAttachmentMetaForMessage(client2, detail.id, detail.files, cache);
     }
     const attachments = await cache.listAttachmentsForMessage(detail.id);
-    return jsonResponse({ ...row, attachments });
+    return jsonResponse({ ...withReadState(row), attachments });
   });
   if (allowSend) server.registerTool("ofw_send_message", {
     description: "Send a message via OurFamilyWizard. To send an existing draft, pass messageId \u2014 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.",
@@ -40548,7 +40575,7 @@ var nodeCacheProvider = () => nodeCache ??= OFWCache.open(getCacheDbPath());
 var nodeAttachmentIO = new NodeAttachmentIO();
 await runMcp({
   name: "ofw",
-  version: "2.6.3",
+  version: "2.6.4",
   // x-release-please-version
   deps: client,
   tools: [

package/dist/index.js

@@ -35,7 +35,7 @@ const nodeAttachmentIO = new NodeAttachmentIO();
 // always succeeds before any credential check runs.
 await runMcp({
     name: 'ofw',
-    version: '2.6.3', // x-release-please-version
+    version: '2.6.4', // x-release-please-version
     deps: client,
     tools: [
         registerUserTools,

package/dist/tools/_shared.js

@@ -10,7 +10,17 @@ export const textResponse = rawTextResult;
 // responses. Used wherever we validate the response of a `/pub/v3/messages*`
 // call. Loose: unknown keys pass through (and survive into cached listData).
 export const ApiRecipientSchema = z.looseObject({
-    user: z.looseObject({ id: z.number().optional(), name: z.string().optional() }).optional(),
+    // Live OFW payloads key the recipient's id as `userId` (verified against a
+    // real /pub/v3/messages record: `recipients[].user.userId === 3039201`). An
+    // earlier guess read `id`, which is absent — so every normalized recipient
+    // came out with `userId: 0`, breaking any "find my own recipient" match. Both
+    // are accepted (userId first, id fallback) so a backend that ever returns `id`
+    // still resolves.
+    user: z.looseObject({
+        userId: z.number().optional(),
+        id: z.number().optional(),
+        name: z.string().optional(),
+    }).optional(),
     viewed: z.looseObject({ dateTime: z.string() }).nullable().optional(),
 });
 // Translates OFW API recipient shape into the cache's normalized Recipient.
@@ -28,7 +38,7 @@ export function mapRecipients(items) {
     return (items ?? []).map((r) => {
         const dt = r.viewed?.dateTime;
         const viewedAt = typeof dt === 'string' && !dt.startsWith('1970-01-01') ? dt : null;
-        return { userId: r.user?.id ?? 0, name: r.user?.name ?? '', viewedAt };
+        return { userId: r.user?.userId ?? r.user?.id ?? 0, name: r.user?.name ?? '', viewedAt };
     });
 }
 // True if any recipient has a *real* "First Viewed" time — i.e. present and
@@ -40,6 +50,62 @@ export function mapRecipients(items) {
 export function hasRealView(recipients) {
     return recipients.some((r) => r.viewedAt !== null && !r.viewedAt.startsWith('1970-01-01'));
 }
+// True when the once-scraped list flags themselves say the message is read.
+// `showNeverViewed === false` is OFW's reliable "has been viewed" signal (per
+// CLAUDE.md); `read === true` is the inbox list's own flag. Both are only ever
+// captured at first sight, so they can go stale — they raise `read` but never
+// lower it (see deriveRead).
+function scrapeSaysRead(listData) {
+    if (typeof listData !== 'object' || listData === null)
+        return false;
+    const ld = listData;
+    return ld.read === true || ld.showNeverViewed === false;
+}
+/**
+ * Derive a message's authoritative read state from the cached record itself,
+ * rather than trusting the `read`/`showNeverViewed` flags scraped once from the
+ * list endpoint. Those flags are frozen at first sight and drift the moment a
+ * message is read after caching — most often when a body fetch
+ * (`ofw_get_message`) marks an inbox message read on OFW as a side effect,
+ * populating `fetchedBodyAt` and the recipient's `viewedAt` but leaving the
+ * stale `read: false` behind.
+ *
+ * The derivation is monotonic — every input can only turn read ON — so a later
+ * resync (which re-scrapes the list flags) can never flip a read message back
+ * to unread:
+ *
+ *  - INBOX: the account holder is the recipient. When we know our own id
+ *    (`selfUserId`), that recipient's `viewedAt` is authoritative; otherwise any
+ *    recipient's `viewedAt` stands in (1:1 co-parent messaging). Fetching the
+ *    body marks the message read on OFW, so a non-null `fetchedBodyAt` is also
+ *    read=true. The stale scrape flag is only a last-resort fallback.
+ *  - SENT: "read" means a *recipient* has opened it — tracked via their
+ *    `viewedAt` (the detail endpoint's real timestamp) — never our own body
+ *    fetch, which is always set for sent messages.
+ */
+export function deriveRead(row, selfUserId) {
+    if (row.folder === 'inbox') {
+        const viewed = selfUserId !== undefined
+            ? row.recipients.some((r) => r.userId === selfUserId && r.viewedAt !== null)
+            : row.recipients.some((r) => r.viewedAt !== null);
+        return viewed || row.fetchedBodyAt !== null || scrapeSaysRead(row.listData);
+    }
+    return row.recipients.some((r) => r.viewedAt !== null) || scrapeSaysRead(row.listData);
+}
+/**
+ * Return the row augmented with an authoritative top-level `read` boolean and a
+ * `listData` whose `read`/`showNeverViewed` flags are forced to agree with it —
+ * so a single response can never contradict itself (the reported bug: a record
+ * carrying `listData.read: false` alongside a populated recipient `viewedAt`).
+ * A non-object `listData` (null / legacy string) is passed through untouched.
+ */
+export function withReadState(row, selfUserId) {
+    const read = deriveRead(row, selfUserId);
+    const listData = (typeof row.listData === 'object' && row.listData !== null)
+        ? { ...row.listData, read, showNeverViewed: !read }
+        : row.listData;
+    return { ...row, read, listData };
+}
 // Expand a user-provided path: ~ → home, relative → absolute. Re-exports
 // @chrischall/mcp-utils' `expandPath`.
 export const expandPath = expandPathUtil;

package/dist/tools/messages.js

@@ -2,7 +2,7 @@ import { z } from 'zod';
 import { syncAll, fetchAttachmentMeta, fetchAttachmentMetaForMessage } from '../sync.js';
 import { getAttachmentsDir, getDefaultInlineAttachments, getSyncMaxRequests, getWriteMode } from '../config.js';
 import { basename, join } from 'node:path';
-import { ApiRecipientSchema, expandPath, hasRealView, jsonResponse, mapRecipients, postMessageAndRefetch, textResponse, verifyWriteLanded } from './_shared.js';
+import { ApiRecipientSchema, expandPath, hasRealView, jsonResponse, mapRecipients, postMessageAndRefetch, textResponse, verifyWriteLanded, withReadState } from './_shared.js';
 import { parseLenient } from '@chrischall/mcp-utils';
 // Schemas for the load-bearing fields of each /pub/v3 response this file
 // reads (issue #83). Loose: unknown keys pass through into cached listData.
@@ -112,7 +112,11 @@ export function registerMessageTools(server, client, cacheProvider, attachmentIO
         const cache = cacheProvider();
         const filter = { folder, since: args.since, until: args.until, q: args.q };
         const total = await cache.countMessages(filter);
-        const messages = await cache.listMessages({ ...filter, page, size });
+        // Reconcile each row's read state at read time: the cached list flags can be
+        // stale (a message read after it was first scraped), so `read` is derived
+        // from the record's own `viewedAt`/`fetchedBodyAt` and `listData` is forced
+        // to agree — see withReadState.
+        const messages = (await cache.listMessages({ ...filter, page, size })).map((m) => withReadState(m));
         const payload = { messages, total, page, size };
         if (total === 0) {
             payload.note = 'No messages match these filters. If you expected results, check ofw_sync_messages was run, or relax the filters.';
@@ -205,7 +209,7 @@ export function registerMessageTools(server, client, cacheProvider, attachmentIO
                     // Backfill is best-effort. Fall through with whatever we have.
                 }
             }
-            return jsonResponse({ ...row, attachments });
+            return jsonResponse({ ...withReadState(row), attachments });
         }
         const detail = parseLenient(MessageDetailSchema, await client.request('GET', `/pub/v3/messages/${encodeURIComponent(args.messageId)}`), { label: 'ofw-mcp', context: 'GET /pub/v3/messages/{id} (ofw_get_message)' });
         // Derive the folder for a live-fetched message. A cached row (reached here
@@ -240,7 +244,7 @@ export function registerMessageTools(server, client, cacheProvider, attachmentIO
             await fetchAttachmentMetaForMessage(client, detail.id, detail.files, cache);
         }
         const attachments = await cache.listAttachmentsForMessage(detail.id);
-        return jsonResponse({ ...row, attachments });
+        return jsonResponse({ ...withReadState(row), attachments });
     });
     if (allowSend)
         server.registerTool('ofw_send_message', {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ofw-mcp",
-  "version": "2.6.3",
+  "version": "2.6.4",
   "license": "MIT",
   "mcpName": "io.github.chrischall/ofw-mcp",
   "description": "OurFamilyWizard MCP server for Claude — developed and maintained by AI (Claude Code)",

package/server.json

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

package/skills/ofw-fpx/references/requests.md

@@ -55,7 +55,7 @@ read on OFW):
 
 ```sh
 curl -s "https://ofw.ourfamilywizard.com/pub/v3/messages/${ID}" "${AUTH_HEADERS[@]}" \
-  | jq '{id, subject, body, sentAt: .date.dateTime, from: .from.name, files, recipients: [.recipients[] | {id: .user.id, name: .user.name, viewedAt: .viewed.dateTime}]}'
+  | jq '{id, subject, body, sentAt: .date.dateTime, from: .from.name, files, recipients: [.recipients[] | {id: .user.userId, name: .user.name, viewedAt: .viewed.dateTime}]}'
 ```
 
 ## 3. Send a message / save a draft (write — confirm-by-re-GET)