From dfa81b040550ab9a628a1a00efc5ce95a5992df1 Mon Sep 17 00:00:00 2001
From: Brad Groux <bradgroux@Brads-Mac-mini.local>
Date: Sun, 1 Feb 2026 15:25:19 -0600
Subject: [PATCH] fix: add Move confirm button to bulk actions (two-step flow)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Previously the 'Move to...' dropdown fired immediately on selection with
no confirm step. Now it's a two-step flow: pick target status from dropdown,
then click 'Move' button to confirm — consistent with Archive and Delete.

Fixes task_20260201_NqmOuf
---
 web/src/components/board/BulkActionsBar.tsx | 32 +++++++++++++++++----
 1 file changed, 26 insertions(+), 6 deletions(-)

diff --git a/web/src/components/board/BulkActionsBar.tsx b/web/src/components/board/BulkActionsBar.tsx
index 8e0a05d3..90929944 100644
--- a/web/src/components/board/BulkActionsBar.tsx
+++ b/web/src/components/board/BulkActionsBar.tsx
@@ -64,6 +64,7 @@ export function BulkActionsBar({ tasks }: BulkActionsBarProps) {
 
   const [showDeleteConfirm, setShowDeleteConfirm] = useState(false);
   const [isProcessing, setIsProcessing] = useState(false);
+  const [moveTarget, setMoveTarget] = useState<TaskStatus | null>(null);
 
   // Group task IDs by status
   const taskIdsByStatus = useMemo(() => {
@@ -109,13 +110,17 @@ export function BulkActionsBar({ tasks }: BulkActionsBarProps) {
     return someSelected && !allSelectedInGroup;
   };
 
-  const handleMoveToStatus = async (status: TaskStatus) => {
+  const handleMoveToStatus = async () => {
+    if (!moveTarget) return;
     setIsProcessing(true);
     try {
       await Promise.all(
-        Array.from(selectedIds).map((id) => updateTask.mutateAsync({ id, input: { status } }))
+        Array.from(selectedIds).map((id) =>
+          updateTask.mutateAsync({ id, input: { status: moveTarget } })
+        )
       );
       clearSelection();
+      setMoveTarget(null);
     } finally {
       setIsProcessing(false);
     }
@@ -210,16 +215,20 @@ export function BulkActionsBar({ tasks }: BulkActionsBarProps) {
 
         {selectedCount > 0 && (
           <div className="flex items-center gap-2">
-            {/* Move to status */}
+            {/* Move to status — two-step: pick target → confirm */}
             <Select
-              value=""
-              onValueChange={(value) => handleMoveToStatus(value as TaskStatus)}
+              value={moveTarget ?? ''}
+              onValueChange={(value) => setMoveTarget(value as TaskStatus)}
               disabled={isProcessing}
             >
               <SelectTrigger className="w-[140px]">
                 <div className="flex items-center gap-1">
                   <ArrowRight className="h-4 w-4" />
-                  <span>Move to...</span>
+                  <span>
+                    {moveTarget
+                      ? (STATUS_BUTTONS.find((s) => s.id === moveTarget)?.label ?? 'Move to...')
+                      : 'Move to...'}
+                  </span>
                 </div>
               </SelectTrigger>
               <SelectContent>
@@ -231,6 +240,17 @@ export function BulkActionsBar({ tasks }: BulkActionsBarProps) {
               </SelectContent>
             </Select>
 
+            {moveTarget && (
+              <Button
+                variant="default"
+                size="sm"
+                onClick={handleMoveToStatus}
+                disabled={isProcessing}
+              >
+                {isProcessing ? 'Moving...' : 'Move'}
+              </Button>
+            )}
+
             {/* Archive */}
             <Button
               variant="outline"