From 5c2ab196143a3e98a36bd9b96fb4f99a686a3466 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Wed, 15 Apr 2026 03:15:36 +0000 Subject: [PATCH] Refine Groq model selection fallback and provider card layout constants Agent-Logs-Url: https://github.com/AkhilKonduru1/Atoll/sessions/d40c666d-f589-48ca-9539-1fd33c799962 Co-authored-by: AkhilKonduru1 <210101196+AkhilKonduru1@users.noreply.github.com> --- .../ScreenAssistant/ModelSelectionPanel.swift | 3 ++- DynamicIsland/managers/ScreenAssistantManager.swift | 10 +++++++--- 2 files changed, 9 insertions(+), 4 deletions(-) diff --git a/DynamicIsland/components/ScreenAssistant/ModelSelectionPanel.swift b/DynamicIsland/components/ScreenAssistant/ModelSelectionPanel.swift index a9f7758..8550708 100644 --- a/DynamicIsland/components/ScreenAssistant/ModelSelectionPanel.swift +++ b/DynamicIsland/components/ScreenAssistant/ModelSelectionPanel.swift @@ -358,6 +358,7 @@ struct ModelSelectionView: View { // MARK: - Provider Card struct ProviderCard: View { + private let wideCardMinHeight: CGFloat = 110 let provider: AIModelProvider let isSelected: Bool let onSelect: () -> Void @@ -402,7 +403,7 @@ struct ProviderCard: View { onSelect() } .animation(.easeInOut(duration: 0.2), value: isSelected) - .frame(maxWidth: .infinity, minHeight: isWide ? 110 : nil) + .frame(maxWidth: .infinity, minHeight: isWide ? wideCardMinHeight : nil) } private func iconForProvider(_ provider: AIModelProvider) -> String { diff --git a/DynamicIsland/managers/ScreenAssistantManager.swift b/DynamicIsland/managers/ScreenAssistantManager.swift index f7f9b06..2f10cba 100644 --- a/DynamicIsland/managers/ScreenAssistantManager.swift +++ b/DynamicIsland/managers/ScreenAssistantManager.swift @@ -459,9 +459,13 @@ class ScreenAssistantManager: NSObject, ObservableObject { // Get selected model or default to llama-3.3-70b-versatile let selectedModel = Defaults[.selectedAIModel] - let modelId = selectedModel?.id.flatMap { selectedId in - AIModelProvider.groq.supportedModels.contains(where: { $0.id == selectedId }) ? selectedId : nil - } ?? "llama-3.3-70b-versatile" + let modelId: String + if let selectedId = selectedModel?.id, + AIModelProvider.groq.supportedModels.contains(where: { $0.id == selectedId }) { + modelId = selectedId + } else { + modelId = "llama-3.3-70b-versatile" + } guard let url = URL(string: "https://api.groq.com/openai/v1/chat/completions") else { print("❌ ScreenAssistant: Invalid Groq API URL")