drop unnecessary change of cachedIndex

Author: Wang Lun

Sources/SourceKitLSP/Swift/GeneratedInterfaceManager.swift

@@ -92,7 +92,13 @@ actor GeneratedInterfaceManager {
     incrementingRefCount: Bool
   ) async throws -> OpenGeneratedInterfaceDocumentDetails {
     func loadFromCache() -> OpenGeneratedInterfaceDocumentDetails? {
-      guard let cachedIndex = openInterfaces.firstIndex(where: { $0.url == document }) else {
+      // Cache by module name and group name, not the full document data
+      // This allows reuse across different buildSettingsFrom URIs
+      guard let cachedIndex = openInterfaces.firstIndex(where: { 
+        $0.url.moduleName == document.moduleName && 
+        $0.url.groupName == document.groupName &&
+        $0.url.sourcekitdDocumentName == document.sourcekitdDocumentName
+      }) else {
         return nil
       }
       if incrementingRefCount {
@@ -160,8 +166,11 @@ actor GeneratedInterfaceManager {
   }
 
   private func decrementRefCount(for document: GeneratedInterfaceDocumentURLData) {
-    guard let cachedIndex = openInterfaces.firstIndex(where: { $0.url == document }) else {
-      logger.fault(
+    guard let cachedIndex = openInterfaces.firstIndex(where: { 
+      $0.url.moduleName == document.moduleName && 
+      $0.url.groupName == document.groupName &&
+      $0.url.sourcekitdDocumentName == document.sourcekitdDocumentName
+    }) else {      logger.fault(
         "Generated interface document for \(document.moduleName) is not open anymore. Unbalanced retain and releases?"
       )
       return