Merge pull request #908 from RooVetGit/cte/fix-restore-checkpoint-dropdown

Fix restore checkpoint popover

Author: cte

src/core/webview/ClineProvider.ts

@@ -1910,7 +1910,6 @@ export class ClineProvider implements vscode.WebviewViewProvider {
 				this.outputChannel.appendLine("Invalid response from Glama API")
 			}
 			await fs.writeFile(glamaModelsFilePath, JSON.stringify(models))
-			this.outputChannel.appendLine(`Glama models fetched and saved: ${JSON.stringify(models, null, 2)}`)
 		} catch (error) {
 			this.outputChannel.appendLine(
 				`Error fetching Glama models: ${JSON.stringify(error, Object.getOwnPropertyNames(error), 2)}`,
@@ -2026,7 +2025,6 @@ export class ClineProvider implements vscode.WebviewViewProvider {
 				this.outputChannel.appendLine("Invalid response from OpenRouter API")
 			}
 			await fs.writeFile(openRouterModelsFilePath, JSON.stringify(models))
-			this.outputChannel.appendLine(`OpenRouter models fetched and saved: ${JSON.stringify(models, null, 2)}`)
 		} catch (error) {
 			this.outputChannel.appendLine(
 				`Error fetching OpenRouter models: ${JSON.stringify(error, Object.getOwnPropertyNames(error), 2)}`,
@@ -2066,7 +2064,6 @@ export class ClineProvider implements vscode.WebviewViewProvider {
 				}
 			}
 			await fs.writeFile(unboundModelsFilePath, JSON.stringify(models))
-			this.outputChannel.appendLine(`Unbound models fetched and saved: ${JSON.stringify(models, null, 2)}`)
 		} catch (error) {
 			this.outputChannel.appendLine(
 				`Error fetching Unbound models: ${JSON.stringify(error, Object.getOwnPropertyNames(error), 2)}`,

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

@@ -3,16 +3,7 @@ import { CheckIcon, Cross2Icon } from "@radix-ui/react-icons"
 
 import { vscode } from "../../../utils/vscode"
 
-import {
-	Button,
-	Popover,
-	PopoverContent,
-	PopoverTrigger,
-	Tooltip,
-	TooltipContent,
-	TooltipProvider,
-	TooltipTrigger,
-} from "@/components/ui"
+import { Button, Popover, PopoverContent, PopoverTrigger } from "@/components/ui"
 
 type CheckpointMenuProps = {
 	ts: number
@@ -52,33 +43,19 @@ export const CheckpointMenu = ({ ts, commitHash, currentCheckpointHash }: Checkp
 
 	return (
 		<div className="flex flex-row gap-1">
-			<TooltipProvider>
-				<Tooltip>
-					<TooltipTrigger asChild>
-						<Button variant="ghost" size="icon" onClick={onCheckpointDiff}>
-							<span className="codicon codicon-diff-single" />
-						</Button>
-					</TooltipTrigger>
-					<TooltipContent align="end">View Diff</TooltipContent>
-				</Tooltip>
-			</TooltipProvider>
+			<Button variant="ghost" size="icon" onClick={onCheckpointDiff} title="View Diff">
+				<span className="codicon codicon-diff-single" />
+			</Button>
 			<Popover
 				open={isOpen}
 				onOpenChange={(open) => {
 					setIsOpen(open)
 					setIsConfirming(false)
 				}}>
 				<PopoverTrigger asChild>
-					<TooltipProvider>
-						<Tooltip>
-							<TooltipTrigger asChild>
-								<Button variant="ghost" size="icon">
-									<span className="codicon codicon-history" />
-								</Button>
-							</TooltipTrigger>
-							<TooltipContent align="end">Restore Checkpoint</TooltipContent>
-						</Tooltip>
-					</TooltipProvider>
+					<Button variant="ghost" size="icon" title="Restore Checkpoint">
+						<span className="codicon codicon-history" />
+					</Button>
 				</PopoverTrigger>
 				<PopoverContent align="end" container={portalContainer}>
 					<div className="flex flex-col gap-2">