network@version for single variants

Author: Marketen

src/tasks/createGithubRelease/getNextGitTag.ts

@@ -16,25 +16,35 @@ type GitTagDetailsMap = {
  *
  * - If the package is a single-variant package, the tag will be in the format:
  *  `v0.1.2`
+ *
+ * - If isMultiVariant is true, always use variant@v<version> format, even for a single variant
  */
-export function getNextGitTag(releaseDetailsMap: GitTagDetailsMap): string {
+export function getNextGitTag(releaseDetailsMap: GitTagDetailsMap, isMultiVariant: boolean): string {
   const variantVersions = Object.entries(
     releaseDetailsMap
   ).map(([, { variant, nextVersion }]) => ({ variant, nextVersion }));
 
   if (variantVersions.length === 0)
     throw Error("Could not generate git tag. Missing variant or nextVersion");
 
-  // Not a multi-variant package
-  if (variantVersions.length === 1) return `v${variantVersions[0].nextVersion}`;
-
   // If any variant is null, throw an error
   if (variantVersions.some(({ variant }) => !variant))
     throw Error("Could not generate git tag. Missing variant");
 
-  // Multi-variant package
+  // If isMultiVariant, always use variant@v<version> format, even for a single variant
+  if (isMultiVariant) {
+    return variantVersions
+      .sort((a, b) => (a.variant || "").localeCompare(b.variant || ""))
+      .map(({ variant, nextVersion }) => `${variant}@v${nextVersion}`)
+      .join("_");
+  }
+
+  // Not a multi-variant package
+  if (variantVersions.length === 1) return `v${variantVersions[0].nextVersion}`;
+
+  // Multi-variant package (fallback, should not hit if isMultiVariant is set correctly)
   return variantVersions
-    .sort((a, b) => (a.variant || "").localeCompare(b.variant || "")) // Sort alphabetically by variant
-    .map(({ variant, nextVersion }) => `${variant}@${nextVersion}`) // Map to string
-    .join("_"); // Join into a single string
+    .sort((a, b) => (a.variant || "").localeCompare(b.variant || ""))
+    .map(({ variant, nextVersion }) => `${variant}@${nextVersion}`)
+    .join("_");
 }

src/tasks/createGithubRelease/index.ts

@@ -26,7 +26,7 @@ export function createGithubRelease({
 
   return new Listr<ListrContextPublish>(
     [
-      getHandleTagsTask({ github }),
+      getHandleTagsTask({ github, isMultiVariant }),
       getCreateReleaseTask({
         github,
         composeFileName,

src/tasks/createGithubRelease/subtasks/getCreateReleaseTask.ts

@@ -32,7 +32,7 @@ export function getCreateReleaseTask({
     task: async (ctx, task) => {
       const releaseDetailsMap = buildReleaseDetailsMap(ctx);
 
-      const tag = getNextGitTag(releaseDetailsMap);
+      const tag = getNextGitTag(releaseDetailsMap, isMultiVariant);
 
       task.output = "Deleting existing release...";
       await github.deleteReleaseAndAssets(tag);

src/tasks/createGithubRelease/subtasks/getHandleTagsTask.ts

@@ -14,9 +14,11 @@ import { buildReleaseDetailsMap } from "../buildReleaseDetailsMap.js";
  * tagged and released on that tag
  */
 export function getHandleTagsTask({
-  github
+  github,
+  isMultiVariant
 }: {
   github: Github;
+  isMultiVariant: boolean;
 }): ListrTask<ListrContextPublish> {
   const isCi = process.env.CI;
   const triggerTag = process.env.GITHUB_REF || process.env.TRAVIS_TAG;
@@ -29,7 +31,7 @@ export function getHandleTagsTask({
 
       const releaseDetailsMap = buildReleaseDetailsMap(ctx);
 
-      const tag = getNextGitTag(releaseDetailsMap);
+      const tag = getNextGitTag(releaseDetailsMap, isMultiVariant);
 
       // If the release is triggered in CI,
       // the trigger tag must be removed ("release/patch")