@sendinel/mcp-server 1.0.57 → 1.0.59

package/build/tools/compound.js

@@ -1,3 +1,4 @@
+import { anthropicResponseText } from '@sendinel/domain';
 import { z } from 'zod';
 import Anthropic from '@anthropic-ai/sdk';
 import { adminDb, db } from '../db.js';
@@ -99,7 +100,7 @@ Do not include body_html. Sendinel renders body_html from blocks.
 Respond with a JSON array.`,
                     }],
             });
-            const raw = msg.content[0].text;
+            const raw = anthropicResponseText(msg);
             steps = extractJsonArray(raw).map((step, index) => {
                 const blocks = normalizeComposerBlocks(step.blocks, brandVisual.resolvedButtonColor);
                 return {
@@ -208,7 +209,7 @@ Do not include body_html. Sendinel renders body_html from blocks.
 Respond with JSON: { "subject": "...", "blocks": [...], "body_text": "..." }`,
                     }],
             });
-            const raw = msg.content[0].text;
+            const raw = anthropicResponseText(msg);
             const parsed = extractJsonObject(raw);
             const blocks = normalizeComposerBlocks(parsed.blocks, brandVisual.resolvedButtonColor);
             if (!blocks.length)
@@ -300,7 +301,7 @@ ${JSON.stringify(diagnosticData, null, 2)}
 Identify the most likely root cause and give 3-5 concise actionable next steps.`,
                 }],
         });
-        const analysis = analysisMsg.content[0].text;
+        const analysis = anthropicResponseText(analysisMsg);
         return j({
             diagnostic_data: diagnosticData,
             analysis,

package/build/tools/gateway.d.ts

@@ -14,4 +14,9 @@ export declare function matchesToolQuery(entry: {
     group: string;
     description: string;
 }, query: string): boolean;
+/**
+ * Keys in args that the target tool's JSON Schema does not declare. Returns [] when the schema
+ * has no `properties` or allows additional properties, so free-form tools keep working.
+ */
+export declare function findUnknownArgKeys(inputSchema: unknown, args: Record<string, unknown>): string[];
 export declare function registerGatewayTools(server: McpServer, options: GatewayOptions): void;

package/build/tools/gateway.js

@@ -54,6 +54,38 @@ async function buildCatalog(options) {
     });
     return { client, entries, byName: new Map(entries.map((entry) => [entry.name, entry])) };
 }
+/** Top-level keys invoke_tool accepts; anything else is rejected by name. */
+function unknownKeyMessage(keys) {
+    const list = keys.map((k) => `'${k}'`).join(', ');
+    const hinted = keys.some((k) => ARGS_LOOKALIKES.has(k.toLowerCase()));
+    return `Unknown key(s) on invoke_tool: ${list}${hinted ? "; did you mean 'args'?" : ''} Valid keys: name, args (alias: arguments).`;
+}
+const ARGS_LOOKALIKES = new Set(['args', 'argument', 'params', 'input', 'parameters']);
+function declaredProperties(inputSchema) {
+    if (!inputSchema || typeof inputSchema !== 'object')
+        return null;
+    const schema = inputSchema;
+    if (schema.additionalProperties)
+        return null;
+    const props = schema.properties;
+    if (!props || typeof props !== 'object' || Array.isArray(props))
+        return null;
+    return Object.keys(props);
+}
+/**
+ * Keys in args that the target tool's JSON Schema does not declare. Returns [] when the schema
+ * has no `properties` or allows additional properties, so free-form tools keep working.
+ */
+export function findUnknownArgKeys(inputSchema, args) {
+    const declared = declaredProperties(inputSchema);
+    if (!declared)
+        return [];
+    return Object.keys(args).filter((k) => !declared.includes(k));
+}
+function unknownArgsMessage(name, inputSchema, unknown) {
+    const valid = (declaredProperties(inputSchema) ?? []).sort();
+    return `Unknown argument(s) for ${name}: ${unknown.map((k) => `'${k}'`).join(', ')}. Valid arguments: ${valid.length ? valid.join(', ') : '(none)'}. Call describe_tool for the full schema.`;
+}
 export function registerGatewayTools(server, options) {
     let catalogPromise = null;
     const getCatalog = () => (catalogPromise ??= buildCatalog(options));
@@ -95,17 +127,33 @@ export function registerGatewayTools(server, options) {
     });
     server.registerTool('invoke_tool', {
         title: 'Invoke Tool',
-        description: 'Execute one permitted Sendinel MCP tool by name with the same scope and approval enforcement as calling it directly. Use search_tools and describe_tool first.',
+        description: 'Execute one permitted Sendinel MCP tool by name with the same scope and approval enforcement as calling it directly. Pass the target tool arguments under args (alias: arguments). Unknown top-level keys and arguments the target tool does not declare are rejected. Use search_tools and describe_tool first.',
         annotations: { readOnlyHint: false, destructiveHint: false, idempotentHint: false, openWorldHint: false },
-        inputSchema: {
+        // Loose object so unknown top-level keys reach the handler (a raw shape would silently strip them).
+        inputSchema: z.looseObject({
             name: z.string().min(1).describe('Tool name to invoke'),
-            args: z.record(z.string(), z.unknown()).optional().default({}).describe('Arguments for the target tool'),
-        },
-    }, async ({ name, args }) => {
+            args: z.record(z.string(), z.unknown()).optional().describe('Arguments for the target tool'),
+            arguments: z.record(z.string(), z.unknown()).optional().describe('Alias for args (the MCP-standard field name). Provide args or arguments, not both.'),
+        }),
+    }, async (input) => {
+        const { name, args, arguments: argumentsAlias, ...rest } = input;
+        const unknownKeys = Object.keys(rest);
+        if (unknownKeys.length > 0) {
+            return notFound(unknownKeyMessage(unknownKeys));
+        }
+        if (args !== undefined && argumentsAlias !== undefined && JSON.stringify(args) !== JSON.stringify(argumentsAlias)) {
+            return notFound("Both 'args' and 'arguments' were provided with different values. Provide either 'args' or 'arguments', not both.");
+        }
+        const effective = args ?? argumentsAlias ?? {};
         const catalog = await getCatalog();
-        if (!catalog.byName.has(name)) {
+        const entry = catalog.byName.get(name);
+        if (!entry) {
             return notFound(`Tool not found or not permitted: ${name}`);
         }
-        return await catalog.client.callTool({ name, arguments: args ?? {} });
+        const unknownArgs = findUnknownArgKeys(entry.inputSchema, effective);
+        if (unknownArgs.length > 0) {
+            return notFound(unknownArgsMessage(name, entry.inputSchema, unknownArgs));
+        }
+        return await catalog.client.callTool({ name, arguments: effective });
     });
 }

package/package.json

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