brand-manager-worker 0.1.0 → 0.1.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "brand-manager-worker",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "The Goose Tools brand-deal worker — your computer reads your brand email and drafts replies in your voice for goosetools.com, using your own Claude account. Drafts only; it never sends.",
   "repository": {
     "type": "git",

package/worker/agent.js

@@ -12,8 +12,21 @@ import { BRAND_DIR } from "./paths.js";
 // Automated / no-reply senders never send brand deals — skip them WITHOUT
 // spending a Claude call. Conservative on purpose: real brands/agencies use
 // human addresses.
-const AUTOMATED_SENDER =
-  /no-?reply|do-?not-?reply|noreply|mailer-daemon|postmaster|notifications?@|accounts\.google\.com|@.*\.(beehiiv|substack)\.com|@e\.|@email\.|@mail\./i;
+const AUTOMATED_SENDER = new RegExp(
+  [
+    // Classic no-reply shapes.
+    "no-?reply", "do-?not-?reply", "noreply", "mailer-daemon", "postmaster",
+    "notifications?@", "accounts\\.google\\.com",
+    "@.*\\.(beehiiv|substack)\\.com",
+    // Bulk-mail SUBDOMAINS. Retail blasts send from things like
+    // news.emailmarket.shein.com, which slipped past the old @mail./@email.
+    // patterns and cost a full Claude call each — 7 SHEIN cart reminders in
+    // one scan. Matched as a leading label so a real company domain like
+    // news-corp.com or mailchimp.com (an agency's own address) is untouched.
+    "@(e|news|newsletter|email|emails|mail|mailer|marketing|promo|promos|campaign|campaigns|updates|offers|deals|shop|store|hello|info)[.-]",
+  ].join("|"),
+  "i",
+);
 
 export const isAutomated = (from) => AUTOMATED_SENDER.test(from);
 

package/worker/handlers.js

@@ -35,18 +35,6 @@ const noGmail = () => ({
   error: "Gmail isn't connected — connect your email on goosetools.com/dashboard/brand-manager",
 });
 
-/** Thread metadata rows for the mirror, from fetched threads. */
-const threadMeta = (threads) =>
-  threads.map((t) => ({
-    threadId: t.id,
-    accountEmail: t.account,
-    fromName: t.last.from.replace(/\s*<[^>]*>\s*/, "").trim() || null,
-    fromEmail: (t.last.from.match(/<([^>]+)>/)?.[1] ?? t.last.from).trim() || null,
-    subject: t.subject,
-    lastMessageAt: t.last.date,
-    hasUnsentDraft: t.hasUnsentDraft,
-  }));
-
 /** Resolve decision attachment paths ("stats/screenshots/x.png") under BRAND_DIR. */
 const resolveAttachments = (paths = []) =>
   paths.map((p) => (p.startsWith("/") ? p : join(BRAND_DIR, p))).filter((p) => existsSync(p));
@@ -73,7 +61,7 @@ async function draftForThread(job, thread) {
     return {
       ok: true,
       summary: `skip — ${decision.summary}`,
-      mirror: buildMirror({ threads: threadMeta([thread]) }),
+      mirror: buildMirror({ threads: [thread] }),
     };
   }
 
@@ -105,7 +93,7 @@ async function draftForThread(job, thread) {
     summary: decision.summary,
     resultBody: decision.body ?? "",
     flags: decision.flags ?? [],
-    mirror: buildMirror({ threads: threadMeta([thread]) }),
+    mirror: buildMirror({ threads: [thread] }),
   };
 }
 
@@ -177,7 +165,7 @@ export async function runScan(job) {
     ok: true,
     summary: lines.length ? lines.join(" · ").slice(0, 1900) : "No new brand mail needing a draft.",
     flags,
-    mirror: buildMirror({ threads: threadMeta(threads) }),
+    mirror: buildMirror({ threads }),
   };
 }
 

package/worker/mirror.js

@@ -74,15 +74,22 @@ export function collectKnowledge() {
  * machine is defined in exactly one place, right here.
  */
 export function collectThreads(threads) {
-  return threads.map((t) => ({
-    threadId: t.id,
-    accountEmail: t.account ?? null,
-    fromName: t.fromName ?? null,
-    fromEmail: t.fromEmail ?? null,
-    subject: t.subject ?? null,
-    lastMessageAt: t.lastMessageAt ?? null,
-    hasUnsentDraft: Boolean(t.hasUnsentDraft),
-  }));
+  return threads.map((t) => {
+    // Raw Thread objects in, mirror rows out. This is the ONLY place that
+    // shape is decided — callers must not pre-map, or threadId silently
+    // becomes undefined and the server drops the row (which is exactly what
+    // happened: deals and knowledge mirrored, threads never did).
+    const from = t.last?.from ?? "";
+    return {
+      threadId: t.id,
+      accountEmail: t.account ?? null,
+      fromName: from.replace(/\s*<[^>]*>\s*/, "").replace(/^"|"$/g, "").trim() || null,
+      fromEmail: (from.match(/<([^>]+)>/)?.[1] ?? from).trim() || null,
+      subject: t.subject ?? null,
+      lastMessageAt: t.last?.date ?? null,
+      hasUnsentDraft: Boolean(t.hasUnsentDraft),
+    };
+  });
 }
 
 /** The full mirror payload for a complete() call. */