From bd5bf996a88fd038a56e1e5e5c86520adf68c05f Mon Sep 17 00:00:00 2001
From: Juan Francisco Lebrero <101231690+frizynn@users.noreply.github.com>
Date: Fri, 13 Mar 2026 19:04:35 -0300
Subject: [PATCH] fix: show 'keep current' option in config when already
 authenticated (#283)

When running `gsd config` with existing LLM auth or web search
configured, show a 'Keep current (provider)' option at the top
so users don't have to re-authenticate.

Co-authored-by: Juan Francisco Lebrero <fran@Juans-MacBook-Air.local>
---
 src/onboarding.ts | 35 +++++++++++++++++++++++++++++------
 1 file changed, 29 insertions(+), 6 deletions(-)

diff --git a/src/onboarding.ts b/src/onboarding.ts
index 28b4f2a06..f2ec1fa62 100644
--- a/src/onboarding.ts
+++ b/src/onboarding.ts
@@ -257,17 +257,30 @@ async function runLlmStep(p: ClackModule, pc: PicoModule, authStorage: AuthStora
   const oauthProviders = authStorage.getOAuthProviders()
   const oauthMap = new Map(oauthProviders.map(op => [op.id, op]))
 
+  // Check if already authenticated
+  const existingAuth = LLM_PROVIDER_IDS.find(id => authStorage.hasAuth(id))
+
   // ── Step 1: How do you want to authenticate? ─────────────────────────────
+  type AuthOption = { value: string; label: string; hint?: string }
+  const authOptions: AuthOption[] = []
+
+  if (existingAuth) {
+    authOptions.push({ value: 'keep', label: `Keep current (${existingAuth})`, hint: 'already configured' })
+  }
+
+  authOptions.push(
+    { value: 'browser', label: 'Sign in with your browser', hint: 'recommended — same login as claude.ai / ChatGPT' },
+    { value: 'api-key', label: 'Paste an API key', hint: 'from your provider dashboard' },
+    { value: 'skip', label: 'Skip for now', hint: 'use /login inside GSD later' },
+  )
+
   const method = await p.select({
-    message: 'How do you want to sign in?',
-    options: [
-      { value: 'browser', label: 'Sign in with your browser', hint: 'recommended — same login as claude.ai / ChatGPT' },
-      { value: 'api-key', label: 'Paste an API key', hint: 'from your provider dashboard' },
-      { value: 'skip', label: 'Skip for now', hint: 'use /login inside GSD later' },
-    ],
+    message: existingAuth ? `LLM provider: ${existingAuth} — change it?` : 'How do you want to sign in?',
+    options: authOptions,
   })
 
   if (p.isCancel(method) || method === 'skip') return false
+  if (method === 'keep') return true
 
   // ── Step 2: Which provider? ──────────────────────────────────────────────
   if (method === 'browser') {
@@ -416,10 +429,19 @@ async function runWebSearchStep(
   const authed = authStorage.list().filter(id => LLM_PROVIDER_IDS.includes(id))
   const isAnthropic = isAnthropicAuth && authed.includes('anthropic')
 
+  // Check if web search is already configured
+  const hasBrave = !!process.env.BRAVE_API_KEY || authStorage.has('brave')
+  const hasTavily = !!process.env.TAVILY_API_KEY || authStorage.has('tavily')
+  const existingSearch = hasBrave ? 'Brave Search' : hasTavily ? 'Tavily' : null
+
   // Build options based on what's available
   type SearchOption = { value: string; label: string; hint?: string }
   const options: SearchOption[] = []
 
+  if (existingSearch) {
+    options.push({ value: 'keep', label: `Keep current (${existingSearch})`, hint: 'already configured' })
+  }
+
   if (isAnthropic) {
     options.push({
       value: 'anthropic-native',
@@ -440,6 +462,7 @@ async function runWebSearchStep(
   })
 
   if (p.isCancel(choice) || choice === 'skip') return null
+  if (choice === 'keep') return existingSearch
 
   if (choice === 'anthropic-native') {
     p.log.success(`Web search: ${pc.green('Anthropic built-in')} — works out of the box`)