@sendinel/mcp-server 1.0.113 → 1.0.115

package/build/tools/campaigns.js

@@ -31,6 +31,7 @@ import { callSendinelV1 } from '../lib/v1-client.js';
 import { countUniqueEngagementByCampaign } from '../lib/campaign-engagement.js';
 const txt = (text) => ({ content: [{ type: 'text', text }] });
 const j = (data) => txt(JSON.stringify(data, null, 2));
+const ACTIVATION_APPROVAL_DESCRIPTION = 'Activation-only: requires_approval gates campaign activation. It does not stop sends of an already-active campaign or sends triggered by other paths.';
 const SENT_EMAIL_STATUSES = ['sent', 'delivered', 'opened', 'clicked', 'bounced', 'complained', 'unsubscribed'];
 function stripHtml(value) {
     return value
@@ -557,13 +558,13 @@ export function registerCampaignTools(server) {
             idempotentHint: true,
             openWorldHint: true,
         },
-        description: 'Run the campaign portfolio review and optionally raise agent proposals for start/end/modify recommendations.',
+        description: 'Run the campaign portfolio review. Defaults to a dry run (no writes); pass dry_run=false to raise agent proposals for start/end/modify recommendations and emit recommendation signals.',
         inputSchema: {
-            dry_run: z.boolean().optional().default(false),
+            dry_run: z.boolean().optional().default(true).describe('Preview only by default: computes findings and writes nothing. Pass false to raise agent proposals and emit recommendation signals.'),
             dormant_draft_days: z.number().int().min(1).max(365).optional(),
             abandoned_days: z.number().int().min(1).max(365).optional(),
         },
-    }, async ({ dry_run = false, dormant_draft_days, abandoned_days }) => {
+    }, async ({ dry_run = true, dormant_draft_days, abandoned_days }) => {
         const result = await callSendinelV1('/api/v1/campaigns/review', {
             body: {
                 dry_run,
@@ -642,7 +643,7 @@ export function registerCampaignTools(server) {
             idempotentHint: true,
             openWorldHint: false,
         },
-        description: 'List campaigns with saved trigger fields, fires_on summaries and trigger_diagnostics; optionally filter by site or status.',
+        description: 'List campaigns with saved trigger fields, fires_on summaries and trigger_diagnostics; optionally filter by site or status. ' + ACTIVATION_APPROVAL_DESCRIPTION,
         inputSchema: {
             site_id: z.string().uuid().optional(),
             period: z.string().min(1).optional().describe("Return one enrollment period in by_period; totals remain all-time."),
@@ -652,7 +653,7 @@ export function registerCampaignTools(server) {
     }, async ({ site_id, status, limit, period }) => {
         let q = db()
             .from('campaigns')
-            .select('id, site_id, name, type, status, sends_per_hour, total_enrolled, total_sent, total_opened, total_clicked, created_at, period_key, segment_id, trigger_type, trigger_event_name, trigger_tag, trigger_config, send_class, unsub_footer_template')
+            .select('id, site_id, name, type, status, sends_per_hour, total_enrolled, total_sent, total_opened, total_clicked, created_at, period_key, segment_id, trigger_type, trigger_event_name, trigger_tag, trigger_config, send_class, unsub_footer_template, requires_approval, approved_by, approved_at')
             .eq('project_id', getProjectId())
             .order('created_at', { ascending: false })
             .limit(limit ?? 25);
@@ -672,6 +673,7 @@ export function registerCampaignTools(server) {
             const { period_key: _periodKey, ...legacy } = row;
             return {
                 ...legacy,
+                requires_approval_description: ACTIVATION_APPROVAL_DESCRIPTION,
                 fires_on: buildCampaignFiresOn(row),
                 trigger_diagnostics: buildCampaignTriggerDiagnostics(row),
                 ...campaignPeriodFields(periods, period),
@@ -735,7 +737,7 @@ export function registerCampaignTools(server) {
             type: z.enum(['sequence', 'broadcast', 'triggered']),
             description: z.string().optional(),
             sends_per_hour: z.number().int().min(1).nullable().optional().describe('Optional hourly send cap for wave sending. Null or omitted sends without a campaign-specific cap.'),
-            requires_approval: z.boolean().optional().default(true),
+            requires_approval: z.boolean().optional().default(true).describe(ACTIVATION_APPROVAL_DESCRIPTION),
             trigger_event_name: z.string().optional().describe('For type=triggered: event name that fires this campaign'),
             trigger_tag: z.string().optional().describe('For type=triggered: contact tag that fires this campaign when applied'),
             trigger_type: z.enum(['date_field']).optional().describe('Set for elapsed-time triggers keyed on a contact custom_properties date field'),
@@ -773,7 +775,7 @@ export function registerCampaignTools(server) {
             .single();
         if (error)
             return { content: [{ type: 'text', text: `Error: ${error.message}` }] };
-        return { content: [{ type: 'text', text: JSON.stringify(data, null, 2) }] };
+        return { content: [{ type: 'text', text: JSON.stringify({ ...data, requires_approval_description: ACTIVATION_APPROVAL_DESCRIPTION }, null, 2) }] };
     });
     const propertyConditions = z.array(z.union([
         z.object({ key: z.string(), op: z.enum(['eq', 'neq']), value: z.string() }).strict(),
@@ -849,7 +851,7 @@ export function registerCampaignTools(server) {
             idempotentHint: true,
             openWorldHint: true,
         },
-        description: 'Activate, pause, or archive a campaign. Activating requires human approval via the dashboard — call without approval_id to request one, then call again with the approved token.',
+        description: 'Activate, pause, or archive a campaign. Activating requires human approval via the dashboard — call without approval_id to request one, then call again with the approved token. ' + ACTIVATION_APPROVAL_DESCRIPTION,
         inputSchema: {
             campaign_id: z.string().uuid(),
             status: z.enum(['draft', 'active', 'paused', 'completed', 'archived']),
@@ -3558,7 +3560,7 @@ Return ONLY valid JSON with this exact shape:
             idempotentHint: true,
             openWorldHint: false,
         },
-        description: 'Get full campaign details including all steps, enrollment counts and saved trigger_type, trigger_event_name, trigger_tag, trigger_config (including where filter), fires_on and trigger_diagnostics. Read what event a dunning campaign fires on; use set_campaign_trigger to change it.',
+        description: 'Get full campaign details including all steps, enrollment counts and saved trigger_type, trigger_event_name, trigger_tag, trigger_config (including where filter), fires_on and trigger_diagnostics. Read what event a dunning campaign fires on; use set_campaign_trigger to change it. ' + ACTIVATION_APPROVAL_DESCRIPTION,
         inputSchema: { campaign_id: z.string().uuid(), period: z.string().min(1).optional().describe("Return results for one enrollment period; totals remain all-time.") },
     }, async ({ campaign_id, period }) => {
         const projectId = getProjectId();
@@ -3629,6 +3631,7 @@ Return ONLY valid JSON with this exact shape:
         const unresolvedCount = dlqUnresolvedCount ?? 0;
         return j({
             ...campaign,
+            requires_approval_description: ACTIVATION_APPROVAL_DESCRIPTION,
             fires_on: buildCampaignFiresOn(campaign),
             trigger_diagnostics: buildCampaignTriggerDiagnostics(campaign),
             total_sent: periods?.totals?.sent ?? sentCount ?? 0,

package/package.json

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