Improve error handling in API requests

jsonify · jsonify · commit 565a0268fc5c · 2025-07-21T15:47:13.000-07:00
Enhance the robustness of API interactions by implementing better error handling mechanisms, ensuring clearer feedback for users and reducing potential crashes.
diff --git a/ClickIt/ClickIt.entitlements b/ClickIt/ClickIt.entitlements
@@ -7,6 +7,10 @@
 	<key>com.apple.security.app-sandbox</key>
 	<false/>
 	
+	<!-- Essential entitlement for Accessibility API access -->
+	<key>com.apple.security.automation.apple-events</key>
+	<true/>
+	
 	<!-- Allow ClickIt to read user-selected files if needed -->
 	<key>com.apple.security.files.user-selected.read-only</key>
 	<true/>
diff --git a/Sources/ClickIt/UI/Components/ActiveTimerView.swift b/Sources/ClickIt/UI/Components/ActiveTimerView.swift
@@ -100,15 +100,17 @@ struct ActiveTimerView: View {
     }
 }
 
-#Preview {
-    ActiveTimerView(viewModel: {
-        let vm = ClickItViewModel()
-        vm.isCountingDown = true
-        vm.remainingTime = 123 // 2:03
-        vm.timerDurationMinutes = 2
-        vm.timerDurationSeconds = 30
-        return vm
-    }())
-    .frame(width: 350)
-    .padding()
+struct ActiveTimerView_Previews: PreviewProvider {
+    static var previews: some View {
+        ActiveTimerView(viewModel: {
+            let vm = ClickItViewModel()
+            vm.isCountingDown = true
+            vm.remainingTime = 123 // 2:03
+            vm.timerDurationMinutes = 2
+            vm.timerDurationSeconds = 30
+            return vm
+        }())
+        .frame(width: 350)
+        .padding()
+    }
 }
diff --git a/Sources/ClickIt/UI/Components/AdvancedSettingsButton.swift b/Sources/ClickIt/UI/Components/AdvancedSettingsButton.swift
@@ -46,8 +46,10 @@ struct AdvancedSettingsButton: View {
     }
 }
 
-#Preview {
-    AdvancedSettingsButton(viewModel: ClickItViewModel())
-        .frame(width: 400)
-        .padding()
+struct AdvancedSettingsButton_Previews: PreviewProvider {
+    static var previews: some View {
+        AdvancedSettingsButton(viewModel: ClickItViewModel())
+            .frame(width: 400)
+            .padding()
+    }
 }
diff --git a/Sources/ClickIt/UI/Components/ClickPointSelector.swift b/Sources/ClickIt/UI/Components/ClickPointSelector.swift
@@ -224,9 +224,11 @@ struct ClickCoordinateCapture {
     }
 }
 
-#Preview {
-    ClickPointSelector { point in
-        print("Selected point: \(point)")
+struct ClickPointSelector_Previews: PreviewProvider {
+    static var previews: some View {
+        ClickPointSelector { point in
+            print("Selected point: \(point)")
+        }
+        .frame(width: 400, height: 500)
     }
-    .frame(width: 400, height: 500)
 }
diff --git a/Sources/ClickIt/UI/Components/ConfigurationPanel.swift b/Sources/ClickIt/UI/Components/ConfigurationPanel.swift
@@ -394,8 +394,10 @@ struct ConfigurationPanel: View {
     }
 }
 
-#Preview {
-    ConfigurationPanel(selectedClickPoint: CGPoint(x: 100, y: 100))
-        .environmentObject(ClickCoordinator.shared)
-        .frame(width: 400, height: 600)
+struct ConfigurationPanel_Previews: PreviewProvider {
+    static var previews: some View {
+        ConfigurationPanel(selectedClickPoint: CGPoint(x: 100, y: 100))
+            .environmentObject(ClickCoordinator.shared)
+            .frame(width: 400, height: 600)
+    }
 }
diff --git a/Sources/ClickIt/UI/Components/ConfigurationPanelCard.swift b/Sources/ClickIt/UI/Components/ConfigurationPanelCard.swift
@@ -184,13 +184,15 @@ struct TimeInputField: View {
     }
 }
 
-#Preview {
-    ConfigurationPanelCard(viewModel: {
-        let vm = ClickItViewModel()
-        vm.intervalSeconds = 1
-        vm.intervalMilliseconds = 500
-        return vm
-    }())
-    .frame(width: 400)
-    .padding()
+struct ConfigurationPanelCard_Previews: PreviewProvider {
+    static var previews: some View {
+        ConfigurationPanelCard(viewModel: {
+            let vm = ClickItViewModel()
+            vm.intervalSeconds = 1
+            vm.intervalMilliseconds = 500
+            return vm
+        }())
+        .frame(width: 400)
+        .padding()
+    }
 }
diff --git a/Sources/ClickIt/UI/Components/DeveloperUpdateButton.swift b/Sources/ClickIt/UI/Components/DeveloperUpdateButton.swift
@@ -54,6 +54,8 @@ struct DeveloperUpdateButton: View {
     }
 }
 
-#Preview {
-    DeveloperUpdateButton(updaterManager: UpdaterManager())
+struct DeveloperUpdateButton_Previews: PreviewProvider {
+    static var previews: some View {
+        DeveloperUpdateButton(updaterManager: UpdaterManager())
+    }
 }
diff --git a/Sources/ClickIt/UI/Components/FooterInfoCard.swift b/Sources/ClickIt/UI/Components/FooterInfoCard.swift
@@ -29,8 +29,10 @@ struct FooterInfoCard: View {
     }
 }
 
-#Preview {
-    FooterInfoCard()
-        .frame(width: 400)
-        .padding()
+struct FooterInfoCard_Previews: PreviewProvider {
+    static var previews: some View {
+        FooterInfoCard()
+            .frame(width: 400)
+            .padding()
+    }
 }
diff --git a/Sources/ClickIt/UI/Components/PermissionStatusIndicator.swift b/Sources/ClickIt/UI/Components/PermissionStatusIndicator.swift
@@ -196,12 +196,14 @@ struct PermissionHealthBadge: View {
     }
 }
 
-#Preview("Permission Status Indicator") {
-    VStack(spacing: 16) {
-        PermissionStatusIndicator()
-        CompactPermissionStatus()
-        PermissionHealthBadge()
+struct PermissionStatusIndicator_Previews: PreviewProvider {
+    static var previews: some View {
+        VStack(spacing: 16) {
+            PermissionStatusIndicator()
+            CompactPermissionStatus()
+            PermissionHealthBadge()
+        }
+        .padding()
+        .environmentObject(PermissionManager.shared)
     }
-    .padding()
-    .environmentObject(PermissionManager.shared)
 }
diff --git a/Sources/ClickIt/UI/Components/RealTimeElapsedView.swift b/Sources/ClickIt/UI/Components/RealTimeElapsedView.swift
@@ -45,45 +45,47 @@ struct ElapsedTimeStatisticView: View {
     }
 }
 
-#Preview("Real Time Elapsed View") {
-    VStack(spacing: 20) {
-        Text("Real-Time Elapsed Time Display")
-            .font(.headline)
-        
-        RealTimeElapsedView(timeManager: {
-            let manager = ElapsedTimeManager.shared
-            manager.startTracking()
-            return manager
-        }())
-        
-        Button("Start/Stop Tracking") {
-            let manager = ElapsedTimeManager.shared
-            if manager.isTracking {
-                manager.stopTracking()
-            } else {
+struct RealTimeElapsedView_Previews: PreviewProvider {
+    static var previews: some View {
+        VStack(spacing: 20) {
+            Text("Real-Time Elapsed Time Display")
+                .font(.headline)
+            
+            RealTimeElapsedView(timeManager: {
+                let manager = ElapsedTimeManager.shared
                 manager.startTracking()
+                return manager
+            }())
+            
+            Button("Start/Stop Tracking") {
+                let manager = ElapsedTimeManager.shared
+                if manager.isTracking {
+                    manager.stopTracking()
+                } else {
+                    manager.startTracking()
+                }
             }
+            .buttonStyle(.borderedProminent)
         }
-        .buttonStyle(.borderedProminent)
-    }
-    .padding()
-    .frame(width: 300)
-}
-
-#Preview("Elapsed Time Statistic") {
-    ElapsedTimeStatisticView(
-        timeManager: ElapsedTimeManager.shared,
-        fallbackStatistics: SessionStatistics(
-            duration: 125,
-            totalClicks: 50,
-            successfulClicks: 48,
-            failedClicks: 2,
-            successRate: 0.96,
-            averageClickTime: 0.05,
-            clicksPerSecond: 2.4,
-            isActive: false
+        .padding()
+        .frame(width: 300)
+        .previewDisplayName("Real Time Elapsed View")
+        
+        ElapsedTimeStatisticView(
+            timeManager: ElapsedTimeManager.shared,
+            fallbackStatistics: SessionStatistics(
+                duration: 125,
+                totalClicks: 50,
+                successfulClicks: 48,
+                failedClicks: 2,
+                successRate: 0.96,
+                averageClickTime: 0.05,
+                clicksPerSecond: 2.4,
+                isActive: false
+            )
         )
-    )
-    .frame(width: 120, height: 100)
-    .padding()
+        .frame(width: 120, height: 100)
+        .padding()
+        .previewDisplayName("Elapsed Time Statistic")
+    }
 }
diff --git a/Sources/ClickIt/UI/Components/StatisticsView.swift b/Sources/ClickIt/UI/Components/StatisticsView.swift
@@ -60,8 +60,10 @@ struct StatisticsView: View {
     }
 }
 
-#Preview {
-    StatisticsView()
-        .environmentObject(ClickCoordinator.shared)
-        .frame(width: 300, height: 100)
+struct StatisticsView_Previews: PreviewProvider {
+    static var previews: some View {
+        StatisticsView()
+            .environmentObject(ClickCoordinator.shared)
+            .frame(width: 300, height: 100)
+    }
 }
diff --git a/Sources/ClickIt/UI/Components/StatusHeaderCard.swift b/Sources/ClickIt/UI/Components/StatusHeaderCard.swift
@@ -123,8 +123,10 @@ struct StatisticView: View {
     }
 }
 
-#Preview {
-    StatusHeaderCard(viewModel: ClickItViewModel())
-        .frame(width: 400)
-        .padding()
+struct StatusHeaderCard_Previews: PreviewProvider {
+    static var previews: some View {
+        StatusHeaderCard(viewModel: ClickItViewModel())
+            .frame(width: 400)
+            .padding()
+    }
 }
diff --git a/Sources/ClickIt/UI/Components/TargetPointSelectionCard.swift b/Sources/ClickIt/UI/Components/TargetPointSelectionCard.swift
@@ -291,12 +291,14 @@ struct CoordinateDisplay: View {
     }
 }
 
-#Preview {
-    TargetPointSelectionCard(viewModel: {
-        let vm = ClickItViewModel()
-        vm.setTargetPoint(CGPoint(x: 1007, y: 260))
-        return vm
-    }())
-    .frame(width: 400)
-    .padding()
+struct TargetPointSelectionCard_Previews: PreviewProvider {
+    static var previews: some View {
+        TargetPointSelectionCard(viewModel: {
+            let vm = ClickItViewModel()
+            vm.setTargetPoint(CGPoint(x: 1007, y: 260))
+            return vm
+        }())
+        .frame(width: 400)
+        .padding()
+    }
 }
diff --git a/Sources/ClickIt/UI/Components/TimerConfigurationView.swift b/Sources/ClickIt/UI/Components/TimerConfigurationView.swift
@@ -148,13 +148,15 @@ struct TimerConfigurationView: View {
     }
 }
 
-#Preview {
-    TimerConfigurationView(viewModel: {
-        let vm = ClickItViewModel()
-        vm.timerDurationMinutes = 0
-        vm.timerDurationSeconds = 15
-        return vm
-    }())
-    .frame(width: 350)
-    .padding()
+struct TimerConfigurationView_Previews: PreviewProvider {
+    static var previews: some View {
+        TimerConfigurationView(viewModel: {
+            let vm = ClickItViewModel()
+            vm.timerDurationMinutes = 0
+            vm.timerDurationSeconds = 15
+            return vm
+        }())
+        .frame(width: 350)
+        .padding()
+    }
 }
diff --git a/Sources/ClickIt/UI/Components/UpdateNotificationCard.swift b/Sources/ClickIt/UI/Components/UpdateNotificationCard.swift
@@ -204,6 +204,8 @@ struct InfoRow: View {
     }
 }
 
-#Preview {
-    UpdateNotificationCard(updaterManager: UpdaterManager())
+struct UpdateNotificationCard_Previews: PreviewProvider {
+    static var previews: some View {
+        UpdateNotificationCard(updaterManager: UpdaterManager())
+    }
 }
diff --git a/Sources/ClickIt/UI/Views/AdvancedSettingsWindow.swift b/Sources/ClickIt/UI/Views/AdvancedSettingsWindow.swift
@@ -106,6 +106,8 @@ struct AdvancedSettingsWindow: View {
     }
 }
 
-#Preview {
-    AdvancedSettingsWindow(viewModel: ClickItViewModel())
+struct AdvancedSettingsWindow_Previews: PreviewProvider {
+    static var previews: some View {
+        AdvancedSettingsWindow(viewModel: ClickItViewModel())
+    }
 }
diff --git a/Sources/ClickIt/UI/Views/ContentView.swift b/Sources/ClickIt/UI/Views/ContentView.swift
@@ -149,9 +149,11 @@ struct ContentView: View {
     }
 }
 
-#Preview {
-    ContentView()
-        .environmentObject(PermissionManager.shared)
-        .environmentObject(HotkeyManager.shared)
-        .environmentObject(ClickItViewModel())
+struct ContentView_Previews: PreviewProvider {
+    static var previews: some View {
+        ContentView()
+            .environmentObject(PermissionManager.shared)
+            .environmentObject(HotkeyManager.shared)
+            .environmentObject(ClickItViewModel())
+    }
 }
diff --git a/Sources/ClickIt/UI/Views/PermissionRequestView.swift b/Sources/ClickIt/UI/Views/PermissionRequestView.swift
@@ -342,7 +342,9 @@ struct PermissionInstructionsView: View {
     }
 }
 
-#Preview {
-    PermissionRequestView()
-        .environmentObject(PermissionManager.shared)
+struct PermissionRequestView_Previews: PreviewProvider {
+    static var previews: some View {
+        PermissionRequestView()
+            .environmentObject(PermissionManager.shared)
+    }
 }
diff --git a/Sources/ClickIt/UI/Views/PermissionsGateView.swift b/Sources/ClickIt/UI/Views/PermissionsGateView.swift
@@ -251,6 +251,8 @@ struct PermissionGateRow: View {
     }
 }
 
-#Preview {
-    PermissionsGateView()
+struct PermissionsGateView_Previews: PreviewProvider {
+    static var previews: some View {
+        PermissionsGateView()
+    }
 }
diff --git a/Sources/ClickIt/UI/Views/WindowDetectionTestView.swift b/Sources/ClickIt/UI/Views/WindowDetectionTestView.swift

Original file line number	Diff line number	Diff line change
`@@ -46,8 +46,10 @@ struct AdvancedSettingsButton: View {`
`46`	`46`	`}`
`47`	`47`	`}`
`48`	`48`
`49`		`-#Preview {`
`50`		`- AdvancedSettingsButton(viewModel: ClickItViewModel())`
`51`		`- .frame(width: 400)`
`52`		`- .padding()`
	`49`	`+struct AdvancedSettingsButton_Previews: PreviewProvider {`
	`50`	`+ static var previews: some View {`
	`51`	`+ AdvancedSettingsButton(viewModel: ClickItViewModel())`
	`52`	`+ .frame(width: 400)`
	`53`	`+ .padding()`
	`54`	`+ }`
`53`	`55`	`}`
Original file line number	Diff line number	Diff line change
`@@ -224,9 +224,11 @@ struct ClickCoordinateCapture {`
`224`	`224`	`}`
`225`	`225`	`}`
`226`	`226`
`227`		`-#Preview {`
`228`		`- ClickPointSelector { point in`
`229`		`- print("Selected point: \(point)")`
	`227`	`+struct ClickPointSelector_Previews: PreviewProvider {`
	`228`	`+ static var previews: some View {`
	`229`	`+ ClickPointSelector { point in`
	`230`	`+ print("Selected point: \(point)")`
	`231`	`+ }`
	`232`	`+ .frame(width: 400, height: 500)`
`230`	`233`	`}`
`231`		`- .frame(width: 400, height: 500)`
`232`	`234`	`}`
Original file line number	Diff line number	Diff line change
`@@ -394,8 +394,10 @@ struct ConfigurationPanel: View {`
`394`	`394`	`}`
`395`	`395`	`}`
`396`	`396`
`397`		`-#Preview {`
`398`		`- ConfigurationPanel(selectedClickPoint: CGPoint(x: 100, y: 100))`
`399`		`- .environmentObject(ClickCoordinator.shared)`
`400`		`- .frame(width: 400, height: 600)`
	`397`	`+struct ConfigurationPanel_Previews: PreviewProvider {`
	`398`	`+ static var previews: some View {`
	`399`	`+ ConfigurationPanel(selectedClickPoint: CGPoint(x: 100, y: 100))`
	`400`	`+ .environmentObject(ClickCoordinator.shared)`
	`401`	`+ .frame(width: 400, height: 600)`
	`402`	`+ }`
`401`	`403`	`}`
Original file line number	Diff line number	Diff line change
`@@ -54,6 +54,8 @@ struct DeveloperUpdateButton: View {`
`54`	`54`	`}`
`55`	`55`	`}`
`56`	`56`
`57`		`-#Preview {`
`58`		`- DeveloperUpdateButton(updaterManager: UpdaterManager())`
	`57`	`+struct DeveloperUpdateButton_Previews: PreviewProvider {`
	`58`	`+ static var previews: some View {`
	`59`	`+ DeveloperUpdateButton(updaterManager: UpdaterManager())`
	`60`	`+ }`
`59`	`61`	`}`
Original file line number	Diff line number	Diff line change
`@@ -29,8 +29,10 @@ struct FooterInfoCard: View {`
`29`	`29`	`}`
`30`	`30`	`}`
`31`	`31`
`32`		`-#Preview {`
`33`		`- FooterInfoCard()`
`34`		`- .frame(width: 400)`
`35`		`- .padding()`
	`32`	`+struct FooterInfoCard_Previews: PreviewProvider {`
	`33`	`+ static var previews: some View {`
	`34`	`+ FooterInfoCard()`
	`35`	`+ .frame(width: 400)`
	`36`	`+ .padding()`
	`37`	`+ }`
`36`	`38`	`}`
Original file line number	Diff line number	Diff line change
`@@ -196,12 +196,14 @@ struct PermissionHealthBadge: View {`
`196`	`196`	`}`
`197`	`197`	`}`
`198`	`198`
`199`		`-#Preview("Permission Status Indicator") {`
`200`		`- VStack(spacing: 16) {`
`201`		`- PermissionStatusIndicator()`
`202`		`- CompactPermissionStatus()`
`203`		`- PermissionHealthBadge()`
	`199`	`+struct PermissionStatusIndicator_Previews: PreviewProvider {`
	`200`	`+ static var previews: some View {`
	`201`	`+ VStack(spacing: 16) {`
	`202`	`+ PermissionStatusIndicator()`
	`203`	`+ CompactPermissionStatus()`
	`204`	`+ PermissionHealthBadge()`
	`205`	`+ }`
	`206`	`+ .padding()`
	`207`	`+ .environmentObject(PermissionManager.shared)`
`204`	`208`	`}`
`205`		`- .padding()`
`206`		`- .environmentObject(PermissionManager.shared)`
`207`	`209`	`}`