@sendinel/mcp-server 1.0.22 → 1.0.24

package/build/tools/analytics.js

@@ -1,3 +1,4 @@
+import { fetchAllRows } from '@sendinel/domain';
 import { z } from 'zod';
 import { db, adminDb } from '../db.js';
 import { getProjectId } from '../project.js';
@@ -431,42 +432,54 @@ export function registerAnalyticsTools(server) {
         if (siteError)
             return { content: [{ type: 'text', text: `Error: ${siteError.message}` }] };
         const trackingEnabled = site?.tracking_enabled ?? false;
-        const { data, error } = await db()
+        const { data, error, pageCount: logPageCount } = await fetchAllRows(async (offset, limit) => db()
             .from('email_log')
             .select('id, provider_message_id, status, sent_at')
             .eq('project_id', projectId)
             .eq('site_id', site_id)
             .gte('sent_at', `${start_date}T00:00:00Z`)
-            .lte('sent_at', `${end}T23:59:59Z`);
+            .lte('sent_at', `${end}T23:59:59Z`)
+            .order('id', { ascending: true })
+            .range(offset, offset + limit - 1));
         if (error)
             return { content: [{ type: 'text', text: `Error: ${error.message}` }] };
         const rows = (data ?? []);
         let eventCounts = { opened: 0, clicked: 0 };
+        let eventRows = [];
+        let eventPageCount = 0;
         if (trackingEnabled) {
-            const { data: events, error: eventsError } = await db()
+            const { data: events, error: eventsError, pageCount } = await fetchAllRows(async (offset, limit) => db()
                 .from('email_events')
                 .select('provider_message_id, event_type')
                 .eq('project_id', projectId)
                 .eq('site_id', site_id)
                 .in('event_type', ['open', 'click'])
                 .gte('occurred_at', `${start_date}T00:00:00Z`)
-                .lte('occurred_at', `${end}T23:59:59Z`);
+                .lte('occurred_at', `${end}T23:59:59Z`)
+                .order('id', { ascending: true })
+                .range(offset, offset + limit - 1));
             if (eventsError)
                 return { content: [{ type: 'text', text: `Error: ${eventsError.message}` }] };
+            eventRows = events ?? [];
+            eventPageCount = pageCount;
             const messageIds = new Set(rows.map(row => row.provider_message_id).filter((id) => Boolean(id)));
             eventCounts = {
-                opened: distinctEventCount(events ?? [], 'open', messageIds),
-                clicked: distinctEventCount(events ?? [], 'click', messageIds),
+                opened: distinctEventCount(eventRows, 'open', messageIds),
+                clicked: distinctEventCount(eventRows, 'click', messageIds),
             };
         }
         const metrics = computeAnalyticsMetrics(rows, trackingEnabled ? eventCounts : undefined);
         const periodDays = daysBetween(start_date, end);
         const priorStart = dateOnly(addDays(new Date(`${start_date}T00:00:00Z`), -periodDays));
         const priorEnd = dateOnly(addDays(new Date(`${start_date}T00:00:00Z`), -1));
-        const { data: subscribers } = await db()
+        const { data: subscribers, error: subscribersError, pageCount: subscriberPageCount } = await fetchAllRows(async (offset, limit) => db()
             .from('subscribers')
             .select('created_at, last_engaged_at, last_opened_at, last_clicked_at')
-            .eq('project_id', projectId);
+            .eq('project_id', projectId)
+            .order('id', { ascending: true })
+            .range(offset, offset + limit - 1));
+        if (subscribersError)
+            return { content: [{ type: 'text', text: `Error: ${subscribersError.message}` }] };
         const subscriberRows = (subscribers ?? []);
         const currentNew = subscriberRows.filter(s => s.created_at && s.created_at >= `${start_date}T00:00:00Z` && s.created_at <= `${end}T23:59:59Z`).length;
         const priorNew = subscriberRows.filter(s => s.created_at && s.created_at >= `${priorStart}T00:00:00Z` && s.created_at <= `${priorEnd}T23:59:59Z`).length;
@@ -488,6 +501,15 @@ export function registerAnalyticsTools(server) {
             list_growth_rate: subscriberRows.length > 0 ? pctNumber(currentNew / subscriberRows.length) : 0,
             inactive_pct: subscriberRows.length > 0 ? pctNumber(inactive / subscriberRows.length) : 0,
             sender_health_score: computeSenderHealthScore(metrics, trackingEnabled),
+            coverage: {
+                log_rows_scanned: rows.length,
+                log_page_count: logPageCount,
+                event_rows_scanned: trackingEnabled ? eventRows.length : null,
+                event_page_count: trackingEnabled ? eventPageCount : null,
+                subscriber_rows_scanned: subscriberRows.length,
+                subscriber_page_count: subscriberPageCount,
+                complete: true,
+            },
         };
         if (trackingEnabled) {
             response.open_rate = metrics.sent > 0 ? percent(metrics.open_rate) : 'N/A';

package/build/tools/contacts.js

@@ -2283,54 +2283,38 @@ 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 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 ?? [];
+        async function loadSchemaKeys(siteScope) {
+            const { data, error } = await db().rpc('get_contact_schema_keys', {
+                p_project_id: projectId,
+                ...(siteScope ? { p_site_id: siteScope } : {}),
+            });
+            if (error)
+                return { error: `Error: ${error.message}` };
+            const result = (Array.isArray(data) ? data[0] : data);
+            if (typeof result?.has_membership !== 'boolean' || !Array.isArray(result.keys)) {
+                return { error: 'Error: get_contact_schema_keys returned an unreadable result' };
+            }
+            return result;
+        }
+        // Site membership is hybrid (#3489): subscribers.source_site OR a site_subscriptions
+        // row, counted once. A real site whose members have no custom_properties yet is still
+        // site-scoped; project scope is only for a site with no members at all.
+        let loaded = await loadSchemaKeys(site_id);
+        if ('error' in loaded)
+            return txt(loaded.error);
         let scope = 'site';
-        if (rows.length === 0) {
-            const projectSample = await sampleSubscriberProperties(false);
-            if (projectSample.error)
-                return txt(`Error: ${projectSample.error.message}`);
-            rows = projectSample.data ?? [];
+        if (!loaded.has_membership) {
+            loaded = await loadSchemaKeys(null);
+            if ('error' in loaded)
+                return txt(loaded.error);
             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 }]) => ({
+        const schema = loaded.keys.map(({ key, contacts_with_this_key, example_values }) => ({
             key,
             merge_tag: `{{custom.${key}}}`,
             merge_tag_with_fallback: `{{custom.${key}|fallback}}`,
-            example_values: Array.from(values),
-            contacts_with_this_key: count,
+            example_values,
+            contacts_with_this_key,
         }));
         return j({
             site_id,
@@ -2338,7 +2322,7 @@ Requirements: urgency without desperation, acknowledge the gap, one clear CTA, v
             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).`
+                ? `No contacts are linked to site ${site_id} via source_site or site_subscriptions — read 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).',
         });
     });

package/package.json

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