P2503-237 Review — Projektpartner-Suchagent

Branch feature/p2503-226-projektpartnerportal · Fixed point 1e92bf0 · Commits db0a93c und 3648dff · 2026-06-16

VERDICT: CLEAN — P2503-237 review-ready

Der Review bewertet nur den finalen P2503-237-Aktivierungs- und Review-Fix-Diff nach dem letzten Feature-Train-Review. Der vorher gebaute Alert-Code wird dadurch in der Site-Config aktiviert, die Rolle berechtigt, die Alert-Mail- und Migrationsconfig synchronisiert, die Drupal-11-Constraint-Annotation ersetzt und die Mail-Platzhalterlogik direkt getestet.

Review Scope

Spec	P2503-237 — Suchagent / Projekt-Alert mit Kriterien-Migration
Diff	`git diff 1e92bf0...HEAD`
Files	10 Dateien, 469 Insertions, 18 Deletions
Canvas	skipped Keine Canvas- oder SDC-Registry-Dateien im Diff.

Standards

clean Keine offenen hard violations und keine judgement calls.

accessCheck(FALSE) in PppAlertMailManager ist mit einem englischen Kommentar begründet: Cron-Mail-Composition läuft im Systemkontext für den Alert-Owner und darf nicht vom aktuellen Request-User abhängen.
PppAlertUniqueConstraint nutzt Drupal-11-Attribute statt der alten @Constraint-Annotation.
Die neue Mail-Platzhalterlogik ist in PppAlertMailManagerTest direkt abgedeckt: HTML, Plain Text, Display-Name-Fallback und HTML-Escaping.
Keine lokalen Overrides oder Secrets sind Teil des Diff; .codex/config.toml blieb unstaged.

Spec

clean Keine fehlenden Anforderungen, kein Scope Creep und keine falsche Implementation im Review-Diff.

Das Alert-Modul ist in config/sync/core.extension.yml aktiviert.
Die Rolle projektpartner erhält manage own ppp alert.
Die Sync-Migration erzeugt ppp_alert-Entities, nutzt migration_lookup auf Projektpartner-User, mappt Skills über neos_doctrine_array und neos_skill_review und validiert gegen die Unique-Constraint.
Das Easy-Email-Template nutzt den neuen Platzhalter [solcom:projektpartner-name] statt ungültiger User-Feld-Tokens.

Validation Evidence

Gate	Result
P2503-237 Kernel tests	OK 14 Tests, 227 Assertions; nur bestehende Drupal-/Contrib-Deprecations.
PHPCS	OK Alert-MailManager, Constraint und Tests.
Diff check	OK `git diff --check`.
Jira	OK P2503-237 steht auf Review.

Remaining Context

Der echte Neos-Dump ist noch nicht importiert. Die lokale Site enthält weiterhin nur die synthetisch verifizierten Projektpartner-Daten, bis die reale neos-Verbindung und der Dump bereitgestellt werden.
Die Deprecations im Testlauf stammen aus Drupal-Core-/Contrib-Pfaden und blockieren diesen Ticket-Review nicht.