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

fix: remove double safety margin in proactive compaction

Browse Source 
Safety margin was applied twice:
1. threshold = (context - reserve) / 1.2
2. estimated = (existing + prompt) * 1.2

This caused compaction to trigger too early (~62% instead of ~83%).
Now only applied once in threshold calculation.
This commit is contained in:
tao 2026-01-30 00:18:23 +08:00
parent 3bf63a89e9
commit f28dacc11b
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/agents/pi-embedded-runner/proactive-compaction.ts
View File

@ -110,8 +110,8 @@ export async function checkProactiveCompaction(params: {
// Add estimate for the new prompt (default ~500 tokens if not provided)
const promptEstimate = params.promptTokenEstimate ?? 500;
// Apply safety margin to account for estimation inaccuracy
const estimatedTotalTokens = Math.floor((existingTokens + promptEstimate) * SAFETY_MARGIN);
// Total estimated tokens (safety margin is already applied in threshold calculation)
const estimatedTotalTokens = existingTokens + promptEstimate;
const shouldCompact = estimatedTotalTokens >= threshold;