@sendinel/mcp-server 1.0.95 → 1.0.97

package/build/tools/campaigns.js

@@ -1,3 +1,4 @@
+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 { assertSendAccepted, 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, } from '@sendinel/domain';
@@ -296,7 +297,6 @@ function normalizeConsistencyResult(value) {
 //   app/api/dashboard/campaigns/goals/route.ts
 // If those lists change, change these. A value MCP accepts that REST rejects (or the
 // reverse) is a divergence between two front doors onto the same engine.
-const CAMPAIGN_BRANCH_CONDITIONS = ['tag_added', 'lifecycle_reached', 'conversion', 'link_clicked', 'custom_event', 'reply', 'click', 'booking'];
 const CAMPAIGN_BRANCH_ACTIONS = ['goto_step', 'goto_campaign', 'exit', 'complete'];
 const CAMPAIGN_GOAL_TYPES = ['conversion', 'tag_added', 'lifecycle_reached', 'link_clicked', 'custom_event'];
 const CAMPAIGN_GOAL_ACTIONS = ['exit', 'complete', 'move_to_campaign'];
@@ -764,6 +764,10 @@ export function registerCampaignTools(server) {
             return { content: [{ type: 'text', text: `Error: ${error.message}` }] };
         return { content: [{ type: 'text', text: JSON.stringify(data, null, 2) }] };
     });
+    const propertyConditions = z.array(z.union([
+        z.object({ key: z.string(), op: z.enum(['eq', 'neq']), value: z.string() }).strict(),
+        z.object({ key: z.string(), op: z.literal('exists') }).strict(),
+    ])).min(1).max(10).refine(isContactPropertyConditionList, STEP_PROPERTY_CONDITIONS_ERROR).nullable().optional();
     server.registerTool('add_campaign_step', {
         title: 'Add Campaign Step',
         annotations: {
@@ -785,10 +789,15 @@ export function registerCampaignTools(server) {
             variant_label: z.string().optional().default('A').describe('Variant label for A/B testing (default: A). Use B, C, etc. for additional variants.'),
             survey_id: z.string().uuid().optional(),
             brand_asset_ids: z.array(z.string().uuid()).optional().describe('Stable brand_media_assets ids explicitly referenced by this step.'),
+            skip_if_property: propertyConditions.describe('Skip when ALL 1–10 conditions match current custom_properties. Missing/null keys never match. null clears.'),
+            send_only_if_property: propertyConditions.describe('Send only when ALL 1–10 conditions match current custom_properties. Unknown/non-match skips. null clears.'),
             skip_if_event: z.string().trim().min(1).optional().describe('Skip this step at send time if the contact already has an event with this name; the journey continues.'),
             send_only_if_event: z.string().trim().min(1).optional().describe('Send this step only if the contact has an event with this name by send time; otherwise skip it and continue the journey.'),
         },
-    }, async ({ campaign_id, step_order, subject, body_html, body_text, blocks, delay_days, delay_hours, variant_label, survey_id, brand_asset_ids, skip_if_event, send_only_if_event }) => {
+    }, async ({ campaign_id, step_order, subject, body_html, body_text, blocks, delay_days, delay_hours, variant_label, survey_id, brand_asset_ids, skip_if_event, send_only_if_event, skip_if_property, send_only_if_property }) => {
+        const properties = validateStepPropertyRules({ skip_if_property, send_only_if_property });
+        if (!properties.ok)
+            return j({ error: properties.error, code: "INVALID_INPUT" });
         const renderVars = {};
         if (survey_id) {
             const surveyLink = await resolveSurveyShortLink(db(), getProjectId(), survey_id);
@@ -803,7 +812,7 @@ export function registerCampaignTools(server) {
         const renderedText = body_text !== undefined && survey_id ? renderTemplateVars(body_text, renderVars) : body_text;
         const { data, error } = await db()
             .from('campaign_steps')
-            .insert({ campaign_id, step_order, subject: renderedSubject, body_html: renderedHtml, body_text: renderedText ?? (normalizedBlocks?.length ? blocksToText(normalizedBlocks) : undefined), blocks: normalizedBlocks ?? null, delay_days, delay_hours, variant_label: variant_label ?? 'A', skip_if_event: skip_if_event ?? null, send_only_if_event: send_only_if_event ?? null })
+            .insert({ campaign_id, step_order, subject: renderedSubject, body_html: renderedHtml, body_text: renderedText ?? (normalizedBlocks?.length ? blocksToText(normalizedBlocks) : undefined), blocks: normalizedBlocks ?? null, delay_days, delay_hours, variant_label: variant_label ?? 'A', skip_if_event: skip_if_event ?? null, send_only_if_event: send_only_if_event ?? null, ...properties.rules })
             .select()
             .single();
         if (error)
@@ -818,6 +827,7 @@ export function registerCampaignTools(server) {
         }).catch((err) => {
             console.warn('Failed to record brand asset usage for MCP campaign step:', err);
         });
+        await logAudit({ action: 'campaign.step_created', resourceType: 'campaign_step', resourceId: data.id, details: { property_rules: properties.rules } });
         return { content: [{ type: 'text', text: JSON.stringify(data, null, 2) }] };
     });
     server.registerTool('update_campaign_status', {
@@ -3101,7 +3111,7 @@ Return ONLY valid JSON with this exact shape:
             return j({ error: campaignErr?.message ?? 'Campaign not found', code: 'NOT_FOUND' });
         const { data, error } = await db()
             .from('campaign_steps')
-            .select('id, step_order, subject, body_html, body_text, blocks, delay_days, delay_hours, skip_if_event, send_only_if_event, created_at, updated_at')
+            .select('id, step_order, subject, body_html, body_text, blocks, delay_days, delay_hours, skip_if_event, send_only_if_event, skip_if_property, send_only_if_property, created_at, updated_at')
             .eq('campaign_id', campaign_id)
             .order('step_order', { ascending: true });
         if (error)
@@ -3127,6 +3137,8 @@ Return ONLY valid JSON with this exact shape:
             delay_hours: z.number().int().min(0).optional(),
             step_order: z.number().int().min(0).optional(),
             survey_id: z.string().uuid().optional(),
+            skip_if_property: propertyConditions.describe('Skip when ALL 1–10 conditions match current custom_properties. Missing/null keys never match. null clears.'),
+            send_only_if_property: propertyConditions.describe('Send only when ALL 1–10 conditions match current custom_properties. Unknown/non-match skips. null clears.'),
             skip_if_event: z.string().trim().min(1).nullable().optional().describe('Event name that skips this step at send time; null clears the rule.'),
             send_only_if_event: z.string().trim().min(1).nullable().optional().describe('Event required for this step to send; null clears the rule.'),
             link_actions: z.array(z.discriminatedUnion('action', [
@@ -3143,6 +3155,9 @@ Return ONLY valid JSON with this exact shape:
     }, async ({ step_id, ...updates }) => {
         const projectId = getProjectId();
         const { survey_id, ...stepUpdates } = updates;
+        const properties = validateStepPropertyRules(stepUpdates);
+        if (!properties.ok)
+            return j({ error: properties.error, code: "INVALID_INPUT" });
         const { data: step, error: stepErr } = await db()
             .from('campaign_steps')
             .select('id, campaign_id')
@@ -3179,10 +3194,11 @@ Return ONLY valid JSON with this exact shape:
             .from('campaign_steps')
             .update({ ...renderedUpdates, updated_at: new Date().toISOString() })
             .eq('id', step_id)
-            .select('id, step_order, subject, delay_days, delay_hours, skip_if_event, send_only_if_event, updated_at')
+            .select('id, step_order, subject, delay_days, delay_hours, skip_if_event, send_only_if_event, skip_if_property, send_only_if_property, updated_at')
             .single();
         if (error)
             return j({ error: error.message });
+        await logAudit({ action: 'campaign.step_updated', resourceType: 'campaign_step', resourceId: data.id, details: { property_rules: properties.rules } });
         return j({ step: data, ok: true });
     });
     server.registerTool('delete_campaign_step', {
@@ -3537,18 +3553,21 @@ Return ONLY valid JSON with this exact shape:
     server.registerTool('add_campaign_branch', {
         title: 'Add Campaign Branch',
         annotations: { readOnlyHint: false, destructiveHint: false, idempotentHint: false, openWorldHint: false },
-        description: 'Add an if/then rule to a campaign step so the sequence branches on what the contact does. Use this for conditional logic like: if they clicked the link send a different follow-up, if they replied skip ahead, if a tag was added move them to another campaign, otherwise continue. When the condition matches, the branch action runs instead of continuing to the next step. Lower priority is evaluated first.',
+        description: 'Add an if/then rule to a campaign step so the sequence branches on what the contact does or their current custom properties. Use this for conditional logic like: if they clicked the link send a different follow-up, if they replied skip ahead, if a tag was added move them to another campaign, otherwise continue. When the condition matches, the branch action runs instead of continuing to the next step. Lower priority is evaluated first.',
         inputSchema: {
             campaign_id: z.string().uuid(),
             from_step_id: z.string().uuid().describe('The step this branch leaves from'),
             condition_type: z.enum(CAMPAIGN_BRANCH_CONDITIONS).describe('What triggers the branch'),
-            condition_value: z.string().optional().describe('Condition argument, e.g. the tag name for tag_added or the URL for link_clicked'),
+            condition_value: z.union([z.string(), z.record(z.string(), z.unknown())]).optional().describe('Tag, URL or event name for existing conditions. For contact_property: {key, op: eq|neq, value: string} or {key, op: exists}; key matches [A-Za-z0-9_.-]{1,64}. Missing/null properties never match.'),
             action: z.enum(CAMPAIGN_BRANCH_ACTIONS).describe('goto_step and goto_campaign require their matching target'),
             target_step_id: z.string().uuid().optional().describe('Required when action is goto_step'),
             target_campaign_id: z.string().uuid().optional().describe('Required when action is goto_campaign'),
             priority: z.number().int().min(0).optional().default(0).describe('Lower runs first (default 0)'),
         },
     }, async ({ campaign_id, from_step_id, condition_type, condition_value, action, target_step_id, target_campaign_id, priority }) => {
+        const conditionValue = campaignBranchConditionValue(condition_type, condition_value);
+        if (!conditionValue.ok)
+            return { ...j({ error: conditionValue.error, code: 'INVALID_CONDITION_VALUE' }), isError: true };
         if (!(await campaignInProject(campaign_id)))
             return j({ error: 'Campaign not found in this project', code: 'CAMPAIGN_NOT_FOUND' });
         // Same requirement the REST route enforces: an action naming a target must have one,
@@ -3562,7 +3581,7 @@ Return ONLY valid JSON with this exact shape:
             campaign_id,
             from_step_id,
             condition_type,
-            condition_value: condition_value?.trim() || null,
+            condition_value: conditionValue.value,
             action,
             target_step_id: action === 'goto_step' ? target_step_id : null,
             target_campaign_id: action === 'goto_campaign' ? target_campaign_id : null,

package/package.json

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