Update dependency vimeo/psalm to v7.0.0-beta15

app/Console/Commands/Ping.php

@@ -31,6 +31,8 @@ class Ping extends Command
 
     /**
      * Execute the console command.
+     *
+     * @psalm-return 0|1
      */
     public function handle(): int
     {

app/Exceptions/QuickBooksFault.php

@@ -11,6 +11,8 @@ class QuickBooksFault extends Exception
 {
     /**
      * Construct a new instance.
+     *
+     * @psalm-mutation-free
      */
     public function __construct(IPPFault $fault)
     {

app/HorizonHealthCheck.php

@@ -22,6 +22,11 @@ class HorizonHealthCheck extends HealthCheck
      */
     protected string $name = 'horizon';
 
+    /**
+     * Construct a new instance of this health check.
+     *
+     * @psalm-mutation-free
+     */
     public function __construct(private readonly MasterSupervisorRepository $supervisorRepository)
     {
     }

app/Http/Controllers/EngagePurchaseRequestController.php

@@ -109,6 +109,11 @@ public function update(EngagePurchaseRequest $purchaseRequest, UpdateEngagePurch
         return response()->json($purchaseRequest);
     }
 
+    /**
+     * Simplify the finance stage name.
+     *
+     * @psalm-pure
+     */
     private static function cleanFinanceStageName(string $input): string
     {
         if (str_contains($input, ':')) {
@@ -137,7 +142,7 @@ private static function getUserByEmailAddress(string $email): User
 
         $result = Sentry::wrapWithChildSpan(
             'ldap.get_user_by_username',
-            static fn (): array => Container::getDefaultConnection()
+            static fn (): array|\LdapRecord\Query\Collection => Container::getDefaultConnection()
                 ->query()
                 ->where('uid', '=', $parts[0])
                 ->select('sn', 'givenName', 'primaryUid', 'mail')

app/Http/Controllers/WorkerController.php

@@ -45,7 +45,7 @@ public function __invoke(UpsertWorker $request): JsonResponse
                 static function () use ($email): ?string {
                     $result = Sentry::wrapWithChildSpan(
                         'ldap.get_username_by_email',
-                        static fn (): array => Container::getDefaultConnection()
+                        static fn (): array|\LdapRecord\Query\Collection => Container::getDefaultConnection()
                             ->query()
                             ->where('mail', '=', $email)
                             ->select('primaryUid')

app/Http/Requests/UpdateEngagePurchaseRequest.php

@@ -10,8 +10,10 @@ class UpdateEngagePurchaseRequest extends FormRequest
 {
     /**
      * Determine if the user is authorized to make this request.
+     *
+     * @psalm-pure
      */
-    public function authorize(): bool
+    public function authorize(): true
     {
         return true;
     }
@@ -20,6 +22,8 @@ public function authorize(): bool
      * Get the validation rules that apply to the request.
      *
      * @return array<string, array<string>>
+     *
+     * @psalm-pure
      */
     public function rules(): array
     {

app/Http/Requests/UpdateExpenseReport.php

@@ -10,8 +10,10 @@ class UpdateExpenseReport extends FormRequest
 {
     /**
      * Determine if the user is authorized to make this request.
+     *
+     * @psalm-pure
      */
-    public function authorize(): bool
+    public function authorize(): true
     {
         return true;
     }
@@ -20,6 +22,8 @@ public function authorize(): bool
      * Get the validation rules that apply to the request.
      *
      * @return array<string, array<string>>
+     *
+     * @psalm-pure
      */
     public function rules(): array
     {

app/Http/Requests/UpdateExpenseReportLine.php

@@ -10,8 +10,10 @@ class UpdateExpenseReportLine extends FormRequest
 {
     /**
      * Determine if the user is authorized to make this request.
+     *
+     * @psalm-pure
      */
-    public function authorize(): bool
+    public function authorize(): true
     {
         return true;
     }
@@ -20,6 +22,8 @@ public function authorize(): bool
      * Get the validation rules that apply to the request.
      *
      * @return array<string, array<string>>
+     *
+     * @psalm-pure
      */
     public function rules(): array
     {

app/Http/Requests/UploadEngageAttachment.php

@@ -10,8 +10,10 @@ class UploadEngageAttachment extends FormRequest
 {
     /**
      * Determine if the user is authorized to make this request.
+     *
+     * @psalm-pure
      */
-    public function authorize(): bool
+    public function authorize(): true
     {
         return true;
     }
@@ -20,6 +22,8 @@ public function authorize(): bool
      * Get the validation rules that apply to the request.
      *
      * @return array<string, array<string>>
+     *
+     * @psalm-pure
      */
     public function rules(): array
     {

app/Http/Requests/UploadWorkdayAttachment.php

@@ -10,8 +10,10 @@ class UploadWorkdayAttachment extends FormRequest
 {
     /**
      * Determine if the user is authorized to make this request.
+     *
+     * @psalm-pure
      */
-    public function authorize(): bool
+    public function authorize(): true
     {
         return true;
     }
@@ -20,6 +22,8 @@ public function authorize(): bool
      * Get the validation rules that apply to the request.
      *
      * @return array<string, array<string>>
+     *
+     * @psalm-pure
      */
     public function rules(): array
     {

app/Http/Requests/UpsertEngagePurchaseRequests.php

@@ -10,8 +10,10 @@ class UpsertEngagePurchaseRequests extends FormRequest
 {
     /**
      * Determine if the user is authorized to make this request.
+     *
+     * @psalm-pure
      */
-    public function authorize(): bool
+    public function authorize(): true
     {
         return true;
     }
@@ -20,6 +22,8 @@ public function authorize(): bool
      * Get the validation rules that apply to the request.
      *
      * @return array<string, array<string>>
+     *
+     * @psalm-pure
      */
     public function rules(): array
     {

app/Http/Requests/UpsertExpenseReports.php

@@ -10,8 +10,10 @@ class UpsertExpenseReports extends FormRequest
 {
     /**
      * Determine if the user is authorized to make this request.
+     *
+     * @psalm-pure
      */
-    public function authorize(): bool
+    public function authorize(): true
     {
         return true;
     }
@@ -20,6 +22,8 @@ public function authorize(): bool
      * Get the validation rules that apply to the request.
      *
      * @return array<string, array<string>>
+     *
+     * @psalm-pure
      */
     public function rules(): array
     {

app/Http/Requests/UpsertExternalCommitteeMember.php

@@ -16,8 +16,10 @@ class UpsertExternalCommitteeMember extends FormRequest
 {
     /**
      * Determine if the user is authorized to make this request.
+     *
+     * @psalm-pure
      */
-    public function authorize(): bool
+    public function authorize(): true
     {
         return true;
     }
@@ -26,6 +28,8 @@ public function authorize(): bool
      * Get the validation rules that apply to the request.
      *
      * @return array<string, array<string>>
+     *
+     * @psalm-pure
      */
     public function rules(): array
     {

app/Http/Requests/UpsertWorker.php

@@ -10,8 +10,10 @@ class UpsertWorker extends FormRequest
 {
     /**
      * Determine if the user is authorized to make this request.
+     *
+     * @psalm-pure
      */
-    public function authorize(): bool
+    public function authorize(): true
     {
         return true;
     }
@@ -20,6 +22,8 @@ public function authorize(): bool
      * Get the validation rules that apply to the request.
      *
      * @return array<string, array<string>>
+     *
+     * @psalm-pure
      */
     public function rules(): array
     {

app/Jobs/AttachAccessWorkdayPermission.php

@@ -24,6 +24,8 @@ class AttachAccessWorkdayPermission implements ShouldBeUnique, ShouldQueue
 
     /**
      * Create a new job instance.
+     *
+     * @psalm-mutation-free
      */
     public function __construct(private readonly User $user)
     {
@@ -52,6 +54,8 @@ public function handle(): void
 
     /**
      * The unique ID of the job.
+     *
+     * @psalm-mutation-free
      */
     public function uniqueId(): string
     {

app/Jobs/CancelExpensePayments.php

@@ -19,6 +19,8 @@ class CancelExpensePayments implements ShouldBeUnique, ShouldQueue
 
     /**
      * Execute the job.
+     *
+     * @psalm-mutation-free
      */
     public function handle(): void
     {

app/Jobs/GenerateThumbnail.php

@@ -22,6 +22,8 @@ class GenerateThumbnail implements ShouldBeUnique, ShouldQueue
 
     /**
      * Create a new job instance.
+     *
+     * @psalm-mutation-free
      */
     public function __construct(private readonly string $original_file_path)
     {

app/Jobs/MatchAttachment.php

@@ -32,6 +32,8 @@ class MatchAttachment implements ShouldBeUnique, ShouldQueue
 
     /**
      * Create a new job instance.
+     *
+     * @psalm-mutation-free
      */
     public function __construct(private readonly Attachment $attachment)
     {
@@ -73,6 +75,8 @@ public function handle(): void
 
     /**
      * The unique ID of the job.
+     *
+     * @psalm-mutation-free
      */
     public function uniqueId(): string
     {

app/Jobs/MatchBankTransaction.php

@@ -29,6 +29,8 @@ class MatchBankTransaction implements ShouldBeUnique, ShouldQueue
 
     /**
      * Create a new job instance.
+     *
+     * @psalm-mutation-free
      */
     public function __construct(private readonly BankTransaction $bankTransaction)
     {
@@ -78,6 +80,8 @@ public function handle(): void
 
     /**
      * The unique ID of the job.
+     *
+     * @psalm-mutation-free
      */
     public function uniqueId(): string
     {

app/Jobs/MatchExpenseReport.php

@@ -35,6 +35,8 @@ class MatchExpenseReport implements ShouldBeUnique, ShouldQueue
 
     /**
      * Create a new job instance.
+     *
+     * @psalm-mutation-free
      */
     public function __construct(private readonly ExpenseReport $expenseReport)
     {
@@ -106,6 +108,8 @@ static function (Attachment $attachment, int $key) use (&$engage_request_numbers
 
     /**
      * The unique ID of the job.
+     *
+     * @psalm-mutation-free
      */
     public function uniqueId(): string
     {

app/Jobs/ProcessSensibleOutput.php

@@ -35,6 +35,8 @@ class ProcessSensibleOutput implements ShouldQueue
 
     /**
      * Create a new job instance.
+     *
+     * @psalm-mutation-free
      */
     public function __construct(private readonly EmailRequest $emailRequest)
     {
@@ -79,6 +81,8 @@ public function handle(): void
      * Get a field value from Sensible or add a validation error to the array.
      *
      * @phan-suppress PhanTypeArraySuspiciousNullable
+     *
+     * @psalm-external-mutation-free
      */
     private function getValueOrAddValidationError(string $field_name): string|float|int|null
     {

app/Jobs/SubmitEmailRequestToSensible.php

@@ -24,6 +24,8 @@ class SubmitEmailRequestToSensible implements ShouldBeUnique, ShouldQueue
 
     /**
      * Create a new job instance.
+     *
+     * @psalm-mutation-free
      */
     public function __construct(private readonly EmailRequest $emailRequest)
     {
@@ -73,6 +75,8 @@ public function handle(): void
 
     /**
      * The unique ID of the job.
+     *
+     * @psalm-mutation-free
      */
     public function uniqueId(): string
     {

app/Mail/EmailRequestProcessed.php

@@ -25,6 +25,8 @@ class EmailRequestProcessed extends Mailable implements ShouldQueue
      * Create a new message instance.
      *
      * @param  array<string>  $validation_errors
+     *
+     * @psalm-mutation-free
      */
     public function __construct(public readonly EmailRequest $email, public readonly array $validation_errors)
     {
@@ -33,7 +35,7 @@ public function __construct(public readonly EmailRequest $email, public readonly
     /**
      * Build the message.
      */
-    public function build(): self
+    public function build(): static
     {
         $this->to(config('services.treasurer_email_address'), config('services.treasurer_name'));