@paean-ai/adk 0.2.24 → 0.2.26

package/dist/cjs/agents/functions.js

@@ -129,6 +129,9 @@ function generateAuthEvent(invocationContext, functionResponseEvent) {
     longRunningToolIds.add(requestEucFunctionCall.id);
     parts.push({ functionCall: requestEucFunctionCall });
   }
+  if (parts.length === 0) {
+    return void 0;
+  }
   return (0, import_event.createEvent)({
     invocationId: invocationContext.invocationId,
     author: invocationContext.agent.name,
@@ -170,6 +173,9 @@ function generateRequestConfirmationEvent({
     longRunningToolIds.add(requestConfirmationFunctionCall.id);
     parts.push({ functionCall: requestConfirmationFunctionCall });
   }
+  if (parts.length === 0) {
+    return void 0;
+  }
   return (0, import_event.createEvent)({
     invocationId: invocationContext.invocationId,
     author: invocationContext.agent.name,
@@ -233,8 +239,9 @@ async function handleFunctionCallList({
       }
     );
     if (!toolAndContext) {
+      const argsPreview = functionCall.args ? JSON.stringify(functionCall.args).substring(0, 300) : "(none)";
       import_logger.logger.warn(
-        `Function "${functionCall.name}" not found in toolsDict (${Object.keys(toolsDict).length} tools registered).`
+        `Function "${functionCall.name}" not found in toolsDict (${Object.keys(toolsDict).length} tools registered). Args: ${argsPreview}`
       );
       const errorResponseEvent = (0, import_event.createEvent)({
         invocationId: invocationContext.invocationId,
@@ -283,6 +290,10 @@ async function handleFunctionCallList({
           toolContext
         );
       } catch (e) {
+        const argsPreview = JSON.stringify(functionArgs).substring(0, 500);
+        import_logger.logger.error(
+          `Tool execution error: "${tool.name}" threw ${e instanceof Error ? e.message : String(e)}. Args: ${argsPreview}`
+        );
         if (e instanceof Error) {
           const onToolErrorResponse = await invocationContext.pluginManager.runOnToolErrorCallback(
             {
@@ -400,6 +411,7 @@ function mergeParallelFunctionResponseEvents(functionResponseEvents) {
   const actionsList = functionResponseEvents.map((event) => event.actions || {});
   const mergedActions = (0, import_event_actions.mergeEventActions)(actionsList);
   return (0, import_event.createEvent)({
+    invocationId: baseEvent.invocationId,
     author: baseEvent.author,
     branch: baseEvent.branch,
     content: { role: "user", parts: mergedParts },

package/dist/cjs/agents/llm_agent.js

@@ -930,12 +930,12 @@ const _LlmAgent = class _LlmAgent extends import_base_agent.BaseAgent {
         consecutiveErrors++;
         if (consecutiveErrors <= _LlmAgent.MAX_AGENT_LOOP_ERROR_RETRIES) {
           import_logger.logger.warn(
-            `[runAsyncImpl] Error event (${lastEvent.errorCode}), retrying agent loop (${consecutiveErrors}/${_LlmAgent.MAX_AGENT_LOOP_ERROR_RETRIES})`
+            `[runAsyncImpl] Error event (${lastEvent.errorCode}: ${lastEvent.errorMessage || "no message"}), retrying agent loop (${consecutiveErrors}/${_LlmAgent.MAX_AGENT_LOOP_ERROR_RETRIES})`
           );
           continue;
         }
         import_logger.logger.error(
-          `[runAsyncImpl] Max agent-loop error retries exhausted for ${lastEvent.errorCode}`
+          `[runAsyncImpl] Max agent-loop error retries exhausted for ${lastEvent.errorCode}: ${lastEvent.errorMessage || "no message"}`
         );
         break;
       }
@@ -1024,7 +1024,7 @@ const _LlmAgent = class _LlmAgent extends import_base_agent.BaseAgent {
     }
   }
   async *postprocess(invocationContext, llmRequest, llmResponse, modelResponseEvent) {
-    var _a, _b;
+    var _a, _b, _c, _d, _e, _f, _g, _h, _i;
     for (const processor of this.responseProcessors) {
       for await (const event of processor.runAsync(invocationContext, llmResponse)) {
         yield event;
@@ -1033,6 +1033,28 @@ const _LlmAgent = class _LlmAgent extends import_base_agent.BaseAgent {
     if (!llmResponse.content && !llmResponse.errorCode && !llmResponse.interrupted) {
       return;
     }
+    if (llmResponse.content && !llmResponse.errorCode) {
+      if (!llmResponse.content.parts || llmResponse.content.parts.length === 0) {
+        import_logger.logger.debug(
+          `[postprocess] Skipping LLM response with no parts (role=${llmResponse.content.role})`
+        );
+        return;
+      }
+      const allEmpty = llmResponse.content.parts.every(
+        (p) => {
+          if (p.functionCall || p.functionResponse || p.executableCode || p.codeExecutionResult) return false;
+          if (p.inlineData || p.fileData) return false;
+          if ("text" in p && typeof p.text === "string" && p.text.length > 0) return false;
+          return true;
+        }
+      );
+      if (allEmpty) {
+        import_logger.logger.debug(
+          `[postprocess] Skipping empty-content LLM response (${llmResponse.content.parts.length} empty parts)`
+        );
+        return;
+      }
+    }
     const mergedEvent = (0, import_event.createEvent)({
       ...modelResponseEvent,
       ...llmResponse
@@ -1054,8 +1076,11 @@ const _LlmAgent = class _LlmAgent extends import_base_agent.BaseAgent {
         }
       }
     }
+    import_logger.logger.debug(
+      `[postprocess] Yielding mergedEvent: role=${(_b = mergedEvent.content) == null ? void 0 : _b.role}, parts=${(_d = (_c = mergedEvent.content) == null ? void 0 : _c.parts) == null ? void 0 : _d.length}, hasFCs=${((_e = (0, import_event.getFunctionCalls)(mergedEvent)) == null ? void 0 : _e.length) || 0}, partial=${mergedEvent.partial}`
+    );
     yield mergedEvent;
-    if (!((_b = (0, import_event.getFunctionCalls)(mergedEvent)) == null ? void 0 : _b.length)) {
+    if (!((_f = (0, import_event.getFunctionCalls)(mergedEvent)) == null ? void 0 : _f.length)) {
       return;
     }
     const functionResponseEvent = await (0, import_functions.handleFunctionCallsAsync)({
@@ -1080,6 +1105,9 @@ const _LlmAgent = class _LlmAgent extends import_base_agent.BaseAgent {
     if (toolConfirmationEvent) {
       yield toolConfirmationEvent;
     }
+    import_logger.logger.debug(
+      `[postprocess] Yielding functionResponseEvent: role=${(_g = functionResponseEvent.content) == null ? void 0 : _g.role}, parts=${(_i = (_h = functionResponseEvent.content) == null ? void 0 : _h.parts) == null ? void 0 : _i.length}`
+    );
     yield functionResponseEvent;
     const nextAgentName = functionResponseEvent.actions.transferToAgent;
     if (nextAgentName) {
@@ -1161,7 +1189,7 @@ const _LlmAgent = class _LlmAgent extends import_base_agent.BaseAgent {
           if (llmResponse.errorCode && _LlmAgent.LLM_RETRYABLE_ERROR_CODES.has(llmResponse.errorCode) && !contentYielded && attempt < maxRetries) {
             shouldRetry = true;
             import_logger.logger.warn(
-              `[callLlmAsync] Transient LLM error: ${llmResponse.errorCode}, usage: ${JSON.stringify(llmResponse.usageMetadata)}`
+              `[callLlmAsync] Transient LLM error: ${llmResponse.errorCode}${llmResponse.errorMessage ? ": " + llmResponse.errorMessage : ""}, finishReason: ${llmResponse.finishReason || "none"}, usage: ${JSON.stringify(llmResponse.usageMetadata)}`
             );
             break;
           }