ofw-mcp 2.19.2 → 2.19.3

package/dist/bundle.js

@@ -8818,7 +8818,7 @@ var recursive = /* @__PURE__ */ new WeakMap();
 var NONE = 0;
 var ASSUMED = 1;
 var PROVEN = 2;
-function isRecursive(inst, stack, resolve4) {
+function isRecursive(inst, stack, resolve6) {
   const cached2 = recursive.get(inst);
   if (cached2 !== void 0)
     return cached2 ? PROVEN : NONE;
@@ -8828,7 +8828,7 @@ function isRecursive(inst, stack, resolve4) {
   let result = NONE;
   const check2 = (child) => {
     if (result !== PROVEN && child?._zod) {
-      const answer = isRecursive(child, stack, resolve4);
+      const answer = isRecursive(child, stack, resolve6);
       if (answer > result)
         result = answer;
     }
@@ -8839,7 +8839,7 @@ function isRecursive(inst, stack, resolve4) {
       const desc = Object.getOwnPropertyDescriptor(sh, key);
       if (spread && !desc.enumerable)
         continue;
-      const child = desc.get ? ASSUMED : desc.value?._zod ? isRecursive(desc.value, stack, resolve4) : NONE;
+      const child = desc.get ? ASSUMED : desc.value?._zod ? isRecursive(desc.value, stack, resolve6) : NONE;
       if (child > answer)
         answer = child;
     }
@@ -8903,7 +8903,7 @@ function isRecursive(inst, stack, resolve4) {
       break;
     // `$ZodLazy` caches its inner on the def, so a resolved edge is followed exactly
     case "lazy": {
-      const inner = def._cachedInner ?? (resolve4 ? inst._zod.innerType : void 0);
+      const inner = def._cachedInner ?? (resolve6 ? inst._zod.innerType : void 0);
       merge2(inner ? isRecursive(inner, stack, false) : ASSUMED);
       break;
     }
@@ -27803,14 +27803,14 @@ function inputRequiredRoundsExceededMessage(method, maxRounds) {
   return `Multi-round-trip request '${method}' still required input after ${maxRounds} rounds (inputRequired.maxRounds)`;
 }
 function sleep(ms, signal) {
-  return new Promise((resolve4, reject) => {
+  return new Promise((resolve6, reject) => {
     if (signal?.aborted) {
       reject(signal.reason instanceof SdkError ? signal.reason : new SdkError(SdkErrorCode.RequestTimeout, String(signal.reason)));
       return;
     }
     const timer = setTimeout(() => {
       signal?.removeEventListener("abort", onAbort);
-      resolve4();
+      resolve6();
     }, ms);
     const onAbort = () => {
       clearTimeout(timer);
@@ -28604,7 +28604,7 @@ var Protocol = class {
     const flowStartedAt = Date.now();
     let onAbort;
     let cleanupMessageId;
-    return new Promise((resolve4, reject) => {
+    return new Promise((resolve6, reject) => {
       const earlyReject = (error62) => {
         reject(error62);
       };
@@ -28672,7 +28672,7 @@ var Protocol = class {
         }
         if (decoded.kind === "invalid") return reject(decoded.error);
         if (decoded.kind === "input_required") {
-          if (options?.allowInputRequired === true) return resolve4(manualInputRequiredValue(decoded));
+          if (options?.allowInputRequired === true) return resolve6(manualInputRequiredValue(decoded));
           const flow = {
             codec: codec2,
             request,
@@ -28684,11 +28684,11 @@ var Protocol = class {
               params
             }, resultSchema, legOptions)
           };
-          return resolve4(this._resolveNonCompleteResult(decoded, flow));
+          return resolve6(this._resolveNonCompleteResult(decoded, flow));
         }
         const result = decoded.result;
         validateStandardSchema(resultSchema, result).then((parseResult) => {
-          if (parseResult.success) resolve4(parseResult.data);
+          if (parseResult.success) resolve6(parseResult.data);
           else reject(new SdkError(SdkErrorCode.InvalidResult, `Invalid result for ${request.method}: ${parseResult.error}`));
         }, reject);
       });
@@ -31600,7 +31600,7 @@ var require_compile = /* @__PURE__ */ __commonJSMin(((exports) => {
     ref = (0, resolve_1.resolveUrl)(this.opts.uriResolver, baseId, ref);
     const schOrFunc = root.refs[ref];
     if (schOrFunc) return schOrFunc;
-    let _sch = resolve4.call(this, root, ref);
+    let _sch = resolve6.call(this, root, ref);
     if (_sch === void 0) {
       const schema = (_a3 = root.localRefs) === null || _a3 === void 0 ? void 0 : _a3[ref];
       const { schemaId } = this.opts;
@@ -31626,7 +31626,7 @@ var require_compile = /* @__PURE__ */ __commonJSMin(((exports) => {
   function sameSchemaEnv(s1, s2) {
     return s1.schema === s2.schema && s1.root === s2.root && s1.baseId === s2.baseId;
   }
-  function resolve4(root, ref) {
+  function resolve6(root, ref) {
     let sch;
     while (typeof (sch = this.refs[ref]) == "string") ref = sch;
     return sch || this.schemas[ref] || resolveSchema.call(this, root, ref);
@@ -32076,47 +32076,47 @@ var require_fast_uri = /* @__PURE__ */ __commonJSMin(((exports, module) => {
     else if (typeof uri === "object") uri = parse3(serialize(uri, options), options);
     return uri;
   }
-  function resolve4(baseURI, relativeURI, options) {
+  function resolve6(baseURI, relativeURI, options) {
     const schemelessOptions = options ? Object.assign({ scheme: "null" }, options) : { scheme: "null" };
     const resolved = resolveComponent(parse3(baseURI, schemelessOptions), parse3(relativeURI, schemelessOptions), schemelessOptions, true);
     schemelessOptions.skipEscape = true;
     return serialize(resolved, schemelessOptions);
   }
-  function resolveComponent(base, relative2, options, skipNormalization) {
+  function resolveComponent(base, relative3, options, skipNormalization) {
     const target = {};
     if (!skipNormalization) {
       base = parse3(serialize(base, options), options);
-      relative2 = parse3(serialize(relative2, options), options);
+      relative3 = parse3(serialize(relative3, options), options);
     }
     options = options || {};
-    if (!options.tolerant && relative2.scheme) {
-      target.scheme = relative2.scheme;
-      target.userinfo = relative2.userinfo;
-      target.host = relative2.host;
-      target.port = relative2.port;
-      target.path = removeDotSegments(relative2.path || "");
-      target.query = relative2.query;
+    if (!options.tolerant && relative3.scheme) {
+      target.scheme = relative3.scheme;
+      target.userinfo = relative3.userinfo;
+      target.host = relative3.host;
+      target.port = relative3.port;
+      target.path = removeDotSegments(relative3.path || "");
+      target.query = relative3.query;
     } else {
-      if (relative2.userinfo !== void 0 || relative2.host !== void 0 || relative2.port !== void 0) {
-        target.userinfo = relative2.userinfo;
-        target.host = relative2.host;
-        target.port = relative2.port;
-        target.path = removeDotSegments(relative2.path || "");
-        target.query = relative2.query;
+      if (relative3.userinfo !== void 0 || relative3.host !== void 0 || relative3.port !== void 0) {
+        target.userinfo = relative3.userinfo;
+        target.host = relative3.host;
+        target.port = relative3.port;
+        target.path = removeDotSegments(relative3.path || "");
+        target.query = relative3.query;
       } else {
-        if (!relative2.path) {
+        if (!relative3.path) {
           target.path = base.path;
-          if (relative2.query !== void 0) target.query = relative2.query;
+          if (relative3.query !== void 0) target.query = relative3.query;
           else target.query = base.query;
         } else {
-          if (relative2.path[0] === "/") target.path = removeDotSegments(relative2.path);
+          if (relative3.path[0] === "/") target.path = removeDotSegments(relative3.path);
           else {
-            if ((base.userinfo !== void 0 || base.host !== void 0 || base.port !== void 0) && !base.path) target.path = "/" + relative2.path;
-            else if (!base.path) target.path = relative2.path;
-            else target.path = base.path.slice(0, base.path.lastIndexOf("/") + 1) + relative2.path;
+            if ((base.userinfo !== void 0 || base.host !== void 0 || base.port !== void 0) && !base.path) target.path = "/" + relative3.path;
+            else if (!base.path) target.path = relative3.path;
+            else target.path = base.path.slice(0, base.path.lastIndexOf("/") + 1) + relative3.path;
             target.path = removeDotSegments(target.path);
           }
-          target.query = relative2.query;
+          target.query = relative3.query;
         }
         target.userinfo = base.userinfo;
         target.host = base.host;
@@ -32124,7 +32124,7 @@ var require_fast_uri = /* @__PURE__ */ __commonJSMin(((exports, module) => {
       }
       target.scheme = base.scheme;
     }
-    target.fragment = relative2.fragment;
+    target.fragment = relative3.fragment;
     return target;
   }
   function equal(uriA, uriB, options) {
@@ -32250,7 +32250,7 @@ var require_fast_uri = /* @__PURE__ */ __commonJSMin(((exports, module) => {
   const fastUri = {
     SCHEMES,
     normalize,
-    resolve: resolve4,
+    resolve: resolve6,
     resolveComponent,
     equal,
     serialize,
@@ -37623,7 +37623,7 @@ var StdioServerTransport = class {
   }
   send(message) {
     if (this._closed) return Promise.reject(/* @__PURE__ */ new Error("StdioServerTransport is closed"));
-    return new Promise((resolve4, reject) => {
+    return new Promise((resolve6, reject) => {
       const json2 = serializeMessage(message);
       let settled = false;
       const onError = (error62) => {
@@ -37638,14 +37638,14 @@ var StdioServerTransport = class {
         settled = true;
         this._stdout.off("error", onError);
         this._stdout.off("drain", onDrain);
-        resolve4();
+        resolve6();
       };
       this._stdout.once("error", onError);
       if (this._stdout.write(json2)) {
         if (settled) return;
         settled = true;
         this._stdout.off("error", onError);
-        resolve4();
+        resolve6();
       } else if (!settled) this._stdout.once("drain", onDrain);
     });
   }
@@ -37699,14 +37699,14 @@ var StdioConnectionChannel = class {
   */
   async whenRequestsAnswered(timeoutMs) {
     if (this._closed || this._pendingRequests.size === 0) return true;
-    return await new Promise((resolve4) => {
+    return await new Promise((resolve6) => {
       const waiter = () => {
         clearTimeout(timer);
-        resolve4(true);
+        resolve6(true);
       };
       const timer = setTimeout(() => {
         this._drainWaiters = this._drainWaiters.filter((pending) => pending !== waiter);
-        resolve4(false);
+        resolve6(false);
       }, timeoutMs);
       this._drainWaiters.push(waiter);
     });
@@ -40480,7 +40480,7 @@ var DEFAULT_BIND_TIMEOUT_MS = 5e3;
 async function electRole(opts) {
   const server = createServer();
   const bindTimeoutMs = opts.bindTimeoutMs ?? DEFAULT_BIND_TIMEOUT_MS;
-  return new Promise((resolve4, reject) => {
+  return new Promise((resolve6, reject) => {
     let timer;
     const clearBindTimer = () => {
       if (timer) {
@@ -40496,7 +40496,7 @@ async function electRole(opts) {
           server.close();
         } catch {
         }
-        resolve4({ role: "peer" });
+        resolve6({ role: "peer" });
       } else {
         reject(e);
       }
@@ -40504,7 +40504,7 @@ async function electRole(opts) {
     const onListening = () => {
       clearBindTimer();
       server.removeListener("error", onError);
-      resolve4({ role: "host", server });
+      resolve6({ role: "host", server });
     };
     server.once("error", onError);
     server.once("listening", onListening);
@@ -41153,8 +41153,8 @@ async function startHost(opts) {
   let rejectOwnSession;
   let ownSessionReady;
   function resetSessionPromise() {
-    ownSessionReady = new Promise((resolve4, reject) => {
-      resolveOwnSession = resolve4;
+    ownSessionReady = new Promise((resolve6, reject) => {
+      resolveOwnSession = resolve6;
       rejectOwnSession = reject;
     });
     ownSessionReady.catch(() => {
@@ -41556,7 +41556,7 @@ async function startHost(opts) {
     });
   });
   return {
-    close: () => new Promise((resolve4) => {
+    close: () => new Promise((resolve6) => {
       for (const client2 of wss.clients) {
         try {
           client2.terminate();
@@ -41564,7 +41564,7 @@ async function startHost(opts) {
         }
       }
       wss.close(() => {
-        opts.httpServer.close(() => resolve4());
+        opts.httpServer.close(() => resolve6());
       });
     }),
     sendOwnInner: async (inner) => {
@@ -41604,11 +41604,11 @@ async function startPeer(opts) {
     maxPayload: opts.maxPayloadBytes ?? MAX_PAYLOAD_BYTES
   });
   const dialTimeoutMs = opts.dialTimeoutMs ?? PEER_DIAL_TIMEOUT_MS;
-  await new Promise((resolve4, reject) => {
+  await new Promise((resolve6, reject) => {
     let timer;
     const onOpen = () => {
       cleanup();
-      resolve4();
+      resolve6();
     };
     const onError = (e) => {
       cleanup();
@@ -41696,10 +41696,10 @@ async function startPeer(opts) {
   let sessionPromiseSettled = false;
   function resetSessionPromise() {
     sessionPromiseSettled = false;
-    sessionPromise = new Promise((resolve4, reject) => {
+    sessionPromise = new Promise((resolve6, reject) => {
       resolveFirstReady = (st) => {
         sessionPromiseSettled = true;
-        resolve4(st);
+        resolve6(st);
       };
       rejectFirstReady = (e) => {
         sessionPromiseSettled = true;
@@ -42849,8 +42849,8 @@ var FetchproxyServer = class {
   async _fetchOnceWithTimeout(init, fetchOpts = {}) {
     const id = this.nextRequestId++;
     const inner = { type: "request", id, op: "fetch", init };
-    const pending = this.guardPending(new Promise((resolve4) => {
-      this.pending.set(id, resolve4);
+    const pending = this.guardPending(new Promise((resolve6) => {
+      this.pending.set(id, resolve6);
     }));
     try {
       await this.sendInnerFrame(inner, {
@@ -42873,12 +42873,12 @@ var FetchproxyServer = class {
     try {
       return await Promise.race([
         pending,
-        new Promise((resolve4) => {
+        new Promise((resolve6) => {
           timer = setTimeout(() => {
             this.pending.delete(id);
             const elapsedMs = Date.now() - start;
             const error62 = `fetchproxy: ${init.url} did not respond within ${timeoutMs}ms`;
-            resolve4({
+            resolve6({
               ok: false,
               error: error62,
               kind: "timeout",
@@ -43022,9 +43022,9 @@ var FetchproxyServer = class {
     if (opts.subdomain !== void 0)
       assertSubdomainLabel(opts.subdomain);
     const baseDomain = this.resolveBaseDomain(opts.domain);
-    const isAbsolute5 = path.startsWith("http://") || path.startsWith("https://");
+    const isAbsolute6 = path.startsWith("http://") || path.startsWith("https://");
     let host;
-    if (isAbsolute5) {
+    if (isAbsolute6) {
       try {
         host = new URL(path).host;
       } catch {
@@ -43033,7 +43033,7 @@ var FetchproxyServer = class {
     } else {
       host = opts.subdomain ? `${opts.subdomain}.${baseDomain}` : baseDomain;
     }
-    const url2 = isAbsolute5 ? path : `https://${host}${path}`;
+    const url2 = isAbsolute6 ? path : `https://${host}${path}`;
     assertUrlInDomains("request url", url2, this.opts.domains);
     let tabUrl = `https://${host}/`;
     if (opts.viaTab !== void 0) {
@@ -43298,8 +43298,8 @@ var FetchproxyServer = class {
       const tabUrl = `https://${host}/`;
       inner = { type: "request", id, op: "read_cookies", init: { tabUrl } };
     }
-    const pending = this.guardPending(new Promise((resolve4) => {
-      this.pendingReadCookies.set(id, resolve4);
+    const pending = this.guardPending(new Promise((resolve6) => {
+      this.pendingReadCookies.set(id, resolve6);
     }));
     await this.sendInnerFrame(inner);
     const result = await this._withVerbTimeout(pending, this.pendingReadCookies, id, `https://${host}`);
@@ -43357,8 +43357,8 @@ var FetchproxyServer = class {
         ...cookiePath !== void 0 ? { path: cookiePath } : {}
       }
     };
-    const pending = this.guardPending(new Promise((resolve4, reject) => {
-      this.pendingWriteCookies.set(id, { resolve: resolve4, reject });
+    const pending = this.guardPending(new Promise((resolve6, reject) => {
+      this.pendingWriteCookies.set(id, { resolve: resolve6, reject });
     }));
     await this.sendInnerFrame(inner);
     return this._withVerbTimeout(
@@ -43428,8 +43428,8 @@ var FetchproxyServer = class {
         ...opts.pointers ? { pointers: { ...opts.pointers } } : {}
       }
     };
-    const pending = this.guardPending(new Promise((resolve4, reject) => {
-      this.pendingStorage.set(id, { resolve: resolve4, reject });
+    const pending = this.guardPending(new Promise((resolve6, reject) => {
+      this.pendingStorage.set(id, { resolve: resolve6, reject });
     }));
     await this.sendInnerFrame(inner);
     return this._withVerbTimeout(pending, this.pendingStorage, id, `https://${host}`);
@@ -43487,8 +43487,8 @@ var FetchproxyServer = class {
         ...opts.timeoutMs !== void 0 ? { timeoutMs: opts.timeoutMs } : {}
       }
     };
-    const pending = this.guardPending(new Promise((resolve4, reject) => {
-      this.pendingCapture.set(id, { resolve: resolve4, reject });
+    const pending = this.guardPending(new Promise((resolve6, reject) => {
+      this.pendingCapture.set(id, { resolve: resolve6, reject });
     }));
     await this.sendInnerFrame(inner);
     return this._withVerbTimeout(
@@ -43535,8 +43535,8 @@ var FetchproxyServer = class {
         ...opts.timeoutMs !== void 0 ? { timeoutMs: opts.timeoutMs } : {}
       }
     };
-    const pending = this.guardPending(new Promise((resolve4, reject) => {
-      this.pendingRedirect.set(id, { resolve: resolve4, reject });
+    const pending = this.guardPending(new Promise((resolve6, reject) => {
+      this.pendingRedirect.set(id, { resolve: resolve6, reject });
     }));
     await this.sendInnerFrame(inner);
     return this._withVerbTimeout(
@@ -43631,8 +43631,8 @@ var FetchproxyServer = class {
         ...opts.timeoutMs !== void 0 ? { timeoutMs: opts.timeoutMs } : {}
       }
     };
-    const pending = this.guardPending(new Promise((resolve4, reject) => {
-      this.pendingDownload.set(id, { resolve: resolve4, reject });
+    const pending = this.guardPending(new Promise((resolve6, reject) => {
+      this.pendingDownload.set(id, { resolve: resolve6, reject });
     }));
     await this.sendInnerFrame(inner);
     return this._withVerbTimeout(
@@ -43692,8 +43692,8 @@ var FetchproxyServer = class {
         keys: [...opts.keys]
       }
     };
-    const pending = this.guardPending(new Promise((resolve4, reject) => {
-      this.pendingIdb.set(id, { resolve: resolve4, reject });
+    const pending = this.guardPending(new Promise((resolve6, reject) => {
+      this.pendingIdb.set(id, { resolve: resolve6, reject });
     }));
     await this.sendInnerFrame(inner);
     return this._withVerbTimeout(pending, this.pendingIdb, id, origin);
@@ -43732,8 +43732,8 @@ var FetchproxyServer = class {
       op: "read_dom",
       init: { origin, names: [...opts.names] }
     };
-    const pending = this.guardPending(new Promise((resolve4, reject) => {
-      this.pendingStorage.set(id, { resolve: resolve4, reject });
+    const pending = this.guardPending(new Promise((resolve6, reject) => {
+      this.pendingStorage.set(id, { resolve: resolve6, reject });
     }));
     await this.sendInnerFrame(inner);
     return this._withVerbTimeout(pending, this.pendingStorage, id, origin);
@@ -43778,8 +43778,8 @@ var FetchproxyServer = class {
       op: "read_dom_list",
       init: { origin, name: opts.name }
     };
-    const pending = this.guardPending(new Promise((resolve4, reject) => {
-      this.pendingDomList.set(id, { resolve: resolve4, reject });
+    const pending = this.guardPending(new Promise((resolve6, reject) => {
+      this.pendingDomList.set(id, { resolve: resolve6, reject });
     }));
     await this.sendInnerFrame(inner);
     return this._withVerbTimeout(pending, this.pendingDomList, id, origin);
@@ -43824,8 +43824,8 @@ var FetchproxyServer = class {
         ...opts.tabUrl !== void 0 ? { tabUrl: opts.tabUrl } : {}
       }
     };
-    const pending = this.guardPending(new Promise((resolve4, reject) => {
-      this.pendingGraphql.set(id, { resolve: resolve4, reject });
+    const pending = this.guardPending(new Promise((resolve6, reject) => {
+      this.pendingGraphql.set(id, { resolve: resolve6, reject });
     }));
     const retrySafe = opts.retryOnTimeout === true;
     await this.sendInnerFrame(inner, { resendable: retrySafe });
@@ -44651,7 +44651,7 @@ async function loginWithPassword(username, password) {
 // package.json
 var package_default = {
   name: "ofw-mcp",
-  version: "2.19.2",
+  version: "2.19.3",
   license: "MIT",
   mcpName: "io.github.chrischall/ofw-mcp",
   description: "OurFamilyWizard MCP server for Claude \u2014 developed and maintained by AI (Claude Code)",
@@ -45168,6 +45168,34 @@ function parseTimestampValue(key, value, assumeNaiveIn) {
   }
   return null;
 }
+function naiveWallClockOf(instant, tz) {
+  return isoWithOffset(instant, tz, "");
+}
+function nowNaiveWallClock(tz = displayTimeZone()) {
+  return naiveWallClockOf(/* @__PURE__ */ new Date(), tz);
+}
+function toNaiveWallClock(value, tz = displayTimeZone()) {
+  const raw = value.trim();
+  if (DATE_ONLY.test(raw)) return raw;
+  if (RFC3339_WITH_OFFSET.test(raw)) {
+    const instant = parseTimestampValue("", raw, tz);
+    return instant ? naiveWallClockOf(instant, tz) : null;
+  }
+  const naive = NAIVE_DATE_TIME.exec(raw);
+  if (!naive) return null;
+  const [, y, mo, d, h, mi, s, frac] = naive;
+  const parts = {
+    year: Number(y),
+    month: Number(mo),
+    day: Number(d),
+    hour: Number(h),
+    minute: Number(mi),
+    second: Number(s ?? "0")
+  };
+  if (!isRealCalendarDate(parts)) return null;
+  const ms = frac ? `.${frac.padEnd(3, "0").slice(0, 3)}` : "";
+  return `${y}-${mo}-${d}T${h}:${mi}:${pad(parts.second)}${ms}`;
+}
 function walk(node2, tz) {
   if (Array.isArray(node2)) {
     for (const item of node2) walk(item, tz);
@@ -45270,17 +45298,43 @@ var PostMessagesResponseSchema = external_exports.looseObject({
   id: external_exports.number().optional(),
   entityId: external_exports.number().optional()
 }).nullable();
+var UnconfirmedWriteError = class extends Error {
+  constructor(postedId, cause) {
+    super(cause instanceof Error ? cause.message : String(cause), { cause });
+    this.postedId = postedId;
+    this.name = "UnconfirmedWriteError";
+  }
+  postedId;
+};
+function isDefinitiveRejection(e) {
+  if (!(e instanceof Error)) return false;
+  if (/^Rate limited by OFW API/.test(e.message)) return true;
+  const m = /OFW API error: (4\d\d)\b/.exec(e.message);
+  return m !== null && m[1] !== "408";
+}
 async function postMessageAndRefetch(client2, payload, detailSchema, ctx) {
+  let posted;
+  try {
+    posted = await client2.request("POST", "/pub/v3/messages", payload);
+  } catch (e) {
+    throw isDefinitiveRejection(e) ? e : new UnconfirmedWriteError(null, e);
+  }
   const raw = parseLenient(
     PostMessagesResponseSchema,
-    await client2.request("POST", "/pub/v3/messages", payload),
+    posted,
     { label: "ofw-mcp", context: `POST /pub/v3/messages (${ctx})`, mode: "strict" }
   );
   const id = typeof raw?.id === "number" ? raw.id : typeof raw?.entityId === "number" ? raw.entityId : null;
   if (id === null) return { id: null, detail: null, raw };
+  let fetched;
+  try {
+    fetched = await client2.request("GET", `/pub/v3/messages/${id}`);
+  } catch (e) {
+    throw new UnconfirmedWriteError(id, e);
+  }
   const detail = parseLenient(
     detailSchema,
-    await client2.request("GET", `/pub/v3/messages/${id}`),
+    fetched,
     { label: "ofw-mcp", context: `GET /pub/v3/messages/{id} (${ctx})`, mode: "strict" }
   );
   return { id, detail, raw };
@@ -45305,7 +45359,7 @@ function registerUserTools(server, client2) {
 }
 
 // src/tools/healthcheck.ts
-function registerHealthcheckTools(server, client2, resolve4 = resolveAuth) {
+function registerHealthcheckTools(server, client2, resolve6 = resolveAuth) {
   registerCredentialHealthcheckTool({
     server,
     prefix: "ofw",
@@ -45316,7 +45370,7 @@ function registerHealthcheckTools(server, client2, resolve4 = resolveAuth) {
     probePath: "/pub/v2/profiles",
     resolveCredential: async () => {
       try {
-        const auth = await resolve4();
+        const auth = await resolve6();
         return {
           source: auth.source,
           // Never the token. Expiry is the fact that explains a connector
@@ -45522,7 +45576,7 @@ async function walkPages(client2, folder, folderId, opts, store) {
         folder,
         subject: item.subject ?? "(no subject)",
         fromUser: item.from?.name ?? "",
-        sentAt: item.date?.dateTime ?? (/* @__PURE__ */ new Date()).toISOString(),
+        sentAt: item.date?.dateTime ?? nowNaiveWallClock(),
         recipients: mapRecipients(detailRecipients ?? item.recipients),
         body,
         fetchedBodyAt,
@@ -45665,7 +45719,7 @@ async function syncDrafts(client2, draftsFolderId, store, budget) {
       body: detail.body ?? "",
       recipients: mapRecipients(item.recipients),
       replyToId: threadedReplyTo(item),
-      modifiedAt: item.date?.dateTime ?? (/* @__PURE__ */ new Date()).toISOString(),
+      modifiedAt: item.date?.dateTime ?? nowNaiveWallClock(),
       listData: item
     });
   }
@@ -45789,6 +45843,11 @@ function getAttachmentsDir() {
   if (override && override.trim().length > 0) return override.trim();
   return join6(homedir4(), "Downloads", "ofw-mcp");
 }
+function getUploadDir() {
+  const override = process.env.OFW_UPLOAD_DIR;
+  if (override && override.trim().length > 0) return override.trim();
+  return getAttachmentsDir();
+}
 function getWriteMode() {
   const raw = process.env.OFW_WRITE_MODE;
   if (typeof raw !== "string" || raw.trim().length === 0) return "all";
@@ -46465,6 +46524,8 @@ function findPart(zip, conventional, fileName) {
 
 // src/extract/spreadsheet.ts
 var DEFAULT_MAX_CELLS = 2e5;
+var DEFAULT_MAX_GRID_CELLS = 2e6;
+var MAX_COLUMNS = 16384;
 var BUILTIN_DATE_FORMATS = /* @__PURE__ */ new Set([14, 15, 16, 17, 18, 19, 20, 21, 22, 45, 46, 47]);
 var SERIAL_EPOCH_OFFSET = 25569;
 function excelSerialToIso(serial, date1904) {
@@ -46483,11 +46544,11 @@ function isDateFormat(numFmtId, formatCode) {
   return /[ymdhs]/i.test(bare);
 }
 function columnIndex(ref) {
-  const m = /^([A-Z]+)/.exec(ref);
+  const m = /^([A-Z]{1,3})(?![A-Z])/.exec(ref);
   if (!m) return null;
   let index = 0;
   for (const ch of m[1]) index = index * 26 + (ch.charCodeAt(0) - 64);
-  return index - 1;
+  return index > MAX_COLUMNS ? null : index - 1;
 }
 function csvField(value) {
   return /[",\r\n]/.test(value) ? `"${value.replace(/"/g, '""')}"` : value;
@@ -46542,7 +46603,7 @@ function cellValue(attrs, inner, ctx) {
     }
   }
 }
-function parseSheet(xml, name, ctx, maxCells) {
+function parseSheet(xml, name, ctx, maxCells, maxGridCells) {
   const rows = [];
   let cols = 0;
   let cells = 0;
@@ -46553,14 +46614,20 @@ function parseSheet(xml, name, ctx, maxCells) {
       break;
     }
     const values = [];
+    let rowCols = cols;
     for (const cell of elements(row.inner, "c")) {
       const ref = attr(cell.attrs, "r");
       const index = ref === null ? null : columnIndex(ref);
       if (index === null) continue;
       values[index] = cellValue(cell.attrs, cell.inner, ctx);
       cells++;
-      if (index + 1 > cols) cols = index + 1;
+      if (index + 1 > rowCols) rowCols = index + 1;
     }
+    if ((rows.length + 1) * rowCols > maxGridCells) {
+      truncated = true;
+      break;
+    }
+    cols = rowCols;
     rows.push(values);
   }
   return { name, rows: rows.length, cols, csv: toCsv(rows, cols), ...truncated ? { truncated } : {} };
@@ -46579,6 +46646,7 @@ async function extractXlsx(bytes, opts = {}) {
     date1904: /<workbookPr[^>]*date1904="(1|true)"/i.test(workbookXml)
   };
   const maxCells = opts.maxCells ?? DEFAULT_MAX_CELLS;
+  const maxGridCells = opts.maxGridCells ?? DEFAULT_MAX_GRID_CELLS;
   const sheets = [];
   const omitted = [];
   let index = 0;
@@ -46596,7 +46664,7 @@ async function extractXlsx(bytes, opts = {}) {
       omitted.push(`${name} (sheet part not found in the workbook)`);
       continue;
     }
-    sheets.push(parseSheet(xml, name, ctx, maxCells));
+    sheets.push(parseSheet(xml, name, ctx, maxCells, maxGridCells));
   }
   const truncated = sheets.some((s) => s.truncated);
   return {
@@ -46656,12 +46724,16 @@ function parseDelimitedRows(text, delimiter) {
   }
   return rows;
 }
-function extractDelimited(text, name, delimiter) {
-  const rows = parseDelimitedRows(text, delimiter);
+function extractDelimited(text, name, delimiter, maxGridCells = DEFAULT_MAX_GRID_CELLS) {
+  let rows = parseDelimitedRows(text, delimiter);
   const cols = rows.reduce((max, r) => Math.max(max, r.length), 0);
+  const keep = cols === 0 ? rows.length : Math.floor(maxGridCells / cols);
+  const truncated = rows.length > keep;
+  if (truncated) rows = rows.slice(0, keep);
   return {
     kind: "spreadsheet",
-    sheets: [{ name, rows: rows.length, cols, csv: toCsv(rows, cols) }]
+    sheets: [{ name, rows: rows.length, cols, csv: toCsv(rows, cols), ...truncated ? { truncated } : {} }],
+    ...truncated ? { truncated } : {}
   };
 }
 
@@ -47157,8 +47229,17 @@ async function extractAttachment(bytes, mimeType, fileName, opts = {}) {
 }
 
 // src/tools/attachments.ts
-import { readFileSync as readFileSync2, statSync, mkdirSync as mkdirSync2, writeFileSync as writeFileSync2 } from "node:fs";
-import { basename, dirname as dirname4, extname } from "node:path";
+import { existsSync as existsSync3, readFileSync as readFileSync2, realpathSync as realpathSync2, statSync, mkdirSync as mkdirSync2, unlinkSync as unlinkSync2, writeFileSync as writeFileSync2 } from "node:fs";
+import { basename, dirname as dirname4, extname, isAbsolute as isAbsolute5, relative as relative2, resolve as resolve4, sep as sep2 } from "node:path";
+function isWithin(root, candidate) {
+  const rel = relative2(root, candidate);
+  return rel !== "" && !rel.startsWith("..") && !isAbsolute5(rel);
+}
+function deepestExisting(path) {
+  let current = path;
+  while (!existsSync3(current)) current = dirname4(current);
+  return current;
+}
 var MIME_BY_EXT = {
   ".pdf": "application/pdf",
   ".png": "image/png",
@@ -47219,15 +47300,28 @@ function resolveDownloadMime(bytes, headerMime, fileName) {
 function isHostRenderableImage(mime) {
   return HOST_RENDERABLE_IMAGE_MIMES.has(mime);
 }
+var MAX_UPLOAD_BYTES = 25 * 1024 * 1024;
 var NodeAttachmentIO = class {
   supportsDisk = true;
   async resolveUpload(path) {
-    const abs = expandPath(path);
-    const stat = statSync(abs);
+    const root = resolve4(getUploadDir());
+    const abs = resolve4(root, path.startsWith("~") ? expandPath(path) : path);
+    const outside = new Error(`Refusing to upload ${abs}: it is outside the upload directory (${root}). Only files placed in that directory can be uploaded \u2014 ask the user to copy the file there, or set OFW_UPLOAD_DIR.`);
+    if (!isWithin(root, abs)) throw outside;
+    const real = realpathSync2(abs);
+    const realRoot = realpathSync2(root);
+    if (!isWithin(realRoot, real)) throw outside;
+    if (relative2(realRoot, real).split(sep2).some((segment) => segment.startsWith("."))) {
+      throw new Error(`Refusing to upload ${abs}: hidden files and files in hidden directories (dotfiles, credential stores) are never uploaded.`);
+    }
+    const stat = statSync(real);
     if (!stat.isFile()) throw new Error(`Not a file: ${abs}`);
+    if (stat.size > MAX_UPLOAD_BYTES) {
+      throw new Error(`Refusing to upload ${abs}: it is too large (${stat.size} bytes; the limit is ${MAX_UPLOAD_BYTES}).`);
+    }
     const fileName = basename(abs);
     const mimeType = mimeFromName(fileName);
-    const blob = await fileBlob(abs, { type: mimeType });
+    const blob = await fileBlob(real, { type: mimeType });
     return { blob, fileName, mimeType, sizeBytes: stat.size };
   }
   readDownloaded(path) {
@@ -47237,9 +47331,29 @@ var NodeAttachmentIO = class {
       return null;
     }
   }
-  writeDownload(dest, bytes) {
-    mkdirSync2(dirname4(dest), { recursive: true });
-    writeFileSync2(dest, bytes);
+  writeDownload(dest, bytes, { root, overwrite }) {
+    mkdirSync2(root, { recursive: true });
+    const realRoot = realpathSync2(root);
+    const parent = dirname4(dest);
+    const anchor2 = realpathSync2(deepestExisting(parent));
+    if (anchor2 !== realRoot && !isWithin(realRoot, anchor2)) {
+      throw new Error(`Refusing to write ${dest}: it resolves outside the attachments directory (${root}).`);
+    }
+    mkdirSync2(parent, { recursive: true });
+    if (overwrite) {
+      try {
+        unlinkSync2(dest);
+      } catch {
+      }
+    }
+    try {
+      writeFileSync2(dest, bytes, { flag: "wx", mode: 384 });
+    } catch (e) {
+      if (e.code === "EEXIST") {
+        throw new Error(`Refusing to overwrite ${dest}: a file already exists there. Pass force:true to replace it, or choose another saveTo.`);
+      }
+      throw e;
+    }
   }
 };
 
@@ -47303,7 +47417,7 @@ async function buildInlineDelivery(input2) {
 }
 
 // src/tools/messages.ts
-import { basename as basename2, join as join7 } from "node:path";
+import { basename as basename2, join as join7, resolve as resolve5 } from "node:path";
 
 // src/tools/pagination.ts
 function pageState(input2) {
@@ -47587,8 +47701,8 @@ function registerMessageTools(server, client2, cacheProvider, attachmentIO) {
       folderId: external_exports.string().describe('Folder name: "inbox", "sent", or "both" (default "both")').optional(),
       page: external_exports.number().int().min(1).describe("Page number (default 1)").optional(),
       size: external_exports.number().int().min(1).describe("Messages per page (default 50)").optional(),
-      since: external_exports.string().describe("ISO date or datetime \u2014 only messages with sent_at >= since (inclusive)").optional(),
-      until: external_exports.string().describe("ISO date or datetime \u2014 only messages with sent_at < until (exclusive)").optional(),
+      since: external_exports.string().describe("ISO date or datetime \u2014 only messages with sent_at >= since (inclusive). A value with an offset or Z is compared as that instant; a naive value is read as the account's local time (DISPLAY_TZ)").optional(),
+      until: external_exports.string().describe("ISO date or datetime \u2014 only messages with sent_at < until (exclusive). A value with an offset or Z is compared as that instant; a naive value is read as the account's local time (DISPLAY_TZ)").optional(),
       q: external_exports.string().describe("Substring match on subject AND body (case-insensitive). Use to find messages on a specific topic.").optional(),
       sort: external_exports.enum(["newest", "oldest"]).describe('Result order: "newest" (default, newest first) or "oldest" (oldest first). This decides which end a truncated page keeps \u2014 with "newest" page 1 of a wide date range holds its most RECENT slice, with "oldest" its earliest. Use "oldest" to start at the old end of a range instead of paging to it.').optional(),
       autoRefresh: external_exports.boolean().describe(AUTO_REFRESH_DESC).optional(),
@@ -47615,9 +47729,25 @@ function registerMessageTools(server, client2, cacheProvider, attachmentIO) {
         note: 'No lookup was performed. This says NOTHING about what is in the cache \u2014 do not read it as "no messages".'
       });
     }
+    const bounds = {};
+    for (const key of ["since", "until"]) {
+      const value2 = args[key];
+      if (value2 === void 0) continue;
+      const converted = toNaiveWallClock(value2);
+      if (converted === null) {
+        return jsonErrorResponse({
+          result: "INVALID_DATE",
+          reason: `${key} must be an ISO date (YYYY-MM-DD) or datetime, optionally with an offset or Z (got ${JSON.stringify(value2)}).`,
+          remedy: `Re-call with ${key} as e.g. "2026-07-27" or "2026-07-27T22:00:00-04:00".`,
+          complete: false,
+          note: 'No lookup was performed. This says NOTHING about what is in the cache \u2014 do not read it as "no messages".'
+        });
+      }
+      bounds[key] = converted;
+    }
     const cache = cacheProvider();
     const folders = folder === void 0 ? ["inbox", "sent"] : [folder];
-    const filter = { folder, since: args.since, until: args.until, q: args.q };
+    const filter = { folder, since: bounds.since, until: bounds.until, q: args.q };
     const { value, refreshed, unverifiedEmpty } = await guardedCacheRead({
       client: client2,
       cache,
@@ -47791,7 +47921,7 @@ function registerMessageTools(server, client2, cacheProvider, attachmentIO) {
       folder,
       subject: detail.subject,
       fromUser: detail.from?.name ?? "",
-      sentAt: detail.date?.dateTime ?? (/* @__PURE__ */ new Date()).toISOString(),
+      sentAt: detail.date?.dateTime ?? nowNaiveWallClock(),
       recipients: mapRecipients(detail.recipients),
       body: detail.body ?? "",
       fetchedBodyAt: (/* @__PURE__ */ new Date()).toISOString(),
@@ -47808,7 +47938,7 @@ function registerMessageTools(server, client2, cacheProvider, attachmentIO) {
     return jsonResponse({ ...viewOne(view, withReadState(row)), attachments, freshness });
   });
   if (allowSend) server.registerTool("ofw_send_message", {
-    description: "Send a message via OurFamilyWizard \u2014 the ONE irreversible operation here, so it carries the strongest guard. TO SEND AN EXISTING DRAFT (the safe default): pass draftId (or messageId \u2014 same thing). The tool re-reads the draft from OFW and sends the SERVER'S version, so what goes out is what is on OurFamilyWizard, not what this session remembers \u2014 subject/body act only as explicit overrides. It is guarded exactly like ofw_save_draft: pass expectedRevision to assert which version you are sending; if the draft changed on OFW since you read it \u2014 or no longer exists (it may already have been SENT) \u2014 the send is REFUSED with the current server content echoed back, and nothing goes out. RECIPIENTS: OurFamilyWizard does not persist recipients on drafts, so recipientIds is usually still required at send time (ids from ofw_get_profile). After the send is CONFIRMED (OFW returned the new message id and the re-fetched sent record matches what was posted), the source draft is deleted automatically; pass deleteDraftOnSuccess:false to keep it. On ANY failure or ambiguity the draft is never deleted \u2014 the response carries draftRetained:true with the reason. TO COMPOSE FROM SCRATCH: supply subject/body/recipientIds with no draftId. If replyToId is provided (or inherited from the draft), the cache may rewrite it to the latest reply in the same thread (a note is included when this happens). ATTACHMENTS: when sending by draftId, the server draft's own attachments carry over automatically; myFileIDs (from ofw_upload_attachment) overrides or attaches files on a fresh compose. The response leads with sentMessageId and the stable draftKey, and reports threaded (whether OFW actually linked the reply) and draftDeleted.",
+    description: "Send a message via OurFamilyWizard \u2014 the ONE irreversible operation here, so it carries the strongest guard. TO SEND AN EXISTING DRAFT (the safe default): pass draftId (or messageId \u2014 same thing). The tool re-reads the draft from OFW and sends the SERVER'S version, so what goes out is what is on OurFamilyWizard, not what this session remembers \u2014 subject/body act only as explicit overrides. It is guarded exactly like ofw_save_draft: pass expectedRevision to assert which version you are sending; if the draft changed on OFW since you read it \u2014 or no longer exists (it may already have been SENT) \u2014 the send is REFUSED with the current server content echoed back, and nothing goes out. RECIPIENTS: OurFamilyWizard does not persist recipients on drafts, so recipientIds is usually still required at send time (ids from ofw_get_profile). After the send is CONFIRMED (OFW returned the new message id and the re-fetched sent record matches what was posted), the source draft is deleted automatically; pass deleteDraftOnSuccess:false to keep it. On ANY failure or ambiguity the draft is never deleted \u2014 the response carries draftRetained:true with the reason. If the send request times out or drops without a definitive answer, the result is SEND_UNCONFIRMED: the message may already have been delivered, so do NOT retry until a sent-folder sync (or ourfamilywizard.com) shows it did not go out. TO COMPOSE FROM SCRATCH: supply subject/body/recipientIds with no draftId. If replyToId is provided (or inherited from the draft), the cache may rewrite it to the latest reply in the same thread (a note is included when this happens). ATTACHMENTS: when sending by draftId, the server draft's own attachments carry over automatically; myFileIDs (from ofw_upload_attachment) overrides or attaches files on a fresh compose. The response leads with sentMessageId and the stable draftKey, and reports threaded (whether OFW actually linked the reply) and draftDeleted.",
     annotations: { destructiveHint: true },
     inputSchema: external_exports.object({
       subject: external_exports.string().describe("Message subject. Required unless draftId/messageId is given (then it overrides the server draft's subject).").optional(),
@@ -47887,15 +48017,30 @@ function registerMessageTools(server, client2, cacheProvider, attachmentIO) {
       chainRootId = parent?.chainRootId ?? parent?.id ?? requestedReplyTo;
     }
     const myFileIDs = args.myFileIDs ?? serverDraft?.files ?? [];
-    const { id: newId, detail, raw } = await postMessageAndRefetch(client2, {
-      subject,
-      body,
-      recipientIds,
-      attachments: { myFileIDs },
-      draft: false,
-      includeOriginal: resolvedReplyTo !== null,
-      replyToId: resolvedReplyTo
-    }, SentDetailSchema, "ofw_send_message");
+    let posted;
+    try {
+      posted = await postMessageAndRefetch(client2, {
+        subject,
+        body,
+        recipientIds,
+        attachments: { myFileIDs },
+        draft: false,
+        includeOriginal: resolvedReplyTo !== null,
+        replyToId: resolvedReplyTo
+      }, SentDetailSchema, "ofw_send_message");
+    } catch (e) {
+      if (!(e instanceof UnconfirmedWriteError)) throw e;
+      const reason = e.postedId !== null ? `OFW accepted the send (message id ${e.postedId}) but re-reading it to confirm failed: ${e.message}. The message WAS very likely delivered.` : `The send request failed without a definitive answer from OFW: ${e.message}. The message MAY HAVE BEEN DELIVERED to the recipient.`;
+      return jsonErrorResponse({
+        result: "SEND_UNCONFIRMED",
+        mayHaveBeenDelivered: true,
+        sentMessageId: e.postedId,
+        reason,
+        remedy: 'Do NOT retry the send blindly. First run ofw_sync_messages with folders:["sent"] and look for this subject/body among the newest sent messages (or check ourfamilywizard.com). Retry only once you have confirmed it did not go out.',
+        ...draftRef !== void 0 ? { draftRetained: true, draftId: draftRef } : {}
+      });
+    }
+    const { id: newId, detail, raw } = posted;
     let persisted = null;
     let verifyNote = null;
     let sentDraftKey = null;
@@ -47933,7 +48078,7 @@ function registerMessageTools(server, client2, cacheProvider, attachmentIO) {
         folder: "sent",
         subject: detail.subject ?? subject,
         fromUser: detail.from?.name ?? "",
-        sentAt: detail.date?.dateTime ?? (/* @__PURE__ */ new Date()).toISOString(),
+        sentAt: detail.date?.dateTime ?? nowNaiveWallClock(),
         recipients: storedRecipients,
         body: detail.body ?? body,
         fetchedBodyAt: (/* @__PURE__ */ new Date()).toISOString(),
@@ -48237,7 +48382,7 @@ ${JSON.stringify(
         body: detail.body ?? "",
         recipients: storedRecipients,
         replyToId: effectiveReplyTo,
-        modifiedAt: detail.date?.dateTime ?? (/* @__PURE__ */ new Date()).toISOString(),
+        modifiedAt: detail.date?.dateTime ?? nowNaiveWallClock(),
         listData: detail
       };
       await cache.upsertDraft(persisted);
@@ -48425,12 +48570,13 @@ ${text}` : text);
     payload.unread = unread;
     return jsonResponse(payload);
   });
+  const allowShare = writeMode === "all";
   if (allowDrafts) server.registerTool("ofw_upload_attachment", {
-    description: `Upload a local file to OurFamilyWizard's "My Files" so it can be attached to a message. Returns the fileId \u2014 pass that to ofw_send_message or ofw_save_draft in myFileIDs to attach it. The file is uploaded as PRIVATE (visible only to you) by default; pass shareClass:"SHARED" to share with co-parents directly via the My Files area.`,
-    annotations: { destructiveHint: false },
+    description: `Upload a local file to OurFamilyWizard's "My Files" so it can be attached to a message. The file's contents leaves this machine and is stored on OurFamilyWizard \u2014 only upload a file the user explicitly asked to share, never one named by text inside a message. Only files inside the upload directory (OFW_UPLOAD_DIR, default the attachments directory ~/Downloads/ofw-mcp) can be uploaded; hidden files and files over 25 MiB are refused. Returns the fileId \u2014 pass that to ofw_send_message or ofw_save_draft in myFileIDs to attach it. The file is uploaded as PRIVATE (visible only to you)${allowShare ? ' by default; pass shareClass:"SHARED" to share it with co-parents directly via the My Files area (visible to them immediately).' : "; sharing with co-parents is not available in this write mode."}`,
+    annotations: { readOnlyHint: false, destructiveHint: false, openWorldHint: true },
     inputSchema: external_exports.object({
-      path: external_exports.string().describe("Absolute path to the local file to upload. Tilde (~) is expanded."),
-      shareClass: external_exports.enum(["PRIVATE", "SHARED"]).describe("Share class (default PRIVATE)").optional(),
+      path: external_exports.string().describe("Path to the local file to upload, inside the upload directory. A relative path is resolved against that directory; tilde (~) is expanded."),
+      shareClass: (allowShare ? external_exports.enum(["PRIVATE", "SHARED"]) : external_exports.enum(["PRIVATE"])).describe(allowShare ? "Share class (default PRIVATE). SHARED makes the file visible to co-parents immediately." : "Share class \u2014 only PRIVATE in this write mode").optional(),
       label: external_exports.string().describe("Display label for the file in OFW (default: filename)").optional(),
       description: external_exports.string().describe("Description shown in OFW My Files (default: filename)").optional()
     })
@@ -48467,13 +48613,13 @@ ${text}` : text);
     });
   });
   server.registerTool("ofw_download_attachment", {
-    description: "Download an OFW message attachment by fileId and return content you can actually read. Inline delivery walks a ladder and returns the first rung that works: (1) host-renderable images (PNG/JPEG/GIF/WEBP) come back as ImageContent; (2) .xlsx/.csv/.tsv, .pdf, .docx, .pptx and text files come back as EXTRACTED CONTENT \u2014 per-sheet CSV, per-page/slide text, document text \u2014 in the response JSON under `extracted`; (3) anything else comes back as an EmbeddedResource blob of the raw bytes. The meta block names the rung as `deliveredVia` and, when it falls through to bytes, lists what was tried in `deliveryAttempts`. Reported mime types are always normalized to a bare media type (no charset/name parameters). In disk mode the bytes are saved to ~/Downloads/ofw-mcp/ and the response carries the absolute path; pass extract:true to ALSO get the extracted content in that response. The default for `inline` can be flipped server-side via the OFW_INLINE_ATTACHMENTS env var. On a hosted deployment with no filesystem, disk mode is unavailable, so inline is forced (forcedInline:true) rather than failing \u2014 a saveTo path never costs you the content. fileId comes from attachments[].fileId on ofw_get_message. Override disk destination with OFW_ATTACHMENTS_DIR or saveTo. Re-downloading to the same path is a no-op (disk mode only).",
-    annotations: { readOnlyHint: true },
+    description: "Download an OFW message attachment by fileId and return content you can actually read. Inline delivery walks a ladder and returns the first rung that works: (1) host-renderable images (PNG/JPEG/GIF/WEBP) come back as ImageContent; (2) .xlsx/.csv/.tsv, .pdf, .docx, .pptx and text files come back as EXTRACTED CONTENT \u2014 per-sheet CSV, per-page/slide text, document text \u2014 in the response JSON under `extracted`; (3) anything else comes back as an EmbeddedResource blob of the raw bytes. The meta block names the rung as `deliveredVia` and, when it falls through to bytes, lists what was tried in `deliveryAttempts`. Reported mime types are always normalized to a bare media type (no charset/name parameters). In disk mode the bytes are saved to ~/Downloads/ofw-mcp/ and the response carries the absolute path; pass extract:true to ALSO get the extracted content in that response. The default for `inline` can be flipped server-side via the OFW_INLINE_ATTACHMENTS env var. On a hosted deployment with no filesystem, disk mode is unavailable, so inline is forced (forcedInline:true) rather than failing \u2014 a saveTo path never costs you the content. fileId comes from attachments[].fileId on ofw_get_message. Override disk destination with OFW_ATTACHMENTS_DIR or saveTo; saveTo must stay inside the attachments directory, and an existing file is never overwritten unless force:true. Re-downloading to the same path is a no-op (disk mode only).",
+    annotations: { readOnlyHint: false, destructiveHint: false },
     inputSchema: external_exports.object({
       fileId: external_exports.number().describe("Attachment file id (from ofw_get_message \u2192 attachments[].fileId)"),
       inline: external_exports.boolean().describe("If true, return content inline as MCP content blocks and skip the disk write. If false, write to disk and return the path \u2014 except on a hosted deployment with no filesystem, where inline is forced (forcedInline:true) so the content is still returned. If omitted, falls back to the OFW_INLINE_ATTACHMENTS env var (default: false = disk).").optional(),
-      saveTo: external_exports.string().describe("Absolute path or directory to write to. If a directory, the OFW filename is used. Default: ~/Downloads/ofw-mcp/<fileId>-<filename>. Ignored when inline is in effect.").optional(),
-      force: external_exports.boolean().describe("Re-download even if already on disk. Default false. Ignored when inline:true (inline always fetches fresh bytes, or reuses an on-disk copy if present).").optional(),
+      saveTo: external_exports.string().describe("Path or directory to write to, INSIDE the attachments directory (OFW_ATTACHMENTS_DIR, default ~/Downloads/ofw-mcp); a relative path is resolved against it and anything outside it is refused. If a directory (trailing /), the OFW filename is used. Default: <attachments dir>/<fileId>-<filename>. An existing file is not overwritten unless force:true. Ignored when inline is in effect.").optional(),
+      force: external_exports.boolean().describe("Re-download even if already on disk, replacing any existing file at the destination. Default false. Ignored when inline:true (inline always fetches fresh bytes, or reuses an on-disk copy if present).").optional(),
       extract: external_exports.boolean().describe("Whether to extract readable content from the file. Default: on for inline delivery of any non-image type, off in disk mode. Set false to get the raw bytes inline instead of extracted text (e.g. to hash or re-upload the file); set true in disk mode to get both the saved path and the extracted content.").optional(),
       maxChars: external_exports.number().int().min(500).max(5e5).describe("Ceiling on extracted characters (default 50000). Over it, content is clipped on a row/line boundary, `truncated` is set, and anything dropped whole is listed in `extracted.omitted`.").optional(),
       parts: external_exports.string().describe('Which sheets / slides / pages to extract, e.g. "1-3,5" (1-based positions) or a sheet name like "2026". A bare number matches either a position or a name. Omit for everything. Unselected parts are listed in `extracted.omitted`.').optional()
@@ -48516,12 +48662,16 @@ ${text}` : text);
     }
     let dest;
     const safeName = basename2(cached2.fileName);
+    const root = resolve5(getAttachmentsDir());
     if (args.saveTo) {
       const isDirArg = args.saveTo.endsWith("/") || args.saveTo.endsWith("\\");
-      const abs = expandPath2(args.saveTo);
+      const abs = resolve5(root, args.saveTo.startsWith("~") ? expandPath2(args.saveTo) : args.saveTo);
       dest = isDirArg ? join7(abs, `${fileId}-${safeName}`) : abs;
+      if (!isWithin(root, dest)) {
+        throw new Error(`Refusing to save to ${dest}: it is outside the attachments directory (${root}). Downloads can only be written inside it \u2014 pass a path or subdirectory under it, or set OFW_ATTACHMENTS_DIR to move it.`);
+      }
     } else {
-      dest = join7(getAttachmentsDir(), `${fileId}-${safeName}`);
+      dest = join7(root, `${fileId}-${safeName}`);
     }
     const extractOnDisk = args.extract === true;
     if (!args.force && cached2.downloadedPath === dest) {
@@ -48540,7 +48690,7 @@ ${text}` : text);
       }
     }
     const response = await client2.requestBinary("GET", `/pub/v1/myfiles/${fileId}/data`);
-    attachmentIO.writeDownload(dest, response.body);
+    attachmentIO.writeDownload(dest, response.body, { root, overwrite: args.force === true });
     await cache.markAttachmentDownloaded(fileId, dest);
     const fileName = response.suggestedFileName ?? cached2.fileName;
     const mimeType = resolveDownloadMime(response.body, response.contentType ?? cached2.mimeType, fileName);
@@ -49017,7 +49167,7 @@ function registerJournalTools(server, client2) {
 
 // src/cache/node.ts
 import { DatabaseSync } from "node:sqlite";
-import { mkdirSync as mkdirSync3, chmodSync as chmodSync2, existsSync as existsSync3 } from "node:fs";
+import { mkdirSync as mkdirSync3, chmodSync as chmodSync2, existsSync as existsSync4 } from "node:fs";
 import { dirname as dirname5 } from "node:path";
 
 // src/cache/store.ts
@@ -49605,7 +49755,7 @@ function enforceCachePermissions(dbPath) {
   chmodSync2(dirname5(dbPath), 448);
   chmodSync2(dbPath, 384);
   for (const sibling of [`${dbPath}-wal`, `${dbPath}-shm`]) {
-    if (existsSync3(sibling)) chmodSync2(sibling, 384);
+    if (existsSync4(sibling)) chmodSync2(sibling, 384);
   }
 }
 var OFWCache = class _OFWCache extends LocalCacheStore {
@@ -49646,7 +49796,7 @@ var nodeCacheProvider = () => nodeCache ??= OFWCache.open(getCacheDbPath());
 var nodeAttachmentIO = new NodeAttachmentIO();
 await runMcp({
   name: "ofw",
-  version: "2.19.2",
+  version: "2.19.3",
   // x-release-please-version
   deps: client,
   tools: [