@ductape/mcp 0.1.60 → 0.1.61

package/dist/index.js

@@ -946,7 +946,7 @@ const payloadGenerateInputSchema = z.object({
     execution_context: z.enum(['user', 'delegated', 'system']).optional().default('user').describe('Actor intent for this runtime operation. "user" means an active request initiated by the authenticated user; ' +
         '"delegated" means work acting on behalf of a user with a short-lived delegated identity or application-owned ' +
         'immutable actor context; "system" means intentionally unattributed background work. This drives session warnings.'),
-    include_cache: z.boolean().optional().default(true).describe('Include the cache tag inside the generated input object so the caller knows which cache to reference for this query.'),
+    include_cache: z.boolean().optional().describe('Include a cache tag inside the generated payload. Defaults to false; set true only when the runtime call should use cache explicitly.'),
     schema_mode: z.enum(['strict', 'best_effort']).optional().default('best_effort').describe('"strict" — fail if any required field cannot be resolved. ' +
         '"best_effort" — fill what is known, leave unknowns as null/placeholder. Use best_effort when exploring.'),
     input_hint: z.record(z.any()).optional().describe('Optional. Partial input values you already know. These are merged into the generated payload template ' +
@@ -1142,7 +1142,12 @@ const ADMIN_SUBCOMMANDS = [
 ];
 function checkCli() {
     try {
-        const out = execSync('ductape --version', { encoding: 'utf8', timeout: 5000, stdio: ['pipe', 'pipe', 'pipe'] }).trim();
+        const out = execSync('ductape --version', {
+            encoding: 'utf8',
+            timeout: 5000,
+            stdio: ['pipe', 'pipe', 'pipe'],
+            env: cliEnvironment(),
+        }).trim();
         return { available: true, version: out || 'unknown' };
     }
     catch {
@@ -1158,6 +1163,7 @@ function checkLoginState() {
             encoding: 'utf8',
             timeout: 10000,
             stdio: ['pipe', 'pipe', 'pipe'],
+            env: cliEnvironment(),
         });
         authState = 'ok';
         return 'ok';
@@ -1173,7 +1179,12 @@ function syncWorkspace() {
     if (!target)
         return;
     try {
-        execSync(`ductape workspaces use "${target}"`, { encoding: 'utf8', timeout: 10000, stdio: ['pipe', 'pipe', 'pipe'] });
+        execSync(`ductape workspaces use "${target}"`, {
+            encoding: 'utf8',
+            timeout: 10000,
+            stdio: ['pipe', 'pipe', 'pipe'],
+            env: cliEnvironment(),
+        });
     }
     catch {
         // best-effort; if it fails the user will see workspace-mismatch errors on subsequent commands
@@ -1200,6 +1211,7 @@ function runCli(command) {
             // instead of this wrapper killing the CLI first and reducing it to "(no data)".
             timeout: 90000,
             stdio: ['pipe', 'pipe', 'pipe'],
+            env: cliEnvironment(),
         });
         return { success: true, output: output.trim() };
     }
@@ -1243,6 +1255,16 @@ function runCli(command) {
         return { success: false, output: msg };
     }
 }
+/**
+ * MCP is publishable-key-only. The standalone CLI may manage its own access-key
+ * credentials, but an access key present in the MCP host environment is never
+ * forwarded into CLI subprocesses.
+ */
+function cliEnvironment() {
+    const environment = { ...process.env };
+    delete environment.DUCTAPE_ACCESS_KEY;
+    return environment;
+}
 const docsInputSchema = z.object({
     topic: z.string().describe('Feature topic to look up. Supported: ' +
         'transactions, presave, triggers, aggregations, migrations, indexes, performance, actions, ' +
@@ -3834,7 +3856,8 @@ async function main() {
             }
             const key = args.publishable_key || process.env.DUCTAPE_PUBLISHABLE_KEY;
             if (!key) {
-                throw new Error('Not authenticated. Set DUCTAPE_PUBLISHABLE_KEY in your MCP server env config, or pass publishable_key on every tool call.');
+                throw new Error('Runtime authentication is missing. Set DUCTAPE_PUBLISHABLE_KEY in the MCP server environment ' +
+                    'or pass publishable_key to ductape_execute. The MCP server never accepts access keys.');
             }
             // The TS SDK uses ductape.events.* for broker operations; the backend proxy uses messageBrokers.
             const proxyModule = args.module === 'events' ? 'messageBrokers' : args.module;
@@ -3857,7 +3880,18 @@ async function main() {
                     return p;
                 });
             }
-            const result = await executeViaProxy(key, proxyModule, args.method, params);
+            let result;
+            try {
+                result = await executeViaProxy(key, proxyModule, args.method, params);
+            }
+            catch (error) {
+                const message = error instanceof Error ? error.message : String(error);
+                if (/authentication failed|unauthorized|invalid.*key/i.test(message)) {
+                    throw new Error('The DUCTAPE_PUBLISHABLE_KEY was rejected by the runtime proxy. ' +
+                        'Use a publishable key for the same workspace/product. The MCP server never accepts or forwards access keys.');
+                }
+                throw error;
+            }
             return { content: [{ type: 'text', text: JSON.stringify(result ?? null, null, 2) }] };
         }
         catch (err) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ductape/mcp",
-  "version": "0.1.60",
+  "version": "0.1.61",
   "description": "MCP server that exposes Ductape SDK operations via the backend proxy",
   "type": "module",
   "main": "dist/index.js",

package/scripts/check-frontend-analytics-guidance.mjs

@@ -6,6 +6,17 @@ import { dirname, resolve } from 'node:path';
 const here = dirname(fileURLToPath(import.meta.url));
 const source = readFileSync(resolve(here, '../src/index.ts'), 'utf8');
 
+assert.doesNotMatch(
+  source,
+  /(?:const|let|var)\s+\w+\s*=\s*process\.env\.DUCTAPE_ACCESS_KEY/,
+  'MCP server must never read DUCTAPE_ACCESS_KEY',
+);
+assert.match(
+  source,
+  /delete environment\.DUCTAPE_ACCESS_KEY/,
+  'MCP CLI subprocess environment must strip DUCTAPE_ACCESS_KEY',
+);
+
 const checks = [
   ['dedicated docs topic', /'frontend-analytics': `/],
   ['anonymous analytics without an invented session', /Before login:[\s\S]*do not invent a session/i],

package/src/index.ts

@@ -973,8 +973,8 @@ const payloadGenerateInputSchema = z.object({
     '"delegated" means work acting on behalf of a user with a short-lived delegated identity or application-owned ' +
     'immutable actor context; "system" means intentionally unattributed background work. This drives session warnings.'
   ),
-  include_cache: z.boolean().optional().default(true).describe(
-    'Include the cache tag inside the generated input object so the caller knows which cache to reference for this query.'
+  include_cache: z.boolean().optional().describe(
+    'Include a cache tag inside the generated payload. Defaults to false; set true only when the runtime call should use cache explicitly.'
   ),
   schema_mode: z.enum(['strict', 'best_effort']).optional().default('best_effort').describe(
     '"strict" — fail if any required field cannot be resolved. ' +
@@ -1204,7 +1204,12 @@ const ADMIN_SUBCOMMANDS = [
 
 function checkCli(): { available: boolean; version?: string } {
   try {
-    const out = execSync('ductape --version', { encoding: 'utf8', timeout: 5000, stdio: ['pipe', 'pipe', 'pipe'] }).trim();
+    const out = execSync('ductape --version', {
+      encoding: 'utf8',
+      timeout: 5000,
+      stdio: ['pipe', 'pipe', 'pipe'],
+      env: cliEnvironment(),
+    }).trim();
     return { available: true, version: out || 'unknown' };
   } catch {
     return { available: false };
@@ -1220,6 +1225,7 @@ function checkLoginState(): 'ok' | 'none' {
       encoding: 'utf8',
       timeout: 10000,
       stdio: ['pipe', 'pipe', 'pipe'],
+      env: cliEnvironment(),
     });
     authState = 'ok';
     return 'ok';
@@ -1234,7 +1240,12 @@ function syncWorkspace(): void {
   workspaceSynced = true; // mark done regardless so we don't retry on every call
   if (!target) return;
   try {
-    execSync(`ductape workspaces use "${target}"`, { encoding: 'utf8', timeout: 10000, stdio: ['pipe', 'pipe', 'pipe'] });
+    execSync(`ductape workspaces use "${target}"`, {
+      encoding: 'utf8',
+      timeout: 10000,
+      stdio: ['pipe', 'pipe', 'pipe'],
+      env: cliEnvironment(),
+    });
   } catch {
     // best-effort; if it fails the user will see workspace-mismatch errors on subsequent commands
   }
@@ -1261,6 +1272,7 @@ function runCli(command: string): { success: boolean; output: string } {
       // instead of this wrapper killing the CLI first and reducing it to "(no data)".
       timeout: 90000,
       stdio: ['pipe', 'pipe', 'pipe'],
+      env: cliEnvironment(),
     });
     return { success: true, output: output.trim() };
   } catch (err: any) {
@@ -1304,6 +1316,17 @@ function runCli(command: string): { success: boolean; output: string } {
   }
 }
 
+/**
+ * MCP is publishable-key-only. The standalone CLI may manage its own access-key
+ * credentials, but an access key present in the MCP host environment is never
+ * forwarded into CLI subprocesses.
+ */
+function cliEnvironment(): NodeJS.ProcessEnv {
+  const environment = { ...process.env };
+  delete environment.DUCTAPE_ACCESS_KEY;
+  return environment;
+}
+
 const docsInputSchema = z.object({
   topic: z.string().describe(
     'Feature topic to look up. Supported: ' +
@@ -3949,7 +3972,10 @@ async function main() {
       }
       const key = args.publishable_key || process.env.DUCTAPE_PUBLISHABLE_KEY;
       if (!key) {
-        throw new Error('Not authenticated. Set DUCTAPE_PUBLISHABLE_KEY in your MCP server env config, or pass publishable_key on every tool call.');
+        throw new Error(
+          'Runtime authentication is missing. Set DUCTAPE_PUBLISHABLE_KEY in the MCP server environment ' +
+          'or pass publishable_key to ductape_execute. The MCP server never accepts access keys.',
+        );
       }
       // The TS SDK uses ductape.events.* for broker operations; the backend proxy uses messageBrokers.
       const proxyModule: SDKModule = args.module === 'events' ? 'messageBrokers' : args.module;
@@ -3974,7 +4000,19 @@ async function main() {
         });
       }
 
-      const result = await executeViaProxy(key, proxyModule, args.method, params);
+      let result: unknown;
+      try {
+        result = await executeViaProxy(key, proxyModule, args.method, params);
+      } catch (error) {
+        const message = error instanceof Error ? error.message : String(error);
+        if (/authentication failed|unauthorized|invalid.*key/i.test(message)) {
+          throw new Error(
+            'The DUCTAPE_PUBLISHABLE_KEY was rejected by the runtime proxy. ' +
+            'Use a publishable key for the same workspace/product. The MCP server never accepts or forwards access keys.',
+          );
+        }
+        throw error;
+      }
       return { content: [{ type: 'text', text: JSON.stringify(result ?? null, null, 2) }] };
     } catch (err) {
       let message = err instanceof Error ? err.message : String(err);