Files
ClawdDotNet/tests/ClawdDotNet.Core.Tests/Engine/ContextCompactorPropertyTests.cs
RichardandClaude Opus 5 cffd6f4b77 Bugtracker-Durchgang: 13 Bugs behoben, Build ohne Warnungen
Alle Behebungen auf der Linux-Test-VM gegengeprueft (Ubuntu, .NET 10.0.111):
Build 0 Warnungen / 0 Fehler, 637 Tests gruen, 1 uebersprungen (WindowsFact).

Sicherheit und Zielplattform
  #373 PathBoundary.Canonicalize fiel offen aus. File.ResolveLinkTarget wirft
       fuer einen fehlenden Abschnitt FileNotFoundException; der catch(IOException)
       verliess daraufhin die GANZE Aufloesung mit dem unaufgeloesten Pfad - und
       der besteht jede Praefixpruefung. Der Symlink-Schutz war damit bei jedem
       Schreibvorgang wirkungslos. Jetzt ueber FileSystemInfo.LinkTarget, und
       TryCanonicalize meldet "nicht bestimmbar", statt fail-open durchzureichen.
  #374 WorkspacePath weist den Backslash rundheraus ab, wie den Doppelpunkt.
       Eine Schreibweise statt einer Sonderregel je Windows-Form.
  #375 Log-Verzeichnis: leer heisst "Anwendungseinstellung"; ein relativer Wert
       zaehlt gegen das Instanzverzeichnis, nicht gegen das Arbeitsverzeichnis
       des Prozesses. Ein systemd-Dienst landet nicht mehr auf /Logs.

Agentenkommunikation
  #206 Delegationskette gegen A->B->A (Test A13). Zyklusfrei per Bauart, dazu
       MaxDelegationDepth gegen die Staffel, die niemand beauftragt hat.
  #461 Zwei unabhaengige Ketten ueber Kreuz - beim Beheben von #206 gefunden und
       reproduziert. Ein delegierter Lauf wartet nur noch begrenzt auf ein fremdes
       Gate; ein Chat vom Menschen reiht sich weiter unbegrenzt ein.
  #234 Instanz-Id wird durchgereicht statt aus einem nur beilaeufig gesetzten Feld
       gelesen.

Betrieb und Oberflaeche
  #376 Log-Ansicht chronologisch statt gruppiert je Moduldatei, mit Stacktraces
       an ihrer Meldung - auch ueber Taktgrenzen hinweg.
  #377 Info-Tabelle: feste Spaltenbreiten, damit ein langer Aenderungstext den
       anderen Spalten nicht den Platz nimmt.
  #378 Sicherung: Mindestabstand zwischen Ueberschrift und Schaltflaechenleiste.
  #379 Hellmodus: drei von vier gedaempften Textpinseln lagen unter 4,5:1. Die
       helle Palette ist als Ganzes verschoben, damit die Staffelung sichtbar
       bleibt statt auf einen Wert zusammenzufallen.

Hygiene
  #233 stock_add: der Indexstand wird innerhalb der Sperre festgehalten.
  #235 Zeitumstellung: eine nicht existierende Ortszeit wird auf den ersten
       Zeitpunkt nach der Luecke verschoben statt uebersprungen (R4, R5).
  #236 45 Nullable-Warnungen und eine CA1416 - nicht unterdrueckt, sondern die
       Stellen so umgebaut, dass die Zusage nachvollziehbar wird. Mail meldet
       jetzt, WELCHER Zugangsschluessel fehlt, statt an einer
       KeyNotFoundException zu scheitern.
  #203 Kompaktiert wird vor dem API-Aufruf statt danach; die gemessenen
       Prompt-Groessen kalibrieren die Schaetzung je Modell.

Neu: tests/ClawdDotNet.App.Tests - fuer #375 und #376 gab es keinen Ort fuer
Tests. Dazu Regressionstests fuer jede Behebung; jeder wurde gegen den alten
Stand geprueft, damit belegt ist, dass er den Fehler faengt.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-09-03 19:04:48 +02:00

151 lines
5.6 KiB
C#

using ClawdDotNet.Core.Api.Models;
using ClawdDotNet.Core.Config;
using ClawdDotNet.Core.Engine;
using ClawdDotNet.Core.Tests.Infrastructure;
using FsCheck;
using FsCheck.Xunit;
namespace ClawdDotNet.Core.Tests.Engine;
/// <summary>
/// Property-Based Tests: Statt einzelner Beispiele werden hunderte zufällige,
/// aber gültige Konversationen erzeugt und geprüft, dass die Compaction die
/// API-Invarianten in JEDEM Fall erhält.
///
/// Genau diese Grenzfälle — wo die Tail-Grenze zufällig mitten in einen
/// Tool-Zyklus fällt — schreibt von Hand niemand auf. Bug B1 fällt in diese Klasse.
/// </summary>
public sealed class ContextCompactorPropertyTests
{
private const string Model = "test/model";
private static readonly LoopGuardConfig AlwaysCompact = new()
{
MaxContextTokens = 1_000,
CompactionThreshold = 0.5
};
[Property(MaxTest = 300, Arbitrary = [typeof(ConversationArbitrary)])]
public Property Compaction_erhaelt_immer_eine_gueltige_Nachrichtenfolge(ConversationShape shape)
{
var messages = shape.ToMessages();
// Vorbedingung: Der Generator erzeugt nur gültige Ausgangsfolgen.
if (!ContextInvariants.IsValid(messages))
return false.ToProperty().Label("Generator hat eine ungültige Folge erzeugt");
var client = new FakeChatClient().AlwaysRespondsWithText("- Zusammenfassung des Verlaufs.");
var compactor = new ContextCompactor(client, TestLogging.Factory);
compactor.CompactIfNeededAsync(messages, AlwaysCompact, Model, default)
.GetAwaiter().GetResult();
var valid = ContextInvariants.IsValid(messages);
return valid.ToProperty()
.Label($"Nach Compaction ungültig. Ausgangsform: {shape}\n{ContextInvariants.Describe(messages)}");
}
[Property(MaxTest = 300, Arbitrary = [typeof(ConversationArbitrary)])]
public Property Compaction_behaelt_den_SystemPrompt_genau_einmal(ConversationShape shape)
{
var messages = shape.ToMessages();
var hadSystem = messages.Any(m => m.Role == "system");
var client = new FakeChatClient().AlwaysRespondsWithText("- Zusammenfassung.");
var compactor = new ContextCompactor(client, TestLogging.Factory);
compactor.CompactIfNeededAsync(messages, AlwaysCompact, Model, default)
.GetAwaiter().GetResult();
var systemCount = messages.Count(m => m.Role == "system");
var expected = hadSystem ? 1 : 0;
return (systemCount == expected).ToProperty()
.Label($"system-Nachrichten: erwartet {expected}, gefunden {systemCount}");
}
}
// ═══════════════════════════════════════════════════════════════
// Generator
// ═══════════════════════════════════════════════════════════════
/// <summary>
/// Beschreibt die Form einer Konversation. FsCheck erzeugt davon zufällige
/// Varianten und schrumpft sie im Fehlerfall auf das minimale Gegenbeispiel.
/// </summary>
public sealed record ConversationShape(bool HasSystemPrompt, IReadOnlyList<TurnShape> Turns)
{
public List<ChatMessage> ToMessages()
{
var c = Conversation.Start(HasSystemPrompt ? "Du bist ein Testagent." : null);
foreach (var turn in Turns)
{
c.User($"Anfrage mit {turn.ToolCalls} Tool-Aufrufen");
if (turn.ToolCalls > 0)
c.ToolCycle(turn.ToolCalls, turn.ResultLength);
if (turn.EndsWithText)
c.Assistant("Abschließende Antwort des Agenten.");
}
return c.Build();
}
public override string ToString()
=> $"system={HasSystemPrompt}, turns=[{string.Join(", ", Turns)}]";
}
public sealed record TurnShape(int ToolCalls, int ResultLength, bool EndsWithText)
{
public override string ToString() => $"{ToolCalls}tc/{ResultLength}b/{(EndsWithText ? "text" : "offen")}";
}
public static class ConversationArbitrary
{
public static Arbitrary<ConversationShape> Conversations()
{
var turnGen =
from toolCalls in Gen.Choose(0, 4)
from resultLength in Gen.Elements(20, 200, 2_500, 9_000)
from endsWithText in Gen.Elements(true, false)
select new TurnShape(toolCalls, resultLength, endsWithText);
var gen =
from hasSystem in Gen.Elements(true, false)
from turnCount in Gen.Choose(1, 12)
from turns in Gen.ListOf(turnCount, turnGen)
select new ConversationShape(hasSystem, turns.ToList());
return Arb.From(gen, Shrink);
}
/// <summary>Im Fehlerfall auf das kleinste Gegenbeispiel reduzieren.</summary>
private static IEnumerable<ConversationShape> Shrink(ConversationShape shape)
{
// Weniger Turns
for (var i = 0; i < shape.Turns.Count; i++)
{
var reduced = shape.Turns.Where((_, idx) => idx != i).ToList();
if (reduced.Count > 0)
yield return shape with { Turns = reduced };
}
// Weniger Tool-Calls pro Turn
for (var i = 0; i < shape.Turns.Count; i++)
{
var turn = shape.Turns[i];
if (turn.ToolCalls <= 0) continue;
var reduced = shape.Turns.ToList();
reduced[i] = turn with { ToolCalls = turn.ToolCalls - 1 };
yield return shape with { Turns = reduced };
}
if (shape.HasSystemPrompt)
yield return shape with { HasSystemPrompt = false };
}
}