Replace opener with open

Author: valscion

package-lock.json

@@ -40,7 +40,7 @@
     "debounce": "^2.0.0",
     "escape-string-regexp": "^4.0.0",
     "html-escaper": "^2.0.2",
-    "opener": "^1.5.2",
+    "open": "^8.4.2",
     "picocolors": "^1.0.0",
     "sirv": "^3.0.2",
     "ws": "^8.18.3"

package.json

@@ -1,5 +1,5 @@
 const {inspect, types} = require('util');
-const opener = require('opener');
+const open = require('open');
 
 const MONTHS = ['Jan', 'Feb', 'Mar', 'Apr', 'May', 'Jun', 'Jul', 'Aug', 'Sep', 'Oct', 'Nov', 'Dec'];
 
@@ -56,12 +56,12 @@ exports.defaultAnalyzerUrl = function (options) {
 };
 
 /**
- * Calls opener on a URI, but silently try / catches it.
+ * Calls open on a URI, but silently try / catches it.
  */
-exports.open = function (uri, logger) {
+exports.open = async function (uri, logger) {
   try {
-    opener(uri);
+    await open(uri);
   } catch (err) {
-    logger.debug(`Opener failed to open "${uri}":\n${err}`);
+    logger.debug(`Open failed to open "${uri}":\n${err}`);
   }
 };

src/utils.js

@@ -95,6 +95,8 @@ async function startServer(bundleStats, opts) {
       );
 
       if (openBrowser) {
+        // We don't await here as the server is already running and
+        // we don't want to stop the process even if browser fails to open.
         open(url, logger);
       }
     });
@@ -169,7 +171,7 @@ async function generateReport(bundleStats, opts) {
   logger.info(`${bold('Webpack Bundle Analyzer')} saved report to ${bold(reportFilepath)}`);
 
   if (openBrowser) {
-    open(`file://${reportFilepath}`, logger);
+    await open(`file://${reportFilepath}`, logger);
   }
 }