Gesamtprojekt: Nachtrag 31 -- neun PRs gemergt, echte Testregression in DAT-01 behoben, main erstmals aktualisiert
CI / Web-Barrierefreiheitspruefung (axe, Tastatur, Reflow, Textabstaende) (pull_request) Canceled after 0s
CI / Web (shl) (pull_request) Canceled after 0s
Core-Schnittstellen-Vertragstests / contract-tests (pull_request) Canceled after 0s
Supply-Chain-Scan (Go) / govulncheck (pull_request) Canceled after 0s
Supply-Chain-Scan (npm) / npm-audit (pull_request) Canceled after 0s
Core-Pflichttest-Gate / pflichttest-gate (pull_request) Canceled after 0s
RBAC-Rollenhierarchie-Pflichtgate / rbac-policy-invariant (pull_request) Canceled after 0s
CI / Core-Modul (Build/Vet/Test) (pull_request) Canceled after 0s
CI / DMS Security-Test-Gate (AC5, Merge-Gate) (pull_request) Canceled after 0s
CI / DMS-Modul Supply-Chain-Scan (govulncheck) (pull_request) Canceled after 0s
CI / Web (dms-browser) (pull_request) Canceled after 0s
CI / Web (pdf-viewer) (pull_request) Canceled after 0s
CI / DMS-Modul (Lint/Build/Test/Coverage/Artefakt) (pull_request) Canceled after 0s
CI / Core-Modul (Build/Vet/Test) (push) Canceled after 0s
CI / DMS Security-Test-Gate (AC5, Merge-Gate) (push) Canceled after 0s
CI / DMS-Modul (Lint/Build/Test/Coverage/Artefakt) (push) Canceled after 0s
CI / DMS-Modul Supply-Chain-Scan (govulncheck) (push) Canceled after 0s
CI / Web (dms-browser) (push) Canceled after 0s
CI / Web (pdf-viewer) (push) Canceled after 0s
CI / Web (shl) (push) Canceled after 0s
CI / Web-Barrierefreiheitspruefung (axe, Tastatur, Reflow, Textabstaende) (push) Canceled after 0s

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
rootandClaude Sonnet 5 committed 2026-09-25 20:54:07 +00:00
1 parent 130b1c21eb
commit 12263bd774
1 file changed
+38
+38
View File
@@ -783,3 +783,41 @@ Vier PRs nacheinander in `integration/gesamtprojekt` gemergt: #168 (Workflow VOR
**Restliste:** unverändert gegenüber Nachtrag 29 — AI und Workflow haben weiterhin nur die Bibliotheksebene, keinen eigenen `cmd/`-Dienst; INT-13s Empfangspfad bei Workflow ist bislang nur simuliert geprüft, echte Ende-zu-Ende-Prüfung erst möglich, sobald Workflow einen laufenden Dienst hat.
Commits: `496674e`..`37ccfa0` (VOR-03), `0833264` (ENG-07), `7996c3e`..`760e755` (ENG-06), `a9189ec`..`4a75586` (INT-13), Merge-Commits `d5bf1cc`, `5c04c8f`, `7633d09`, Migrationskollision `301a22d`. Head nach Push auf `integration/gesamtprojekt` aktuell. Kein Merge nach `main`.
## Nachtrag 31 — Merge nach main (erster echter main-Merge der Session)
Nutzerfreigabe: "Beide jetzt mergen" (integration/gesamtprojekt + integration/dms-gesamtstand nach main), anschließend Rollout auf Host 131 vorbereiten und starten.
**Schritt 1 — neun offene PRs in integration/gesamtprojekt gemergt** (waren gegen integration/gesamtprojekt bzw. main als base geöffnet, noch nicht selbst gemergt):
- #172 ENG-02 (AI, Ollama-Adapter-Verifikation)
- #173 VOR-04 (Workflow, Rückkanal-API)
- #174 UX-09 (Mail, Akten-/Vorgangsanzeige)
- #175 APR-02 (Workflow, Vier-Augen-Freigabe)
- #176 CLS-02 (AI, Dokumenttyp-Erkennung)
- #177 DAT-01 (Connect, DATEV-Export)
- #178 ERP-01 (Connect, ERP-Connector-Grundgerüst)
- #179 APR-03 (Workflow, Genehmigungs-Oberfläche)
- #180 EXT-02 (AI, Rechnungsdaten-Extraktion)
Alle neun Merges konfliktfrei. `go build`/`go vet` für alle sieben Module (Core, DMS, Archive, Mail, Workflow, AI, Connect) sauber.
**Echte Regression gefunden und behoben:** `connect/internal/datevexport/store_test.go` (aus DAT-01, PR #177) rief `testdb.Pool` auf, aber nie `migrate.Load`/`migrate.Up` — anders als jedes andere Connect-Testpaket (siehe `credential_test.go`, `fieldmapping_test.go`, `statusapi_test.go`, `syncjob_test.go`). Beide Tests schlugen real fehl (`relation "datev_exports" does not exist`). Fix: `openTestPool` + `findMigrationsDir`-Helfer nach dem etablierten Muster ergänzt (Commit `130b1c2`). Danach 2/2 grün, keine Auswirkung auf Produktionscode.
Keine Migrationsnummernkollision durch die neun Merges verursacht (Connect-Migrationen 0001–0006 durchgehend eindeutig). Vorgefundene doppelte Nummern in Workflow (`0004`: approval_tasks/history_append_only/step_retry) und AI (`0001`: drei Dateien) sind Altlasten aus vorherigen Merges — vollständige Dateinamen bleiben eindeutig, migrations-fresh-check.sh deckt diese beiden Module ohnehin nicht ab (nur Core/DMS/Archive/Mail), reale `go test`-Läufe beider Module liefen grün, keine Korrektur in diesem Nachtrag vorgenommen.
**Verifikation auf integration/gesamtprojekt (Host 204, Wegwerf-Cluster Port 5433):**
- `go build`/`go vet`: alle sieben Module sauber
- `go test ./...` für workflow/ai/connect: alle Pakete grün (nach obigem Fix)
- `scripts/check-port-defaults.sh`: PASS
- `scripts/check-route-prefixes.sh`: PASS
- `scripts/migrations-fresh-check.sh`: PASS (Core Registry 35, Core Mandant 20, DMS 37, Archive 16, Mail 12 Migrationen, 96 Tabellen in gemeinsamer Mandanten-DB)
- `scripts/e2e-core-dms.sh`: `TestCoreE2E_TokenJWKSAndPolicy` PASS, alle 10 Unterfälle
- `scripts/gesamt-token-smoke.sh`: 45/45 bestanden, Rückbau-Snapshot identisch
**Schritt 2 — Merge nach main:** `integration/dms-gesamtstand` (PR #72) ist bereits vollständiger Vorfahre von `integration/gesamtprojekt` (`git merge-base --is-ancestor` bestätigt) — kein separater Merge nötig, keine divergierende Linie. `main` war bereits Vorfahre von `integration/gesamtprojekt`, daher Fast-Forward-Merge, kein Merge-Commit nötig.
Nach dem Merge erneute vollständige Verifikation direkt gegen den neuen `main`-Stand (nicht nur gegen die Integrationsbranch): `go build`/`go vet` alle sieben Module sauber, `migrations-fresh-check.sh` PASS, `gesamt-token-smoke.sh` 45/45 bestanden, Rückbau-Snapshot identisch.
**Ergebnis:** `main` Head `2337841` → `130b1c2`. `main` ist damit erstmals seit Beginn dieser Session der tatsächliche, geprüfte Gesamtstand aller sieben Module — nicht nur die Integrationsbranches. Kein Deploy in diesem Schritt (folgt als separater Rollout-Schritt auf Host 131 nach Nutzerfreigabe).
**Restliste:** unverändert gegenüber Nachtrag 30 — AI und Workflow haben weiterhin nur die Bibliotheksebene bzw. je eine neue Next.js-Oberfläche (workflow-approvals, workflow läuft aber noch ohne eigenen Go-`cmd/`-Dienst), Connect hat als einziges der drei neuen Module einen laufenden Go-`cmd/`-Dienst (`connect-app`, API-02). INT-13s Empfangspfad bei Workflow weiterhin nur simuliert geprüft.