fixed scope

app/Models/User.php

@@ -4,13 +4,14 @@ namespace App\Models;
 
 // use Illuminate\Contracts\Auth\MustVerifyEmail;
 use App\Actions\Permissions\PermissionService;
+use App\Enums\Permission;
 use Database\Factories\UserFactory;
 use Illuminate\Database\Eloquent\Attributes\Fillable;
 use Illuminate\Database\Eloquent\Attributes\Hidden;
+use Illuminate\Database\Eloquent\Builder;
 use Illuminate\Database\Eloquent\Factories\HasFactory;
 use Illuminate\Database\Eloquent\Relations\BelongsToMany;
 use Illuminate\Database\Eloquent\Relations\HasMany;
-use Illuminate\Database\Eloquent\Builder;
 use Illuminate\Database\Eloquent\Relations\MorphMany;
 use Illuminate\Foundation\Auth\User as Authenticatable;
 use Illuminate\Notifications\Notifiable;
@@ -76,12 +77,17 @@ class User extends Authenticatable
     {
         $permissionService = app(PermissionService::class);
 
-        return $query->where(function (Builder $builder) use ($permissionService, $project): void {
+        $allowedUserIds = User::query()
-            foreach (User::query()->select('id')->cursor() as $candidate) {
+            ->select('id')
-                if ($permissionService->allows($candidate, \App\Enums\Permission::ProjectRead, $project)) {
+            ->cursor()
-                    $builder->orWhereKey($candidate->id);
+            ->filter(fn (User $candidate) => $permissionService->allows(
-                }
+                $candidate,
-            }
+                Permission::ProjectRead,
-        });
+                $project
+            ))
+            ->pluck('id')
+            ->all();
+
+        return $query->whereKey($allowedUserIds);
     }
 }