@eshal-bot/chat-widget 0.1.92 → 0.1.94

package/dist/chat-widget-markdown-chunk.js

@@ -89272,12 +89272,6 @@
   const MONTH_NAMES = ['Jan', 'Feb', 'Mar', 'Apr', 'May', 'Jun', 'Jul', 'Aug', 'Sep', 'Oct', 'Nov', 'Dec'];
   const CURRENCY_KEY_RE = /(^|[_\W])(price|salary|cost|amount|revenue|expense|total|value|budget|spend|income|sale|balance)/i;
   const MONTH_KEY_RE = /(^|[_\W])month/i;
-  const USD_FORMATTER = new Intl.NumberFormat('en-US', {
-    style: 'currency',
-    currency: 'USD',
-    minimumFractionDigits: 2,
-    maximumFractionDigits: 2
-  });
   function prettifyLabel(value) {
     if (value === null || value === undefined) return '';
     const s = String(value);
@@ -89340,31 +89334,52 @@
   }
 
   // Numeric string under a money column, e.g. "81,200" / "105444" / "-1,250.50".
-  // Markdown tables (the common concierge path) arrive as text cells, so a
-  // salary value is the STRING "81,200" - not a number - and skips the
-  // USD_FORMATTER branch below. We prefix "$" while keeping the source's own
-  // grouping/decimals so it reads "$81,200" (not "$81,200.00"). A leading "$"
-  // already present makes the regex miss, so we never double-prefix.
-  const _NUMERIC_STRING_RE = /^-?\d[\d,]*(\.\d+)?$/;
+  // Table-cell formatting, kept IN LOCKSTEP with the Testing-Lab (frontend
+  // chartShared.tsx: formatDateCell / formatCurrencyCellString) and SDK copies so
+  // all three surfaces render DB tables identically. Two rules:
+  //   1. ISO date/datetime → "Dec 9, 2021" (parity with Testing Lab).
+  //   2. Bare numeric string in a money-named column → comma-group, and NEVER
+  //      synthesize a currency symbol — the DB is the source of truth for "$".
+  //      A value that already carries a symbol ("$71000", "₹560") fails
+  //      _BARE_NUMBER_RE and is passed through verbatim, so we don't double-format.
+  const _BARE_NUMBER_RE = /^-?\d+(\.\d+)?$/;
+  const _MONTH_NAMES = ['Jan', 'Feb', 'Mar', 'Apr', 'May', 'Jun', 'Jul', 'Aug', 'Sep', 'Oct', 'Nov', 'Dec'];
+  const _ISO_DATETIME_CELL_RE = /^\d{4}-\d{2}-\d{2}([T ]\d{2}:\d{2}(:\d{2})?(\.\d+)?(Z|[+-]\d{2}:?\d{2})?)?$/;
+  function _formatDateCell(value) {
+    if (typeof value !== 'string') return null;
+    if (!_ISO_DATETIME_CELL_RE.test(value.trim())) return null;
+    const d = new Date(value);
+    if (isNaN(d.getTime())) return null;
+    const hasTime = /[T ]\d{2}:\d{2}/.test(value) && (d.getUTCHours() !== 0 || d.getUTCMinutes() !== 0 || d.getUTCSeconds() !== 0);
+    const dateStr = "".concat(_MONTH_NAMES[d.getUTCMonth()], " ").concat(d.getUTCDate(), ", ").concat(d.getUTCFullYear());
+    if (!hasTime) return dateStr;
+    const h = d.getUTCHours();
+    const ampm = h >= 12 ? 'PM' : 'AM';
+    const h12 = h % 12 || 12;
+    const mm = String(d.getUTCMinutes()).padStart(2, '0');
+    return "".concat(dateStr, ", ").concat(h12, ":").concat(mm, " ").concat(ampm);
+  }
+  function _commaGroup(n) {
+    return Number.isInteger(n) ? n.toLocaleString('en-US') : Number(n.toFixed(2)).toLocaleString('en-US');
+  }
   function formatCell(v, columnKey) {
     if (v == null) return '';
     if (typeof v === 'number') {
-      // Currency-keyed numeric cells in tables also get $ + decimals so
-      // the table next to a chart agrees on its number formatting.
-      if (isCurrencyKey(columnKey)) return USD_FORMATTER.format(v);
-      return Number.isInteger(v) ? String(v) : v.toFixed(2);
+      // Comma-group; NEVER synthesize a currency symbol (DB owns symbols).
+      return _commaGroup(v);
     }
     if (typeof v === 'object') return JSON.stringify(v);
-    // Numeric string under a money-named column → prefix "$" (Bug: concierge
-    // salary table showed "81,200" with no currency symbol).
-    if (isCurrencyKey(columnKey) && _NUMERIC_STRING_RE.test(String(v).trim())) {
-      return "$".concat(String(v).trim());
-    }
-    // String cells: prettify snake_case category labels (e.g.
-    // `office_supplies` → "Office Supplies") so the table reads
-    // consistently with the chart's X-axis ticks on the same data.
-    // Conservative: anything not a clean lowercase_underscore token is
-    // returned verbatim, so paths/IDs/emails/slugs are preserved.
+    const s = String(v).trim();
+    // 1. ISO date/datetime → friendly date.
+    const dateCell = _formatDateCell(s);
+    if (dateCell) return dateCell;
+    // 2. Bare numeric string in a money column → comma-group, NO "$".
+    if (isCurrencyKey(columnKey) && _BARE_NUMBER_RE.test(s)) {
+      const n = Number(s);
+      if (Number.isFinite(n)) return _commaGroup(n);
+    }
+    // 3. Everything else: prettify snake_case labels; IDs/paths/emails/slugs and
+    // any DB value already carrying a symbol pass through verbatim.
     return prettifyCategoryCell(v);
   }
 
@@ -91218,6 +91233,20 @@
     return lines.slice(0, start).join('\n');
   }
 
+  // EV-2782: SQL aggregates (AVG(salary), etc.) come back from Postgres as a
+  // NUMERIC with a fixed decimal scale, so the DB tool round-trips it as e.g.
+  // `106500.00000000000` - trailing zero padding from the column's declared
+  // precision, not a real value the agent computed. The agent quotes it
+  // verbatim into its prose, so it reaches here as plain text. Collapse any
+  // decimal whose digits past the 2nd place are ALL zero down to 2 decimals -
+  // safe because a genuinely precise value (19.99, 3.14159) never has an
+  // all-zero tail past position 2, so this only ever fires on exactly this
+  // padding artifact. Mirrors the same fix in the frontend's
+  // StreamingMarkdown.tsx (Testing Lab renderer).
+  function sanitizeExcessDecimalPrecision(text) {
+    return text.replace(/(\d+\.\d{2})0+(?!\d)/g, '$1');
+  }
+
   // The rich markdown renderer. Receives the already-animated `content`
   // (displayedContent from StreamingMarkdown) plus styling props. Renders only
   // the ReactMarkdown tree - the outer wrapper <div> and the streaming cursor
@@ -91665,7 +91694,7 @@
           }));
         }
       },
-      children: sanitizeStreamingContent(content, isStreaming)
+      children: sanitizeExcessDecimalPrecision(sanitizeStreamingContent(content, isStreaming))
     });
   }