fix(memory): keep pre-compaction flush prompt out of user transcript#73380
Conversation
Greptile SummaryThis PR fixes a transcript-leak bug where the pre-compaction memory-flush prompt was being saved as a normal user message in Confidence Score: 5/5Safe to merge — the one-line fix correctly uses the established runtimeOnly mechanism and has adequate layered test coverage. The change is minimal (one added field), the underlying resolveRuntimeContextPromptParts path for transcriptPrompt: '' is already unit-tested in runtime-context-prompt.test.ts, and no logic in attempt.ts is altered. No P0 or P1 issues found. No files require special attention. Reviews (1): Last reviewed commit: "fix(memory): keep pre-compaction flush p..." | Re-trigger Greptile |
a18411c to
25c4487
Compare
ee51412 to
f9697c6
Compare
|
Thanks for the Aisle pass. I checked this before landing. I’m leaving The run is still tagged on the internal path via |
f9697c6 to
9f69cb1
Compare
Summary
Context
Fixes #54408 and covers duplicate #58956. Related: #43567 for broader internal-system UI treatment and #12590/#51421 for the separate memoryFlush cadence bug.
Credit
Thanks @markgong for the canonical report and @guoyuhang9 for the v2026.3.31 reproduction details.
Validation
ProjectClownfish replacement details:
memoryFlushdoes not fire reliably #12590/fix(memory): memoryFlush fires every compaction cycle instead of every other #51421 cadence behavior is fixed by this PR unless the patch also intentionally changes that path.