@ductape/mcp 0.1.45 → 0.1.47

package/dist/index.js

@@ -522,7 +522,7 @@ ALL params are passed as a JSON array in positional order matching the SDK signa
   events.topics.list   [product_tag, broker_tag]  ← safe via ductape_execute
   events.produce  [{ product, env, event: "broker_tag:topic_tag", message: { key: value }, session?, cache? }]
   events.consume  [{ product, env, event: "broker_tag:topic_tag", callback: "function_ref" }]
-  events.dispatch [{ product, env, broker, event, input: { message }, retries?, session?, cache?, schedule?: { cron?, every?, start_at? } }]  — requires redisUrl in ductape initialization
+  events.dispatch [{ product, env, event: "broker_tag:topic_tag", input: { message }, retries?, session?, cache?, schedule?: { cron?, every?, start_at? } }]  — event is always the fully-qualified "broker:topic" string; do NOT pass broker separately — requires redisUrl in ductape initialization
   events.messages.query [{ product, env, brokerTag, topicTag?, producerTag?, consumerTag?, status?, startDate?, endDate?, page?, limit? }]
   events.messages.getProducers [{ product, env, brokerTag, topicTag?, page?, limit? }]
   events.messages.getConsumers [{ product, env, brokerTag, topicTag?, page?, limit? }]
@@ -2579,8 +2579,7 @@ Import (register an EXISTING cloud resource):
   SCHEDULED DISPATCH (background job):
     ductape_execute("events.dispatch", [{
       product, env,
-      broker: "order-events",                // broker tag
-      event: "order-events:reminder-due",    // "broker:topic"
+      event: "order-events:reminder-due",    // fully-qualified "broker-tag:topic-tag" — no separate broker field
       input: { message: { orderId: "123" } },
       retries?: 3,
       session?: "session-tag:jwt",
@@ -3396,7 +3395,26 @@ async function main() {
             }
             // The TS SDK uses ductape.events.* for broker operations; the backend proxy uses messageBrokers.
             const proxyModule = args.module === 'events' ? 'messageBrokers' : args.module;
-            const result = await executeViaProxy(key, proxyModule, args.method, args.params);
+            // Normalize events.dispatch params: event must be fully-qualified "broker:topic".
+            // If an agent passes both broker and event="broker:topic", strip broker and de-duplicate.
+            let params = args.params;
+            if ((args.module === 'events' || args.module === 'messageBrokers') && args.method === 'dispatch') {
+                params = params.map((p) => {
+                    if (p && typeof p === 'object' && !Array.isArray(p)) {
+                        const obj = p;
+                        const broker = typeof obj['broker'] === 'string' ? obj['broker'] : '';
+                        const event = typeof obj['event'] === 'string' ? obj['event'] : '';
+                        if (broker && event) {
+                            // Strip broker prefix if event is already "broker:..." to avoid "broker:broker:topic"
+                            const normalized = event.startsWith(broker + ':') ? event : `${broker}:${event}`;
+                            const { broker: _removed, ...rest } = obj;
+                            return { ...rest, event: normalized };
+                        }
+                    }
+                    return p;
+                });
+            }
+            const result = await executeViaProxy(key, proxyModule, args.method, params);
             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.45",
+  "version": "0.1.47",
   "description": "MCP server that exposes Ductape SDK operations via the backend proxy",
   "type": "module",
   "main": "dist/index.js",

package/src/index.ts

@@ -533,7 +533,7 @@ ALL params are passed as a JSON array in positional order matching the SDK signa
   events.topics.list   [product_tag, broker_tag]  ← safe via ductape_execute
   events.produce  [{ product, env, event: "broker_tag:topic_tag", message: { key: value }, session?, cache? }]
   events.consume  [{ product, env, event: "broker_tag:topic_tag", callback: "function_ref" }]
-  events.dispatch [{ product, env, broker, event, input: { message }, retries?, session?, cache?, schedule?: { cron?, every?, start_at? } }]  — requires redisUrl in ductape initialization
+  events.dispatch [{ product, env, event: "broker_tag:topic_tag", input: { message }, retries?, session?, cache?, schedule?: { cron?, every?, start_at? } }]  — event is always the fully-qualified "broker:topic" string; do NOT pass broker separately — requires redisUrl in ductape initialization
   events.messages.query [{ product, env, brokerTag, topicTag?, producerTag?, consumerTag?, status?, startDate?, endDate?, page?, limit? }]
   events.messages.getProducers [{ product, env, brokerTag, topicTag?, page?, limit? }]
   events.messages.getConsumers [{ product, env, brokerTag, topicTag?, page?, limit? }]
@@ -2652,8 +2652,7 @@ Import (register an EXISTING cloud resource):
   SCHEDULED DISPATCH (background job):
     ductape_execute("events.dispatch", [{
       product, env,
-      broker: "order-events",                // broker tag
-      event: "order-events:reminder-due",    // "broker:topic"
+      event: "order-events:reminder-due",    // fully-qualified "broker-tag:topic-tag" — no separate broker field
       input: { message: { orderId: "123" } },
       retries?: 3,
       session?: "session-tag:jwt",
@@ -3494,7 +3493,28 @@ async function main() {
       }
       // The TS SDK uses ductape.events.* for broker operations; the backend proxy uses messageBrokers.
       const proxyModule: SDKModule = args.module === 'events' ? 'messageBrokers' : args.module;
-      const result = await executeViaProxy(key, proxyModule, args.method, args.params);
+
+      // Normalize events.dispatch params: event must be fully-qualified "broker:topic".
+      // If an agent passes both broker and event="broker:topic", strip broker and de-duplicate.
+      let params = args.params;
+      if ((args.module === 'events' || args.module === 'messageBrokers') && args.method === 'dispatch') {
+        params = params.map((p) => {
+          if (p && typeof p === 'object' && !Array.isArray(p)) {
+            const obj = p as Record<string, unknown>;
+            const broker = typeof obj['broker'] === 'string' ? obj['broker'] : '';
+            const event = typeof obj['event'] === 'string' ? obj['event'] : '';
+            if (broker && event) {
+              // Strip broker prefix if event is already "broker:..." to avoid "broker:broker:topic"
+              const normalized = event.startsWith(broker + ':') ? event : `${broker}:${event}`;
+              const { broker: _removed, ...rest } = obj;
+              return { ...rest, event: normalized };
+            }
+          }
+          return p;
+        });
+      }
+
+      const result = await executeViaProxy(key, proxyModule, args.method, params);
       return { content: [{ type: 'text', text: JSON.stringify(result ?? null, null, 2) }] };
     } catch (err) {
       const message = err instanceof Error ? err.message : String(err);