Make SKCore build with strict concurrency

Author: ahoppen

Package.swift

@@ -178,7 +178,8 @@ let package = Package(
         .product(name: "SwiftPMDataModel-auto", package: "swift-package-manager"),
         .product(name: "SwiftToolsSupport-auto", package: "swift-tools-support-core"),
       ],
-      exclude: ["CMakeLists.txt"]
+      exclude: ["CMakeLists.txt"],
+      swiftSettings: [.enableExperimentalFeature("StrictConcurrency")]
     ),
 
     .testTarget(

Sources/SKCore/BuildSetup.swift

@@ -12,11 +12,11 @@
 
 import SKSupport
 
-import struct PackageModel.BuildFlags
+@preconcurrency import struct PackageModel.BuildFlags
 import struct TSCBasic.AbsolutePath
 
 /// Build configuration
-public struct BuildSetup {
+public struct BuildSetup: Sendable {
 
   /// Default configuration
   public static let `default` = BuildSetup(

Sources/SKCore/BuildSystem.swift

@@ -16,7 +16,7 @@ import LanguageServerProtocol
 import struct TSCBasic.AbsolutePath
 
 /// Defines how well a `BuildSystem` can handle a file with a given URI.
-public enum FileHandlingCapability: Comparable {
+public enum FileHandlingCapability: Comparable, Sendable {
   /// The build system can't handle the file at all
   case unhandled
 
@@ -36,7 +36,7 @@ public enum FileHandlingCapability: Comparable {
 ///
 /// For example, a SwiftPMWorkspace provides compiler arguments for the files
 /// contained in a SwiftPM package root directory.
-public protocol BuildSystem: AnyObject {
+public protocol BuildSystem: AnyObject, Sendable {
 
   /// The root of the project that this build system manages. For example, for SwiftPM packages, this is the folder
   /// containing Package.swift. For compilation databases it is the root folder based on which the compilation database

Sources/SKCore/BuildSystemDelegate.swift

@@ -13,7 +13,7 @@ import BuildServerProtocol
 import LanguageServerProtocol
 
 /// Handles build system events, such as file build settings changes.
-public protocol BuildSystemDelegate: AnyObject {
+public protocol BuildSystemDelegate: AnyObject, Sendable {
   /// Notify the delegate that the build targets have changed.
   ///
   /// The callee should request new sources and outputs for the build targets of

Sources/SKCore/CompilationDatabase.swift

@@ -141,7 +141,7 @@ extension FixedCompilationDatabase {
     var fixedArgs: [String] = ["clang"]
     try bytes.withUnsafeData { data in
       guard let fileContents = String(data: data, encoding: .utf8) else {
-        throw CompilationDatabaseDecodingError.fixedDatabaseDecordingError
+        throw CompilationDatabaseDecodingError.fixedDatabaseDecodingError
       }
 
       fileContents.enumerateLines { line, _ in
@@ -238,7 +238,7 @@ extension JSONCompilationDatabase {
 
 enum CompilationDatabaseDecodingError: Error {
   case missingCommandOrArguments
-  case fixedDatabaseDecordingError
+  case fixedDatabaseDecodingError
 }
 
 extension CompilationDatabase.Command: Codable {

Sources/SKCore/FallbackBuildSystem.swift

@@ -16,7 +16,7 @@ import Foundation
 import LanguageServerProtocol
 import SKSupport
 
-import enum PackageLoading.Platform
+@preconcurrency import enum PackageLoading.Platform
 import struct TSCBasic.AbsolutePath
 import class TSCBasic.Process
 

Sources/SKCore/FileBuildSettings.swift

@@ -17,7 +17,7 @@ import LanguageServerProtocol
 ///
 /// Encapsulates all the settings needed to compile a single file, including the compiler arguments
 /// and working directory. FileBuildSettings are typically the result of a BuildSystem query.
-public struct FileBuildSettings: Equatable {
+public struct FileBuildSettings: Equatable, Sendable {
 
   /// The compiler arguments to use for this file.
   public var compilerArguments: [String]

Sources/SKCore/MainFilesProvider.swift

@@ -13,7 +13,7 @@
 import LanguageServerProtocol
 
 /// A type that can provide the set of main files that include a particular file.
-public protocol MainFilesProvider: AnyObject {
+public protocol MainFilesProvider: AnyObject, Sendable {
 
   /// Returns the set of main files that contain the given file.
   ///

Sources/SKCore/PathPrefixMapping.swift

@@ -1,6 +1,6 @@
 import Foundation
 
-public struct PathPrefixMapping {
+public struct PathPrefixMapping: Sendable {
   /// Path prefix to be replaced, typically the canonical or hermetic path.
   public let original: String
 

Sources/SKCore/Toolchain.swift

@@ -14,7 +14,7 @@ import LSPLogging
 import LanguageServerProtocol
 import SKSupport
 
-import enum PackageLoading.Platform
+@preconcurrency import enum PackageLoading.Platform
 import struct TSCBasic.AbsolutePath
 import protocol TSCBasic.FileSystem
 import var TSCBasic.localFileSystem
@@ -114,7 +114,6 @@ public final class Toolchain {
 }
 
 extension Toolchain {
-
   /// Create a toolchain for the given path, if it contains at least one tool, otherwise return nil.
   ///
   /// This initializer looks for a toolchain using the following basic layout: