@canonmsg/agent-tools 0.4.0 → 0.5.0

package/dist/verb-tools.d.ts

@@ -77,6 +77,8 @@ export interface VerbExecutionContext {
     turnId?: string;
     /** Default responder (e.g. the author of the turn being served). */
     responseUserId?: string;
+    /** Canon message id of the inbound message that triggered this turn. */
+    sourceMessageId?: string;
 }
 export interface ExecuteVerbToolOptions {
     /**

package/dist/verb-tools.js

@@ -94,6 +94,15 @@ export function canonVerbToolDefinitions(options = {}) {
                 delete properties[field];
             inputSchema = { ...inputSchema, properties };
         }
+        if (verb === 'no_reply') {
+            // Binding-owned like turnId: injected from turn context, never asked
+            // of the model — a guessed id is worse than none.
+            const properties = {
+                ...(inputSchema.properties ?? {}),
+            };
+            delete properties.messageId;
+            inputSchema = { ...inputSchema, properties };
+        }
         // The contract marks conversationId optional because bindings default it
         // to the active conversation — a projection without one must require it
         // from the model instead. Context-bound fields are either injected by
@@ -172,15 +181,23 @@ const DEFAULT_INTERACTION_TIMEOUT_MS = 5 * 60 * 1000;
  */
 export function normalizeVerbToolArgs(verb, args, context, now = Date.now()) {
     if (verb === 'no_reply') {
-        // Silence carries a conversation for the server's audit line only, so it
-        // defaults like an interaction verb but never FAILS without one — a turn
-        // can decline to reply from a binding that has no active conversation.
-        if (typeof args.conversationId === 'string' && args.conversationId)
-            return { args };
-        if (typeof context?.conversationId === 'string' && context.conversationId) {
-            return { args: { ...args, conversationId: context.conversationId } };
+        const completed = { ...args };
+        // messageId is binding-owned (like turnId on interaction verbs): the
+        // model never knows the triggering id more reliably than the binding,
+        // and a wrong id misattributes the silence record.
+        delete completed.messageId;
+        if (typeof context?.sourceMessageId === 'string' && context.sourceMessageId) {
+            completed.messageId = context.sourceMessageId;
         }
-        return { args };
+        // Silence carries a conversation for the server's durable silence marker
+        // (/runtime-silence/{convoId}/{agentId}), so it defaults like an
+        // interaction verb but never FAILS without one — a turn can decline to
+        // reply from a binding that has no active conversation.
+        if ((typeof completed.conversationId !== 'string' || !completed.conversationId)
+            && typeof context?.conversationId === 'string' && context.conversationId) {
+            completed.conversationId = context.conversationId;
+        }
+        return { args: completed };
     }
     if (!INTERACTION_VERBS.has(verb))
         return { args };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@canonmsg/agent-tools",
-  "version": "0.4.0",
+  "version": "0.5.0",
   "description": "Canonical Canon verb tools — shared projections of canon.verbs.v1 for runtime bindings",
   "type": "module",
   "main": "dist/index.js",
@@ -40,7 +40,7 @@
     "access": "public"
   },
   "dependencies": {
-    "@canonmsg/core": "^10.0.0",
+    "@canonmsg/core": "^10.2.0",
     "@canonmsg/rich-cards": "^0.9.1",
     "@modelcontextprotocol/sdk": "^1.29.0"
   },