@paro.io/expert-shared-components 1.14.55 → 1.14.56

package/lib/tax-axis/components/dashboard/StrategyDetailPanel.js

@@ -25,49 +25,46 @@ function positionStrengthLabel(positionStrength) {
     return { label: positionStrength, color: "#9498B8", dots: 2 };
 }
 /**
- * Split engagement_recommendation into two parts:
- * - savingsSection: sentences/paragraphs that contain dollar figures or formula references
- *   → shown as "HOW SAVINGS BREAK DOWN"
- * - cpaSection: the remaining implementation/CPA-action paragraphs
- *   → shown as "CPA ENGAGEMENT NOTES"
+ * Split engagement_recommendation at the implementation/action boundary.
  *
- * Strategy: split on double-newlines or sentence boundaries. Sentences with "$" or
- * formulas go to savings; implementation/deadline/CPA sentences go to cpa.
+ * The LLM writes engagement_recommendation as a single paragraph covering both
+ * the savings math AND the CPA implementation steps. We split at the first
+ * implementation keyword so the savings math shows under "HOW SAVINGS BREAK DOWN"
+ * and the CPA action steps show under "CPA ENGAGEMENT NOTES".
+ *
+ * Heuristic: split at "Implementation:", "CPA should", "CPA must", or numbered
+ * action lists "(1)". If no split point found, entire text goes to savings section.
  */
 function splitEngagementText(text) {
     if (!text)
         return { savingsSection: "", cpaSection: "" };
-    // Split into paragraphs first, then sentences within paragraphs
-    const rawParas = text.split(/\n{2,}/).map((p) => p.trim()).filter(Boolean);
-    const savingsParas = [];
-    const cpaParas = [];
-    for (const para of rawParas) {
-        const hasDollar = /\$[\d,]+/.test(para);
-        const hasFormula = /SAVINGS\s*=|CREDIT\s*=|DEDUCTION\s*=|ADDITIONAL_CONTRIBUTION/i.test(para);
-        const hasPercent = /\d+\.?\d*%/.test(para);
-        const hasCpaKeyword = /\bCPA\b|actuar|implement|deadline|establish|file|form\s+\d|verify|engage|model|recommend/i.test(para);
-        if ((hasDollar || hasFormula || hasPercent) && !hasCpaKeyword) {
-            savingsParas.push(para);
-        }
-        else if (savingsParas.length === 0) {
-            // First para with dollar amounts seeds savings section even if it has CPA keywords
-            if (hasDollar || hasFormula)
-                savingsParas.push(para);
-            else
-                cpaParas.push(para);
-        }
-        else {
-            cpaParas.push(para);
-        }
+    // Try splitting on double-newline first (well-structured LLM output)
+    const paraBreak = text.indexOf("\n\n");
+    if (paraBreak !== -1) {
+        return {
+            savingsSection: text.slice(0, paraBreak).trim(),
+            cpaSection: text.slice(paraBreak).trim(),
+        };
     }
-    // If everything ended up in one bucket, put first para in savings, rest in cpa
-    if (!savingsParas.length && cpaParas.length) {
-        savingsParas.push(cpaParas.shift());
+    // Single-paragraph: find the first implementation boundary
+    const splitPatterns = [
+        /\s+Implementation:\s+/i,
+        /\s+CPA should\s+/i,
+        /\s+CPA must\s+/i,
+        /\s+To implement:\s+/i,
+        /\s+\(1\)\s+/, // numbered list start
+    ];
+    for (const pattern of splitPatterns) {
+        const match = text.search(pattern);
+        if (match > 80) { // only split if there's meaningful content before it
+            return {
+                savingsSection: text.slice(0, match).trim(),
+                cpaSection: text.slice(match).trim(),
+            };
+        }
     }
-    return {
-        savingsSection: savingsParas.join("\n\n"),
-        cpaSection: cpaParas.join("\n\n"),
-    };
+    // No clean split point — show everything as savings breakdown, nothing as CPA notes
+    return { savingsSection: text, cpaSection: "" };
 }
 function SourceDocRows({ docs, trace }) {
     if (docs && docs.length > 0) {

package/lib/tax-axis/components/dashboard/TaxAxisDashboard.js

@@ -27,6 +27,7 @@ exports.TaxAxisDashboard = TaxAxisDashboard;
 const react_1 = __importStar(require("react"));
 const data_1 = require("../../lib/data");
 const compute_1 = require("../../lib/compute");
+const useEngineOutput_1 = require("../../lib/adapters/useEngineOutput");
 const TaxAxisButton_1 = require("../shared/TaxAxisButton");
 const TaxAxisBadge_1 = require("../shared/TaxAxisBadge");
 const DashboardSummary_1 = require("./DashboardSummary");
@@ -163,8 +164,29 @@ function TaxAxisDashboard({ profile, llmResult, parsedDocuments, onDownloadClien
     // Extracted documents: live from session if provided, otherwise empty (no mock fallback)
     const extractionDocs = parsedDocuments !== null && parsedDocuments !== void 0 ? parsedDocuments : [];
     // ─── Derived data ──────────────────────────────────────────────
-    const llmStrategies = (0, react_1.useMemo)(() => (hasLlm ? mapLlmToStrategies(llmResult) : []), [hasLlm, llmResult]);
-    const llmComputed = (0, react_1.useMemo)(() => (hasLlm ? buildLlmComputed(llmStrategies) : new Map()), [hasLlm, llmStrategies]);
+    // Prefer useEngineOutput (full rich data: sourceDocuments, positionStrength,
+    // clientBrief from why_it_applies, etc.) over the thin mapLlmToStrategies fallback.
+    // engineOutput is at llmResult.engineOutput or llmResult.rawOutput (both present).
+    const engineOutputForAdapter = (0, react_1.useMemo)(() => { var _a, _b; return (_b = (_a = llmResult === null || llmResult === void 0 ? void 0 : llmResult.engineOutput) !== null && _a !== void 0 ? _a : llmResult === null || llmResult === void 0 ? void 0 : llmResult.rawOutput) !== null && _b !== void 0 ? _b : null; }, [llmResult]);
+    const adapted = (0, useEngineOutput_1.useEngineOutput)(engineOutputForAdapter);
+    const llmStrategies = (0, react_1.useMemo)(() => {
+        var _a;
+        if (!hasLlm)
+            return [];
+        // Use adapted strategies when available — they carry the full engine data.
+        if ((_a = adapted === null || adapted === void 0 ? void 0 : adapted.strategies) === null || _a === void 0 ? void 0 : _a.length)
+            return adapted.strategies;
+        // Fallback to thin mapping for old payload shapes without engineOutput.
+        return mapLlmToStrategies(llmResult);
+    }, [hasLlm, adapted, llmResult]);
+    const llmComputed = (0, react_1.useMemo)(() => {
+        var _a;
+        if (!hasLlm)
+            return new Map();
+        if ((_a = adapted === null || adapted === void 0 ? void 0 : adapted.computedMap) === null || _a === void 0 ? void 0 : _a.size)
+            return adapted.computedMap;
+        return buildLlmComputed(llmStrategies);
+    }, [hasLlm, adapted, llmStrategies]);
     const computed = (0, react_1.useMemo)(() => (hasLlm ? llmComputed : (0, compute_1.computeAllStrategies)(profile)), [hasLlm, llmComputed, profile]);
     const dashEligible = (0, react_1.useMemo)(() => (hasLlm ? llmStrategies : (0, compute_1.filterEligibleStrategies)(profile)), [hasLlm, llmStrategies, profile]);
     const maxSavings = Math.max(...dashEligible.map((s) => { var _a, _b; return (_b = (_a = computed.get(s.rank)) === null || _a === void 0 ? void 0 : _a.hi) !== null && _b !== void 0 ? _b : s.hi; }), 1);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@paro.io/expert-shared-components",
-  "version": "1.14.55",
+  "version": "1.14.56",
   "description": "",
   "main": "lib/index.js",
   "scripts": {