Ayuriel/openclaw
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: use SessionManager for transcript writes in appendUserTranscriptMessage

Browse Source 
Replace raw fs.appendFileSync with SessionManager.open + appendMessage
to maintain the parentId chain (guardrail test was correctly catching this).

Co-authored-by: Cursor <cursoragent@cursor.com>
This commit is contained in:
kumarabhirup 2026-02-17 00:21:13 -08:00
parent 835a36e741
commit a5343992dc
No known key found for this signature in database
GPG Key ID: DB7CA2289CAB0167
1 changed files with 5 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/gateway/server-methods/chat.ts
View File

@ -269,7 +269,6 @@ function appendUserTranscriptMessage(params: {
}
const now = Date.now();
const messageId = randomUUID().slice(0, 8);
const content: Array<Record<string, unknown>> = [];
if (params.message.trim()) {
content.push({ type: "text", text: params.message });
@ -289,20 +288,15 @@ function appendUserTranscriptMessage(params: {
content: content.length > 0 ? content : [],
timestamp: now,
};
const transcriptEntry = {
type: "message",
id: messageId,
timestamp: new Date(now).toISOString(),
message: messageBody,
};
try {
fs.appendFileSync(transcriptPath, `${JSON.stringify(transcriptEntry)}\n`, "utf-8");
// IMPORTANT: Use SessionManager so the entry is attached to the current leaf via parentId.
// Raw jsonl appends break the parent chain and can hide compaction summaries from context.
const sessionManager = SessionManager.open(transcriptPath);
const messageId = sessionManager.appendMessage(messageBody);
return { ok: true, messageId, message: messageBody };
} catch (err) {
return { ok: false, error: err instanceof Error ? err.message : String(err) };
}
return { ok: true, messageId, message: transcriptEntry.message };
}
function collectSessionAbortPartials(params: {