fix: always show checkpoint restore options regardless of change detection

Previously, checkpoint restore options were hidden when the system detected no changes (isCurrent === true). This caused the checkpoint menu popover to appear empty for current checkpoints.

Changes:
- Removed conditional rendering in CheckpointMenu.tsx that hid restore options for current checkpoints
- Prefixed unused currentHash parameter with underscore to satisfy linter
- Removed conditional rendering in App.tsx for CheckpointRestoreDialog
- Always show 'Restore to Checkpoint' button in CheckpointRestoreDialog.tsx
- Updated tests to reflect the new behavior

All checkpoint restore options are now always visible, ensuring users can restore to any checkpoint even when change detection indicates no differences.

Author: daniel-lxs

webview-ui/src/components/chat/checkpoints/CheckpointMenu.tsx

@@ -27,7 +27,7 @@ type CheckpointMenuProps = CheckpointMenuBaseProps & (CheckpointMenuControlledPr
 export const CheckpointMenu = ({
 	ts,
 	commitHash,
-	currentHash,
+	currentHash: _currentHash,
 	checkpoint,
 	open,
 	onOpenChange,
@@ -37,8 +37,6 @@ export const CheckpointMenu = ({
 	const [isConfirming, setIsConfirming] = useState(false)
 	const portalContainer = useRooPortal("roo-portal")
 
-	const isCurrent = currentHash === commitHash
-
 	const previousCommitHash = checkpoint?.from
 
 	const isOpen = open ?? internalOpen
@@ -88,60 +86,56 @@ export const CheckpointMenu = ({
 				</StandardTooltip>
 				<PopoverContent align="end" container={portalContainer}>
 					<div className="flex flex-col gap-2">
-						{!isCurrent && (
-							<div className="flex flex-col gap-1 group hover:text-foreground">
-								<Button variant="secondary" onClick={onPreview} data-testid="restore-files-btn">
-									{t("chat:checkpoint.menu.restoreFiles")}
-								</Button>
-								<div className="text-muted transition-colors group-hover:text-foreground">
-									{t("chat:checkpoint.menu.restoreFilesDescription")}
-								</div>
+						<div className="flex flex-col gap-1 group hover:text-foreground">
+							<Button variant="secondary" onClick={onPreview} data-testid="restore-files-btn">
+								{t("chat:checkpoint.menu.restoreFiles")}
+							</Button>
+							<div className="text-muted transition-colors group-hover:text-foreground">
+								{t("chat:checkpoint.menu.restoreFilesDescription")}
 							</div>
-						)}
-						{!isCurrent && (
+						</div>
+						<div className="flex flex-col gap-1 group hover:text-foreground">
 							<div className="flex flex-col gap-1 group hover:text-foreground">
-								<div className="flex flex-col gap-1 group hover:text-foreground">
-									{!isConfirming ? (
+								{!isConfirming ? (
+									<Button
+										variant="secondary"
+										onClick={() => setIsConfirming(true)}
+										data-testid="restore-files-and-task-btn">
+										{t("chat:checkpoint.menu.restoreFilesAndTask")}
+									</Button>
+								) : (
+									<>
 										<Button
-											variant="secondary"
-											onClick={() => setIsConfirming(true)}
-											data-testid="restore-files-and-task-btn">
-											{t("chat:checkpoint.menu.restoreFilesAndTask")}
+											variant="default"
+											onClick={onRestore}
+											className="grow"
+											data-testid="confirm-restore-btn">
+											<div className="flex flex-row gap-1">
+												<CheckIcon />
+												<div>{t("chat:checkpoint.menu.confirm")}</div>
+											</div>
+										</Button>
+										<Button variant="secondary" onClick={() => setIsConfirming(false)}>
+											<div className="flex flex-row gap-1">
+												<Cross2Icon />
+												<div>{t("chat:checkpoint.menu.cancel")}</div>
+											</div>
 										</Button>
-									) : (
-										<>
-											<Button
-												variant="default"
-												onClick={onRestore}
-												className="grow"
-												data-testid="confirm-restore-btn">
-												<div className="flex flex-row gap-1">
-													<CheckIcon />
-													<div>{t("chat:checkpoint.menu.confirm")}</div>
-												</div>
-											</Button>
-											<Button variant="secondary" onClick={() => setIsConfirming(false)}>
-												<div className="flex flex-row gap-1">
-													<Cross2Icon />
-													<div>{t("chat:checkpoint.menu.cancel")}</div>
-												</div>
-											</Button>
-										</>
-									)}
-									{isConfirming ? (
-										<div
-											data-testid="checkpoint-confirm-warning"
-											className="text-destructive font-bold">
-											{t("chat:checkpoint.menu.cannotUndo")}
-										</div>
-									) : (
-										<div className="text-muted transition-colors group-hover:text-foreground">
-											{t("chat:checkpoint.menu.restoreFilesAndTaskDescription")}
-										</div>
-									)}
-								</div>
+									</>
+								)}
+								{isConfirming ? (
+									<div
+										data-testid="checkpoint-confirm-warning"
+										className="text-destructive font-bold">
+										{t("chat:checkpoint.menu.cannotUndo")}
+									</div>
+								) : (
+									<div className="text-muted transition-colors group-hover:text-foreground">
+										{t("chat:checkpoint.menu.restoreFilesAndTaskDescription")}
+									</div>
+								)}
 							</div>
-						)}
+						</div>
 					</div>
 				</PopoverContent>
 			</Popover>