formlab-mcp 0.6.37 → 0.6.38

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "formlab-mcp",
-  "version": "0.6.37",
+  "version": "0.6.38",
   "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.37",
+  "version": "0.6.38",
   "websiteUrl": "https://formvix.com/mcp",
   "repository": {
     "url": "https://github.com/juliu1980/FormLab",
@@ -13,7 +13,7 @@
     {
       "registryType": "npm",
       "identifier": "formlab-mcp",
-      "version": "0.6.37",
+      "version": "0.6.38",
       "runtimeHint": "npx",
       "transport": {
         "type": "stdio"

package/tools/analytics.js

@@ -164,8 +164,8 @@ const list_test_results = {
         const found = (t.measurements || []).some(m => {
           if (args.parameter && !norm(m.parameter).includes(pn)) return false;
           if (hasRange) {
-            const v = parseFloat(m.value);
-            if (!Number.isFinite(v)) return false;
+            const v = _mcpRepNumber(m);   // a curve's final value, not its point count (13a)
+            if (v == null) return false;
             if (args.value_min != null && v < args.value_min) return false;
             if (args.value_max != null && v > args.value_max) return false;
           }
@@ -352,7 +352,9 @@ const get_doe_matrix = {
     const cellParamValue = (r, col) => {
       const ms = r.measurements.filter(m => m.parameter === col.id && (col.cond == null || _mcpCondSig(m) === col.cond));
       if (!ms.length) return null;
-      const nums = ms.map(m => parseFloat(m.value)).filter(v => isFinite(v));
+      // The app's DOE Matrix numbers (13a): a curve's final value, a
+      // distribution's D50, an ordinal's 1-based rank; text → its first reading.
+      const nums = ms.map(m => _mcpRepNumber(m, { ordinal: 'rank' })).filter(v => v != null);
       if (!nums.length) return ms[0].value ?? null;
       const mean = nums.reduce((s, v) => s + v, 0) / nums.length;
       return +mean.toFixed(4);
@@ -405,18 +407,44 @@ const get_doe_matrix = {
 // so a parameter measured 3× on one occasion counts once in failure rates and
 // coverage, exactly as Test Analytics does.
 const FAIL_MIN_N = 5;   // app: _TA_FAIL_MIN_N — rates on fewer readings are "low n"
-function _mcpRepNumber(m) {
-  const type = m.type || 'scalar';
-  if (type === 'time_series' && Array.isArray(m.points) && m.points.length) {
+// One number per measurement — the app's flMeasNum (js/utils.js, 13a), kept
+// in step. Never parseFloat(m.value): a saved time series / distribution has
+// "12 points" / "8 bins" there, and an ordinal label "6B" parses to 6.
+//   scalar → value; time_series → opts.ts ('final' default | 'initial' | 'peak');
+//   distribution → opts.dist ('D50' default, a named bin only — no stand-in);
+//   ordinal → null, or with opts.ordinal === 'rank' its 1-based scale position;
+//   categorical → null.
+function _mcpMeasType(m) {
+  const T = ['scalar', 'ordinal', 'categorical', 'time_series', 'distribution'];
+  if (m && m.type && T.includes(m.type)) return m.type;
+  return m && m.valueType === 'Categorical' ? 'categorical' : 'scalar';
+}
+function _mcpRepNumber(m, opts) {
+  const o = opts || {};
+  if (!m) return null;
+  const type = _mcpMeasType(m);
+  if (type === 'time_series') {
+    if (!Array.isArray(m.points) || !m.points.length) return null;
     const agg = _mcpTsAggregate(m.points);
-    const last = agg[agg.length - 1];
-    return last ? last.mean : null;
+    if (!agg.length) return null;
+    const means = agg.map(g => g.mean);
+    const red = o.ts || 'final';
+    const v = red === 'initial' ? means[0] : red === 'peak' ? Math.max(...means) : means[means.length - 1];
+    return isFinite(v) ? v : null;
   }
-  if (type === 'distribution' && Array.isArray(m.bins) && m.bins.length) {
-    const d50 = m.bins.find(b => String(b.label || '').toUpperCase().replace(/\s/g, '') === 'D50');
-    const v = d50 ? parseFloat(d50.value) : NaN;
+  if (type === 'distribution') {
+    if (!Array.isArray(m.bins) || !m.bins.length) return null;
+    const want = String(o.dist || 'D50').toUpperCase();
+    const b = m.bins.find(x => String((x && x.label) || '').toUpperCase().replace(/\s/g, '') === want);
+    const v = b ? parseFloat(b.value) : NaN;
     return isFinite(v) ? v : null;
   }
+  if (type === 'ordinal') {
+    if (o.ordinal !== 'rank' || !Array.isArray(m.scale)) return null;
+    const i = m.scale.map(x => String(x).trim().toUpperCase()).indexOf(String(m.value == null ? '' : m.value).trim().toUpperCase());
+    return i >= 0 ? i + 1 : null;
+  }
+  if (type === 'categorical') return null;
   const v = parseFloat(m.value);
   return isFinite(v) ? v : null;
 }
@@ -1153,6 +1181,7 @@ const get_stability = {
     const paramLc = (args.parameter || '').trim().toLowerCase();
     if (!paramLc) return { error: 'parameter is required.' };
     const points = [];
+    const skipped = { ordinal: 0, categorical: 0 };
     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
@@ -1165,11 +1194,12 @@ const get_stability = {
       if (!Number.isFinite(date.getTime())) return;
       (t.measurements || []).forEach(m => {
         if ((m.parameter || '').trim().toLowerCase() !== paramLc) return;
-        let value = parseFloat(m.value);
-        if (!Number.isFinite(value) && Array.isArray(m.points) && m.points.length) {
-          const last = m.points[m.points.length - 1]; value = parseFloat(last && (last.value ?? last.y));   // time-series → final
-        }
-        if (!Number.isFinite(value)) return;
+        // As the app's Stability chart (13a): a curve → its final value (not
+        // "12 points" → 12), and ordinal / text readings left out and counted.
+        const _ty = _mcpMeasType(m);
+        if (_ty === 'ordinal' || _ty === 'categorical') { skipped[_ty]++; return; }
+        const value = _mcpRepNumber(m);
+        if (value == null) return;
         if (m.unit && !unit) unit = m.unit;
         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;
@@ -1208,7 +1238,7 @@ const get_stability = {
       points.push({ date: new Date(g[0].day), value, cond: g[0].cond, spec: (g.find(r => r.spec) || {}).spec || null, readings: g.length });
     });
     points.sort((a, b) => (a.date - b.date) || a.cond.localeCompare(b.cond));
-    if (!points.length) return { error: `No dated "${args.parameter}" measurements on ${entity.uid || entity.id}.` };
+    if (!points.length) return { error: `No dated numeric "${args.parameter}" measurements on ${entity.uid || entity.id}.${skipped.ordinal || skipped.categorical ? ` ${skipped.ordinal} ordinal and ${skipped.categorical} text reading(s) exist but have no number to trend — use list_test_results / find_failures for their pass/fail.` : ''}` };
 
     const spec = { min: specMin, max: specMax };
     // Target shelf life: the formula's own Shelf life, else the workspace
@@ -1275,14 +1305,14 @@ const get_stability = {
 
     return {
       entity: { type: entityType, uid: entity.uid, name: entity.name || entity.uid },
-      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,
+      parameter: args.parameter, unit, spec, ...((skipped.ordinal || skipped.categorical) ? { skipped, skippedNote: 'Ordinal (ranked scale) and text readings have no number to trend and are left out, as in the app.' } : {}), ...(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: medium <3 mo or overdue, else info). A projection never reaches \"high\": high is reserved for a MEASURED out-of-spec value (spec.latestInSpec=false). short/overdue → medium, marginal/meets → info. 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.',
     };
   },
 };
 
-export { _evalPassFail, _resolveSpec, _shelfLife, _shelfLifeAssess, _mcpCondTags, _mcpArmNames, _mcpCondDefaultScore };
+export { _mcpRepNumber, _mcpMeasType, _evalPassFail, _resolveSpec, _shelfLife, _shelfLifeAssess, _mcpCondTags, _mcpArmNames, _mcpCondDefaultScore };
 export const tools = {
   list_test_results,
   get_test_result,

package/tools/designspace.js

@@ -27,6 +27,7 @@
 // ============================================================
 
 import { getStore, resolveById, flattenComposition } from '../data.js';
+import { _mcpRepNumber } from './analytics.js';
 
 // ---- composition helper -----------------------------------------------------
 // Ingredient wt-% map for one formula, mirroring the app's _asmIngredientMap:
@@ -344,12 +345,20 @@ function _roleCandidates(forms, roleMaps) {
   });
   return out.sort((a, b) => b.score - a.score);
 }
-// A formula's value for a measured parameter: the mean of every numeric
-// reading of it across the formula's samples (the app's _asmFormulaValueForCol).
+// A formula's value for a measured parameter, as the app's
+// _asmFormulaValueForCol (13a): readings of the primary run condition (the one
+// with the most readings), each as the DOE Matrix reads it (a curve's final
+// value, a distribution's D50, an ordinal's rank), then their mean.
 function _paramValue(f, param, db) {
   const sids = new Set((db.samples || []).filter(s => s.formulationId === f.id).map(s => s.id));
-  const vals = [];
-  (db.testResults || []).forEach(t => { if (!sids.has(t.sampleId)) return; (t.measurements || []).forEach(m => { if (m.parameter === param) { const v = parseFloat(m.value); if (!isNaN(v)) vals.push(v); } }); });
+  const ms = [];
+  (db.testResults || []).forEach(t => { if (!sids.has(t.sampleId)) return; (t.measurements || []).forEach(m => { if (m.parameter === param) ms.push(m); }); });
+  const sig = (m) => { const c = m && m.conditions; return c && typeof c === 'object' ? Object.keys(c).filter(k => c[k] != null && c[k] !== '').sort().map(k => k + '=' + c[k]).join('; ') : ''; };
+  const byCond = new Map();
+  ms.forEach(m => { if (_mcpRepNumber(m) == null) return; const k = sig(m); byCond.set(k, (byCond.get(k) || 0) + 1); });
+  const primary = [...byCond.entries()].sort((a, b) => b[1] - a[1])[0];
+  const pool = primary ? ms.filter(m => sig(m) === primary[0]) : ms;
+  const vals = pool.map(m => _mcpRepNumber(m, { ordinal: 'rank' })).filter(v => v != null);
   return vals.length ? vals.reduce((a, b) => a + b, 0) / vals.length : null;
 }
 function _spearman(xs, ys) {