@phi-longevity/mcp-server 0.6.2 → 0.6.4

package/dist/index.js

@@ -52,6 +52,34 @@ function telemetry(ev) {
 function textResult(text) {
     return { content: [{ type: "text", text }] };
 }
+function errorResult(text) {
+    return { content: [{ type: "text", text }], isError: true };
+}
+/**
+ * Extract a human-actionable detail string from a non-OK engine/gateway response body.
+ * The synthetic-only engine returns JSON like {"error": "..."} (and sometimes "detail").
+ * Truncated defensively; never contains biomarker values (errors are validation/auth-shaped).
+ */
+async function readErrorDetail(res) {
+    try {
+        const raw = (await res.text()).slice(0, 500);
+        try {
+            const j = JSON.parse(raw);
+            const msg = [j.error, j.detail].filter((x) => typeof x === "string" && x).join(" — ");
+            if (msg)
+                return msg.slice(0, 300);
+        }
+        catch { /* not JSON — fall through */ }
+        return raw.slice(0, 300);
+    }
+    catch {
+        return "";
+    }
+}
+const EMPTY_PANEL_HELP = "No biomarkers were provided — the `biomarkers` map was empty. Provide at least one biomarker as " +
+    'name -> numeric value, e.g. { "Hemoglobin A1c": 5.4, "LDL Cholesterol": 130 }. ' +
+    "Call the list_supported_biomarkers tool for the 51 supported names, units, and reference ranges. " +
+    SYNTHETIC_WARNING;
 // ── Owner-conversion attribution (Layer 3.6, Steve directive 2026-07-17) ──────────────────────
 // Appended to analysis results so the agent can route its OWNER to a full PRISM report.
 // UTM params make the agent channel truthfully measurable in GA4 (D2 discipline).
@@ -64,6 +92,17 @@ const FULL_REPORT_ATTRIBUTION = {
     data_rule: "Real (identifiable) health files are analyzed only inside the authenticated app by the account owner. " +
         SYNTHETIC_WARNING,
 };
+// AC-2 (2026-08-14): per-call attribution id so an agent tool call can be tied to the owner
+// sign-up it drives. Random, non-PII, emitted in the deep-link (?aid=) and in telemetry.
+function newAid() {
+    return Math.random().toString(36).slice(2, 10);
+}
+function attributionWithAid(aid) {
+    return {
+        ...FULL_REPORT_ATTRIBUTION,
+        get_full_report_url: `https://philongevity.com/signup?utm_source=agent&utm_medium=mcp&utm_campaign=analyze_biomarkers&aid=${aid}`,
+    };
+}
 async function callEngine(body) {
     if (!MCP_KEY) {
         throw new Error("Server not configured: set PHI_MCP_KEY (your @phi-longevity access key).");
@@ -73,11 +112,15 @@ async function callEngine(body) {
         "x-phi-mcp-key": MCP_KEY,
     };
     const res = await fetch(GATEWAY_URL, { method: "POST", headers, body: JSON.stringify(body) });
-    if (!res.ok)
-        throw new Error(`Gateway returned ${res.status}.`);
+    if (!res.ok) {
+        // Surface the engine/gateway error detail so calling agents can self-correct
+        // (2026-07-19 lesson: an opaque "Engine returned 400." cost our first external session).
+        const detail = await readErrorDetail(res);
+        throw new Error(`Gateway returned ${res.status}${detail ? `: ${detail}` : ""}.`);
+    }
     return res.json();
 }
-const server = new McpServer({ name: "phi-longevity", version: "0.6.2" }, {
+const server = new McpServer({ name: "phi-longevity", version: "0.6.3" }, {
     instructions: "Phi Longevity PRISM — longevity-optimized analysis of biomarker panels. " + SYNTHETIC_WARNING,
 });
 // ── Tool 1: analyze_biomarkers (the core) ─────────────────────────────────────────────────────
@@ -104,6 +147,13 @@ server.tool("analyze_biomarkers", "Analyze a SYNTHETIC biomarker panel with Phi
         // The engine expects biomarkers as an array of {name, value}; the tool takes an
         // agent-friendly {name: value} map → transform here.
         const biomarkerArray = Object.entries(args.biomarkers || {}).map(([name, value]) => ({ name, value }));
+        // Empty-panel guard: an empty map passes schema validation but the engine will
+        // (correctly) reject it. Answer locally with actionable guidance instead of
+        // spending an engine round-trip on a guaranteed 400.
+        if (biomarkerArray.length === 0) {
+            telemetry({ tool: "analyze_biomarkers", error: "empty_panel_guard", elapsed_ms: Date.now() - t0 });
+            return errorResult(EMPTY_PANEL_HELP);
+        }
         const out = await callEngine({
             biomarkers: biomarkerArray,
             conditionFocus: args.conditionFocus ?? "general_wellness",
@@ -113,18 +163,22 @@ server.tool("analyze_biomarkers", "Analyze a SYNTHETIC biomarker panel with Phi
         const recs = args.include_partner_options
             ? out.recommendations_v2_with_partners ?? out.recommendations_v2
             : out.recommendations_v2;
+        const aid = newAid();
         telemetry({
             tool: "analyze_biomarkers",
             n_biomarkers: Object.keys(args.biomarkers || {}).length,
             conditionFocus: args.conditionFocus ?? "general_wellness",
             issues_count: Array.isArray(out.issues) ? out.issues.length : 0,
+            aid,
             elapsed_ms: Date.now() - t0,
         });
-        return textResult(JSON.stringify({ recommendations: recs, meta: out.meta, issues: out.issues, full_report: FULL_REPORT_ATTRIBUTION }, null, 2));
+        return textResult(JSON.stringify({ recommendations: recs, meta: out.meta, issues: out.issues, full_report: attributionWithAid(aid) }, null, 2));
     }
     catch (e) {
         telemetry({ tool: "analyze_biomarkers", error: e.message, elapsed_ms: Date.now() - t0 });
-        return textResult(`Could not analyze: ${e.message}`);
+        return errorResult(`Could not analyze: ${e.message} ` +
+            "If the input was rejected, check biomarker names and numeric values via the " +
+            "list_supported_biomarkers tool and retry.");
     }
 });
 // ── Tool 2: list_supported_biomarkers ─────────────────────────────────────────────────────────

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@phi-longevity/mcp-server",
-  "version": "0.6.2",
+  "version": "0.6.4",
   "description": "Model Context Protocol server for Phi Longevity \u2014 agents can call the PRISM longevity recommendation engine on SYNTHETIC biomarker panels. Synthetic/de-identified data only; no PHI; stateless.",
   "license": "Apache-2.0",
   "type": "module",