Files
RichardandClaude Opus 5 373e1f4caf Deploymentcenter-Anbindung auf SDK 2.8.2 nachgezogen (#218)
Der Changelog seit 2.6.0 nennt sechs Fassungen, 16 action_items und fuenf
brechende Aenderungen. Davon trifft fast nichts diese Anwendung - und das
ist ein Befund, kein Versaeumnis:

- finding_status (brechend, 2.7.0): kein Aufrufer. Weder
  /api/supervisor/v1/observe noch /status werden hier angesprochen.
  Vorgemerkt fuer #528 Stufe 3 - wer den Manager baut, liest den
  Befundzustand von Anfang an aus diesem Feld.
- Nachpruefungs-Status awaiting_verification, in_verification,
  fix_insufficient (brechend, 2.6.1): Die Anwendung wertet
  bugtracker_items.status nirgends aus. Betrifft den Arbeitsablauf der
  Agenten, und der steht in CLAUDE.md bereits richtig beschrieben.
- Meldungen ohne project_slug (brechend, 2.8.1): BugtrackerClient und
  ErrorReporter schicken project_slug und build_version seit jeher mit.
- Leseweg ohne project_slug und schmales dc_roadmap (brechend, 2.8.1):
  betrifft den MCP-Adapter, nicht den Anwendungscode.

Geblieben ist die eine Zusage, die man nicht sieht, wenn man sie verletzt:
Seit 2.8.1 erbt ein Watchdog-Monitor das Projekt seines Ping-Tokens. Ohne
Bindung landen die Befunde im Scheinprojekt "default", und eine Abfrage
nach dem eigenen Projekt liefert fuer eine laufende, fleissig meldende
Instanz null Befunde - die Aufsicht sieht aus wie "nichts auffaellig". Das
ist genau die Falle, in die hier schon einmal Meldungen gelaufen sind.

Deshalb neu: TokenProvisioner.WhoAmIAsync gegen GET
/api/tokens/v1/whoami (SDK 2.7.1) und der Typ TokenIdentity mit
IsProjectBound. DeploymentcenterService fragt beim Start das eigene
Melde-Token, protokolliert das Projekt und warnt, wenn keines dransteht.
Bewusst nur eine Warnung: Monitoring, das bei unklarer Rechtelage gar nicht
erst anlaeuft, ist genau dann still, wenn man es braucht. Faellt die
Auskunft aus - aelterer Server, kein Netz -, bleibt es bei Debug.

Gemessen statt geraten: TokenProvisioner schickt keinen project_slug mit,
und ob der Server die Bindung vom Master-Token erbt, steht in keiner
Beschreibung, die vorliegt. Ein Sub-Token dafuer eigens auf dem
Produktivserver auszustellen waere zu viel Nebenwirkung gewesen; die erste
Instanz, die eines zieht, sagt es jetzt selbst ins Protokoll.

#176 ist erledigt und die Regel in CLAUDE.md gestrichen: dc_roadmap ohne
project_slug liefert jetzt das eigene Projekt (nachgemessen: 51 Eintraege,
alle clawddotnet). Wer projektuebergreifend lesen will, schickt
project_slug=all.

dc_link_project auf 2.8.2 gesetzt, CLAUDE.md-Zeile hochgesetzt.

Build ohne Warnungen, 717 Tests gruen, 8 uebersprungen.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-09-08 22:14:19 +02:00

131 lines
5.4 KiB
C#

using System.Text.Json;
namespace ClawdDotNet.Core.Deploymentcenter;
/// <summary>Ein für diese Instanz ausgestelltes Sub-Token.</summary>
/// <param name="Token">Der Klartext — wird nur einmal ausgeliefert.</param>
/// <param name="TokenId">Kennung zum Widerrufen in der Verwaltung.</param>
/// <param name="Scopes">Welche Rechte tatsächlich durchgereicht wurden.</param>
public sealed record ProvisionedToken(string Token, string TokenId, IReadOnlyList<string> Scopes);
/// <summary>
/// Was ein Token über sich selbst sagt (<c>GET /api/tokens/v1/whoami</c>, seit SDK 2.7.1).
///
/// <para><see cref="ProjectSlug"/> ist das Feld, auf das es ankommt. Seit SDK 2.8.1 erbt
/// ein Watchdog-Monitor das Projekt seines Ping-Tokens — ist dort nichts eingetragen,
/// landen die Befunde im Scheinprojekt „default", und eine Abfrage nach dem eigenen
/// Projekt liefert für eine laufende, meldende Instanz <b>null Befunde</b>. Die Aufsicht
/// sähe dann aus wie „nichts auffällig", und das ist der schlechteste aller
/// Irrtümer.</para>
/// </summary>
public sealed record TokenIdentity(
string TokenId,
string Name,
string Type,
string? ProjectSlug,
string Environment,
IReadOnlyList<string> Scopes)
{
/// <summary>Ob dieses Token an ein Projekt gebunden ist — siehe Klassenkommentar.</summary>
public bool IsProjectBound => !string.IsNullOrWhiteSpace(ProjectSlug);
}
/// <summary>
/// Der Zugang zu <c>/api/tokens/v1/…</c>: ein Sub-Token je Instanz ausstellen
/// (<c>provision</c>) und ein Token nach sich selbst fragen (<c>whoami</c>).
///
/// <para>Das ersetzt die frühere Selbstregistrierung über <c>POST /api/register</c> —
/// diesen Endpunkt gibt es im Deploymentcenter nicht (und im alten WatchDog-Server war
/// er der einzige Weg, überhaupt an einen Token zu kommen). Der Zweck bleibt derselbe
/// und ist es wert, erhalten zu bleiben: Auf den Instanzen liegt danach nicht das
/// Master-Token, sondern ein eingeschränktes, einzeln widerrufbares.</para>
///
/// <para>Rechte lassen sich dabei nur einschränken, nie erweitern — was das
/// Master-Token nicht hat, bekommt auch das Sub-Token nicht.</para>
/// </summary>
public sealed class TokenProvisioner(DeploymentcenterApi api)
{
/// <summary>Was eine Instanz braucht: Heartbeats senden und Fehler melden.</summary>
public static readonly string[] InstanceScopes = ["watchdog:ping", "bugtracker:report"];
public async Task<ProvisionedToken> ProvisionAsync(
string clientName,
string instanceId,
IReadOnlyList<string> scopes,
string environment = "production",
CancellationToken ct = default)
{
var payload = new
{
client_name = clientName,
instance_id = instanceId,
scopes,
environment
};
var response = await api.PostAsync("/api/tokens/v1/provision", payload, ct)
.ConfigureAwait(false);
var token = response.TryGetProperty("sub_token", out var t) && t.ValueKind == JsonValueKind.String
? t.GetString()
: null;
if (string.IsNullOrWhiteSpace(token))
{
throw new DeploymentcenterException(
"no_token",
"Die Provisionierung lieferte kein Token.",
System.Net.HttpStatusCode.OK);
}
var tokenId = response.TryGetProperty("token_id", out var i) && i.ValueKind == JsonValueKind.String
? i.GetString() ?? ""
: "";
var granted = new List<string>();
if (response.TryGetProperty("scopes", out var s) && s.ValueKind == JsonValueKind.Array)
{
granted.AddRange(s.EnumerateArray()
.Where(e => e.ValueKind == JsonValueKind.String)
.Select(e => e.GetString()!));
}
return new ProvisionedToken(token, tokenId, granted);
}
/// <summary>
/// Fragt das Token, mit dem diese Verbindung arbeitet, nach sich selbst.
///
/// <para>Nicht Neugier, sondern die einzige Möglichkeit, die Projektbindung zu
/// erfahren: Sie steht nicht im Token und ergibt sich nicht aus dem, was beim
/// Ausstellen mitgeschickt wurde — sie ist eine Eigenschaft, die der Server führt.
/// Siehe <see cref="TokenIdentity"/>, warum das nicht gleichgültig ist.</para>
/// </summary>
public async Task<TokenIdentity> WhoAmIAsync(CancellationToken ct = default)
{
var response = await api.GetAsync("/api/tokens/v1/whoami", ct).ConfigureAwait(false);
var scopes = new List<string>();
if (response.TryGetProperty("scopes", out var s) && s.ValueKind == JsonValueKind.Array)
{
scopes.AddRange(s.EnumerateArray()
.Where(e => e.ValueKind == JsonValueKind.String)
.Select(e => e.GetString()!));
}
return new TokenIdentity(
Text(response, "token_id") ?? "",
Text(response, "name") ?? "",
Text(response, "type") ?? "unknown",
Text(response, "project_slug"),
Text(response, "environment") ?? "",
scopes);
}
/// <summary>Eine Zeichenkette aus der Antwort; <c>null</c> auch bei JSON-<c>null</c>.</summary>
private static string? Text(JsonElement root, string name)
=> root.TryGetProperty(name, out var v) && v.ValueKind == JsonValueKind.String
? v.GetString()
: null;
}