feat: the update command should update to compatible versions by default

e2e/harmony/update-cmd.e2e.ts

@@ -11,7 +11,7 @@ describe('update command', function () {
   after(() => {
     helper.scopeHelper.destroy();
   });
-  describe('updates policies', function () {
+  describe('updates policies to latest versions', function () {
     describe('policies added by the user', function () {
       let configFile;
       let componentJson;
@@ -39,7 +39,7 @@ describe('update command', function () {
         };
         helper.componentJson.write(componentJson, 'comp2');
         helper.command.install('[email protected]');
-        helper.command.update('--yes');
+        helper.command.update('--yes --latest');
         configFile = helper.bitJsonc.read(helper.scopes.localPath);
         componentJson = helper.componentJson.read('comp2');
       });
@@ -82,7 +82,7 @@ describe('update command', function () {
           },
         });
         helper.command.install();
-        helper.command.update('--yes is-posit*');
+        helper.command.update('--yes is-posit* --latest');
         configFile = helper.bitJsonc.read(helper.scopes.localPath);
       });
       it('should update the version range of the selected package', function () {
@@ -102,14 +102,41 @@ describe('update command', function () {
         helper.scopeHelper.reInitLocalScope();
         helper.fixtures.populateComponents(1);
         helper.command.dependenciesSet('comp1', '[email protected]');
-        helper.command.update('--yes');
+        helper.command.update('--yes --latest');
       });
       it('should update the version range', function () {
         const showOutput = helper.command.showComponentParsed('comp1');
         expect(showOutput.packageDependencies['is-negative']).not.to.equal('1.0.0');
       });
     });
   });
+  describe('updates policies to compatible versions', function () {
+    describe('policies added by deps set', function () {
+      before(() => {
+        helper.scopeHelper.reInitLocalScope();
+        helper.fixtures.populateComponents(1);
+        helper.command.dependenciesSet('comp1', '[email protected]');
+        helper.command.update('--yes');
+      });
+      it('should update the version range', function () {
+        const showOutput = helper.command.showComponentParsed('comp1');
+        expect(showOutput.packageDependencies['is-negative']).to.equal('1.0.1');
+      });
+    });
+    describe('policies added by deps set. savePrefix is present', function () {
+      before(() => {
+        helper.scopeHelper.reInitLocalScope();
+        helper.extensions.bitJsonc.addKeyValToDependencyResolver('savePrefix', '^');
+        helper.fixtures.populateComponents(1);
+        helper.command.dependenciesSet('comp1', '[email protected]');
+        helper.command.update('--yes');
+      });
+      it('should update the version range', function () {
+        const showOutput = helper.command.showComponentParsed('comp1');
+        expect(showOutput.packageDependencies['is-negative']).to.equal('^1.0.1');
+      });
+    });
+  });
   (supportNpmCiRegistryTesting ? describe : describe.skip)('updates dependencies from the model', () => {
     let configFile;
     let npmCiRegistry: NpmCiRegistry;
@@ -166,7 +193,7 @@ const isPositive = require("is-positive");`
         helper.bitJsonc.setupDefault();
         helper.extensions.bitJsonc.addKeyValToDependencyResolver('savePrefix', '^');
         helper.command.import(`${helper.scopes.remote}/comp1`);
-        helper.command.update('--yes');
+        helper.command.update('--yes --latest');
         configFile = helper.bitJsonc.read(helper.scopes.localPath);
       });
       it('should add an updated version of the dependency from the model to the workspace policies', function () {
@@ -201,7 +228,7 @@ const isPositive = require("is-positive");`
         helper.scopeHelper.addRemoteScope();
         helper.bitJsonc.setupDefault();
         helper.command.import(`${helper.scopes.remote}/comp1`);
-        helper.command.update('--yes');
+        helper.command.update('--yes --latest');
         configFile = helper.bitJsonc.read(helper.scopes.localPath);
       });
       it('should add an updated version of the dependency from the model to the bitmap', function () {
@@ -265,7 +292,7 @@ const isPositive = require("is-positive");`
       helper.scopeHelper.addRemoteScope();
       helper.command.import(`${helper.scopes.remote}/[email protected]`);
       helper.command.import(`${helper.scopes.remote}/[email protected]`);
-      helper.command.update('--yes');
+      helper.command.update('--yes --latest');
       configFile = helper.bitJsonc.read(helper.scopes.localPath);
     });
     after(() => {

scopes/dependencies/dependency-resolver/dependency-resolver.main.runtime.ts

@@ -50,7 +50,7 @@ import {
 import { DependencyResolverAspect } from './dependency-resolver.aspect';
 import { DependencyVersionResolver } from './dependency-version-resolver';
 import { DepLinkerContext, DependencyLinker, LinkingOptions } from './dependency-linker';
-import { ComponentModelVersion, getAllPolicyPkgs, OutdatedPkg } from './get-all-policy-pkgs';
+import { ComponentModelVersion, getAllPolicyPkgs, OutdatedPkg, CurrentPkgSource } from './get-all-policy-pkgs';
 import { InvalidVersionWithPrefix, PackageManagerNotFound } from './exceptions';
 import {
   CreateFromComponentsOptions,
@@ -1405,7 +1405,7 @@ export class DependencyResolverMain {
     componentPolicies: Array<{ componentId: ComponentID; policy: any }>;
     components: Component[];
     patterns?: string[];
-    forceVersionBump?: 'major' | 'minor' | 'patch';
+    forceVersionBump?: 'major' | 'minor' | 'patch' | 'compatible';
   }): Promise<MergedOutdatedPkg[]> {
     const localComponentPkgNames = new Set(components.map((component) => this.getPackageName(component)));
     const componentModelVersions: ComponentModelVersion[] = (
@@ -1460,7 +1460,7 @@ export class DependencyResolverMain {
       forceVersionBump,
     }: {
       rootDir: string;
-      forceVersionBump?: 'major' | 'minor' | 'patch';
+      forceVersionBump?: 'major' | 'minor' | 'patch' | 'compatible';
     },
     pkgs: Array<
       { name: string; currentRange: string; source: 'variants' | 'component' | 'rootPolicy' | 'component-model' } & T
@@ -1483,7 +1483,9 @@ export class DependencyResolverMain {
     const outdatedPkgs = compact(
       await Promise.all(
         pkgs.map(async (pkg) => {
-          const latestVersion = await tryResolve(`${pkg.name}@${newVersionRange(pkg.currentRange, forceVersionBump)}`);
+          const latestVersion = await tryResolve(
+            `${pkg.name}@${newVersionRange(pkg.currentRange, { pkgSource: pkg.source, forceVersionBump })}`
+          );
           if (!latestVersion) return null;
           const currentVersion = semver.valid(pkg.currentRange.replace(/[\^~]/, ''));
           // If the current version is newer than the latest, then no need to update the dependency
@@ -1690,12 +1692,21 @@ function repeatPrefix(originalSpec: string, newVersion: string): string {
   }
 }
 
-function newVersionRange(currentRange: string, forceVersionBump?: 'major' | 'minor' | 'patch') {
-  if (forceVersionBump == null || forceVersionBump === 'major') return 'latest';
+function newVersionRange(
+  currentRange: string,
+  opts: { pkgSource: CurrentPkgSource; forceVersionBump?: 'major' | 'minor' | 'patch' | 'compatible' }
+) {
+  if (opts.forceVersionBump == null || opts.forceVersionBump === 'major') return 'latest';
   const currentVersion = semver.valid(currentRange.replace(/[\^~]/, ''));
+  if (opts.forceVersionBump === 'compatible') {
+    if ((opts.pkgSource === 'component' || opts.pkgSource === 'component-model') && currentVersion === currentRange) {
+      return `^${currentVersion}`;
+    }
+    return currentRange;
+  }
   if (!currentVersion) return null;
   const [major, minor] = currentVersion.split('.');
-  switch (forceVersionBump) {
+  switch (opts.forceVersionBump) {
     case 'patch':
       return `>=${currentVersion} <${major}.${+minor + 1}.0`;
     case 'minor':

scopes/dependencies/dependency-resolver/get-all-policy-pkgs.ts

@@ -4,10 +4,20 @@ import { VariantPolicyConfigObject, WorkspacePolicy } from './policy';
 import { DependencyLifecycleType } from './dependencies/dependency';
 import { KEY_NAME_BY_LIFECYCLE_TYPE } from './dependencies';
 
+export type CurrentPkgSource =
+  // the variants section of "workspace.jsonc"
+  | 'variants'
+  // these are dependencies set via "bit deps set" or "component.json"
+  | 'component'
+  // these are dependencies from the dependencies policy in "workspace.jsonc"
+  | 'rootPolicy'
+  // these are dependencies stored in the component object (snapped/tagged version)
+  | 'component-model';
+
 type CurrentPkg = {
   name: string;
   currentRange: string;
-  source: 'variants' | 'component' | 'rootPolicy' | 'component-model';
+  source: CurrentPkgSource;
   variantPattern?: string | null;
   componentId?: ComponentID;
   isAuto?: boolean;

scopes/workspace/install/install.main.runtime.ts

@@ -619,7 +619,7 @@ export class InstallMain {
    * @param options.all {Boolean} updates all outdated dependencies without showing a prompt.
    */
   async updateDependencies(options: {
-    forceVersionBump?: 'major' | 'minor' | 'patch';
+    forceVersionBump?: 'major' | 'minor' | 'patch' | 'compatible';
     patterns?: string[];
     all: boolean;
   }): Promise<ComponentMap<string> | null> {

scopes/workspace/install/update.cmd.tsx

@@ -8,11 +8,12 @@ type UpdateCmdOptions = {
   major?: boolean;
   minor?: boolean;
   patch?: boolean;
+  latest?: boolean;
 };
 
 export default class UpdateCmd implements Command {
   name = 'update [package-patterns...]';
-  description = 'update dependencies';
+  description = 'update dependencies. By default, dependencies are updated to the highest semver compatible versions.';
   helpUrl = 'reference/dependencies/configuring-dependencies/#update-dependencies';
   alias = 'up';
   group = 'development';
@@ -32,18 +33,21 @@ export default class UpdateCmd implements Command {
     ['', 'patch', 'update to the latest patch version. Semver rules are ignored'],
     ['', 'minor', 'update to the latest minor version. Semver rules are ignored'],
     ['', 'major', 'update to the latest major version. Semver rules are ignored'],
+    ['', 'latest', 'update to the latest version. Semver rules are ignored'],
   ] as CommandOptions;
 
   constructor(private install: InstallMain) {}
 
   async report([patterns = []]: [string[]], options: UpdateCmdOptions) {
-    let forceVersionBump: 'major' | 'minor' | 'patch' | undefined;
+    let forceVersionBump: 'major' | 'minor' | 'patch' | 'compatible' | undefined;
     if (options.major) {
       forceVersionBump = 'major';
     } else if (options.minor) {
       forceVersionBump = 'minor';
     } else if (options.patch) {
       forceVersionBump = 'patch';
+    } else if (!options.latest) {
+      forceVersionBump = 'compatible';
     }
     await this.install.updateDependencies({
       all: options.yes === true,