If a build server doesn’t specify -index-store-path in the SourceKit options, add it during background indexing

Author: ahoppen

Documentation/Configuration File.md

@@ -37,8 +37,6 @@ The structure of the file is currently not guaranteed to be stable. Options may
 - `buildSettingsTimeout: integer`: Number of milliseconds to wait for build settings from the build system before using fallback build settings.
 - `clangdOptions: string[]`: Extra command line arguments passed to `clangd` when launching it.
 - `index`: Options related to indexing.
-  - `indexStorePath: string`: Directory in which a separate compilation stores the index store. By default, inferred from the build system.
-  - `indexDatabasePath: string`: Directory in which the indexstore-db should be stored. By default, inferred from the build system.
   - `indexPrefixMap: [string: string]`: Path remappings for remapping index data for local use.
   - `updateIndexStoreTimeout: integer`: Number of seconds to wait for an update index store task to finish before killing it.
 - `logging`: Options related to logging, changing SourceKit-LSP’s logging behavior on non-Apple platforms. On Apple platforms, logging is done through the [system log](Diagnose%20Bundle.md#Enable%20Extended%20Logging). These options can only be set globally and not per workspace.

Sources/BuildSystemIntegration/FixedCompilationDatabaseBuildSystem.swift

@@ -24,10 +24,10 @@ import LanguageServerProtocol
 #endif
 
 func lastIndexStorePathArgument(in compilerArgs: [String]) -> String? {
-  for i in compilerArgs.indices.reversed() {
-    if compilerArgs[i] == "-index-store-path" && i + 1 < compilerArgs.count {
-      return compilerArgs[i + 1]
-    }
+  if let indexStorePathIndex = compilerArgs.lastIndex(of: "-index-store-path"),
+    indexStorePathIndex + 1 < compilerArgs.count
+  {
+    return compilerArgs[indexStorePathIndex + 1]
   }
   return nil
 }

Sources/SemanticIndex/UpdateIndexStoreTaskDescription.swift

@@ -311,7 +311,7 @@ package struct UpdateIndexStoreTaskDescription: IndexTaskDescription {
       return
     }
 
-    let args =
+    var args =
       try [swiftc.filePath] + buildSettings.compilerArguments + [
         "-index-file",
         "-index-file-path", uri.pseudoPath,
@@ -320,6 +320,25 @@ package struct UpdateIndexStoreTaskDescription: IndexTaskDescription {
         // Fake an output path so that we get a different unit file for every Swift file we background index
         "-index-unit-output-path", uri.pseudoPath + ".o",
       ]
+    guard let buildSystemIndexStorePath = await self.buildSystemManager.initializationData?.indexStorePath else {
+      struct NoIndexStorePathError: Error {}
+      throw NoIndexStorePathError()
+    }
+    if let indexStorePathIndex = args.lastIndex(of: "-index-store-path"), indexStorePathIndex + 1 < args.count {
+      let indexStorePath = args[indexStorePathIndex + 1]
+      if indexStorePath != buildSystemIndexStorePath {
+        logger.error(
+          """
+          Compiler arguments for \(uri) specify index store path \(indexStorePath) but build system specified an \
+          incompatible index store path \(buildSystemIndexStorePath). Overriding with the path specified by the build \
+          system.
+          """
+        )
+        args[indexStorePathIndex + 1] = buildSystemIndexStorePath
+      }
+    } else {
+      args += ["-index-store-path", buildSystemIndexStorePath]
+    }
 
     try await runIndexingProcess(
       indexFile: uri,

config.schema.json

@@ -126,11 +126,6 @@
       "description" : "Options related to indexing.",
       "markdownDescription" : "Options related to indexing.",
       "properties" : {
-        "indexDatabasePath" : {
-          "description" : "Directory in which the indexstore-db should be stored. By default, inferred from the build system.",
-          "markdownDescription" : "Directory in which the indexstore-db should be stored. By default, inferred from the build system.",
-          "type" : "string"
-        },
         "indexPrefixMap" : {
           "additionalProperties" : {
             "type" : "string"
@@ -139,11 +134,6 @@
           "markdownDescription" : "Path remappings for remapping index data for local use.",
           "type" : "object"
         },
-        "indexStorePath" : {
-          "description" : "Directory in which a separate compilation stores the index store. By default, inferred from the build system.",
-          "markdownDescription" : "Directory in which a separate compilation stores the index store. By default, inferred from the build system.",
-          "type" : "string"
-        },
         "updateIndexStoreTimeout" : {
           "description" : "Number of seconds to wait for an update index store task to finish before killing it.",
           "markdownDescription" : "Number of seconds to wait for an update index store task to finish before killing it.",