Default multi-target preparation to true

Author: rockbruno

Contributor Documentation/BSP Extensions.md

@@ -41,8 +41,9 @@ export interface SourceKitInitializeBuildResponseData {
 }
 
 export interface MultiTargetPreparationSupport {
-  /** Whether the build server can prepare multiple targets in parallel. */
-  supported: bool;
+  /** Whether the build server can prepare multiple targets in parallel.
+   * If this value is not provided, it will be assumed to be `true`. */
+  supported?: bool;
 
   /** The number of targets to prepare in parallel.
    * If not provided, SourceKit-LSP will calculate an appropriate value based on the environment. */

Sources/BuildServerIntegration/FixedCompilationDatabaseBuildServer.swift

@@ -59,7 +59,7 @@ package actor FixedCompilationDatabaseBuildServer: BuiltInBuildServer {
     indexStorePath?.deletingLastPathComponent().appendingPathComponent("IndexDatabase")
   }
 
-  package nonisolated var supportsMultiTargetPreparation: Bool { false }
+  package nonisolated var supportsMultiTargetPreparation: Bool { true }
 
   package nonisolated var supportsPreparationAndOutputPaths: Bool { false }
 

Sources/BuildServerIntegration/JSONCompilationDatabaseBuildServer.swift

@@ -96,7 +96,7 @@ package actor JSONCompilationDatabaseBuildServer: BuiltInBuildServer {
     indexStorePath?.deletingLastPathComponent().appendingPathComponent("IndexDatabase")
   }
 
-  package nonisolated var supportsMultiTargetPreparation: Bool { false }
+  package nonisolated var supportsMultiTargetPreparation: Bool { true }
 
   package nonisolated var supportsPreparationAndOutputPaths: Bool { false }
 

Sources/BuildServerIntegration/LegacyBuildServer.swift

@@ -138,7 +138,7 @@ actor LegacyBuildServer: MessageHandler, BuiltInBuildServer {
     connectionToSourceKitLSP.send(OnBuildTargetDidChangeNotification(changes: nil))
   }
 
-  package nonisolated var supportsMultiTargetPreparation: Bool { false }
+  package nonisolated var supportsMultiTargetPreparation: Bool { true }
 
   package nonisolated var supportsPreparationAndOutputPaths: Bool { false }
 

Sources/BuildServerProtocol/Messages/InitializeBuildRequest.swift

@@ -373,6 +373,7 @@ public struct SourceKitInitializeBuildResponseData: LSPAnyCodable, Codable, Send
 
 public struct MultiTargetPreparationSupport: LSPAnyCodable, Codable, Sendable {
   /// Whether the build server can prepare multiple targets in parallel.
+  /// If this value is not provided, it will be assumed to be `true`.
   public var supported: Bool?
 
   /// The number of targets to prepare in parallel.

Sources/SourceKitLSP/Workspace.swift

@@ -162,7 +162,7 @@ package final class Workspace: Sendable, BuildServerManagerDelegate {
     if options.backgroundIndexingOrDefault, let uncheckedIndex,
       await buildServerManager.initializationData?.prepareProvider ?? false
     {
-      let shouldIndexInParallel = await buildServerManager.initializationData?.multiTargetPreparation?.supported ?? false
+      let shouldIndexInParallel = await buildServerManager.initializationData?.multiTargetPreparation?.supported ?? true
       let batchSize: Int
       if shouldIndexInParallel {
         if let customBatchSize = await buildServerManager.initializationData?.multiTargetPreparation?.batchSize {