@agentforge-io/core 2.2.3 → 2.2.4

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.3",
+  "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",