@sendinel/mcp-server 1.0.171 → 1.0.173

package/build/lib/email-generation.d.ts

@@ -1,5 +1,9 @@
 import type Anthropic from '@anthropic-ai/sdk';
 import { type ComposerBlock } from './composer-blocks.js';
+/** Known unsupported targeting must fail before a model can invent a property rule. */
+export declare function unsupportedAudienceReason(description: string): string | null;
+export declare class AudienceNotUnderstoodError extends Error {
+}
 export declare function parseAudienceRules(anthropic: Anthropic, description: string): Promise<Record<string, unknown>>;
 export type GeneratedEmail = {
     subject: string;

package/build/lib/email-generation.js

@@ -11,24 +11,39 @@ import { BLOCK_GENERATION_SCHEMA, blocksToHtml, blocksToText, normalizeComposerB
 function parseJsonResponse(raw) {
     return JSON.parse(raw.replace(/```json?\n?|```/g, '').trim());
 }
+const EMAIL_DOMAIN_AUDIENCE_MESSAGE = 'Audiences by email address or domain are not supported. Create a segment using a tag or property rule, or pass segment_id for a saved segment.';
+/** Known unsupported targeting must fail before a model can invent a property rule. */
+export function unsupportedAudienceReason(description) {
+    return /\be-?mail[\s_-]+domains?\b|\be-?mail(?:\s+addresses?)?\s+(?:ends?\s+with|contains?|matches?|is\b|at\b)|@[a-z0-9](?:[a-z0-9.-]*\.)[a-z]{2,}\b/i.test(description)
+        ? EMAIL_DOMAIN_AUDIENCE_MESSAGE : null;
+}
+export class AudienceNotUnderstoodError extends Error {
+}
 export async function parseAudienceRules(anthropic, description) {
+    const unsupported = unsupportedAudienceReason(description);
+    if (unsupported)
+        throw new AudienceNotUnderstoodError(unsupported);
     const resp = await anthropic.messages.create({
         model: getAnthropicModelId(),
         max_tokens: AI_MAX_TOKENS,
         ...outputConfig("low"),
-        system: 'You translate natural language audience descriptions into a JSON filter rules object. Return ONLY valid JSON. Available keys: lifecycle_stage (lead|trial|customer|churned|win-back), payment (object with plan, mrr_cents, ltv_cents, subscription_status, trial_ends_at, churned_at, paid_at), min_score, max_score, source, subscribed_after, subscribed_before, has_opened, has_clicked, tag, custom_property, exclude_suppressed. Example: {"payment":{"plan":"pro","trial_ends_at":{"within_days":7}},"lifecycle_stage":"trial"}. Today is ' + new Date().toISOString().slice(0, 10) + '.',
+        system: 'You translate natural language audience descriptions into a JSON filter rules object. Return ONLY valid JSON. Available keys: match_all, lifecycle_stage (lead|trial|customer|churned|win-back), payment (object with plan, mrr_cents, ltv_cents, subscription_status, trial_ends_at, churned_at, paid_at), min_score, max_score, source, subscribed_after, subscribed_before, has_opened, has_clicked, tag, custom_property (a string like "plan=pro"), exclude_suppressed. Everyone, all contacts, all subscribers or all means {"match_all":true}. There is no email address or email-domain rule. Never invent keys or use custom_property for email targeting; return {"error":"' + EMAIL_DOMAIN_AUDIENCE_MESSAGE + '"} instead. For any other unsupported audience return an error object explaining why. Example: {"payment":{"plan":"pro","trial_ends_at":{"within_days":7}},"lifecycle_stage":"trial"}. Today is ' + new Date().toISOString().slice(0, 10) + '.',
         messages: [{ role: 'user', content: description }],
     });
     const text = anthropicCheckedText(resp);
+    let rules;
     try {
-        const rules = parseJsonResponse(text);
+        rules = parseJsonResponse(text);
         if (!rules || typeof rules !== 'object' || Array.isArray(rules))
             throw new Error('not an object');
-        return rules;
     }
     catch {
-        throw new Error(`Could not understand the audience "${description}". Try a saved segment instead.`);
+        throw new AudienceNotUnderstoodError(`Could not understand the audience "${description}". Try a saved segment instead.`);
     }
+    if ('error' in rules)
+        throw new AudienceNotUnderstoodError(typeof rules.error === 'string'
+            ? rules.error : 'This audience is not supported. Try a saved segment instead.');
+    return rules;
 }
 export async function generateEmailFromBrief(anthropic, projectId, siteId, brief) {
     const [{ data: site, error: siteError }, { data: project, error: projectError }] = await Promise.all([

package/build/tools/campaigns.js

@@ -4,7 +4,7 @@ import { resolveCampaignReentry, readCampaignPeriods, campaignPeriodFields } fro
 import { CAMPAIGN_BRANCH_CONDITIONS, campaignBranchConditionValue, isContactPropertyConditionList, STEP_PROPERTY_CONDITIONS_ERROR, validateStepPropertyRules } from "@sendinel/domain";
 import { z } from 'zod';
 import Anthropic from '@anthropic-ai/sdk';
-import { stripActiveEmailContent, assertSendAccepted, providerSendError, sendError, SendRefusedError, BLOCKING_CONTENT_ISSUE_KINDS, computeSendContentKey, findContentIssues, sendingHostOf, NAME_FALLBACKS, ONE_SEND_MATCH_STATUSES, ONE_SEND_SENT_STATUSES, ONE_SEND_WINDOW_DAYS, normalizeTriggerConfig, renderTemplateString as renderTemplateVars, resolveBrandTokens, brandWarnings, readAllRows, isSegmentRulesError, scopeSegmentRulesToSite, SegmentRulesError, validateSegmentRules, validateUnsubFooterTemplate, } from '@sendinel/domain';
+import { stripActiveEmailContent, assertSendAccepted, providerSendError, sendError, SendRefusedError, BLOCKING_CONTENT_ISSUE_KINDS, computeSendContentKey, findContentIssues, sendingHostOf, NAME_FALLBACKS, ONE_SEND_MATCH_STATUSES, ONE_SEND_SENT_STATUSES, ONE_SEND_WINDOW_DAYS, normalizeTriggerConfig, renderTemplateString as renderTemplateVars, resolveBrandTokens, brandWarnings, readAllRows, isSegmentRulesError, scopeSegmentRulesToSite, SegmentRulesError, validateSegmentRules, validateUnsubFooterTemplate, SEGMENT_MATCH_ALL, } from '@sendinel/domain';
 // Subpath import, not the barrel above -- reply-actions.ts uses node:crypto,
 // and MCP-server-side that's fine (this never runs in an Edge Function), but
 // keeping it off the shared barrel avoids re-creating the Edge-Function
@@ -22,7 +22,7 @@ import { fireWebhookEvent } from '../lib/webhook-events.js';
 import { approvalNext, approvalExcerpt, assertChannelCanExecute, currentChannelContext, requestApproval, checkApproval, withApproval } from '../lib/action-approvals.js';
 import { sandboxAudience, broadcastApprovalNext } from '../lib/sandbox-audience.js';
 import { resolveSegmentContactIds, resolveSendAudience } from '../lib/send-audience.js';
-import { generateEmailFromBrief, parseAudienceRules } from '../lib/email-generation.js';
+import { AudienceNotUnderstoodError, generateEmailFromBrief, parseAudienceRules, unsupportedAudienceReason } from '../lib/email-generation.js';
 import { getAnthropicModelId } from '../anthropic-model.js';
 import { AI_MAX_TOKENS, outputConfig, anthropicCheckedText } from '@sendinel/domain';
 import { getScheduleWindowBounds, groupScheduleByCampaign, parseFutureDate } from '../lib/schedule.js';
@@ -1580,8 +1580,12 @@ export function registerCampaignTools(server) {
                 sites: (sites ?? []).map((s) => ({ id: s.id, name: s.name, from_email: s.from_email })),
             });
         }
+        const everyone = audience !== undefined && ['everyone', 'all contacts', 'all subscribers', 'all'].includes(audience.trim().toLowerCase().replace(/[.!?,;:]+$/, '').trim());
+        const unsupported = audience ? unsupportedAudienceReason(audience) : null;
+        if (unsupported)
+            return j({ error: unsupported, code: 'AUDIENCE_NOT_UNDERSTOOD' });
         const apiKey = process.env.ANTHROPIC_API_KEY;
-        if ((audience || brief) && !apiKey)
+        if (((audience && !everyone) || brief) && !apiKey)
             return j({ error: 'ANTHROPIC_API_KEY not configured', code: 'AI_NOT_CONFIGURED' });
         const anthropic = apiKey ? new Anthropic({ apiKey }) : null;
         let rules;
@@ -1600,12 +1604,17 @@ export function registerCampaignTools(server) {
             rules = scopeSegmentRulesToSite(segment.rules, segment.site_id);
             segmentName = segment.name;
         }
+        else if (everyone) {
+            rules = { ...SEGMENT_MATCH_ALL };
+            segmentName = audience;
+        }
         else {
             try {
                 rules = await parseAudienceRules(anthropic, audience);
             }
             catch (error) {
-                return j({ error: error instanceof Error ? dbToolError(error).error : String(error), code: 'AUDIENCE_NOT_UNDERSTOOD' });
+                return j({ error: error instanceof AudienceNotUnderstoodError ? error.message
+                        : error instanceof Error ? dbToolError(error).error : String(error), code: 'AUDIENCE_NOT_UNDERSTOOD' });
             }
             segmentName = audience;
         }

package/build/tools/contacts.js

@@ -2,7 +2,7 @@ import { normalizeTagInput, tagInputWarnings } from '../lib/tags.js';
 import { ResourceTemplate } from '@modelcontextprotocol/sdk/server/mcp.js';
 import { CONTACT_ENRICHMENT_PROVIDERS } from '@desler/models';
 import { createContactEnrichmentAdapters, enrichContactFromConnections, } from '@desler/models/contact-enrichment';
-import { buildPreviewAttentionEnvelope, buildSafeIlikeOrFilter, literalIlikePattern, applySegmentRules, isSegmentRulesError, ContactPropertyAnomalyLogger, normalizeContactProperties, validateContactPropertyPatch, getAttentionDecision, getContactNeighborhood, getScheduledTouches, findContactDuplicateGroups, readAllPages, readAllRows, chunkIds, } from '@sendinel/domain';
+import { buildPreviewAttentionEnvelope, buildSafeIlikeOrFilter, literalIlikePattern, applySegmentRules, assertValidSegmentRules, isSegmentRulesError, ContactPropertyAnomalyLogger, normalizeContactProperties, validateContactPropertyPatch, getAttentionDecision, getContactNeighborhood, getScheduledTouches, findContactDuplicateGroups, readAllPages, readAllRows, chunkIds, } from '@sendinel/domain';
 import { z } from 'zod';
 import { callSendinelV1 } from '../lib/v1-client.js';
 import { db, projectClientForWriteback } from '../db.js';
@@ -2315,6 +2315,17 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
     }, async (params) => {
         const projectId = getProjectId();
         const sampleSize = params.sample_size ?? 5;
+        // Validate the same flat-rule shape as saved segments before any query.
+        // No filters deliberately means all contacts for this estimation tool.
+        const { sample_size: _sampleSize, ...rules } = params;
+        try {
+            assertValidSegmentRules(rules, { allowEmpty: true });
+        }
+        catch (error) {
+            if (isSegmentRulesError(error))
+                return toolError({ error: error.message, code: 'validation_failed' });
+            throw error;
+        }
         let q = db()
             .from('subscribers')
             .select('id, email, first_name, last_name, engagement_score, lifecycle_stage, source, created_at, last_opened_at, last_clicked_at')
@@ -2352,7 +2363,7 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
             }
             catch (error) {
                 if (isSegmentRulesError(error))
-                    return txt(`Error: ${dbToolError(error).error}`);
+                    return toolError({ error: error.message, code: 'validation_failed' });
                 throw error;
             }
         }
@@ -2365,11 +2376,13 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
             q = siteFiltered.query;
         }
         if (params.company_domain) {
-            const { data: companies } = await db()
+            const { data: companies, error } = await db()
                 .from('companies')
                 .select('id')
                 .eq('project_id', projectId)
                 .eq('domain', params.company_domain.trim().toLowerCase());
+            if (error)
+                return dbErrorResult(error);
             const ids = (companies ?? []).map((company) => company.id);
             if (!ids.length)
                 return j({ total: 0, sample: [] });
@@ -2377,7 +2390,7 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
         }
         const { data, error } = await q.order('engagement_score', { ascending: false }).limit(sampleSize);
         if (error)
-            return txt(`Error: ${dbToolError(error).error}`);
+            return dbErrorResult(error);
         // Get full count separately
         let countQ = db()
             .from('subscribers')
@@ -2412,7 +2425,9 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
         // 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, {}, { allowEmpty: true }));
-        const { count } = await countQ;
+        const { count, error: countError } = await countQ;
+        if (countError)
+            return dbErrorResult(countError);
         return j({ total: count ?? (data ?? []).length, sample: data ?? [] });
     });
     server.registerTool('identify_inactive', {

package/build/tools/segments.js

@@ -63,7 +63,9 @@ const j = (data) => txt(JSON.stringify(data, null, 2));
  * never an audience of everyone. Returns null for any other error.
  */
 function invalidRules(error) {
-    return isSegmentRulesError(error) ? txt(`Error: ${error.message}`) : null;
+    return isSegmentRulesError(error)
+        ? toolError({ error: error.message, code: 'validation_failed' })
+        : null;
 }
 export function registerSegmentTools(server) {
     server.registerTool('create_segment', {
@@ -89,7 +91,7 @@ export function registerSegmentTools(server) {
             catch (rulesError) {
                 const invalid = invalidRules(rulesError);
                 if (invalid)
-                    return toolError({ error: rulesError instanceof Error ? rulesError.message : 'Invalid segment rules', code: 'validation_failed' });
+                    return invalid;
                 throw rulesError;
             }
             const { data, error } = await db()
@@ -100,7 +102,7 @@ export function registerSegmentTools(server) {
             if (error?.code === '23505')
                 return toolError({ error: 'A segment with this name already exists', code: 'conflict' });
             if (error)
-                return toolError(dbToolError(error));
+                return dbErrorResult(error);
             // Count against the stored (normalized) tree, not the raw input: a tree skips
             // applySegmentRules' SQL push-down and filters in JS, which is what list_segments
             // does for this same segment a moment later. Pushing the flat rules down instead
@@ -171,7 +173,7 @@ export function registerSegmentTools(server) {
                 updates.name = name;
             if (description !== undefined)
                 updates.description = description;
-            if (rules) {
+            if (rules !== undefined) {
                 try {
                     updates.rules = prepareSegmentRulesForStorage(rules);
                 }
@@ -242,11 +244,13 @@ export function registerSegmentTools(server) {
                 .eq('id', segment_id)
                 .eq('project_id', getProjectId())
                 .maybeSingle();
-            if (error || !seg)
+            if (error)
+                return dbErrorResult(error);
+            if (!seg)
                 return txt('Error: segment not found');
             segRules = seg.rules;
         }
-        else if (rules) {
+        else if (rules !== undefined) {
             segRules = rules;
         }
         else {
@@ -265,10 +269,12 @@ export function registerSegmentTools(server) {
         const sample = ids.slice(0, 10);
         let sampleContacts = [];
         if (sample.length) {
-            const { data } = await db()
+            const { data, error } = await db()
                 .from('subscribers')
                 .select('id, email, first_name, last_name, engagement_score, lifecycle_stage, tags')
                 .in('id', sample);
+            if (error)
+                return dbErrorResult(error);
             sampleContacts = data ?? [];
         }
         return j({ total_contacts: ids.length, sample: sampleContacts });

package/package.json

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