@sendinel/mcp-server 1.0.20 → 1.0.22

package/build/tools/contacts.js

@@ -14,6 +14,7 @@ import { diffCustomProperties } from '../lib/custom-properties-diff.js';
 import Anthropic from '@anthropic-ai/sdk';
 import { getAnthropicModelId } from '../anthropic-model.js';
 const txt = (text) => ({ content: [{ type: 'text', text }] });
+const err = (text) => ({ ...txt(text), isError: true });
 const j = (data) => txt(JSON.stringify(data, null, 2));
 const CUSTOM_FIELD_TYPES = [
     'email', 'text', 'textarea', 'select', 'checkbox', 'number', 'tel', 'url', 'date',
@@ -183,6 +184,19 @@ function normalizeCompanyDomain(value) {
 function normalizeEmail(value) {
     return String(value ?? '').trim().toLowerCase();
 }
+function normalizePhoneE164(value) {
+    const digits = String(value ?? '').replace(/[^\d+]/g, '');
+    if (!digits)
+        return null;
+    if (digits.startsWith('+1') && digits.length === 12)
+        return digits;
+    const numeric = digits.replace(/\D/g, '');
+    if (numeric.length === 10)
+        return `+1${numeric}`;
+    if (numeric.length === 11 && numeric.startsWith('1'))
+        return `+${numeric}`;
+    return null;
+}
 function isPlainObject(value) {
     return Boolean(value) && typeof value === 'object' && !Array.isArray(value);
 }
@@ -398,8 +412,8 @@ const SUBSCRIBER_SELECT_WITH_COMPANY = 'id, email, first_name, last_name, compan
 const SUBSCRIBER_SELECT_NO_COMPANY = 'id, email, first_name, last_name, company_id, global_unsubscribed, source, created_at, last_emailed_at, total_emails_sent, total_opens, total_clicks, engagement_score, lifecycle_stage';
 const SEGMENT_SUBSCRIBER_SELECT_WITH_COMPANY = 'id, email, first_name, last_name, company_id, company:companies(id, name, domain), engagement_score, lifecycle_stage, last_opened_at, last_clicked_at, total_opens, total_clicks';
 const SEGMENT_SUBSCRIBER_SELECT_NO_COMPANY = 'id, email, first_name, last_name, company_id, engagement_score, lifecycle_stage, last_opened_at, last_clicked_at, total_opens, total_clicks';
-const UPDATE_SUBSCRIBER_SELECT_WITH_COMPANY = 'id, email, first_name, last_name, company_id, company:companies(id, name, domain), custom_properties, tags';
-const UPDATE_SUBSCRIBER_SELECT_NO_COMPANY = 'id, email, first_name, last_name, company_id, custom_properties, tags';
+const UPDATE_SUBSCRIBER_SELECT_WITH_COMPANY = 'id, email, first_name, last_name, company_id, company:companies(id, name, domain), custom_properties, tags, phone_e164, timezone';
+const UPDATE_SUBSCRIBER_SELECT_NO_COMPANY = 'id, email, first_name, last_name, company_id, custom_properties, tags, phone_e164, timezone';
 const GET_SUBSCRIBER_SELECT_WITH_COMPANY = '*, company:companies(id, name, domain), site_subscriptions(site_id, subscribed)';
 const GET_SUBSCRIBER_SELECT_NO_COMPANY = '*, site_subscriptions(site_id, subscribed)';
 const CONTACT_ENRICHMENT_ADAPTERS = createContactEnrichmentAdapters({
@@ -1464,7 +1478,7 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
             idempotentHint: true,
             openWorldHint: false,
         },
-        description: 'Update a contact\'s email, name, tags, or custom properties',
+        description: 'Update a contact\'s email, name, phone, timezone, tags, or custom properties',
         inputSchema: {
             contact_id: z.string().uuid(),
             email: z.string().email().optional(),
@@ -1475,8 +1489,10 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
             company_domain: z.string().nullable().optional(),
             tags: z.array(z.string()).optional(),
             custom_properties: z.record(z.string(), z.unknown()).optional().describe('Merged into existing custom_properties'),
+            phone_e164: z.string().nullable().optional().describe('E.164 phone number, e.g. +15551234567. SMS is US-only for now.'),
+            timezone: z.string().nullable().optional().describe('IANA time zone identifier, e.g. America/Los_Angeles'),
         },
-    }, async ({ contact_id, email, first_name, last_name, company_id, company_name, company_domain, tags, custom_properties }) => {
+    }, async ({ contact_id, email, first_name, last_name, company_id, company_name, company_domain, tags, custom_properties, phone_e164, timezone }) => {
         const projectId = getProjectId();
         const embedOk = await hasCompaniesEmbed(projectId);
         const updates = {};
@@ -1541,6 +1557,36 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
                 ...companyPatch,
             };
         }
+        if (phone_e164 !== undefined) {
+            if (phone_e164 === null) {
+                updates.phone_e164 = null;
+            }
+            else {
+                const normalized = normalizePhoneE164(phone_e164);
+                if (!normalized)
+                    return err('Invalid phone_e164: must be a valid E.164 phone number (SMS is US-only for now).');
+                updates.phone_e164 = normalized;
+            }
+        }
+        if (timezone !== undefined) {
+            if (timezone === null) {
+                updates.timezone = null;
+            }
+            else {
+                // Region form only; mirrors normalizeTimezoneField in the dashboard. Intl accepts UTC offsets
+                // ("+05:00") and abbreviations ("EST"), which Postgres AT TIME ZONE reads differently.
+                if (timezone !== 'UTC' && !/^[A-Za-z_]+(\/[A-Za-z0-9_+-]+)+$/.test(timezone)) {
+                    return err('Invalid timezone: must be an IANA region name such as America/Los_Angeles (not a UTC offset or abbreviation).');
+                }
+                try {
+                    new Intl.DateTimeFormat('en-US', { timeZone: timezone });
+                }
+                catch {
+                    return err('Invalid timezone: must be a valid IANA time zone identifier.');
+                }
+                updates.timezone = timezone;
+            }
+        }
         if (!Object.keys(updates).length)
             return txt('No fields to update.');
         updates.updated_at = new Date().toISOString();
@@ -1551,6 +1597,9 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
             .eq('project_id', projectId)
             .select(embedOk ? UPDATE_SUBSCRIBER_SELECT_WITH_COMPANY : UPDATE_SUBSCRIBER_SELECT_NO_COMPANY)
             .single();
+        if (error?.code === '23505' && updates.phone_e164 !== undefined) {
+            return err('Another contact in this project already has this phone number.');
+        }
         if (error)
             return txt(`Error: ${error.message}`);
         if (customPropertyChanges.length > 0) {
@@ -2235,81 +2284,62 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
     }, async ({ site_id }) => {
         const projectId = getProjectId();
         // Sample recent contacts to discover keys (LIMIT 500 is enough for schema discovery)
-        const { data, error } = await db().rpc('get_contact_schema_keys', {
-            p_project_id: projectId,
-            p_site_id: site_id,
-        });
-        if (error) {
-            // Fallback: inline query if RPC not yet available
-            const SCHEMA_SAMPLE_LIMIT = 500;
-            async function sampleSubscriberProperties(scopeToSite) {
-                let query = db()
-                    .from('subscribers')
-                    .select('custom_properties')
-                    .eq('project_id', projectId)
-                    .not('custom_properties', 'is', null)
-                    .order('created_at', { ascending: false });
-                if (scopeToSite)
-                    query = query.eq('source_site', site_id);
-                return query.limit(SCHEMA_SAMPLE_LIMIT);
-            }
-            const siteSample = await sampleSubscriberProperties(true);
-            if (siteSample.error)
-                return txt(`Error: ${siteSample.error.message}`);
-            let rows = siteSample.data ?? [];
-            let scope = 'site';
-            if (rows.length === 0) {
-                const projectSample = await sampleSubscriberProperties(false);
-                if (projectSample.error)
-                    return txt(`Error: ${projectSample.error.message}`);
-                rows = projectSample.data ?? [];
-                scope = 'project';
-            }
-            const keyMap = new Map();
-            for (const row of rows ?? []) {
-                const props = row.custom_properties;
-                if (!props)
-                    continue;
-                for (const [k, v] of Object.entries(props)) {
-                    if (!keyMap.has(k))
-                        keyMap.set(k, { values: new Set(), count: 0 });
-                    const entry = keyMap.get(k);
-                    entry.count++;
-                    if (v !== null && v !== undefined && entry.values.size < 3) {
-                        entry.values.add(String(v));
-                    }
+        const SCHEMA_SAMPLE_LIMIT = 500;
+        async function sampleSubscriberProperties(scopeToSite) {
+            let query = db()
+                .from('subscribers')
+                .select('custom_properties')
+                .eq('project_id', projectId)
+                .not('custom_properties', 'is', null)
+                .order('created_at', { ascending: false });
+            if (scopeToSite)
+                query = query.eq('source_site', site_id);
+            return query.limit(SCHEMA_SAMPLE_LIMIT);
+        }
+        const siteSample = await sampleSubscriberProperties(true);
+        if (siteSample.error)
+            return txt(`Error: ${siteSample.error.message}`);
+        let rows = siteSample.data ?? [];
+        let scope = 'site';
+        if (rows.length === 0) {
+            const projectSample = await sampleSubscriberProperties(false);
+            if (projectSample.error)
+                return txt(`Error: ${projectSample.error.message}`);
+            rows = projectSample.data ?? [];
+            scope = 'project';
+        }
+        const keyMap = new Map();
+        for (const row of rows ?? []) {
+            const props = row.custom_properties;
+            if (!props)
+                continue;
+            for (const [k, v] of Object.entries(props)) {
+                if (!keyMap.has(k))
+                    keyMap.set(k, { values: new Set(), count: 0 });
+                const entry = keyMap.get(k);
+                entry.count++;
+                if (v !== null && v !== undefined && entry.values.size < 3) {
+                    entry.values.add(String(v));
                 }
             }
-            const schema = Array.from(keyMap.entries())
-                .sort((a, b) => b[1].count - a[1].count)
-                .map(([key, { values, count }]) => ({
-                key,
-                merge_tag: `{{custom.${key}}}`,
-                merge_tag_with_fallback: `{{custom.${key}|fallback}}`,
-                example_values: Array.from(values),
-                contacts_with_this_key: count,
-            }));
-            return j({
-                site_id,
-                scope,
-                total_keys: schema.length,
-                schema,
-                note: scope === 'project'
-                    ? `No contacts are tagged to site ${site_id} via source_site — sampled custom_properties across the whole project instead. Use merge_tag syntax in email body/subject. For sensitive data, add a fallback: {{custom.plan|subscriber}}. ⚠️ custom_properties data passes through the Anthropic API — avoid storing sensitive PII (SSNs, passwords, financial account numbers).`
-                    : 'Use merge_tag syntax in email body/subject. For sensitive data, add a fallback: {{custom.plan|subscriber}}. ⚠️ custom_properties data passes through the Anthropic API — avoid storing sensitive PII (SSNs, passwords, financial account numbers).',
-            });
         }
+        const schema = Array.from(keyMap.entries())
+            .sort((a, b) => b[1].count - a[1].count)
+            .map(([key, { values, count }]) => ({
+            key,
+            merge_tag: `{{custom.${key}}}`,
+            merge_tag_with_fallback: `{{custom.${key}|fallback}}`,
+            example_values: Array.from(values),
+            contacts_with_this_key: count,
+        }));
         return j({
             site_id,
-            total_keys: data.length,
-            schema: data.map(r => ({
-                key: r.key,
-                merge_tag: `{{custom.${r.key}}}`,
-                merge_tag_with_fallback: `{{custom.${r.key}|fallback}}`,
-                example_values: r.examples ?? [],
-                contacts_with_this_key: r.count,
-            })),
-            note: 'Use merge_tag syntax in email body/subject. For sensitive data, add a fallback: {{custom.plan|subscriber}}. ⚠️ custom_properties data passes through the Anthropic API — avoid storing sensitive PII (SSNs, passwords, financial account numbers).',
+            scope,
+            total_keys: schema.length,
+            schema,
+            note: scope === 'project'
+                ? `No contacts are tagged to site ${site_id} via source_site — sampled custom_properties across the whole project instead. Use merge_tag syntax in email body/subject. For sensitive data, add a fallback: {{custom.plan|subscriber}}. ⚠️ custom_properties data passes through the Anthropic API — avoid storing sensitive PII (SSNs, passwords, financial account numbers).`
+                : 'Use merge_tag syntax in email body/subject. For sensitive data, add a fallback: {{custom.plan|subscriber}}. ⚠️ custom_properties data passes through the Anthropic API — avoid storing sensitive PII (SSNs, passwords, financial account numbers).',
         });
     });
     server.registerTool('create_custom_field', {

package/package.json

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