revert cache strategy

Author: Wang Lun

Sources/SourceKitLSP/Swift/GeneratedInterfaceManager.swift

@@ -92,13 +92,7 @@ actor GeneratedInterfaceManager {
     incrementingRefCount: Bool
   ) async throws -> OpenGeneratedInterfaceDocumentDetails {
     func loadFromCache() -> OpenGeneratedInterfaceDocumentDetails? {
-      // 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 {
+      guard let cachedIndex = openInterfaces.firstIndex(where: { $0.url == document }) else {
         return nil
       }
       if incrementingRefCount {
@@ -166,11 +160,7 @@ actor GeneratedInterfaceManager {
   }
 
   private func decrementRefCount(for document: GeneratedInterfaceDocumentURLData) {
-    guard let cachedIndex = openInterfaces.firstIndex(where: { 
-      $0.url.moduleName == document.moduleName && 
-      $0.url.groupName == document.groupName &&
-      $0.url.sourcekitdDocumentName == document.sourcekitdDocumentName
-    }) else {
+    guard let cachedIndex = openInterfaces.firstIndex(where: { $0.url == document }) else {
       logger.fault(
         "Generated interface document for \(document.moduleName) is not open anymore. Unbalanced retain and releases?"
       )