@sendinel/mcp-server 1.0.172 → 1.0.173

package/build/tools/contacts.js

@@ -2,7 +2,7 @@ import { normalizeTagInput, tagInputWarnings } from '../lib/tags.js';
 import { ResourceTemplate } from '@modelcontextprotocol/sdk/server/mcp.js';
 import { CONTACT_ENRICHMENT_PROVIDERS } from '@desler/models';
 import { createContactEnrichmentAdapters, enrichContactFromConnections, } from '@desler/models/contact-enrichment';
-import { buildPreviewAttentionEnvelope, buildSafeIlikeOrFilter, literalIlikePattern, applySegmentRules, isSegmentRulesError, ContactPropertyAnomalyLogger, normalizeContactProperties, validateContactPropertyPatch, getAttentionDecision, getContactNeighborhood, getScheduledTouches, findContactDuplicateGroups, readAllPages, readAllRows, chunkIds, } from '@sendinel/domain';
+import { buildPreviewAttentionEnvelope, buildSafeIlikeOrFilter, literalIlikePattern, applySegmentRules, assertValidSegmentRules, isSegmentRulesError, ContactPropertyAnomalyLogger, normalizeContactProperties, validateContactPropertyPatch, getAttentionDecision, getContactNeighborhood, getScheduledTouches, findContactDuplicateGroups, readAllPages, readAllRows, chunkIds, } from '@sendinel/domain';
 import { z } from 'zod';
 import { callSendinelV1 } from '../lib/v1-client.js';
 import { db, projectClientForWriteback } from '../db.js';
@@ -2315,6 +2315,17 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
     }, async (params) => {
         const projectId = getProjectId();
         const sampleSize = params.sample_size ?? 5;
+        // Validate the same flat-rule shape as saved segments before any query.
+        // No filters deliberately means all contacts for this estimation tool.
+        const { sample_size: _sampleSize, ...rules } = params;
+        try {
+            assertValidSegmentRules(rules, { allowEmpty: true });
+        }
+        catch (error) {
+            if (isSegmentRulesError(error))
+                return toolError({ error: error.message, code: 'validation_failed' });
+            throw error;
+        }
         let q = db()
             .from('subscribers')
             .select('id, email, first_name, last_name, engagement_score, lifecycle_stage, source, created_at, last_opened_at, last_clicked_at')
@@ -2352,7 +2363,7 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
             }
             catch (error) {
                 if (isSegmentRulesError(error))
-                    return txt(`Error: ${dbToolError(error).error}`);
+                    return toolError({ error: error.message, code: 'validation_failed' });
                 throw error;
             }
         }
@@ -2365,11 +2376,13 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
             q = siteFiltered.query;
         }
         if (params.company_domain) {
-            const { data: companies } = await db()
+            const { data: companies, error } = await db()
                 .from('companies')
                 .select('id')
                 .eq('project_id', projectId)
                 .eq('domain', params.company_domain.trim().toLowerCase());
+            if (error)
+                return dbErrorResult(error);
             const ids = (companies ?? []).map((company) => company.id);
             if (!ids.length)
                 return j({ total: 0, sample: [] });
@@ -2377,7 +2390,7 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
         }
         const { data, error } = await q.order('engagement_score', { ascending: false }).limit(sampleSize);
         if (error)
-            return txt(`Error: ${dbToolError(error).error}`);
+            return dbErrorResult(error);
         // Get full count separately
         let countQ = db()
             .from('subscribers')
@@ -2412,7 +2425,9 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
         // The total must count the same site the sample came from (#3674).
         if (params.site_id)
             ({ query: countQ } = await applySegmentRules(countQ, { site_id: params.site_id }, db(), projectId, {}, { allowEmpty: true }));
-        const { count } = await countQ;
+        const { count, error: countError } = await countQ;
+        if (countError)
+            return dbErrorResult(countError);
         return j({ total: count ?? (data ?? []).length, sample: data ?? [] });
     });
     server.registerTool('identify_inactive', {

package/build/tools/segments.js

@@ -63,7 +63,9 @@ const j = (data) => txt(JSON.stringify(data, null, 2));
  * never an audience of everyone. Returns null for any other error.
  */
 function invalidRules(error) {
-    return isSegmentRulesError(error) ? txt(`Error: ${error.message}`) : null;
+    return isSegmentRulesError(error)
+        ? toolError({ error: error.message, code: 'validation_failed' })
+        : null;
 }
 export function registerSegmentTools(server) {
     server.registerTool('create_segment', {
@@ -89,7 +91,7 @@ export function registerSegmentTools(server) {
             catch (rulesError) {
                 const invalid = invalidRules(rulesError);
                 if (invalid)
-                    return toolError({ error: rulesError instanceof Error ? rulesError.message : 'Invalid segment rules', code: 'validation_failed' });
+                    return invalid;
                 throw rulesError;
             }
             const { data, error } = await db()
@@ -100,7 +102,7 @@ export function registerSegmentTools(server) {
             if (error?.code === '23505')
                 return toolError({ error: 'A segment with this name already exists', code: 'conflict' });
             if (error)
-                return toolError(dbToolError(error));
+                return dbErrorResult(error);
             // Count against the stored (normalized) tree, not the raw input: a tree skips
             // applySegmentRules' SQL push-down and filters in JS, which is what list_segments
             // does for this same segment a moment later. Pushing the flat rules down instead
@@ -171,7 +173,7 @@ export function registerSegmentTools(server) {
                 updates.name = name;
             if (description !== undefined)
                 updates.description = description;
-            if (rules) {
+            if (rules !== undefined) {
                 try {
                     updates.rules = prepareSegmentRulesForStorage(rules);
                 }
@@ -242,11 +244,13 @@ export function registerSegmentTools(server) {
                 .eq('id', segment_id)
                 .eq('project_id', getProjectId())
                 .maybeSingle();
-            if (error || !seg)
+            if (error)
+                return dbErrorResult(error);
+            if (!seg)
                 return txt('Error: segment not found');
             segRules = seg.rules;
         }
-        else if (rules) {
+        else if (rules !== undefined) {
             segRules = rules;
         }
         else {
@@ -265,10 +269,12 @@ export function registerSegmentTools(server) {
         const sample = ids.slice(0, 10);
         let sampleContacts = [];
         if (sample.length) {
-            const { data } = await db()
+            const { data, error } = await db()
                 .from('subscribers')
                 .select('id, email, first_name, last_name, engagement_score, lifecycle_stage, tags')
                 .in('id', sample);
+            if (error)
+                return dbErrorResult(error);
             sampleContacts = data ?? [];
         }
         return j({ total_contacts: ids.length, sample: sampleContacts });

package/package.json

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