@sendinel/mcp-server 1.0.207 → 1.0.208

package/build/lib/action-approvals.js

@@ -159,7 +159,7 @@ function broadcastSummary(d) {
     const audience = d.segment_name ? `segment "${d.segment_name}"` : (d.audience_label ? String(d.audience_label) : 'the chosen audience');
     const when = d.send_at ? ` at ${d.send_at}` : '';
     const excerpt = d.body_excerpt ? ` — "${d.body_excerpt}"` : '';
-    return `Send "${d.subject}" to ${audience}${count ? ` (${count})` : ''}${when}${excerpt}`;
+    return `Send "${d.subject}" to ${audience}${count ? ` (${count})` : ''}${when}${excerpt}${d.one_send_summary ? ` — ${d.one_send_summary}` : ''}`;
 }
 /**
  * launch_campaign approvals carry the written emails in params.generated (#3669), so the summary

package/build/tools/campaigns.js

@@ -464,18 +464,15 @@ async function validateBranchTarget(campaignId, fromStepId, action, targetStepId
     }
     return null;
 }
-/**
- * Create, enroll and activate a one-email broadcast (#3669). Shared by broadcast_to_segment and
- * send_broadcast so both run the same steps. Enrolls exactly the given contact ids.
- */
-async function executeBroadcast(input) {
+/** Read-only match shared by approval previews and execution. */
+async function findBroadcastOneSend(input) {
     // #3749 One Send: the same email (site + subject + body) joins its existing active/completed
     // campaign from the last 90 days instead of creating another. Same rule as the dashboard helper.
     const contentKey = computeSendContentKey({ siteId: input.siteId, subject: input.subject, bodyHtml: input.bodyHtml });
     const since = new Date(Date.now() - ONE_SEND_WINDOW_DAYS * 24 * 60 * 60 * 1000).toISOString();
     const { data: matches, error: matchErr } = await db()
-        .from('campaigns')
-        .select('id, status')
+        .schema('email').from('campaigns')
+        .select('id, name, status')
         .eq('project_id', input.projectId)
         .eq('metadata->>content_key', contentKey)
         .in('status', [...ONE_SEND_MATCH_STATUSES])
@@ -486,6 +483,17 @@ async function executeBroadcast(input) {
         return { ok: false, ...dbToolError(matchErr), code: 'ONE_SEND_LOOKUP_FAILED' };
     }
     const match = (matches ?? [])[0];
+    return { ok: true, contentKey, match };
+}
+/**
+ * Create, enroll and activate a one-email broadcast (#3669). Shared by broadcast_to_segment and
+ * send_broadcast so both run the same steps. Enrolls exactly the given contact ids.
+ */
+async function executeBroadcast(input) {
+    const found = await findBroadcastOneSend(input);
+    if (!found.ok)
+        return found;
+    const { contentKey, match } = found;
     if (match)
         return attachBroadcastToExisting(input, match);
     const limit = await checkPlanLimit('active_campaigns');
@@ -564,15 +572,15 @@ async function executeBroadcast(input) {
     }
     return { ok: true, campaignId: campaign.id, stepId: step.id, enrolled: input.contactIds.length, nextSendAt };
 }
-/** #3749: enroll into the matched campaign, skipping anyone enrolled or already sent to. */
-async function attachBroadcastToExisting(input, match) {
+/** The same enrollment/log skip set is used before approval and immediately before writes. */
+async function broadcastOneSendAudience(input, match) {
     const skip = new Set();
     const sentStatuses = new Set(ONE_SEND_SENT_STATUSES);
     const chunk = 200;
     for (let i = 0; i < input.contactIds.length; i += chunk) {
         const ids = input.contactIds.slice(i, i + chunk);
         const { data: enrolledRows, error: enrolledErr } = await db()
-            .from('campaign_enrollments')
+            .schema('email').from('campaign_enrollments')
             .select('contact_id')
             .eq('campaign_id', match.id)
             .in('contact_id', ids)
@@ -584,7 +592,7 @@ async function attachBroadcastToExisting(input, match) {
             skip.add(row.contact_id);
         let logRows;
         try {
-            logRows = await readAllRows(db().from('email_log')
+            logRows = await readAllRows(db().schema('email').from('email_log')
                 .select('contact_id, status')
                 .eq('project_id', input.projectId)
                 .eq('campaign_id', match.id)
@@ -599,6 +607,37 @@ async function attachBroadcastToExisting(input, match) {
     }
     const fresh = input.contactIds.filter((id) => !skip.has(id));
     const alreadySent = input.contactIds.length - fresh.length;
+    return { ok: true, fresh, alreadySent };
+}
+async function previewOneSend(input) {
+    const found = await findBroadcastOneSend(input);
+    if (!found.ok)
+        return found;
+    if (!found.match)
+        return { ok: true, match: undefined, fresh: input.contactIds, alreadySent: 0 };
+    const audience = await broadcastOneSendAudience(input, found.match);
+    if (!audience.ok)
+        return audience;
+    return { ...audience, match: found.match };
+}
+function broadcastNoOp(executed) {
+    return {
+        campaign_id: executed.campaignId,
+        step_id: executed.stepId,
+        status: 'not_sent',
+        code: 'ALREADY_SENT_TO_AUDIENCE',
+        contacts_enrolled: 0,
+        grouped: true,
+        already_sent: executed.alreadySent ?? 0,
+        message: `Nothing was sent: all ${executed.alreadySent ?? 0} recipients already received this email. Changing the subject or body creates a new send.`,
+    };
+}
+/** #3749: enroll into the matched campaign, skipping anyone enrolled or already sent to. */
+async function attachBroadcastToExisting(input, match) {
+    const audience = await broadcastOneSendAudience(input, match);
+    if (!audience.ok)
+        return audience;
+    const { fresh, alreadySent } = audience;
     const nextSendAt = input.sendAt ?? new Date().toISOString();
     let enrolled = 0;
     for (let i = 0; i < fresh.length; i += 500) {
@@ -642,7 +681,7 @@ async function attachBroadcastToExisting(input, match) {
         .order('step_order', { ascending: true })
         .limit(1);
     const stepId = (steps ?? [])[0]?.id ?? '';
-    return { ok: true, campaignId: match.id, stepId, enrolled, nextSendAt, grouped: true, alreadySent };
+    return { ok: true, campaignId: match.id, stepId, enrolled, nextSendAt, grouped: true, alreadySent: alreadySent + fresh.length - enrolled };
 }
 /** #3714: a send nobody could receive leaves one audit row (counts by reason, no addresses). */
 function logRefusedSend(kind, siteId, sendClass, audience, extra = {}) {
@@ -1441,9 +1480,13 @@ export function registerCampaignTools(server) {
                 site_id,
                 segment_id,
                 contacts_enrolled: executed.enrolled,
+                ...(executed.grouped && executed.enrolled === 0 ? { outcome: 'no_op_already_sent', already_sent: executed.alreadySent ?? 0 } : {}),
                 source: 'mcp',
             },
         }).catch(() => { });
+        if (executed.grouped && executed.enrolled === 0)
+            return j({ ...broadcastNoOp(executed),
+                name: campaignName, segment: { id: segment.id, name: segment.name }, next_send_at: sendAt });
         fireWebhookEvent('campaign.activated', {
             campaign_id: campaign.id,
             campaign_name: campaignName,
@@ -1459,7 +1502,9 @@ export function registerCampaignTools(server) {
             segment: { id: segment.id, name: segment.name },
             contacts_enrolled: executed.enrolled,
             next_send_at: sendAt,
-            message: `Broadcast "${campaignName}" is active with ${executed.enrolled} contact(s) enrolled for immediate sending.`,
+            message: executed.grouped && executed.alreadySent
+                ? `${executed.enrolled} sent now; ${executed.alreadySent} skipped because they already received it.`
+                : `Broadcast "${campaignName}" is active with ${executed.enrolled} contact(s) enrolled for immediate sending.`,
             ...(executed.grouped ? { grouped: true, already_sent: executed.alreadySent ?? 0 } : {}),
         });
     });
@@ -1519,8 +1564,11 @@ export function registerCampaignTools(server) {
                 action: 'campaign.send_broadcast',
                 resourceType: 'campaign',
                 resourceId: executed.campaignId,
-                details: { approval_id, decided_by: decidedBy ?? null, site_id: payload.site_id, segment_id: payload.segment_id ?? null, contacts_enrolled: executed.enrolled, send_at: payload.send_at ?? null, source: 'mcp' },
+                details: { approval_id, decided_by: decidedBy ?? null, site_id: payload.site_id, segment_id: payload.segment_id ?? null, contacts_enrolled: executed.enrolled, send_at: payload.send_at ?? null, source: 'mcp',
+                    ...(executed.grouped && executed.enrolled === 0 ? { outcome: 'no_op_already_sent', already_sent: executed.alreadySent ?? 0 } : {}) },
             }).catch(() => { });
+            if (executed.grouped && executed.enrolled === 0)
+                return j({ ...broadcastNoOp(executed), send_at: payload.send_at ?? 'now' });
             fireWebhookEvent('campaign.activated', {
                 campaign_id: executed.campaignId,
                 campaign_name: payload.name,
@@ -1535,9 +1583,11 @@ export function registerCampaignTools(server) {
                 contacts_enrolled: executed.enrolled,
                 send_at: payload.send_at ?? 'now',
                 ...(executed.grouped ? { grouped: true, already_sent: executed.alreadySent ?? 0 } : {}),
-                message: payload.send_at
-                    ? `Scheduled: "${payload.subject}" goes to ${executed.enrolled} people at ${payload.send_at}.`
-                    : `Sending: "${payload.subject}" is going to ${executed.enrolled} people now. Delivery happens over the next few minutes; check the campaign for results.`,
+                message: executed.grouped && executed.alreadySent
+                    ? `${executed.enrolled} ${payload.send_at ? `scheduled at ${payload.send_at}` : 'sent now'}; ${executed.alreadySent} skipped because they already received it.`
+                    : payload.send_at
+                        ? `Scheduled: "${payload.subject}" goes to ${executed.enrolled} people at ${payload.send_at}.`
+                        : `Sending: "${payload.subject}" is going to ${executed.enrolled} people now. Delivery happens over the next few minutes; check the campaign for results.`,
             });
         }
         // ── Call 2: execute exactly the approved, frozen payload. Nothing else is trusted. ──
@@ -1666,6 +1716,34 @@ export function registerCampaignTools(server) {
                 code: 'validation_failed', ...delivery,
                 audience: { label: segmentName, segment_id: segment_id ?? null, ...heldBackLines(resolved) } });
         }
+        const preview = await previewOneSend({ projectId, siteId: site.id, subject: content.subject,
+            bodyHtml: content.body_html, contactIds: resolved.contactIds });
+        if (!preview.ok) {
+            const { ok: _ok, ...failure } = preview;
+            return j(failure);
+        }
+        const oneSendCounts = preview.match ? {
+            already_sent_count: preview.alreadySent,
+            will_send_count: preview.fresh.length,
+            matches_campaign_id: preview.match.id,
+        } : {};
+        if (preview.match && preview.fresh.length === 0) {
+            return j({ code: 'ALREADY_SENT_TO_AUDIENCE', ...oneSendCounts, recipient_count: 0,
+                error: `Everyone in this audience already got this exact email from this sender (campaign ${preview.match.id}). Changing the subject or body creates a new send.` });
+        }
+        // Preserve sandbox policy counts for the full audience, and intersect that policy with the
+        // fresh audience for the displayed count. Never narrow the frozen contact_ids.
+        const recipientCount = preview.match
+            ? (await sandboxAudience(projectId, preview.fresh)).recipient_count
+            : delivery.recipient_count;
+        if (recipientCount === 0) {
+            return j({ error: 'No new recipients can be delivered under the sandbox policy. Choose an allowed audience before requesting approval.',
+                code: 'validation_failed', ...delivery, recipient_count: 0, ...oneSendCounts,
+                audience: { label: segmentName, segment_id: segment_id ?? null, ...heldBackLines(resolved) } });
+        }
+        const oneSendSummary = preview.match
+            ? `${preview.alreadySent} of ${resolved.contactIds.length} already received this email (campaign ${preview.match.name}); it will send to ${recipientCount ?? 'an unknown number of recipients'}.`
+            : undefined;
         const campaignName = name ?? `Broadcast - ${content.subject.slice(0, 120)}`;
         const payload = {
             site_id: site.id,
@@ -1679,8 +1757,9 @@ export function registerCampaignTools(server) {
             send_at: sendAt,
             send_class: sendClass,
             brand_asset_ids: brand_asset_ids ?? null,
-            // Frozen at approval time: the people approved are the people enrolled.
+            // Frozen at approval time: execution checks One Send again for these same people.
             contact_ids: resolved.contactIds,
+            ...oneSendCounts,
         };
         const approvalParams = { payload };
         const channelContext = currentChannelContext();
@@ -1693,7 +1772,8 @@ export function registerCampaignTools(server) {
             subject: content.subject,
             segment_name: segment_id ? segmentName : null,
             audience_label: segment_id ? null : `"${segmentName}"`,
-            recipient_count: delivery.recipient_count,
+            recipient_count: recipientCount,
+            one_send_summary: oneSendSummary,
             body_excerpt: approvalExcerpt(content.body_html),
             send_at: sendAt,
         });
@@ -1701,6 +1781,8 @@ export function registerCampaignTools(server) {
             ...approval,
             rendered: { subject: content.subject, body_html: stripActiveEmailContent(content.body_html).html, body_text: content.body_text },
             ...delivery,
+            recipient_count: recipientCount,
+            ...oneSendCounts,
             audience: { label: segmentName, segment_id: segment_id ?? null, ...heldBackLines(resolved) },
             site: { id: site.id, name: site.name, from_email: site.from_email },
             send_at: sendAt ?? 'right after approval',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sendinel/mcp-server",
-  "version": "1.0.207",
+  "version": "1.0.208",
   "description": "AI-controlled email operations control plane for campaigns, contacts, segments, analytics, and multi-provider sending via MCP.",
   "mcpName": "ai.sendinel/mcp-server",
   "type": "module",