Address review feedback for Groq provider changes

Agent-Logs-Url: https://github.com/AkhilKonduru1/Atoll/sessions/d40c666d-f589-48ca-9539-1fd33c799962

Co-authored-by: AkhilKonduru1 <210101196+AkhilKonduru1@users.noreply.github.com>
This commit is contained in:
copilot-swe-agent[bot] 2026-04-15 03:14:55 +00:00 • committed by GitHub
commit e8b8ea5602
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 5 additions and 4 deletions

View file

@ -117,6 +117,7 @@ class ModelSelectionPanel: NSPanel {
// MARK: - Model Selection View // MARK: - Model Selection View
struct ModelSelectionView: View { struct ModelSelectionView: View {
private let primaryProviders: [AIModelProvider] = [.gemini, .openai, .claude, .local]
@State private var selectedProvider: AIModelProvider = Defaults[.selectedAIProvider] @State private var selectedProvider: AIModelProvider = Defaults[.selectedAIProvider]
@State private var selectedModel: AIModel? = Defaults[.selectedAIModel] @State private var selectedModel: AIModel? = Defaults[.selectedAIModel]
@State private var enableThinking: Bool = Defaults[.enableThinkingMode] @State private var enableThinking: Bool = Defaults[.enableThinkingMode]
@ -171,7 +172,7 @@ struct ModelSelectionView: View {
.foregroundColor(.primary) .foregroundColor(.primary)
LazyVGrid(columns: Array(repeating: GridItem(.flexible()), count: 2), spacing: 12) { LazyVGrid(columns: Array(repeating: GridItem(.flexible()), count: 2), spacing: 12) {
ForEach(AIModelProvider.allCases.filter { $0 != .groq }) { provider in ForEach(primaryProviders) { provider in
ProviderCard( ProviderCard(
provider: provider, provider: provider,
isSelected: selectedProvider == provider, isSelected: selectedProvider == provider,

View file

@ -459,9 +459,9 @@ class ScreenAssistantManager: NSObject, ObservableObject {
// Get selected model or default to llama-3.3-70b-versatile // Get selected model or default to llama-3.3-70b-versatile
let selectedModel = Defaults[.selectedAIModel] let selectedModel = Defaults[.selectedAIModel]
let modelId = AIModelProvider.groq.supportedModels.contains(where: { $0.id == selectedModel?.id }) let modelId = selectedModel?.id.flatMap { selectedId in
? selectedModel!.id AIModelProvider.groq.supportedModels.contains(where: { $0.id == selectedId }) ? selectedId : nil
: "llama-3.3-70b-versatile" } ?? "llama-3.3-70b-versatile"
guard let url = URL(string: "https://api.groq.com/openai/v1/chat/completions") else { guard let url = URL(string: "https://api.groq.com/openai/v1/chat/completions") else {
print("❌ ScreenAssistant: Invalid Groq API URL") print("❌ ScreenAssistant: Invalid Groq API URL")