ofw-mcp 2.13.0 → 2.15.0

package/.claude-plugin/marketplace.json

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

package/.claude-plugin/plugin.json

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

package/dist/auth.js

@@ -69,6 +69,38 @@ function fetchproxyDisabled() {
  * return value as opaque credentials — they should not branch on `source`.
  * The field exists for logging / future cache-keying only.
  */
+/**
+ * The message raised when NEITHER auth path is configured.
+ *
+ * Exported with {@link isNoAuthConfigured} because a second reader needs to
+ * tell this case apart from every other auth failure: "nothing is set up" and
+ * "the password was rejected" or "the bridge is down" need opposite advice, and
+ * `ofw_healthcheck` gives the wrong one if it confuses them. A prefix match on
+ * a copy of this string in the other module would pass its own test while
+ * silently stopping matching the day this wording changed.
+ */
+export const NO_AUTH_CONFIGURED = 'OFW auth: set OFW_USERNAME + OFW_PASSWORD, ' +
+    'or install the fetchproxy extension and sign into ourfamilywizard.com ' +
+    '(unset OFW_DISABLE_FETCHPROXY if it is set).';
+/**
+ * Prefix of the message raised when the fetchproxy bridge is unreachable.
+ *
+ * A PREFIX rather than a whole constant because the upstream `.hint` — the
+ * actionable "click the toolbar icon" copy — is appended per failure. Exported
+ * with {@link isBridgeDown} for the reason {@link NO_AUTH_CONFIGURED} is: a
+ * reader that matched a copy of this text would keep its own test green on the
+ * day the wording changed, while silently misreporting a downed bridge as an
+ * unconfigured server.
+ */
+export const BRIDGE_DOWN_PREFIX = 'OFW auth: fetchproxy bridge is down';
+/** True for the {@link BRIDGE_DOWN_PREFIX} failure and nothing else. */
+export function isBridgeDown(e) {
+    return e instanceof Error && e.message.startsWith(BRIDGE_DOWN_PREFIX);
+}
+/** True for the {@link NO_AUTH_CONFIGURED} failure and nothing else. */
+export function isNoAuthConfigured(e) {
+    return e instanceof Error && e.message === NO_AUTH_CONFIGURED;
+}
 export async function resolveAuth() {
     // Which paths are CONFIGURED. `resolveAuthPattern` runs the first one
     // provided, in the fleet's fixed priority order (token → oauth →
@@ -125,7 +157,7 @@ export async function resolveAuth() {
                 // FetchproxyBridgeDownError only escapes bootstrap() after the lazy-revive retry fails — surface .hint verbatim (actionable "click toolbar icon" copy).
                 if (classifyBridgeError(e) === 'bridge_down') {
                     const downErr = e;
-                    throw new Error(`OFW auth: fetchproxy bridge is down (extension service worker unreachable after retry). ${downErr.hint}`);
+                    throw new Error(`${BRIDGE_DOWN_PREFIX} (extension service worker unreachable after retry). ${downErr.hint}`);
                 }
                 const msg = e instanceof Error ? e.message : String(e);
                 throw new Error(`OFW auth: no OFW_USERNAME/OFW_PASSWORD set, and fetchproxy fallback failed: ${msg}`);
@@ -137,9 +169,7 @@ export async function resolveAuth() {
     // because this one names OFW's own two fixes side-by-side and the generic
     // one cannot.
     if (!pattern.sessionScrape && !pattern.fetchproxy) {
-        throw new Error('OFW auth: set OFW_USERNAME + OFW_PASSWORD, ' +
-            'or install the fetchproxy extension and sign into ourfamilywizard.com ' +
-            '(unset OFW_DISABLE_FETCHPROXY if it is set).');
+        throw new Error(NO_AUTH_CONFIGURED);
     }
     // Errors from the winning path propagate UNWRAPPED, which is what keeps the
     // bridge-down `.hint` above intact.