Make BuildServerManager.toolchain only take a target, no document

The document wasn’t used in here.

Author: ahoppen

Sources/BuildServerIntegration/BuildServerManager.swift

@@ -757,16 +757,12 @@ package actor BuildServerManager: QueueBasedMessageHandler {
 
   // MARK: Build server queries
 
-  /// Returns the toolchain that should be used to process the given document.
+  /// Returns the toolchain that should be used to process the given target.
   package func toolchain(
-    for uri: DocumentURI,
-    in target: BuildTargetIdentifier?,
+    for target: BuildTargetIdentifier,
     language: Language
   ) async -> Toolchain? {
     let toolchainPath = await orLog("Getting toolchain from build targets") { () -> URL? in
-      guard let target else {
-        return nil
-      }
       let targets = try await self.buildTargets()
       guard let target = targets[target]?.target else {
         logger.error("Failed to find target \(target.forLogging) to determine toolchain")

Sources/SemanticIndex/UpdateIndexStoreTaskDescription.swift

@@ -280,7 +280,7 @@ package struct UpdateIndexStoreTaskDescription: IndexTaskDescription {
       logger.error("Not indexing \(file.forLogging) because it has fallback compiler arguments")
       return
     }
-    guard let toolchain = await buildServerManager.toolchain(for: file.mainFile, in: target, language: language) else {
+    guard let toolchain = await buildServerManager.toolchain(for: target, language: language) else {
       logger.error(
         "Not updating index store for \(file.forLogging) because no toolchain could be determined for the document"
       )

Sources/SourceKitLSP/SourceKitLSPServer.swift

@@ -567,11 +567,12 @@ package actor SourceKitLSPServer {
       return existingLanguageServices
     }
 
-    let toolchain = await workspace.buildServerManager.toolchain(
-      for: uri,
-      in: workspace.buildServerManager.canonicalTarget(for: uri),
-      language: language
-    )
+    let toolchain: Toolchain? =
+      if let target = await workspace.buildServerManager.canonicalTarget(for: uri) {
+        await workspace.buildServerManager.toolchain(for: target, language: language)
+      } else {
+        nil
+      }
     guard let toolchain else {
       logger.error("Failed to determine toolchain for \(uri)")
       return []