fix: release mechanism with Lerna

Author: jankarres

.github/workflows/ci.yml

@@ -61,6 +61,7 @@ jobs:
         name: LAME install on ${{ matrix.os }}
         runs-on: ${{ matrix.os }}
         needs: quality
+        if: github.ref_name != 'master'
         strategy:
             fail-fast: false
             matrix:

.github/workflows/prepare-release.yml

@@ -72,6 +72,7 @@ jobs:
           SKIP_CHANGELOG: ${{ steps.changelog.outputs.skip }}
         run: |
           set -eo pipefail
+          git checkout -B master ${{ github.event.workflow_run.head_commit.id }}
           pnpm run release:version > version.log 2>&1 || STATUS=$?
           cat version.log
           if [ -n "$STATUS" ]; then
@@ -88,4 +89,4 @@ jobs:
         if: steps.version.outputs.skip != 'true'
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-        run: git push --follow-tags
+        run: git push --set-upstream origin master --follow-tags

.github/workflows/release.yml

@@ -2,7 +2,7 @@ name: Release
 
 on:
     workflow_run:
-        workflows: ["CI"]
+        workflows: ["Prepare Release"]
         types:
             - completed
 

package.json

@@ -79,7 +79,7 @@
     "@typescript-eslint/eslint-plugin": "~8.46.2",
     "@typescript-eslint/parser": "~8.46.2",
     "@vitest/coverage-istanbul": "~4.0.4",
-    "conventional-changelog-cli": "^5.0.0",
+    "conventional-changelog-cli": "~5.0.0",
     "conventional-changelog-conventionalcommits": "~9.1.0",
     "conventional-recommended-bump": "~11.2.0",
     "eslint": "~9.38.0",

pnpm-lock.yaml

@@ -5,6 +5,10 @@ const skipChangelog = process.env.SKIP_CHANGELOG === "1";
 const packageJson = JSON.parse(
     readFileSync(new URL("../package.json", import.meta.url), "utf-8"),
 );
+const semverModule = await import("semver");
+const semver = semverModule.default ?? semverModule;
+const targetVersion = "2.0.0";
+const isBeforeTarget = semver.lt(packageJson.version, targetVersion);
 
 const runCommand = (args) => {
     const result = spawnSync("pnpm", args, {
@@ -21,33 +25,39 @@ const runCommand = (args) => {
 };
 
 if (skipChangelog) {
-    const [{ default: conventionalRecommendedBump }, semver] = await Promise.all(
-        [
-            import("conventional-recommended-bump"),
-            import("semver"),
-        ],
-    );
+    let releaseType = "patch";
 
-    const releaseType = await new Promise((resolve, reject) => {
-        conventionalRecommendedBump(
-            { preset: "conventionalcommits" },
-            (error, recommendation) => {
-                if (error) {
-                    reject(error);
-                    return;
-                }
+    try {
+        const module = await import("conventional-recommended-bump");
+        const candidate =
+            typeof module.default === "function"
+                ? module.default
+                : undefined;
 
-                resolve(recommendation.releaseType ?? "patch");
-            },
+        if (candidate) {
+            const recommendation = await candidate({
+                preset: "conventionalcommits",
+            });
+            releaseType = recommendation?.releaseType ?? "patch";
+        } else {
+            console.warn(
+                "[release] conventional-recommended-bump did not expose a callable default export. Defaulting to patch bump.",
+            );
+        }
+    } catch (error) {
+        console.warn(
+            `[release] Failed to determine recommended bump (${error.message}). Defaulting to patch.`,
         );
-    });
+    }
 
-    const nextVersion = semver.inc(
-        packageJson.version,
-        releaseType ?? "patch",
-    );
-    if (!nextVersion) {
-        throw new Error("Unable to determine next version for release.");
+    let nextVersion =
+        semver.inc(packageJson.version, releaseType) ?? targetVersion;
+
+    if (isBeforeTarget && semver.lt(nextVersion, targetVersion)) {
+        console.log(
+            `[release] Forcing version to ${targetVersion} to begin the 2.x line.`,
+        );
+        nextVersion = targetVersion;
     }
 
     console.log(
@@ -60,18 +70,32 @@ if (skipChangelog) {
         nextVersion,
         "--yes",
         "--no-commit-hooks",
-        "--no-changelog",
-        "--no-conventional-commits",
+        "--conventional-commits",
+        "--no-push",
     ]);
 } else {
     console.log(
         "[release] Using Conventional Commits to generate the changelog.",
     );
 
-    runCommand([
-        "lerna",
-        "version",
-        "--yes",
-        "--no-commit-hooks",
-    ]);
+    if (isBeforeTarget) {
+        console.log(
+            `[release] Forcing version to ${targetVersion} to begin the 2.x line.`,
+        );
+        runCommand([
+            "lerna",
+            "version",
+            targetVersion,
+            "--yes",
+            "--no-commit-hooks",
+            "--conventional-commits",
+        ]);
+    } else {
+        runCommand([
+            "lerna",
+            "version",
+            "--yes",
+            "--no-commit-hooks",
+        ]);
+    }
 }