refactor: mark classes as final to prevent inheritance

app/Actions/Posts/CreatePostAction.php

@@ -6,7 +6,7 @@
 
 use App\Models\Post;
 
-class CreatePostAction
+final class CreatePostAction
 {
     /**
      * Create a new post.

app/Actions/Posts/DeletePostAction.php

@@ -6,7 +6,7 @@
 
 use App\Models\Post;
 
-class DeletePostAction
+final class DeletePostAction
 {
     public function execute(Post $post): bool
     {

app/Actions/Posts/TogglePostFeatureAction.php

@@ -7,7 +7,7 @@
 use App\Enums\FeaturedStatus;
 use App\Models\Post;
 
-class TogglePostFeatureAction
+final class TogglePostFeatureAction
 {
     public function execute(Post $post): bool
     {

app/Actions/Posts/UpdatePostAction.php

@@ -6,7 +6,7 @@
 
 use App\Models\Post;
 
-class UpdatePostAction
+final class UpdatePostAction
 {
     /**
      * Update a post.

app/Builders/PostBuilder.php

@@ -12,7 +12,7 @@
  *
  * @extends Builder<Post>
  */
-class PostBuilder extends Builder
+final class PostBuilder extends Builder
 {
     public function search(string $search): static
     {

app/Http/Controllers/Api/PostController.php

@@ -14,7 +14,7 @@
 use Illuminate\Http\Resources\Json\JsonResource;
 use Illuminate\Http\Response;
 
-class PostController extends Controller
+final class PostController extends Controller
 {
     /**
      * Display a listing of the resource.

app/Http/Controllers/FeaturedPostController.php

@@ -9,7 +9,7 @@
 use App\Models\Post;
 use Illuminate\Http\RedirectResponse;
 
-class FeaturedPostController extends Controller
+final class FeaturedPostController extends Controller
 {
     public function __invoke(Post $post, TogglePostFeatureAction $action): RedirectResponse
     {

app/Http/Controllers/LocaleController.php

@@ -6,7 +6,7 @@
 
 use Illuminate\Http\RedirectResponse;
 
-class LocaleController extends Controller
+final class LocaleController extends Controller
 {
     /**
      * Handle the incoming request.

app/Http/Controllers/PostController.php

@@ -19,7 +19,7 @@
 use Illuminate\Http\Request;
 use Illuminate\View\View;
 
-class PostController extends Controller
+final class PostController extends Controller
 {
     /**
      * Display a listing of the resource.

app/Http/Middleware/SetLocale.php

@@ -8,7 +8,7 @@
 use Illuminate\Http\Request;
 use Symfony\Component\HttpFoundation\Response;
 
-class SetLocale
+final class SetLocale
 {
     /**
      * Handle an incoming request.

app/Http/Requests/StorePostRequest.php

@@ -9,7 +9,7 @@
 use Illuminate\Foundation\Http\FormRequest;
 use Override;
 
-class StorePostRequest extends FormRequest
+final class StorePostRequest extends FormRequest
 {
     use HasFileFromUrl;
 

app/Http/Requests/UpdatePostRequest.php

@@ -10,7 +10,7 @@
 use Illuminate\Foundation\Http\FormRequest;
 use Override;
 
-class UpdatePostRequest extends FormRequest
+final class UpdatePostRequest extends FormRequest
 {
     use HasFileFromUrl;
 

app/Models/Category.php

@@ -22,7 +22,7 @@
  *
  * @mixin Model
  */
-class Category extends Model
+final class Category extends Model
 {
     /** @use HasFactory<\Database\Factories\CategoryFactory> */
     use HasFactory;

app/Models/Post.php

@@ -38,7 +38,7 @@
  *
  * @mixin Model
  */
-class Post extends Model
+final class Post extends Model
 {
     /** @use HasFactory<\Database\Factories\PostFactory> */
     use HasFactory;
@@ -88,26 +88,10 @@ public function category(): BelongsTo
         return $this->belongsTo(Category::class);
     }
 
-    /**
-     * The attributes that should be cast.
-     *
-     * @return array<string, string>
-     */
-    #[Override]
-    protected function casts(): array
-    {
-        return [
-            'tags' => 'array',
-            'published_at' => 'datetime',
-            'is_featured' => FeaturedStatus::class,
-            'content' => CleanHtmlInput::class,
-        ];
-    }
-
     /**
      * @return Attribute<string|null, string|null>
      */
-    protected function image(): Attribute
+    public function image(): Attribute
     {
         return Attribute::make(
             get: function (mixed $value): ?string {
@@ -130,4 +114,20 @@ protected function image(): Attribute
             }
         );
     }
+
+    /**
+     * The attributes that should be cast.
+     *
+     * @return array<string, string>
+     */
+    #[Override]
+    protected function casts(): array
+    {
+        return [
+            'tags' => 'array',
+            'published_at' => 'datetime',
+            'is_featured' => FeaturedStatus::class,
+            'content' => CleanHtmlInput::class,
+        ];
+    }
 }

app/Models/User.php

@@ -33,7 +33,7 @@
  *
  * @mixin Model
  */
-class User extends Authenticatable // implements MustVerifyEmail
+final class User extends Authenticatable // implements MustVerifyEmail
 {
     use HasApiTokens;
 

app/Providers/AppServiceProvider.php

@@ -9,7 +9,7 @@
 use Illuminate\Support\ServiceProvider;
 use Override;
 
-class AppServiceProvider extends ServiceProvider
+final class AppServiceProvider extends ServiceProvider
 {
     /**
      * Register any application services.

app/Support/FileUploaderFromUrl.php

@@ -9,7 +9,7 @@
 use Illuminate\Support\Facades\Http;
 use Illuminate\Support\Str;
 
-class FileUploaderFromUrl
+final class FileUploaderFromUrl
 {
     public function __invoke(string $url): ?UploadedFile
     {

app/Support/QueryResolver.php

@@ -7,12 +7,12 @@
 use Illuminate\Support\Collection;
 use Illuminate\Support\Facades\Route;
 
-class QueryResolver
+final readonly class QueryResolver
 {
     /**
      * @var Collection<string, string>
      */
-    private readonly Collection $query;
+    private Collection $query;
 
     public function __construct()
     {
@@ -30,9 +30,9 @@ public function __construct()
     }
 
     /**
-     * @return array<string, string>|null
+     * @return array<string, string>
      */
-    public static function getPreviousQuery(string $routeName): ?array
+    public static function getPreviousQuery(string $routeName): array
     {
         /** @var array<string, string> */
         $query = session()->get($routeName.'.previous.query', []);

database/factories/CategoryFactory.php

@@ -11,7 +11,7 @@
 /**
  * @extends Factory<Category>
  */
-class CategoryFactory extends Factory
+final class CategoryFactory extends Factory
 {
     /**
      * Define the model's default state.

database/factories/PostFactory.php

@@ -12,7 +12,7 @@
 /**
  * @extends Factory<Post>
  */
-class PostFactory extends Factory
+final class PostFactory extends Factory
 {
     /**
      * Define the model's default state.

database/factories/UserFactory.php

@@ -6,17 +6,14 @@
 
 use App\Models\User;
 use Illuminate\Database\Eloquent\Factories\Factory;
-use Illuminate\Support\Facades\Hash;
 use Illuminate\Support\Str;
 use Override;
 
 /**
  * @extends Factory<User>
  */
-class UserFactory extends Factory
+final class UserFactory extends Factory
 {
-    protected static ?string $password = null;
-
     /**
      * Define the model's default state.
      *
@@ -29,7 +26,7 @@ public function definition(): array
             'name' => fake()->name(),
             'email' => fake()->unique()->safeEmail(),
             'email_verified_at' => now(),
-            'password' => static::$password ??= Hash::make('password'),
+            'password' => 'password',
             'remember_token' => Str::random(10),
         ];
     }

database/seeders/DatabaseSeeder.php

@@ -11,7 +11,7 @@
 use Illuminate\Database\Seeder;
 use Illuminate\Support\Facades\Schema;
 
-class DatabaseSeeder extends Seeder
+final class DatabaseSeeder extends Seeder
 {
     /**
      * Seed the application's database.

pint.json

@@ -4,6 +4,7 @@
         "declare_strict_types": true,
         "strict_param": true,
         "strict_comparison": true,
+        "final_class": true,
         "fully_qualified_strict_types": {
             "import_symbols": true,
             "leading_backslash_in_global_namespace": false,

tests/Feature/ArchTest.php

@@ -5,3 +5,4 @@
 arch()->preset()->php();
 arch()->preset()->security();
 arch()->preset()->laravel();
+// arch()->preset()->strict(); // todo