Move away from deprecated methods

Author: plemarquand

Sources/PackageMetadata/PackageMetadata.swift

@@ -329,37 +329,32 @@ public struct PackageSearchClient {
         timeout: DispatchTimeInterval? = .none,
         observabilityScope: ObservabilityScope,
         callbackQueue: DispatchQueue,
-        completion: @escaping (Result<Set<PackageIdentity>, Error>) -> Void
+        completion: @Sendable @escaping (Result<Set<PackageIdentity>, Error>) -> Void
     ) {
-        registryClient.lookupIdentities(
-            scmURL: scmURL,
-            timeout: timeout,
-            observabilityScope: observabilityScope,
-            callbackQueue: callbackQueue,
-            completion: completion
-        )
+        callbackQueue.asyncResult(completion) {
+            try await registryClient.lookupIdentities(
+                scmURL: scmURL,
+                timeout: timeout,
+                observabilityScope: observabilityScope
+            )
+        }
     }
 
     public func lookupSCMURLs(
         package: PackageIdentity,
         timeout: DispatchTimeInterval? = .none,
         observabilityScope: ObservabilityScope,
         callbackQueue: DispatchQueue,
-        completion: @escaping (Result<Set<SourceControlURL>, Error>) -> Void
+        completion: @Sendable @escaping (Result<Set<SourceControlURL>, Error>) -> Void
     ) {
-        registryClient.getPackageMetadata(
-            package: package,
-            timeout: timeout,
-            observabilityScope: observabilityScope,
-            callbackQueue: callbackQueue
-        ) { result in
-            do {
-                let metadata = try result.get()
-                let alternateLocations = metadata.alternateLocations
-                return completion(.success(Set(alternateLocations)))
-            } catch {
-                return completion(.failure(error))
-            }
+        callbackQueue.asyncResult(completion) {
+            let metadata = try await registryClient.getPackageMetadata(
+                package: package,
+                timeout: timeout,
+                observabilityScope: observabilityScope
+            )
+            let alternateLocations = metadata.alternateLocations
+            return Set(alternateLocations)
         }
     }
 }