monorepo-next 9.4.2 → 9.4.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "monorepo-next",
-  "version": "9.4.2",
+  "version": "9.4.4",
   "description": "Detach monorepo packages from normal linking",
   "bin": {
     "next": "bin/next.js"

package/src/build-release-graph.js

@@ -11,7 +11,7 @@ const semver = require('semver');
 const dependencyTypes = require('./dependency-types');
 const { loadPackageConfig } = require('./config');
 const debug = require('./debug');
-const { createLogger } = require('./log');
+const { createSyncLogger, createAsyncLogger } = require('./log');
 
 const defaultReleaseType = 'patch';
 
@@ -139,11 +139,11 @@ async function secondPass({
         return;
       }
 
-      visitedNodes.add(dag.node.packageName);
-
       let nextConfig = loadPackageConfig(dag.node.cwd);
 
       if (!nextConfig.shouldBumpVersion) {
+        visitedNodes.add(dag.node.packageName);
+
         return;
       }
 
@@ -168,6 +168,8 @@ async function secondPass({
         }
       }
 
+      visitedNodes.add(dag.node.packageName);
+
       for (let group of dag.node.dependents) {
         if (group.isCycle) {
           continue;
@@ -306,18 +308,19 @@ async function buildReleaseGraph({
   shouldInheritGreaterReleaseType,
   shouldExcludeDevChanges,
 }) {
-  let log = createLogger(_debug);
+  let logSync = createSyncLogger(_debug);
+  let logAsync = createAsyncLogger(_debug);
 
   let releaseTrees = {};
 
-  await log(firstPass, {
+  await logAsync(firstPass, {
     releaseTrees,
     packagesWithChanges,
   });
 
   // only packages with changes have been analyzed
 
-  await log(secondPass, {
+  await logAsync(secondPass, {
     releaseTrees,
     packagesWithChanges,
     shouldBumpInRangeDependencies,
@@ -327,7 +330,7 @@ async function buildReleaseGraph({
 
   // packages without changes, but need to be analyzed because of options
 
-  log(thirdPass, {
+  logSync(thirdPass, {
     releaseTrees,
     packagesWithChanges,
     shouldInheritGreaterReleaseType,
@@ -336,7 +339,7 @@ async function buildReleaseGraph({
 
   // dependents have now inherited release type
 
-  log(fourthPass, {
+  logSync(fourthPass, {
     releaseTrees,
     packagesWithChanges,
     shouldBumpInRangeDependencies,

package/src/log.js

@@ -2,24 +2,56 @@
 
 const _debug = require('./debug');
 
-function createLogger(debug = _debug) {
-  return async function log(callback, ...args) {
-    let _debug = debug.extend(callback.name);
+function prepare(callback, debug = _debug) {
+  let _debug = debug.extend(callback.name);
+
+  _debug('begin');
+
+  let before = new Date();
+
+  return {
+    debug: _debug,
+    before,
+  };
+}
 
-    _debug('begin');
+function finish(debug, before) {
+  let elapsed = new Date() - before;
+
+  debug(`end in ${elapsed}ms`);
+}
 
-    let before = new Date();
+function createSyncLogger(debug) {
+  return function log(callback, ...args) {
+    let {
+      debug: _debug,
+      before,
+    } = prepare(callback, debug);
 
     try {
-      return await callback(...args);
+      return callback.apply(this, args);
     } finally {
-      let elapsed = new Date() - before;
+      finish(_debug, before);
+    }
+  };
+}
 
-      _debug(`end in ${elapsed}ms`);
+function createAsyncLogger(debug) {
+  return async function log(callback, ...args) {
+    let {
+      debug: _debug,
+      before,
+    } = prepare(callback, debug);
+
+    try {
+      return await callback.apply(this, args);
+    } finally {
+      finish(_debug, before);
     }
   };
 }
 
 module.exports = {
-  createLogger,
+  createSyncLogger,
+  createAsyncLogger,
 };

package/src/release.js

@@ -18,7 +18,7 @@ const {
 const semver = require('semver');
 const { builder } = require('../bin/commands/release');
 const debug = require('./debug');
-const { createLogger } = require('./log');
+const { createAsyncLogger } = require('./log');
 
 async function release({
   cwd = process.cwd(),
@@ -43,7 +43,7 @@ async function release({
   cached,
 } = {}) {
   let _debug = debug.extend(release.name);
-  let log = createLogger(_debug);
+  let logAsync = createAsyncLogger(_debug);
 
   let currentBranch = await getCurrentBranch(cwd);
   if (currentBranch !== defaultBranch) {
@@ -52,9 +52,9 @@ async function release({
 
   let workspaceCwd = await getWorkspaceCwd(cwd);
 
-  let workspaceMeta = await log(buildDepGraph, { workspaceCwd });
+  let workspaceMeta = await logAsync(buildDepGraph, { workspaceCwd });
 
-  let packagesWithChanges = await log(buildChangeGraph, {
+  let packagesWithChanges = await logAsync(buildChangeGraph, {
     workspaceMeta,
     shouldExcludeDevChanges,
     cached,
@@ -69,7 +69,7 @@ async function release({
     return;
   }
 
-  let releaseTrees = await log(buildReleaseGraph, {
+  let releaseTrees = await logAsync(buildReleaseGraph, {
     debug: _debug,
     packagesWithChanges,
     shouldBumpInRangeDependencies,