@measurequick/measurequick-report-generator 1.5.192 → 1.5.194

package/.claude/settings.local.json

@@ -10,7 +10,10 @@
       "Bash(npm publish:*)",
       "Bash(node -e:*)",
       "Bash(npm version patch)",
-      "Bash(node:*)"
+      "Bash(node:*)",
+      "Bash(git add:*)",
+      "Bash(git commit:*)",
+      "Bash(npm view:*)"
     ]
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@measurequick/measurequick-report-generator",
-  "version": "1.5.192",
+  "version": "1.5.194",
   "description": "Generates PDF documents for various measureQuick applications.",
   "main": "index.js",
   "scripts": {

package/report-gen-scripts/notes-summary-page.js

@@ -108,7 +108,7 @@ export async function getReport(payload, options = {}) {
     // Combine content
     let fullContent = "";
     if (aiSummary) {
-      fullContent += "AI SUMMARY\n\n" + aiSummary + "\n\n";
+      fullContent += "SUMMARY\n\n" + aiSummary + "\n\n";
     }
     if (notes) {
       fullContent += "NOTES\n\n" + notes;
@@ -166,7 +166,7 @@ export async function getReport(payload, options = {}) {
       }
 
       // Check if this is a title line
-      const isTitle = line === "AI SUMMARY" || line === "NOTES";
+      const isTitle = line === "SUMMARY" || line === "NOTES";
 
       if (isTitle) {
         // Add some spacing before title

package/report-gen-scripts/vitals-heatpump-heating-report.js

@@ -230,19 +230,34 @@ export async function getReport(payload, _test) {
     form.getTextField("YourSystemScorePage2").setText(`${systemScorePercentage} ${systemScoreGrade}`);
 
     // Print targets and range icons (using appropriate values for heat pump heating)
-    const measureLabels = ["Superheat", "Subcooling", "Condenser", "TempSplit", "Tesp", "FilterFace"];
-    for (let i = 0; i < measureLabels.length; i++) {
-      // For heat pump heating, show green icons for valid measurements
-      let icon = iconRangeGreen;
-      let iconPlacement = "Mid";
-      try {
-        form.getButton(`Image${measureLabels[i]}${iconPlacement}_af_image`).setImage(icon);
-      } catch (e) {
-        // Field may not exist
+    // Only show range icons for measurements that exist
+    // Map field names to textFields values and check if they exist
+    const measureChecks = [
+      { label: "Superheat", value: textFields.tempRise },      // Temperature Rise
+      { label: "Subcooling", value: textFields.llt },          // Liquid Line Temp
+      { label: "Condenser", value: textFields.cop },           // COP/Efficiency
+      { label: "TempSplit", value: textFields.oat },           // Outdoor Air Temp
+      { label: "Tesp", value: textFields.tesp },               // Total External Static Pressure
+      { label: "FilterFace", value: textFields.airflow }       // Airflow
+    ];
+
+    for (let i = 0; i < measureChecks.length; i++) {
+      const { label, value } = measureChecks[i];
+      // Only show range indicator if value exists (is not "--")
+      if (value && value !== "--") {
+        // For heat pump heating, we don't have typical target ranges yet
+        // Show green icon as placeholder for valid measurements
+        let icon = iconRangeGreen;
+        let iconPlacement = "Mid";
+        try {
+          form.getButton(`Image${label}${iconPlacement}_af_image`).setImage(icon);
+        } catch (e) {
+          // Field may not exist
+        }
       }
       // Clear target zone text for heat pump heating (different targets apply)
       try {
-        form.getTextField(`${measureLabels[i]}Target`).setText("");
+        form.getTextField(`${label}Target`).setText("");
       } catch (e) {
         // Field may not exist
       }
@@ -299,7 +314,8 @@ function getTextFields(payload, test) {
     cop: test.data.cop !== undefined ? (+test.data.cop).toFixed(2) : "--",
     oat: test.data.temperature_outdoor_dry_bulb !== undefined ? (+test.data.temperature_outdoor_dry_bulb).toFixed(1) : "--",
     tesp: test.data.pressure_static_total_external !== undefined ? (+test.data.pressure_static_total_external).toFixed(2) : "--",
-    airflow: test.data.airflow !== undefined ? (+test.data.airflow).toFixed(0) : "--",
+    airflow: test.data.airflow !== undefined ? (+test.data.airflow).toFixed(0) :
+             (test.data.airflow_estimated !== undefined ? (+test.data.airflow_estimated).toFixed(0) : "--"),
     // Heat pump heating scoring factors (points earned, not losses)
     ptsFilter: test.data.hp_heating_pts_filter !== undefined ? (+test.data.hp_heating_pts_filter).toFixed(0) : "--",
     ptsA: test.data.hp_heating_pts_a !== undefined ? (+test.data.hp_heating_pts_a).toFixed(0) : "--",

package/util.js

@@ -1061,10 +1061,31 @@ export function shouldShowAiSummary(payload) {
 }
 
 export function getAiSummary(payload) {
-  // Check multiple possible sources for AI summary
+  // Check multiple possible sources for AI summary (both snake_case and camelCase)
   if (payload.ai_summary) return payload.ai_summary;
+  if (payload.aiSummary) return payload.aiSummary;
   if (payload.project && payload.project.ai_summary) return payload.project.ai_summary;
+  if (payload.project && payload.project.aiSummary) return payload.project.aiSummary;
   if (payload.project && payload.project.mq_ai_summary) return payload.project.mq_ai_summary;
+  if (payload.project && payload.project.mqAiSummary) return payload.project.mqAiSummary;
+  // Check in tests array
+  if (payload.tests && payload.tests[0]) {
+    if (payload.tests[0].testInfo && payload.tests[0].testInfo.aiSummary) {
+      return payload.tests[0].testInfo.aiSummary;
+    }
+    if (payload.tests[0].testInfo && payload.tests[0].testInfo.ai_summary) {
+      return payload.tests[0].testInfo.ai_summary;
+    }
+  }
+  // Check aiSummaries array on project
+  if (payload.project && payload.project.aiSummaries && payload.project.aiSummaries.length > 0) {
+    // Return the most recent summary - extract text if it's an object
+    const summary = payload.project.aiSummaries[payload.project.aiSummaries.length - 1];
+    if (typeof summary === 'string') return summary;
+    if (summary && summary.text) return summary.text;
+    if (summary && summary.summary) return summary.summary;
+    if (summary && summary.content) return summary.content;
+  }
   return null;
 }
 
@@ -1072,8 +1093,13 @@ export function getAiNotes(payload) {
   // Check multiple possible sources for notes
   if (payload.notes) return payload.notes;
   if (payload.project && payload.project.notes) return payload.project.notes;
+  // Check in tests array
   if (payload.tests && payload.tests[0] && payload.tests[0].testInfo && payload.tests[0].testInfo.notes) {
     return payload.tests[0].testInfo.notes;
   }
+  // Check testInfo directly on payload (for single test payloads)
+  if (payload.testInfo && payload.testInfo.notes) {
+    return payload.testInfo.notes;
+  }
   return null;
 }