compaction: decider sur le VRAI decompte de tokens (ctx_used du client) au lieu de l'estimation

L'heuristique ~4 char/token sous-estimait le contexte (elle ne voit ni le prompt
systeme injecte ni le gabarit de chat), donc le seuil 75% n'etait jamais franchi.
Le client envoie desormais ctx_used (prompt_tokens_total deja affiche) ; repli sur
l'estimation quand inconnu (terminal).
This commit is contained in:
nathaninline committed 2026-07-21 16:18:50 +02:00
1 parent e6c0bbe3b7
commit 2daea32ade
4 files changed
+20 -5

No files matched your search

+13 -2
View File
@@ -96,11 +96,22 @@ func estimateTokens(msgs []Message) int {
// proactif. Renvoie l'historique (compacté ou inchangé) et un booléen indiquant
// s'il a changé. À appeler sur l'historique BRUT (avant InjectSkills) pour que
// le résultat puisse être renvoyé au client sans le préfixe système injecté.
func MaybeCompact(ctx context.Context, msgs []Message, caps Caps) ([]Message, bool) {
//
// knownTokens = taille RÉELLE du contexte au tour précédent (usage.prompt_tokens
// + tokens générés), telle que rapportée par llama.cpp et affichée par l'UI. On
// la préfère à estimateTokens() car cette dernière n'est qu'une heuristique et,
// surtout, ne « voit » pas le prompt système injecté (machine briefing) ni le
// gabarit de chat — donc elle sous-estime largement le vrai contexte. 0 = inconnu
// (clients sans compteur, ex. terminal) → repli sur l'estimation.
func MaybeCompact(ctx context.Context, msgs []Message, caps Caps, knownTokens int) ([]Message, bool) {
if !compactEnabled() {
return msgs, false
}
if estimateTokens(msgs) < int(float64(ctxWindow())*compactTriggerFrac) {
used := knownTokens
if used <= 0 {
used = estimateTokens(msgs)
}
if used < int(float64(ctxWindow())*compactTriggerFrac) {
return msgs, false
}
return compactMessages(ctx, msgs, caps)