payload-plugin-newsletter 0.16.9 → 0.16.10

package/CHANGELOG.md

@@ -1,3 +1,18 @@
+## [0.16.10] - 2025-01-29
+
+### Fixed
+- **Corrected Syntax Errors** - Fixed TypeScript compilation errors in broadcast afterChange hook
+  - Simplified deferred create logic to avoid nested try-catch blocks
+  - Removed duplicate error handling and extra braces
+  - Fixed all TypeScript compilation errors
+  - Maintained functionality for deferred provider sync
+
+### Technical
+- Cleaned up afterChange hook structure
+- Simplified error handling flow
+- Removed redundant code blocks
+- All TypeScript errors resolved
+
 ## [0.16.9] - 2025-01-29
 
 ### Fixed

package/dist/collections.cjs

@@ -1569,77 +1569,47 @@ var createBroadcastsCollection = (pluginConfig) => {
                   req.payload.logger.info("Still missing required fields for provider sync");
                   return doc;
                 }
-                try {
-                  req.payload.logger.info("Creating broadcast in provider (deferred from initial create)...");
-                  const htmlContent = await convertToEmailSafeHtml(doc.contentSection?.content);
-                  if (!htmlContent || htmlContent.trim() === "") {
-                    req.payload.logger.info("Skipping provider sync - content is empty after conversion");
-                    return doc;
-                  }
-                  const createData = {
-                    name: doc.subject,
-                    subject: doc.subject,
-                    preheader: doc.contentSection?.preheader,
-                    content: htmlContent,
-                    trackOpens: doc.settings?.trackOpens,
-                    trackClicks: doc.settings?.trackClicks,
-                    replyTo: doc.settings?.replyTo || providerConfig.replyTo,
-                    audienceIds: doc.audienceIds?.map((a) => a.audienceId)
-                  };
-                  req.payload.logger.info("Creating broadcast with data:", {
-                    name: createData.name,
-                    subject: createData.subject,
-                    preheader: createData.preheader || "NONE",
-                    contentLength: htmlContent ? htmlContent.length : 0,
-                    contentPreview: htmlContent ? htmlContent.substring(0, 100) + "..." : "EMPTY",
-                    apiUrl: providerConfig.apiUrl,
-                    hasToken: !!providerConfig.token
-                  });
-                  const providerBroadcast = await provider.create(createData);
-                  await req.payload.update({
-                    collection: "broadcasts",
-                    id: doc.id,
-                    data: {
-                      providerId: providerBroadcast.id,
-                      providerData: providerBroadcast.providerData
-                    },
-                    req
-                  });
-                  req.payload.logger.info(`Broadcast ${doc.id} created in provider successfully (deferred)`);
-                  return {
-                    ...doc,
-                    providerId: providerBroadcast.id,
-                    providerData: providerBroadcast.providerData
-                  };
-                } catch (error) {
-                  req.payload.logger.error("Raw error from broadcast provider (deferred create):");
-                  req.payload.logger.error(error);
-                  if (error instanceof Error) {
-                    req.payload.logger.error("Error is instance of Error:", {
-                      message: error.message,
-                      stack: error.stack,
-                      name: error.name,
-                      ...error.details,
-                      ...error.response,
-                      ...error.data,
-                      ...error.status,
-                      ...error.statusText
-                    });
-                  } else if (typeof error === "string") {
-                    req.payload.logger.error("Error is a string:", error);
-                  } else if (error && typeof error === "object") {
-                    req.payload.logger.error("Error is an object:", JSON.stringify(error, null, 2));
-                  } else {
-                    req.payload.logger.error("Unknown error type:", typeof error);
-                  }
-                  req.payload.logger.error("Failed broadcast document (deferred create):", {
-                    id: doc.id,
-                    subject: doc.subject,
-                    hasContent: !!doc.contentSection?.content,
-                    contentType: doc.contentSection?.content ? typeof doc.contentSection.content : "none"
-                  });
+                req.payload.logger.info("Creating broadcast in provider (deferred from initial create)...");
+                const htmlContent = await convertToEmailSafeHtml(doc.contentSection?.content);
+                if (!htmlContent || htmlContent.trim() === "") {
+                  req.payload.logger.info("Skipping provider sync - content is empty after conversion");
                   return doc;
                 }
+                const createData = {
+                  name: doc.subject,
+                  subject: doc.subject,
+                  preheader: doc.contentSection?.preheader,
+                  content: htmlContent,
+                  trackOpens: doc.settings?.trackOpens,
+                  trackClicks: doc.settings?.trackClicks,
+                  replyTo: doc.settings?.replyTo || providerConfig.replyTo,
+                  audienceIds: doc.audienceIds?.map((a) => a.audienceId)
+                };
+                req.payload.logger.info("Creating broadcast with data:", {
+                  name: createData.name,
+                  subject: createData.subject,
+                  preheader: createData.preheader || "NONE",
+                  contentLength: htmlContent ? htmlContent.length : 0,
+                  contentPreview: htmlContent ? htmlContent.substring(0, 100) + "..." : "EMPTY",
+                  apiUrl: providerConfig.apiUrl,
+                  hasToken: !!providerConfig.token
+                });
+                const providerBroadcast = await provider.create(createData);
+                await req.payload.update({
+                  collection: "broadcasts",
+                  id: doc.id,
+                  data: {
+                    providerId: providerBroadcast.id,
+                    providerData: providerBroadcast.providerData
+                  },
+                  req
+                });
+                req.payload.logger.info(`Broadcast ${doc.id} created in provider successfully (deferred)`);
+                return {
+                  ...doc,
+                  providerId: providerBroadcast.id,
+                  providerData: providerBroadcast.providerData
+                };
               }
               if (doc.providerId) {
                 const capabilities = provider.getCapabilities();
@@ -1684,16 +1654,32 @@ var createBroadcastsCollection = (pluginConfig) => {
                 }
               }
             } catch (error) {
+              req.payload.logger.error("Raw error from broadcast update operation:");
+              req.payload.logger.error(error);
               if (error instanceof Error) {
-                req.payload.logger.error("Failed to handle broadcast update operation:", {
+                req.payload.logger.error("Error is instance of Error:", {
                   message: error.message,
                   stack: error.stack,
                   name: error.name,
-                  ...error.details
+                  ...error.details,
+                  ...error.response,
+                  ...error.data,
+                  ...error.status,
+                  ...error.statusText
                 });
+              } else if (typeof error === "string") {
+                req.payload.logger.error("Error is a string:", error);
+              } else if (error && typeof error === "object") {
+                req.payload.logger.error("Error is an object:", JSON.stringify(error, null, 2));
               } else {
-                req.payload.logger.error("Failed to handle broadcast update operation:", error);
+                req.payload.logger.error("Unknown error type:", typeof error);
               }
+              req.payload.logger.error("Failed broadcast document (update operation):", {
+                id: doc.id,
+                subject: doc.subject,
+                hasContent: !!doc.contentSection?.content,
+                contentType: doc.contentSection?.content ? typeof doc.contentSection.content : "none"
+              });
             }
           }
           return doc;