From 0c4f8796d7d306279d227368f58d6662fa557559 Mon Sep 17 00:00:00 2001
From: Jiayuan Zhang <forrestchang7@gmail.com>
Date: Tue, 10 Feb 2026 22:44:36 +0800
Subject: [PATCH] fix(desktop): require active provider before Continue in
 setup step

Changed check from availableProviders.length > 0 to current?.available,
so Continue is only enabled when a provider is both configured and
selected as the active default.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 apps/desktop/src/renderer/src/pages/onboarding/setup.tsx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/apps/desktop/src/renderer/src/pages/onboarding/setup.tsx b/apps/desktop/src/renderer/src/pages/onboarding/setup.tsx
index a66b60daf..b8391f512 100644
--- a/apps/desktop/src/renderer/src/pages/onboarding/setup.tsx
+++ b/apps/desktop/src/renderer/src/pages/onboarding/setup.tsx
@@ -23,7 +23,7 @@ export default function SetupStep() {
   const [focusedProvider, setFocusedProvider] =
     useState<ProviderStatus | null>(null)
 
-  const hasConfiguredProvider = availableProviders.length > 0
+  const hasActiveProvider = current?.available === true
 
   const handleConfigure = (provider: ProviderStatus) => {
     setSelectedProvider(provider)
@@ -91,7 +91,7 @@ export default function SetupStep() {
             <Button
               size="lg"
               onClick={handleContinue}
-              disabled={!hasConfiguredProvider}
+              disabled={!hasActiveProvider}
             >
               Continue
             </Button>