beachball 2.51.2 → 2.52.0

package/lib/bump/bumpInPlace.js

@@ -28,8 +28,8 @@ function bumpInPlace(bumpInfo, options) {
         }
     }
     // Calculate change types for packages and dependencies
-    for (const { changeFile } of changeFileChangeInfos) {
-        (0, updateRelatedChangeType_1.updateRelatedChangeType)({ changeFile, bumpInfo, dependents, bumpDeps });
+    for (const { change } of changeFileChangeInfos) {
+        (0, updateRelatedChangeType_1.updateRelatedChangeType)({ change, bumpInfo, dependents, bumpDeps });
     }
     // pass 3: actually bump the packages in the bumpInfo in memory (no disk writes at this point)
     Object.keys(calculatedChangeTypes).forEach(pkgName => {

package/lib/bump/bumpInPlace.js.map

@@ -1 +1 @@
-{"version":3,"file":"bumpInPlace.js","sourceRoot":"","sources":["../../src/bump/bumpInPlace.ts"],"names":[],"mappings":";;;AACA,yEAAsE;AACtE,uEAAoE;AACpE,qEAAkE;AAElE,iEAA8D;AAE9D;;;;GAIG;AACH,SAAgB,WAAW,CAAC,QAAkB,EAAE,OAAyB;IACvE,MAAM,EAAE,QAAQ,EAAE,GAAG,OAAO,CAAC;IAC7B,MAAM,EAAE,qBAAqB,EAAE,qBAAqB,EAAE,gBAAgB,EAAE,GAAG,QAAQ,CAAC;IAEpF,0HAA0H;IAC1H,MAAM,UAAU,GAAG,QAAQ,CAAC,CAAC,CAAC,IAAA,mDAAwB,EAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC;IAEtE,0GAA0G;IAC1G,sFAAsF;IACtF,kFAAkF;IAElF,4DAA4D;IAC5D,KAAK,MAAM,EAAE,MAAM,EAAE,UAAU,EAAE,IAAI,qBAAqB,EAAE;QAC1D,MAAM,KAAK,GAAG,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,aAAa,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,YAAY,CAAC,QAAQ,CAAC,UAAU,CAAC,WAAW,CAAC,CAAC,CAAC;QAE/G,IAAI,KAAK,EAAE;YACT,KAAK,MAAM,kBAAkB,IAAI,KAAK,CAAC,YAAY,EAAE;gBACnD,qBAAqB,CAAC,kBAAkB,CAAC,GAAG,UAAU,CAAC,IAAI,CAAC;aAC7D;SACF;KACF;IAED,uDAAuD;IACvD,KAAK,MAAM,EAAE,UAAU,EAAE,IAAI,qBAAqB,EAAE;QAClD,IAAA,iDAAuB,EAAC,EAAE,UAAU,EAAE,QAAQ,EAAE,UAAU,EAAE,QAAQ,EAAE,CAAC,CAAC;KACzE;IAED,8FAA8F;IAC9F,MAAM,CAAC,IAAI,CAAC,qBAAqB,CAAC,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE;QACnD,IAAA,+CAAsB,EAAC,OAAO,EAAE,QAAQ,EAAE,OAAO,CAAC,CAAC;IACrD,CAAC,CAAC,CAAC;IAEH,6CAA6C;IAC7C,QAAQ,CAAC,kBAAkB,GAAG,IAAA,2CAAoB,EAAC,QAAQ,EAAE,OAAO,CAAC,CAAC;IACtE,MAAM,CAAC,IAAI,CAAC,QAAQ,CAAC,kBAAkB,CAAC,CAAC,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC,gBAAgB,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC,CAAC;AACrF,CAAC;AAnCD,kCAmCC"}
+{"version":3,"file":"bumpInPlace.js","sourceRoot":"","sources":["../../src/bump/bumpInPlace.ts"],"names":[],"mappings":";;;AACA,yEAAsE;AACtE,uEAAoE;AACpE,qEAAkE;AAElE,iEAA8D;AAE9D;;;;GAIG;AACH,SAAgB,WAAW,CAAC,QAAkB,EAAE,OAAyB;IACvE,MAAM,EAAE,QAAQ,EAAE,GAAG,OAAO,CAAC;IAC7B,MAAM,EAAE,qBAAqB,EAAE,qBAAqB,EAAE,gBAAgB,EAAE,GAAG,QAAQ,CAAC;IAEpF,0HAA0H;IAC1H,MAAM,UAAU,GAAG,QAAQ,CAAC,CAAC,CAAC,IAAA,mDAAwB,EAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC;IAEtE,0GAA0G;IAC1G,sFAAsF;IACtF,kFAAkF;IAElF,4DAA4D;IAC5D,KAAK,MAAM,EAAE,MAAM,EAAE,UAAU,EAAE,IAAI,qBAAqB,EAAE;QAC1D,MAAM,KAAK,GAAG,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,aAAa,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,YAAY,CAAC,QAAQ,CAAC,UAAU,CAAC,WAAW,CAAC,CAAC,CAAC;QAE/G,IAAI,KAAK,EAAE;YACT,KAAK,MAAM,kBAAkB,IAAI,KAAK,CAAC,YAAY,EAAE;gBACnD,qBAAqB,CAAC,kBAAkB,CAAC,GAAG,UAAU,CAAC,IAAI,CAAC;aAC7D;SACF;KACF;IAED,uDAAuD;IACvD,KAAK,MAAM,EAAE,MAAM,EAAE,IAAI,qBAAqB,EAAE;QAC9C,IAAA,iDAAuB,EAAC,EAAE,MAAM,EAAE,QAAQ,EAAE,UAAU,EAAE,QAAQ,EAAE,CAAC,CAAC;KACrE;IAED,8FAA8F;IAC9F,MAAM,CAAC,IAAI,CAAC,qBAAqB,CAAC,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE;QACnD,IAAA,+CAAsB,EAAC,OAAO,EAAE,QAAQ,EAAE,OAAO,CAAC,CAAC;IACrD,CAAC,CAAC,CAAC;IAEH,6CAA6C;IAC7C,QAAQ,CAAC,kBAAkB,GAAG,IAAA,2CAAoB,EAAC,QAAQ,EAAE,OAAO,CAAC,CAAC;IACtE,MAAM,CAAC,IAAI,CAAC,QAAQ,CAAC,kBAAkB,CAAC,CAAC,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC,gBAAgB,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC,CAAC;AACrF,CAAC;AAnCD,kCAmCC"}

package/lib/bump/gatherBumpInfo.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"gatherBumpInfo.d.ts","sourceRoot":"","sources":["../../src/bump/gatherBumpInfo.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,mBAAmB,CAAC;AAElD,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,2BAA2B,CAAC;AAElE,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,sBAAsB,CAAC;AAGzD;;GAEG;AACH,wBAAgB,cAAc,CAAC,OAAO,EAAE,gBAAgB,EAAE,YAAY,EAAE,YAAY,GAAG,QAAQ,CAyB9F"}
+{"version":3,"file":"gatherBumpInfo.d.ts","sourceRoot":"","sources":["../../src/bump/gatherBumpInfo.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,mBAAmB,CAAC;AAElD,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,2BAA2B,CAAC;AAElE,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,sBAAsB,CAAC;AAGzD;;GAEG;AACH,wBAAgB,cAAc,CAAC,OAAO,EAAE,gBAAgB,EAAE,YAAY,EAAE,YAAY,GAAG,QAAQ,CAmB9F"}

package/lib/bump/gatherBumpInfo.js

@@ -10,15 +10,9 @@ const getPackageGroups_1 = require("../monorepo/getPackageGroups");
  * Gather bump info and bump versions in memory.
  */
 function gatherBumpInfo(options, packageInfos) {
-    // Collate the changes per package
     const changes = (0, readChangeFiles_1.readChangeFiles)(options, packageInfos);
-    // Clear non-existent packages from changefiles infos
+    // Determine base change types for each package (not considering disallowedChangeTypes or groups)
     const calculatedChangeTypes = (0, changeTypes_1.initializePackageChangeTypes)(changes);
-    Object.keys(calculatedChangeTypes).forEach(packageName => {
-        if (!packageInfos[packageName]) {
-            delete calculatedChangeTypes[packageName];
-        }
-    });
     const bumpInfo = {
         calculatedChangeTypes,
         packageInfos,

package/lib/bump/gatherBumpInfo.js.map

@@ -1 +1 @@
-{"version":3,"file":"gatherBumpInfo.js","sourceRoot":"","sources":["../../src/bump/gatherBumpInfo.ts"],"names":[],"mappings":";;;AAAA,2DAAyE;AACzE,mEAAgE;AAEhE,+CAA4C;AAE5C,qEAAkE;AAElE,mEAAgE;AAEhE;;GAEG;AACH,SAAgB,cAAc,CAAC,OAAyB,EAAE,YAA0B;IAClF,kCAAkC;IAClC,MAAM,OAAO,GAAG,IAAA,iCAAe,EAAC,OAAO,EAAE,YAAY,CAAC,CAAC;IAEvD,qDAAqD;IACrD,MAAM,qBAAqB,GAAG,IAAA,0CAA4B,EAAC,OAAO,CAAC,CAAC;IACpE,MAAM,CAAC,IAAI,CAAC,qBAAqB,CAAC,CAAC,OAAO,CAAC,WAAW,CAAC,EAAE;QACvD,IAAI,CAAC,YAAY,CAAC,WAAW,CAAC,EAAE;YAC9B,OAAO,qBAAqB,CAAC,WAAW,CAAC,CAAC;SAC3C;IACH,CAAC,CAAC,CAAC;IAEH,MAAM,QAAQ,GAAa;QACzB,qBAAqB;QACrB,YAAY;QACZ,aAAa,EAAE,IAAA,mCAAgB,EAAC,YAAY,EAAE,OAAO,CAAC,IAAI,EAAE,OAAO,CAAC,MAAM,CAAC;QAC3E,qBAAqB,EAAE,OAAO;QAC9B,gBAAgB,EAAE,IAAI,GAAG,EAAU;QACnC,cAAc,EAAE,IAAI,GAAG,CAAC,IAAA,qCAAiB,EAAC,OAAO,EAAE,YAAY,CAAC,CAAC;QACjE,kBAAkB,EAAE,EAAE;KACvB,CAAC;IAEF,IAAA,yBAAW,EAAC,QAAQ,EAAE,OAAO,CAAC,CAAC;IAE/B,OAAO,QAAQ,CAAC;AAClB,CAAC;AAzBD,wCAyBC"}
+{"version":3,"file":"gatherBumpInfo.js","sourceRoot":"","sources":["../../src/bump/gatherBumpInfo.ts"],"names":[],"mappings":";;;AAAA,2DAAyE;AACzE,mEAAgE;AAEhE,+CAA4C;AAE5C,qEAAkE;AAElE,mEAAgE;AAEhE;;GAEG;AACH,SAAgB,cAAc,CAAC,OAAyB,EAAE,YAA0B;IAClF,MAAM,OAAO,GAAG,IAAA,iCAAe,EAAC,OAAO,EAAE,YAAY,CAAC,CAAC;IAEvD,iGAAiG;IACjG,MAAM,qBAAqB,GAAG,IAAA,0CAA4B,EAAC,OAAO,CAAC,CAAC;IAEpE,MAAM,QAAQ,GAAa;QACzB,qBAAqB;QACrB,YAAY;QACZ,aAAa,EAAE,IAAA,mCAAgB,EAAC,YAAY,EAAE,OAAO,CAAC,IAAI,EAAE,OAAO,CAAC,MAAM,CAAC;QAC3E,qBAAqB,EAAE,OAAO;QAC9B,gBAAgB,EAAE,IAAI,GAAG,EAAU;QACnC,cAAc,EAAE,IAAI,GAAG,CAAC,IAAA,qCAAiB,EAAC,OAAO,EAAE,YAAY,CAAC,CAAC;QACjE,kBAAkB,EAAE,EAAE;KACvB,CAAC;IAEF,IAAA,yBAAW,EAAC,QAAQ,EAAE,OAAO,CAAC,CAAC;IAE/B,OAAO,QAAQ,CAAC;AAClB,CAAC;AAnBD,wCAmBC"}

package/lib/bump/updateRelatedChangeType.d.ts

@@ -1,25 +1,31 @@
 import type { BumpInfo, PackageDependents } from '../types/BumpInfo';
+import { ChangeFileInfo } from '../types/ChangeInfo';
 /**
- * This is the core of the bumpInfo dependency bumping logic - done once per change file
+ * This is the core of the `bumpInfo` dependency bumping logic - done once per change info.
  *
- * The algorithm is an iterative graph traversal algorithm (breadth first)
- * - it searches up the parent `dependents` and modifies the "calculatedChangeTypes" entries inside `BumpInfo`
- * - one single root entry from `pkgName` as given by a change file
- * - for all dependents
- *   - apply the `dependentChangeType` as change type
- * - this function is the primary way for package groups to get the same dependent change type by queueing up
- *   all packages within a group to be visited by the loop
+ * The algorithm is an iterative graph traversal algorithm (breadth first):
+ * - One root entry: `change.packageName`
+ * - For each parent in `dependents`:
+ *   - Update its `calculatedChangeTypes` entry to `max(current value, change.dependentChangeType)`
+ *   - Enqueue all of its dependents to be seen
+ * - If the package is part of a group, enqueue all packages in the group
+ *
+ * Preconditions:
+ * - `bumpInfo.calculatedChangeTypes` includes:
+ *   - For non-grouped packages that have changed, the highest change type from any change file
+ *   - For each grouped package where anything in the group has changed, the highest change type
+ *     from any change file in the group
  *
  * What it mutates:
- * - bumpInfo.calculatedChangeTypes: updates packages change type modifed by this function
- * - all dependents change types as part of a group update
+ * - `bumpInfo.calculatedChangeTypes`: updates dependents' change types
+ * - all dependents' change types as part of a group update
  *
  * What it does not do:
- * - bumpInfo.calculatedChangeTypes: will not mutate the entryPoint `pkgName` change type
+ * - `bumpInfo.calculatedChangeTypes`: will not mutate the `change.packageName` change type
  */
 export declare function updateRelatedChangeType(params: {
-    changeFile: string;
-    bumpInfo: Pick<BumpInfo, 'calculatedChangeTypes' | 'changeFileChangeInfos' | 'packageGroups' | 'packageInfos'>;
+    change: ChangeFileInfo;
+    bumpInfo: Pick<BumpInfo, 'calculatedChangeTypes' | 'packageGroups' | 'packageInfos'>;
     dependents: PackageDependents;
     bumpDeps: boolean;
 }): void;

package/lib/bump/updateRelatedChangeType.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"updateRelatedChangeType.d.ts","sourceRoot":"","sources":["../../src/bump/updateRelatedChangeType.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,QAAQ,EAAE,iBAAiB,EAAE,MAAM,mBAAmB,CAAC;AAErE;;;;;;;;;;;;;;;;;GAiBG;AACH,wBAAgB,uBAAuB,CAAC,MAAM,EAAE;IAC9C,UAAU,EAAE,MAAM,CAAC;IACnB,QAAQ,EAAE,IAAI,CAAC,QAAQ,EAAE,uBAAuB,GAAG,uBAAuB,GAAG,eAAe,GAAG,cAAc,CAAC,CAAC;IAC/G,UAAU,EAAE,iBAAiB,CAAC;IAC9B,QAAQ,EAAE,OAAO,CAAC;CACnB,GAAG,IAAI,CAqFP"}
+{"version":3,"file":"updateRelatedChangeType.d.ts","sourceRoot":"","sources":["../../src/bump/updateRelatedChangeType.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,QAAQ,EAAE,iBAAiB,EAAE,MAAM,mBAAmB,CAAC;AACrE,OAAO,EAAE,cAAc,EAAE,MAAM,qBAAqB,CAAC;AAErD;;;;;;;;;;;;;;;;;;;;;;GAsBG;AACH,wBAAgB,uBAAuB,CAAC,MAAM,EAAE;IAC9C,MAAM,EAAE,cAAc,CAAC;IACvB,QAAQ,EAAE,IAAI,CAAC,QAAQ,EAAE,uBAAuB,GAAG,eAAe,GAAG,cAAc,CAAC,CAAC;IACrF,UAAU,EAAE,iBAAiB,CAAC;IAC9B,QAAQ,EAAE,OAAO,CAAC;CACnB,GAAG,IAAI,CAsEP"}

package/lib/bump/updateRelatedChangeType.js

@@ -3,81 +3,78 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.updateRelatedChangeType = void 0;
 const changeTypes_1 = require("../changefile/changeTypes");
 /**
- * This is the core of the bumpInfo dependency bumping logic - done once per change file
+ * This is the core of the `bumpInfo` dependency bumping logic - done once per change info.
  *
- * The algorithm is an iterative graph traversal algorithm (breadth first)
- * - it searches up the parent `dependents` and modifies the "calculatedChangeTypes" entries inside `BumpInfo`
- * - one single root entry from `pkgName` as given by a change file
- * - for all dependents
- *   - apply the `dependentChangeType` as change type
- * - this function is the primary way for package groups to get the same dependent change type by queueing up
- *   all packages within a group to be visited by the loop
+ * The algorithm is an iterative graph traversal algorithm (breadth first):
+ * - One root entry: `change.packageName`
+ * - For each parent in `dependents`:
+ *   - Update its `calculatedChangeTypes` entry to `max(current value, change.dependentChangeType)`
+ *   - Enqueue all of its dependents to be seen
+ * - If the package is part of a group, enqueue all packages in the group
+ *
+ * Preconditions:
+ * - `bumpInfo.calculatedChangeTypes` includes:
+ *   - For non-grouped packages that have changed, the highest change type from any change file
+ *   - For each grouped package where anything in the group has changed, the highest change type
+ *     from any change file in the group
  *
  * What it mutates:
- * - bumpInfo.calculatedChangeTypes: updates packages change type modifed by this function
- * - all dependents change types as part of a group update
+ * - `bumpInfo.calculatedChangeTypes`: updates dependents' change types
+ * - all dependents' change types as part of a group update
  *
  * What it does not do:
- * - bumpInfo.calculatedChangeTypes: will not mutate the entryPoint `pkgName` change type
+ * - `bumpInfo.calculatedChangeTypes`: will not mutate the `change.packageName` change type
  */
 function updateRelatedChangeType(params) {
-    const { changeFile, bumpInfo, dependents, bumpDeps } = params;
-    const { calculatedChangeTypes, changeFileChangeInfos, packageGroups, packageInfos } = bumpInfo;
-    for (const info of changeFileChangeInfos) {
-        if (info.changeFile !== changeFile) {
-            continue;
-        }
-        const { change: { packageName: entryPointPackageName, dependentChangeType }, } = info;
-        // Do not do anything if packageInfo is not present: it means this was an invalid changefile that
-        // somehow got checked in. (This should have already been caught by readChangeFiles, but just in case.)
-        if (!packageInfos[entryPointPackageName]) {
-            continue;
-        }
-        const updatedChangeType = (0, changeTypes_1.getMaxChangeType)(dependentChangeType, changeTypes_1.MinChangeType);
-        const queue = [{ subjectPackage: entryPointPackageName, changeType: changeTypes_1.MinChangeType }];
-        // visited is a set of package names that already has been seen by this algorithm - this allows the algo to scale
-        const visited = new Set();
-        while (queue.length > 0) {
-            const { subjectPackage, changeType } = queue.shift();
-            // Step 1. Update change type of the subjectPackage according to the dependent change type propagation
-            const packageInfo = packageInfos[subjectPackage];
-            if (!packageInfo) {
+    const { change, bumpInfo, dependents, bumpDeps } = params;
+    const { calculatedChangeTypes, packageGroups, packageInfos } = bumpInfo;
+    // If dependentChangeType is none (or somehow unset), there's nothing to do.
+    const updatedChangeType = (0, changeTypes_1.getMaxChangeType)(change.dependentChangeType);
+    if (updatedChangeType === 'none') {
+        return;
+    }
+    const entryPointPackageName = change.packageName;
+    // Enqueue the first package.
+    // This part of the bump algorithm is a performance bottleneck, so it's important to bail early
+    // whenever possible, and to use `seen` to reduce queue insertion.
+    // https://github.com/microsoft/beachball/pull/1042
+    const queue = [{ subjectPackage: entryPointPackageName, changeType: changeTypes_1.MinChangeType }];
+    const seen = new Set();
+    while (queue.length > 0) {
+        const { subjectPackage, changeType } = queue.shift();
+        // Step 1. Update change type of the subjectPackage according to dependentChangeType if needed.
+        if (subjectPackage !== entryPointPackageName) {
+            const oldType = calculatedChangeTypes[subjectPackage];
+            calculatedChangeTypes[subjectPackage] = (0, changeTypes_1.getMaxChangeType)(oldType, changeType, packageInfos[subjectPackage]?.combinedOptions?.disallowedChangeTypes);
+            if (calculatedChangeTypes[subjectPackage] === oldType) {
+                // We didn't change this type, so keep going.
                 continue;
             }
-            const disallowedChangeTypes = packageInfo.combinedOptions?.disallowedChangeTypes ?? [];
-            if (subjectPackage !== entryPointPackageName) {
-                const oldType = calculatedChangeTypes[subjectPackage];
-                calculatedChangeTypes[subjectPackage] = (0, changeTypes_1.getMaxChangeType)(oldType, changeType, disallowedChangeTypes);
-                // We didn't change this type, so keep going.
-                if (calculatedChangeTypes[subjectPackage] === oldType) {
+        }
+        // Step 2. For all dependent packages of the current subjectPackage, place in queue to be updated at least to the "updatedChangeType"
+        const dependentPackages = dependents[subjectPackage];
+        if (bumpDeps && dependentPackages?.length) {
+            for (const dependentPackage of dependentPackages) {
+                if (seen.has(dependentPackage)) {
                     continue;
                 }
+                seen.add(dependentPackage);
+                queue.push({ subjectPackage: dependentPackage, changeType: updatedChangeType });
             }
-            // Step 2. For all dependent packages of the current subjectPackage, place in queue to be updated at least to the "updatedChangeType"
-            const dependentPackages = dependents[subjectPackage];
-            if (bumpDeps && dependentPackages?.length) {
-                for (const dependentPackage of dependentPackages) {
-                    if (visited.has(dependentPackage)) {
-                        continue;
-                    }
-                    visited.add(dependentPackage);
-                    queue.push(({ subjectPackage: dependentPackage, changeType: updatedChangeType }));
-                }
-            }
-            // TODO: when we do "locked", or "lock step" versioning, we could simply skip this grouped traversal,
-            //       - set the version for all packages in the group in bumpPackageInfoVersion()
-            //       - the main concern is how to capture the bump reason in grouped changelog
-            // Step 3. For group-linked packages, update the change type to the max(change file info's change type, propagated update change type)
-            const group = Object.values(packageGroups).find(g => g.packageNames.includes(packageInfo.name));
-            if (group) {
-                for (const packageNameInGroup of group.packageNames) {
-                    if (!group.disallowedChangeTypes?.includes(updatedChangeType) && !visited.has(packageNameInGroup)) {
-                        visited.add(packageNameInGroup);
-                        queue.push({
-                            subjectPackage: packageNameInGroup,
-                            changeType: updatedChangeType,
-                        });
-                    }
+        }
+        // TODO: when we do "locked", or "lock step" versioning, we could simply skip this grouped traversal,
+        //       - set the version for all packages in the group in bumpPackageInfoVersion()
+        //       - the main concern is how to capture the bump reason in grouped changelog
+        // Step 3. For group-linked packages, update the change type to the max(change file info's change type, propagated update change type)
+        const group = Object.values(packageGroups).find(g => g.packageNames.includes(subjectPackage));
+        if (group && !group.disallowedChangeTypes?.includes(updatedChangeType)) {
+            for (const packageNameInGroup of group.packageNames) {
+                if (!seen.has(packageNameInGroup)) {
+                    seen.add(packageNameInGroup);
+                    queue.push({
+                        subjectPackage: packageNameInGroup,
+                        changeType: updatedChangeType,
+                    });
                 }
             }
         }

package/lib/bump/updateRelatedChangeType.js.map

@@ -1 +1 @@
-{"version":3,"file":"updateRelatedChangeType.js","sourceRoot":"","sources":["../../src/bump/updateRelatedChangeType.ts"],"names":[],"mappings":";;;AAAA,2DAA4E;AAG5E;;;;;;;;;;;;;;;;;GAiBG;AACH,SAAgB,uBAAuB,CAAC,MAKvC;IACC,MAAM,EAAE,UAAU,EAAE,QAAQ,EAAE,UAAU,EAAE,QAAQ,EAAE,GAAG,MAAM,CAAC;IAC9D,MAAM,EAAE,qBAAqB,EAAE,qBAAqB,EAAE,aAAa,EAAE,YAAY,EAAE,GAAG,QAAQ,CAAC;IAE/F,KAAK,MAAM,IAAI,IAAI,qBAAqB,EAAE;QACxC,IAAI,IAAI,CAAC,UAAU,KAAK,UAAU,EAAE;YAClC,SAAS;SACV;QAED,MAAM,EACJ,MAAM,EAAE,EAAE,WAAW,EAAE,qBAAqB,EAAE,mBAAmB,EAAE,GACpE,GAAG,IAAI,CAAC;QAET,iGAAiG;QACjG,uGAAuG;QACvG,IAAI,CAAC,YAAY,CAAC,qBAAqB,CAAC,EAAE;YACxC,SAAS;SACV;QAED,MAAM,iBAAiB,GAAG,IAAA,8BAAgB,EAAC,mBAAmB,EAAE,2BAAa,CAAC,CAAC;QAE/E,MAAM,KAAK,GAAG,CAAC,EAAE,cAAc,EAAE,qBAAqB,EAAE,UAAU,EAAE,2BAAa,EAAE,CAAC,CAAC;QAErF,iHAAiH;QACjH,MAAM,OAAO,GAAG,IAAI,GAAG,EAAU,CAAC;QAElC,OAAO,KAAK,CAAC,MAAM,GAAG,CAAC,EAAE;YACvB,MAAM,EAAE,cAAc,EAAE,UAAU,EAAE,GAAG,KAAK,CAAC,KAAK,EAAG,CAAC;YAEtD,sGAAsG;YACtG,MAAM,WAAW,GAAG,YAAY,CAAC,cAAc,CAAC,CAAC;YACjD,IAAI,CAAC,WAAW,EAAE;gBAChB,SAAS;aACV;YAED,MAAM,qBAAqB,GAAG,WAAW,CAAC,eAAe,EAAE,qBAAqB,IAAI,EAAE,CAAC;YAEvF,IAAI,cAAc,KAAK,qBAAqB,EAAE;gBAC5C,MAAM,OAAO,GAAG,qBAAqB,CAAC,cAAc,CAAC,CAAC;gBACtD,qBAAqB,CAAC,cAAc,CAAC,GAAG,IAAA,8BAAgB,EACtD,OAAO,EACP,UAAU,EACV,qBAAqB,CACtB,CAAC;gBAEF,6CAA6C;gBAC7C,IAAI,qBAAqB,CAAC,cAAc,CAAC,KAAK,OAAO,EAAE;oBACrD,SAAS;iBACV;aACF;YAED,qIAAqI;YACrI,MAAM,iBAAiB,GAAG,UAAU,CAAC,cAAc,CAAC,CAAC;YAErD,IAAI,QAAQ,IAAI,iBAAiB,EAAE,MAAM,EAAE;gBACzC,KAAK,MAAM,gBAAgB,IAAI,iBAAiB,EAAE;oBAChD,IAAI,OAAO,CAAC,GAAG,CAAC,gBAAgB,CAAC,EAAE;wBACjC,SAAS;qBACV;oBAED,OAAO,CAAC,GAAG,CAAC,gBAAgB,CAAC,CAAC;oBAC9B,KAAK,CAAC,IAAI,CAAC,CAAC,EAAE,cAAc,EAAE,gBAAgB,EAAE,UAAU,EAAE,iBAAiB,EAAE,CAAC,CAAC,CAAC;iBACnF;aACF;YAED,qGAAqG;YACrG,oFAAoF;YACpF,kFAAkF;YAElF,sIAAsI;YACtI,MAAM,KAAK,GAAG,MAAM,CAAC,MAAM,CAAC,aAAa,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,YAAY,CAAC,QAAQ,CAAC,WAAW,CAAC,IAAI,CAAC,CAAC,CAAC;YAEhG,IAAI,KAAK,EAAE;gBACT,KAAK,MAAM,kBAAkB,IAAI,KAAK,CAAC,YAAY,EAAE;oBACnD,IAAI,CAAC,KAAK,CAAC,qBAAqB,EAAE,QAAQ,CAAC,iBAAiB,CAAC,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,kBAAkB,CAAC,EAAE;wBACjG,OAAO,CAAC,GAAG,CAAC,kBAAkB,CAAC,CAAC;wBAChC,KAAK,CAAC,IAAI,CAAC;4BACT,cAAc,EAAE,kBAAkB;4BAClC,UAAU,EAAE,iBAAiB;yBAC9B,CAAC,CAAC;qBACJ;iBACF;aACF;SACF;KACF;AACH,CAAC;AA1FD,0DA0FC"}
+{"version":3,"file":"updateRelatedChangeType.js","sourceRoot":"","sources":["../../src/bump/updateRelatedChangeType.ts"],"names":[],"mappings":";;;AAAA,2DAA4E;AAI5E;;;;;;;;;;;;;;;;;;;;;;GAsBG;AACH,SAAgB,uBAAuB,CAAC,MAKvC;IACC,MAAM,EAAE,MAAM,EAAE,QAAQ,EAAE,UAAU,EAAE,QAAQ,EAAE,GAAG,MAAM,CAAC;IAC1D,MAAM,EAAE,qBAAqB,EAAE,aAAa,EAAE,YAAY,EAAE,GAAG,QAAQ,CAAC;IAExE,4EAA4E;IAC5E,MAAM,iBAAiB,GAAG,IAAA,8BAAgB,EAAC,MAAM,CAAC,mBAAmB,CAAC,CAAC;IACvE,IAAI,iBAAiB,KAAK,MAAM,EAAE;QAChC,OAAO;KACR;IAED,MAAM,qBAAqB,GAAG,MAAM,CAAC,WAAW,CAAC;IAEjD,6BAA6B;IAC7B,+FAA+F;IAC/F,kEAAkE;IAClE,mDAAmD;IACnD,MAAM,KAAK,GAAG,CAAC,EAAE,cAAc,EAAE,qBAAqB,EAAE,UAAU,EAAE,2BAAa,EAAE,CAAC,CAAC;IACrF,MAAM,IAAI,GAAG,IAAI,GAAG,EAAU,CAAC;IAE/B,OAAO,KAAK,CAAC,MAAM,GAAG,CAAC,EAAE;QACvB,MAAM,EAAE,cAAc,EAAE,UAAU,EAAE,GAAG,KAAK,CAAC,KAAK,EAAG,CAAC;QAEtD,+FAA+F;QAC/F,IAAI,cAAc,KAAK,qBAAqB,EAAE;YAC5C,MAAM,OAAO,GAAG,qBAAqB,CAAC,cAAc,CAAC,CAAC;YACtD,qBAAqB,CAAC,cAAc,CAAC,GAAG,IAAA,8BAAgB,EACtD,OAAO,EACP,UAAU,EACV,YAAY,CAAC,cAAc,CAAC,EAAE,eAAe,EAAE,qBAAqB,CACrE,CAAC;YAEF,IAAI,qBAAqB,CAAC,cAAc,CAAC,KAAK,OAAO,EAAE;gBACrD,6CAA6C;gBAC7C,SAAS;aACV;SACF;QAED,qIAAqI;QACrI,MAAM,iBAAiB,GAAG,UAAU,CAAC,cAAc,CAAC,CAAC;QAErD,IAAI,QAAQ,IAAI,iBAAiB,EAAE,MAAM,EAAE;YACzC,KAAK,MAAM,gBAAgB,IAAI,iBAAiB,EAAE;gBAChD,IAAI,IAAI,CAAC,GAAG,CAAC,gBAAgB,CAAC,EAAE;oBAC9B,SAAS;iBACV;gBAED,IAAI,CAAC,GAAG,CAAC,gBAAgB,CAAC,CAAC;gBAC3B,KAAK,CAAC,IAAI,CAAC,EAAE,cAAc,EAAE,gBAAgB,EAAE,UAAU,EAAE,iBAAiB,EAAE,CAAC,CAAC;aACjF;SACF;QAED,qGAAqG;QACrG,oFAAoF;QACpF,kFAAkF;QAElF,sIAAsI;QACtI,MAAM,KAAK,GAAG,MAAM,CAAC,MAAM,CAAC,aAAa,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,YAAY,CAAC,QAAQ,CAAC,cAAc,CAAC,CAAC,CAAC;QAE9F,IAAI,KAAK,IAAI,CAAC,KAAK,CAAC,qBAAqB,EAAE,QAAQ,CAAC,iBAAiB,CAAC,EAAE;YACtE,KAAK,MAAM,kBAAkB,IAAI,KAAK,CAAC,YAAY,EAAE;gBACnD,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,kBAAkB,CAAC,EAAE;oBACjC,IAAI,CAAC,GAAG,CAAC,kBAAkB,CAAC,CAAC;oBAC7B,KAAK,CAAC,IAAI,CAAC;wBACT,cAAc,EAAE,kBAAkB;wBAClC,UAAU,EAAE,iBAAiB;qBAC9B,CAAC,CAAC;iBACJ;aACF;SACF;KACF;AACH,CAAC;AA3ED,0DA2EC"}

package/lib/changefile/changeTypes.d.ts

@@ -1,23 +1,21 @@
+import { BumpInfo } from '../types/BumpInfo';
 import type { ChangeSet, ChangeType } from '../types/ChangeInfo';
 /**
  * List of all change types from least to most significant.
  */
-export declare const SortedChangeTypes: ChangeType[];
-/**
- * Change type with the smallest weight.
- */
+export declare const SortedChangeTypes: readonly ["none", "prerelease", "prepatch", "patch", "preminor", "minor", "premajor", "major"];
+/** `'none'` change type (smallest weight) */
 export declare const MinChangeType: ChangeType;
 /**
  * Get initial package change types based on the greatest change type set for each package in any
  * change file, accounting for any disallowed change types or nonexistent packages.
+ * Anything with change type "none" will be ignored.
  */
-export declare function initializePackageChangeTypes(changeSet: ChangeSet): {
-    [pkgName: string]: ChangeType;
-};
+export declare function initializePackageChangeTypes(changeSet: ChangeSet): BumpInfo['calculatedChangeTypes'];
 /**
  * Get the max allowed change type based on `a` and `b`, accounting for disallowed change types:
  * e.g. if `a` is "major" and `b` is "patch", and "major" is disallowed, the result will be "minor"
  * (the greatest allowed change type).
  */
-export declare function getMaxChangeType(a: ChangeType | undefined, b: ChangeType | undefined, disallowedChangeTypes?: ChangeType[] | null): ChangeType;
+export declare function getMaxChangeType(a: ChangeType | undefined, b?: ChangeType, disallowedChangeTypes?: ReadonlyArray<ChangeType> | null): ChangeType;
 //# sourceMappingURL=changeTypes.d.ts.map

package/lib/changefile/changeTypes.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"changeTypes.d.ts","sourceRoot":"","sources":["../../src/changefile/changeTypes.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,UAAU,EAAE,MAAM,qBAAqB,CAAC;AAEjE;;GAEG;AACH,eAAO,MAAM,iBAAiB,EAAE,UAAU,EASzC,CAAC;AAEF;;GAEG;AACH,eAAO,MAAM,aAAa,YAAuB,CAAC;AAQlD;;;GAGG;AACH,wBAAgB,4BAA4B,CAAC,SAAS,EAAE,SAAS,GAAG;IAAE,CAAC,OAAO,EAAE,MAAM,GAAG,UAAU,CAAA;CAAE,CASpG;AAeD;;;;GAIG;AACH,wBAAgB,gBAAgB,CAC9B,CAAC,EAAE,UAAU,GAAG,SAAS,EACzB,CAAC,EAAE,UAAU,GAAG,SAAS,EACzB,qBAAqB,CAAC,EAAE,UAAU,EAAE,GAAG,IAAI,GAC1C,UAAU,CAUZ"}
+{"version":3,"file":"changeTypes.d.ts","sourceRoot":"","sources":["../../src/changefile/changeTypes.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,MAAM,mBAAmB,CAAC;AAC7C,OAAO,KAAK,EAAE,SAAS,EAAE,UAAU,EAAE,MAAM,qBAAqB,CAAC;AAEjE;;GAEG;AACH,eAAO,MAAM,iBAAiB,gGASY,CAAC;AAE3C,6CAA6C;AAC7C,eAAO,MAAM,aAAa,EAAE,UAAmB,CAAC;AAQhD;;;;GAIG;AACH,wBAAgB,4BAA4B,CAAC,SAAS,EAAE,SAAS,GAAG,QAAQ,CAAC,uBAAuB,CAAC,CAapG;AAeD;;;;GAIG;AACH,wBAAgB,gBAAgB,CAC9B,CAAC,EAAE,UAAU,GAAG,SAAS,EACzB,CAAC,CAAC,EAAE,UAAU,EACd,qBAAqB,CAAC,EAAE,aAAa,CAAC,UAAU,CAAC,GAAG,IAAI,GACvD,UAAU,CAOZ"}

package/lib/changefile/changeTypes.js

@@ -14,10 +14,8 @@ exports.SortedChangeTypes = [
     'premajor',
     'major',
 ];
-/**
- * Change type with the smallest weight.
- */
-exports.MinChangeType = exports.SortedChangeTypes[0];
+/** `'none'` change type (smallest weight) */
+exports.MinChangeType = 'none';
 /**
  * Change type weights.
  * Note: the order in which this is defined is IMPORTANT.
@@ -26,12 +24,17 @@ const ChangeTypeWeights = Object.fromEntries(exports.SortedChangeTypes.map((t, i
 /**
  * Get initial package change types based on the greatest change type set for each package in any
  * change file, accounting for any disallowed change types or nonexistent packages.
+ * Anything with change type "none" will be ignored.
  */
 function initializePackageChangeTypes(changeSet) {
     const pkgChangeTypes = {};
     for (const { change } of changeSet) {
-        const { packageName: pkg, type } = change;
-        pkgChangeTypes[pkg] = getMaxChangeType(type, pkgChangeTypes[pkg] || 'none', null);
+        const { packageName: pkg } = change;
+        const changeType = getMaxChangeType(change.type, pkgChangeTypes[pkg]);
+        // It's best to totally ignore "none" changes to do a bit less processing.
+        if (changeType !== 'none') {
+            pkgChangeTypes[pkg] = changeType;
+        }
     }
     return pkgChangeTypes;
 }
@@ -52,13 +55,11 @@ function getAllowedChangeType(changeType, disallowedChangeTypes) {
  * (the greatest allowed change type).
  */
 function getMaxChangeType(a, b, disallowedChangeTypes) {
-    a ?? (a = exports.MinChangeType);
-    b ?? (b = exports.MinChangeType);
-    if (disallowedChangeTypes) {
-        a = getAllowedChangeType(a, disallowedChangeTypes);
-        b = getAllowedChangeType(b, disallowedChangeTypes);
+    if (disallowedChangeTypes?.length) {
+        a = a && getAllowedChangeType(a, disallowedChangeTypes);
+        b = b && getAllowedChangeType(b, disallowedChangeTypes);
     }
-    return ChangeTypeWeights[a] > ChangeTypeWeights[b] ? a : b;
+    return a && b ? (ChangeTypeWeights[a] > ChangeTypeWeights[b] ? a : b) : a || b || 'none';
 }
 exports.getMaxChangeType = getMaxChangeType;
 //# sourceMappingURL=changeTypes.js.map

package/lib/changefile/changeTypes.js.map

@@ -1 +1 @@
-{"version":3,"file":"changeTypes.js","sourceRoot":"","sources":["../../src/changefile/changeTypes.ts"],"names":[],"mappings":";;;AAEA;;GAEG;AACU,QAAA,iBAAiB,GAAiB;IAC7C,MAAM;IACN,YAAY;IACZ,UAAU;IACV,OAAO;IACP,UAAU;IACV,OAAO;IACP,UAAU;IACV,OAAO;CACR,CAAC;AAEF;;GAEG;AACU,QAAA,aAAa,GAAG,yBAAiB,CAAC,CAAC,CAAC,CAAC;AAElD;;;GAGG;AACH,MAAM,iBAAiB,GAAG,MAAM,CAAC,WAAW,CAAC,yBAAiB,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAkC,CAAC;AAEvH;;;GAGG;AACH,SAAgB,4BAA4B,CAAC,SAAoB;IAC/D,MAAM,cAAc,GAAsC,EAAE,CAAC;IAE7D,KAAK,MAAM,EAAE,MAAM,EAAE,IAAI,SAAS,EAAE;QAClC,MAAM,EAAE,WAAW,EAAE,GAAG,EAAE,IAAI,EAAE,GAAG,MAAM,CAAC;QAC1C,cAAc,CAAC,GAAG,CAAC,GAAG,gBAAgB,CAAC,IAAI,EAAE,cAAc,CAAC,GAAG,CAAC,IAAI,MAAM,EAAE,IAAI,CAAC,CAAC;KACnF;IAED,OAAO,cAAc,CAAC;AACxB,CAAC;AATD,oEASC;AAED,SAAS,oBAAoB,CAAC,UAAsB,EAAE,qBAAmC;IACvF,IAAI,CAAC,UAAU,EAAE;QACf,OAAO,MAAM,CAAC,CAAC,wCAAwC;KACxD;IAED,OAAO,qBAAqB,CAAC,QAAQ,CAAC,UAAU,CAAC,IAAI,UAAU,KAAK,MAAM,EAAE;QAC1E,MAAM,oBAAoB,GAAG,iBAAiB,CAAC,UAAU,CAAC,GAAG,CAAC,CAAC;QAC/D,UAAU,GAAG,yBAAiB,CAAC,oBAAoB,CAAC,CAAC;KACtD;IAED,OAAO,UAAU,CAAC;AACpB,CAAC;AAED;;;;GAIG;AACH,SAAgB,gBAAgB,CAC9B,CAAyB,EACzB,CAAyB,EACzB,qBAA2C;IAE3C,CAAC,KAAD,CAAC,GAAK,qBAAa,EAAC;IACpB,CAAC,KAAD,CAAC,GAAK,qBAAa,EAAC;IAEpB,IAAI,qBAAqB,EAAE;QACzB,CAAC,GAAG,oBAAoB,CAAC,CAAC,EAAE,qBAAqB,CAAC,CAAC;QACnD,CAAC,GAAG,oBAAoB,CAAC,CAAC,EAAE,qBAAqB,CAAC,CAAC;KACpD;IAED,OAAO,iBAAiB,CAAC,CAAC,CAAC,GAAG,iBAAiB,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;AAC7D,CAAC;AAdD,4CAcC"}
+{"version":3,"file":"changeTypes.js","sourceRoot":"","sources":["../../src/changefile/changeTypes.ts"],"names":[],"mappings":";;;AAGA;;GAEG;AACU,QAAA,iBAAiB,GAAG;IAC/B,MAAM;IACN,YAAY;IACZ,UAAU;IACV,OAAO;IACP,UAAU;IACV,OAAO;IACP,UAAU;IACV,OAAO;CACiC,CAAC;AAE3C,6CAA6C;AAChC,QAAA,aAAa,GAAe,MAAM,CAAC;AAEhD;;;GAGG;AACH,MAAM,iBAAiB,GAAG,MAAM,CAAC,WAAW,CAAC,yBAAiB,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAA+B,CAAC;AAEpH;;;;GAIG;AACH,SAAgB,4BAA4B,CAAC,SAAoB;IAC/D,MAAM,cAAc,GAAsC,EAAE,CAAC;IAE7D,KAAK,MAAM,EAAE,MAAM,EAAE,IAAI,SAAS,EAAE;QAClC,MAAM,EAAE,WAAW,EAAE,GAAG,EAAE,GAAG,MAAM,CAAC;QACpC,MAAM,UAAU,GAAG,gBAAgB,CAAC,MAAM,CAAC,IAAI,EAAE,cAAc,CAAC,GAAG,CAAC,CAAC,CAAC;QACtE,0EAA0E;QAC1E,IAAI,UAAU,KAAK,MAAM,EAAE;YACzB,cAAc,CAAC,GAAG,CAAC,GAAG,UAAU,CAAC;SAClC;KACF;IAED,OAAO,cAAc,CAAC;AACxB,CAAC;AAbD,oEAaC;AAED,SAAS,oBAAoB,CAAC,UAAsB,EAAE,qBAAgD;IACpG,IAAI,CAAC,UAAU,EAAE;QACf,OAAO,MAAM,CAAC,CAAC,wCAAwC;KACxD;IAED,OAAO,qBAAqB,CAAC,QAAQ,CAAC,UAAU,CAAC,IAAI,UAAU,KAAK,MAAM,EAAE;QAC1E,MAAM,oBAAoB,GAAG,iBAAiB,CAAC,UAAU,CAAC,GAAG,CAAC,CAAC;QAC/D,UAAU,GAAG,yBAAiB,CAAC,oBAAoB,CAAC,CAAC;KACtD;IAED,OAAO,UAAU,CAAC;AACpB,CAAC;AAED;;;;GAIG;AACH,SAAgB,gBAAgB,CAC9B,CAAyB,EACzB,CAAc,EACd,qBAAwD;IAExD,IAAI,qBAAqB,EAAE,MAAM,EAAE;QACjC,CAAC,GAAG,CAAC,IAAI,oBAAoB,CAAC,CAAC,EAAE,qBAAqB,CAAC,CAAC;QACxD,CAAC,GAAG,CAAC,IAAI,oBAAoB,CAAC,CAAC,EAAE,qBAAqB,CAAC,CAAC;KACzD;IAED,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,iBAAiB,CAAC,CAAC,CAAC,GAAG,iBAAiB,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,MAAM,CAAC;AAC3F,CAAC;AAXD,4CAWC"}

package/lib/types/BumpInfo.d.ts

@@ -2,14 +2,22 @@ import type { ChangeSet, ChangeType } from './ChangeInfo';
 import type { DeepReadonly } from './DeepReadonly';
 import type { PackageInfos, PackageGroups } from './PackageInfo';
 export type BumpInfo = {
-    /** Changes coming from the change files */
+    /**
+     * Changes coming from the change files.
+     * `readChangeFiles` ensures that this will only contain changes for packages that exist.
+     */
     changeFileChangeInfos: DeepReadonly<ChangeSet>;
     /**
      * Cached version of package info (e.g. package.json, package path).
      * This will be updated to reflect the bumped versions and dependencies.
      */
     packageInfos: PackageInfos;
-    /** Change types collated by the package names */
+    /**
+     * Mapping from package name to change type.
+     *
+     * Initially (after `gatherBumpInfo`), this just has change types based on the change files.
+     * It's updated by the early steps of `bumpInPlace` to consider groups and `disallowedChangeTypes`.
+     */
     calculatedChangeTypes: {
         [pkgName: string]: ChangeType;
     };

package/lib/types/BumpInfo.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"BumpInfo.d.ts","sourceRoot":"","sources":["../../src/types/BumpInfo.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,UAAU,EAAE,MAAM,cAAc,CAAC;AAC1D,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,gBAAgB,CAAC;AACnD,OAAO,KAAK,EAAE,YAAY,EAAE,aAAa,EAAE,MAAM,eAAe,CAAC;AAEjE,MAAM,MAAM,QAAQ,GAAG;IACrB,2CAA2C;IAC3C,qBAAqB,EAAE,YAAY,CAAC,SAAS,CAAC,CAAC;IAE/C;;;OAGG;IACH,YAAY,EAAE,YAAY,CAAC;IAE3B,iDAAiD;IACjD,qBAAqB,EAAE;QAAE,CAAC,OAAO,EAAE,MAAM,GAAG,UAAU,CAAA;KAAE,CAAC;IAEzD,uBAAuB;IACvB,aAAa,EAAE,YAAY,CAAC,aAAa,CAAC,CAAC;IAE3C,6CAA6C;IAC7C,gBAAgB,EAAE,GAAG,CAAC,MAAM,CAAC,CAAC;IAE9B,+EAA+E;IAC/E,kBAAkB,EAAE;QAAE,CAAC,OAAO,EAAE,MAAM,GAAG,GAAG,CAAC,MAAM,CAAC,CAAA;KAAE,CAAC;IAEvD,sDAAsD;IACtD,cAAc,EAAE,WAAW,CAAC,MAAM,CAAC,CAAC;CACrC,CAAC;AAEF,uFAAuF;AACvF,MAAM,MAAM,iBAAiB,GAAG;IAAE,QAAQ,EAAE,OAAO,EAAE,MAAM,GAAG,aAAa,CAAC,MAAM,CAAC,CAAA;CAAE,CAAC;AAEtF;;;GAGG;AACH,MAAM,MAAM,eAAe,GAAG,QAAQ,GAAG;IACvC;;;OAGG;IACH,WAAW,CAAC,EAAE,aAAa,CAAC,MAAM,CAAC,CAAC;CACrC,CAAC"}
+{"version":3,"file":"BumpInfo.d.ts","sourceRoot":"","sources":["../../src/types/BumpInfo.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,UAAU,EAAE,MAAM,cAAc,CAAC;AAC1D,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,gBAAgB,CAAC;AACnD,OAAO,KAAK,EAAE,YAAY,EAAE,aAAa,EAAE,MAAM,eAAe,CAAC;AAEjE,MAAM,MAAM,QAAQ,GAAG;IACrB;;;OAGG;IACH,qBAAqB,EAAE,YAAY,CAAC,SAAS,CAAC,CAAC;IAE/C;;;OAGG;IACH,YAAY,EAAE,YAAY,CAAC;IAE3B;;;;;OAKG;IACH,qBAAqB,EAAE;QAAE,CAAC,OAAO,EAAE,MAAM,GAAG,UAAU,CAAA;KAAE,CAAC;IAEzD,uBAAuB;IACvB,aAAa,EAAE,YAAY,CAAC,aAAa,CAAC,CAAC;IAE3C,6CAA6C;IAC7C,gBAAgB,EAAE,GAAG,CAAC,MAAM,CAAC,CAAC;IAE9B,+EAA+E;IAC/E,kBAAkB,EAAE;QAAE,CAAC,OAAO,EAAE,MAAM,GAAG,GAAG,CAAC,MAAM,CAAC,CAAA;KAAE,CAAC;IAEvD,sDAAsD;IACtD,cAAc,EAAE,WAAW,CAAC,MAAM,CAAC,CAAC;CACrC,CAAC;AAEF,uFAAuF;AACvF,MAAM,MAAM,iBAAiB,GAAG;IAAE,QAAQ,EAAE,OAAO,EAAE,MAAM,GAAG,aAAa,CAAC,MAAM,CAAC,CAAA;CAAE,CAAC;AAEtF;;;GAGG;AACH,MAAM,MAAM,eAAe,GAAG,QAAQ,GAAG;IACvC;;;OAGG;IACH,WAAW,CAAC,EAAE,aAAa,CAAC,MAAM,CAAC,CAAC;CACrC,CAAC"}

package/lib/types/ChangeInfo.d.ts

@@ -31,8 +31,12 @@ export interface ChangeInfoMultiple {
 /**
  * List of change file infos (not actually a set).
  */
-export type ChangeSet = {
-    changeFile: string;
+export type ChangeSet = Array<{
     change: ChangeFileInfo;
-}[];
+    /**
+     * Filename the change came from (under `BeachballOptions.changeDir`).
+     * Multiple entries in the array might have come from the same file.
+     */
+    changeFile: string;
+}>;
 //# sourceMappingURL=ChangeInfo.d.ts.map

package/lib/types/ChangeInfo.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"ChangeInfo.d.ts","sourceRoot":"","sources":["../../src/types/ChangeInfo.ts"],"names":[],"mappings":"AAAA,MAAM,MAAM,UAAU,GAAG,YAAY,GAAG,UAAU,GAAG,OAAO,GAAG,UAAU,GAAG,OAAO,GAAG,UAAU,GAAG,OAAO,GAAG,MAAM,CAAC;AAEpH;;;GAGG;AACH,MAAM,WAAW,cAAc;IAC7B,IAAI,EAAE,UAAU,CAAC;IACjB,qBAAqB;IACrB,OAAO,EAAE,MAAM,CAAC;IAChB,qCAAqC;IACrC,WAAW,EAAE,MAAM,CAAC;IACpB,mBAAmB;IACnB,KAAK,EAAE,MAAM,CAAC;IACd,mDAAmD;IACnD,mBAAmB,EAAE,UAAU,CAAC;IAChC,6DAA6D;IAC7D,CAAC,SAAS,EAAE,MAAM,GAAG,GAAG,CAAC;CAC1B;AAED;;GAEG;AACH,MAAM,WAAW,UAAW,SAAQ,cAAc;IAChD,MAAM,EAAE,MAAM,CAAC;CAChB;AAED;;GAEG;AACH,MAAM,WAAW,kBAAkB;IACjC,OAAO,EAAE,UAAU,EAAE,CAAC;CACvB;AAED;;GAEG;AACH,MAAM,MAAM,SAAS,GAAG;IAAE,UAAU,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,cAAc,CAAA;CAAE,EAAE,CAAC"}
+{"version":3,"file":"ChangeInfo.d.ts","sourceRoot":"","sources":["../../src/types/ChangeInfo.ts"],"names":[],"mappings":"AAAA,MAAM,MAAM,UAAU,GAAG,YAAY,GAAG,UAAU,GAAG,OAAO,GAAG,UAAU,GAAG,OAAO,GAAG,UAAU,GAAG,OAAO,GAAG,MAAM,CAAC;AAEpH;;;GAGG;AACH,MAAM,WAAW,cAAc;IAC7B,IAAI,EAAE,UAAU,CAAC;IACjB,qBAAqB;IACrB,OAAO,EAAE,MAAM,CAAC;IAChB,qCAAqC;IACrC,WAAW,EAAE,MAAM,CAAC;IACpB,mBAAmB;IACnB,KAAK,EAAE,MAAM,CAAC;IACd,mDAAmD;IACnD,mBAAmB,EAAE,UAAU,CAAC;IAChC,6DAA6D;IAC7D,CAAC,SAAS,EAAE,MAAM,GAAG,GAAG,CAAC;CAC1B;AAED;;GAEG;AACH,MAAM,WAAW,UAAW,SAAQ,cAAc;IAChD,MAAM,EAAE,MAAM,CAAC;CAChB;AAED;;GAEG;AACH,MAAM,WAAW,kBAAkB;IACjC,OAAO,EAAE,UAAU,EAAE,CAAC;CACvB;AAED;;GAEG;AACH,MAAM,MAAM,SAAS,GAAG,KAAK,CAAC;IAC5B,MAAM,EAAE,cAAc,CAAC;IACvB;;;OAGG;IACH,UAAU,EAAE,MAAM,CAAC;CACpB,CAAC,CAAC"}

package/lib/validation/isValidDependentChangeType.d.ts

@@ -1,3 +1,7 @@
 import { ChangeType } from '../types/ChangeInfo';
+/**
+ * Returns whether `dependentChangeType` is valid and not disallowed.
+ * Note that `'patch'` is always allowed.
+ */
 export declare function isValidDependentChangeType(dependentChangeType: ChangeType, disallowedChangeTypes: ChangeType[] | null): boolean;
 //# sourceMappingURL=isValidDependentChangeType.d.ts.map

package/lib/validation/isValidDependentChangeType.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"isValidDependentChangeType.d.ts","sourceRoot":"","sources":["../../src/validation/isValidDependentChangeType.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,qBAAqB,CAAC;AAEjD,wBAAgB,0BAA0B,CACxC,mBAAmB,EAAE,UAAU,EAC/B,qBAAqB,EAAE,UAAU,EAAE,GAAG,IAAI,GACzC,OAAO,CAST"}
+{"version":3,"file":"isValidDependentChangeType.d.ts","sourceRoot":"","sources":["../../src/validation/isValidDependentChangeType.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,UAAU,EAAE,MAAM,qBAAqB,CAAC;AAEjD;;;GAGG;AACH,wBAAgB,0BAA0B,CACxC,mBAAmB,EAAE,UAAU,EAC/B,qBAAqB,EAAE,UAAU,EAAE,GAAG,IAAI,GACzC,OAAO,CAOT"}

package/lib/validation/isValidDependentChangeType.js

@@ -1,10 +1,17 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.isValidDependentChangeType = void 0;
+const changeTypes_1 = require("../changefile/changeTypes");
+/**
+ * Returns whether `dependentChangeType` is valid and not disallowed.
+ * Note that `'patch'` is always allowed.
+ */
 function isValidDependentChangeType(dependentChangeType, disallowedChangeTypes) {
-    // patch is always allowed as a dependentChangeType
-    const disallowedDependentChangeTypes = (disallowedChangeTypes || []).filter(t => t !== 'patch');
-    return (['patch', 'major', 'minor', 'prerelease', 'prepatch', 'praminor', 'premajor', 'none'].includes(dependentChangeType) && !disallowedDependentChangeTypes.includes(dependentChangeType));
+    if (dependentChangeType === 'patch') {
+        // patch is always allowed as a dependentChangeType
+        return true;
+    }
+    return changeTypes_1.SortedChangeTypes.includes(dependentChangeType) && !disallowedChangeTypes?.includes(dependentChangeType);
 }
 exports.isValidDependentChangeType = isValidDependentChangeType;
 //# sourceMappingURL=isValidDependentChangeType.js.map

package/lib/validation/isValidDependentChangeType.js.map

@@ -1 +1 @@
-{"version":3,"file":"isValidDependentChangeType.js","sourceRoot":"","sources":["../../src/validation/isValidDependentChangeType.ts"],"names":[],"mappings":";;;AAEA,SAAgB,0BAA0B,CACxC,mBAA+B,EAC/B,qBAA0C;IAE1C,mDAAmD;IACnD,MAAM,8BAA8B,GAAG,CAAC,qBAAqB,IAAI,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,KAAK,OAAO,CAAC,CAAC;IAEhG,OAAO,CACL,CAAC,OAAO,EAAE,OAAO,EAAE,OAAO,EAAE,YAAY,EAAE,UAAU,EAAE,UAAU,EAAE,UAAU,EAAE,MAAM,CAAC,CAAC,QAAQ,CAC5F,mBAAmB,CACpB,IAAI,CAAC,8BAA8B,CAAC,QAAQ,CAAC,mBAAmB,CAAC,CACnE,CAAC;AACJ,CAAC;AAZD,gEAYC"}
+{"version":3,"file":"isValidDependentChangeType.js","sourceRoot":"","sources":["../../src/validation/isValidDependentChangeType.ts"],"names":[],"mappings":";;;AAAA,2DAA8D;AAG9D;;;GAGG;AACH,SAAgB,0BAA0B,CACxC,mBAA+B,EAC/B,qBAA0C;IAE1C,IAAI,mBAAmB,KAAK,OAAO,EAAE;QACnC,mDAAmD;QACnD,OAAO,IAAI,CAAC;KACb;IAED,OAAO,+BAAiB,CAAC,QAAQ,CAAC,mBAAmB,CAAC,IAAI,CAAC,qBAAqB,EAAE,QAAQ,CAAC,mBAAmB,CAAC,CAAC;AAClH,CAAC;AAVD,gEAUC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "beachball",
-  "version": "2.51.2",
+  "version": "2.52.0",
   "description": "The Sunniest Semantic Version Bumper",
   "repository": {
     "type": "git",