fix(agents): track normalized message target evidence

vincentkoc · vincentkoc · commit 928b5932a386 · 2026-06-15T15:10:20.000+08:00
diff --git a/src/agents/embedded-agent-subscribe.tools.extract.test.ts b/src/agents/embedded-agent-subscribe.tools.extract.test.ts
@@ -248,6 +248,32 @@ describe("extractMessagingToolSend", () => {
     expect(result?.to).toBe("telegram:123");
   });
 
+  it("accepts channelId alias when earlier target aliases are blank", () => {
+    const result = extractMessagingToolSend("message", {
+      action: "send",
+      channel: "telegram",
+      target: " ",
+      to: "",
+      channelId: "123",
+    });
+
+    expect(result?.tool).toBe("message");
+    expect(result?.provider).toBe("telegram");
+    expect(result?.to).toBe("telegram:123");
+  });
+
+  it("prefers canonical target over legacy target aliases", () => {
+    const result = extractMessagingToolSend("message", {
+      action: "send",
+      channel: "telegram",
+      target: "123",
+      to: "456",
+      channelId: "789",
+    });
+
+    expect(result?.to).toBe("telegram:123");
+  });
+
   it("recognizes attachment-style message tool sends", () => {
     const upload = extractMessagingToolSend("message", {
       action: "upload-file",
diff --git a/src/agents/embedded-agent-subscribe.tools.ts b/src/agents/embedded-agent-subscribe.tools.ts
@@ -633,11 +633,11 @@ export function extractToolErrorMessage(result: unknown): string | undefined {
 }
 
 function resolveMessageToolTarget(args: Record<string, unknown>): string | undefined {
-  const toRaw = readStringValue(args.to);
-  if (toRaw) {
-    return toRaw;
-  }
-  return readStringValue(args.target);
+  return (
+    normalizeOptionalString(args.target) ??
+    normalizeOptionalString(args.to) ??
+    normalizeOptionalString(args.channelId)
+  );
 }
 
 function resolveMessagingToolThreadEvidence(params: {
@@ -744,7 +744,7 @@ export function extractMessagingToolSend(
     const providerId = providerHint ? normalizeChannelId(providerHint) : null;
     const provider = providerId ?? normalizeOptionalLowercaseString(providerHint) ?? "message";
     const to = normalizeTargetForProvider(provider, toRaw);
-    const pluginExtractionArgs = readStringValue(args.to) ? args : { ...args, to: toRaw };
+    const pluginExtractionArgs = { ...args, to: toRaw };
     const pluginExtracted = providerId
       ? getChannelPlugin(providerId)?.actions?.extractToolSend?.({ args: pluginExtractionArgs })
       : null;
