Remove unused kind param

Author: rockbruno

Contributor Documentation/BSP Extensions.md

@@ -39,6 +39,11 @@ export interface SourceKitInitializeBuildResponseData {
    * `FileSystemWatcher` is the same as in LSP. */
   watchers?: [FileSystemWatcher]
 }
+
+export interface MultiTargetPreparationSupport {
+  /** Whether the build server can prepare multiple targets in parallel. If this value is omitted, it is assumed to be `true`. */
+  supported: bool;
+}
 ```
 
 ## `build/taskStart`

Sources/BuildServerIntegration/BuildServerManager.swift

@@ -325,10 +325,6 @@ package actor BuildServerManager: QueueBasedMessageHandler {
   /// The build server adapter that is used to answer build server queries.
   private var buildServerAdapter: BuildServerAdapter?
 
-  /// The kind of underlying build server adapter that is being managed.
-  /// `nil` if the `BuildServerManager` does not have an underlying build system.
-  private var kind: BuildServerSpec.Kind?
-
   /// The build server adapter after initialization finishes. When sending messages to the BSP server, this should be
   /// preferred over `buildServerAdapter` because no messages must be sent to the build server before initialization
   /// finishes.
@@ -455,7 +451,6 @@ package actor BuildServerManager: QueueBasedMessageHandler {
     self.toolchainRegistry = toolchainRegistry
     self.options = options
     self.connectionToClient = connectionToClient
-    self.kind = buildServerSpec?.kind
     self.configPath = buildServerSpec?.configPath
     self.buildServerAdapter = await buildServerSpec?.createBuildServerAdapter(
       toolchainRegistry: toolchainRegistry,

Sources/SKTestSupport/CustomBuildServerTestProject.swift

@@ -183,7 +183,7 @@ package extension CustomBuildServer {
         indexStorePath: try projectRoot.appendingPathComponent("index-store").filePath,
         outputPathsProvider: outputPathsProvider,
         prepareProvider: true,
-        sourceKitOptionsProvider: true,
+        sourceKitOptionsProvider: true
       )
     )
   }

Sources/SemanticIndex/SemanticIndexManager.swift

@@ -654,6 +654,7 @@ package final actor SemanticIndexManager {
     guard !targetsToPrepare.isEmpty else {
       return
     }
+
     let taskDescription = AnyIndexTaskDescription(
       PreparationTaskDescription(
         targetsToPrepare: targetsToPrepare,

Sources/SemanticIndex/TaskScheduler.swift

@@ -370,6 +370,7 @@ package actor TaskScheduler<TaskDescription: TaskDescriptionProtocol> {
         // We didn't actually change anything, so we don't need to perform any validation or task processing.
         return
       }
+
       // Check we are over-subscribed in currently executing tasks by walking through all currently executing tasks and
       // checking if we could schedule them within the new execution limits. Cancel any tasks that do not fit within the
       // new limit to be rescheduled when we are within the limit again.

Tests/SourceKitLSPTests/BackgroundIndexingTests.swift

@@ -2624,6 +2624,7 @@ final class BackgroundIndexingTests: XCTestCase {
     let symbols = try await project.testClient.send(WorkspaceSymbolsRequest(query: "myTestFu"))
     XCTAssertEqual(symbols?.count, 1)
   }
+}
 
 extension HoverResponseContents {
   var markupContent: MarkupContent? {