fix: prevent WebpackLogger 'done hook' error when callback throws

- Replace try/catch with try/finally to always call callback()
- Improve test to explicitly assert WebpackLogger error is not logged

Author: Srushti-33

src/BundleAnalyzerPlugin.js

@@ -69,7 +69,7 @@ class BundleAnalyzerPlugin {
         // Making analyzer logs to be after all webpack logs in the console
         setImmediate(async () => {
           try {
-            await Promise.all(actions.map(action => action()));
+            await Promise.all(actions.map((action) => action()));
           } finally {
             callback();
           }

test/issue-499.test.js

@@ -0,0 +1,43 @@
+"use strict";
+
+const webpack = require("webpack");
+const BundleAnalyzerPlugin = require("../src/BundleAnalyzerPlugin");
+
+describe("Issue #499", () => {
+  it("should not cause WebpackLogger 'done hook' error when callback throws", (done) => {
+    expect.assertions(1);
+
+    const compiler = webpack({
+      mode: "development",
+      entry: __filename,
+      plugins: [new BundleAnalyzerPlugin({ analyzerMode: "disabled" })],
+    });
+
+    let webpackLoggerError = false;
+    const originalConsoleError = console.error;
+
+    console.error = (...args) => {
+      const message = args.join(" ");
+      if (message.includes("No such label 'done hook'")) {
+        webpackLoggerError = true;
+      }
+      // eslint-disable-next-line no-console
+      originalConsoleError.apply(console, args);
+    };
+
+    compiler.run(() => {
+      try {
+        throw new Error("Intentional test error");
+      } catch {
+        // Swallow expected error
+      }
+    });
+
+    setTimeout(() => {
+      console.error = originalConsoleError;
+
+      expect(webpackLoggerError).toBe(false);
+      done();
+    }, 1000);
+  });
+});