chore: respect ALLOWED_INCONSISTENT_DEPENDENCIES for workspace packages; simplify logic

Author: legobeat

yarn.config.cjs

@@ -12,6 +12,15 @@ const { basename, resolve } = require('path');
 const semver = require('semver');
 const { inspect } = require('util');
 
+/**
+ * These packages and ranges are allowed to mismatch expected consistency checks
+ * Only intended as temporary measures to faciliate upgrades and releases.
+ * This should trend towards empty.
+ */
+const ALLOWED_INCONSISTENT_DEPENDENCIES = {
+  '@metamask/rpc-errors': ['^7.0.0'],
+};
+
 /**
  * Aliases for the Yarn type definitions, to make the code more readable.
  *
@@ -594,6 +603,12 @@ function expectUpToDateWorkspaceDependenciesAndDevDependencies(
       dependencyWorkspace !== null &&
       dependency.type !== 'peerDependencies'
     ) {
+
+      const ignoredRanges = ALLOWED_INCONSISTENT_DEPENDENCIES[dependency.ident];
+      if (ignoredRanges?.includes(dependency.range)) {
+        continue;
+      }
+
       dependency.update(`^${dependencyWorkspace.manifest.version}`);
     }
   }
@@ -714,10 +729,6 @@ function expectControllerDependenciesListedAsPeerDependencies(
   }
 }
 
-const ALLOWED_INCONSISTENT_DEPENDENCIES = Object.entries({
-  '@metamask/rpc-errors': ['^7.0.0'],
-});
-
 /**
  * Filter out dependency ranges which are not to be considered in `expectConsistentDependenciesAndDevDependencies`.
  *
@@ -729,19 +740,15 @@ function getInconsistentDependenciesAndDevDependencies(
   dependencyIdent,
   dependenciesByRange,
 ) {
-  for (const [
-    allowedPackage,
-    ignoredRange,
-  ] of ALLOWED_INCONSISTENT_DEPENDENCIES) {
-    if (allowedPackage === dependencyIdent) {
-      return new Map(
-        Object.entries(dependenciesByRange).filter(
-          ([range]) => !ignoredRange.includes(range),
-        ),
-      );
-    }
+  const ignoredRanges = ALLOWED_INCONSISTENT_DEPENDENCIES[dependencyIdent];
+  if (!ignoredRanges) {
+    return dependenciesByRange;
   }
-  return dependenciesByRange;
+  return new Map(
+    Object.entries(dependenciesByRange).filter(
+      ([range]) => !ignoredRanges.includes(range),
+    ),
+  );
 }
 
 /**