@hogsend/plugin-resend 0.12.1 → 0.12.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hogsend/plugin-resend",
-  "version": "0.12.1",
+  "version": "0.12.2",
   "type": "module",
   "license": "MIT",
   "repository": {
@@ -24,8 +24,8 @@
   "dependencies": {
     "resend": "^6.12.3",
     "svix": "^1.94.0",
-    "@hogsend/core": "^0.12.1",
-    "@hogsend/email": "^0.12.1"
+    "@hogsend/core": "^0.12.2",
+    "@hogsend/email": "^0.12.2"
   },
   "devDependencies": {
     "@types/node": "^22.0.0",

package/src/__tests__/send.test.ts

@@ -118,6 +118,38 @@ describe("sendEmail", () => {
     expect(arg.tags).toEqual(tags);
   });
 
+  it("sanitizes tag names/values to Resend's allowed charset", async () => {
+    const sendFn = vi.fn().mockResolvedValue({
+      data: { id: "resend_123" },
+      error: null,
+    });
+    const client = mockResendClient({ sendFn });
+
+    await sendEmail({
+      client,
+      options: {
+        from: "test@hogsend.com",
+        to: "user@example.com",
+        subject: "Test",
+        html: HTML,
+        // The engine's neutral tags carry journey names and slashed template
+        // keys — Resend only allows [A-Za-z0-9_-].
+        tags: [
+          { name: "journeyId", value: "Docs Subscriber" },
+          { name: "templateKey", value: "docs/welcome" },
+        ],
+      },
+    });
+
+    const arg = sendFn.mock.calls[0]?.[0] as {
+      tags?: Array<{ name: string; value: string }>;
+    };
+    expect(arg.tags).toEqual([
+      { name: "journeyId", value: "Docs-Subscriber" },
+      { name: "templateKey", value: "docs-welcome" },
+    ]);
+  });
+
   it("omits Resend tags when none are set", async () => {
     const sendFn = vi.fn().mockResolvedValue({
       data: { id: "resend_123" },

package/src/send.ts

@@ -9,6 +9,21 @@ import type { BatchEmailItem, SendEmailOptions, SendResult } from "./types.js";
 
 const BATCH_CHUNK_SIZE = 100;
 
+/**
+ * Resend rejects tag names/values outside ASCII letters, numbers, underscores
+ * and dashes. The engine's neutral tags carry journey names ("Docs Subscriber")
+ * and template keys ("docs/welcome"), so map anything else to "-" rather than
+ * failing the whole send.
+ */
+function sanitizeTags(
+  tags: SendEmailOptions["tags"],
+): SendEmailOptions["tags"] {
+  return tags?.map((tag) => ({
+    name: tag.name.replace(/[^a-zA-Z0-9_-]/g, "-"),
+    value: tag.value.replace(/[^a-zA-Z0-9_-]/g, "-"),
+  }));
+}
+
 function isRetryableStatusCode(statusCode: number): boolean {
   return statusCode === 429 || statusCode >= 500;
 }
@@ -109,8 +124,9 @@ export async function sendEmail(args: {
       cc: options.cc,
       bcc: options.bcc,
       scheduledAt: options.scheduledAt,
-      // Resend accepts neutral `{name,value}[]` tags natively — pass them through.
-      tags: options.tags,
+      // Resend accepts neutral `{name,value}[]` tags natively — sanitized to
+      // its allowed charset (see sanitizeTags).
+      tags: sanitizeTags(options.tags),
       headers: options.headers,
     });
 
@@ -182,8 +198,8 @@ async function sendBatchChunk(
         replyTo: email.replyTo,
         cc: email.cc,
         bcc: email.bcc,
-        // Resend accepts neutral `{name,value}[]` tags natively.
-        tags: email.tags,
+        // Resend accepts neutral `{name,value}[]` tags — sanitized to its charset.
+        tags: sanitizeTags(email.tags),
         headers: email.headers,
       })),
     );