Fix --tag parameter to properly push tags and update latest (#168)

Fix --tag parameter to properly push tags and update latest

Motivation
----------

The --tag parameter was accepted by containertool but had no effect on
the
actual tags pushed to the registry. When running:

swift package build-container-image --repository ghcr.io/org/image --tag
v1.0.0

The image was only pushed to `ghcr.io/org/image:latest` (the default
reference
from --repository), while the output misleadingly reported
`ghcr.io/org/image:v1.0.0`.

This made it impossible to publish versioned releases using the --tag
flag.

Modifications
-------------

- Collect all tags to publish: the --tag value (if provided) plus the
default
reference from --repository (typically 'latest')
- Push the manifest and index to each tag in the collection
- Skip duplicate pushes when --tag matches the repository reference
- Return the --tag reference (if provided) as the primary result

Result
------

When --tag is provided, the image is pushed to both the specified tag
AND the
default reference:

  --tag v1.0.0  →  pushes to v1.0.0 and latest
  --tag latest  →  pushes to latest (once, no duplicate)
  (no --tag)    →  pushes to latest (existing behavior)

Test Plan
---------

1. Build with --tag flag: `swift package build-container-image
--repository ghcr.io/org/image --tag v1.0.0`
2. Verify both tags exist: `docker pull ghcr.io/org/image:v1.0.0` and
`docker pull ghcr.io/org/image:latest`
3. Build with --tag latest: verify no errors and single push
4. Build without --tag: verify existing behavior unchanged

Author: the-braveknight

Sources/containertool/Extensions/RegistryClient+publish.swift

@@ -179,56 +179,58 @@ func publishContainerImage<Source: ImageSource, Destination: ImageDestination>(
         )
     }
 
-    // MARK: Upload application manifest
-
-    let manifestDescriptor = try await destination.putManifest(
-        repository: destinationImage.repository,
-        reference: destinationImage.reference,
-        manifest: manifest
-    )
-
-    if verbose {
-        log("manifest: \(manifestDescriptor.digest) (\(manifestDescriptor.size) bytes)")
+    // Determine the tags to push. Always include the reference from --repository (defaults to 'latest').
+    // If --tag is provided and differs from the repository reference, push to both.
+    var tagsToPublish: [any ImageReference.Reference] = [destinationImage.reference]
+    if let tag {
+        let tagReference = try ImageReference.Tag(tag)
+        // Avoid duplicates if --tag matches the reference already in --repository
+        if "\(tagReference)" != "\(destinationImage.reference)" {
+            tagsToPublish.insert(tagReference, at: 0)
+        }
     }
 
-    // MARK: Create application index
+    // MARK: Upload application manifest and index for each tag
 
-    let index = ImageIndex(
-        schemaVersion: 2,
-        mediaType: "application/vnd.oci.image.index.v1+json",
-        manifests: [
-            ContentDescriptor(
-                mediaType: manifestDescriptor.mediaType,
-                digest: manifestDescriptor.digest,
-                size: Int64(manifestDescriptor.size),
-                platform: .init(architecture: architecture, os: os)
-            )
-        ]
-    )
+    for tagReference in tagsToPublish {
+        let manifestDescriptor = try await destination.putManifest(
+            repository: destinationImage.repository,
+            reference: tagReference,
+            manifest: manifest
+        )
 
-    // MARK: Upload application manifest
+        if verbose {
+            log("manifest (\(tagReference)): \(manifestDescriptor.digest) (\(manifestDescriptor.size) bytes)")
+        }
 
-    let indexDescriptor = try await destination.putIndex(
-        repository: destinationImage.repository,
-        reference: destinationImage.reference,
-        index: index
-    )
+        let index = ImageIndex(
+            schemaVersion: 2,
+            mediaType: "application/vnd.oci.image.index.v1+json",
+            manifests: [
+                ContentDescriptor(
+                    mediaType: manifestDescriptor.mediaType,
+                    digest: manifestDescriptor.digest,
+                    size: Int64(manifestDescriptor.size),
+                    platform: .init(architecture: architecture, os: os)
+                )
+            ]
+        )
 
-    if verbose {
-        log("index: \(indexDescriptor.digest) (\(indexDescriptor.size) bytes)")
-    }
+        let indexDescriptor = try await destination.putIndex(
+            repository: destinationImage.repository,
+            reference: tagReference,
+            index: index
+        )
 
-    // Use the index digest if the user did not provide a human-readable tag
-    // To support multiarch images, we should also create an an index pointing to
-    // this manifest.
-    let reference: ImageReference.Reference
-    if let tag {
-        reference = try ImageReference.Tag(tag)
-    } else {
-        reference = try ImageReference.Digest(indexDescriptor.digest)
+        if verbose {
+            log("index (\(tagReference)): \(indexDescriptor.digest) (\(indexDescriptor.size) bytes)")
+        }
     }
 
+    // Return the primary tag (--tag if provided, otherwise the repository reference)
     var result = destinationImage
-    result.reference = reference
+    if let tag {
+        result.reference = try ImageReference.Tag(tag)
+    }
     return result
 }