feat: add sentry error logging

Signed-off-by: ItsNeil17 <neil@hytalemodding.dev>

Author: ItsNeil17

app/Http/Controllers/PageController.php

@@ -236,9 +236,11 @@ public function edit(Mod $mod, Page $page)
         $this->ensureManualPageEditingAllowed($mod);
 
         $page->load(['parent']);
+        $descendantIds = $this->getDescendantIds($page);
 
         $potentialParents = $mod->pages()
             ->where('id', '!=', $page->id)
+            ->when($descendantIds !== [], fn ($query) => $query->whereNotIn('id', $descendantIds))
             ->orderBy('title')
             ->get();
 
@@ -282,6 +284,10 @@ public function update(Request $request, Mod $mod, Page $page)
             if ($parent->mod_id !== $mod->id || $parent->id === $page->id) {
                 abort(422, 'Invalid parent page.');
             }
+
+            if (in_array($parent->id, $this->getDescendantIds($page), true)) {
+                abort(422, 'Invalid parent page.');
+            }
         }
 
         if (($validated['kind'] ?? $page->kind) === Page::KIND_CATEGORY) {
@@ -406,6 +412,34 @@ private function ensureManualPageEditingAllowed(Mod $mod): void
         }
     }
 
+    /**
+     * Get all descendant page IDs for a page in the same mod.
+     *
+     * @return array<int, string>
+     */
+    private function getDescendantIds(Page $page): array
+    {
+        $descendantIds = [];
+        $currentParentIds = [$page->id];
+
+        while ($currentParentIds !== []) {
+            $childIds = Page::query()
+                ->where('mod_id', $page->mod_id)
+                ->whereIn('parent_id', $currentParentIds)
+                ->pluck('id')
+                ->all();
+
+            if ($childIds === []) {
+                break;
+            }
+
+            $descendantIds = array_merge($descendantIds, $childIds);
+            $currentParentIds = $childIds;
+        }
+
+        return $descendantIds;
+    }
+
     /**
      * Search pages within a mod.
      */

bootstrap/app.php

@@ -10,6 +10,7 @@
 use Illuminate\Foundation\Configuration\Exceptions;
 use Illuminate\Foundation\Configuration\Middleware;
 use Illuminate\Http\Middleware\AddLinkHeadersForPreloadedAssets;
+use Sentry\Laravel\Integration;
 
 return Application::configure(basePath: dirname(__DIR__))
     ->withRouting(
@@ -44,5 +45,5 @@
         ]);
     })
     ->withExceptions(function (Exceptions $exceptions): void {
-        //
+        Integration::handles($exceptions);
     })->create();

composer.json

@@ -14,7 +14,8 @@
     "laravel/fortify": "^1.30",
     "laravel/framework": "^12.0",
     "laravel/tinker": "^2.10.1",
-    "laravel/wayfinder": "^0.1.9"
+    "laravel/wayfinder": "^0.1.9",
+    "sentry/sentry-laravel": "^4.22"
   },
   "require-dev": {
     "fakerphp/faker": "^1.23",