smashimo/oh-my-pi
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(coding-agent): keep failed tails visible in read-only session history

Browse Source 
Route loadSessionMessagesReadOnly through transcript mode (collapsed to the
latest compaction) so history:// transcripts of on-disk sessions retain
failed/aborted assistant tails that the provider-context builder now drops.
This commit is contained in:
can1357
2026-07-23 11:37:10 +02:00
parent 6fe0cc99bc
commit e488d09751
1 changed files with 8 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/coding-agent/src/session/session-loader.ts
+8 -5
View File
@@ -306,10 +306,12 @@ export async function resolveBlobRefsInEntries(entries: FileEntry[], blobStore:
}
/**
* Read-only message view of a session file: load entries, migrate to the
* current version, resolve blob refs, and build the context along the
* persisted leaf path (last entry). Does NOT create a writer or take the
* session lock — safe to call against a file another session is writing.
* Read-only transcript view of a session file: load entries, migrate to the
* current version, resolve blob refs, and build the display transcript along
* the persisted leaf path (last entry). Uses transcript mode (collapsed to the
* latest compaction) so failed/aborted tail turns stay visible, unlike the
* provider-context builder which drops them. Does NOT create a writer or take
* the session lock — safe to call against a file another session is writing.
*/
export async function loadSessionMessagesReadOnly(filePath: string): Promise<AgentMessage[]> {
const entries = await loadEntriesFromFile(filePath);
@@ -317,5 +319,6 @@ export async function loadSessionMessagesReadOnly(filePath: string): Promise<Age
migrateToCurrentVersion(entries);
await resolveBlobRefsInEntries(entries, new BlobStore(getBlobsDir()));
const sessionEntries = entries.filter((e): e is SessionEntry => e.type !== "session");
return buildSessionContext(sessionEntries).messages;
return buildSessionContext(sessionEntries, undefined, undefined, { transcript: true, collapseCompactedHistory: true })
.messages;
}