From 97e9beea5f516a46e43e5f9ca6dd86abddf039cc Mon Sep 17 00:00:00 2001 From: Alan Wizemann Date: Thu, 23 Apr 2026 02:13:49 +0200 Subject: [PATCH] refactor(settings): remove unused providers list MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The hardcoded `providers` array in SettingsViewModel was never referenced — no view reads `viewModel.providers`; the Model picker uses the models.dev catalog via `ModelCatalogService.loadProviders()` and Provider is shown as a `ReadOnlyRow` in the General tab. Leaving the dead list around makes issues like #33 look plausible (users reasonably guess a stale enum is normalising `openai-codex` → `openai` on save, which the code does not actually do). Co-Authored-By: Claude Opus 4.7 (1M context) --- scarf/scarf/Features/Settings/ViewModels/SettingsViewModel.swift | 1 - 1 file changed, 1 deletion(-) diff --git a/scarf/scarf/Features/Settings/ViewModels/SettingsViewModel.swift b/scarf/scarf/Features/Settings/ViewModels/SettingsViewModel.swift index 91724bf..8aec927 100644 --- a/scarf/scarf/Features/Settings/ViewModels/SettingsViewModel.swift +++ b/scarf/scarf/Features/Settings/ViewModels/SettingsViewModel.swift @@ -20,7 +20,6 @@ final class SettingsViewModel { var hermesRunning = false var rawConfigYAML = "" var personalities: [String] = [] - var providers = ["anthropic", "openrouter", "nous", "openai-codex", "google-ai-studio", "xai", "ollama-cloud", "zai", "kimi-coding", "minimax"] var terminalBackends = ["local", "docker", "singularity", "modal", "daytona", "ssh"] var browserBackends = ["browseruse", "firecrawl", "local"] var ttsProviders = ["edge", "elevenlabs", "openai", "minimax", "mistral", "neutts"]