fix(email): strip plain-text quoted replies whose attribution line is quote-prefixed (#324)

Some mail clients quote-prefix the "On <date>, <name> wrote:" attribution
line itself (e.g. "> On ... wrote:") instead of leaving it bare before the
quoted body. The plain-text branch of stripQuotedReply only matched an
unprefixed attribution line, so replies in this format kept the entire
original message quoted underneath instead of trimming it.

Broadened the boundary regex to tolerate an optional leading ">" quote
marker, in both the Gmail and Outlook connectors (duplicated logic, kept
in sync). Added regression tests for the quote-prefixed case to both
connectors' test suites.

Author: KrisBraun

connectors/google/src/mail/gmail-api.test.ts

@@ -298,6 +298,20 @@ describe("forwarded email body extraction", () => {
     expect(reply).not.toContain("old message text");
   });
 
+  it("strips plain-text quoted replies whose 'On ... wrote:' attribution is itself quote-prefixed", () => {
+    // Some clients (observed from a real reply) emit the attribution line
+    // WITH a leading "> " rather than leaving it unquoted, e.g.:
+    //   Reply text.
+    //   > On Jun 1, 2026, at 9:00 AM, Bob <bob@example.com> wrote:
+    //   > old message text
+    const reply = stripQuotedReply(
+      "My answer is yes.\r\n> On Mon, Jun 1, 2026, at 9:00 AM, Bob <bob@example.com> wrote:\r\n> old message text",
+      "text"
+    );
+    expect(reply).toContain("My answer is yes.");
+    expect(reply).not.toContain("old message text");
+  });
+
   it("strips Apple Mail quoted replies (<blockquote type=\"cite\">)", () => {
     // Apple Mail (Mail.app / iPhone Mail) wraps the "On <date>, <name> wrote:"
     // attribution and the quoted history in <blockquote type="cite">. The

connectors/google/src/mail/gmail-api.ts

@@ -862,8 +862,11 @@ export function stripQuotedReply(
   for (let i = 0; i < lines.length; i++) {
     const line = lines[i].trim();
 
-    // "On [date], [name] wrote:" or "On [date], [name] <email> wrote:"
-    if (/^On .+ wrote:\s*$/.test(line)) {
+    // "On [date], [name] wrote:" or "On [date], [name] <email> wrote:".
+    // Some clients quote-prefix the attribution line itself (e.g. "> On ...
+    // wrote:") instead of leaving it bare — allow an optional leading ">"
+    // quote marker so that case is still recognized as the reply boundary.
+    if (/^(?:>+\s*)*On .+ wrote:\s*$/.test(line)) {
       // Verify next non-empty line starts with ">" (actual quoted content)
       const nextContentLine = lines.slice(i + 1).find((l) => l.trim() !== "");
       if (nextContentLine && nextContentLine.trim().startsWith(">")) {

connectors/outlook/src/mail/email-parsing.test.ts

@@ -38,4 +38,9 @@ describe("stripQuotedReply", () => {
     const text = "Thanks!\nOn Tue, Jun 10, 2026, Bob wrote:\n> earlier";
     expect(stripQuotedReply(text, "text")).toBe("Thanks!");
   });
+
+  it("cuts plain-text quotes whose 'On ... wrote:' attribution is itself quote-prefixed", () => {
+    const text = "Thanks!\n> On Tue, Jun 10, 2026, at 9:00 AM, Bob <bob@example.com> wrote:\n> earlier";
+    expect(stripQuotedReply(text, "text")).toBe("Thanks!");
+  });
 });

connectors/outlook/src/mail/email-parsing.ts

@@ -136,8 +136,11 @@ export function stripQuotedReply(
   for (let i = 0; i < lines.length; i++) {
     const line = lines[i].trim();
 
-    // "On [date], [name] wrote:" or "On [date], [name] <email> wrote:"
-    if (/^On .+ wrote:\s*$/.test(line)) {
+    // "On [date], [name] wrote:" or "On [date], [name] <email> wrote:".
+    // Some clients quote-prefix the attribution line itself (e.g. "> On ...
+    // wrote:") instead of leaving it bare — allow an optional leading ">"
+    // quote marker so that case is still recognized as the reply boundary.
+    if (/^(?:>+\s*)*On .+ wrote:\s*$/.test(line)) {
       // Verify next non-empty line starts with ">" (actual quoted content)
       const nextContentLine = lines.slice(i + 1).find((l) => l.trim() !== "");
       if (nextContentLine && nextContentLine.trim().startsWith(">")) {