root/fusion
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(engine): remove leading newline from reviewer memorySection — buildPromptLayers handles joining

Browse Source
This commit is contained in:
Matthew Greenberg
2026-05-10 13:57:11 -04:00
parent 7a299ca543
commit ea3299f0b9
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
packages/engine/src/reviewer.ts
View File

@@ -405,8 +405,12 @@ export async function reviewStep(
} }
} }
const reviewerBasePrompt = resolveAgentPrompt("reviewer", options.agentPrompts) || REVIEWER_SYSTEM_PROMPT; const reviewerBasePrompt = resolveAgentPrompt("reviewer", options.agentPrompts) || REVIEWER_SYSTEM_PROMPT;
// Memory goes in the dynamic layer (not concatenated onto basePrompt) so the
// stable prefix is byte-identical across sessions even if memory changes.
// The leading "\n" separator is no longer needed — buildPromptLayers handles
// section joining with "\n\n".
const memorySection = options.rootDir && options.settings?.memoryEnabled !== false const memorySection = options.rootDir && options.settings?.memoryEnabled !== false
? "\n" + buildReviewerMemoryInstructions(options.rootDir, options.settings) ? buildReviewerMemoryInstructions(options.rootDir, options.settings)
: ""; : "";
// Build structured layers for cross-session prompt caching. // Build structured layers for cross-session prompt caching.