@sendinel/mcp-server 1.0.29 → 1.0.31

package/build/tools/campaigns.js

@@ -458,8 +458,9 @@ export function registerCampaignTools(server) {
             trigger_config: z.record(z.string(), z.unknown()).nullable().optional().describe('For trigger_type=date_field: { field, days_before?, recurring? }. Otherwise: { events?: string[], tags?: string[] }.'),
             segment_id: z.string().uuid().nullable().optional().describe('Segment UUID for ongoing audience targeting; membership is re-evaluated on sends'),
             send_at: z.string().datetime().nullable().optional().describe('For type=broadcast only: ISO 8601 instant to schedule the send. Must be in the future. Set null for no schedule. The campaign must still be activated via update_campaign_status.'),
+            send_class: z.enum(['broadcast', 'product']).optional().describe("Who may receive it when the project's consent gate is on: 'broadcast' (default) = only contacts who opted in to marketing; 'product' = also the customer's signed-up users (onboarding, trial, account mail)."),
         },
-    }, async ({ site_id, name, type, description, sends_per_hour, requires_approval, trigger_event_name, trigger_tag, trigger_type, trigger_config, segment_id, send_at }) => {
+    }, async ({ site_id, name, type, description, sends_per_hour, requires_approval, trigger_event_name, trigger_tag, trigger_type, trigger_config, segment_id, send_at, send_class }) => {
         const limit = await checkPlanLimit('active_campaigns');
         if (!limit.allowed) {
             return { content: [{ type: 'text', text: `Active campaign limit reached (${limit.current}/${limit.limit}). Upgrade to Pro for unlimited campaigns.` }] };
@@ -473,6 +474,7 @@ export function registerCampaignTools(server) {
             description,
             sends_per_hour: sends_per_hour ?? null,
             requires_approval,
+            ...(send_class ? { send_class } : {}),
         };
         const scheduleError = applyCampaignSendAtField(insertPayload, send_at, type);
         if (scheduleError)
@@ -2691,7 +2693,7 @@ Return ONLY valid JSON with this exact shape:
             idempotentHint: true,
             openWorldHint: false,
         },
-        description: 'Update a campaign name, site, segment, send time, topic, or trigger settings. send_at schedules a broadcast send (type=broadcast only) and must be a future ISO 8601 instant; set null to clear a pending schedule. Only provided fields are updated.',
+        description: 'Update a campaign name, site, segment, send time, topic, trigger settings, or send class (broadcast vs product). send_at schedules a broadcast send (type=broadcast only) and must be a future ISO 8601 instant; set null to clear a pending schedule. Only provided fields are updated.',
         inputSchema: {
             campaign_id: z.string().uuid(),
             name: z.string().min(1).max(200).optional(),
@@ -2705,8 +2707,9 @@ Return ONLY valid JSON with this exact shape:
             trigger_config: z.record(z.string(), z.unknown()).nullable().optional(),
             sends_per_hour: z.number().int().min(1).nullable().optional().describe('Optional hourly send cap for wave sending. Set null to clear.'),
             send_in_contact_timezone: z.boolean().optional(),
+            send_class: z.enum(['broadcast', 'product']).optional().describe("Who may receive it when the project's consent gate is on: 'broadcast' (default) = only contacts who opted in to marketing; 'product' = also the customer's signed-up users (onboarding, trial, account mail)."),
         },
-    }, async ({ campaign_id, name, site_id, segment_id, send_at, topic, trigger_event_name, trigger_tag, trigger_type, trigger_config, sends_per_hour, send_in_contact_timezone }) => {
+    }, async ({ campaign_id, name, site_id, segment_id, send_at, topic, trigger_event_name, trigger_tag, trigger_type, trigger_config, sends_per_hour, send_in_contact_timezone, send_class }) => {
         const projectId = getProjectId();
         let campaignType;
         if (send_at !== undefined && send_at !== null) {
@@ -2736,6 +2739,8 @@ Return ONLY valid JSON with this exact shape:
             updates.sends_per_hour = sends_per_hour;
         if (send_in_contact_timezone !== undefined)
             updates.send_in_contact_timezone = send_in_contact_timezone;
+        if (send_class !== undefined)
+            updates.send_class = send_class;
         const targetingError = await applyCampaignTargetingFields(updates, { trigger_event_name, trigger_tag, trigger_type, trigger_config, segment_id }, projectId);
         if (targetingError)
             return j({ error: targetingError, code: targetingError === 'Segment not found' ? 'NOT_FOUND' : 'INVALID_FIELD' });

package/build/tools/contacts.js

@@ -1921,15 +1921,11 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
             ({ query: q } = await applySegmentRules(q, { custom_property: params.custom_property }, db(), projectId, fieldTypes));
         }
         if (params.site_id) {
-            const { data: subs } = await db()
-                .from('site_subscriptions')
-                .select('contact_id')
-                .eq('site_id', params.site_id)
-                .eq('subscribed', true);
-            const ids = (subs ?? []).map((s) => s.contact_id);
-            if (!ids.length)
+            // Shared site filter: an inner join, not an id list that overflows past ~350 contacts (#3674).
+            const siteFiltered = await applySegmentRules(q, { site_id: params.site_id }, db(), projectId);
+            if (siteFiltered.siteContactIds && siteFiltered.siteContactIds.length === 0)
                 return j({ total: 0, sample: [] });
-            q = q.in('id', ids);
+            q = siteFiltered.query;
         }
         if (params.company_domain) {
             const { data: companies } = await db()
@@ -1976,6 +1972,9 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
         if (params.custom_property) {
             ({ query: countQ } = await applySegmentRules(countQ, { custom_property: params.custom_property }, db(), projectId, fieldTypes));
         }
+        // The total must count the same site the sample came from (#3674).
+        if (params.site_id)
+            ({ query: countQ } = await applySegmentRules(countQ, { site_id: params.site_id }, db(), projectId));
         const { count } = await countQ;
         return j({ total: count ?? (data ?? []).length, sample: data ?? [] });
     });

package/build/tools/lists.js

@@ -1,5 +1,5 @@
 import { z } from 'zod';
-import { applySegmentRules, buildSegmentBaseQuery, evaluateSegmentRules, normalizeSegmentRulesToTree, } from '@sendinel/domain';
+import { applySegmentRules, buildSegmentBaseQuery, evaluateSegmentRules, normalizeSegmentRulesToTree, readAllRows, } from '@sendinel/domain';
 import { db } from '../db.js';
 import { getProjectId } from '../project.js';
 import { logAudit } from '../audit.js';
@@ -34,17 +34,13 @@ async function getSegmentContactIds(rules) {
     const { query: filtered, siteContactIds } = await applySegmentRules(q, rules, db(), projectId, fieldTypes);
     if (siteContactIds && siteContactIds.length === 0)
         return [];
-    const { data, error } = await filtered;
-    if (error || !data)
-        return [];
+    // Paged and loud (#3674): an unpaged read stopped at 1,000 rows, and an error read as "no one".
+    const data = await readAllRows(filtered);
     const contacts = data.filter((contact) => evaluateSegmentRules(contact, tree, fieldTypes));
     let ids = contacts.map((contact) => String(contact.id));
     if (tree.exclude_suppressed !== false && ids.length > 0) {
-        const { data: suppressed } = await db()
-            .from('suppressions')
-            .select('email')
-            .eq('project_id', projectId);
-        if (suppressed?.length) {
+        const suppressed = await readAllRows(db().from('suppressions').select('id, email').eq('project_id', projectId));
+        if (suppressed.length) {
             const suppressedEmails = new Set(suppressed.map((row) => row.email));
             ids = contacts
                 .filter((contact) => !suppressedEmails.has(String(contact.email)))

package/build/tools/segments.js

@@ -3,7 +3,7 @@ import Anthropic from '@anthropic-ai/sdk';
 import { db } from '../db.js';
 import { getProjectId } from '../project.js';
 import { getAnthropicModelId } from '../anthropic-model.js';
-import { applySegmentRules, buildSegmentBaseQuery, evaluateSegmentRules, normalizeSegmentRulesToTree, } from '@sendinel/domain';
+import { applySegmentRules, buildSegmentBaseQuery, evaluateSegmentRules, normalizeSegmentRulesToTree, readAllRows, } from '@sendinel/domain';
 async function loadFieldTypes(projectId) {
     const client = db();
     const scoped = typeof client.schema === 'function' ? client.schema('email') : client;
@@ -34,18 +34,14 @@ async function getSegmentContactIds(rules) {
     const { query: filtered, siteContactIds } = await applySegmentRules(q, rules, db(), projectId, fieldTypes);
     if (siteContactIds && siteContactIds.length === 0)
         return [];
-    const { data, error } = await filtered;
-    if (error || !data)
-        return [];
+    // Paged and loud (#3674): an unpaged read stopped at 1,000 rows, and an error read as "no one".
+    const data = await readAllRows(filtered);
     const contacts = data.filter((contact) => evaluateSegmentRules(contact, tree, fieldTypes));
     let ids = contacts.map((c) => String(c.id));
     if (tree.exclude_suppressed !== false && ids.length > 0) {
-        const { data: suppressed } = await db()
-            .from('suppressions')
-            .select('email')
-            .eq('project_id', projectId);
-        if (suppressed?.length) {
-            const suppressedEmails = new Set(suppressed.map((s) => s.email));
+        const suppressed = await readAllRows(db().from('suppressions').select('id, email').eq('project_id', projectId));
+        if (suppressed.length) {
+            const suppressedEmails = new Set(suppressed.map((row) => row.email));
             ids = contacts
                 .filter((c) => !suppressedEmails.has(String(c.email)))
                 .map((c) => String(c.id));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sendinel/mcp-server",
-  "version": "1.0.29",
+  "version": "1.0.31",
   "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",