#643: Entscheidungen wecken den Agenten nur noch bei Ablehnung oder gescheiterter Ausfuehrung, gesammelt in einer Aufgabe je Agent (5 min Ruhefenster), frisch per @new:agent statt im fortgefuehrten Chatverlauf. Eine einmalige Aufgabe, deren Lauf wegen Budget oder Herunterfahren gar nicht stattfand, bleibt faellig (Wiedervorlage nach 30 min) statt mit gesetztem Marker liegenzubleiben. #641: Rechte stehen in der Werkzeugkonfiguration des Agenten (erlaubt/verboten). Erlaubtes laeuft, Nicht-Erlaubtes wird einzeln vorgelegt statt verweigert, Verbotenes gar nicht erst. "Gleichartiges kuenftig erlauben" erteilt eine widerrufbare Dauerfreigabe je Agent/Werkzeug/Operation; nach drei gleichartigen Freigaben schlaegt die Oberflaeche sie vor. Aenderungen an Agenten bleiben ausnahmslos freigabepflichtig. Database: Operation ist das SQL-Schluesselwort; die eigene Zugriffspruefung greift nur noch ohne Freigabestelle (AgentToolContext.FreigabeGeprueft). WebUI: Rechte-Tabelle je Werkzeug, Dauerfreigaben mit Widerruf. Version 0.1.11. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
100 lines
4.6 KiB
C#
100 lines
4.6 KiB
C#
using System.Text.Json;
|
|
using ClawdDotNet.Core.Staging;
|
|
using ClawdDotNet.Core.Tools;
|
|
using ClawdDotNet.Tools.Database;
|
|
using ClawdDotNet.Tools.Tests.FileRW;
|
|
using Microsoft.Extensions.Logging.Abstractions;
|
|
using Shouldly;
|
|
|
|
namespace ClawdDotNet.Tools.Tests.Database;
|
|
|
|
/// <summary>
|
|
/// Vorpruefung (#579), Freigabeschluessel (#609) und Rechte (#641) des Database-Werkzeugs -
|
|
/// alles ohne Datenbankverbindung pruefbar, weil es nur Konfiguration und Statement ansieht.
|
|
/// </summary>
|
|
public sealed class DatabaseVorpruefungTests
|
|
{
|
|
private static readonly DatabaseTool Tool = new();
|
|
|
|
private static AgentToolContext Kontext(string accessLevel, params string[] tabellen) => new(
|
|
"agent", "inst",
|
|
new Dictionary<string, object?>
|
|
{
|
|
["type"] = "mysql",
|
|
["connectionString"] = "Server=nirgends",
|
|
["accessLevel"] = accessLevel,
|
|
["allowedTables"] = JsonSerializer.SerializeToElement(tabellen)
|
|
},
|
|
new NullStateStore(), NullLogger.Instance, CancellationToken.None);
|
|
|
|
private static JsonElement Sql(string sql, string action = "query")
|
|
=> JsonSerializer.SerializeToElement(new { action, sql });
|
|
|
|
[Fact]
|
|
public void Fremde_Tabelle_wird_vorab_abgelehnt()
|
|
=> Tool.Preflight(Sql("SELECT * FROM protokoll"), Kontext("ReadOnly", "messwerte"))
|
|
.ShouldNotBeNull().ShouldContain("protokoll");
|
|
|
|
[Fact]
|
|
public void Kommentare_werden_vorab_abgelehnt()
|
|
=> Tool.Preflight(Sql("DELETE FROM messwerte -- messwerte"), Kontext("Admin", "messwerte"))
|
|
.ShouldNotBeNull();
|
|
|
|
[Theory]
|
|
[InlineData("SELECT * FROM messwerte", "ReadOnly")]
|
|
[InlineData("INSERT INTO messwerte (ort) VALUES ('Berlin')", "ReadOnly")]
|
|
[InlineData("ALTER TABLE messwerte ADD COLUMN x INT", "ReadWrite")]
|
|
[InlineData("CREATE TABLE IF NOT EXISTS messwerte (id INT)", "Admin")]
|
|
[InlineData("DESCRIBE messwerte", "ReadOnly")]
|
|
public void Die_Vorpruefung_urteilt_nicht_mehr_ueber_die_Operation(string sql, string level)
|
|
{
|
|
// #641: Ob ein INSERT eines lesenden Agenten laeuft, vorgelegt oder verboten wird,
|
|
// entscheidet die Freigabestelle - nicht mehr eine stille Ablehnung im Werkzeug.
|
|
Tool.Preflight(Sql(sql), Kontext(level, "messwerte")).ShouldBeNull();
|
|
}
|
|
|
|
[Theory]
|
|
[InlineData("SELECT * FROM messwerte", "query", "select")]
|
|
[InlineData("DESCRIBE messwerte", "query", "select")]
|
|
[InlineData("SHOW COLUMNS FROM messwerte", "query", "select")]
|
|
[InlineData("INSERT INTO messwerte (ort) VALUES ('Berlin')", "query", "insert")]
|
|
[InlineData("UPDATE messwerte SET ort = 'X'", "insert", "update")]
|
|
[InlineData("DELETE FROM messwerte WHERE id = 1", "query", "delete")]
|
|
[InlineData("CREATE TABLE messwerte (id INT)", "query", "create")]
|
|
[InlineData("ALTER TABLE messwerte ADD COLUMN x INT", "query", "alter")]
|
|
[InlineData("DROP TABLE IF EXISTS messwerte", "query", "drop")]
|
|
[InlineData("insert into messwerte (ort) values ('select')", "query", "insert")]
|
|
public void Der_Freigabeschluessel_ist_die_SQL_Operation_nicht_das_action_Argument(string sql, string action, string erwartet)
|
|
=> Tool.StagingAktion(Sql(sql, action)).ShouldBe(erwartet);
|
|
|
|
[Theory]
|
|
[InlineData("query", "select")]
|
|
[InlineData("insert", "insert")]
|
|
[InlineData("upsert", "update")]
|
|
public void MongoDB_Aktionen_werden_auf_dieselben_Operationen_abgebildet(string action, string erwartet)
|
|
=> Tool.StagingAktion(JsonSerializer.SerializeToElement(new { action, collection = "c" })).ShouldBe(erwartet);
|
|
|
|
[Fact]
|
|
public void Die_Operationen_sind_fuer_die_Oberflaeche_aufgezaehlt()
|
|
=> Tool.Operationen.ShouldBe(["select", "insert", "update", "delete", "replace", "create", "alter", "drop", "truncate"]);
|
|
|
|
[Theory]
|
|
[InlineData("select", StagingDecision.Auto)]
|
|
[InlineData("insert", StagingDecision.Approve)]
|
|
[InlineData("alter", StagingDecision.Approve)]
|
|
public void Ohne_erlaubt_liest_der_Agent_frei_und_legt_alles_andere_vor(string operation, StagingDecision erwartet)
|
|
=> Tool.Vorgabe(operation, new Dictionary<string, object?>()).ShouldBe(erwartet);
|
|
|
|
[Fact]
|
|
public async Task Ohne_Freigabestelle_bleibt_die_Zugriffsstufe_eine_harte_Grenze()
|
|
{
|
|
// Kein Gate, keine Pruefung: dann muss das Werkzeug selbst verweigern, bevor es
|
|
// ueberhaupt eine Verbindung aufbaut.
|
|
var ergebnis = await Tool.ExecuteAsync(
|
|
Sql("INSERT INTO messwerte (ort) VALUES ('Berlin')"), Kontext("ReadOnly", "messwerte"), default);
|
|
|
|
ergebnis.Success.ShouldBeFalse();
|
|
ergebnis.ErrorMessage.ShouldNotBeNull().ShouldContain("ReadOnly");
|
|
}
|
|
}
|