rebuild depstree

packages/n4s/src/exports/compose.ts

@@ -1,7 +1,7 @@
+import { ctx } from 'n4s';
 import { invariant, StringObject, assign, mapFirst } from 'vest-utils';
 
 import type { ComposeResult, LazyRuleRunners } from 'genEnforceLazy';
-import { ctx } from 'n4s';
 import { defaultToPassing, RuleDetailedResult } from 'ruleReturn';
 import runLazyRule from 'runLazyRule';
 

packages/n4s/src/exports/compounds.ts

@@ -1,8 +1,9 @@
+import { enforce } from 'n4s';
+
 import { allOf } from 'allOf';
 import { anyOf } from 'anyOf';
 import { EnforceCustomMatcher } from 'enforceUtilityTypes';
 import { Lazy } from 'genEnforceLazy';
-import { enforce } from 'n4s';
 import { noneOf } from 'noneOf';
 import { oneOf } from 'oneOf';
 import { RuleDetailedResult } from 'ruleReturn';

packages/n4s/src/exports/schema.ts

@@ -1,7 +1,8 @@
+import { enforce } from 'n4s';
+
 import { EnforceCustomMatcher } from 'enforceUtilityTypes';
 import { isArrayOf } from 'isArrayOf';
 import { loose } from 'loose';
-import { enforce } from 'n4s';
 import { optional } from 'optional';
 import { shape } from 'shape';
 

packages/n4s/src/plugins/schema/isArrayOf.ts

@@ -1,7 +1,7 @@
+import { ctx } from 'n4s';
 import { mapFirst } from 'vest-utils';
 
 import type { LazyRuleRunners } from 'genEnforceLazy';
-import { ctx } from 'n4s';
 import type { RuleDetailedResult } from 'ruleReturn';
 import * as ruleReturn from 'ruleReturn';
 import runLazyRule from 'runLazyRule';

packages/n4s/src/plugins/schema/loose.ts

@@ -1,4 +1,5 @@
 import { ctx } from 'n4s';
+
 import type { RuleDetailedResult } from 'ruleReturn';
 import * as ruleReturn from 'ruleReturn';
 import runLazyRule from 'runLazyRule';

vx/package.json

@@ -10,7 +10,6 @@
     "dotenv": "^16.0.1",
     "fs-extra": "^10.1.0",
     "glob": "^8.0.3",
-    "madge": "^5.0.1",
     "onchange": "^7.1.0",
     "rollup": "^2.48.0",
     "rollup-plugin-terser": "^7.0.2",

vx/scripts/release/depsTree.js

@@ -1,53 +1,45 @@
-const path = require('path');
-
 const { memoize } = require('lodash');
-const madge = require('madge');
-const opts = require('vx/opts');
-const vxPath = require('vx/vxPath');
+const packageNames = require('vx/packageNames');
+const packageJson = require('vx/util/packageJson');
 
 // Takes import map and turns it into a dependency map
-const buildDepsTree = memoize(async function buildDepsTree() {
-  const baseTree = (await getAllDeps()).tree;
-
-  const depTree = {};
-  for (const file in baseTree) {
-    const [packageName] = file.split(path.sep);
-    depTree[packageName] = depTree[packageName] ?? {};
-    baseTree[file].forEach(dep => {
-      const [depName] = dep.split(path.sep);
-
-      if (depName === packageName) {
-        return;
-      }
-      depTree[depName] = depTree[depName] || {};
-
-      if (!depTree[depName][packageName]) {
-        depTree[depName][packageName] = depTree[packageName];
-      }
-    });
-  }
-  return depTree;
+const buildDepsMemo = memoize(function (package, deps) {
+  const pkgJson = packageJson(package);
+
+  // Is circular object ok?
+  deps[package] = deps[package] || {};
+
+  const dependencies = Object.keys(pkgJson.dependencies || {});
+
+  dependencies.forEach(dependency => {
+    deps[dependency] = deps[dependency] || {};
+    deps[dependency][package] =
+      deps[package] ?? buildDepsMemo(dependency, deps);
+  });
+
+  return deps;
 });
 
+function buildDepsTree() {
+  return packageNames.list.reduce(
+    (deps, packageName) => buildDepsMemo(packageName, deps),
+    {}
+  );
+}
+
 // Sorts an array of packages by their dependency depth
-async function sortDependencies(packagesList) {
-  const deps = await buildDepsTree();
+function sortDependencies(packagesList) {
+  const deps = buildDepsTree();
 
   return packagesList.sort(
     (a, b) => countMaxDepth(deps[b]) - countMaxDepth(deps[a])
   );
 }
 
-module.exports = { buildDepsTree, sortDependencies };
-
-// Uses madge to get all the packages. Note: must be run from the root of the repo.
-const getAllDeps = memoize(function getAllDeps() {
-  return madge(vxPath.PACKAGES_PATH, {
-    excludeRegExp: [`${opts.dir.TESTS}|.d.ts|shared`, /^((?!src).)*$/],
-    fileExtensions: ['ts'],
-    tsConfig: vxPath.TSCONFIG_PATH,
-  });
-});
+module.exports = {
+  buildDepsTree,
+  sortDependencies,
+};
 
 // Counts max dependency depth
 const countMaxDepth = memoize(function countMaxDepth(node) {

vx/scripts/release/packagesToRelease.js

@@ -1,18 +1,27 @@
+const logger = require('vx/logger');
 const {
   buildDepsTree,
   sortDependencies,
 } = require('vx/scripts/release/depsTree');
 const listAllChangedPackages = require('vx/scripts/release/github/listAllChangedPackages');
 
 // Gets all the packages that need to be released in the correct order
-async function packagesToRelease() {
-  const deps = await buildDepsTree();
+function packagesToRelease() {
+  const deps = buildDepsTree();
 
   const changedPackages = listAllChangedPackages();
 
+  logger.info(
+    `💡 The following packages were changed: \n  - ${changedPackages.join(
+      '\n  - '
+    )}\n`
+  );
+
   const queue = [...changedPackages];
   const release = new Set();
 
+  const unchangedDependents = new Set();
+
   while (queue.length) {
     const name = queue.shift();
 
@@ -24,12 +33,27 @@ async function packagesToRelease() {
 
     for (const dep in dependents) {
       queue.push(dep);
+      unchangedDependents.add(dep);
     }
 
     release.add(name);
   }
 
-  return await sortDependencies([...release]);
+  logger.info(
+    `🧱 The following packages did not change, but will be released because they depend on changed packages: \n  - ${[
+      ...unchangedDependents,
+    ].join('\n  - ')} \n`
+  );
+
+  const allPackagesToRelease = sortDependencies([...release]);
+
+  logger.info(
+    `✅ The packages will be released in the following order: \n  - ${allPackagesToRelease.join(
+      '\n  - '
+    )}\n`
+  );
+
+  return allPackagesToRelease;
 }
 
 module.exports = packagesToRelease;