@bobfrankston/rmfmail 1.1.115 → 1.1.116

package/packages/mailx-service/index.ts

@@ -12,7 +12,6 @@ import { MailxDB, parseSerial } from "@bobfrankston/mailx-store";
 const __dirname = import.meta.dirname;
 import { ImapManager } from "@bobfrankston/mailx-imap";
 import * as gsync from "./google-sync.js";
-import { sniffAndFixCharset } from "./charset.js";
 import { Store } from "@bobfrankston/mailx-store";
 import { SyncQueue } from "./sync-queue.js";
 import { Reconciler } from "./reconciler.js";
@@ -20,18 +19,10 @@ import { loadSettings, saveSettings, loadAccounts, loadAccountsAsync, saveAccoun
 import type { AccountConfig, Folder, AutocompleteRequest, AutocompleteResponse, AutocompleteSettings, AiTransformRequest, AiTransformResponse, ExtractedEvent, MailxApi } from "@bobfrankston/mailx-types";
 import { sanitizeHtml, encodeQuotedPrintable, htmlToPlainText } from "@bobfrankston/mailx-types";
 
-/** Detect mis-labeled charset and rewrite the part header to `utf-8` when
- *  the body bytes are actually valid UTF-8. PHPMailer-driven senders are
- *  the chronic offender — they declare `charset=iso-8859-1` (the PHP
- *  default) and then encode "—", "'", "…" etc. as UTF-8, which simpleParser
- *  faithfully decodes as Latin-1 and produces "â??" garbage. We only
- *  override the obviously-wrong declarations (`iso-8859-1`, `us-ascii`,
- *  `windows-1252`, `latin1`); anything else passes through unchanged.
- *  Operates byte-wise so MIME boundaries / base64 / etc. are preserved.
- *  The `utf-8` validity test rejects 0xC0–0xC1 / 0xF5–0xFF and continuation
- *  bytes out of place, which would be common in genuine Latin-1 text. */
-// sniffAndFixCharset moved to ./charset.ts so LocalStore can use it
-// without creating a circular dep with index.ts.
+// Charset normalization (sniffAndFixCharset) is owned solely by
+// mailx-store/charset.ts — the single live parse path (LocalStore.getMessage)
+// calls it there. mailx-service used to keep a duplicate copy that nothing
+// invoked; removed 2026-05-21 (economy of mechanism — one charset fixer).
 
 // parseListUnsubscribe moved to ./local-store.ts (only consumer is the
 // body-read path, which is now part of LocalStore).