RichardandClaude Opus 4.8
eae13771cf
B1 — Die Compaction behielt blind die letzten 6 Nachrichten. Fiel diese Grenze mitten in eine Tool-Sequenz, entstand eine tool-Antwort ohne zugehoerigen assistant-tool_call; die API lehnt das mit HTTP 400 ab. FindSafeTailStart verschiebt die Grenze jetzt rueckwaerts auf eine Blockgrenze. B14 — Bei Konversationen mit hoechstens 6 Nachrichten enthielt der Tail auch die system-Nachricht, die anschliessend ein zweites Mal angehaengt wurde. Ergebnis war ein doppelter System-Prompt und eine duplizierte Konversation — die Compaction vergroesserte den Kontext, statt ihn zu verkleinern. Der Tail beginnt nun grundsaetzlich hinter dem System-Prompt; liegt davor nichts Nennenswertes, wird die Kompaktierung uebersprungen. Gefunden durch den Property-Test. Nebeneffekt: Zusammengefasst wird nur noch der Teil, der tatsaechlich wegfaellt. Der Tail bleibt woertlich erhalten und musste bisher doppelt bezahlt werden. B3 — maxTokens zaehlte kumulativ ueber alle Schritte, wurde aber wie eine Kontextgrenze konfiguriert. Da jeder Schritt den vollen Kontext erneut sendet, brach ein Chat mit 20k Kontext nach vier Schritten ab. Aufgeteilt in maxCumulativeTokens (Kostenbudget, Default 500k) und maxContextTokens (Kontextgroesse). Alte Konfigurationen werden beim Laden migriert, die Fehlermeldungen unterscheiden jetzt Schritt- und Kostenlimit. Alle 31 Tests gruen. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
69 lines
1.9 KiB
C#
69 lines
1.9 KiB
C#
using ClawdDotNet.Core.Config;
|
|
using Shouldly;
|
|
|
|
namespace ClawdDotNet.Core.Tests.Config;
|
|
|
|
/// <summary>
|
|
/// Alte Konfigurationen enthalten "maxTokens" — ein Wert, der als Kontextgrenze gemeint
|
|
/// war, aber kumulativ über alle Schritte zählte und Läufe vorzeitig abbrach (Bug B3).
|
|
/// Die Migration darf diesen Wert nicht unverändert als Kostenbudget übernehmen.
|
|
/// </summary>
|
|
public sealed class ConfigMigrationTests
|
|
{
|
|
private static InstanceConfig WithLegacyMaxTokens(int? legacy) => new()
|
|
{
|
|
OpenRouterApiKey = "sk-test",
|
|
Agents =
|
|
[
|
|
new AgentConfig
|
|
{
|
|
AgentId = "test",
|
|
LoopGuard = new LoopGuardConfig { LegacyMaxTokens = legacy }
|
|
}
|
|
]
|
|
};
|
|
|
|
[Fact]
|
|
public void Ein_altes_zu_knappes_maxTokens_wird_durch_den_neuen_Default_ersetzt()
|
|
{
|
|
var config = WithLegacyMaxTokens(80_000); // der alte, problematische Wert
|
|
|
|
ConfigLoader.Migrate(config);
|
|
|
|
var guard = config.Agents[0].LoopGuard;
|
|
guard.MaxCumulativeTokens.ShouldBe(500_000);
|
|
guard.LegacyMaxTokens.ShouldBeNull();
|
|
}
|
|
|
|
[Fact]
|
|
public void Ein_bewusst_grosszuegiges_maxTokens_bleibt_erhalten()
|
|
{
|
|
var config = WithLegacyMaxTokens(2_000_000);
|
|
|
|
ConfigLoader.Migrate(config);
|
|
|
|
config.Agents[0].LoopGuard.MaxCumulativeTokens.ShouldBe(2_000_000);
|
|
}
|
|
|
|
[Fact]
|
|
public void Ohne_altes_Feld_bleibt_der_Default_unveraendert()
|
|
{
|
|
var config = WithLegacyMaxTokens(null);
|
|
|
|
ConfigLoader.Migrate(config);
|
|
|
|
config.Agents[0].LoopGuard.MaxCumulativeTokens.ShouldBe(500_000);
|
|
}
|
|
|
|
[Fact]
|
|
public void Migration_ist_mehrfach_anwendbar()
|
|
{
|
|
var config = WithLegacyMaxTokens(80_000);
|
|
|
|
ConfigLoader.Migrate(config);
|
|
ConfigLoader.Migrate(config);
|
|
|
|
config.Agents[0].LoopGuard.MaxCumulativeTokens.ShouldBe(500_000);
|
|
}
|
|
}
|