fix(email): split two fused domain entries in the vendored list

Upstream joins mail2moldova.com/mail2molly.com and
smileyface.com/smithemail.net into one entry each, a CSV line-join
artifact that made all four read as work addresses. Splitting them keeps
the list sorted and adds a guard against a naive refresh.

Author: waleedlatif1

apps/sim/lib/messaging/email/free-email-domains.json

@@ -2590,7 +2590,8 @@
   "mail2missouri.com",
   "mail2mitch.com",
   "mail2model.com",
-  "mail2moldova.commail2molly.com",
+  "mail2moldova.com",
+  "mail2molly.com",
   "mail2mom.com",
   "mail2monaco.com",
   "mail2money.com",
@@ -3974,7 +3975,8 @@
   "smapxsmap.net",
   "smashmail.de",
   "smellrear.com",
-  "smileyface.comsmithemail.net",
+  "smileyface.com",
+  "smithemail.net",
   "smoothmail.com",
   "sms.at",
   "snail-mail.net",

apps/sim/lib/messaging/email/free-email.test.ts

@@ -2,6 +2,7 @@
  * @vitest-environment node
  */
 import { describe, expect, it } from 'vitest'
+import freeEmailDomains from '@/lib/messaging/email/free-email-domains.json'
 import { isFreeEmailDomain } from './free-email'
 
 describe('isFreeEmailDomain', () => {
@@ -24,4 +25,28 @@ describe('isFreeEmailDomain', () => {
     expect(isFreeEmailDomain('jane')).toBe(false)
     expect(isFreeEmailDomain('')).toBe(false)
   })
+
+  /**
+   * Upstream joins each of these pairs into one entry, which made all four read as work
+   * addresses. They are split in the vendored list, so this guards against a naive refresh.
+   */
+  it('returns true for the providers upstream fuses into a single entry', () => {
+    expect(isFreeEmailDomain('jane@mail2moldova.com')).toBe(true)
+    expect(isFreeEmailDomain('jane@mail2molly.com')).toBe(true)
+    expect(isFreeEmailDomain('jane@smileyface.com')).toBe(true)
+    expect(isFreeEmailDomain('jane@smithemail.net')).toBe(true)
+  })
+
+  it('carries no entry that is two domains concatenated', () => {
+    const suffixes = ['.com', '.net', '.org', '.info', '.biz']
+    const fused = freeEmailDomains.filter((entry) =>
+      suffixes.some((suffix) => {
+        const at = entry.indexOf(suffix)
+        if (at === -1 || at + suffix.length >= entry.length) return false
+        const rest = entry.slice(at + suffix.length)
+        return rest.includes('.') && rest.split('.')[0].length >= 3
+      })
+    )
+    expect(fused).toEqual(['cable.comcast.com'])
+  })
 })

apps/sim/lib/messaging/email/free-email.ts

@@ -15,6 +15,11 @@ const FREE_EMAIL_DOMAINS = new Set<string>(freeEmailDomains)
  * downloads a CDN CSV and overwrites its own `domains.json` — so the lockfile hash covers
  * the tarball but not the installed data. Refresh from
  * https://github.com/Kikobeats/free-email-domains (MIT) and review the diff.
+ *
+ * The vendored list deviates from upstream in one place: upstream fuses two pairs of
+ * domains into single entries (`mail2moldova.com`/`mail2molly.com` and
+ * `smileyface.com`/`smithemail.net`), a CSV line-join artifact that made all four read as
+ * work addresses. They are split here, so re-apply that split after any refresh.
  */
 export function isFreeEmailDomain(email: string): boolean {
   const domain = email.split('@')[1]?.toLowerCase()