From c35f367b39185990d4948cde6f542e09231e5d63 Mon Sep 17 00:00:00 2001
From: Simon <10131203+gaomeng1900@users.noreply.github.com>
Date: Fri, 20 Mar 2026 16:36:23 +0800
Subject: [PATCH] feat(ext): style adjust; rm `rerunDisabled`

---
 .../src/components/HistoryDetail.tsx          | 28 ++++++----
 .../extension/src/components/HistoryList.tsx  | 52 +++++++++----------
 .../src/entrypoints/sidepanel/App.tsx         |  2 -
 3 files changed, 42 insertions(+), 40 deletions(-)

diff --git a/packages/extension/src/components/HistoryDetail.tsx b/packages/extension/src/components/HistoryDetail.tsx
index 9fe2d59..eff8ff9 100644
--- a/packages/extension/src/components/HistoryDetail.tsx
+++ b/packages/extension/src/components/HistoryDetail.tsx
@@ -1,8 +1,8 @@
-import { ArrowLeft, RotateCcw } from 'lucide-react'
+import { ArrowLeft, RotateCcw, Trash2 } from 'lucide-react'
 import { useEffect, useState } from 'react'
 
 import { Button } from '@/components/ui/button'
-import { type SessionRecord, getSession } from '@/lib/db'
+import { type SessionRecord, deleteSession, getSession } from '@/lib/db'
 
 import { EventCard } from './cards'
 
@@ -10,12 +10,10 @@ export function HistoryDetail({
 	sessionId,
 	onBack,
 	onRerun,
-	rerunDisabled = false,
 }: {
 	sessionId: string
 	onBack: () => void
 	onRerun: (task: string) => void
-	rerunDisabled?: boolean
 }) {
 	const [session, setSession] = useState<SessionRecord | null>(null)
 
@@ -47,18 +45,26 @@ export function HistoryDetail({
 				<div className="text-xs font-medium" title={session.task}>
 					{session.task}
 				</div>
-				<div className="mt-2">
-					<Button
+				<div className="mt-2 flex items-center gap-2">
+					<button
 						type="button"
-						variant="secondary"
-						size="sm"
 						onClick={() => onRerun(session.task)}
-						disabled={rerunDisabled}
-						className="h-7 cursor-pointer text-[10px]"
+						className="flex items-center gap-1 text-[10px] text-muted-foreground hover:text-foreground transition-colors cursor-pointer disabled:opacity-50 disabled:cursor-not-allowed"
 					>
 						<RotateCcw className="size-3" />
 						Run again
-					</Button>
+					</button>
+					<button
+						type="button"
+						onClick={async () => {
+							await deleteSession(sessionId)
+							onBack()
+						}}
+						className="flex items-center gap-1 text-[10px] text-muted-foreground hover:text-destructive transition-colors cursor-pointer"
+					>
+						<Trash2 className="size-3" />
+						Delete
+					</button>
 				</div>
 			</div>
 
diff --git a/packages/extension/src/components/HistoryList.tsx b/packages/extension/src/components/HistoryList.tsx
index 096ac0f..176633b 100644
--- a/packages/extension/src/components/HistoryList.tsx
+++ b/packages/extension/src/components/HistoryList.tsx
@@ -19,12 +19,10 @@ export function HistoryList({
 	onSelect,
 	onBack,
 	onRerun,
-	rerunDisabled = false,
 }: {
 	onSelect: (id: string) => void
 	onBack: () => void
 	onRerun: (task: string) => void
-	rerunDisabled?: boolean
 }) {
 	const [sessions, setSessions] = useState<SessionRecord[]>([])
 	const [loading, setLoading] = useState(true)
@@ -112,31 +110,31 @@ export function HistoryList({
 						{/* Content */}
 						<div className="flex-1 min-w-0">
 							<p className="text-xs font-medium truncate">{session.task}</p>
-							<p className="text-[10px] text-muted-foreground mt-0.5">
-								{timeAgo(session.createdAt)} · {session.history.length} steps
-							</p>
-						</div>
-
-						<div className="flex items-center gap-1 shrink-0">
-							<button
-								type="button"
-								onClick={(e) => handleRerun(e, session.task)}
-								disabled={rerunDisabled}
-								className="p-1 text-muted-foreground hover:text-foreground transition-colors cursor-pointer disabled:opacity-50 disabled:cursor-not-allowed"
-								title="Run task again"
-								aria-label={`Run history task again: ${session.task}`}
-							>
-								<RotateCcw className="size-3" />
-							</button>
-							<button
-								type="button"
-								onClick={(e) => handleDelete(e, session.id)}
-								className="opacity-0 group-hover:opacity-100 transition-opacity p-1 hover:text-destructive cursor-pointer shrink-0"
-								title="Delete history"
-								aria-label={`Delete history for ${session.task}`}
-							>
-								<Trash2 className="size-3" />
-							</button>
+							<div className="flex items-center mt-0.5">
+								<p className="text-[10px] text-muted-foreground">
+									{timeAgo(session.createdAt)} · {session.history.length} steps
+								</p>
+								<div className="flex items-center gap-0.5 ml-auto opacity-0 group-hover:opacity-100 transition-opacity">
+									<button
+										type="button"
+										onClick={(e) => handleRerun(e, session.task)}
+										className="p-0.5 text-muted-foreground hover:text-foreground transition-colors cursor-pointer"
+										title="Run task again"
+										aria-label={`Run history task again: ${session.task}`}
+									>
+										<RotateCcw className="size-3" />
+									</button>
+									<button
+										type="button"
+										onClick={(e) => handleDelete(e, session.id)}
+										className="p-0.5 text-muted-foreground hover:text-destructive transition-colors cursor-pointer"
+										title="Delete history"
+										aria-label={`Delete history for ${session.task}`}
+									>
+										<Trash2 className="size-3" />
+									</button>
+								</div>
+							</div>
 						</div>
 					</div>
 				))}
diff --git a/packages/extension/src/entrypoints/sidepanel/App.tsx b/packages/extension/src/entrypoints/sidepanel/App.tsx
index 9835460..229c630 100644
--- a/packages/extension/src/entrypoints/sidepanel/App.tsx
+++ b/packages/extension/src/entrypoints/sidepanel/App.tsx
@@ -112,7 +112,6 @@ export default function App() {
 				onSelect={(id) => setView({ name: 'history-detail', sessionId: id })}
 				onBack={() => setView({ name: 'chat' })}
 				onRerun={runTask}
-				rerunDisabled={status === 'running'}
 			/>
 		)
 	}
@@ -123,7 +122,6 @@ export default function App() {
 				sessionId={view.sessionId}
 				onBack={() => setView({ name: 'history' })}
 				onRerun={runTask}
-				rerunDisabled={status === 'running'}
 			/>
 		)
 	}