fix: prevent duplicating interface

Author: Wang Lun

Sources/SourceKitLSP/SourceKitLSPServer.swift

@@ -2065,31 +2065,7 @@ extension SourceKitLSPServer {
     originatorUri: DocumentURI,
     languageService: LanguageService
   ) async throws -> Location {
-    // Check if we're already in the target interface with the same module/group/symbol
-    if case .generatedInterface(let interfaceData) = try? ReferenceDocumentURL(from: originatorUri),
-      interfaceData.moduleName == moduleName && interfaceData.groupName == groupName
-    {
-      // If we have a specific symbol USR, try to find its position in the current interface
-      if let symbolUSR = symbolUSR,
-        let swiftLanguageService = languageService as? SwiftLanguageService
-      {
-        do {
-          let position = try await swiftLanguageService.generatedInterfaceManager.position(
-            ofUsr: symbolUSR,
-            in: interfaceData
-          )
-          return Location(uri: originatorUri, range: Range(position))
-        } catch {
-          // If we can't find the symbol, just return the top of the current interface
-          return Location(uri: originatorUri, range: Range(Position(line: 0, utf16index: 0)))
-        }
-      } else {
-        // No specific symbol, just return the current interface location
-        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 openGeneratedInterface handle all the logic, including checking if we're already in the right interface
     let documentForBuildSettings = originatorUri.buildSettingsFile
 
     guard

Sources/SourceKitLSP/Swift/OpenInterface.swift

@@ -21,10 +21,16 @@ extension SwiftLanguageService {
     groupName: String?,
     symbolUSR symbol: String?
   ) async throws -> GeneratedInterfaceDetails? {
+    // Include build settings context to distinguish different versions/configurations
+    let buildSettingsFileHash = "\(abs(document.buildSettingsFile.stringValue.hashValue))"
+    let sourcekitdDocumentName = [moduleName, groupName, buildSettingsFileHash].compactMap(\.self).joined(
+      separator: "."
+    )
+
     let urlData = GeneratedInterfaceDocumentURLData(
       moduleName: moduleName,
       groupName: groupName,
-      sourcekitdDocumentName: "\(moduleName)-\(UUID())",
+      sourcekitdDocumentName: sourcekitdDocumentName,
       primaryFile: document
     )
     let position: Position? =