From 2a3da3e6e17420d7654924f0a2dd561bcdff9d91 Mon Sep 17 00:00:00 2001 From: iamsivin Date: Tue, 5 May 2026 12:24:12 +0530 Subject: [PATCH] chore: Review fix --- .../dashboard/helper/emailQuoteExtractor.js | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/app/javascript/dashboard/helper/emailQuoteExtractor.js b/app/javascript/dashboard/helper/emailQuoteExtractor.js index 162fb96f2..b6eead184 100644 --- a/app/javascript/dashboard/helper/emailQuoteExtractor.js +++ b/app/javascript/dashboard/helper/emailQuoteExtractor.js @@ -141,7 +141,15 @@ const findTopLevelTailStart = root => { if (isNeutral(n)) return false; const t = nodeText(n); if (!t.trim()) return false; - if (HARD_HEADERS.some(re => re.test(t)) || ATTRIBUTION.test(t)) { + // For element nodes the trigger must occupy the FIRST non-empty line — + // a container like 
/ wrapping reply text ABOVE a buried
+    // header would otherwise be marked as the entire quote tail and the
+    // user's reply would be dropped with it.
+    const probe =
+      n.nodeType === ELEM
+        ? (t.split('\n').find(l => l.trim()) ?? '').trim()
+        : t;
+    if (HARD_HEADERS.some(re => re.test(probe)) || ATTRIBUTION.test(probe)) {
       // No reply text above the trigger → could be bottom-posted. Mirror
       // the RFC branch: only fire when every following node is `>`-quoted
       // or neutral. Otherwise leave the body alone.
@@ -149,7 +157,7 @@ const findTopLevelTailStart = root => {
         return kids.slice(i + 1).every(c => isRfcQuoted(c) || isNeutral(c));
       return true;
     }
-    return HEADER_LINE.test(t) && countHeaderLines(tailText(i)) >= 2;
+    return HEADER_LINE.test(probe) && countHeaderLines(tailText(i)) >= 2;
   });
   return idx === -1 ? -1 : walkBack(kids, idx);
 };