@sendinel/mcp-server 1.0.137 → 1.0.138

package/build/tools/contacts.js

@@ -1670,7 +1670,7 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
             idempotentHint: true,
             openWorldHint: false,
         },
-        description: 'Update a contact\'s email, name, phone, timezone, tags, or custom properties',
+        description: 'Update a contact\'s email, name, phone, timezone, tags, or custom properties. tags replaces the whole tag set; use set_subscriber_tags to add tags.',
         inputSchema: {
             contact_id: z.string().uuid(),
             email: z.string().email().optional(),
@@ -1756,7 +1756,7 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
             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).');
+                    return contactInputError('validation_failed', 'Invalid phone_e164: must be a valid E.164 phone number (SMS is US-only for now).');
                 updates.phone_e164 = normalized;
             }
         }
@@ -1768,19 +1768,49 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
                 // 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).');
+                    return contactInputError('validation_failed', '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.');
+                    return contactInputError('validation_failed', 'Invalid timezone: must be a valid IANA time zone identifier.');
                 }
                 updates.timezone = timezone;
             }
         }
         if (!Object.keys(updates).length)
             return txt('No fields to update.');
+        const tagResult = {};
+        if (tags !== undefined) {
+            let previousTags;
+            try {
+                const { data: existing, error: readError } = await db()
+                    .from('subscribers')
+                    .select('tags')
+                    .eq('id', contact_id)
+                    .eq('project_id', projectId)
+                    .maybeSingle();
+                if (!readError && Array.isArray(existing?.tags) && existing.tags.every((tag) => typeof tag === 'string')) {
+                    previousTags = existing.tags;
+                }
+            }
+            catch {
+                // The replacement can still succeed, but the previous set is unknown.
+            }
+            if (previousTags === undefined) {
+                tagResult.warnings = ['Tags were REPLACED, but previous tags could not be read; removed and added tags are unknown. Use set_subscriber_tags to add tags without replacing the whole set.'];
+            }
+            else {
+                const previousSet = new Set(previousTags);
+                const nextSet = new Set(tags);
+                tagResult.tags_removed = [...previousSet].filter((tag) => !nextSet.has(tag));
+                tagResult.tags_added = [...nextSet].filter((tag) => !previousSet.has(tag));
+                if (tagResult.tags_removed.length > 0) {
+                    tagResult.warnings = ['Tags were REPLACED. Use set_subscriber_tags to add tags without replacing the whole set.'];
+                }
+            }
+        }
         updates.updated_at = new Date().toISOString();
         const { data, error } = await db()
             .from('subscribers')
@@ -1790,7 +1820,7 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
             .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.');
+            return contactInputError('validation_failed', 'Another contact in this project already has this phone number.');
         }
         if (error)
             return txt(`Error: ${error.message}`);
@@ -1809,7 +1839,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 } : {}),
         });
-        return j(data);
+        return j({ ...data, ...tagResult });
     });
     server.registerTool('get_subscriber', {
         title: 'Get Subscriber',

package/package.json

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