fix(api): create Copilot provider from device flow token, load custom provider IDs

EndpointManager.setupProviders() only checked config.apiKey to decide whether
to create a provider. GitHub Copilot uses device flow tokens stored in
CopilotTokenStore, so also check CopilotTokenStore.shared.isSignedIn. Also
iterate saved_provider_ids to pick up user-created providers with timestamp
IDs that weren't covered by ProviderType.allCases. Guard ModelListManager
against duplicate initialization.

Author: fewtarius

Sources/APIFramework/EndpointManager.swift

@@ -642,8 +642,11 @@ public class EndpointManager: ObservableObject {
             }
 
             /// Only create provider if it has API key or doesn't require one.
+            /// GitHub Copilot also accepts device flow tokens from CopilotTokenStore.
             let requiresKey = providerType.requiresApiKey
-            if !requiresKey || config.apiKey != nil {
+            let hasAuth = config.apiKey != nil ||
+                (providerType == .githubCopilot && CopilotTokenStore.shared.isSignedIn)
+            if !requiresKey || hasAuth {
                 /// For local llama, create one provider per model using registry identifiers.
                 if providerType == .localLlama {
                     if let modelManager = localModelManager {
@@ -751,6 +754,30 @@ public class EndpointManager: ObservableObject {
             }
         }
 
+        // Also check saved_provider_ids for any user-created provider IDs (may have timestamps)
+        if let savedProviderIds = UserDefaults.standard.stringArray(forKey: "saved_provider_ids") {
+            for savedProviderId in savedProviderIds {
+                // Skip if we already loaded this provider
+                if providers[savedProviderId] != nil {
+                    continue
+                }
+
+                // Try to load configuration for this custom ID
+                if let config = loadProviderConfiguration(for: savedProviderId) {
+                    let requiresKey = config.providerType.requiresApiKey
+                    let hasAuth = config.apiKey != nil ||
+                        (config.providerType == .githubCopilot && CopilotTokenStore.shared.isSignedIn)
+                    if !requiresKey || hasAuth {
+                        providers[savedProviderId] = createProvider(type: config.providerType, config: config)
+                        providerConfigs[savedProviderId] = config
+                        logger.debug("Loaded saved provider \(savedProviderId) with \(config.models.count) models")
+                    } else {
+                        logger.debug("Saved provider \(savedProviderId) not loaded - requires API key")
+                    }
+                }
+            }
+        }
+
         logger.debug("Setup complete. Active providers: \(self.providers.keys.sorted().joined(separator: ", "))")
     }
 
@@ -1200,8 +1227,11 @@ public class EndpointManager: ObservableObject {
             }
 
             /// Only create provider if it has API key or doesn't require one.
+            /// GitHub Copilot also accepts device flow tokens from CopilotTokenStore.
             let requiresKey = providerType.requiresApiKey
-            if !requiresKey || config.apiKey != nil {
+            let hasAuth = config.apiKey != nil ||
+                (providerType == .githubCopilot && CopilotTokenStore.shared.isSignedIn)
+            if !requiresKey || hasAuth {
                 /// For local llama, create one provider per model using registry identifiers.
                 if providerType == .localLlama {
                     if let modelManager = localModelManager {

Sources/APIFramework/ModelListManager.swift

@@ -30,6 +30,11 @@ public class ModelListManager: ObservableObject {
 
     /// Initialize the manager with required dependencies
     public func initialize(endpointManager: EndpointManager) {
+        // Skip if already initialized with this endpoint manager
+        if self.endpointManager != nil {
+            logger.debug("ModelListManager already initialized, skipping")
+            return
+        }
         logger.info("Initializing ModelListManager with EndpointManager")
         self.endpointManager = endpointManager