acdev 1.0.7 → 1.0.9

package/public/app.js

@@ -130,11 +130,34 @@ let settingsTab = 'ticket';
 /** Sentinel: no model selected (jobs blocked until user picks one). */
 const NO_MODEL = '-';
 /** @type {string} */
+let modelSourceHintBase = '';
+
+/**
+ * @param {HTMLElement} btn
+ * @param {{ id: string, label?: string, name?: string }} model
+ */
+function populateModelPickerOption(btn, model) {
+  const title = document.createElement('span');
+  title.className = 'model-picker-option-title';
+  title.textContent = modelPickerLabel(model);
+  btn.appendChild(title);
+
+  if (model.id !== modelPickerLabel(model)) {
+    const sub = document.createElement('span');
+    sub.className = 'model-picker-option-id';
+    sub.textContent = model.id;
+    btn.appendChild(sub);
+  }
+}
+
+function refreshModelSourceHint() {
+  if (!els.settingsModelHint) return;
+  els.settingsModelHint.textContent = modelSourceHintBase;
+}
+/** @type {string} */
 let currentModel = 'claude-sonnet-5';
 /** @type {Array<{id:string,label?:string,name?:string}>} */
 let availableModels = [];
-/** @type {'claude' | 'openrouter' | null} */
-let availableModelsProvider = null;
 /** @type {'github' | 'jira'} */
 let ticketSource = 'github';
 /** @type {{
@@ -167,11 +190,6 @@ let ticketSource = 'github';
  *   anthropicApiKeyMasked?: string | null,
  *   claudeOauthTokenSet?: boolean,
  *   claudeOauthTokenMasked?: string | null,
- *   llmProvider?: 'claude' | 'openrouter',
- *   lastModelsByProvider?: Record<string, string>,
- *   openrouterAuthOk?: boolean,
- *   openrouterApiKeySet?: boolean,
- *   openrouterApiKeyMasked?: string | null,
  *   llmAuthOk?: boolean,
  *   stubAgent?: boolean,
  * }} */
@@ -187,7 +205,6 @@ const els = {
   modelProvider: document.getElementById('model-provider'),
   modelCurrent: document.getElementById('model-current'),
   modelCombined: document.getElementById('model-combined'),
-  overviewLlmProvider: document.getElementById('overview-llm-provider'),
   overviewModel: document.getElementById('overview-model'),
   overviewAgentPicker: document.getElementById('overview-agent-picker'),
   overviewAgentTrigger: document.getElementById('overview-agent-trigger'),
@@ -273,8 +290,6 @@ const els = {
   settingsModelList: document.getElementById('settings-model-list'),
   settingsModelSearch: document.getElementById('settings-model-search'),
   settingsModelHint: document.getElementById('settings-model-hint'),
-  settingsLlmProvider: document.getElementById('settings-llm-provider'),
-  settingsLlmProviderHint: document.getElementById('settings-llm-provider-hint'),
   settingsMaxTurns: document.getElementById('settings-max-turns'),
   settingsTimeout: document.getElementById('settings-timeout'),
   settingsTimeoutMs: document.getElementById('settings-timeout-ms'),
@@ -315,10 +330,6 @@ const els = {
   settingsClaudeOauth: document.getElementById('settings-claude-oauth'),
   settingsClaudeOauthClear: document.getElementById('settings-claude-oauth-clear'),
   settingsClaudeOauthHint: document.getElementById('settings-claude-oauth-hint'),
-  settingsOpenrouterStatus: document.getElementById('settings-openrouter-status'),
-  settingsOpenrouterKey: document.getElementById('settings-openrouter-key'),
-  settingsOpenrouterKeyClear: document.getElementById('settings-openrouter-key-clear'),
-  settingsOpenrouterKeyHint: document.getElementById('settings-openrouter-key-hint'),
   settingsTabs: document.getElementById('settings-tabs'),
 };
 
@@ -2975,34 +2986,21 @@ async function readJson(res) {
   }
 }
 
-function llmProviderLabel(provider) {
-  return provider === 'openrouter' ? 'OpenRouter' : 'Claude';
-}
-
 /**
- * @param {'claude' | 'openrouter'} [provider]
  * @param {string} [modelId]
  * @returns {string}
  */
-function agentDisplayText(provider = currentLlmProviderSelection(), modelId = currentModel) {
-  const providerName = llmProviderLabel(provider);
-  const modelText = isNoModelSelection(modelId) ? '—' : modelPickerDisplayText(modelId);
-  return `${providerName} • ${modelText}`;
+function agentDisplayText(modelId = currentModel) {
+  return isNoModelSelection(modelId) ? '—' : modelPickerDisplayText(modelId);
 }
 
 function updateModelLabel() {
-  const provider = appConfig.llmProvider === 'openrouter' ? 'openrouter' : 'claude';
-  const providerName = llmProviderLabel(provider);
   const modelText = isNoModelSelection(currentModel) ? '—' : currentModel;
   const modelTitle = isNoModelSelection(currentModel)
     ? 'none selected'
     : currentModel;
-  const combined = agentDisplayText(provider, currentModel);
+  const combined = agentDisplayText(currentModel);
 
-  if (els.modelProvider) {
-    els.modelProvider.textContent = providerName;
-    els.modelProvider.title = `${providerName} — switch in Overview or Settings`;
-  }
   if (els.modelCurrent) {
     els.modelCurrent.textContent = modelText;
     els.modelCurrent.title = `Model: ${modelTitle} — switch in Overview or Settings`;
@@ -3012,29 +3010,19 @@ function updateModelLabel() {
     els.modelCombined.title = combined;
   }
   syncAgentPickerDisplay();
+  refreshModelSourceHint();
 }
 
-/**
- * @param {'claude' | 'openrouter'} [provider]
- * @returns {Array<{id:string,label?:string,name?:string}>}
- */
-function getAvailableModels(provider = currentLlmProviderSelection()) {
-  if (availableModelsProvider === provider) return availableModels;
-  return [];
+function getAvailableModels() {
+  return availableModels.length ? availableModels : defaultModels();
 }
 
-/**
- * @param {'claude' | 'openrouter'} provider
- * @param {Array<{id:string,label?:string,name?:string}>} models
- */
-function setAvailableModels(provider, models) {
-  availableModelsProvider = provider;
+function setAvailableModels(models) {
   availableModels = models;
 }
 
 function clearAvailableModels() {
   availableModels = [];
-  availableModelsProvider = null;
 }
 
 /**
@@ -3281,16 +3269,7 @@ function renderModelComboboxList(instance, selectedId = currentModel) {
       btn.classList.add('is-active');
     }
 
-    const title = document.createElement('span');
-    title.textContent = modelPickerLabel(m);
-    btn.appendChild(title);
-
-    if (m.id !== modelPickerLabel(m)) {
-      const sub = document.createElement('span');
-      sub.className = 'model-picker-option-id';
-      sub.textContent = m.id;
-      btn.appendChild(sub);
-    }
+    populateModelPickerOption(btn, m);
 
     btn.addEventListener('click', () => {
       instance.onSelect(m.id);
@@ -3330,76 +3309,8 @@ function renderAllModelComboboxLists(selectedId = currentModel) {
 /** @type {AgentPickerInstance[]} */
 const agentPickers = [];
 
-const PROVIDER_UNAVAILABLE_TOOLTIP =
-  'Not configured — add credentials in Settings → Authentication';
-
-/**
- * @param {'claude' | 'openrouter'} provider
- * @param {typeof appConfig} [cfg]
- */
-function isLlmProviderAvailable(provider, cfg = appConfig) {
-  if (cfg.stubAgent) return true;
-  if (provider === 'openrouter') return cfg.openrouterAuthOk === true;
-  return cfg.claudeAuthOk === true;
-}
-
-/**
- * Disable provider tabs and select options when auth is missing.
- * @param {typeof appConfig} [cfg]
- */
-function syncLlmProviderAvailability(cfg = appConfig) {
-  for (const ap of agentPickers) {
-    for (const tab of ap.tabs) {
-      const provider =
-        tab.dataset.provider === 'openrouter' ? 'openrouter' : 'claude';
-      const available = isLlmProviderAvailable(provider, cfg);
-      tab.disabled = !available;
-      tab.classList.toggle('is-disabled', !available);
-      tab.setAttribute('aria-disabled', available ? 'false' : 'true');
-      if (available) {
-        tab.removeAttribute('title');
-      } else {
-        tab.title = PROVIDER_UNAVAILABLE_TOOLTIP;
-      }
-    }
-  }
-
-  for (const select of [els.settingsLlmProvider, els.overviewLlmProvider]) {
-    if (!select) continue;
-    for (const opt of select.options) {
-      const provider = opt.value === 'openrouter' ? 'openrouter' : 'claude';
-      const available = isLlmProviderAvailable(provider, cfg);
-      opt.disabled = !available;
-      if (available) {
-        opt.removeAttribute('title');
-      } else {
-        opt.title = PROVIDER_UNAVAILABLE_TOOLTIP;
-      }
-    }
-  }
-}
-
-/**
- * @param {'claude' | 'openrouter'} provider
- */
-function syncAgentPickerTabs(provider) {
-  const value = provider === 'openrouter' ? 'openrouter' : 'claude';
-  for (const ap of agentPickers) {
-    for (const tab of ap.tabs) {
-      const active = tab.dataset.provider === value;
-      tab.classList.toggle('is-active', active);
-      tab.setAttribute('aria-selected', active ? 'true' : 'false');
-    }
-    if (ap.iconEl) {
-      ap.iconEl.classList.toggle('agent-pill-icon--claude', value === 'claude');
-      ap.iconEl.classList.toggle('agent-pill-icon--openrouter', value === 'openrouter');
-    }
-  }
-}
-
 function syncAgentPickerDisplay(selectedId = currentModel) {
-  const provider = currentLlmProviderSelection();
-  const text = agentDisplayText(provider, selectedId);
+  const text = agentDisplayText(selectedId);
   for (const ap of agentPickers) {
     ap.valueEl.textContent = text;
     ap.valueEl.classList.toggle('is-empty', isNoModelSelection(selectedId));
@@ -3407,7 +3318,6 @@ function syncAgentPickerDisplay(selectedId = currentModel) {
       ap.hiddenInput.value = isNoModelSelection(selectedId) ? NO_MODEL : selectedId;
     }
   }
-  syncAgentPickerTabs(provider);
 }
 
 /**
@@ -3431,7 +3341,6 @@ function setAgentPickerOpen(instance, open) {
   if (open) {
     instance.activeIndex = -1;
     instance.search.value = '';
-    syncAgentPickerTabs(currentLlmProviderSelection());
     renderAgentPickerModelList(instance, currentModel);
     focusWithoutScroll(instance.search);
   } else {
@@ -3477,16 +3386,7 @@ function renderAgentPickerModelList(instance, selectedId = currentModel) {
       btn.classList.add('is-active');
     }
 
-    const title = document.createElement('span');
-    title.textContent = modelPickerLabel(m);
-    btn.appendChild(title);
-
-    if (m.id !== modelPickerLabel(m)) {
-      const sub = document.createElement('span');
-      sub.className = 'model-picker-option-id';
-      sub.textContent = m.id;
-      btn.appendChild(sub);
-    }
+    populateModelPickerOption(btn, m);
 
     btn.addEventListener('click', () => {
       instance.onSelect(m.id);
@@ -3577,19 +3477,6 @@ function initAgentPicker(opts) {
     }
   });
 
-  for (const tab of instance.tabs) {
-    tab.addEventListener('click', () => {
-      if (tab.disabled) return;
-      const provider =
-        tab.dataset.provider === 'openrouter' ? 'openrouter' : 'claude';
-      if (!isLlmProviderAvailable(provider)) return;
-      if (provider === currentLlmProviderSelection()) return;
-      syncLlmProviderSelects(provider);
-      syncAgentPickerTabs(provider);
-      void onLlmProviderChange(provider);
-    });
-  }
-
   return instance;
 }
 
@@ -3622,26 +3509,10 @@ function reconcileModelForProvider(models, candidate = currentModel, storedCandi
   return NO_MODEL;
 }
 
-/**
- * @param {'claude' | 'openrouter'} provider
- * @returns {string | undefined}
- */
-function lastModelForProvider(provider) {
-  const stored = appConfig.lastModelsByProvider?.[provider];
-  return typeof stored === 'string' && !isNoModelSelection(stored) ? stored : undefined;
-}
-
 /**
  * @returns {Array<{id:string,label?:string,name?:string}>}
  */
-function defaultModelsForProvider(provider = 'claude') {
-  if (provider === 'openrouter') {
-    return [
-      { id: 'anthropic/claude-sonnet-4', label: 'Claude Sonnet 4 (Anthropic)' },
-      { id: 'anthropic/claude-opus-4', label: 'Claude Opus 4 (Anthropic)' },
-      { id: 'anthropic/claude-3.5-sonnet', label: 'Claude 3.5 Sonnet (Anthropic)' },
-    ];
-  }
+function defaultModels() {
   return [
     { id: 'claude-sonnet-5', label: 'Sonnet 5' },
     { id: 'claude-opus-5', label: 'Opus 5' },
@@ -3653,29 +3524,20 @@ function defaultModelsForProvider(provider = 'claude') {
 }
 
 /**
- * @param {'anthropic' | 'openrouter' | 'fallback' | string | undefined} source
- * @param {'claude' | 'openrouter'} [provider]
+ * @param {'anthropic' | 'fallback' | string | undefined} source
  */
-function updateModelSourceHint(source, provider = 'claude') {
+function updateModelSourceHint(source) {
   if (!els.settingsModelHint) return;
-  const name = provider === 'openrouter' ? 'OpenRouter' : 'Claude';
   if (source === 'anthropic') {
-    els.settingsModelHint.textContent =
-      `${name} model for agent runs. List loaded from Anthropic Models API.`;
-  } else if (source === 'openrouter') {
-    els.settingsModelHint.textContent =
-      'OpenRouter model for agent runs (anthropic/* only — Claude Agent SDK uses Anthropic-compatible API). List loaded from OpenRouter Models API.';
+    modelSourceHintBase =
+      'Claude model for agent runs. List loaded from Anthropic Models API.';
   } else if (source === 'fallback') {
-    if (provider === 'openrouter') {
-      els.settingsModelHint.textContent =
-        'OpenRouter model for agent runs (anthropic/* only). Showing curated Anthropic models (live list unavailable — set OPENROUTER_API_KEY in Authentication).';
-    } else {
-      els.settingsModelHint.textContent =
-        'Claude model for agent runs. Showing curated Claude Code models (live list unavailable — set an API key or use claude auth login).';
-    }
+    modelSourceHintBase =
+      'Claude model for agent runs. Showing curated Claude Code models (live list unavailable — set an API key or use claude auth login).';
   } else {
-    els.settingsModelHint.textContent = `${name} model for agent runs.`;
+    modelSourceHintBase = 'Claude model for agent runs.';
   }
+  refreshModelSourceHint();
 }
 
 /**
@@ -3699,90 +3561,55 @@ function modelMatchesSearchQuery(model, query) {
     .some((part) => String(part).toLowerCase().includes(q));
 }
 
-/**
- * @returns {'claude' | 'openrouter'}
- */
-function currentLlmProviderSelection() {
-  const fromOverview = els.overviewLlmProvider?.value;
-  if (fromOverview === 'openrouter' || fromOverview === 'claude') return fromOverview;
-  const fromForm = els.settingsLlmProvider?.value;
-  if (fromForm === 'openrouter' || fromForm === 'claude') return fromForm;
-  return appConfig.llmProvider === 'openrouter' ? 'openrouter' : 'claude';
-}
-
-/**
- * @param {'claude' | 'openrouter'} provider
- */
-function syncLlmProviderSelects(provider) {
-  const value = provider === 'openrouter' ? 'openrouter' : 'claude';
-  if (els.settingsLlmProvider) els.settingsLlmProvider.value = value;
-  if (els.overviewLlmProvider) els.overviewLlmProvider.value = value;
-  syncAgentPickerTabs(value);
-  if (els.settingsLlmProviderHint) {
-    els.settingsLlmProviderHint.textContent =
-      value === 'openrouter'
-        ? 'Agent runs route through OpenRouter (requires OPENROUTER_API_KEY). Use anthropic/* models only.'
-        : 'Agent runs use Claude via Anthropic API, OAuth token, or claude auth login.';
-  }
-}
-
-/**
- * @param {{ models?: Array<{id:string,label?:string,name?:string}>, selected?: string, source?: string, provider?: string }} data
- */
 function populateSettingsModels(data) {
-  const provider = data?.provider || currentLlmProviderSelection();
-  const cached = getAvailableModels(provider);
-  const models = Array.isArray(data?.models)
+  let models = Array.isArray(data?.models)
     ? data.models
-    : data?.reconcileProvider
+    : data?.reconcile
       ? []
-      : cached.length
-        ? cached
-        : defaultModelsForProvider(provider);
-  setAvailableModels(provider, models);
+      : getAvailableModels().length
+        ? getAvailableModels()
+        : defaultModels();
+  if (!models.length && data?.source) {
+    models = defaultModels();
+  }
+  setAvailableModels(models);
 
-  if (data?.reconcileProvider) {
+  if (data?.reconcile) {
     currentModel = reconcileModelForProvider(
       models,
-      data.reconcileCandidate,
-      lastModelForProvider(provider)
+      data.reconcileCandidate
     );
   } else if (data?.selected != null) {
     const next = isNoModelSelection(data.selected) ? NO_MODEL : data.selected;
-    currentModel = reconcileModelForProvider(models, next, lastModelForProvider(provider));
+    currentModel = reconcileModelForProvider(models, next);
   }
-  if (data?.source) updateModelSourceHint(data.source, provider);
+  if (data?.source) updateModelSourceHint(data.source);
 
   syncAllModelComboboxValues(currentModel);
   renderAllModelComboboxLists(currentModel);
   updateModelLabel();
 }
 
-function modelsForFetchFallback(provider, reconcileProvider) {
-  if (reconcileProvider) return [];
-  const cached = getAvailableModels(provider);
-  if (cached.length) return cached;
-  return defaultModelsForProvider(provider);
+function modelsForFetchFallback() {
+  return getAvailableModels().length ? getAvailableModels() : defaultModels();
 }
 
 async function fetchModels(opts = {}) {
   const refresh = opts.refresh === true;
-  const provider = opts.provider || currentLlmProviderSelection();
-  const reconcileProvider = opts.reconcileProvider === true;
+  const reconcile = opts.reconcile === true;
   const reconcileCandidate = opts.reconcileCandidate;
-  if (reconcileProvider) {
+  if (reconcile) {
     clearAvailableModels();
   }
-  const query = new URLSearchParams({ provider });
+  const query = new URLSearchParams();
   if (refresh) query.set('refresh', '1');
   try {
     const res = await fetch(`/api/models?${query.toString()}`);
     if (!res.ok) {
       populateSettingsModels({
-        models: modelsForFetchFallback(provider, reconcileProvider),
+        models: modelsForFetchFallback(),
         selected: currentModel,
-        provider,
-        reconcileProvider,
+        reconcile,
         reconcileCandidate,
       });
       return;
@@ -3790,17 +3617,15 @@ async function fetchModels(opts = {}) {
     const data = await readJson(res);
     populateSettingsModels({
       ...data,
-      provider: data.provider || provider,
-      reconcileProvider,
+      reconcile,
       reconcileCandidate,
     });
   } catch (err) {
     console.error('Failed to fetch models:', err);
     populateSettingsModels({
-      models: modelsForFetchFallback(provider, reconcileProvider),
+      models: modelsForFetchFallback(),
       selected: currentModel,
-      provider,
-      reconcileProvider,
+      reconcile,
       reconcileCandidate,
     });
   }
@@ -3920,13 +3745,6 @@ function claudeStatusText(cfg) {
   }
 }
 
-function openrouterStatusText(cfg) {
-  if (cfg.openrouterAuthOk) {
-    return 'Authenticated via OPENROUTER_API_KEY';
-  }
-  return 'Not authenticated — add an OpenRouter API key below';
-}
-
 function fillAuthSettings(cfg) {
   if (els.settingsGhStatus) {
     els.settingsGhStatus.textContent = ghStatusText(cfg);
@@ -3985,25 +3803,6 @@ function fillAuthSettings(cfg) {
       ? 'OAuth token stored in <code>.acdev/.env</code> as <code>CLAUDE_CODE_OAUTH_TOKEN</code>. Settings cannot complete browser OAuth — that still needs <code>claude auth login</code> on this host.'
       : 'From <code>claude setup-token</code> for non-interactive subscription auth. Stored as <code>CLAUDE_CODE_OAUTH_TOKEN</code>. Settings cannot complete browser OAuth — that still needs <code>claude auth login</code> on this host.';
   }
-
-  if (els.settingsOpenrouterStatus) {
-    els.settingsOpenrouterStatus.textContent = openrouterStatusText(cfg);
-    els.settingsOpenrouterStatus.className = cfg.openrouterAuthOk
-      ? 'auth-status ok'
-      : 'auth-status err';
-  }
-  fillSecretInput(
-    els.settingsOpenrouterKey,
-    els.settingsOpenrouterKeyClear,
-    Boolean(cfg.openrouterApiKeySet),
-    cfg.openrouterApiKeyMasked,
-    'Paste an OpenRouter API key'
-  );
-  if (els.settingsOpenrouterKeyHint) {
-    els.settingsOpenrouterKeyHint.innerHTML = cfg.openrouterApiKeySet
-      ? 'API key stored in <code>.acdev/.env</code> as <code>OPENROUTER_API_KEY</code> (not committed).'
-      : 'Required when LLM provider is OpenRouter. Get a key at <a href="https://openrouter.ai/keys" target="_blank" rel="noopener noreferrer">openrouter.ai/keys</a>. Stored as <code>OPENROUTER_API_KEY</code> in <code>.acdev/.env</code>.';
-  }
 }
 
 function fillSettingsForm(cfg) {
@@ -4083,23 +3882,17 @@ function fillSettingsForm(cfg) {
     els.settingsBaseBranch.value = cfg.baseBranch || '';
   }
 
-  if (els.settingsLlmProvider) {
-    syncLlmProviderSelects(cfg.llmProvider === 'openrouter' ? 'openrouter' : 'claude');
-  }
-
   if (els.settingsModelPicker) {
-    const provider = cfg.llmProvider === 'openrouter' ? 'openrouter' : 'claude';
-    if (getAvailableModels(provider).length) {
+    if (getAvailableModels().length) {
       populateSettingsModels({
-        models: getAvailableModels(provider),
+        models: getAvailableModels(),
         selected: cfg.model ?? currentModel,
-        provider,
       });
     } else if (cfg.model != null) {
       currentModel = isNoModelSelection(cfg.model) ? NO_MODEL : cfg.model;
       syncModelPickerValue(currentModel);
     }
-    void fetchModels({ provider });
+    void fetchModels();
   }
 
   if (els.settingsMaxTurns) {
@@ -4292,16 +4085,14 @@ function setOverviewLlmFeedback(message, kind = 'ok') {
  */
 function fillOverviewLlmControls(cfg) {
   if (!cfg) return;
-  const provider = cfg.llmProvider === 'openrouter' ? 'openrouter' : 'claude';
-  syncLlmProviderSelects(provider);
   syncAllModelComboboxValues(currentModel);
 }
 
 let llmConfigSaving = false;
 
 /**
- * PATCH llmProvider and/or model immediately (Overview switcher).
- * @param {{ llmProvider?: 'claude' | 'openrouter', model?: string }} patch
+ * PATCH model immediately (Overview switcher).
+ * @param {{ model?: string }} patch
  */
 async function patchLlmConfig(patch) {
   if (llmConfigSaving) return;
@@ -4323,7 +4114,7 @@ async function patchLlmConfig(patch) {
     if (currentView === 'settings') {
       fillSettingsForm(data);
     }
-    setOverviewLlmFeedback('LLM updated. Next jobs will use these values.', 'ok');
+    setOverviewLlmFeedback('Model updated. Next jobs will use this value.', 'ok');
     window.setTimeout(() => setOverviewLlmFeedback(null), 2500);
   } catch (err) {
     setOverviewLlmFeedback(err.message || 'Update failed', 'error');
@@ -4332,29 +4123,6 @@ async function patchLlmConfig(patch) {
   }
 }
 
-/**
- * @param {'claude' | 'openrouter'} provider
- */
-async function onLlmProviderChange(provider) {
-  const previousModel = currentModel;
-  syncLlmProviderSelects(provider);
-  clearAvailableModels();
-  currentModel = NO_MODEL;
-  syncAllModelComboboxValues(currentModel);
-  renderAllModelComboboxLists(currentModel);
-  updateModelLabel();
-  await fetchModels({
-    refresh: true,
-    provider,
-    reconcileProvider: true,
-    reconcileCandidate: previousModel,
-  });
-  await patchLlmConfig({
-    llmProvider: provider,
-    model: currentModel,
-  });
-}
-
 function applyConfigSnapshot(data) {
   appConfig = data || {};
   if (data?.model != null) {
@@ -4370,7 +4138,6 @@ function applyConfigSnapshot(data) {
   if (els.repoBranch) {
     els.repoBranch.textContent = data?.baseBranch || '—';
   }
-  syncLlmProviderAvailability(appConfig);
   if (currentView === 'settings') {
     fillSettingsForm(appConfig);
   }
@@ -4379,11 +4146,13 @@ function applyConfigSnapshot(data) {
 async function fetchConfig() {
   try {
     const res = await fetch('/api/config');
-    if (!res.ok) return;
+    if (!res.ok) return null;
     const data = await readJson(res);
     applyConfigSnapshot(data);
+    return data;
   } catch (err) {
     console.error('Failed to fetch config:', err);
+    return null;
   }
 }
 
@@ -4404,7 +4173,6 @@ function readSettingsForm() {
   const patch = {
     baseBranch,
     model,
-    llmProvider: els.settingsLlmProvider?.value === 'openrouter' ? 'openrouter' : 'claude',
     maxAgentTurns,
     agentTimeoutMs: Math.round(timeoutMinutes * 60_000),
     testCommand: testTrimmed === '' ? null : testTrimmed,
@@ -4418,8 +4186,6 @@ function readSettingsForm() {
   if (anthropicApiKey !== undefined) patch.anthropicApiKey = anthropicApiKey;
   const claudeOauthToken = secretPatchValue(els.settingsClaudeOauth);
   if (claudeOauthToken !== undefined) patch.claudeOauthToken = claudeOauthToken;
-  const openrouterApiKey = secretPatchValue(els.settingsOpenrouterKey);
-  if (openrouterApiKey !== undefined) patch.openrouterApiKey = openrouterApiKey;
 
   if (ticketSource === 'jira' || els.settingsJiraBaseUrl?.value) {
     patch.jiraBaseUrl = els.settingsJiraBaseUrl?.value?.trim() || '';
@@ -4587,12 +4353,6 @@ bindSecretClear(
   els.settingsClaudeOauthHint,
   'Saved CLAUDE_CODE_OAUTH_TOKEN will be removed when you click Save settings.'
 );
-bindSecretClear(
-  els.settingsOpenrouterKeyClear,
-  els.settingsOpenrouterKey,
-  els.settingsOpenrouterKeyHint,
-  'Saved OPENROUTER_API_KEY will be removed when you click Save settings.'
-);
 
 
 document.addEventListener('click', (ev) => {
@@ -4625,38 +4385,6 @@ document.addEventListener('keydown', (ev) => {
   }
 });
 
-els.settingsLlmProvider?.addEventListener('change', () => {
-  const provider = currentLlmProviderSelection();
-  if (!isLlmProviderAvailable(provider)) {
-    const fallback =
-      appConfig.llmProvider === 'openrouter' ? 'openrouter' : 'claude';
-    syncLlmProviderSelects(fallback);
-    return;
-  }
-  for (const cb of modelComboboxes) {
-    setModelComboboxOpen(cb, false);
-  }
-  void onLlmProviderChange(provider);
-});
-
-els.overviewLlmProvider?.addEventListener('change', () => {
-  const provider =
-    els.overviewLlmProvider.value === 'openrouter' ? 'openrouter' : 'claude';
-  if (!isLlmProviderAvailable(provider)) {
-    const fallback =
-      appConfig.llmProvider === 'openrouter' ? 'openrouter' : 'claude';
-    syncLlmProviderSelects(fallback);
-    return;
-  }
-  for (const cb of modelComboboxes) {
-    setModelComboboxOpen(cb, false);
-  }
-  for (const ap of agentPickers) {
-    setAgentPickerOpen(ap, false);
-  }
-  void onLlmProviderChange(provider);
-});
-
 if (
   els.settingsModelPicker &&
   els.settingsModelTrigger &&
@@ -4699,9 +4427,6 @@ if (
     list: els.overviewAgentModelList,
     valueEl: els.overviewAgentValue,
     iconEl: els.overviewAgentIcon,
-    tabs: [
-      ...(els.overviewAgentPanel.querySelectorAll('.agent-picker-tab') || []),
-    ],
     hiddenInput: els.overviewModel,
     optionIdPrefix: 'overview-agent-model-opt',
     onSelect: (modelId) => {
@@ -5031,7 +4756,9 @@ els.logJobFilter.addEventListener('change', () => {
 });
 
 applyTheme(loadTheme());
-fetchConfig();
-fetchModels();
+void (async () => {
+  await fetchConfig();
+  await fetchModels();
+})();
 fetchJobs();
 setInterval(fetchJobs, 3000);