From 00bfac62eb7a70244ed34a07cb4cc3117da00ba0 Mon Sep 17 00:00:00 2001
From: Louis Knight-Webb <louis@bloop.ai>
Date: Tue, 24 Jun 2025 23:52:18 +0100
Subject: [PATCH] Task attempt a858e1e1-9dde-47a2-82e8-15daa836b9c5 - Final
 changes

---
 .../src/components/tasks/TaskDetailsPanel.tsx | 114 +++++++++---------
 1 file changed, 58 insertions(+), 56 deletions(-)

diff --git a/frontend/src/components/tasks/TaskDetailsPanel.tsx b/frontend/src/components/tasks/TaskDetailsPanel.tsx
index c3845f91..dac121fd 100644
--- a/frontend/src/components/tasks/TaskDetailsPanel.tsx
+++ b/frontend/src/components/tasks/TaskDetailsPanel.tsx
@@ -889,90 +889,92 @@ export function TaskDetailsPanel({
                             </DropdownMenuContent>
                           </DropdownMenu>
                         )}
-                        <div className="flex">
+                        {(isAttemptRunning || isStopping) ? (
                           <TooltipProvider>
                             <Tooltip>
                               <TooltipTrigger asChild>
                                 <Button
                                   variant="outline"
                                   size="sm"
-                                  onClick={() => createNewAttempt()}
-                                  className="rounded-r-none border-r-0"
+                                  onClick={stopAllExecutions}
+                                  disabled={isStopping}
+                                  className="text-red-600 hover:text-red-700 hover:bg-red-50 disabled:opacity-50"
                                 >
-                                  {selectedAttempt ? "Retry" : "Start"}
+                                  <StopCircle className="h-4 w-4 mr-2" />
+                                  {isStopping ? "Stopping..." : "Stop Attempt"}
                                 </Button>
                               </TooltipTrigger>
                               <TooltipContent>
-                                <p>{selectedAttempt ? "Retry task with current executor" : "Start task with current executor"}</p>
+                                <p>{isStopping ? "Stopping execution..." : "Stop execution"}</p>
                               </TooltipContent>
                             </Tooltip>
                           </TooltipProvider>
-                          <DropdownMenu>
+                        ) : (
+                          <div className="flex">
                             <TooltipProvider>
                               <Tooltip>
                                 <TooltipTrigger asChild>
-                                  <DropdownMenuTrigger asChild>
-                                    <Button
-                                      variant="outline"
-                                      size="sm"
-                                      className="rounded-l-none px-2"
-                                    >
-                                      <Settings2 className="h-4 w-4" />
-                                    </Button>
-                                  </DropdownMenuTrigger>
+                                  <Button
+                                    variant="outline"
+                                    size="sm"
+                                    onClick={() => createNewAttempt()}
+                                    className="rounded-r-none border-r-0"
+                                  >
+                                    {selectedAttempt ? "Retry" : "Start"}
+                                  </Button>
                                 </TooltipTrigger>
                                 <TooltipContent>
-                                  <p>Choose executor</p>
+                                  <p>{selectedAttempt ? "Retry task with current executor" : "Start task with current executor"}</p>
                                 </TooltipContent>
                               </Tooltip>
                             </TooltipProvider>
-                            <DropdownMenuContent align="end">
-                              {availableExecutors.map((executor) => (
-                                <DropdownMenuItem
-                                  key={executor.id}
-                                  onClick={() => setSelectedExecutor(executor.id)}
-                                  className={
-                                    selectedExecutor === executor.id
-                                      ? "bg-accent"
-                                      : ""
-                                  }
-                                >
-                                  {executor.name}
-                                  {config?.executor.type === executor.id &&
-                                    " (Default)"}
-                                </DropdownMenuItem>
-                              ))}
-                            </DropdownMenuContent>
-                          </DropdownMenu>
-                        </div>
+                            <DropdownMenu>
+                              <TooltipProvider>
+                                <Tooltip>
+                                  <TooltipTrigger asChild>
+                                    <DropdownMenuTrigger asChild>
+                                      <Button
+                                        variant="outline"
+                                        size="sm"
+                                        className="rounded-l-none px-2"
+                                      >
+                                        <Settings2 className="h-4 w-4" />
+                                      </Button>
+                                    </DropdownMenuTrigger>
+                                  </TooltipTrigger>
+                                  <TooltipContent>
+                                    <p>Choose executor</p>
+                                  </TooltipContent>
+                                </Tooltip>
+                              </TooltipProvider>
+                              <DropdownMenuContent align="end">
+                                {availableExecutors.map((executor) => (
+                                  <DropdownMenuItem
+                                    key={executor.id}
+                                    onClick={() => setSelectedExecutor(executor.id)}
+                                    className={
+                                      selectedExecutor === executor.id
+                                        ? "bg-accent"
+                                        : ""
+                                    }
+                                  >
+                                    {executor.name}
+                                    {config?.executor.type === executor.id &&
+                                      " (Default)"}
+                                  </DropdownMenuItem>
+                                ))}
+                              </DropdownMenuContent>
+                            </DropdownMenu>
+                          </div>
+                        )}
                       </div>
 
                       {selectedAttempt && (
                         <>
                           <div className="h-4 w-px bg-border" />
                           
-                          {/* Execution Control Group */}
+                          {/* Dev Server Control Group */}
                           <div className="flex items-center gap-1">
-                            {(isAttemptRunning || isStopping) && (
-                              <TooltipProvider>
-                                <Tooltip>
-                                  <TooltipTrigger asChild>
-                                    <Button
-                                      variant="outline"
-                                      size="sm"
-                                      onClick={stopAllExecutions}
-                                      disabled={isStopping}
-                                      className="text-red-600 hover:text-red-700 hover:bg-red-50 disabled:opacity-50"
-                                    >
-                                      <StopCircle className="h-4 w-4" />
-                                    </Button>
-                                  </TooltipTrigger>
-                                  <TooltipContent>
-                                    <p>{isStopping ? "Stopping execution..." : "Stop execution"}</p>
-                                  </TooltipContent>
-                                </Tooltip>
-                              </TooltipProvider>
-                            )}
                             <TooltipProvider>
                               <Tooltip>
                                 <TooltipTrigger asChild>