From 2fa31ff74ca46a6413ac0a2ba837d4c52bf1af54 Mon Sep 17 00:00:00 2001
From: myrmidex <myrmidex@myrmidex.net>
Date: Thu, 19 Mar 2026 01:26:58 +0100
Subject: [PATCH] Fix priority shift to update in reverse order for SQLite
 compatibility

---
 app/Actions/CreateBucketAction.php | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/app/Actions/CreateBucketAction.php b/app/Actions/CreateBucketAction.php
index 8a09cf7..18ff266 100644
--- a/app/Actions/CreateBucketAction.php
+++ b/app/Actions/CreateBucketAction.php
@@ -39,10 +39,14 @@ public function execute(
                 // Check if priority already exists and shift others if needed
                 $existingBucket = $scenario->buckets()->where('priority', $priority)->first();
                 if ($existingBucket) {
-                    // Shift priorities to make room
+                    // Shift priorities in reverse order to avoid unique constraint violations
+                    // (SQLite checks constraints per-row during bulk updates)
                     $scenario->buckets()
                         ->where('priority', '>=', $priority)
-                        ->increment('priority');
+                        ->orderByDesc('priority')
+                        ->each(function ($bucket) {
+                            $bucket->increment('priority');
+                        });
                 }
             }