From 87cf0457155e750cc9b839e36cdddf045f8562c4 Mon Sep 17 00:00:00 2001
From: Oleg <ogrechko@top-sysops.ru>
Date: Mon, 10 Aug 2026 13:53:19 +0000
Subject: [PATCH] Fix garbled ticket text from HTML emails (Outlook
 VML/entities)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

extractBody() used a naive tag-stripping regex as a fallback, which
strips <style> tags but leaves their CSS content behind as visible
text, and never decodes HTML entities. Outlook's VML style block
(v\:* {behavior:url(#default#VML);} ...) and &nbsp;/&laquo; entities
were leaking straight into ticket bodies as a result.

Now uses html-to-text (already a transitive dep via mailparser,
promoted to direct) to convert the HTML part whenever present, which
correctly drops <head>/<style> content and decodes entities. Also
stopped trusting parsed.text unconditionally — some senders' plain-text
alternative isn't actually clean, so HTML is now the primary source
whenever available.
---
 package-lock.json           |  1 +
 package.json                |  1 +
 src/lib/mail/imap.ts        | 10 ++++++++--
 src/types/html-to-text.d.ts | 10 ++++++++++
 4 files changed, 20 insertions(+), 2 deletions(-)
 create mode 100644 src/types/html-to-text.d.ts

diff --git a/package-lock.json b/package-lock.json
index 5610287..660820f 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -12,6 +12,7 @@
         "better-sqlite3": "^12.11.1",
         "drizzle-orm": "^0.45.2",
         "framer-motion": "^12.4.7",
+        "html-to-text": "^10.0.0",
         "imapflow": "^1.5.0",
         "ldapjs": "^3.0.7",
         "lucide-react": "^0.545.0",
diff --git a/package.json b/package.json
index 4f16ab8..825abd4 100644
--- a/package.json
+++ b/package.json
@@ -18,6 +18,7 @@
     "better-sqlite3": "^12.11.1",
     "drizzle-orm": "^0.45.2",
     "framer-motion": "^12.4.7",
+    "html-to-text": "^10.0.0",
     "imapflow": "^1.5.0",
     "ldapjs": "^3.0.7",
     "lucide-react": "^0.545.0",
diff --git a/src/lib/mail/imap.ts b/src/lib/mail/imap.ts
index 5779e1a..988bf19 100644
--- a/src/lib/mail/imap.ts
+++ b/src/lib/mail/imap.ts
@@ -1,5 +1,6 @@
 import { ImapFlow } from "imapflow";
 import { simpleParser, type ParsedMail } from "mailparser";
+import { htmlToText } from "html-to-text";
 import { getMailboxSettings, saveMailboxSettings, disableMailboxConfig, type MailboxSettings } from "@/lib/mail/config";
 import { recordEmailInboundMessage } from "@/lib/tickets/service";
 import { saveAttachment } from "@/lib/attachments/storage";
@@ -28,11 +29,16 @@ function createClient(settings: MailboxSettings): ImapFlow {
   });
 }
 
+// Prefer converting the HTML part ourselves rather than trusting parsed.text
+// blindly — some senders (Outlook in particular) produce a text/plain
+// alternative that isn't actually clean, while html-to-text reliably drops
+// <head>/<style> content (e.g. Outlook's VML behavior: url(...) rules) and
+// decodes entities, which a naive tag-stripping regex cannot do.
 function extractBody(parsed: ParsedMail): string {
-  if (parsed.text) return parsed.text;
   if (typeof parsed.html === "string") {
-    return parsed.html.replace(/<[^>]+>/g, " ").replace(/\s+/g, " ").trim();
+    return htmlToText(parsed.html, { baseElements: { selectors: ["body"] }, wordwrap: false }).trim();
   }
+  if (parsed.text) return parsed.text;
   return "(пустое письмо)";
 }
 
diff --git a/src/types/html-to-text.d.ts b/src/types/html-to-text.d.ts
new file mode 100644
index 0000000..e192b0f
--- /dev/null
+++ b/src/types/html-to-text.d.ts
@@ -0,0 +1,10 @@
+declare module "html-to-text" {
+  interface HtmlToTextOptions {
+    wordwrap?: number | false;
+    baseElements?: { selectors?: string[] };
+    [key: string]: unknown;
+  }
+
+  export function htmlToText(html: string, options?: HtmlToTextOptions): string;
+  export function convert(html: string, options?: HtmlToTextOptions): string;
+}