@sendinel/mcp-server 1.0.37 → 1.0.39

package/build/tools/campaigns.js

@@ -305,6 +305,33 @@ async function executeBroadcast(input) {
     }
     return { ok: true, campaignId: campaign.id, stepId: step.id, enrolled: input.contactIds.length, nextSendAt };
 }
+/** #3714: a send nobody could receive leaves one audit row (counts by reason, no addresses). */
+function logRefusedSend(kind, siteId, sendClass, audience, extra = {}) {
+    const reasons = {};
+    if (audience.notOnSender > 0)
+        reasons.not_subscribed_to_sender = audience.notOnSender;
+    for (const [reason, count] of Object.entries(audience.heldBack)) {
+        if (count > 0)
+            reasons[`consent_${reason}`] = count;
+    }
+    return logAudit({
+        action: 'send.refused',
+        resourceType: 'site',
+        resourceId: siteId,
+        details: {
+            kind,
+            outcome: 'nothing_sent',
+            send_class: sendClass,
+            site_id: siteId,
+            campaign_id: null,
+            requested: audience.matched,
+            enrolled: 0,
+            refused: Object.values(reasons).reduce((sum, count) => sum + count, 0),
+            reasons,
+            ...extra,
+        },
+    });
+}
 function heldBackLines(audience) {
     return {
         matched: audience.matched,
@@ -909,6 +936,7 @@ export function registerCampaignTools(server) {
             });
         }
         if (contactIds.length === 0) {
+            await logRefusedSend('broadcast_to_segment', site_id, 'broadcast', audience, { audience: { segment_id } });
             return j({
                 error: 'Nobody in this segment can get email from this sender right now.',
                 code: 'EMPTY_SEGMENT',
@@ -1153,6 +1181,7 @@ export function registerCampaignTools(server) {
             return j({ error: `Could not load this audience: ${error instanceof Error ? error.message : String(error)}`, code: 'AUDIENCE_LOOKUP_FAILED' });
         }
         if (resolved.contactIds.length === 0) {
+            await logRefusedSend('send_broadcast', site.id, sendClass, resolved, { audience: segment_id ? { segment_id } : { inline_rules: true } });
             return j({ error: 'Nobody in this audience can get this email from this sender.', code: 'EMPTY_AUDIENCE', audience: heldBackLines(resolved), rules });
         }
         let content;

package/build/tools/delivery-ops.js

@@ -703,14 +703,15 @@ export function registerDeliveryOpsTools(server) {
             idempotentHint: true,
             openWorldHint: false,
         },
-        description: 'Export contacts, email log, campaign, or suppression data as JSON.',
+        description: 'Export contacts, email log, campaign, or suppression data as JSON, newest first. Returns up to `limit` rows per call plus `total` (all matching rows). When `truncated` is true, call again with `offset` set to `next_offset` until it is false to get every row.',
         inputSchema: {
             resource: z.enum(['contacts', 'email_log', 'campaigns', 'suppressions']),
             site_id: z.string().uuid().optional(),
             since: z.string().datetime().optional(),
             limit: z.number().int().min(1).max(5000).default(1000).optional(),
+            offset: z.number().int().min(0).default(0).optional().describe('Rows to skip. Use the next_offset from the previous call to continue.'),
         },
-    }, async ({ resource, site_id, since, limit = 1000 }) => {
+    }, async ({ resource, site_id, since, limit = 1000, offset = 0 }) => {
         const projectId = getProjectId();
         const tableMap = {
             contacts: 'subscribers',
@@ -718,16 +719,40 @@ export function registerDeliveryOpsTools(server) {
             campaigns: 'campaigns',
             suppressions: 'suppressions',
         };
-        let query = db().from(tableMap[resource]).select('*').eq('project_id', projectId).limit(limit);
-        if (site_id && (resource === 'email_log' || resource === 'campaigns'))
-            query = query.eq('site_id', site_id);
-        if (since)
-            query = query.gte('created_at', since);
-        query = query.order('created_at', { ascending: false });
-        const { data, error } = await query;
-        if (error)
-            return j({ error: error.message });
-        return j({ resource, count: data?.length ?? 0, data: data ?? [], truncated: (data?.length ?? 0) === limit });
+        // #3715: one request returns at most 1,000 rows however large `limit` is, with no error.
+        // Read `limit` rows in pages, and report the real total so the caller can tell when it has
+        // everything. id breaks ties so pages do not overlap.
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any
+        const filtered = (query) => {
+            let q = query.eq('project_id', projectId);
+            if (site_id && (resource === 'email_log' || resource === 'campaigns'))
+                q = q.eq('site_id', site_id);
+            if (since)
+                q = q.gte('created_at', since);
+            return q;
+        };
+        const PAGE = 1000;
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any
+        const rows = [];
+        while (rows.length < limit) {
+            const size = Math.min(PAGE, limit - rows.length);
+            const from = offset + rows.length;
+            const { data, error } = await filtered(db().from(tableMap[resource]).select('*'))
+                .order('created_at', { ascending: false })
+                .order('id', { ascending: true })
+                .range(from, from + size - 1);
+            if (error)
+                return j({ error: error.message });
+            rows.push(...(data ?? []));
+            if ((data?.length ?? 0) < size)
+                break;
+        }
+        const { count, error: countError } = await filtered(db().from(tableMap[resource]).select('id', { count: 'exact', head: true }));
+        // An unknown total is reported as null, never as a number.
+        const total = countError || typeof count !== 'number' ? null : count;
+        const nextOffset = offset + rows.length;
+        const truncated = total === null ? rows.length === limit : nextOffset < total;
+        return j({ resource, count: rows.length, total, offset, next_offset: truncated ? nextOffset : null, truncated, data: rows });
     });
     server.registerTool('list_domains', {
         title: 'List Domains',

package/package.json

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