F13 PHPStan-Level-8 und harte Code-Regeln vollständig erfüllen

Frühere Version vom 2026-07-16T18:39:13.335375+02:00 · zur aktuellen Fassung

PHPStan-Level-8 und harte Code-Regeln vollständig erfüllen

Finding: F13 Priorität: P1 Status: OFFEN Stand: 2026-07-16

Problembeschreibung

PHPStan Level 8 meldet 58 Fehler.
Nichtkommentierte LOC über 150: InvoiceRecord 304, HttpWetellSourceAdapter 231, run-rechnung 204, Ingest 174, LexofficeAdapter 155.
Nullable Strings, cURL string|true, untypisierte Iterables, tote Bedingungen und Constructorreihenfolge.
Syntax/Tests grün, Level-8 rot.

Ziel

Produktiver PHP-Code ist Level-8-fehlerfrei, unter Größenlimits und in kleine kohäsive Verantwortlichkeiten zerlegt.

Aufgabe

58 Befunde ohne Baseline/Ignore lösen.
Großklassen in Mapper/Parser/Retry/Transitions/HTTP teilen.
Shapes/List-Typen und Rückgabewertprüfungen ergänzen.

Scope

In Scope: alle unter Code Blast genannten Komponenten, Daten, Verträge und Tests.
Out of Scope: neue Features und produktive externe Mutation ohne separate Freigabe.

Harte Spezifikation

MUST: 0 PHPStan ohne Ignore.
MUST: jede PHP-Datei <=150 nichtkommentierte LOC.
MUST: final/strict/Docblocks/Constructor Injection.
MUST: Dependencies nach innen.

Harte Quality Gateways

PHPStan Level 8 Exit 0.
LOC 0 und php -l Exit 0.
Architekturtest grün.
PHPUnit bleibt grün.
Global: PHPStan Level 8 und PHPUnit liefern 0 Fehler; kein Ignore oder Baseline.

Code Blast

Direkt: mindestens 12 Fehlerdateien/5 LOC-Dateien.
Indirekt: Constructors, Komposition, Tests.

Code-Impact-Analyse

Großer Refactor mit Regressionseinfluss.
Zuerst Characterization, dann Struktur.

Risikoanalyse

| Risiko | Eintritt | Schaden | |---|---|---| | Refactor-Regression | mittel | hoch | | Blindes Casting/Baseline | hoch | hoch |

Strategie zur Risikonullierung

Prevent: Characterization.
Detect: Level8/LOC/Architektur CI.
Contain: kleine Pakete.
Recover: Fixturevergleich.

Abschlussnachweis

Erledigt erst, wenn alle MUST-Spezifikationen implementiert, alle Gateways mit unveränderlicher Evidenz grün und Restrisiken technisch ausgeschlossen oder ausdrücklich fachlich akzeptiert sind.