feat: Add name of package that fails to publish (#3644)

libs/commands/publish/src/index.ts

@@ -922,6 +922,7 @@ class PublishCommand extends Command {
               }
 
               this.logger.silly("", err);
+              this.logger.warn("notice", `Package failed to publish: ${pkg.name}`);
               this.logger.error(err.code, (err.body && err.body.error) || err.message);
 
               // avoid dumping logs, this isn't a lerna problem

libs/commands/publish/src/lib/publish-from-package.spec.ts

@@ -47,9 +47,11 @@ const promptConfirmation = jest.mocked(_promptConfirmation);
 const throwIfUncommitted = jest.mocked(_throwIfUncommitted);
 
 // The mock differs from the real thing
-const npmPublish = _npmPublish as any;
-const writePkg = _writePkg as any;
-const output = _output as any;
+const npmPublish = _npmPublish as jest.MockedFunction<typeof _npmPublish> & { order: () => string[] };
+const writePkg = _writePkg as jest.MockedFunction<typeof _writePkg> & {
+  updatedManifest: (name: string) => { gitHead?: string };
+};
+const output = _output as jest.MockedFunction<typeof _output> & { logged: () => string[] };
 
 const initFixture = initFixtureFactory(__dirname);
 
@@ -105,6 +107,27 @@ describe("publish from-package", () => {
     ]);
   });
 
+  it("logs the name of the package that fails to be published", async () => {
+    const cwd = await initFixture("independent");
+
+    getUnpublishedPackages.mockImplementationOnce((packageGraph) => Array.from(packageGraph.values()));
+
+    npmPublish.mockImplementation(async (pkg) => {
+      if (pkg.name === "package-2") {
+        throw new Error("some-error");
+      }
+    });
+
+    try {
+      await lernaPublish(cwd)("from-package");
+    } catch {
+      // ignore error
+    }
+
+    const logMessages = loggingOutput("notice");
+    expect(logMessages).toContain("Package failed to publish: package-2");
+  });
+
   it("exits early when all packages are published", async () => {
     const cwd = await initFixture("normal");