@sendinel/mcp-server 1.0.145 → 1.0.146

package/build/lib/tags.d.ts

@@ -0,0 +1,10 @@
+/** Normalize input only: preserve first-seen spelling for exact-match consumers. */
+export declare function normalizeTagInput(input: string[]): {
+    tags: string[];
+    dropped_empty: number;
+    case_duplicates: {
+        kept: string;
+        dropped: string;
+    }[];
+};
+export declare function tagInputWarnings(result: ReturnType<typeof normalizeTagInput>): string[];

package/build/lib/tags.js

@@ -0,0 +1,29 @@
+/** Normalize input only: preserve first-seen spelling for exact-match consumers. */
+export function normalizeTagInput(input) {
+    const tags = [];
+    const seen = new Map();
+    let dropped_empty = 0;
+    const case_duplicates = [];
+    for (const value of input) {
+        const tag = value.trim();
+        if (!tag) {
+            dropped_empty++;
+            continue;
+        }
+        const key = tag.toLowerCase();
+        const kept = seen.get(key);
+        if (kept !== undefined) {
+            case_duplicates.push({ kept, dropped: tag });
+            continue;
+        }
+        seen.set(key, tag);
+        tags.push(tag);
+    }
+    return { tags, dropped_empty, case_duplicates };
+}
+export function tagInputWarnings(result) {
+    return [
+        ...(result.dropped_empty ? [`Dropped ${result.dropped_empty} empty tag(s).`] : []),
+        ...result.case_duplicates.map(({ kept, dropped }) => `Tag "${dropped}" duplicates "${kept}" (case-insensitive); kept "${kept}".`),
+    ];
+}

package/build/tools/contacts.js

@@ -1,3 +1,4 @@
+import { normalizeTagInput, tagInputWarnings } from '../lib/tags.js';
 import { dbErrorResult } from '../lib/db-errors.js';
 import { ResourceTemplate } from '@modelcontextprotocol/sdk/server/mcp.js';
 import { CONTACT_ENRICHMENT_PROVIDERS } from '@desler/models';
@@ -1299,6 +1300,7 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
             company_id: z.string().uuid().optional(),
             company_name: z.string().optional(),
             company_domain: z.string().optional(),
+            tags: z.array(z.string()).optional().describe('Initial tags for new subscribers; existing contacts keep their stored tags.'),
             source: z.enum(['signup', 'import', 'form', 'manual']).optional().default('manual'),
             utm_source: z.string().optional(),
             utm_medium: z.string().optional(),
@@ -1306,8 +1308,11 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
             utm_content: z.string().optional(),
             utm_term: z.string().optional(),
         },
-    }, async ({ email, site_id, first_name, last_name, company_id, company_name, company_domain, source, utm_source, utm_medium, utm_campaign, utm_content, utm_term }) => {
+    }, async ({ email, site_id, first_name, last_name, company_id, company_name, company_domain, tags, source, utm_source, utm_medium, utm_campaign, utm_content, utm_term }) => {
         try {
+            const normalizedTags = normalizeTagInput(tags ?? []);
+            const warnings = tagInputWarnings(normalizedTags);
+            const tagWarnings = warnings.length ? { warnings } : {};
             email = email.trim().toLowerCase();
             const projectId = getProjectId();
             // Block distribution list / system addresses
@@ -1351,6 +1356,8 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
             let contact;
             if (!existingContact) {
                 const row = { project_id: projectId, email, first_name, last_name, company_id: resolvedCompanyId, source, source_site: site_id };
+                if (tags !== undefined)
+                    row.tags = normalizedTags.tags;
                 if (customProperties)
                     row.custom_properties = customProperties;
                 if (!needsConfirm)
@@ -1426,15 +1433,15 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
             if (needsConfirm) {
                 // Create opt-in token (confirmation email must be sent externally in MCP context)
                 await db().from('opt_in_tokens').insert({ contact_id: contact.id, site_id });
-                await writeBackMcpContact(projectId, contact, { email, first_name: contact.first_name ?? null, last_name: contact.last_name ?? null });
+                await writeBackMcpContact(projectId, contact, { email, first_name: contact.first_name ?? null, last_name: contact.last_name ?? null, ...(!existing && tags !== undefined ? { tags: normalizedTags.tags } : {}) });
                 logConsent({ contactId: contact.id, siteId: site_id, action: 'grant', consentType: 'email_marketing', source: 'mcp_pending_double_opt_in' });
-                return { content: [{ type: 'text', text: JSON.stringify({ ...contact, existing, confirmation: 'pending', message: 'Double opt-in required. Confirmation token created.', ...existingMessage }, null, 2) }] };
+                return { content: [{ type: 'text', text: JSON.stringify({ ...contact, ...tagWarnings, existing, confirmation: 'pending', message: 'Double opt-in required. Confirmation token created.', ...existingMessage }, null, 2) }] };
             }
-            await writeBackMcpContact(projectId, contact, { email, first_name: contact.first_name ?? null, last_name: contact.last_name ?? null });
+            await writeBackMcpContact(projectId, contact, { email, first_name: contact.first_name ?? null, last_name: contact.last_name ?? null, ...(!existing && tags !== undefined ? { tags: normalizedTags.tags } : {}) });
             logConsent({ contactId: contact.id, siteId: site_id, action: 'grant', consentType: 'email_marketing', source: 'mcp_add_subscriber' });
             if (!existing)
                 logAudit({ action: 'contact.created', resourceType: 'contact', resourceId: contact.id, details: { email, source } });
-            return { content: [{ type: 'text', text: JSON.stringify({ ...contact, existing, ...existingMessage }, null, 2) }] };
+            return { content: [{ type: 'text', text: JSON.stringify({ ...contact, ...tagWarnings, existing, ...existingMessage }, null, 2) }] };
         }
         catch (error) {
             return dbErrorResult(error);
@@ -1815,6 +1822,9 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
     }, async ({ contact_id, email, first_name, last_name, company_id, company_name, company_domain, tags, custom_properties, phone_e164, timezone }) => {
         try {
             const projectId = getProjectId();
+            const normalizedTags = tags !== undefined ? normalizeTagInput(tags) : undefined;
+            if (normalizedTags)
+                tags = normalizedTags.tags;
             let propertyWarnings = [];
             let fields;
             if (custom_properties !== undefined) {
@@ -2000,7 +2010,7 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
                 ...(updates.tags !== undefined ? { tags: updates.tags } : {}),
                 ...(updates.custom_properties !== undefined ? { custom_properties: updates.custom_properties } : {}),
             });
-            const mergedWarnings = [...(tagResult.warnings ?? []), ...propertyWarnings];
+            const mergedWarnings = [...(tagResult.warnings ?? []), ...propertyWarnings, ...(normalizedTags ? tagInputWarnings(normalizedTags) : [])];
             return j({ ...data, ...tagResult, ...(mergedWarnings.length ? { warnings: mergedWarnings } : {}) });
         }
         catch (error) {
@@ -2136,57 +2146,78 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
             idempotentHint: true,
             openWorldHint: false,
         },
-        description: 'Add or remove tags from one or more contacts',
+        description: 'Add or remove tags from one or more contacts. Tags are trimmed; empty values are ignored; case variants of an existing tag are not added again. Removal is case-insensitive. Ids not found are returned in not_found.',
         inputSchema: {
             contact_ids: z.array(z.string().uuid()).min(1),
             add_tags: z.array(z.string()).optional(),
             remove_tags: z.array(z.string()).optional(),
         },
     }, async ({ contact_ids, add_tags = [], remove_tags = [] }) => {
+        const normalized = normalizeTagInput(add_tags);
+        add_tags = normalized.tags;
+        remove_tags = remove_tags.map(tag => tag.trim()).filter(Boolean);
+        const warnings = new Set(tagInputWarnings(normalized));
         if (!add_tags.length && !remove_tags.length)
             return toolError({ error: 'No tags to add or remove.', code: 'validation_failed' });
         const projectId = getProjectId();
         let updated = 0;
         const missing_contact_ids = [];
+        const failed = [];
         for (const cid of contact_ids) {
-            const { data: contact, error: readError } = await db()
-                .from('subscribers')
-                .select('id, email, first_name, last_name, custom_properties, tags')
-                .eq('id', cid)
-                .eq('project_id', projectId)
-                .maybeSingle();
-            if (readError)
-                return toolError({ ...dbToolError(readError), updated });
-            if (!contact) {
-                missing_contact_ids.push(cid);
-                continue;
-            }
-            const currentTags = contact.tags ?? [];
-            const removeSet = new Set(remove_tags);
-            const merged = currentTags.filter((t) => !removeSet.has(t));
-            for (const t of add_tags) {
-                if (!merged.includes(t))
-                    merged.push(t);
+            try {
+                const { data: contact, error: readError } = await db()
+                    .from('subscribers')
+                    .select('id, email, first_name, last_name, custom_properties, tags')
+                    .eq('id', cid)
+                    .eq('project_id', projectId)
+                    .maybeSingle();
+                if (readError) {
+                    failed.push({ id: cid, ...dbToolError(readError) });
+                    continue;
+                }
+                if (!contact) {
+                    missing_contact_ids.push(cid);
+                    continue;
+                }
+                const currentTags = contact.tags ?? [];
+                const removeSet = new Set(remove_tags.map(tag => tag.toLowerCase()));
+                const merged = currentTags.filter((t) => !removeSet.has(t.trim().toLowerCase()));
+                for (const tag of add_tags) {
+                    const existing = merged.find(value => value.trim().toLowerCase() === tag.toLowerCase());
+                    if (existing !== undefined)
+                        warnings.add(`${tag} already exists as ${existing}`);
+                    else
+                        merged.push(tag);
+                }
+                const { data: saved, error } = await db()
+                    .from('subscribers')
+                    .update({ tags: merged })
+                    .eq('id', cid)
+                    .eq('project_id', projectId)
+                    .select('id').maybeSingle();
+                if (error) {
+                    failed.push({ id: cid, ...dbToolError(error) });
+                    continue;
+                }
+                if (!saved) {
+                    missing_contact_ids.push(cid);
+                    continue;
+                }
+                updated++;
+                await writeBackMcpContact(projectId, contact, { tags: merged });
+                await logAudit({ action: 'subscriber.tags_updated', resourceType: 'subscriber', resourceId: cid, details: { add_tags, remove_tags } });
             }
-            const { data: saved, error } = await db()
-                .from('subscribers')
-                .update({ tags: merged })
-                .eq('id', cid)
-                .eq('project_id', projectId)
-                .select('id').maybeSingle();
-            if (error)
-                return toolError({ ...dbToolError(error), updated });
-            if (!saved) {
-                missing_contact_ids.push(cid);
-                continue;
+            catch {
+                failed.push({ id: cid, ...dbToolError({}) });
             }
-            updated++;
-            await writeBackMcpContact(projectId, contact, { tags: merged });
-            await logAudit({ action: 'subscriber.tags_updated', resourceType: 'subscriber', resourceId: cid, details: { add_tags, remove_tags } });
         }
+        const result = { missing_contact_ids, not_found: missing_contact_ids, failed, updated, total: contact_ids.length, add_tags, remove_tags, ...(warnings.size ? { warnings: [...warnings] } : {}) };
+        // Preserve the existing coded-error convention when no requested write succeeded.
+        if (!updated && failed.length)
+            return toolError({ ...dbToolError({}), ...result, code: failed[0].code, error: failed[0].error });
         if (!updated)
-            return toolError({ error: 'Subscriber not found', code: 'not_found', updated, missing_contact_ids });
-        return j({ missing_contact_ids, updated, total: contact_ids.length, add_tags, remove_tags });
+            return toolError({ error: 'Subscriber not found', code: 'not_found', ...result });
+        return j(result);
     });
     server.registerTool('bulk_update_subscribers', {
         title: 'Bulk Update Subscribers',

package/package.json

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