@google/adk 2.4.0 → 2.5.0

package/dist/cjs/a2a/a2a_remote_agent.js

@@ -33,6 +33,7 @@ var import_sdk = require("@a2a-js/sdk");
 var import_client = require("@a2a-js/sdk/client");
 var import_base_agent = require("../agents/base_agent.js");
 var import_event = require("../events/event.js");
+var import_instrumentation = require("../telemetry/api/instrumentation.js");
 var import_env_aware_utils = require("../utils/env_aware_utils.js");
 var import_logger = require("../utils/logger.js");
 var import_a2a_event = require("./a2a_event.js");
@@ -46,6 +47,18 @@ var import_metadata_converter_utils = require("./metadata_converter_utils.js");
  * Copyright 2026 Google LLC
  * SPDX-License-Identifier: Apache-2.0
  */
+function createTracePropagatingClientFactory() {
+  const fetchImpl = (0, import_instrumentation.propagateTraceContext)();
+  return new import_client.ClientFactory(
+    import_client.ClientFactoryOptions.createFrom(import_client.ClientFactoryOptions.default, {
+      transports: [
+        new import_client.JsonRpcTransportFactory({ fetchImpl }),
+        new import_client.RestTransportFactory({ fetchImpl })
+      ]
+    })
+  );
+}
+__name(createTracePropagatingClientFactory, "createTracePropagatingClientFactory");
 const _RemoteA2AAgent = class _RemoteA2AAgent extends import_base_agent.BaseAgent {
   constructor(a2aConfig) {
     super(a2aConfig);
@@ -56,6 +69,7 @@ const _RemoteA2AAgent = class _RemoteA2AAgent extends import_base_agent.BaseAgen
     }
   }
   async init() {
+    var _a;
     if (this.isInitialized) {
       return;
     }
@@ -68,7 +82,7 @@ const _RemoteA2AAgent = class _RemoteA2AAgent extends import_base_agent.BaseAgen
         this.a2aConfig.resolveAgentCardOptions
       );
       if (!this.client) {
-        const factory = this.a2aConfig.clientFactory || new import_client.ClientFactory();
+        const factory = (_a = this.a2aConfig.clientFactory) != null ? _a : createTracePropagatingClientFactory();
         this.client = await factory.createFromAgentCard(this.card);
       }
     }
@@ -84,7 +98,8 @@ const _RemoteA2AAgent = class _RemoteA2AAgent extends import_base_agent.BaseAgen
       }
       const userFnCall = (0, import_a2a_remote_agent_utils.getUserFunctionCallAt)(
         context.session,
-        events.length - 1
+        events.length - 1,
+        this.name
       );
       let parts;
       let taskId = void 0;

package/dist/cjs/a2a/a2a_remote_agent_utils.js

@@ -46,7 +46,8 @@ var import_part_converter_utils = require("./part_converter_utils.js");
  * Copyright 2026 Google LLC
  * SPDX-License-Identifier: Apache-2.0
  */
-function getUserFunctionCallAt(session, index) {
+function getUserFunctionCallAt(session, index, agentName) {
+  var _a, _b;
   const events = session.events;
   if (index < 0 || index >= events.length) {
     return void 0;
@@ -55,13 +56,16 @@ function getUserFunctionCallAt(session, index) {
   if (candidate.author !== "user") {
     return void 0;
   }
-  const fnCallId = getFunctionResponseCallId(candidate);
-  if (!fnCallId) {
+  const fnCallIds = (_b = (_a = candidate.content) == null ? void 0 : _a.parts) == null ? void 0 : _b.map((part) => {
+    var _a2;
+    return (_a2 = part.functionResponse) == null ? void 0 : _a2.id;
+  }).filter((id) => !!id);
+  if (!(fnCallIds == null ? void 0 : fnCallIds.length)) {
     return void 0;
   }
   for (let i = index - 1; i >= 0; i--) {
     const request = events[i];
-    if (!isFunctionCallEvent(request, fnCallId)) {
+    if (request.author !== agentName || !fnCallIds.some((callId) => isFunctionCallEvent(request, callId))) {
       continue;
     }
     const metadata = request.customMetadata || {};

package/dist/cjs/a2a/agent_card.js

@@ -43,12 +43,11 @@ module.exports = __toCommonJS(agent_card_exports);
 var import_client = require("@a2a-js/sdk/client");
 var fs = __toESM(require("node:fs/promises"), 1);
 var import_node_url = require("node:url");
-var import_invocation_context = require("../agents/invocation_context.js");
 var import_llm_agent = require("../agents/llm_agent.js");
 var import_loop_agent = require("../agents/loop_agent.js");
 var import_parallel_agent = require("../agents/parallel_agent.js");
-var import_readonly_context = require("../agents/readonly_context.js");
 var import_sequential_agent = require("../agents/sequential_agent.js");
+var import_instrumentation = require("../telemetry/api/instrumentation.js");
 var import_base_tool = require("../tools/base_tool.js");
 var import_base_toolset = require("../tools/base_toolset.js");
 var import_logger = require("../utils/logger.js");
@@ -67,7 +66,7 @@ async function resolveAgentCard(agentCard, options = {}) {
   if (source.startsWith("http://") || source.startsWith("https://")) {
     await refuseLinkLocalHost(source);
     const resolver = new import_client.DefaultAgentCardResolver({
-      fetchImpl: refuseRedirectFetch
+      fetchImpl: tracePropagatingRefuseRedirectFetch
     });
     const card = await resolver.resolve(source);
     validateCardRpcTargets(card, source, options);
@@ -107,6 +106,7 @@ async function refuseRedirectFetch(url, init) {
   return response;
 }
 __name(refuseRedirectFetch, "refuseRedirectFetch");
+const tracePropagatingRefuseRedirectFetch = (0, import_instrumentation.propagateTraceContext)(refuseRedirectFetch);
 function parseCardUrl(source) {
   let url;
   try {
@@ -273,7 +273,7 @@ async function buildLLMAgentSkills(agent) {
     {
       id: agent.name,
       name: "model",
-      description: await buildDescriptionFromInstructions(agent),
+      description: buildAgentDescription(agent),
       tags: ["llm"]
     }
   ];
@@ -342,7 +342,7 @@ function buildAgentDescription(agent) {
   if (agent.description) {
     descriptionParts.push(agent.description);
   }
-  if (agent.subAgents.length > 0) {
+  if (!(0, import_llm_agent.isLlmAgent)(agent) && agent.subAgents.length > 0) {
     if ((0, import_loop_agent.isLoopAgent)(agent)) {
       descriptionParts.push(buildLoopAgentDescription(agent));
     } else if ((0, import_parallel_agent.isParallelAgent)(agent)) {
@@ -420,82 +420,6 @@ function buildLoopAgentDescription(agent) {
   return `${descriptions.join(" ")} in a loop (max ${maxIterations} iterations).`;
 }
 __name(buildLoopAgentDescription, "buildLoopAgentDescription");
-async function buildDescriptionFromInstructions(agent) {
-  const descriptionParts = [];
-  if (agent.description) {
-    descriptionParts.push(agent.description);
-  }
-  if (agent.instruction) {
-    let instructionStr;
-    if (typeof agent.instruction === "function") {
-      const dummyContext = new import_readonly_context.ReadonlyContext(
-        new import_invocation_context.InvocationContext({
-          agent
-        })
-      );
-      try {
-        instructionStr = await agent.instruction(dummyContext);
-      } catch (e) {
-        import_logger.logger.warn("Failed to resolve dynamic instruction for AgentCard", e);
-        instructionStr = "";
-      }
-    } else {
-      instructionStr = agent.instruction;
-    }
-    if (instructionStr) {
-      descriptionParts.push(replacePronouns(instructionStr));
-    }
-  }
-  const root = agent.rootAgent;
-  if ((0, import_llm_agent.isLlmAgent)(root) && root.globalInstruction) {
-    let globalInstructionStr;
-    if (typeof root.globalInstruction === "function") {
-      const dummyContext = new import_readonly_context.ReadonlyContext(
-        new import_invocation_context.InvocationContext({
-          agent
-        })
-      );
-      try {
-        globalInstructionStr = await root.globalInstruction(dummyContext);
-      } catch (e) {
-        import_logger.logger.warn(
-          "Failed to resolve dynamic global instruction for AgentCard",
-          e
-        );
-        globalInstructionStr = "";
-      }
-    } else {
-      globalInstructionStr = root.globalInstruction;
-    }
-    if (globalInstructionStr) {
-      descriptionParts.push(replacePronouns(globalInstructionStr));
-    }
-  }
-  if (descriptionParts.length > 0) {
-    return descriptionParts.join(" ");
-  } else {
-    return getDefaultAgentDescription(agent);
-  }
-}
-__name(buildDescriptionFromInstructions, "buildDescriptionFromInstructions");
-function replacePronouns(instruction) {
-  const substitutions = [
-    { original: "you were", target: "I was" },
-    { original: "you are", target: "I am" },
-    { original: "you're", target: "I am" },
-    { original: "you've", target: "I have" },
-    { original: "yours", target: "mine" },
-    { original: "your", target: "my" },
-    { original: "you", target: "I" }
-  ];
-  let result = instruction;
-  for (const sub of substitutions) {
-    const pattern = new RegExp(`\\b${sub.original}\\b`, "gi");
-    result = result.replace(pattern, sub.target);
-  }
-  return result;
-}
-__name(replacePronouns, "replacePronouns");
 function getDefaultAgentDescription(agent) {
   if ((0, import_loop_agent.isLoopAgent)(agent)) {
     return "A loop workflow agent";

package/dist/cjs/a2a/agent_to_a2a.js

@@ -44,6 +44,7 @@ var import_sdk = require("@a2a-js/sdk");
 var import_server = require("@a2a-js/sdk/server");
 var import_run_config = require("../agents/run_config.js");
 var import_in_memory_session_service = require("../sessions/in_memory_session_service.js");
+var import_instrumentation = require("../telemetry/api/instrumentation.js");
 var import_logger = require("../utils/logger.js");
 var import_optional_peer = require("../utils/optional_peer.js");
 var import_agent_card = require("./agent_card.js");
@@ -63,6 +64,10 @@ async function loadExpressBindings() {
   return { express: expressModule.default, a2aExpress };
 }
 __name(loadExpressBindings, "loadExpressBindings");
+function continueRemoteTrace(req, _res, next) {
+  (0, import_instrumentation.runWithRemoteTraceContext)(req.headers, () => next());
+}
+__name(continueRemoteTrace, "continueRemoteTrace");
 function resolveA2aUserBuilder(options, a2aExpress) {
   if (options.authentication) {
     return options.authentication;
@@ -124,6 +129,7 @@ async function toA2a(agent, options = {}) {
   );
   app.use(
     `${basePath}/rest`,
+    continueRemoteTrace,
     a2aExpress.restHandler({
       requestHandler,
       userBuilder
@@ -131,6 +137,7 @@ async function toA2a(agent, options = {}) {
   );
   app.use(
     `${basePath}/jsonrpc`,
+    continueRemoteTrace,
     a2aExpress.jsonRpcHandler({
       requestHandler,
       userBuilder

package/dist/cjs/a2a/event_converter_utils.js

@@ -79,7 +79,12 @@ function messageToAdkEvent(msg, invocationId, agentName, branch) {
   return {
     ...createAdkEventFromMetadata(msg),
     invocationId,
-    author: msg.role === import_a2a_event.MessageRole.USER ? import_a2a_event.MessageRole.USER : agentName,
+    // The role on a peer's message is the peer's own claim and must not decide
+    // authorship: letting `user` through here lets a remote agent post events
+    // that read as the local user's, which is a trust anchor elsewhere (tool
+    // confirmations are approved only from user-authored events). Attribute the
+    // event to the peer, the same way the task and status converters below do.
+    author: agentName,
     branch,
     content,
     turnComplete: true,
@@ -194,8 +199,10 @@ function createAdkEventFromMetadata(a2aEvent) {
     // (set it to a shared ancestor branch, or omit it) to leak its content
     // into an unrelated sibling agent's LLM context. Every caller of the
     // `*ToAdkEvent` functions in this file force-sets `branch` from its own
-    // local `InvocationContext` instead, the same way `author` is handled.
-    author: metadata[import_metadata_converter_utils.A2AMetadataKeys.AUTHOR],
+    // local `InvocationContext` instead, the same way `author` is handled:
+    // `author` is likewise never restored from peer metadata here, since every
+    // converter sets it from the local context after the spread, so reading it
+    // from metadata would be dead and read as if a peer could set the author.
     partial: metadata[import_metadata_converter_utils.A2AMetadataKeys.PARTIAL],
     errorCode: metadata[import_metadata_converter_utils.A2AMetadataKeys.ERROR_CODE],
     errorMessage: metadata[import_metadata_converter_utils.A2AMetadataKeys.ERROR_MESSAGE],