Feature/concurrent card dragging

database/factories/TaskFactory.php

@@ -3,7 +3,7 @@
 namespace Relaticle\Flowforge\Database\Factories;
 
 use Illuminate\Database\Eloquent\Factories\Factory;
-use Relaticle\Flowforge\Services\Rank;
+use Relaticle\Flowforge\Services\DecimalPosition;
 use Relaticle\Flowforge\Tests\Fixtures\Task;
 
 class TaskFactory extends Factory
@@ -176,15 +176,15 @@ private function generateResearchTitle(): string
 
     private function generatePosition(): string
     {
-        // Generate positions in a realistic range
-        static $baseRank = null;
+        // Generate positions in a realistic range using decimal positions
+        static $position = null;
 
-        if ($baseRank === null) {
-            $baseRank = Rank::forEmptySequence();
+        if ($position === null) {
+            $position = DecimalPosition::forEmptyColumn();
         } else {
-            $baseRank = Rank::after($baseRank);
+            $position = DecimalPosition::after($position);
         }
 
-        return $baseRank->get();
+        return $position;
     }
 }

database/migrations/add_unique_position_constraint.php.stub

@@ -0,0 +1,35 @@
+<?php
+
+use Illuminate\Database\Migrations\Migration;
+use Illuminate\Database\Schema\Blueprint;
+use Illuminate\Support\Facades\Schema;
+
+return new class extends Migration
+{
+    /**
+     * Run the migrations.
+     *
+     * IMPORTANT: Before running this migration, ensure you have no duplicate
+     * positions in your table by running:
+     *
+     *   php artisan flowforge:repair-positions YourModel --column=status
+     */
+    public function up(): void
+    {
+        Schema::table('your_table_name', function (Blueprint $table) {
+            // Add unique constraint on (status_column, position_column) combination
+            // This prevents duplicate positions within the same column/status
+            $table->unique(['status_column', 'position_column'], 'unique_position_per_column');
+        });
+    }
+
+    /**
+     * Reverse the migrations.
+     */
+    public function down(): void
+    {
+        Schema::table('your_table_name', function (Blueprint $table) {
+            $table->dropUnique('unique_position_per_column');
+        });
+    }
+};

phpstan.neon.dist

@@ -2,7 +2,7 @@ includes:
     - phpstan-baseline.neon
 
 parameters:
-    level: 4
+    level: 5
     paths:
         - src
         - config

resources/js/flowforge.js

@@ -25,7 +25,7 @@ export default function flowforge({state}) {
             const afterCardId = cardIndex > 0 ? newOrder[cardIndex - 1] : null;
             const beforeCardId = cardIndex < newOrder.length - 1 ? newOrder[cardIndex + 1] : null;
 
-            this.$wire.moveCard(cardId, targetColumn, beforeCardId, afterCardId)
+            this.$wire.moveCard(cardId, targetColumn, afterCardId, beforeCardId)
                 .then(() => this.setCardState(cardElement, false))
                 .catch(() => this.setCardState(cardElement, false));
         },