@agentforge-io/core 2.2.2 → 2.2.4

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/conversation.service.js

@@ -27,11 +27,23 @@ class ConversationService {
         });
     }
     async addMessage(params) {
+        // Coerce empty/whitespace-only content to a sentinel so we don't
+        // poison the history with rows that getAnthropicMessages would
+        // have to filter (and that Anthropic would 400 on replay).
+        // Assistant rows with structured metadata (approval bubbles,
+        // blocked-tool cards, etc.) keep their original empty content
+        // because `getAnthropicMessages` excludes them indirectly when
+        // the metadata kind doesn't map to natural prose.
+        const content = typeof params.content === 'string' && params.content.trim().length > 0
+            ? params.content
+            : params.role === 'assistant'
+                ? '(no response)'
+                : params.content;
         const msg = await this.msgRepo.create({
             conversationId: params.conversationId,
             userId: params.userId,
             role: params.role,
-            content: params.content,
+            content,
             toolCalls: params.toolCalls,
             usage: params.usage,
             metadata: params.metadata,
@@ -70,8 +82,18 @@ class ConversationService {
     async getAnthropicMessages(conversationId, userId) {
         const conv = await this.loadOwned(conversationId, userId);
         const messages = await this.msgRepo.listForConversation(conv.id, userId);
+        // Drop system rows AND any message whose content is empty /
+        // whitespace-only. Anthropic rejects empty user/assistant content
+        // with a 400 ("messages.N: ... must have non-empty content"),
+        // which used to crash the chat the moment the next turn replayed
+        // a malformed history row (e.g. a tool-only assistant turn
+        // persisted with empty `content`, or a streaming aborted before
+        // the user message text settled). Filtering is the cheapest fix —
+        // we don't lose semantic information because Anthropic's API
+        // wouldn't accept those rows anyway.
         return messages
             .filter((m) => m.role !== 'system')
+            .filter((m) => typeof m.content === 'string' && m.content.trim().length > 0)
             .map((m) => ({ role: m.role, content: m.content }));
     }
     async listForUser(userId, options = {}) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agentforge-io/core",
-  "version": "2.2.2",
+  "version": "2.2.4",
   "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",