@agentforge-io/core 2.2.1 → 2.2.3

package/dist/services/agent-runner.service.js

@@ -172,11 +172,24 @@ class AgentRunnerService {
                         toolResults.push({
                             type: 'tool_result',
                             tool_use_id: block.id,
-                            content: output,
+                            // Sentinel keeps Anthropic happy when a tool produced
+                            // no string output (e.g. a mutation that returned void).
+                            content: output || '(tool completed with no output)',
                             is_error: !!error,
                         });
                     }
                 }
+                // Same defensive break as the stream path — if tool_use was
+                // signalled but we resolved zero tool calls, don't append
+                // an empty user message.
+                if (toolResults.length === 0) {
+                    this.logger.warn(`Agent "${agent.id}" reported tool_use but emitted no resolvable tool calls. Closing the turn.`);
+                    finalContent = response.content
+                        .filter((b) => b.type === 'text')
+                        .map((b) => b.text)
+                        .join('');
+                    break;
+                }
                 currentMessages = [...currentMessages, { role: 'user', content: toolResults }];
             }
             else {
@@ -295,9 +308,27 @@ class AgentRunnerService {
                             output = `Error: ${err instanceof Error ? err.message : String(err)}`;
                         }
                         yield { type: 'tool_result', toolName: block.name, result: output };
-                        toolResults.push({ type: 'tool_result', tool_use_id: block.id, content: output });
+                        toolResults.push({
+                            type: 'tool_result',
+                            tool_use_id: block.id,
+                            // Anthropic rejects empty tool_result content (part of
+                            // the "messages.N: user messages must have non-empty
+                            // content" 400). When the tool returned no string,
+                            // substitute a sentinel so the next planning step still
+                            // sees a coherent transcript.
+                            content: output || '(tool completed with no output)',
+                        });
                     }
                 }
+                // Defensive: if the model said `tool_use` but emitted zero
+                // tool_use blocks (or all were filtered for unknown names),
+                // appending `{role:'user', content:[]}` triggers the same
+                // Anthropic 400. Break and let whatever text the model
+                // already produced stand as the final answer.
+                if (toolResults.length === 0) {
+                    this.logger.warn(`Agent "${agent.id}" reported tool_use but emitted no resolvable tool calls. Closing the turn.`);
+                    break;
+                }
                 currentMessages = [...currentMessages, { role: 'user', content: toolResults }];
             }
             else {

package/dist/services/orchestrator.service.js

@@ -310,6 +310,18 @@ class OrchestratorService {
                     content: assembled || '(member completed silently — no textual response)',
                 });
             }
+            // Defensive: when the model returned `tool_use` as stop_reason
+            // but every emitted block was a non-tool_use block (or pointed
+            // at a delegate_to_* whose subAgent is no longer in our list),
+            // `toolResults` ends up empty. Appending `{role:'user', content:
+            // []}` makes Anthropic reject the next turn with "messages.N:
+            // user messages must have non-empty content" (400). Break out
+            // gracefully and let whatever text the model already emitted
+            // serve as the final answer.
+            if (toolResults.length === 0) {
+                this.logger.warn(`Orchestrator "${agent.id}" reported tool_use but produced no resolvable delegation tool calls. Closing the turn.`);
+                break;
+            }
             // Feed the tool results back into the orchestrator's next loop.
             currentMessages = [
                 ...currentMessages,
@@ -386,9 +398,23 @@ class OrchestratorService {
                     toolResults.push({
                         type: 'tool_result',
                         tool_use_id: block.id,
-                        content: subResult.content,
+                        // Anthropic rejects empty tool_result content as part of
+                        // the "non-empty content" rule. Substitute a sentinel
+                        // when the sub-agent finished without text.
+                        content: subResult.content || '(member completed silently — no textual response)',
                     });
                 }
+                // Same defensive break as the stream path — if tool_use was
+                // signalled but we resolved zero delegation calls, don't
+                // append `{role:'user', content:[]}` (Anthropic 400).
+                if (toolResults.length === 0) {
+                    this.logger.warn(`Orchestrator "${orchestrator.id}" reported tool_use but produced no resolvable delegation tool calls. Closing the turn.`);
+                    finalContent = response.content
+                        .filter((b) => b.type === 'text')
+                        .map((b) => b.text)
+                        .join('');
+                    break;
+                }
                 currentMessages = [...currentMessages, { role: 'user', content: toolResults }];
             }
             else {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agentforge-io/core",
-  "version": "2.2.1",
+  "version": "2.2.3",
   "description": "Framework-free AI runtime SDK. Owns: agent loop (Anthropic), conversations, tools, streaming, agent-job queue, SdkHooks. Identity, billing, infra (email/uploads/secrets) live in the host's modules — not here.",
   "license": "MIT",
   "main": "dist/index.js",