@sendinel/mcp-server 1.0.212 → 1.0.213

package/build/lib/how-to-recipes.js

@@ -112,12 +112,12 @@ export const HOW_TO_RECIPES = [
         requires: { scopes: ['write'], tester_ok: false },
         steps: [sender,
             { n: 2, tool: 'create_campaign', args_example: { site_id: from(1, '0.id'), name: 'Welcome new signups', type: 'triggered', trigger_event_name: 'signup', send_class: 'product' }, note: 'Confirm the actual event emitted by the integration; signup is an example. This creates a draft.' },
-            { n: 3, tool: 'add_campaign_step', args_example: { campaign_id: from(2, 'id'), step_order: 0, subject: 'Welcome!', body_html: '<p>Welcome aboard.</p>', delay_days: 0 } },
+            { n: 3, tool: 'add_campaign_step', args_example: { campaign_id: from(2, 'id'), step_order: 0, subject: 'Welcome!', body_html: '<p>Welcome aboard.</p>', delay_days: 0 }, note: 'Optional: pass template_id (+ template_vars) from create_template/list_templates instead of body_html.' },
             { n: 4, tool: 'send_test_email', args_example: { to_email: 'person@example.com', site_id: from(1, '0.id'), step_id: from(3, 'id'), dry_run: true }, note: 'Use an authorized preview address. Keep dry_run:true: render only, without approval.' },
             { n: 5, tool: 'activate_campaigns', args_example: { campaign_ids: [from(2, 'id')] }, approval, note: 'Show the campaign content and approve_url to the person; stop until approved.' },
             { n: 6, tool: 'activate_campaigns', args_example: { approval_id: from(5, 'approval_id') }, approval, note: 'Only after approval. Activation turns on the saved campaign; launch_campaign instead generates a NEW campaign from briefs.' },
         ],
-        gotchas: [projectRule, previewRule, 'Activation does not emit signup events or enroll historical signups. Verify the integration fires the configured event.'],
+        gotchas: [projectRule, previewRule, 'Activation does not emit signup events or enroll historical signups. Verify the integration fires the configured event.', 'A template_id on add_campaign_step is rendered once at step creation, so later template edits do not change existing steps.'],
         done_when: 'Activation succeeds and the integration’s signup event is configured; end-to-end delivery remains unverified until an authorized signup test completes.',
     },
     {
@@ -155,9 +155,9 @@ export const HOW_TO_RECIPES = [
             { n: 4, tool: 'preview_email', args_example: { site_id: from(1, '0.id'), subject: from(3, 'template.subject_hint'), body_html: from(3, 'template.body_html') }, note: 'Review the rendered content with sample contact data; this sends nothing.' },
             { n: 5, tool: 'list_templates', args_example: { search: 'Product update' }, note: 'Verify the reusable template is discoverable; paginate if truncated.' },
             { n: 6, tool: 'create_campaign', args_example: { site_id: from(1, '0.id'), name: 'Product update sequence', type: 'sequence' }, note: 'Optional use in a campaign: creates a new draft; use an existing draft instead if appropriate.' },
-            { n: 7, tool: 'add_campaign_step', args_example: { campaign_id: from(6, 'id'), step_order: 0, subject: from(3, 'template.subject_hint'), body_html: from(3, 'template.body_html') }, note: 'Copy content from get_template. No template_id parameter exists here; template_id/template_vars in the response are reserved for this path. This draft is not activated.' },
+            { n: 7, tool: 'add_campaign_step', args_example: { campaign_id: from(6, 'id'), step_order: 0, subject: from(3, 'template.subject_hint'), body_html: from(3, 'template.body_html') }, note: 'Optional: pass template_id (+ template_vars) from create_template/list_templates instead of body_html. This draft is not activated.' },
         ],
-        gotchas: [projectRule, previewRule, 'Later template edits do not update copied step content. Sequences need a separate enrollment/activation plan and approval. For a one-off segment broadcast use send_broadcast with copied html and subject, segment_id and optional send_at; approval is required.'],
+        gotchas: [projectRule, previewRule, 'A template_id on add_campaign_step is rendered once at step creation, so later template edits do not change existing steps. Sequences need a separate enrollment/activation plan and approval. For a one-off segment broadcast use send_broadcast with copied html and subject, segment_id and optional send_at; approval is required.'],
         done_when: 'The template is saved, previewed and listed; if requested, its content is copied into an unactivated campaign step.',
     },
     {

package/build/tools/campaigns.js

@@ -67,6 +67,54 @@ function stripHtml(value) {
         .replace(/\s+/g, ' ')
         .trim();
 }
+// One render path for send_transactional, send_awaiting_reply, and add_campaign_step.
+async function renderStoredCampaignTemplate(args) {
+    const { data: template, error: templateError } = await db()
+        .from('campaign_templates')
+        .select('id, subject_hint, body_html, body_text, format')
+        .eq('id', args.templateId)
+        .eq('project_id', args.projectId)
+        .maybeSingle();
+    if (templateError)
+        return { ok: false, reason: 'lookup_failed', dbError: templateError };
+    if (!template)
+        return { ok: false, reason: 'not_found' };
+    const { data: project } = await adminDb().from('projects').select('brand_settings').eq('id', args.projectId).maybeSingle();
+    const projectBrand = (project?.brand_settings ?? null);
+    const siteBrand = (args.site.brand_voice ?? null);
+    const mergedBrand = projectBrand || siteBrand ? { ...(projectBrand ?? {}), ...(siteBrand ?? {}) } : null;
+    const brandVars = resolveBrandTokens(mergedBrand, args.site.name || undefined, { address: args.site.physical_address ?? undefined });
+    const brandWarningCodes = brandWarnings(mergedBrand);
+    const vars = args.templateVars ?? {};
+    const renderVars = { ...brandVars, ...vars };
+    const subjectFallingBack = !args.subject?.trim();
+    const requiredVars = deriveTemplateVariables({
+        body_html: template.format === 'text' ? null : (template.body_html ?? ''),
+        body_text: template.format === 'text' ? (template.body_text ?? '') : null,
+        subject_hint: subjectFallingBack ? (template.subject_hint ?? '') : null,
+    });
+    const satisfied = new Set([
+        ...Object.keys(vars),
+        'brand_name', 'brand_logo_url', 'brand_bg_color', 'brand_text_color', 'brand_accent_color',
+        'brand_logo_html', 'brand_font', 'brand_address',
+        ...Object.keys(NAME_FALLBACKS),
+    ]);
+    const missing = requiredVars.filter((name) => !satisfied.has(name));
+    if (missing.length)
+        return { ok: false, reason: 'missing_vars', missing, brandWarningCodes };
+    let html;
+    let text;
+    if (template.format === 'text') {
+        text = renderTemplateVars(template.body_text ?? '', renderVars, { brandTokens: brandVars });
+        html = undefined;
+    }
+    else {
+        html = renderTemplateVars(template.body_html ?? '', renderVars, { brandTokens: brandVars });
+        text = args.callerText ?? stripHtml(html);
+    }
+    const subject = args.subject?.trim() || renderTemplateVars(template.subject_hint ?? '', renderVars, { brandTokens: brandVars });
+    return { ok: true, subject, html, text, brandWarningCodes };
+}
 function normalizeTriggerValue(value) {
     if (typeof value !== 'string')
         return null;
@@ -974,14 +1022,16 @@ export function registerCampaignTools(server) {
             idempotentHint: false,
             openWorldHint: false,
         },
-        description: 'Add an email step to a campaign. Requires a non-blank subject and body_html, body_text, or blocks that render to content. To use a stored template, call get_template and copy subject_hint into subject and body_html into body_html (and body_text if available). This tool accepts no template_id or template_vars; those response fields are reserved here and are not set by this tool. For A/B testing, add multiple steps with the same step_order but different variant_label values.',
+        description: 'Add an email step to a campaign. Requires a non-blank subject and body_html, body_text, or blocks that render to content, unless template_id is set. Optionally render from a template via template_id. When template_id is set without body content, the template is rendered once into subject and body_html. For A/B testing, add multiple steps with the same step_order but different variant_label values.',
         inputSchema: z.object({
             campaign_id: z.string().uuid(),
             step_order: z.number().int().min(0).describe('Position in sequence (0 = first)'),
-            subject: z.string().min(1),
+            subject: z.string().min(1).optional().describe('Email subject. Required unless template_id supplies a subject_hint.'),
             body_html: z.string().optional(),
             body_text: z.string().optional(),
             blocks: z.array(z.record(z.string(), z.unknown())).optional().describe('Optional structured Composer blocks that produced body_html.'),
+            template_id: z.string().uuid().optional().describe('campaign_templates row to render into this step. When set, body_html/blocks are not required; explicit body_html/body_text/blocks win over the template body.'),
+            template_vars: z.record(z.string(), z.string()).optional().describe('Variable values for template_id rendering. Wins over brand tokens on key collision.'),
             delay_days: z.number().int().min(0).optional().default(0),
             delay_hours: z.number().int().min(0).max(23).optional().default(0),
             variant_label: z.string().optional().default('A').describe('Variant label for A/B testing (default: A). Use B, C, etc. for additional variants.'),
@@ -994,7 +1044,7 @@ export function registerCampaignTools(server) {
             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.'),
         }).refine(absencePairValid, { message: absencePairError }),
-    }, 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_events, send_only_if_events, send_only_if_events_mode, send_only_if_no_event, send_only_if_no_event_days, skip_if_property, send_only_if_property }) => {
+    }, async ({ campaign_id, step_order, subject, body_html, body_text, blocks, template_id, template_vars, delay_days, delay_hours, variant_label, survey_id, brand_asset_ids, skip_if_event, send_only_if_event, skip_if_events, send_only_if_events, send_only_if_events_mode, send_only_if_no_event, send_only_if_no_event_days, skip_if_property, send_only_if_property }) => {
         try {
             const absence = validateStepAbsenceRule({ send_only_if_no_event, send_only_if_no_event_days });
             if (!absence.ok)
@@ -1005,22 +1055,29 @@ export function registerCampaignTools(server) {
             const properties = validateStepPropertyRules({ skip_if_property, send_only_if_property });
             if (!properties.ok)
                 return j({ error: properties.error, code: "INVALID_INPUT" });
+            if (template_id && body_html !== undefined) {
+                return j({ error: "Provide either 'template_id' or 'body_html', not both", code: 'INVALID_INPUT' });
+            }
             const normalizedBlocks = blocks ? normalizeComposerBlocks(blocks) : undefined;
             const htmlFromBlocks = normalizedBlocks?.length ? blocksToHtml(normalizedBlocks) : undefined;
-            if (!subject.trim())
-                return j({ error: 'A step needs a non-blank subject.', code: 'validation_failed' });
-            const hasBody = body_html?.trim() || body_text?.trim()
+            const renderingTemplate = Boolean(template_id) && body_html === undefined && body_text === undefined && blocks === undefined;
+            const explicitHasBody = () => Boolean(body_html?.trim() || body_text?.trim()
                 || htmlFromBlocks?.replace(/<[^>]*>/g, '').trim()
-                || normalizedBlocks?.some((block) => block.type === 'image' && block.url);
-            if (!hasBody) {
-                return j({ error: 'A step needs body_html or body_text (or blocks that render to content).', code: 'validation_failed' });
+                || normalizedBlocks?.some((block) => block.type === 'image' && block.url));
+            if (!renderingTemplate) {
+                if (!subject?.trim())
+                    return j({ error: 'A step needs a non-blank subject.', code: 'validation_failed' });
+                if (!explicitHasBody()) {
+                    return j({ error: 'A step needs body_html or body_text (or blocks that render to content).', code: 'validation_failed' });
+                }
             }
+            const projectId = getProjectId();
             const { data: campaign, error: campaignError } = await db()
                 .schema('email')
                 .from('campaigns')
-                .select('id')
+                .select('id, site_id')
                 .eq('id', campaign_id)
-                .eq('project_id', getProjectId())
+                .eq('project_id', projectId)
                 .maybeSingle();
             if (campaignError)
                 return dbErrorResult(campaignError);
@@ -1028,24 +1085,81 @@ export function registerCampaignTools(server) {
                 return { ...j({ error: 'Campaign not found.', code: 'not_found' }), isError: true };
             const renderVars = {};
             if (survey_id) {
-                const surveyLink = await resolveSurveyShortLink(db(), getProjectId(), survey_id);
+                const surveyLink = await resolveSurveyShortLink(db(), projectId, survey_id);
                 if (!surveyLink)
                     return j({ error: 'Survey has no short link yet', code: 'SURVEY_LINK_NOT_READY' });
                 renderVars.survey_link = surveyLink;
             }
-            const renderedSubject = survey_id ? renderTemplateVars(subject, renderVars) : subject;
-            const renderedHtml = body_html !== undefined && survey_id ? renderTemplateVars(body_html, renderVars) : body_html ?? htmlFromBlocks;
-            const renderedText = body_text !== undefined && survey_id ? renderTemplateVars(body_text, renderVars) : body_text;
+            let bakedSubject = subject;
+            let bakedHtml = body_html;
+            let bakedText = body_text;
+            if (renderingTemplate && template_id) {
+                let siteForTemplate = {};
+                if (campaign.site_id) {
+                    const { data: site, error: siteError } = await db()
+                        .from('sites')
+                        .select('name, brand_voice, physical_address')
+                        .eq('id', campaign.site_id)
+                        .eq('project_id', projectId)
+                        .maybeSingle();
+                    if (siteError)
+                        return dbErrorResult(siteError);
+                    if (site)
+                        siteForTemplate = site;
+                }
+                const rendered = await renderStoredCampaignTemplate({
+                    templateId: template_id,
+                    templateVars: { ...(template_vars ?? {}), ...renderVars },
+                    subject,
+                    projectId,
+                    site: siteForTemplate,
+                });
+                if (!rendered.ok) {
+                    if (rendered.reason === 'lookup_failed')
+                        return dbErrorResult(rendered.dbError);
+                    if (rendered.reason === 'not_found')
+                        return { ...j({ error: 'Template not found.', code: 'not_found' }), isError: true };
+                    return j({ error: `Missing required template variable(s): ${rendered.missing.join(', ')}`, code: 'validation_failed' });
+                }
+                bakedSubject = rendered.subject;
+                bakedHtml = rendered.html === undefined ? undefined : stripActiveEmailContent(rendered.html).html;
+                bakedText = rendered.text;
+                if (!bakedSubject?.trim())
+                    return j({ error: 'A step needs a non-blank subject.', code: 'validation_failed' });
+                const bakedHasBody = bakedHtml?.trim() || bakedText?.trim();
+                if (!bakedHasBody) {
+                    return j({ error: 'A step needs body_html or body_text (or blocks that render to content).', code: 'validation_failed' });
+                }
+            }
+            else if (template_id) {
+                const { data: template, error: templateError } = await db()
+                    .from('campaign_templates')
+                    .select('id')
+                    .eq('id', template_id)
+                    .eq('project_id', projectId)
+                    .maybeSingle();
+                if (templateError)
+                    return dbErrorResult(templateError);
+                if (!template)
+                    return { ...j({ error: 'Template not found.', code: 'not_found' }), isError: true };
+            }
+            const renderedSubject = survey_id && bakedSubject !== undefined ? renderTemplateVars(bakedSubject, renderVars) : bakedSubject;
+            const renderedHtml = renderingTemplate
+                ? (survey_id && bakedHtml !== undefined ? renderTemplateVars(bakedHtml, renderVars) : bakedHtml)
+                : (body_html !== undefined && survey_id ? renderTemplateVars(body_html, renderVars) : body_html ?? htmlFromBlocks);
+            const renderedText = renderingTemplate
+                ? (survey_id && bakedText !== undefined ? renderTemplateVars(bakedText, renderVars) : bakedText)
+                : (body_text !== undefined && survey_id ? renderTemplateVars(body_text, renderVars) : body_text);
             const { data, error } = await withCreatedItems('campaign_steps', 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, ...events.rules, ...absence.rules, ...properties.rules })
+                .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', template_id: template_id ?? null, template_vars: template_vars ?? null, skip_if_event: skip_if_event ?? null, send_only_if_event: send_only_if_event ?? null, ...events.rules, ...absence.rules, ...properties.rules })
                 .select()
                 .single());
             if (error)
                 return dbErrorResult(error);
             await recordBrandAssetUsage({
                 client: db(),
-                projectId: getProjectId(),
+                projectId,
                 source: 'campaign_step',
                 resourceId: data.id,
                 html: renderedHtml,
@@ -1053,7 +1167,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 } });
+            await logAudit({ action: 'campaign.step_created', resourceType: 'campaign_step', resourceId: data.id, details: { property_rules: properties.rules, template_id: template_id ?? null } });
             return { content: [{ type: 'text', text: JSON.stringify(data, null, 2) }] };
         }
         catch (error) {
@@ -2708,53 +2822,31 @@ Do not include body_html in your response; Sendinel renders body_html from block
         let resolvedText = text;
         let brandWarningCodes = [];
         if (template_id) {
-            const { data: template, error: templateError } = await db()
-                .from('campaign_templates')
-                .select('id, subject_hint, body_html, body_text, format')
-                .eq('id', template_id)
-                .eq('project_id', projectId)
-                .maybeSingle();
-            if (templateError) {
-                console.error('MCP send template lookup failed:', templateError);
-                return { ...txt('Error: Internal server error'), ...sendError('internal_error') };
+            const rendered = await renderStoredCampaignTemplate({
+                templateId: template_id,
+                templateVars: template_vars,
+                subject,
+                projectId,
+                site,
+                callerText: text,
+            });
+            if (!rendered.ok) {
+                if (rendered.reason === 'lookup_failed') {
+                    console.error('MCP send template lookup failed:', rendered.dbError);
+                    return { ...txt('Error: Internal server error'), ...sendError('internal_error') };
+                }
+                if (rendered.reason === 'not_found')
+                    return txt('Error: Template not found');
+                if (rendered.brandWarningCodes.length)
+                    console.warn('MCP send brand warnings', { projectId, site_id, warnings: rendered.brandWarningCodes });
+                return txt(`Error: Missing required template variable(s): ${rendered.missing.join(', ')}`);
             }
-            if (!template)
-                return txt('Error: Template not found');
-            const { data: project } = await adminDb().from('projects').select('brand_settings').eq('id', projectId).maybeSingle();
-            const projectBrand = (project?.brand_settings ?? null);
-            const siteBrand = (site.brand_voice ?? null);
-            const mergedBrand = projectBrand || siteBrand ? { ...(projectBrand ?? {}), ...(siteBrand ?? {}) } : null;
-            const brandVars = resolveBrandTokens(mergedBrand, site.name || undefined, { address: site.physical_address ?? undefined });
-            brandWarningCodes = brandWarnings(mergedBrand);
+            brandWarningCodes = rendered.brandWarningCodes;
             if (brandWarningCodes.length)
                 console.warn('MCP send brand warnings', { projectId, site_id, warnings: brandWarningCodes });
-            const vars = template_vars ?? {};
-            const renderVars = { ...brandVars, ...vars };
-            const subjectFallingBack = !subject?.trim();
-            const requiredVars = deriveTemplateVariables({
-                body_html: template.format === 'text' ? null : (template.body_html ?? ''),
-                body_text: template.format === 'text' ? (template.body_text ?? '') : null,
-                subject_hint: subjectFallingBack ? (template.subject_hint ?? '') : null,
-            });
-            const satisfied = new Set([
-                ...Object.keys(vars),
-                'brand_name', 'brand_logo_url', 'brand_bg_color', 'brand_text_color', 'brand_accent_color',
-                'brand_logo_html', 'brand_font', 'brand_address',
-                ...Object.keys(NAME_FALLBACKS),
-            ]);
-            const missing = requiredVars.filter((name) => !satisfied.has(name));
-            if (missing.length) {
-                return txt(`Error: Missing required template variable(s): ${missing.join(', ')}`);
-            }
-            if (template.format === 'text') {
-                resolvedText = renderTemplateVars(template.body_text ?? '', renderVars, { brandTokens: brandVars });
-                resolvedHtml = undefined;
-            }
-            else {
-                resolvedHtml = renderTemplateVars(template.body_html ?? '', renderVars, { brandTokens: brandVars });
-                resolvedText = text ?? stripHtml(resolvedHtml);
-            }
-            resolvedSubject = subject?.trim() || renderTemplateVars(template.subject_hint ?? '', renderVars, { brandTokens: brandVars });
+            resolvedSubject = rendered.subject;
+            resolvedHtml = rendered.html;
+            resolvedText = rendered.text;
         }
         if (resolvedHtml !== undefined)
             resolvedHtml = stripActiveEmailContent(resolvedHtml).html;
@@ -2904,51 +2996,29 @@ Do not include body_html in your response; Sendinel renders body_html from block
         let resolvedText = text;
         let brandWarningCodes = [];
         if (template_id) {
-            const { data: template, error: templateError } = await db()
-                .from('campaign_templates')
-                .select('id, subject_hint, body_html, body_text, format')
-                .eq('id', template_id)
-                .eq('project_id', projectId)
-                .maybeSingle();
-            if (templateError)
-                return txt(`Error: ${dbToolError(templateError).error}`);
-            if (!template)
-                return txt('Error: Template not found');
-            const { data: project } = await adminDb().from('projects').select('brand_settings').eq('id', projectId).maybeSingle();
-            const projectBrand = (project?.brand_settings ?? null);
-            const siteBrand = (site.brand_voice ?? null);
-            const mergedBrand = projectBrand || siteBrand ? { ...(projectBrand ?? {}), ...(siteBrand ?? {}) } : null;
-            const brandVars = resolveBrandTokens(mergedBrand, site.name || undefined, { address: site.physical_address ?? undefined });
-            brandWarningCodes = brandWarnings(mergedBrand);
-            if (brandWarningCodes.length)
-                console.warn('MCP send brand warnings', { projectId, site_id, warnings: brandWarningCodes });
-            const vars = template_vars ?? {};
-            const renderVars = { ...brandVars, ...vars };
-            const subjectFallingBack = !subject?.trim();
-            const requiredVars = deriveTemplateVariables({
-                body_html: template.format === 'text' ? null : (template.body_html ?? ''),
-                body_text: template.format === 'text' ? (template.body_text ?? '') : null,
-                subject_hint: subjectFallingBack ? (template.subject_hint ?? '') : null,
+            const rendered = await renderStoredCampaignTemplate({
+                templateId: template_id,
+                templateVars: template_vars,
+                subject,
+                projectId,
+                site,
+                callerText: text,
             });
-            const satisfied = new Set([
-                ...Object.keys(vars),
-                'brand_name', 'brand_logo_url', 'brand_bg_color', 'brand_text_color', 'brand_accent_color',
-                'brand_logo_html', 'brand_font', 'brand_address',
-                ...Object.keys(NAME_FALLBACKS),
-            ]);
-            const missing = requiredVars.filter((name) => !satisfied.has(name));
-            if (missing.length) {
-                return txt(`Error: Missing required template variable(s): ${missing.join(', ')}`);
+            if (!rendered.ok) {
+                if (rendered.reason === 'lookup_failed')
+                    return txt(`Error: ${dbToolError(rendered.dbError).error}`);
+                if (rendered.reason === 'not_found')
+                    return txt('Error: Template not found');
+                if (rendered.brandWarningCodes.length)
+                    console.warn('MCP send brand warnings', { projectId, site_id, warnings: rendered.brandWarningCodes });
+                return txt(`Error: Missing required template variable(s): ${rendered.missing.join(', ')}`);
             }
-            if (template.format === 'text') {
-                resolvedText = renderTemplateVars(template.body_text ?? '', renderVars, { brandTokens: brandVars });
-                resolvedHtml = undefined;
-            }
-            else {
-                resolvedHtml = renderTemplateVars(template.body_html ?? '', renderVars, { brandTokens: brandVars });
-                resolvedText = text ?? stripHtml(resolvedHtml);
-            }
-            resolvedSubject = subject?.trim() || renderTemplateVars(template.subject_hint ?? '', renderVars, { brandTokens: brandVars });
+            brandWarningCodes = rendered.brandWarningCodes;
+            if (brandWarningCodes.length)
+                console.warn('MCP send brand warnings', { projectId, site_id, warnings: brandWarningCodes });
+            resolvedSubject = rendered.subject;
+            resolvedHtml = rendered.html;
+            resolvedText = rendered.text;
         }
         if (resolvedHtml !== undefined)
             resolvedHtml = stripActiveEmailContent(resolvedHtml).html;

package/build/tools/templates.js

@@ -241,10 +241,10 @@ export function registerTemplateTools(server) {
             idempotentHint: false,
             openWorldHint: false,
         },
-        description: 'Save an email template from supplied content. brief is a required description/label stored as brief_template, even with body_html; this tool performs no AI generation and consumes no AI credits. Use create_template_from_brief to AI-generate content using the site brand voice.',
+        description: 'Create a template record from content you supply. `brief` is a stored note and does NOT trigger AI generation or use credits; use create_template_from_brief to AI-generate.',
         inputSchema: {
             name: templateName,
-            brief: z.string().min(1).describe('Required description/label; stored only, no AI generation or AI credits.'),
+            brief: z.string().min(1).describe('Stored as brief_template only. Does NOT trigger AI generation or use credits. Use create_template_from_brief to AI-generate.'),
             subject_hint: z.string().optional(),
             body_html: z.string().optional(),
             category: z.string().optional(),

package/package.json

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