mirror of
https://github.com/moltbot/moltbot.git
synced 2026-04-26 16:06:16 +00:00
* fix(gateway): safely extract text from message content arrays in prompt builder
When HistoryEntry.body is a content array (e.g. [{type:"text",
text:"hello"}]) rather than a plain string, template literal
interpolation produces "[object Object]" instead of the actual message
text. This affects users whose session messages were stored with array
content format.
Add a safeBody helper that detects non-string body values and uses
extractTextFromChatContent to extract the text, preventing the
[object Object] serialization in both the current-message return path
and the history formatting path.
Fixes openclaw#24688
Co-authored-by: Cursor <cursoragent@cursor.com>
* fix: format gateway agent prompt helper (#24946)
---------
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Peter Steinberger <steipete@gmail.com>
57 lines
1.7 KiB
TypeScript
57 lines
1.7 KiB
TypeScript
import { buildHistoryContextFromEntries, type HistoryEntry } from "../auto-reply/reply/history.js";
|
|
import { extractTextFromChatContent } from "../shared/chat-content.js";
|
|
|
|
export type ConversationEntry = {
|
|
role: "user" | "assistant" | "tool";
|
|
entry: HistoryEntry;
|
|
};
|
|
|
|
/**
|
|
* Coerce body to string. Handles cases where body is a content array
|
|
* (e.g. [{type:"text", text:"hello"}]) that would serialize as
|
|
* [object Object] if used directly in a template literal.
|
|
*/
|
|
function safeBody(body: unknown): string {
|
|
if (typeof body === "string") {
|
|
return body;
|
|
}
|
|
return extractTextFromChatContent(body) ?? "";
|
|
}
|
|
|
|
export function buildAgentMessageFromConversationEntries(entries: ConversationEntry[]): string {
|
|
if (entries.length === 0) {
|
|
return "";
|
|
}
|
|
|
|
// Prefer the last user/tool entry as "current message" so the agent responds to
|
|
// the latest user input or tool output, not the assistant's previous message.
|
|
let currentIndex = -1;
|
|
for (let i = entries.length - 1; i >= 0; i -= 1) {
|
|
const role = entries[i]?.role;
|
|
if (role === "user" || role === "tool") {
|
|
currentIndex = i;
|
|
break;
|
|
}
|
|
}
|
|
if (currentIndex < 0) {
|
|
currentIndex = entries.length - 1;
|
|
}
|
|
|
|
const currentEntry = entries[currentIndex]?.entry;
|
|
if (!currentEntry) {
|
|
return "";
|
|
}
|
|
|
|
const historyEntries = entries.slice(0, currentIndex).map((e) => e.entry);
|
|
if (historyEntries.length === 0) {
|
|
return safeBody(currentEntry.body);
|
|
}
|
|
|
|
const formatEntry = (entry: HistoryEntry) => `${entry.sender}: ${safeBody(entry.body)}`;
|
|
return buildHistoryContextFromEntries({
|
|
entries: [...historyEntries, currentEntry],
|
|
currentMessage: formatEntry(currentEntry),
|
|
formatEntry,
|
|
});
|
|
}
|