@sendinel/mcp-server 1.0.147 → 1.0.148

package/build/tools/analytics.js

@@ -1,3 +1,4 @@
+import { isSentRow } from "@sendinel/domain";
 import { readPeriodCampaigns, campaignPeriodFields, filterPlanContacts } from "@sendinel/domain";
 import { isTrueConversionEvent, TRUE_CONVERSION_DEFINITION } from "@sendinel/domain";
 import { percentRate, groupMetricRows, readRevenue, isComplainedRow, isOpenedRow, countPeople, isClickedRow, summarizeEmailRows, countEmailRows, isDeliveredRow, isBouncedRow, fetchAllRows, readAllRows } from "@sendinel/domain";
@@ -42,7 +43,7 @@ export function computeSenderHealthScore(metrics, trackingEnabled) {
 function sendVolumeByDay(rows) {
     const byDay = new Map();
     for (const row of rows) {
-        if (!row.sent_at)
+        if (!row.sent_at || !isSentRow(row))
             continue;
         const date = new Date(row.sent_at).toISOString().slice(0, 10);
         byDay.set(date, (byDay.get(date) ?? 0) + 1);
@@ -133,12 +134,12 @@ function distinctEventCount(events, eventType, messageIds) {
         .map(event => event.provider_message_id)).size;
 }
 function buildTrueConversionAnalytics(params) {
-    const sent = params.logs.length;
+    const sent = countEmailRows(params.logs).sent;
     const delivered = params.logs.filter(isDeliveredRow).length;
     const trueRows = params.attributionRows.filter(row => isTrueConversionEvent(row.conversion_event));
     const campaignDenominators = new Map();
     for (const row of params.logs) {
-        if (typeof row.campaign_id !== 'string')
+        if (typeof row.campaign_id !== 'string' || !isSentRow(row))
             continue;
         campaignDenominators.set(row.campaign_id, (campaignDenominators.get(row.campaign_id) ?? 0) + 1);
     }
@@ -815,7 +816,7 @@ export function registerAnalyticsTools(server) {
             .eq('site_id', site_id)
             .gte('sent_at', sevenDaysAgo);
         const logs = recentLogs ?? [];
-        const totalSent = logs.length;
+        const totalSent = countEmailRows(logs).sent;
         const bounced = logs.filter((l) => isBouncedRow(l)).length;
         const complained = logs.filter(isComplainedRow).length;
         const bounceRate = percentRate(bounced, totalSent, 8) ?? 0;
@@ -1012,7 +1013,7 @@ export function registerAnalyticsTools(server) {
         if (firstHalf.length > 0 && secondHalf.length > 0) {
             trend.open_rate = `${(parseRate(secondMetrics.open_rate) - parseRate(firstMetrics.open_rate) >= 0 ? '+' : '')}${(parseRate(secondMetrics.open_rate) - parseRate(firstMetrics.open_rate)).toFixed(1)}pp`;
             trend.click_rate = `${(parseRate(secondMetrics.click_rate) - parseRate(firstMetrics.click_rate) >= 0 ? '+' : '')}${(parseRate(secondMetrics.click_rate) - parseRate(firstMetrics.click_rate)).toFixed(1)}pp`;
-            trend.volume = `${firstHalf.length} → ${secondHalf.length} emails`;
+            trend.volume = `${firstMetrics.total_sent} → ${secondMetrics.total_sent} emails`;
         }
         return j({
             period: `Last ${days} days`,
@@ -1077,7 +1078,7 @@ export function registerAnalyticsTools(server) {
                     .eq('project_id', proj.id)
                     .gte('sent_at', since);
                 const rows = logs ?? [];
-                const sent = rows.length;
+                const sent = countEmailRows(rows).sent;
                 const delivered = rows.filter(isDeliveredRow).length;
                 const opened = countPeople(rows, isOpenedRow);
                 const clicked = countPeople(rows, isClickedRow);

package/build/tools/campaigns.js

@@ -1,3 +1,4 @@
+import { readCampaignMetricGroups, SENT_STATUSES } from "@sendinel/domain";
 import { resolveCampaignReentry, readCampaignPeriods, campaignPeriodFields } from "@sendinel/domain";
 import { CAMPAIGN_BRANCH_CONDITIONS, campaignBranchConditionValue, isContactPropertyConditionList, STEP_PROPERTY_CONDITIONS_ERROR, validateStepPropertyRules } from "@sendinel/domain";
 import { z } from 'zod';
@@ -39,7 +40,6 @@ const reentryFields = {
     reentry_mode: z.enum(['once_ever', 'per_event_period']).optional().describe(REENTRY_DESCRIPTION),
     period_key: z.string().min(1).max(64).nullable().optional().describe('Flat event-data key naming the period (starts with a letter; letters, digits and underscores only). Key alone selects per_event_period; null clears to once ever. ' + REENTRY_DESCRIPTION),
 };
-const SENT_EMAIL_STATUSES = ['sent', 'delivered', 'opened', 'clicked', 'bounced', 'complained', 'unsubscribed'];
 function stripHtml(value) {
     return value
         .replace(/<style[\s\S]*?<\/style>/gi, ' ')
@@ -758,7 +758,7 @@ export function registerCampaignTools(server) {
                 fires_on: buildCampaignFiresOn(row),
                 trigger_diagnostics: buildCampaignTriggerDiagnostics(row),
                 ...campaignPeriodFields(periods, period),
-                ...(periods?.totals ? { total_sent: periods.totals.sent } : {}),
+                total_sent: periods?.totals?.sent ?? (engagement.ok ? engagement.sent.get(row.id) ?? 0 : null),
                 unique_opened: periods?.totals?.opened ?? (engagement.ok ? engagement.opened.get(row.id) ?? 0 : null),
                 unique_clicked: periods?.totals?.clicked ?? (engagement.ok ? engagement.clicked.get(row.id) ?? 0 : null),
             };
@@ -1157,7 +1157,7 @@ export function registerCampaignTools(server) {
                 .eq('project_id', projectId)
                 .eq('campaign_id', campaign_id)
                 .eq('contact_id', contact_id)
-                .not('status', 'eq', 'failed'),
+                .in('status', SENT_STATUSES),
             enrollment.current_step == null
                 ? Promise.resolve({ data: null, error: null })
                 : db()
@@ -3746,7 +3746,7 @@ Return ONLY valid JSON with this exact shape:
                 .select('id', { count: 'exact', head: true })
                 .eq('project_id', projectId)
                 .eq('campaign_id', campaign_id)
-                .in('status', SENT_EMAIL_STATUSES),
+                .in('status', SENT_STATUSES),
             db()
                 .from('email_log')
                 .select('id', { count: 'exact', head: true })
@@ -3775,6 +3775,9 @@ Return ONLY valid JSON with this exact shape:
         const visibilityError = stepsError ?? enrollError ?? sentError ?? queuedEmailError ?? failedSendError ?? queueError ?? dlqError;
         if (visibilityError)
             return j({ ...dbToolError(visibilityError), code: 'CAMPAIGN_VISIBILITY_QUERY_FAILED' });
+        const stepMetrics = await readCampaignMetricGroups(db(), projectId, (steps ?? []).map((step) => step.id), "step_id");
+        if (!stepMetrics.ok)
+            return j({ ...dbToolError(stepMetrics.error), code: 'CAMPAIGN_VISIBILITY_QUERY_FAILED' });
         const periods = await readCampaignPeriods(db(), projectId, campaign);
         const outstandingRows = queueRows ?? [];
         const pendingOrRetryRows = outstandingRows.filter((row) => row.status !== 'processing');
@@ -3795,7 +3798,7 @@ Return ONLY valid JSON with this exact shape:
             total_sent: periods?.totals?.sent ?? sentCount ?? 0,
             ...(periods?.totals ? { unique_opened: periods.totals.opened, unique_clicked: periods.totals.clicked } : {}),
             ...campaignPeriodFields(periods, period),
-            steps: steps ?? [],
+            steps: (steps ?? []).map((step) => ({ ...step, total_sent: stepMetrics.sent.get(step.id) ?? 0 })),
             active_enrollments: enrollCount ?? 0,
             send_incomplete: queuedCount > 0 || failedCount > 0 || outstandingRows.length > 0 || unresolvedCount > 0,
             queued_email_count: queuedCount,

package/package.json

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