Move more parsing code to DestinationsBundle.swift

Author: MaxDesiatov

Sources/Commands/DestinationTools/ListDestinations.swift

@@ -60,12 +60,7 @@ extension DestinationCommand {
             // Enumerate available bundles and parse manifests for each of them, then validate supplied destinations.
             for bundlePath in destinationBundles {
                 do {
-                    let parsedManifest = try ArtifactsArchiveMetadata.parse(
-                        fileSystem: fileSystem,
-                        rootPath: bundlePath
-                    )
-
-                    let destinationsBundle = try parsedManifest.validateDestinationsBundle(
+                    let destinationsBundle = try DestinationsBundle.parseAndValidate(
                         bundlePath: bundlePath,
                         fileSystem: fileSystem,
                         observabilityScope: observabilityScope

Sources/SPMBuildCore/DestinationsBundle.swift

@@ -25,10 +25,35 @@ public struct DestinationsBundle {
 
     /// Mapping of artifact IDs to variants available for a corresponding artifact.
     public fileprivate(set) var artifacts = [String: [Variant]]()
+
+    /// Parses metadata of an `.artifactbundle` and validates it as a bundle containing
+    /// cross-compilation destinations.
+    /// - Parameters:
+    ///   - bundlePath: path to the bundle root directory.
+    ///   - fileSystem: filesystem containing the bundle.
+    ///   - observabilityScope: observability scope to log validation warnings.
+    /// - Returns: Validated `DestinationsBundle` containing validated `Destination` values for
+    /// each artifact and its variants.
+    public static func parseAndValidate(
+        bundlePath: AbsolutePath,
+        fileSystem: FileSystem,
+        observabilityScope: ObservabilityScope
+    ) throws -> Self {
+        let parsedManifest = try ArtifactsArchiveMetadata.parse(
+            fileSystem: fileSystem,
+            rootPath: bundlePath
+        )
+
+        return try parsedManifest.validateDestinationsBundle(
+            bundlePath: bundlePath,
+            fileSystem: fileSystem,
+            observabilityScope: observabilityScope
+        )
+    }
 }
 
-extension ArtifactsArchiveMetadata {
-    public func validateDestinationsBundle(
+private extension ArtifactsArchiveMetadata {
+    func validateDestinationsBundle(
         bundlePath: AbsolutePath,
         fileSystem: FileSystem,
         observabilityScope: ObservabilityScope