buildSettingFrom can be only file://

Author: Wang Lun

Sources/SourceKitLSP/SourceKitLSPServer.swift

@@ -2086,10 +2086,10 @@ extension SourceKitLSPServer {
         return Location(uri: originatorUri, range: Range(Position(line: 0, utf16index: 0)))
       }
     }
-    
+
     // If the originator URI is already a generated interface, use its primary file for build settings
-    let documentForBuildSettings = originatorUri.primaryFile ?? originatorUri
-    
+    let documentForBuildSettings = originatorUri.buildSettingsFile
+
     guard
       let interfaceDetails = try await languageService.openGeneratedInterface(
         document: documentForBuildSettings,

Sources/SourceKitLSP/Swift/GeneratedInterfaceDocumentURLData.swift

@@ -67,7 +67,14 @@ package struct GeneratedInterfaceDocumentURLData: Hashable, ReferenceURLData {
     self.moduleName = moduleName
     self.groupName = groupName
     self.sourcekitdDocumentName = sourcekitdDocumentName
-    self.buildSettingsFrom = primaryFile
+    if let referenceDocumentURL = try? ReferenceDocumentURL(from: primaryFile),
+       let fileURL = referenceDocumentURL.buildSettingsFile.fileURL {
+      self.buildSettingsFrom = DocumentURI(fileURL)
+    } else if let fileURL = primaryFile.fileURL {
+      self.buildSettingsFrom = DocumentURI(fileURL)
+    } else {
+      self.buildSettingsFrom = primaryFile
+    }
   }
 
   init(queryItems: [URLQueryItem]) throws {

Sources/SourceKitLSP/Swift/ReferenceDocumentURL.swift

@@ -123,8 +123,9 @@ package enum ReferenceDocumentURL {
     switch self {
     case .macroExpansion(let data): return data.primaryFile
     case .generatedInterface(let data):
-      if let referenceDocumentURL = try? ReferenceDocumentURL(from: data.buildSettingsFrom) {
-        return referenceDocumentURL.buildSettingsFile
+      if let referenceDocumentURL = try? ReferenceDocumentURL(from: data.buildSettingsFrom),
+         let fileURL = referenceDocumentURL.buildSettingsFile.fileURL {
+        return DocumentURI(fileURL)
       }
       return data.buildSettingsFrom
     }
@@ -134,10 +135,12 @@ package enum ReferenceDocumentURL {
     switch self {
     case .macroExpansion(let data): return data.primaryFile
     case .generatedInterface(let data):
-      if let referenceDocumentURL = try? ReferenceDocumentURL(from: data.buildSettingsFrom) {
-        return referenceDocumentURL.primaryFile
+      if let referenceDocumentURL = try? ReferenceDocumentURL(from: data.buildSettingsFrom),
+         let primaryFile = referenceDocumentURL.primaryFile,
+         let fileURL = primaryFile.fileURL {
+        return DocumentURI(fileURL)
       }
-    return data.buildSettingsFrom.primaryFile
+      return data.buildSettingsFrom.primaryFile
     }
   }
 }