@aglyn/plugins-marketing 1.0.0-beta.218 → 1.0.0-beta.220

package/src/lib/server/campaign-send.js

@@ -14,7 +14,7 @@ import { _ as _extends } from "@swc/helpers/_/_extends";
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
  * See the License for the specific language governing permissions and
  * limitations under the License.
- */ import { checkQuota, contactMatchesSegment, isEmailTopicId, DEFAULT_CAMPAIGN_TOPIC_ID, readMarketingBasis, resolveMarketingConsentPolicy, splitByMarketingConsent, createResourceUid, decodeStoredNodes, emailStarterSendBlock, resolveBrandingProfile, siteLockdownFromDocs, visibleToHost } from "@aglyn/aglyn/server";
+ */ import { checkQuota, contactMatchesSegment, isSubscriptionTopicId, DEFAULT_SUBSCRIPTION_TOPIC_ID, readMarketingBasis, resolveMarketingConsentPolicy, splitByMarketingConsent, createResourceUid, decodeStoredNodes, emailStarterSendBlock, resolveBrandingProfile, seenOnlyThroughGrant, siteLockdownFromDocs, visibleToHost } from "@aglyn/aglyn/server";
 import { renderRecipientEmail } from "@aglyn/aglyn/app-utils/recipient-email-render";
 import { composeHostComponentNodes } from "@aglyn/aglyn/app-utils/load-referenced-components";
 import { assignExperimentVariant } from "../model/index.js";
@@ -70,7 +70,9 @@ import { campaignSendSearchFields } from "../model/campaign-list-query.js";
  * against a harness that cannot serve it.
  */ import { claimOrgEmailSendDay, orgAgeDays, readSenderReputation, reconcileOrgEmailSendDay, recordCampaignAccepted, resolveOrgEmailRamp } from "@aglyn/tenant-data-admin/server/email-sender-reputation";
 import { createHash } from "crypto";
-import { EMAIL_MAX_AUDIENCE_PER_SEND, EMAIL_MAX_RECIPIENTS_PER_SEND, campaignBatchPlan, createProviderRequestPacer, effectiveReputationPolicy, HOST_SENDERS_COLLECTION, isEmailConfigured, rateLimitedRetryAtMs, readHostSender, sendEmail, sendingIdentityRefusal, sentAsStamp } from "@aglyn/shared-util-email";
+import { EMAIL_MAX_AUDIENCE_PER_SEND, EMAIL_MAX_RECIPIENTS_PER_SEND, createProviderRequestPacer, effectiveReputationPolicy, HOST_SENDERS_COLLECTION, isEmailConfigured, rateLimitedRetryAtMs, readHostSender, sendEmail, sendingIdentityRefusal, sentAsStamp } from "@aglyn/shared-util-email";
+import { campaignBatchPlan } from "./campaign-batch-plan.js";
+import { stampRecordEmailReach } from "@aglyn/aglyn/plugin-manager/plugin-record-email-state";
 /*
  * The List-Unsubscribe setting sequences share (AGL-3307), from its LEAF
  * module: the specs that reach this file spread the real barrel under their
@@ -449,10 +451,10 @@ export async function performCampaignSend(options) {
    * two different parameter tuples — see `signedSubject` in the email plugin's
    * `unsubscribe-link.ts`. Refused at the point the topic ENTERS the send, so
    * the link that leaves it is unambiguous by construction.
-   */ if (options.topicId && !isEmailTopicId(options.topicId)) {
+   */ if (options.topicId && !isSubscriptionTopicId(options.topicId)) {
         throw new CampaignSendError('Invalid topicId', 400);
     }
-    const topicId = options.topicId || DEFAULT_CAMPAIGN_TOPIC_ID;
+    const topicId = options.topicId || DEFAULT_SUBSCRIPTION_TOPIC_ID;
     /**
    * Whether this send ADDS to an email that already exists, rather than
    * starting one.
@@ -714,7 +716,9 @@ export async function performCampaignSend(options) {
         // MAIL a sibling brand's leads under this site's campaign.
         const leads = await sweepAudience((await orgDataQueryForHost(hostId, 'leads')).query);
         audienceTruncated = leads.truncated;
-        recipients = leads.docs.map((doc)=>{
+        // Held, not merely seen: a lead another site SHARED with this one
+        // (AGL-3336) is visible here and was never this site's audience.
+        recipients = leads.docs.filter((doc)=>!seenOnlyThroughGrant(doc.data(), hostId)).map((doc)=>{
             var _doc_get;
             const email = String((_doc_get = doc.get('email')) != null ? _doc_get : '');
             collectName(email, doc.get('name'));
@@ -780,7 +784,9 @@ export async function performCampaignSend(options) {
      * reports a small `audienceSize` with `audienceTruncated` set, reading as
      * "at least this many, we stopped counting" and not as a total.
      */ audienceTruncated = contacts.truncated;
-        recipients = contacts.docs.filter((doc)=>{
+        recipients = contacts.docs// Held, not merely seen — a shared contact is no site's audience but
+        // its holders' (AGL-3336).
+        .filter((doc)=>!seenOnlyThroughGrant(doc.data(), hostId)).filter((doc)=>{
             var _doc_get, _doc_get1;
             return contactMatchesSegment({
                 tags: (_doc_get = doc.get('tags')) != null ? _doc_get : [],
@@ -1127,14 +1133,21 @@ export async function performCampaignSend(options) {
      * merchant who cannot send and cannot find out why will open a ticket
      * that says the product is broken.
      */ if (reputation.blocked) {
+            var _ref4;
+            var _orgForHost_org4;
             // The merchant is told by the refusal; the operator, once a day per
             // workspace, so a tripped breaker on a shared domain is looked at
             // before it is reinstated (AGL-3377).
             await raiseOperatorAlert(MARKETING_REPUTATION_BREAKER, {
                 dedupeKey: orgId,
+                // The findings in numbers, and the workspace by name off the org doc
+                // this send already holds. `reputation.reason` is the merchant's
+                // refusal, written to them, so it is not staff's copy.
                 context: {
                     orgId,
-                    reason: reputation.reason
+                    orgName: String((_ref4 = orgForHost == null ? void 0 : (_orgForHost_org4 = orgForHost.org) == null ? void 0 : _orgForHost_org4['name']) != null ? _ref4 : '').trim(),
+                    windowDays: reputation.windowDays,
+                    detail: reputation.findings.filter((finding)=>finding.actionable).map((finding)=>finding.detail).join(' ')
                 },
                 orgId
             });
@@ -1472,14 +1485,14 @@ export async function performCampaignSend(options) {
    * back on the clock and this screen finds it released — and actioning the
    * row cancels it.
    */ if (!options.followUp) {
-        var _ref4, _hostSnapshot_data, _options_fromName, _sendingIdentity_from, _options_replyTo, _options_preheader, _options_plainText, _experiment_variants;
+        var _ref5, _hostSnapshot_data, _options_fromName, _sendingIdentity_from, _options_replyTo, _options_preheader, _options_plainText, _experiment_variants;
         const sendRef = sends.doc(campaignId);
         const screened = await screenOutboundSend({
             kind: 'campaign',
             path: sendRef.path,
             hostId,
             orgId,
-            org: (_ref4 = orgForHost == null ? void 0 : orgForHost.org) != null ? _ref4 : null,
+            org: (_ref5 = orgForHost == null ? void 0 : orgForHost.org) != null ? _ref5 : null,
             host: (_hostSnapshot_data = hostSnapshot.data()) != null ? _hostSnapshot_data : null,
             subject,
             fromName: (_options_fromName = options.fromName) != null ? _options_fromName : null,
@@ -1804,7 +1817,7 @@ export async function performCampaignSend(options) {
                 siteBase,
                 hostId,
                 email,
-                campaignId,
+                sendId: campaignId,
                 topicId,
                 secret: unsubscribeSecret
             };
@@ -2066,6 +2079,14 @@ export async function performCampaignSend(options) {
    * follow-up, which is also safe but loses the feature over a transient.
    */ if (options.recordCampaign !== false) {
         await recordCampaignReach(sends.doc(campaignId), reached);
+        // The record system hears who it reached, so a lead nobody had touched
+        // moves to Nurturing (AGL-3446). A test send reached nobody's lead, so it
+        // is left out with the reach record. Never throws.
+        await stampRecordEmailReach({
+            orgId,
+            hostId,
+            emails: reached
+        });
         /*
      * And who it decided NOT to mail, under a field of its own.
      *
@@ -2860,7 +2881,7 @@ function storedSendOptionsFrom(snapshot, hostId, senderUid, followUp) {
    * An empty value is not an error: it means "the composer did not say", which
    * `performCampaignSend` resolves to the default topic.
    */ const topicId = String((_ref10 = (_req_body14 = req.body) == null ? void 0 : _req_body14.topicId) != null ? _ref10 : '');
-    if (topicId && !isEmailTopicId(topicId)) {
+    if (topicId && !isSubscriptionTopicId(topicId)) {
         return res.status(400).json({
             error: 'Unknown topic'
         });