@measurequick/measurequick-report-generator 1.5.191 → 1.5.193

package/.claude/settings.local.json

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

package/package.json

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

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

@@ -39,7 +39,7 @@ export async function getReport(payload, options = {}) {
     let companyImage = null;
     let companyImageType = null;
     try {
-      companyImageType = util.checkCompanyLogo?.(payload.meta);
+      companyImageType = util.checkCompanyLogo ? util.checkCompanyLogo(payload.meta) : null;
     } catch (e) {}
     if (companyImageType === "jpg") {
       companyImage = await pdfDoc.embedJpg(
@@ -55,7 +55,7 @@ export async function getReport(payload, options = {}) {
     let profileImage = null;
     let profileImageType = null;
     try {
-      profileImageType = util.checkProfilePicture?.(payload.meta);
+      profileImageType = util.checkProfilePicture ? util.checkProfilePicture(payload.meta) : null;
     } catch (e) {}
     if (profileImageType) {
       let profilePicBase64;
@@ -89,7 +89,7 @@ export async function getReport(payload, options = {}) {
     }
 
     // Set tech name
-    if (payload.meta.profile_settings?.techFirstName || payload.meta.profile_settings?.techLastName) {
+    if ((payload.meta.profile_settings && payload.meta.profile_settings.techFirstName) || (payload.meta.profile_settings && payload.meta.profile_settings.techLastName)) {
       const techName = `${payload.meta.profile_settings.techFirstName || ""} ${payload.meta.profile_settings.techLastName || ""}`.trim();
       form.getTextField("TechName").setText(techName);
     }
@@ -102,13 +102,13 @@ export async function getReport(payload, options = {}) {
     }
 
     // Get AI summary and notes from payload or options
-    const aiSummary = options.aiSummary || util.getAiSummary?.(payload) || "";
-    const notes = options.notes || util.getAiNotes?.(payload) || "";
+    const aiSummary = options.aiSummary || (util.getAiSummary ? util.getAiSummary(payload) : "") || "";
+    const notes = options.notes || (util.getAiNotes ? util.getAiNotes(payload) : "") || "";
 
     // 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
@@ -260,8 +260,8 @@ function wrapText(text, font, fontSize, maxWidth) {
  * @param {Object} payload - The report payload
  */
 export async function addNotesSummaryPages(pdfDoc, payload) {
-  const aiSummary = util.getAiSummary?.(payload) || "";
-  const notes = util.getAiNotes?.(payload) || "";
+  const aiSummary = (util.getAiSummary ? util.getAiSummary(payload) : "") || "";
+  const notes = (util.getAiNotes ? util.getAiNotes(payload) : "") || "";
 
   if (!aiSummary && !notes) return;
 

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,27 @@ 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
+    return payload.project.aiSummaries[payload.project.aiSummaries.length - 1];
+  }
   return null;
 }
 
@@ -1072,8 +1089,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;
 }