Fruehjahrsputz: toter Code, ungenutzte Symbole, Dokumentenstand

Alles Entfernte war nachweislich ohne Aufrufer. Build, 198/198 Tests, Smoke-UI und Daemon-Prueflauf sind vor und nach jedem Schritt gruen. Code: - AIModelService: Platzhalter, der immer 0.5 lieferte. Im DI registriert, aber nie irgendwo injiziert. Ordner Core/AI faellt mit weg. - CtApiWrapper + CtMeta: JSON-Modelle fuer einen {data,meta}-Umschlag, den CapitolTrades nicht mehr liefert. Der Scraper deserialisiert seit laengerem direkt List<CtTrade>. - IBKRGatewayService: DisconnectAsync, InitBrokerageSessionAsync und SearchStocksBySymbolAsync. Der Dienst selbst bleibt - er versorgt Instrument-Sync, Kurshistorie und den Watchdog-Heartbeat. - Je eine Methode ohne Aufrufer: BudgetService.GetAvailableBudgetAsync, TradeHistoryService.GetRecentTradesAsync, CongressRepository. GetAllTradeIdsAsync und .ResetHistoryImportAsync, IbkrMapping.DefaultPortFor, SecretProtection.IsEncrypted. - CongressRepository bekam damit einen LoggingService injiziert, den es nicht mehr benutzt - Abhaengigkeit samt Konstruktorparameter raus. Ressourcen: - 17 Symbole der WinForms-Oberflaeche entfernt. Das Wildcard-Muster im csproj nahm sie in die Binaerdatei auf, ViewIcons.cs bildet aber nur sieben Schluessel ab. Resources/ enthaelt jetzt genau die sieben. NuGet-Allowlist: - Dapper und HtmlAgilityPack sind seit R3 bzw. R1 aus dem Projekt raus, Microsoft.WindowsDesktop.* seit L5. Muster entfernt. - MySqlConnector und Newtonsoft.Json stehen NUR transitiv in den Projektdateien und wurden zuerst mitentfernt - ein Restore in einen leeren Paket-Ordner scheiterte darauf mit NU1100. Beide wieder aufgenommen, jetzt mit Begruendung, damit der naechste Aufraeumlauf nicht dieselbe Falle tritt. Dokumente an den tatsaechlichen Stand angeglichen: - ARCHITECTURE: R2 fuehrte die Umstellung auf IHostedService als offen, obwohl R4 sie erledigt hat. L6 und die Deploymentcenter-Phase fehlten ganz. - DC-Konzept: Schritte 0-8 standen auf "dieser Durchlauf", sind aber umgesetzt. Jetzt je Schritt der wirkliche Stand - inklusive der beiden Halbfertigen: Update-PRUEFUNG laeuft, das Anwenden hat keinen Aufrufer; die Release-Pipeline steht, ist aber nie gelaufen. P5 ist eingetreten. - Accounting und Supervisor trugen keinen Umsetzungsvermerk, obwohl beide Module gebaut sind. Vermerk nach dem Muster des Linux-Konzepts ergaenzt, mit dem, was jeweils offen bleibt. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-08-23 12:25:03 +02:00
parent 0a72ea7e28
commit e1546bd1b1
33 changed files with 78 additions and 163 deletions
@@ -97,7 +97,7 @@ src/IBKRTrader.Hosting        (classlib, net10.0) – AppHostBuilder + RunStartu
 - [x] `ShellUiHost` + `LauncherForm` (View-Buttons + gemeinsames Fenster-Menü); Core-Views (Workers/Logs/Settings) als eigene Fenster
 - [x] CongressTrading auf neuen `IModule`-Vertrag; Modul-Worker als `IWorker` registriert
 - [x] `--smoke-ui` Headless-Test (alle Views + Launcher konstruieren) → grün
- [ ] **Offen (R4/R5):** Worker von `WorkerEngine`/`IWorker` auf `IHostedService` umstellen (aktuell noch WorkerEngine)
+- [x] Worker von `WorkerEngine`/`IWorker` auf `IHostedService` umgestellt — **in R4 erledigt**, die Zeile stand hier bis zum 2026-08-23 faelschlich noch offen

 ### R3 – Persistenz auf EF Core
 **Festgelegt:** EF-Migrationen **extern** (wie PolytraderSharp) – Schema per `dotnet ef database update`,
@@ -163,6 +163,34 @@ Anzeigegerät mehr und ist damit erstmals Teil der CI.
 **Offen:** IB Gateway kopflos betreiben (IBC + Xvfb) – eigene Baustelle, unabhängig vom Code;
 LiveCharts2 kommt mit den neuen Modulen (Avalonia deshalb auf der 11er-Linie gepinnt).

+### L6 – Namensgebung bereinigt ✅ (2026-08-07)
+- [x] `IBKRTrader.App.Avalonia` → **`IBKRTrader.App`**. Das Suffix gab es nur, solange daneben eine
+      WinForms-`IBKRTrader.App` stand; seit L5 ist die weg. Assembly, Wurzel-Namensraum und die
+      `avares://`-Ressourcen-URI sind mitgezogen, Git erkennt alles als Umbenennung.
+- [x] Die `global::Avalonia`-Qualifizierungen entfallen – sie waren nur nötig, weil der Namensraum
+      `IBKRTrader.App.Avalonia` das gleichnamige Paket verdeckte.
+- [x] **Inhaltlich falsch gewordene Aussagen berichtigt** – das waren die eigentlichen Überbleibsel,
+      nicht die Kommentare: `.agents/rules/grundregeln.md` schrieb weiterhin „C# .NET 10 WinForms",
+      RichTextBox-Logging, `LauncherForm` und `PropertyGrid` vor und hätte die Portierung Stück für
+      Stück rückgängig gemacht. Dazu Core-Kommentare (`LogEntry`, `IWorker`, `ModuleView`) und Doku.
+
+### DC – Deploymentcenter-Integration ✅ (2026-08-23)
+Konzept und Begründung: [konzepte/KONZEPT-Deploymentcenter-Integration.md](konzepte/KONZEPT-Deploymentcenter-Integration.md).
+Schritte 0–8 der dortigen Reihenfolge sind umgesetzt; der Stand je Schritt steht in §9 dieses Konzepts.
+
+- [x] Lizenz (Sperrbetrieb statt Abbruch), Watchdog-Heartbeat, Fehler-Stream inkl. globaler
+      Ausnahmebehandler, Update-Prüfung mit `ReleaseCredentials`, `setup.json` + Release-Pipeline.
+      Einbauort ist `IBKRTrader.Hosting/Deploymentcenter/` – den Host teilen sich Shell und Daemon.
+- [x] Vier Projekt-Befunde vorab bereinigt: Zugangsdaten aus `AppSettings` (P1), `AppPaths`-Rückfall
+      unter Windows (P2), globale Handler (P3), Version zentral in `Directory.Build.props` (P4).
+- [ ] **Offen (P5):** Die Gitea-CI checkt das Schwester-Repo `Deploymentcenter` nicht aus. Solange
+      das SDK als Cross-Repo-`ProjectReference` hängt, ist der CI-Lauf rot. Behebt sich mit
+      Schritt 10 (SDK als NuGet-Paket in der Gitea-Registry).
+- [ ] **Offen (Schritt 9):** Bugtracker-Baustein – setzt voraus, dass das Projekt `ibkrtrader` im
+      DC-WebUI angelegt ist und ein Token mit `bugtracker:report` vorliegt (serverseitige Handarbeit).
+- [ ] **Offen:** Das Anwenden eines gefundenen Updates ist nicht verdrahtet. `DcUpdateService.LaunchAgent`
+      ist fertig und dokumentiert, es fehlt der Aufrufer, der danach geordnet herunterfährt.
+
 ---

 ## Kurskorrektur abgeschlossen (R1–R7)
@@ -523,6 +523,9 @@ Packager, der genau das abfängt — kein Fehlschlag, sondern die Absicherung, d

 ## 8. Befunde und offene Punkte

+> **Status 2026-08-23:** P1–P4 sind umgesetzt (Einzelheiten in §9). **P5 ist eingetreten**,
+> P6 entscheidet sich erst beim ersten Release. Der ursprüngliche Vermerk von 2026-08-14:
+>
 > **Status 2026-08-14:** Alle sieben ursprünglich an den Deploymentcenter-Entwickler
 > gemeldeten Befunde (D1–D7) sind mit den Versionen 2.5.0/2.5.1 behoben — bestätigt über
 > `GET /api/updateservice/v1/changelog?since=2.4`. Der Abschnitt bleibt als Nachweis
@@ -567,6 +570,13 @@ ersten Release ab — zu Recht.
 nur IBKRTrader aus. Eine `ProjectReference` nach `..\..\..\..\Deploymentcenter\…`
 macht den Build auf beiden Matrix-Zielen rot. Siehe die Entscheidung in §2.2.

+> **Seit 2026-08-23 eingetreten, nicht mehr nur vorhergesagt.** Die
+> `ProjectReference` ist mit der Integration auf `main` gelandet, die CI ist
+> damit rot. Das war die bewusst in Kauf genommene Folge der Interimslösung aus
+> §2.2 — lokal baut die Projektmappe, sobald das Schwester-Repo daneben liegt.
+> Auflösung ist Schritt 10: SDK als NuGet-Paket in die Gitea-Registry, dann
+> `PackageReference` statt Cross-Repo-Pfad.
+
 **P6 · Konfiguration liegt im Installationsverzeichnis.** Unter Windows schreibt
 `AppPaths` neben die Binärdatei. Das ist mit `preservePatterns` beherrschbar
 (§7.3), muss aber beim ersten Release stimmen — ein Update, das `settings.json`
@@ -627,16 +637,16 @@ nicht durchgängig beziffert). ✅ **Behoben (2.5.1, Doku)** — Changelog nennt

 | Schritt | Inhalt | Abhängig von | Stand |
 |---|---|---|---|
-| 0 | **P1** Zugangsdaten wechseln, Vorgaben auf Platzhalter | — | ⬜ dieser Durchlauf |
-| 1 | **P2** `AppPaths`-Rückfall plattformabhängig, **P4** `Directory.Build.props` | — | ⬜ dieser Durchlauf |
-| 2 | SDK-Bezug Stufe 1: Cross-Repo-`ProjectReference` (§2.2) | — | ⬜ dieser Durchlauf |
-| 3 | `DcConfig`, `DeploymentcenterSettings`, `DcApiClient`, `BuildInfo` | 1, 2 | ⬜ dieser Durchlauf |
-| 4 | **P3** globale Handler + Fehler-Stream | 3 | ⬜ dieser Durchlauf |
-| 5 | Watchdog-Heartbeat inkl. `stopped` | 3 | ⬜ dieser Durchlauf |
-| 6 | Lizenz mit Sperrbetrieb statt Abbruch (§5.2), `EnsureLicensedAsync` | 3 | ⬜ dieser Durchlauf |
-| 7 | Update-Prüfung **mit** `ReleaseCredentials`, `exitCurrentApp:false` bei offenem Zustand | 6 (braucht den Schlüssel) | ⬜ dieser Durchlauf |
-| 8 | `setup.json`, `preserve`/`exclude`, Release-Pipeline (Vorlage per `curl` von `/docs/release-template/`) | 1–7 | ⬜ dieser Durchlauf |
-| 9 | Bugtracker-Baustein nach `AGENTS.md` / `.agents/rules` | Projekt-Slug angelegt | ⬜ separat (Projekt `ibkrtrader` muss im WebUI erst angelegt werden) |
+| 0 | **P1** Zugangsdaten wechseln, Vorgaben auf Platzhalter | — | ✅ Vorgaben sind Platzhalter. **Die Rotation des geleakten Passworts steht weiterhin aus** (Nutzer-Aktion) |
+| 1 | **P2** `AppPaths`-Rückfall plattformabhängig, **P4** `Directory.Build.props` | — | ✅ 2026-08-23 |
+| 2 | SDK-Bezug Stufe 1: Cross-Repo-`ProjectReference` (§2.2) | — | ✅ 2026-08-23, inkl. Prüf-Target mit lesbarer Fehlermeldung |
+| 3 | `DcConfig`, `DeploymentcenterSettings`, `DcApiClient`, `BuildInfo` | 1, 2 | ✅ 2026-08-23 |
+| 4 | **P3** globale Handler + Fehler-Stream | 3 | ✅ 2026-08-23 (`DcCrashHandlers`, `DcErrorSink`, `DcErrorReporter`) |
+| 5 | Watchdog-Heartbeat inkl. `stopped` | 3 | ✅ 2026-08-23 (`DcHeartbeatWorker`) |
+| 6 | Lizenz mit Sperrbetrieb statt Abbruch (§5.2), `EnsureLicensedAsync` | 3 | ✅ 2026-08-23 (`LicenseGuard`) |
+| 7 | Update-Prüfung **mit** `ReleaseCredentials`, `exitCurrentApp:false` bei offenem Zustand | 6 (braucht den Schlüssel) | 🔶 Die **Prüfung** läuft beim Start. Das **Anwenden** ist es nicht: `LaunchAgent` ist fertig, aber kein Aufrufer fährt danach geordnet herunter |
+| 8 | `setup.json`, `preserve`/`exclude`, Release-Pipeline (Vorlage per `curl` von `/docs/release-template/`) | 1–7 | ✅ 2026-08-23 (`setup.json`, `scripts/release.*`). **Noch nie ausgeführt** – das erste Release steht aus |
+| 9 | Bugtracker-Baustein nach `AGENTS.md` / `.agents/rules` | Projekt-Slug angelegt | ⬜ offen — Projekt `ibkrtrader` muss im WebUI erst angelegt werden |
 | 10 | **Folgeschritt, nicht Teil dieses Durchlaufs:** `Deploymentcenter.Client` als NuGet-Paket in die Gitea-Registry pushen, `<Import>` durch `PackageReference` ersetzen, Gitea-CI (**P5**) auf den Paketbezug umstellen | 2 | ⬜ offen — siehe §2.2 |

 Schritt 7 und 8 hängen zusammen: **der erste ausgelieferte Build muss die
@@ -1,5 +1,15 @@
 # Konzept: Modul „Accounting" (Buchhaltung/Reporting aller Konten)

+> **UMGESETZT (Modulgerüst).** Das Modul steht: `acc_`-Schema mit Migration `InitialAccounting`,
+> `AccountingIngestService` (append-only, idempotent über `IdempotencyKey`), `AccountingClassifier`,
+> `AccountingEngine`, `FxConverter`, `AccountingReportService` sowie CSV- und PDF-Export. Die
+> Ingest-Quellen liegen hinter Interfaces mit **Offline-Null-Stubs** — das Modul läuft vollständig
+> und bucht dabei korrekt nichts.
+>
+> **Weiterhin offen ist genau die Zielland-Arbeit aus §6** — vor allem der Live-Flex-Abruf, ohne den
+> keine echten Buchungen entstehen, und die Steuerschicht, deren Jurisdiktion nicht festgelegt ist.
+> Das Modul ist damit lauffähig, aber noch nicht in Betrieb.
+
 > Stand: 2026-07-30
 > Ziel: Vollständige, **von unserer Trading-DB unabhängige**, buchhalterisch korrekte Erfassung ALLER
 > Kontobewegungen der IBKR-Konten. Periodische (meist monatliche), vor einer Steuerbehörde
@@ -1,5 +1,13 @@
 # Konzept: Modul „Supervisor" (KI-gestützte Handels-Analyse & Forensik)

+> **UMGESETZT (S-0 bis S-4).** Datenfundament im Core (`core_decision_journal`, `core_order_events`,
+> durchgereichte `SignalId`, JSONL-Log-Sink), `DossierService`/`DossierBuilder`, der
+> OpenRouter-Agent mit read-only Tool-Registry und Profilen, `sup_reports`, `CounterfactualJob`,
+> `DailyReportService` und der MCP-Server (`McpLightServer`, `McpJsonRpc`).
+>
+> **Weiterhin offen** sind die beiden Punkte am Ende dieses Dokuments: die Counterfactual-Kursauflösung
+> (Interface + Stub vorhanden) und der externe Versand des Tagesberichts.
+
 > Stand: 2026-07-30
 > Ziel: ALLES, was IBKRTrader getan (und bewusst NICHT getan) hat, detailliert analysierbar machen —
 > Entscheidungen, Orders, Trades und Logs — und die Analyse durch ein KI-Modell (OpenRouter) durchführen