@vnejs/monorepo 0.2.4 → 0.2.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vnejs/monorepo",
-  "version": "0.2.4",
+  "version": "0.2.6",
   "description": "Monorepo tooling for @vnejs",
   "main": "src/index.js",
   "bin": {
@@ -17,9 +17,9 @@
   "dependencies": {
     "@vnejs/configs.vitest": "~0.2.0",
     "commander": "~15.0.0",
-    "happy-dom": "~20.10.6",
+    "happy-dom": "~21.0.4",
     "p-limit": "~7.3.0",
     "typescript": "7.0.2",
-    "vitest": "~4.1.9"
+    "vitest": "~5.1.0"
   }
 }

package/src/publish/index.js

@@ -39,9 +39,10 @@ module.exports = async (bump, options = {}) => {
     }
 
     console.log(`Пакет ${pkgLabel} успешно опубликован с версией ${publishedVersion}`);
-  } catch {
+  } catch (error) {
     if (bump && readVersion(pkgDir) !== oldVersion) writeVersion(pkgDir, oldVersion);
-    console.error(`Не удалось опубликовать пакет${pkgLabel}. Версия не была изменена (${oldVersion}).`);
+    const detail = error?.message ? ` ${error.message}` : "";
+    console.error(`\n!!! Не удалось опубликовать пакет${pkgLabel} (было ${oldVersion}). Версия не изменена.${detail}\n`);
     process.exit(1);
   }
 };

package/src/publish/utils.js

@@ -39,7 +39,7 @@ const runAsync = (command, pkgDir) =>
         return;
       }
 
-      reject(new Error(`${command} exited with ${code}`));
+      reject(new Error(`${command} in ${pkgDir} exited with ${code}`));
     });
   });
 

package/src/run-changed/index.js

@@ -6,7 +6,7 @@ const { buildDependencyGraph, restrictGraph } = require("../workspace/build-grap
 const { topoSortLayers } = require("../workspace/topo-sort");
 const { runAsync } = require("../publish/utils");
 const confirm = require("../utils/confirm");
-const { formatPackageLine } = require("../utils/format-package");
+const { formatPackageLine, formatPackageRef } = require("../utils/format-package");
 const schedulePackages = require("./schedule");
 
 const groupChangedByGraph = (root, changedPackages) => {
@@ -111,8 +111,15 @@ module.exports = async (options = {}) => {
     firstSerial,
     runFn: async (pkg, index) => {
       const pkgDir = path.join(root, pkg.dir);
-      console.log(formatPackageLine(pkg, { leadingNewline: true, tag: `[${index + 1}/${packages.length}]` }));
-      await runAsync(`npm run ${script}`, pkgDir);
+      const tag = `[${index + 1}/${packages.length}]`;
+      console.log(formatPackageLine(pkg, { leadingNewline: true, tag }));
+
+      try {
+        await runAsync(`npm run ${script}`, pkgDir);
+      } catch (error) {
+        const detail = error?.message ?? String(error);
+        throw new Error(`${tag} ${formatPackageRef(pkg)} (${pkg.dir}): ${detail}`, { cause: error });
+      }
     },
   });
 };

package/src/run-changed/schedule.js

@@ -13,12 +13,14 @@ const rejectFailed = (results, log) => {
 
   if (!failed.length) return;
 
-  for (const result of failed) {
+  const messages = failed.map((result) => {
     const reason = result.reason;
-    log(reason?.stack ?? String(reason));
-  }
+    const message = reason?.message ?? String(reason);
+    log(reason?.stack ?? message);
+    return message;
+  });
 
-  throw new Error(`${failed.length} package script(s) failed.`);
+  throw new Error(`${failed.length} package script(s) failed.\n${messages.map((message) => `- ${message}`).join("\n")}`);
 };
 
 const schedulePackages = async ({ packages = [], layers, runFn, concurrency = 1, parallelMs, firstSerial = false, log = console.log }) => {

package/src/run-changed/schedule.test.js

@@ -236,4 +236,20 @@ describe("schedulePackages", () => {
     await expect(done).rejects.toThrow("1 package script(s) failed.");
     expect(logs.some((message) => String(message).includes("a failed"))).toBe(true);
   });
+
+  it("lists each failed package message in the thrown error", async () => {
+    const { runFn, resolve, reject } = createTrackedRunner();
+    const done = schedulePackages({
+      packages: packages("a", "b"),
+      runFn,
+      concurrency: 2,
+      log: () => {},
+    });
+
+    await flush();
+    reject("a", new Error("[3/89] @vnejs/foo (plugins/foo): npm run publish:patch exited with 1"));
+    resolve("b");
+
+    await expect(done).rejects.toThrow(/1 package script\(s\) failed\.\n- \[3\/89\] @vnejs\/foo/);
+  });
 });