payload-plugin-newsletter 0.25.4 → 0.25.6

package/CHANGELOG.md

@@ -1,3 +1,20 @@
+## [0.25.6] - 2025-08-19
+
+### Changed
+- Enhanced error logging for Broadcast API provider to diagnose connection issues
+  - Added detailed response logging for successful and failed API calls
+  - Improved error messages to show actual API responses
+  - Added logging for GET requests after broadcast creation
+  - Better JSON parsing error handling with response body details
+
+## [0.25.5] - 2025-08-18
+
+### Fixed
+- Fixed "document not found" error when creating broadcasts by removing unnecessary update call during creation
+  - The afterChange hook now properly returns the modified document instead of trying to update it
+  - Added debug logging to help diagnose provider sync issues
+  - Ensures smooth broadcast creation without transaction conflicts
+
 ## [0.25.4] - 2025-08-18
 
 ### Fixed

package/dist/{broadcast-VMCYSZRY.js → broadcast-7DI2WOB3.js}

@@ -1,6 +1,6 @@
 import {
   BroadcastApiProvider
-} from "./chunk-XVMYJQRQ.js";
+} from "./chunk-LLPN5SQN.js";
 export {
   BroadcastApiProvider
 };

package/dist/{chunk-XVMYJQRQ.js → chunk-LLPN5SQN.js}

@@ -157,6 +157,7 @@ var BroadcastApiProvider = class extends BaseBroadcastProvider {
   }
   async get(id) {
     try {
+      console.log("[BroadcastApiProvider] Getting broadcast with ID:", id);
       const response = await fetch(`${this.apiUrl}/api/v1/broadcasts/${id}`, {
         method: "GET",
         headers: {
@@ -231,7 +232,18 @@ var BroadcastApiProvider = class extends BaseBroadcastProvider {
         }
         throw new Error(`Broadcast API error: ${response.status} - ${errorText}`);
       }
-      const result = await response.json();
+      const responseText = await response.text();
+      console.log("[BroadcastApiProvider] Success response body:", responseText);
+      let result;
+      try {
+        result = JSON.parse(responseText);
+      } catch (e) {
+        throw new Error(`Failed to parse response as JSON: ${responseText}`);
+      }
+      console.log("[BroadcastApiProvider] Parsed result:", result);
+      if (!result.id) {
+        throw new Error(`Response missing expected 'id' field: ${JSON.stringify(result)}`);
+      }
       return this.get(result.id.toString());
     } catch (error) {
       if (error instanceof BroadcastProviderError) throw error;

package/dist/collections.cjs

@@ -229,6 +229,7 @@ var init_broadcast2 = __esm({
       }
       async get(id) {
         try {
+          console.log("[BroadcastApiProvider] Getting broadcast with ID:", id);
           const response = await fetch(`${this.apiUrl}/api/v1/broadcasts/${id}`, {
             method: "GET",
             headers: {
@@ -303,7 +304,18 @@ var init_broadcast2 = __esm({
             }
             throw new Error(`Broadcast API error: ${response.status} - ${errorText}`);
           }
-          const result = await response.json();
+          const responseText = await response.text();
+          console.log("[BroadcastApiProvider] Success response body:", responseText);
+          let result;
+          try {
+            result = JSON.parse(responseText);
+          } catch (e) {
+            throw new Error(`Failed to parse response as JSON: ${responseText}`);
+          }
+          console.log("[BroadcastApiProvider] Parsed result:", result);
+          if (!result.id) {
+            throw new Error(`Response missing expected 'id' field: ${JSON.stringify(result)}`);
+          }
           return this.get(result.id.toString());
         } catch (error) {
           if (error instanceof BroadcastProviderError) throw error;
@@ -2294,6 +2306,12 @@ var createBroadcastsCollection = (pluginConfig) => {
           if (!hasProviders) return doc;
           if (operation === "create") {
             try {
+              req.payload.logger.info("Broadcast afterChange create hook - doc info:", {
+                docId: doc.id,
+                docIdType: typeof doc.id,
+                hasDoc: !!doc,
+                operation
+              });
               const providerConfig = await getBroadcastConfig(req, pluginConfig);
               if (!providerConfig || !providerConfig.token) {
                 req.payload.logger.error("Broadcast provider not configured in Newsletter Settings or environment variables");
@@ -2329,16 +2347,11 @@ var createBroadcastsCollection = (pluginConfig) => {
                 hasActualContent: !!doc.contentSection?.content
               });
               const providerBroadcast = await provider.create(createData);
-              await req.payload.update({
-                collection: "broadcasts",
-                id: doc.id,
-                data: {
-                  providerId: providerBroadcast.id,
-                  externalId: providerBroadcast.id,
-                  // Set externalId to match providerId for webhook lookup
-                  providerData: providerBroadcast.providerData
-                },
-                req
+              req.payload.logger.info("Provider broadcast created:", {
+                providerBroadcastId: providerBroadcast.id,
+                providerBroadcastIdType: typeof providerBroadcast.id,
+                docId: doc.id,
+                docIdType: typeof doc.id
               });
               req.payload.logger.info(`Broadcast ${doc.id} created in provider with ID ${providerBroadcast.id}`);
               return {
@@ -2349,7 +2362,18 @@ var createBroadcastsCollection = (pluginConfig) => {
                 providerData: providerBroadcast.providerData
               };
             } catch (error) {
-              req.payload.logger.error("Failed to create broadcast in provider during initial creation:", error);
+              req.payload.logger.error("Failed to create broadcast in provider during initial creation:");
+              if (error instanceof Error) {
+                req.payload.logger.error("Error details:", {
+                  message: error.message,
+                  stack: error.stack,
+                  name: error.name,
+                  // Check for any additional properties
+                  ...error
+                });
+              } else {
+                req.payload.logger.error("Raw error:", error);
+              }
               return doc;
             }
           }