payload-plugin-newsletter 0.16.1 → 0.16.3

package/CHANGELOG.md

@@ -1,3 +1,27 @@
+## [0.16.3] - 2025-07-27
+
+### Improved
+- **Enhanced Error Logging** - Improved error logging for broadcast operations
+  - Now logs full error details including message, stack trace, and any additional error properties
+  - Helps diagnose API connection issues, authentication failures, and validation errors
+  - Structured error logging makes it easier to identify root causes
+  - Applied to all broadcast hooks: create, update, delete, and send
+
+## [0.16.2] - 2025-07-27
+
+### Fixed
+- **Configuration Consistency** - Fixed broadcast operations to read from Newsletter Settings collection
+  - Broadcast create/update/delete operations now check Newsletter Settings first before falling back to env vars
+  - This matches the behavior of email operations (magic links, welcome emails)
+  - Resolves issue where broadcasts failed when env vars were missing despite settings being configured
+  - Added `getBroadcastConfig` utility for consistent configuration retrieval
+
+### Added
+- **Configuration Utilities** - New utilities for consistent provider configuration
+  - `getBroadcastConfig` - Gets Broadcast provider config from settings or env vars
+  - `getResendConfig` - Gets Resend provider config from settings or env vars
+  - Both utilities handle errors gracefully with fallback to env vars
+
 ## [0.16.1] - 2025-07-27
 
 ### Fixed

package/dist/collections.cjs

@@ -1162,6 +1162,29 @@ function wrapInEmailTemplate(content, preheader) {
 </html>`;
 }
 
+// src/utils/getBroadcastConfig.ts
+async function getBroadcastConfig(req, pluginConfig) {
+  try {
+    const settings = await req.payload.findGlobal({
+      slug: pluginConfig.settingsSlug || "newsletter-settings",
+      req
+    });
+    if (settings?.provider === "broadcast" && settings?.broadcastSettings) {
+      return {
+        apiUrl: settings.broadcastSettings.apiUrl || pluginConfig.providers?.broadcast?.apiUrl || "",
+        token: settings.broadcastSettings.token || pluginConfig.providers?.broadcast?.token || "",
+        fromAddress: settings.fromAddress || pluginConfig.providers?.broadcast?.fromAddress || "",
+        fromName: settings.fromName || pluginConfig.providers?.broadcast?.fromName || "",
+        replyTo: settings.replyTo || pluginConfig.providers?.broadcast?.replyTo
+      };
+    }
+    return pluginConfig.providers?.broadcast || null;
+  } catch (error) {
+    req.payload.logger.error("Failed to get broadcast config from settings:", error);
+    return pluginConfig.providers?.broadcast || null;
+  }
+}
+
 // src/collections/Broadcasts.ts
 var createBroadcastsCollection = (pluginConfig) => {
   const hasProviders = !!(pluginConfig.providers?.broadcast || pluginConfig.providers?.resend);
@@ -1402,9 +1425,9 @@ var createBroadcastsCollection = (pluginConfig) => {
         async ({ doc, operation, req }) => {
           if (!hasProviders || operation !== "create") return doc;
           try {
-            const providerConfig = pluginConfig.providers?.broadcast;
-            if (!providerConfig) {
-              req.payload.logger.error("Broadcast provider not configured");
+            const providerConfig = await getBroadcastConfig(req, pluginConfig);
+            if (!providerConfig || !providerConfig.token) {
+              req.payload.logger.error("Broadcast provider not configured in Newsletter Settings or environment variables");
               return doc;
             }
             const { BroadcastApiProvider: BroadcastApiProvider2 } = await Promise.resolve().then(() => (init_broadcast2(), broadcast_exports));
@@ -1436,7 +1459,17 @@ var createBroadcastsCollection = (pluginConfig) => {
               providerData: providerBroadcast.providerData
             };
           } catch (error) {
-            req.payload.logger.error("Failed to create broadcast in provider:", error);
+            if (error instanceof Error) {
+              req.payload.logger.error("Failed to create broadcast in provider:", {
+                message: error.message,
+                stack: error.stack,
+                name: error.name,
+                // If it's a BroadcastProviderError, it might have additional details
+                ...error.details
+              });
+            } else {
+              req.payload.logger.error("Failed to create broadcast in provider:", error);
+            }
             return doc;
           }
         },
@@ -1450,13 +1483,13 @@ var createBroadcastsCollection = (pluginConfig) => {
               return doc;
             }
             try {
-              const broadcastConfig = pluginConfig.providers?.broadcast;
+              const broadcastConfig = await getBroadcastConfig(req, pluginConfig);
               const resendConfig = pluginConfig.providers?.resend;
               if (!broadcastConfig && !resendConfig) {
-                req.payload.logger.error("No provider configured for sending");
+                req.payload.logger.error("No provider configured for sending in Newsletter Settings or environment variables");
                 return doc;
               }
-              if (broadcastConfig) {
+              if (broadcastConfig && broadcastConfig.token) {
                 const { BroadcastApiProvider: BroadcastApiProvider2 } = await Promise.resolve().then(() => (init_broadcast2(), broadcast_exports));
                 const provider = new BroadcastApiProvider2(broadcastConfig);
                 await provider.send(doc.providerId);
@@ -1472,7 +1505,17 @@ var createBroadcastsCollection = (pluginConfig) => {
               });
               req.payload.logger.info(`Broadcast ${doc.id} sent successfully`);
             } catch (error) {
-              req.payload.logger.error(`Failed to send broadcast ${doc.id}:`, error);
+              if (error instanceof Error) {
+                req.payload.logger.error(`Failed to send broadcast ${doc.id}:`, {
+                  message: error.message,
+                  stack: error.stack,
+                  name: error.name,
+                  // If it's a BroadcastProviderError, it might have additional details
+                  ...error.details
+                });
+              } else {
+                req.payload.logger.error(`Failed to send broadcast ${doc.id}:`, error);
+              }
               await req.payload.update({
                 collection: "broadcasts",
                 id: doc.id,
@@ -1491,9 +1534,9 @@ var createBroadcastsCollection = (pluginConfig) => {
         async ({ data, originalDoc, operation, req }) => {
           if (!hasProviders || !originalDoc?.providerId || operation !== "update") return data;
           try {
-            const providerConfig = pluginConfig.providers?.broadcast;
-            if (!providerConfig) {
-              req.payload.logger.error("Broadcast provider not configured");
+            const providerConfig = await getBroadcastConfig(req, pluginConfig);
+            if (!providerConfig || !providerConfig.token) {
+              req.payload.logger.error("Broadcast provider not configured in Newsletter Settings or environment variables");
               return data;
             }
             const { BroadcastApiProvider: BroadcastApiProvider2 } = await Promise.resolve().then(() => (init_broadcast2(), broadcast_exports));
@@ -1527,7 +1570,17 @@ var createBroadcastsCollection = (pluginConfig) => {
               await provider.update(originalDoc.providerId, updates);
             }
           } catch (error) {
-            req.payload.logger.error("Failed to update broadcast in provider:", error);
+            if (error instanceof Error) {
+              req.payload.logger.error("Failed to update broadcast in provider:", {
+                message: error.message,
+                stack: error.stack,
+                name: error.name,
+                // If it's a BroadcastProviderError, it might have additional details
+                ...error.details
+              });
+            } else {
+              req.payload.logger.error("Failed to update broadcast in provider:", error);
+            }
           }
           return data;
         }
@@ -1537,9 +1590,9 @@ var createBroadcastsCollection = (pluginConfig) => {
         async ({ doc, req }) => {
           if (!hasProviders || !doc?.providerId) return doc;
           try {
-            const providerConfig = pluginConfig.providers?.broadcast;
-            if (!providerConfig) {
-              req.payload.logger.error("Broadcast provider not configured");
+            const providerConfig = await getBroadcastConfig(req, pluginConfig);
+            if (!providerConfig || !providerConfig.token) {
+              req.payload.logger.error("Broadcast provider not configured in Newsletter Settings or environment variables");
               return doc;
             }
             const { BroadcastApiProvider: BroadcastApiProvider2 } = await Promise.resolve().then(() => (init_broadcast2(), broadcast_exports));
@@ -1549,7 +1602,17 @@ var createBroadcastsCollection = (pluginConfig) => {
               await provider.delete(doc.providerId);
             }
           } catch (error) {
-            req.payload.logger.error("Failed to delete broadcast from provider:", error);
+            if (error instanceof Error) {
+              req.payload.logger.error("Failed to delete broadcast from provider:", {
+                message: error.message,
+                stack: error.stack,
+                name: error.name,
+                // If it's a BroadcastProviderError, it might have additional details
+                ...error.details
+              });
+            } else {
+              req.payload.logger.error("Failed to delete broadcast from provider:", error);
+            }
           }
           return doc;
         }