payload-plugin-newsletter 0.25.6 → 0.25.8

package/CHANGELOG.md

@@ -1,3 +1,21 @@
+## [0.25.8] - 2025-08-19
+
+### Fixed
+- Added check to prevent double broadcast creation if externalId/providerId already exists
+- Enhanced logging to show document status and existing IDs during creation
+- Added logging for GET broadcast response to diagnose sync issues
+- Helps identify root cause of duplicate broadcast creation
+
+### Changed
+- More descriptive hook logging to distinguish between sync and publish hooks
+
+## [0.25.7] - 2025-08-19
+
+### Fixed
+- Fixed TypeScript type errors in error handling code
+- Resolved all linting errors (warnings remain but don't block the build)
+- Simplified error logging to avoid type conversion issues
+
 ## [0.25.6] - 2025-08-19
 
 ### Changed

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

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

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

@@ -177,6 +177,7 @@ var BroadcastApiProvider = class extends BaseBroadcastProvider {
         throw new Error(`Broadcast API error: ${response.status} - ${error}`);
       }
       const broadcast = await response.json();
+      console.log("[BroadcastApiProvider] GET response:", broadcast);
       return this.transformBroadcastFromApi(broadcast);
     } catch (error) {
       if (error instanceof BroadcastProviderError) throw error;
@@ -237,7 +238,7 @@ var BroadcastApiProvider = class extends BaseBroadcastProvider {
       let result;
       try {
         result = JSON.parse(responseText);
-      } catch (e) {
+      } catch {
         throw new Error(`Failed to parse response as JSON: ${responseText}`);
       }
       console.log("[BroadcastApiProvider] Parsed result:", result);

package/dist/collections.cjs

@@ -249,6 +249,7 @@ var init_broadcast2 = __esm({
             throw new Error(`Broadcast API error: ${response.status} - ${error}`);
           }
           const broadcast = await response.json();
+          console.log("[BroadcastApiProvider] GET response:", broadcast);
           return this.transformBroadcastFromApi(broadcast);
         } catch (error) {
           if (error instanceof BroadcastProviderError) throw error;
@@ -309,7 +310,7 @@ var init_broadcast2 = __esm({
           let result;
           try {
             result = JSON.parse(responseText);
-          } catch (e) {
+          } catch {
             throw new Error(`Failed to parse response as JSON: ${responseText}`);
           }
           console.log("[BroadcastApiProvider] Parsed result:", result);
@@ -2306,12 +2307,22 @@ var createBroadcastsCollection = (pluginConfig) => {
           if (!hasProviders) return doc;
           if (operation === "create") {
             try {
-              req.payload.logger.info("Broadcast afterChange create hook - doc info:", {
+              req.payload.logger.info("Broadcast afterChange CREATE hook (sync) - doc info:", {
                 docId: doc.id,
                 docIdType: typeof doc.id,
                 hasDoc: !!doc,
-                operation
+                operation,
+                status: doc._status,
+                hasExternalId: !!doc.externalId,
+                hasProviderId: !!doc.providerId
               });
+              if (doc.externalId || doc.providerId) {
+                req.payload.logger.info("Broadcast already has provider IDs, skipping creation", {
+                  externalId: doc.externalId,
+                  providerId: doc.providerId
+                });
+                return doc;
+              }
               const providerConfig = await getBroadcastConfig(req, pluginConfig);
               if (!providerConfig || !providerConfig.token) {
                 req.payload.logger.error("Broadcast provider not configured in Newsletter Settings or environment variables");
@@ -2367,9 +2378,7 @@ var createBroadcastsCollection = (pluginConfig) => {
                 req.payload.logger.error("Error details:", {
                   message: error.message,
                   stack: error.stack,
-                  name: error.name,
-                  // Check for any additional properties
-                  ...error
+                  name: error.name
                 });
               } else {
                 req.payload.logger.error("Raw error:", error);