Fixed SwiftLint errors and made a few adjustments

Author: austincondiff

CodeEdit.xcodeproj/project.pbxproj

@@ -3256,7 +3256,7 @@
 			attributes = {
 				BuildIndependentTargetsInParallel = 1;
 				LastSwiftUpdateCheck = 1330;
-				LastUpgradeCheck = 1540;
+				LastUpgradeCheck = 1620;
 				TargetAttributes = {
 					2BE487EB28245162003F3F64 = {
 						CreatedOnToolsVersion = 13.3.1;

CodeEdit/Features/Activities/ActivityManager.swift

@@ -10,6 +10,7 @@ import Combine
 import SwiftUI
 
 /// Manages activities for a workspace
+@MainActor
 final class ActivityManager: ObservableObject {
     /// Currently displayed activities
     @Published private(set) var activities: [CEActivity] = []
@@ -59,13 +60,13 @@ final class ActivityManager: ObservableObject {
     func update(
         id: String,
         title: String? = nil,
-        message: String? = nil, 
+        message: String? = nil,
         percentage: Double? = nil,
         isLoading: Bool? = nil
     ) {
         if let index = activities.firstIndex(where: { $0.id == id }) {
             var activity = activities[index]
-            
+
             if let title = title {
                 activity.title = title
             }
@@ -96,8 +97,9 @@ final class ActivityManager: ObservableObject {
     ///   - id: ID of activity to delete
     ///   - delay: Time to wait before deleting
     func delete(id: String, delay: TimeInterval) {
-        DispatchQueue.main.asyncAfter(deadline: .now() + delay) { [weak self] in
-            self?.delete(id: id)
+        Task { @MainActor in
+            try? await Task.sleep(for: .seconds(delay))
+            delete(id: id)
         }
     }
 }

CodeEdit/Features/ActivityViewer/Models/CEActivity.swift

@@ -1,7 +1,14 @@
+//
+//  CEActivity.swift
+//  CodeEdit
+//
+//  Created by Austin Condiff on 2/20/25.
+//
+
 struct CEActivity: Equatable {
     var id: String
     var title: String
     var message: String?
     var percentage: Double?
     var isLoading: Bool = false
-} 
+}

CodeEdit/Features/Documents/Controllers/CodeEditSplitViewController.swift

@@ -41,7 +41,7 @@ final class CodeEditSplitViewController: NSSplitViewController {
 
     override func viewDidLoad() {
         super.viewDidLoad()
-        guard let windowRef else {
+        guard windowRef != nil else {
             // swiftlint:disable:next line_length
             assertionFailure("No WindowRef found, not initialized properly or the window was dereferenced and the controller was not.")
             return

CodeEdit/Features/Documents/WorkspaceDocument/WorkspaceDocument+Index.swift

@@ -10,18 +10,20 @@ import Foundation
 extension WorkspaceDocument.SearchState {
     /// Adds the contents of the current workspace URL to the search index.
     /// That means that the contents of the workspace will be indexed and searchable.
-    func addProjectToIndex() {
+    func addProjectToIndex() async {
         guard let indexer = indexer else { return }
         guard let url = workspace.fileURL else { return }
 
         indexStatus = .indexing(progress: 0.0)
-        
+
         // Create activity using new API
-        let activity = workspace.activityManager.post(
-            title: "Indexing | Processing files",
-            message: "Creating an index to enable fast and accurate searches within your codebase.",
-            isLoading: true
-        )
+        let activity = await MainActor.run {
+            workspace.activityManager.post(
+                title: "Indexing | Processing files",
+                message: "Creating an index to enable fast and accurate searches within your codebase.",
+                isLoading: true
+            )
+        }
 
         Task.detached {
             let filePaths = self.getFileURLs(at: url)
@@ -39,8 +41,7 @@ extension WorkspaceDocument.SearchState {
                     await MainActor.run {
                         self.indexStatus = .indexing(progress: progress)
                     }
-                    // Update activity using new API
-                    self.workspace.activityManager.update(
+                    await self.workspace.activityManager.update(
                         id: activity.id,
                         percentage: progress
                     )
@@ -51,15 +52,14 @@ extension WorkspaceDocument.SearchState {
             await MainActor.run {
                 self.indexStatus = .done
             }
-            
-            // Update and delete activity using new API
-            self.workspace.activityManager.update(
+
+            await self.workspace.activityManager.update(
                 id: activity.id,
                 title: "Finished indexing",
                 isLoading: false
             )
-            
-            self.workspace.activityManager.delete(
+
+            await self.workspace.activityManager.delete(
                 id: activity.id,
                 delay: 4.0
             )

CodeEdit/Features/Documents/WorkspaceDocument/WorkspaceDocument+SearchState.swift

@@ -47,7 +47,9 @@ extension WorkspaceDocument {
         init(_ workspace: WorkspaceDocument) {
             self.workspace = workspace
             self.indexer = SearchIndexer.Memory.create()
-            addProjectToIndex()
+            Task {
+                await addProjectToIndex()
+            }
         }
 
         /// Represents the compare options to be used for find and replace.

CodeEdit/Features/InspectorArea/InternalDevelopmentInspector/InternalDevelopmentInspectorView.swift

@@ -9,7 +9,7 @@ import SwiftUI
 
 struct InternalDevelopmentInspectorView: View {
     @EnvironmentObject var activityManager: ActivityManager
-    
+
     var body: some View {
         Form {
             InternalDevelopmentActivitiesView()

CodeEditTests/Features/Activities/ActivityManagerTests.swift

@@ -41,7 +41,7 @@ final class ActivityManagerTests: XCTestCase {
 
     func testUpdateTask() {
         let activity = activityManager.post(title: "Task Title")
-        
+
         activityManager.update(
             id: activity.id,
             title: "Updated Task Title"
@@ -53,7 +53,7 @@ final class ActivityManagerTests: XCTestCase {
     func testDeleteTask() {
         let activity = activityManager.post(title: "Task Title")
         activityManager.delete(id: activity.id)
-        
+
         XCTAssertTrue(activityManager.activities.isEmpty)
     }
 
@@ -62,7 +62,7 @@ final class ActivityManagerTests: XCTestCase {
         activityManager.delete(id: activity.id, delay: 0.2)
 
         XCTAssertFalse(activityManager.activities.isEmpty)
-        
+
         let expectation = XCTestExpectation()
         DispatchQueue.main.asyncAfter(deadline: .now() + 0.3) {
             XCTAssertTrue(self.activityManager.activities.isEmpty)