formlab-mcp 0.6.29 → 0.6.31

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "formlab-mcp",
-  "version": "0.6.29",
+  "version": "0.6.31",
   "mcpName": "io.github.juliu1980/formlab-mcp",
   "description": "Read-only Model Context Protocol server for FormLab \u2014 lets Claude (and other MCP clients) read and analyze your FormLab data, from a local export file OR your live cloud workspace.",
   "type": "module",

package/server.json

@@ -2,7 +2,7 @@
   "$schema": "https://static.modelcontextprotocol.io/schemas/2025-12-11/server.schema.json",
   "name": "io.github.juliu1980/formlab-mcp",
   "description": "Read-only MCP for FormLab — let Claude query your formulation lab. Free reads a local export; Pro connects to your live cloud workspace with a dedicated read-only token.",
-  "version": "0.6.29",
+  "version": "0.6.31",
   "websiteUrl": "https://formvix.com/mcp",
   "repository": {
     "url": "https://github.com/juliu1980/FormLab",
@@ -13,7 +13,7 @@
     {
       "registryType": "npm",
       "identifier": "formlab-mcp",
-      "version": "0.6.29",
+      "version": "0.6.31",
       "runtimeHint": "npx",
       "transport": {
         "type": "stdio"

package/tools/analytics.js

@@ -1002,11 +1002,12 @@ function _shelfLifeAssess(sl, targetMonths, now) {
   if (targetMonths) {
     const verdict = pointTotal < targetMonths ? 'short' : (ciTotal != null && ciTotal < targetMonths) ? 'marginal' : 'meets';
     if (verdict === 'meets' && sl.remainingMonths > 60) return null;
-    const level = (overdue && verdict !== 'meets') ? 'high' : verdict === 'short' ? 'high' : verdict === 'marginal' ? 'medium' : 'info';
+    // Mirrors the app: a projection never reaches 'high' (reserved for a measured out-of-spec value).
+    const level = (overdue && verdict !== 'meets') || verdict === 'short' ? 'medium' : 'info';
     return { verdict, level, pointTotal, ciTotal, overdue, crossDate };
   }
   if (sl.remainingMonths > 60) return null;
-  return { verdict: null, level: (overdue || sl.remainingMonths < 3) ? 'high' : sl.remainingMonths < 12 ? 'medium' : 'info', pointTotal, ciTotal, overdue, crossDate };
+  return { verdict: null, level: (overdue || sl.remainingMonths < 3) ? 'medium' : 'info', pointTotal, ciTotal, overdue, crossDate };
 }
 
 const get_stability = {
@@ -1039,6 +1040,10 @@ const get_stability = {
     if (!paramLc) return { error: 'parameter is required.' };
     const points = [];
     let specMin = null, specMax = null, unit = '';
+    // Each reading keeps the spec it was judged by (reading → panel row →
+    // method). Mirrors the app: one band only when they all agree; 2+ distinct
+    // specs (two panels) → no band + `specVaries`, never an intersection.
+    const specByKey = new Map();
     (db.testResults || []).filter(t => sampleIds.has(t.sampleId)).forEach(t => {
       const dStr = t.testDate || (t.createdAt ? String(t.createdAt).slice(0, 10) : null);
       if (!dStr) return;
@@ -1052,15 +1057,25 @@ const get_stability = {
         }
         if (!Number.isFinite(value)) return;
         if (m.unit && !unit) unit = m.unit;
-        const ac = _resolveSpec(m, t.templateId).spec;
-        if (ac) { if (Number.isFinite(ac.min)) specMin = specMin == null ? ac.min : Math.max(specMin, ac.min); if (Number.isFinite(ac.max)) specMax = specMax == null ? ac.max : Math.min(specMax, ac.max); }
+        const rs = _resolveSpec(m, t.templateId), ac = rs.spec;
+        const sp = (ac && (Number.isFinite(ac.min) || Number.isFinite(ac.max))) ? { min: Number.isFinite(ac.min) ? ac.min : null, max: Number.isFinite(ac.max) ? ac.max : null } : null;
+        if (sp) {
+          const k = `${sp.min}|${sp.max}`;
+          if (!specByKey.has(k)) specByKey.set(k, { ...sp, sources: new Set(), readings: 0 });
+          const e = specByKey.get(k); e.readings++;
+          const tmpl = rs.source === 'template' ? (db.templates || []).find(x => x.id === t.templateId) : null;
+          e.sources.add(tmpl ? (tmpl.name || tmpl.uid || 'panel') : rs.source === 'parameter' ? 'Test Method default' : 'set on the reading');
+        }
         const cond = (m.conditions && Object.keys(m.conditions).length)
           ? Object.entries(m.conditions).map(([k, v]) => `${k} ${v}`).join(' · ') : '';
-        points.push({ date, value, cond });
+        points.push({ date, value, cond, spec: sp });
       });
     });
+    const specList = [...specByKey.values()].map(e => ({ min: e.min, max: e.max, readings: e.readings, sources: [...e.sources] }));
+    if (specList.length === 1) { specMin = specList[0].min; specMax = specList[0].max; }
+    const specVaries = specList.length > 1 ? specList : null;
     // Spec fallback to the Test Method library entry when no measurement carried one.
-    if (specMin == null && specMax == null) {
+    if (specMin == null && specMax == null && !specVaries) {
       const pdef = (db.parameters || []).find(p => (p.name || '').trim().toLowerCase() === paramLc);
       if (pdef && pdef.acceptanceCriteria) { specMin = pdef.acceptanceCriteria.min ?? null; specMax = pdef.acceptanceCriteria.max ?? null; }
     }
@@ -1098,6 +1113,10 @@ const get_stability = {
       const ic = _iChart(ys);
       if (ic) out.controlChart = { mean: _round(ic.mean), ucl: _round(ic.ucl), lcl: _round(ic.lcl), sigma: _round(ic.sigma), outOfControlPoints: ic.ooc };
       if (spec.min != null || spec.max != null) out.spec = { ...spec, latestInSpec: (spec.min == null || ys[n - 1] >= spec.min) && (spec.max == null || ys[n - 1] <= spec.max) };
+      else if (specVaries) {
+        const ls = [...pts].reverse().find(p => p.spec);
+        if (ls) out.spec = { ...ls.spec, judgedBy: 'the latest reading\'s own panel spec (specs differ by panel)', latestInSpec: (ls.spec.min == null || ys[n - 1] >= ls.spec.min) && (ls.spec.max == null || ys[n - 1] <= ls.spec.max) };
+      }
       const sl = _shelfLife(pts, spec);
       const as = _shelfLifeAssess(sl, target && target.months);
       if (sl) out.shelfLife = {
@@ -1128,7 +1147,7 @@ const get_stability = {
 
     return {
       entity: { type: entityType, uid: entity.uid, name: entity.name || entity.uid },
-      parameter: args.parameter, unit, spec, pointCount: points.length, byCondition: split, series,
+      parameter: args.parameter, unit, spec, ...(specVaries ? { specVaries, specNote: 'Readings were judged against different specs (different panels): no single band, no shelf-life projection; each reading keeps its own pass/fail.' } : {}), pointCount: points.length, byCondition: split, series,
       targetShelfLife: target ? { months: target.months, source: target.source } : null,
       note: 'shelfLife.fromT0Months / ci95FromT0Months are the projected shelf life counted from the first test (T0): point estimate and the conservative (ICH Q1E) 95%-bound crossing. remainingMonths / ci95Months are the same, counted from the last test. verdict compares against targetShelfLife: "short" = the trend crosses spec before the target, "marginal" = the trend clears it but the 95% bound does not, "meets" = both clear it; null = no target set (then concern is by time left: high <3 mo, medium <12 mo). overdue=true means projectedCrossDate is already past today (time left runs from the last test), so the sample is probably out of spec now — recommend a re-test. reportable=false means the app shows no projection (R² < 0.3, crossing already past, or beyond 5 years). Numbers are computed here — narrate them, do not recompute.',
     };