Merge remote-tracking branch 'origin/feature/IO-3515-bill-ocr-feedback' into hotfix/2026-03-20

server/ai/bill-ai-feedback.js

@@ -0,0 +1,73 @@
+const { isString } = require("lodash");
+const { sendServerEmail } = require("../email/sendemail");
+const logger = require("../utils/logger");
+const { raw } = require("express");
+
+const SUPPORT_EMAIL = "patrick@imexsystems.ca";
+
+const safeJsonParse = (maybeJson) => {
+    if (!isString(maybeJson)) return null;
+    try {
+        return JSON.parse(maybeJson);
+    } catch {
+        return null;
+    }
+};
+
+
+const handleBillAiFeedback = async (req, res) => {
+    try {
+        const rating = req.body?.rating;
+        const comments = isString(req.body?.comments) ? req.body?.comments?.trim() : "";
+
+        const billFormValues = safeJsonParse(req.body?.billFormValues);
+        const rawAIData = safeJsonParse(req.body?.rawAIData);
+
+        const jobid = billFormValues?.jobid || billFormValues?.jobId || "unknown";
+        const shopname = req.body?.shopname || "unknown";
+        const subject = `Bill AI Feedback (${rating === "up" ? "+" : "-"}) Shop=${shopname} jobid=${jobid}`;
+
+        const text = [
+            `User: ${req?.user?.email || "unknown"}`,
+            `Rating: ${rating}`,
+            comments ? `Comments: ${comments}` : "Comments: (none)",
+            "",
+            "Form Values (User):",
+            JSON.stringify(billFormValues, null, 4),
+            "",
+            "Raw AI Data:",
+            JSON.stringify(rawAIData, null, 4)
+        ]
+            .filter(Boolean)
+            .join("\n");
+
+        const attachments = [];
+        if (req.file?.buffer) {
+            attachments.push({
+                filename: req.file.originalname || `bill-${jobid}.pdf`,
+                content: req.file.buffer,
+                contentType: req.file.mimetype || "application/pdf"
+            });
+        }
+
+        await sendServerEmail({
+            to: [SUPPORT_EMAIL],
+            subject,
+            type: "text",
+            text,
+            attachments
+        });
+
+        return res.json({ success: true });
+    } catch (error) {
+        logger.log("bill-ai-feedback-error", "ERROR", req?.user?.email, null, {
+            message: error?.message,
+            stack: error?.stack
+        });
+        return res.status(500).json({ message: "Failed to submit feedback" });
+    }
+};
+
+module.exports = {
+    handleBillAiFeedback
+};

server/ai/bill-ocr/bill-ocr.js

@@ -212,7 +212,8 @@ async function processSinglePageDocument(pdfBuffer) {
 
     return {
         ...processedData,
-        originalTextractResponse: result
+        //Removed as this is a large object that provides minimal value to send to client.
+        // originalTextractResponse: result
     };
 }
 
@@ -392,7 +393,8 @@ async function handleTextractNotification(message) {
                     status: 'COMPLETED',
                     data: {
                         ...processedData,
-                        originalTextractResponse: originalResponse
+                        //Removed as this is a large object that provides minimal value to send to client.
+                        // originalTextractResponse: originalResponse 
                     },
                     completedAt: new Date().toISOString()
                 }

server/email/sendemail.js

@@ -44,7 +44,7 @@ const logEmail = async (req, email) => {
   }
 };
 
-const sendServerEmail = async ({ subject, text, to = [] }) => {
+const sendServerEmail = async ({ subject, text, to = [], attachments }) => {
   if (process.env.NODE_ENV === undefined) return;
 
   try {
@@ -57,6 +57,7 @@ const sendServerEmail = async ({ subject, text, to = [] }) => {
         to: ["support@imexsystems.ca", ...to],
         subject: subject,
         text: text,
+        attachments: attachments,
         ses: {
           // optional extra arguments for SendRawEmail
           Tags: [

server/routes/aiRoutes.js

@@ -4,9 +4,14 @@ const multer = require("multer");
 const validateFirebaseIdTokenMiddleware = require("../middleware/validateFirebaseIdTokenMiddleware");
 const withUserGraphQLClientMiddleware = require("../middleware/withUserGraphQLClientMiddleware");
 const { handleBillOcr, handleBillOcrStatus } = require("../ai/bill-ocr/bill-ocr");
+const { handleBillAiFeedback } = require("../ai/bill-ai-feedback");
 
-// Configure multer for form data parsing
-const upload = multer();
+// Configure multer for form data parsing (memory storage)
+const upload = multer({
+	limits: {
+		fileSize: 5 * 1024 * 1024 // 5MB
+	}
+});
 
 router.use(validateFirebaseIdTokenMiddleware);
 router.use(withUserGraphQLClientMiddleware);
@@ -14,4 +19,6 @@ router.use(withUserGraphQLClientMiddleware);
 router.post("/bill-ocr", upload.single('billScan'), handleBillOcr);
 router.get("/bill-ocr/status/:textractJobId", handleBillOcrStatus);
 
+router.post("/bill-feedback", upload.single("billPdf"), handleBillAiFeedback);
+
 module.exports = router;