Kevin Adametz
e783adc54e
phpstan lief mit 390 Meldungen außerhalb der Baseline, der größte Teil davon Falschmeldungen. Ursache: Larastan löste keinen einzigen Cast auf, weil alle Models ihre Casts über die casts()-METHODE deklarieren (Laravel-11-Stil). Laravel führt die erst zur Laufzeit mit $casts zusammen, Larastan instanziiert Models aber ohne Konstruktor — $pressRelease->status galt deshalb als string und jeder Enum-Vergleich als "immer falsch". Dafür gibt es den Schalter parseModelCastsMethod, dessen Larastan-Default false ist. Zweite Sammelursache waren fehlende Generics an den Relations-Methoden: Ohne sie landete phpstan auf Illuminate\Database\Eloquent\Model und meldete jede Modell-Eigenschaft und jeden eigenen Scope als unbekannt. Alle 102 Relationen tragen jetzt ihr @return <Related, $this>; das verbessert nebenbei die Vervollständigung in der IDE. Zusammen fällt die Zahl von 390 auf 169. Die neu erzeugte Baseline umfasst 166 Einträge, überwiegend Closure-Parameter, die als blankes Builder typisiert sind, sowie zwei nachgeprüfte Falschmeldungs-Gruppen: die Concerns unter app/Livewire werden ausschließlich aus Volt-Komponenten eingebunden, die phpstan nicht analysiert, und Cashier typisiert findBillable() gegen den TRAIT Billable, was phpstan nicht mit User vereinbaren kann — die Laufzeitprüfung dort ist richtig und bleibt. Was die Analyse an echten toten Zweigen zutage gefördert hat, ist mit entfernt: eine doppelte Null-Prüfung im Nachweis-PDF, Prüfungen gegen NOT-NULL-Spalten in der Rechnungsstellung und im Vorschau-Controller, eine immer wahre Bedingung im Logo-Pfad und in RetireAgencyCompanyType, ein überflüssiges ?? [] auf einer Symfony-Methode, die immer ein Array liefert, sowie der unerreichbare null-Arm in publicPortalLinks(). MigrateLegacyMedia deklariert seine Medientypen jetzt als Literal-Union, womit das match ohne default nachweislich vollständig ist statt nur faktisch. Damit das nicht wieder abdriftet — die Baseline stand seit Mai unverändert — läuft die Analyse ab jetzt als eigener Schritt in der Test-Pipeline. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
24 lines
861 B
Text
24 lines
861 B
Text
includes:
|
|
- vendor/larastan/larastan/extension.neon
|
|
- vendor/nesbot/carbon/extension.neon
|
|
- phpstan-baseline.neon
|
|
|
|
parameters:
|
|
# ZWINGEND für dieses Projekt: Alle Models deklarieren ihre Casts über die
|
|
# `casts()`-METHODE (Laravel-11-Stil). Laravel führt sie erst zur Laufzeit
|
|
# mit `$this->casts` zusammen, Larastan instanziiert Models aber ohne
|
|
# Konstruktor und sieht sie deshalb nicht — ohne diesen Schalter gilt
|
|
# `$pressRelease->status` als `string` statt als `PressReleaseStatus` und
|
|
# jeder Enum-Vergleich wird als „immer falsch" gemeldet. Larastans Default
|
|
# ist `false`.
|
|
parseModelCastsMethod: true
|
|
|
|
level: 4
|
|
paths:
|
|
- app
|
|
- bootstrap/app.php
|
|
- config
|
|
- database/factories
|
|
- database/seeders
|
|
- routes
|
|
tmpDir: storage/framework/cache/phpstan
|