@sendinel/mcp-server 1.0.74 → 1.0.75

package/build/tools/campaigns.js

@@ -2288,7 +2288,7 @@ Do not include body_html in your response; Sendinel renders body_html from block
             idempotentHint: false,
             openWorldHint: true,
         },
-        description: 'Send a test email to a specified address using the site\'s configured provider. Content from step_id, draft_id, or raw subject/body_html. Logged as test, not counted in campaign metrics.',
+        description: 'Send a test email to a specified address using the site\'s configured provider. Content from step_id, draft_id, or raw subject/body_html uses the real send renderer. Optional dry_run previews without sending; suppressed recipients are refused. Logged as test, not counted in campaign metrics.',
         inputSchema: {
             approval_id: z.string().uuid().optional().describe('Get this by calling once without it, then have a human approve the returned approve_url. Re-call with the same arguments plus this approval_id to run the action.'),
             to_email: z.string().email().describe('Email address to send the test to'),
@@ -2298,91 +2298,34 @@ Do not include body_html in your response; Sendinel renders body_html from block
             subject: z.string().optional(),
             body_html: z.string().optional(),
             body_text: z.string().optional(),
-        },
-    }, withApproval('send_test_email', async ({ to_email, site_id, step_id, draft_id, subject, body_html, body_text }) => {
-        const projectId = getProjectId();
-        let subj = subject ?? '';
-        let html = body_html ?? '';
-        let text = body_text ?? '';
-        if (!subj && !html) {
-            if (step_id) {
-                const { data: step } = await db().from('campaign_steps').select('subject, body_html, body_text').eq('id', step_id).maybeSingle();
-                if (!step)
-                    return txt('Error: Step not found');
-                subj = step.subject;
-                html = step.body_html ?? '';
-                text = step.body_text ?? '';
-            }
-            else if (draft_id) {
-                const { data: draft } = await db().from('ai_drafts').select('subject, body_html, body_text').eq('id', draft_id).eq('project_id', projectId).maybeSingle();
-                if (!draft)
-                    return txt('Error: Draft not found');
-                subj = draft.subject;
-                html = draft.body_html ?? '';
-                text = draft.body_text ?? '';
-            }
-            else {
+            dry_run: z.boolean().optional().describe('Render a preview through the real send renderer without sending or writing email_log.'),
+        },
+    }, async (args, extra) => {
+        // Fail before the approval flow can touch the DB when stdio is not configured.
+        if (!requestContext()?.v1Caller && !process.env.SENDINEL_API_KEY) {
+            const result = await callSendinelV1('/api/v1/campaigns/test-send');
+            if (!result.ok)
+                return j({ error: result.error, status: result.status });
+        }
+        return withApproval('send_test_email', async ({ to_email, site_id, step_id, draft_id, subject, body_html, body_text, dry_run }) => {
+            // Same precedence the tool always had, so callers passing several sources keep
+            // working: raw subject/body_html wins, then step_id, then draft_id. The route
+            // takes exactly one source.
+            const source = subject || body_html ? { subject, body_html, body_text }
+                : step_id ? { step_id }
+                    : draft_id ? { draft_id }
+                        : body_text ? { body_text }
+                            : null;
+            if (!source)
                 return txt('Provide content via subject/body_html, step_id, or draft_id.');
-            }
-        }
-        const resolve = (t) => t
-            .replace(/\{\{first_name\}\}/g, 'Test')
-            .replace(/\{\{email\}\}/g, to_email)
-            .replace(/\{\{unsubscribe_url\}\}/g, '#test-unsubscribe');
-        // Suppression check — don't send test emails to bounced/complained addresses
-        const { data: suppressed } = await db()
-            .from('suppressions')
-            .select('reason')
-            .eq('project_id', projectId)
-            .eq('email', to_email.toLowerCase())
-            .maybeSingle();
-        if (suppressed)
-            return txt(`Error: Recipient suppressed (${suppressed.reason}). Test emails to suppressed addresses risk domain reputation.`);
-        const { data: site } = await db()
-            .from('sites')
-            .select('from_email, from_name')
-            .eq('id', site_id)
-            .eq('project_id', projectId)
-            .maybeSingle();
-        if (!site)
-            return txt('Error: Site not found');
-        const resolvedHtml = resolve(html);
-        const resolvedText = resolve(text);
-        const payload = {
-            from: `${site.from_name} <${site.from_email}>`,
-            to: to_email,
-            subject: `[TEST] ${resolve(subj)}`,
-            html: resolvedHtml || undefined,
-            text: resolvedText || undefined,
-        };
-        let sendResult;
-        try {
-            sendResult = await sendEmailForSite(projectId, site_id, payload);
-        }
-        catch (error) {
-            return txt(`Error: Send failed — ${error instanceof Error ? error.message : 'unknown provider error'}`);
-        }
-        await db().from('email_log').insert({
-            project_id: projectId,
-            site_id,
-            contact_id: null,
-            to_email,
-            from_email: site.from_email,
-            subject: resolve(subj),
-            status: 'sent',
-            category: 'test',
-            sent_at: new Date().toISOString(),
-            provider_message_id: sendResult.messageId ?? null,
-            body_html: resolvedHtml ?? null,
-        });
-        return j({
-            sent: true,
-            to: to_email,
-            provider: sendResult.provider,
-            provider_message_id: sendResult.messageId,
-            resend_id: sendResult.provider === 'resend' ? sendResult.messageId : null,
-        });
-    }));
+            // All sources use the dashboard's shared renderer. In stdio mode callSendinelV1
+            // requires SENDINEL_API_KEY; never fall back to a separate local renderer.
+            const result = await callSendinelV1('/api/v1/campaigns/test-send', {
+                body: { to_email, site_id, ...source, dry_run },
+            });
+            return result.ok ? j(result.data) : j({ error: result.error, status: result.status });
+        })(args, extra);
+    });
     server.registerTool('send_transactional', {
         title: 'Send Transactional',
         annotations: {

package/package.json

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