Update app/Models/UserRecentPage.php

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>

Author: andrasbacsai

app/Models/UserRecentPage.php

@@ -76,45 +76,48 @@ public static function recordVisit(int $userId, int $teamId, string $url, string
 
     public static function togglePin(int $userId, int $teamId, string $url): bool
     {
-        $record = self::where('user_id', $userId)
-            ->where('team_id', $teamId)
-            ->first();
+        return \DB::transaction(function () use ($userId, $teamId, $url) {
+            $record = self::where('user_id', $userId)
+                ->where('team_id', $teamId)
+                ->lockForUpdate()
+                ->first();
+
+            if (! $record) {
+                return false;
+            }
 
-        if (! $record) {
-            return false;
-        }
+            $pages = collect($record->pages);
+            $index = $pages->search(fn ($p) => $p['url'] === $url);
 
-        $pages = collect($record->pages);
-        $index = $pages->search(fn ($p) => $p['url'] === $url);
-
-        if ($index === false) {
-            return false;
-        }
+            if ($index === false) {
+                return false;
+            }
 
-        $page = $pages[$index];
-        $currentlyPinned = $page['pinned'] ?? false;
+            $page = $pages[$index];
+            $currentlyPinned = $page['pinned'] ?? false;
 
-        // Check if we can pin (max 5 pinned)
-        if (! $currentlyPinned) {
-            $pinnedCount = $pages->filter(fn ($p) => ! empty($p['pinned']))->count();
-            if ($pinnedCount >= 5) {
-                return false; // Can't pin more
+            // Check if we can pin (max 5 pinned)
+            if (! $currentlyPinned) {
+                $pinnedCount = $pages->filter(fn ($p) => ! empty($p['pinned']))->count();
+                if ($pinnedCount >= 5) {
+                    return false; // Can't pin more
+                }
             }
-        }
 
-        $page['pinned'] = ! $currentlyPinned;
-        $page['pinned_at'] = $page['pinned'] ? now()->toISOString() : null;
+            $page['pinned'] = ! $currentlyPinned;
+            $page['pinned_at'] = $page['pinned'] ? now()->toISOString() : null;
 
-        $pages[$index] = $page;
+            $pages[$index] = $page;
 
-        // Re-sort: pinned first (by pinned_at desc), then unpinned (by visited_at desc)
-        $pinned = $pages->filter(fn ($p) => ! empty($p['pinned']))->sortByDesc('pinned_at');
-        $unpinned = $pages->reject(fn ($p) => ! empty($p['pinned']))->sortByDesc('visited_at');
+            // Re-sort: pinned first (by pinned_at desc), then unpinned (by visited_at desc)
+            $pinned = $pages->filter(fn ($p) => ! empty($p['pinned']))->sortByDesc('pinned_at');
+            $unpinned = $pages->reject(fn ($p) => ! empty($p['pinned']))->sortByDesc('visited_at');
 
-        $record->pages = $pinned->merge($unpinned)->values()->all();
-        $record->save();
+            $record->pages = $pinned->merge($unpinned)->values()->all();
+            $record->save();
 
-        return $page['pinned'];
+            return $page['pinned'];
+        });
     }
 
     public static function getRecent(int $userId, int $teamId): array