@sendinel/mcp-server 1.0.146 → 1.0.148

package/build/tools/templates.js

@@ -6,6 +6,7 @@ import { db, requestContext } from '../db.js';
 import { getProjectId } from '../project.js';
 import { recordBrandAssetUsage } from '../lib/brand-asset-usage.js';
 import { deriveTemplateVariables } from '../lib/voice.js';
+import { dbToolError } from '../lib/db-error.js';
 const txt = (text) => ({ content: [{ type: 'text', text }] });
 const j = (data) => txt(JSON.stringify(data, null, 2));
 function stripHtml(value) {
@@ -91,7 +92,7 @@ export function registerTemplateTools(server) {
             query = query.ilike('name', `%${search}%`);
         const { data, error } = await query;
         if (error)
-            return j({ error: error.message });
+            return j(dbToolError(error));
         return j({ templates: data ?? [], count: data?.length ?? 0 });
     });
     server.registerTool('translate_template', {
@@ -117,7 +118,7 @@ export function registerTemplateTools(server) {
         const projectId = getProjectId();
         const { data: project, error: projectError } = await db().from('projects').select('org_id').eq('id', projectId).single();
         if (projectError || !project?.org_id)
-            return j({ error: projectError?.message ?? 'Project not found' });
+            return j({ error: (projectError ? dbToolError(projectError).error : undefined) ?? 'Project not found' });
         const source = String(html ?? text ?? '').trim();
         if (!source)
             return j({ error: 'html or text is required' });
@@ -159,7 +160,7 @@ export function registerTemplateTools(server) {
             extracted_json: translated,
         }).select('id').single();
         if (error)
-            return j({ error: error.message });
+            return j(dbToolError(error));
         let savedTemplateId = null;
         if (save) {
             const { data: saved, error: saveError } = await db().from('campaign_templates').insert({
@@ -179,7 +180,7 @@ export function registerTemplateTools(server) {
                 external_id: source_template_id ?? null,
             }).select('id').single();
             if (saveError)
-                return j({ error: saveError.message });
+                return j(dbToolError(saveError));
             savedTemplateId = saved?.id ?? null;
             if (savedTemplateId) {
                 await recordBrandAssetUsage({
@@ -215,7 +216,7 @@ export function registerTemplateTools(server) {
             .or(`project_id.is.null,project_id.eq.${projectId}`)
             .maybeSingle();
         if (error)
-            return j({ error: error.message });
+            return j(dbToolError(error));
         return j({ template: data });
     });
     server.registerTool('create_template', {
@@ -256,7 +257,7 @@ export function registerTemplateTools(server) {
             .select('id, name, category, description, subject_hint, brief_template, body_html, body_source, body_text, format, variables, tags, created_at')
             .single();
         if (error)
-            return j({ error: error.message });
+            return j(dbToolError(error));
         await recordBrandAssetUsage({
             client: db(),
             projectId,
@@ -311,7 +312,7 @@ export function registerTemplateTools(server) {
             .eq('id', id)
             .maybeSingle();
         if (existingErr || !existing)
-            return j({ error: existingErr?.message ?? 'Template not found', code: 'NOT_FOUND' });
+            return j({ error: (existingErr ? dbToolError(existingErr).error : undefined) ?? 'Template not found', code: 'NOT_FOUND' });
         if (!existing.project_id || existing.project_id !== projectId)
             return j({ error: 'Cannot edit system templates', code: 'FORBIDDEN' });
         const row = { updated_at: new Date().toISOString() };
@@ -342,7 +343,7 @@ export function registerTemplateTools(server) {
             .select('id, name, category, description, subject_hint, brief_template, body_html, body_source, body_text, format, variables, tags, updated_at')
             .single();
         if (error)
-            return j({ error: error.message });
+            return j(dbToolError(error));
         await recordBrandAssetUsage({
             client: db(),
             projectId,
@@ -386,7 +387,7 @@ export function registerTemplateTools(server) {
             .eq('id', id)
             .maybeSingle();
         if (existingErr || !existing)
-            return j({ error: existingErr?.message ?? 'Template not found', code: 'NOT_FOUND' });
+            return j({ error: (existingErr ? dbToolError(existingErr).error : undefined) ?? 'Template not found', code: 'NOT_FOUND' });
         if (!existing.project_id || existing.project_id !== projectId)
             return j({ error: 'Cannot delete system templates', code: 'FORBIDDEN' });
         const { error } = await db().from('campaign_templates')
@@ -394,7 +395,7 @@ export function registerTemplateTools(server) {
             .eq('id', id)
             .eq('project_id', projectId);
         if (error)
-            return j({ error: error.message });
+            return j(dbToolError(error));
         return j({ ok: true, deleted: id });
     }));
     server.registerTool('restore_template_version', {
@@ -417,7 +418,7 @@ export function registerTemplateTools(server) {
             .eq('id', id)
             .maybeSingle();
         if (existingErr || !existing)
-            return j({ error: existingErr?.message ?? 'Template not found', code: 'NOT_FOUND' });
+            return j({ error: (existingErr ? dbToolError(existingErr).error : undefined) ?? 'Template not found', code: 'NOT_FOUND' });
         if (!existing.project_id || existing.project_id !== projectId)
             return j({ error: 'Cannot restore a system template', code: 'FORBIDDEN' });
         const { data: version, error: versionErr } = await db().from('template_versions')
@@ -426,7 +427,7 @@ export function registerTemplateTools(server) {
             .eq('template_id', id)
             .maybeSingle();
         if (versionErr || !version)
-            return j({ error: versionErr?.message ?? 'Version not found', code: 'NOT_FOUND' });
+            return j({ error: (versionErr ? dbToolError(versionErr).error : undefined) ?? 'Version not found', code: 'NOT_FOUND' });
         const bodyRestored = versionHasBodySnapshot(version);
         const row = {
             updated_at: new Date().toISOString(),
@@ -455,7 +456,7 @@ export function registerTemplateTools(server) {
             .select('id, name, category, description, subject_hint, brief_template, body_html, body_source, body_text, format, variables, tags, updated_at')
             .single();
         if (error)
-            return j({ error: error.message });
+            return j(dbToolError(error));
         const nextVersion = await snapshotTemplateVersion(db(), {
             templateId: data.id,
             name: data.name,

package/build/tools/warmup.js

@@ -1,6 +1,7 @@
 import { z } from 'zod';
 import { db } from '../db.js';
 import { getProjectId } from '../project.js';
+import { dbToolError } from '../lib/db-error.js';
 const j = (data) => ({ content: [{ type: 'text', text: JSON.stringify(data, null, 2) }] });
 export function registerWarmupTools(server) {
     server.registerTool('get_warmup_status', {
@@ -22,7 +23,7 @@ export function registerWarmupTools(server) {
             .limit(1)
             .maybeSingle();
         if (error)
-            return j({ error: error.message });
+            return j(dbToolError(error));
         const day = data?.warmup_day ?? 0;
         return j({
             phase: day >= 42 ? 'complete' : day > 0 ? 'warming' : 'not_started',
@@ -47,7 +48,7 @@ export function registerWarmupTools(server) {
         const { error } = await db().from('warmup_schedules')
             .upsert({ project_id: getProjectId(), site_id, target_volume: daily_limit ?? null, status: phase ?? 'active', updated_at: new Date().toISOString() }, { onConflict: 'site_id' });
         if (error)
-            return j({ error: error.message });
+            return j(dbToolError(error));
         return j({ updated: true });
     });
     server.registerTool('pause_warmup', {
@@ -63,7 +64,7 @@ export function registerWarmupTools(server) {
     }, async ({ site_id, reason }) => {
         const { error } = await db().from('warmup_schedules').upsert({ project_id: getProjectId(), site_id, status: 'paused', pause_reason: reason ?? null, updated_at: new Date().toISOString() }, { onConflict: 'site_id' });
         if (error)
-            return j({ error: error.message });
+            return j(dbToolError(error));
         return j({ paused: true });
     });
     server.registerTool('resume_warmup', {
@@ -79,7 +80,7 @@ export function registerWarmupTools(server) {
     }, async ({ site_id }) => {
         const { error } = await db().from('warmup_schedules').upsert({ project_id: getProjectId(), site_id, status: 'active', updated_at: new Date().toISOString() }, { onConflict: 'site_id' });
         if (error)
-            return j({ error: error.message });
+            return j(dbToolError(error));
         return j({ resumed: true });
     });
 }

package/build/tools/webhooks.js

@@ -4,6 +4,7 @@ import { listSandboxWebhookEvents } from '@sendinel/domain';
 import { db, adminDb } from '../db.js';
 import { getProjectId } from '../project.js';
 import { validateWebhookUrl, generateWebhookSecret } from '../lib/webhook-events.js';
+import { dbToolError } from '../lib/db-error.js';
 const txt = (text) => ({ content: [{ type: 'text', text }] });
 const j = (data) => txt(JSON.stringify(data, null, 2));
 const VALID_EVENTS = [
@@ -72,7 +73,7 @@ export function registerWebhookTools(server) {
             .select('id, url, events, active, created_at')
             .single();
         if (error)
-            return txt(`Error creating subscription: ${error.message}`);
+            return txt(`Error creating subscription: ${dbToolError(error).error}`);
         return j({
             ...data,
             secret, // Only returned on creation — store it securely
@@ -102,7 +103,7 @@ export function registerWebhookTools(server) {
             q = q.eq('active', true);
         const { data, error } = await q;
         if (error)
-            return txt(`Error: ${error.message}`);
+            return txt(`Error: ${dbToolError(error).error}`);
         return j({ subscriptions: data ?? [], count: data?.length ?? 0 });
     });
     server.registerTool('update_webhook_subscription', {
@@ -145,7 +146,7 @@ export function registerWebhookTools(server) {
             .select('id, url, events, active, description, updated_at')
             .single();
         if (error)
-            return txt(`Error: ${error.message}`);
+            return txt(`Error: ${dbToolError(error).error}`);
         return j(data);
     }));
     server.registerTool('delete_webhook_subscription', {
@@ -168,7 +169,7 @@ export function registerWebhookTools(server) {
             .eq('id', subscription_id)
             .eq('project_id', getProjectId());
         if (error)
-            return txt(`Error: ${error.message}`);
+            return txt(`Error: ${dbToolError(error).error}`);
         return txt(`Subscription ${subscription_id} deleted.`);
     }));
 }

package/package.json

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

package/build/lib/db-errors.d.ts

@@ -1,12 +0,0 @@
-/** Caller-safe database errors. Never serialize database messages, details or hints. */
-export declare function mapDbError(error: unknown): {
-    error: string;
-    code: string;
-};
-export declare function dbErrorResult(error: unknown): {
-    isError: boolean;
-    content: {
-        type: 'text';
-        text: string;
-    }[];
-};

package/build/lib/db-errors.js

@@ -1,20 +0,0 @@
-/** Caller-safe database errors. Never serialize database messages, details or hints. */
-export function mapDbError(error) {
-    const code = error && typeof error === 'object' && 'code' in error ? String(error.code) : '';
-    if (code === '23505')
-        return { error: 'A record with these values already exists.', code: 'conflict' };
-    if (code === '23503' || code === 'PGRST116') {
-        return { error: 'The requested or referenced record was not found.', code: 'not_found' };
-    }
-    if (code.startsWith('23') || code.startsWith('22')) {
-        return { error: 'The supplied values are not valid. Check the input and try again.', code: 'validation_failed' };
-    }
-    return { error: 'The operation could not be completed. Please try again later.', code: 'internal_error' };
-}
-export function dbErrorResult(error) {
-    console.warn('[mcp] Database operation failed:', error);
-    return {
-        isError: true,
-        content: [{ type: 'text', text: JSON.stringify(mapDbError(error)) }],
-    };
-}