refactor: weitere identische Pakete nach common/ (Batch 2) #377

Closed

patrick wants to merge 1 commits from refactor/common-batch-2 into refactor/common-modul-pilot

pull from: refactor/common-batch-2

merge into: scripte:refactor/common-modul-pilot

scripte:main

scripte:feat/ai-app-mandantenmodell

scripte:feat/ai-app-restpakete

scripte:feat/ai-app-rollout-131

scripte:fix/ai-app-write-timeout

scripte:chore/migration-kanban-board

scripte:chore/connect-kanban-board

scripte:chore/workflow-kanban-board

scripte:chore/archive-kanban-board

scripte:chore/mail-kanban-board

scripte:chore/dms-kanban-board

scripte:chore/core-kanban-board

scripte:chore/ai-kanban-board

scripte:chore/ai-kanban-security-folgetickets

scripte:feat/ai-app-deployment

scripte:feat/ai-app-vorschlaege

scripte:feat/ai-cls09-herkunftsmarkierung

scripte:feat/ai-eng09-gemini

scripte:feat/ai-cls08-mehrsprachig

scripte:feat/ai-titel-korrespondent

scripte:feat/dms-backup-s3

scripte:feat/dms-s3-verdrahten

scripte:docs/alfresco-vergleich

scripte:refactor/common-modul-pilot

scripte:fix/dms-httpharden-normalize

scripte:feature/shl-02-pagecontainer-dedup

scripte:test/dedup-fakekek-testutil

scripte:feature/nav-01-mail-postfach-registry-eintrag

scripte:feature/sec-08-09-rbac-cmis-audit

scripte:feature/shl-04-globale-modul-navigation

scripte:chore/mail-kanban-security-folgetickets

scripte:chore/dms-kanban-security-folgetickets

scripte:chore/core-kanban-security-folgetickets

scripte:chore/archive-kanban-security-folgetickets

scripte:chore/workflow-kanban-security-folgetickets

scripte:chore/connect-kanban-security-folgetickets

scripte:feature/qa-08-barrierefreiheits-audit-workflow

scripte:feature/tsk-04-aufgaben-oberflaeche-workflow

scripte:feature/qa-08-barrierefreiheits-audit-connect

scripte:feature/int-18-smtp-gateway-weiterleitung-an-feste-adresse-dms

scripte:docs/rollout-131-vorbereitung

scripte:feature/met-08-facturx-zugferd-szenarien-validator-dms

scripte:feature/qa-15-sicherheitspruefung-gesamtprojekt

scripte:fix/testrun-2026-09-21-dms

scripte:feature/cmp-07-dsgvo-loeschantrag-recht-auf-loeschung

scripte:feature/cmp-06-vier-augen-freigabe-fuer-loeschungen

scripte:feature/cmp-02-dsgvo-datenschutz-berichte

scripte:feature/ret-03-loeschworkflow-und-aufbewahrungssperre-legal-hold

Conversation 1 Commits 1 Files Changed 96

+81 -930

patrick commented 2026-10-27 14:06:11 +01:00

Owner

Copy link

Copy source

Stackt auf #376 (Pilot common/), Base-Branch refactor/common-modul-pilot. Nach Merge von #376 auf main umstellen.

Änderung

Weitere Pakete nach common/, jeweils nur dort, wo der Code ohne Kommentare und Importpfade byte-gleich ist (per normalisierter Prüfsumme belegt, nicht nur nach Paketnamen):

Paket	zusammengeführt aus	bleibt lokal
httpclient	ai, connect	Root (strengerer Wächtertest http.Client{}), dms/mail/archive schon in #376
svcauth	archive, ai, workflow	Root (anderes Paket)
auditclient	ai, connect, archive	–
tenantregistryclient	dms, ai, workflow	archive (nutzt httpclient.Default())
policyclient	mail, connect	dms (eigene, größere Variante)
httpharden	Root, archive	dms (ohne Normalize, hängt an #375)
loadtest	mail, ai	archive, connect (abweichend)

• go.mod (require + replace) für ai, connect, workflow
• Sicherheit: svcauth (Konstantzeit-Vergleich) und tenantregistryclient waren in den Pflichttest-Gates von ai/workflow als sicherheitskritisch gelistet. Die Gates kennen jetzt zusätzlich common/svcauth/ und common/tenantregistryclient/, die CI-Pfadfilter common/**, sonst fiele die Absicherung beim Verschieben still weg
• reine Import-Sortierung per gofmt in 34 berührten Dateien (je 1 Zeile)

Bewusst nicht zusammengeführt

notifyclient (87/85/83 Zeilen, verschiedene Fassungen), pflichttestgate (modulspezifische Pfadlisten, müsste parametrisiert werden), migrate und testdb (brauchen pgx in common, also externe Abhängigkeit + go.sum; bei testdb zusätzlich Schema-Präfix-Kollisionsgefahr zwischen Modulen), policyclient/tenantregistryclient/httpharden jeweils die abweichende Variante.

Verifikation (Host 131, frisches Wurfverzeichnis)

• go build ./... sauber in common, Root, dms, mail, archive, ai, connect, workflow
• go test grün in common, dms, archive, ai, connect
• Rot, wie auf main: internal/archtest (ai-Fixture ruft ListenAndServe direkt), mail/internal/mailapi (OpenAPI), workflow/internal/contracttest (Signatur, behoben in #374), mail/internal/qa09gate (nur wegen fehlendem .git im Wurfverzeichnis)
• Nicht geprüft: DB-abhängige Tests (kein TEST_ADMIN_DSN)

🤖 Generated with Claude Code

Stackt auf #376 (Pilot `common/`), Base-Branch `refactor/common-modul-pilot`. Nach Merge von #376 auf `main` umstellen. ## Änderung Weitere Pakete nach `common/`, jeweils nur dort, wo der Code ohne Kommentare und Importpfade byte-gleich ist (per normalisierter Prüfsumme belegt, nicht nur nach Paketnamen): | Paket | zusammengeführt aus | bleibt lokal | |---|---|---| | `httpclient` | ai, connect | Root (strengerer Wächtertest `http.Client{}`), dms/mail/archive schon in #376 | | `svcauth` | archive, ai, workflow | Root (anderes Paket) | | `auditclient` | ai, connect, archive | – | | `tenantregistryclient` | dms, ai, workflow | archive (nutzt `httpclient.Default()`) | | `policyclient` | mail, connect | dms (eigene, größere Variante) | | `httpharden` | Root, archive | dms (ohne `Normalize`, hängt an #375) | | `loadtest` | mail, ai | archive, connect (abweichend) | - `go.mod` (`require` + `replace`) für ai, connect, workflow - **Sicherheit:** `svcauth` (Konstantzeit-Vergleich) und `tenantregistryclient` waren in den Pflichttest-Gates von ai/workflow als sicherheitskritisch gelistet. Die Gates kennen jetzt zusätzlich `common/svcauth/` und `common/tenantregistryclient/`, die CI-Pfadfilter `common/**`, sonst fiele die Absicherung beim Verschieben still weg - reine Import-Sortierung per `gofmt` in 34 berührten Dateien (je 1 Zeile) ## Bewusst nicht zusammengeführt `notifyclient` (87/85/83 Zeilen, verschiedene Fassungen), `pflichttestgate` (modulspezifische Pfadlisten, müsste parametrisiert werden), `migrate` und `testdb` (brauchen `pgx` in `common`, also externe Abhängigkeit + `go.sum`; bei `testdb` zusätzlich Schema-Präfix-Kollisionsgefahr zwischen Modulen), `policyclient`/`tenantregistryclient`/`httpharden` jeweils die abweichende Variante. ## Verifikation (Host 131, frisches Wurfverzeichnis) - `go build ./...` sauber in `common`, Root, dms, mail, archive, ai, connect, workflow - `go test` grün in `common`, dms, archive, ai, connect - Rot, **wie auf `main`**: `internal/archtest` (ai-Fixture ruft `ListenAndServe` direkt), `mail/internal/mailapi` (OpenAPI), `workflow/internal/contracttest` (Signatur, behoben in #374), `mail/internal/qa09gate` (nur wegen fehlendem `.git` im Wurfverzeichnis) - Nicht geprüft: DB-abhängige Tests (kein `TEST_ADMIN_DSN`) 🤖 Generated with [Claude Code](https://claude.com/claude-code)

patrick added 1 commit 2026-10-27 14:06:11 +01:00

refactor: weitere identische Pakete nach common/ (svcauth, auditclient, tenantregistryclient, policyclient, httpharden, loadtest, httpclient) ... 7515b8db4d

Nur Pakete, deren Code ohne Kommentare/Importpfade byte-gleich ist:
- httpclient: ai, connect (Root behaelt eigene Kopie wegen strengerem Waechtertest)
- svcauth: archive, ai, workflow
- auditclient: ai, connect, archive
- tenantregistryclient: dms, ai, workflow (archive weicht ab, bleibt)
- policyclient: mail, connect (dms weicht ab, bleibt)
- httpharden: Root, archive (dms weicht ab, bleibt)
- loadtest: mail, ai (archive, connect weichen ab, bleiben)

go.mod (require + replace) fuer ai, connect, workflow; Pflichttest-Gates
von ai/workflow und CI-Pfadfilter um common/ erweitert, damit die
sicherheitskritischen Pakete (svcauth, tenantregistryclient) weiter
gegated werden.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

patrick commented 2026-10-27 14:37:36 +01:00

Author

Owner

Copy link

Copy source

Geschlossen: replace => ../common bricht Einzel-Builds (Docker nur mit Modul-Ordner, eigene Repos, Partner-Builds). Wird neu aufgesetzt als versioniertes Modul common per Git-Tag. Code wird wiederverwendet.

Geschlossen: `replace => ../common` bricht Einzel-Builds (Docker nur mit Modul-Ordner, eigene Repos, Partner-Builds). Wird neu aufgesetzt als versioniertes Modul `common` per Git-Tag. Code wird wiederverwendet.

patrick closed this pull request 2026-10-27 14:37:38 +01:00

Pull request closed

Please reopen this pull request to perform a merge.

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

No items

No labels

Milestone

No items

No Milestone

Projects

Clear projects

No projects

Assignees

Clear assignees

patrick

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: scripte/nexarch#377