bunset 1.0.6 → 1.0.7

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 1.0.7
+
+### Features
+
+- prompt for commit/tag when not explicitly set
+
+### Bug Fixes
+
+- sync package versions in monorepo when using shared tags
+
 ## 1.0.6
 
 ### Bug Fixes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bunset",
-  "version": "1.0.6",
+  "version": "1.0.7",
   "module": "src/index.ts",
   "type": "module",
   "bin": {

package/src/cli.ts

@@ -92,8 +92,8 @@ export function resolveOptions(
         patch: { type: "boolean", default: false },
         minor: { type: "boolean", default: false },
         major: { type: "boolean", default: false },
-        commit: { type: "boolean", default: true },
-        tag: { type: "boolean", default: true },
+        commit: { type: "boolean" },
+        tag: { type: "boolean" },
         "per-package-tags": { type: "boolean", default: false },
         sections: { type: "string" },
         "dry-run": { type: "boolean", default: false },
@@ -124,8 +124,8 @@ export function resolveOptions(
   const bump = cliBump ?? config.bump ?? null;
   const scope = cliScope ?? config.scope ?? (isWs ? null : "all");
 
-  const commit = values.commit === false ? false : (config.commit ?? true);
-  const tag = values.tag === false ? false : (config.tag ?? true);
+  const commit = values.commit !== undefined ? (values.commit as boolean) : (config.commit ?? null);
+  const tag = values.tag !== undefined ? (values.tag as boolean) : (config.tag ?? null);
   const perPackageTags = values["per-package-tags"]
     ? true
     : (config.perPackageTags ?? false);
@@ -145,7 +145,7 @@ export function resolveOptions(
     ?? config.tagPrefix
     ?? null;
 
-  if (bump && scope) {
+  if (bump && scope && commit !== null && tag !== null) {
     return { scope, bump, commit, tag, perPackageTags, sections, dryRun, filterByPackage, tagPrefix, debug };
   }
 
@@ -185,8 +185,8 @@ function parseSections(raw: string | undefined): CommitType[] | null {
 }
 
 interface MergedDefaults {
-  commit: boolean;
-  tag: boolean;
+  commit: boolean | null;
+  tag: boolean | null;
   perPackageTags: boolean;
   sections: CommitType[];
   dryRun: boolean;
@@ -220,10 +220,26 @@ async function promptForMissing(
     else bump = "patch";
   }
 
+  let { commit, tag } = merged;
+
+  if (commit === null) {
+    process.stdout.write("Commit changes? (y/n) [default: y]: ");
+    const answer = await readLine();
+    commit = answer.trim().toLowerCase() !== "n";
+  }
+
+  if (tag === null) {
+    process.stdout.write("Tag release? (y/n) [default: y]: ");
+    const answer = await readLine();
+    tag = answer.trim().toLowerCase() !== "n";
+  }
+
   return {
     scope: scope ?? "all",
     bump,
     ...merged,
+    commit,
+    tag,
   };
 }
 

package/src/index.ts

@@ -21,7 +21,7 @@ import {
   getAllPackages,
   getChangedPackages,
 } from "./workspace.ts";
-import { bumpVersion, updatePackageVersion } from "./version.ts";
+import { bumpVersion, parseSemver, updatePackageVersion, setPackageVersion } from "./version.ts";
 import type { ParsedCommit, GroupedCommits } from "./types.ts";
 
 const cwd = process.cwd();
@@ -145,6 +145,22 @@ function packageHasChanges(groups: GroupedCommits): boolean {
   return options.sections.some((type) => groups[type].length > 0);
 }
 
+// When using shared tags, sync all packages to the same target version
+let targetVersion: string | null = null;
+if (!options.perPackageTags && packages.length > 1) {
+  const maxVersion = packages.reduce((max, pkg) => {
+    const v = pkg.version ?? "0.0.0";
+    const [mj1, mn1, p1] = parseSemver(max);
+    const [mj2, mn2, p2] = parseSemver(v);
+    if (mj2 > mj1) return v;
+    if (mj2 === mj1 && mn2 > mn1) return v;
+    if (mj2 === mj1 && mn2 === mn1 && p2 > p1) return v;
+    return max;
+  }, "0.0.0");
+  targetVersion = bumpVersion(maxVersion, options.bump);
+  debug(`shared tag mode: max version = ${maxVersion}, target version = ${targetVersion}`);
+}
+
 if (options.dryRun) {
   if (!dbg) console.log("--- Dry Run ---\n");
 
@@ -179,7 +195,7 @@ if (options.dryRun) {
     }
 
     const oldVersion = pkg.version ?? "0.0.0";
-    const newVersion = bumpVersion(oldVersion, options.bump);
+    const newVersion = targetVersion ?? bumpVersion(oldVersion, options.bump);
     console.log(`${pkg.name}: ${oldVersion} → ${newVersion}`);
     filesToCommit.push(pkg.packageJsonPath, `${pkg.path}/CHANGELOG.md`);
 
@@ -212,9 +228,10 @@ if (options.dryRun) {
   filesToCommit.push(`${cwd}/bun.lock`);
 
   if (options.commit) {
+    const releaseVersion = targetVersion ?? bumpVersion(packages[0]!.version ?? "0.0.0", options.bump);
     const msg =
       packages.length === 1
-        ? `chore: release ${packages[0]!.name}@${bumpVersion(packages[0]!.version ?? "0.0.0", options.bump)}`
+        ? `chore: release ${packages[0]!.name}@${releaseVersion}`
         : `chore: release ${packages.length} packages`;
     console.log(`Would commit: ${msg}`);
   } else {
@@ -247,10 +264,9 @@ for (const pkg of packages) {
     continue;
   }
 
-  const { oldVersion, newVersion } = await updatePackageVersion(
-    pkg.packageJsonPath,
-    options.bump,
-  );
+  const { oldVersion, newVersion } = targetVersion
+    ? await setPackageVersion(pkg.packageJsonPath, targetVersion)
+    : await updatePackageVersion(pkg.packageJsonPath, options.bump);
   changedFiles.push(pkg.packageJsonPath);
   console.log(`${pkg.name}: ${oldVersion} → ${newVersion}`);
 

package/src/version.ts

@@ -30,3 +30,15 @@ export async function updatePackageVersion(
   await Bun.write(packageJsonPath, JSON.stringify(pkg, null, 2) + "\n");
   return { oldVersion, newVersion };
 }
+
+export async function setPackageVersion(
+  packageJsonPath: string,
+  newVersion: string,
+): Promise<{ oldVersion: string; newVersion: string }> {
+  const file = Bun.file(packageJsonPath);
+  const pkg = await file.json();
+  const oldVersion = pkg.version ?? "0.0.0";
+  pkg.version = newVersion;
+  await Bun.write(packageJsonPath, JSON.stringify(pkg, null, 2) + "\n");
+  return { oldVersion, newVersion };
+}