Phpstan fixes

Author: ksassnowski

phpstan-baseline.neon

@@ -10,11 +10,6 @@ parameters:
 			count: 1
 			path: src/AbstractWizard.php
 
-		-
-			message: "#^Method Arcanist\\\\AbstractWizard\\:\\:summary\\(\\) should return array\\{id\\: int\\|string, slug\\: string, title\\: string, steps\\: array\\<int, array\\{slug\\: string, isComplete\\: bool, title\\: string, active\\: bool, url\\: string\\|null\\}\\>\\} but returns array\\{id\\: int\\|string\\|null, slug\\: string, title\\: string, steps\\: array\\<int, array\\{slug\\: string, isComplete\\: bool, title\\: string, active\\: bool, url\\: string\\|null\\}\\>\\}\\.$#"
-			count: 1
-			path: src/AbstractWizard.php
-
 		-
 			message: "#^Parameter \\#1 \\$callback of method Illuminate\\\\Support\\\\Collection\\<int,mixed\\>\\:\\:filter\\(\\) expects \\(callable\\(mixed, int\\)\\: bool\\)\\|null, Closure\\(Arcanist\\\\WizardStep\\)\\: bool given\\.$#"
 			count: 1
@@ -56,9 +51,14 @@ parameters:
 			path: src/AbstractWizard.php
 
 		-
-			message: "#^Property Arcanist\\\\AbstractWizard\\:\\:\\$id \\(int\\|string\\|null\\) does not accept mixed\\.$#"
+			message: "#^Cannot access offset 'config' on Illuminate\\\\Contracts\\\\Foundation\\\\Application\\.$#"
 			count: 1
-			path: src/AbstractWizard.php
+			path: src/ArcanistServiceProvider.php
+
+		-
+			message: "#^Cannot access offset 'events' on Illuminate\\\\Contracts\\\\Foundation\\\\Application\\.$#"
+			count: 1
+			path: src/ArcanistServiceProvider.php
 
 		-
 			message: "#^Parameter \\#1 \\$implementation of method Illuminate\\\\Contracts\\\\Container\\\\ContextualBindingBuilder\\:\\:give\\(\\) expects array\\|Closure\\|string, mixed given\\.$#"
@@ -76,6 +76,6 @@ parameters:
 			path: src/ArcanistServiceProvider.php
 
 		-
-			message: "#^Cannot access offset '\\:\\:step\\-with…' on mixed\\.$#"
-			count: 2
-			path: tests/InvalidateDependentFieldsTest.php
+			message: "#^Parameter \\#1 \\$id of method Arcanist\\\\AbstractWizard\\:\\:setId\\(\\) expects int\\|string\\|null, Ramsey\\\\Uuid\\\\UuidInterface given\\.$#"
+			count: 1
+			path: src/Repository/CacheWizardRepository.php

phpstan.neon

@@ -7,14 +7,9 @@ parameters:
 
     paths:
         - src
-        - tests
 
     level: 9
 
-    ignoreErrors:
-        - '/Parameter #1 \$response of class Illuminate\\Testing\\TestResponse constructor expects*/'
-        - /Cannot access offset '\w+' on Illuminate\\Contracts\\Foundation\\Application/
-
     excludePaths:
         - tests/__snapshots__/*
 

src/AbstractWizard.php

@@ -148,6 +148,9 @@ public function getId(): mixed
         return $this->id;
     }
 
+    /**
+     * @param int|string|null $id
+     */
     public function setId(mixed $id): void
     {
         $this->id = $id;
@@ -311,7 +314,7 @@ public function exists(): bool
      * Returns a summary about the current wizard and its steps.
      *
      * @return array{
-     *     id: string|int,
+     *     id: string|int|null,
      *     slug: string,
      *     title: string,
      *     steps: array<int, SummaryStep>,

src/Repository/Wizard.php

@@ -13,18 +13,27 @@
 
 namespace Arcanist\Repository;
 
+use Arcanist\AbstractWizard;
 use Illuminate\Database\Eloquent\Factories\HasFactory;
 use Illuminate\Database\Eloquent\Model;
 
 /**
- * @property string $class
- * @property array  $data
+ * @property string|class-string<AbstractWizard> $class
+ * @property array<string, mixed>  $data
  * @property int    $id
  */
 class Wizard extends Model
 {
     use HasFactory;
+
+    /**
+     * @var array<int, string>
+     */
     protected $guarded = [];
+
+    /**
+     * @var array<string, string>
+     */
     protected $casts = [
         'data' => 'array',
     ];