Switched default browser library

index.js

@@ -1,10 +1,10 @@
-const path = require('path');
-const childProcess = require('child_process');
-const {promises: fs, constants: fsConstants} = require('fs');
-const isWsl = require('is-wsl');
-const isDocker = require('is-docker');
-const defineLazyProperty = require('define-lazy-prop');
-const defaultBrowser = require('x-default-browser');
+import path from 'path';
+import childProcess from 'child_process';
+import {promises as fs, constants as fsConstants} from 'fs';
+import isWsl from 'is-wsl';
+import isDocker from 'is-docker';
+import defineLazyProperty from 'define-lazy-prop';
+import defaultBrowser from 'default-browser';
 
 // Path to included `xdg-open`.
 const localXdgOpenPath = path.join(__dirname, 'xdg-open');
@@ -119,20 +119,14 @@ const baseOpen = async options => {
 	}
 
 	if (app === 'browser') {
-		return defaultBrowser((error, browser) => {
-			if (error) {
-				throw error;
+		const browser = await defaultBrowser();
+		const browserName = browser.name.toLowerCase();
+		return baseOpen({
+			...options,
+			app: {
+				name: open.apps[browserName],
+				arguments: appArguments
 			}
-
-			const browserName = browser.commonName;
-
-			baseOpen({
-				...options,
-				app: {
-					name: open.apps[browserName],
-					arguments: appArguments
-				}
-			});
 		});
 	}
 
@@ -144,20 +138,14 @@ const baseOpen = async options => {
 			edge: '--inPrivate'
 		};
 
-		return defaultBrowser((error, browser) => {
-			if (error) {
-				throw error;
+		const browser = await defaultBrowser();
+		const browserName = browser.name.toLowerCase();
+		return baseOpen({
+			...options,
+			app: {
+				name: open.apps[browserName],
+				arguments: [...appArguments, flags[browserName]]
 			}
-
-			const browserName = browser.commonName;
-
-			baseOpen({
-				...options,
-				app: {
-					name: open.apps[browserName],
-					arguments: [...appArguments, flags[browserName]]
-				}
-			});
 		});
 	}
 
@@ -379,4 +367,4 @@ defineLazyProperty(apps, 'browserPrivate', () => 'browserPrivate');
 open.apps = apps;
 open.openApp = openApp;
 
-module.exports = open;
+export {open};

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "open",
-	"version": "8.4.2",
+	"version": "8.4.0",
 	"description": "Open stuff like URLs, files, executables. Cross-platform.",
 	"license": "MIT",
 	"repository": "sindresorhus/open",
@@ -14,7 +14,7 @@
 		"node": ">=12"
 	},
 	"scripts": {
-		"test": "xo && tsd"
+		"test": "xo && tsd && ava"
 	},
 	"files": [
 		"index.js",
@@ -51,6 +51,7 @@
 		"define-lazy-prop": "^2.0.0",
 		"is-docker": "^2.1.1",
 		"is-wsl": "^2.2.0",
+		"default-browser": "^3.1.0",
 		"x-default-browser": "^0.5.0"
 	},
 	"devDependencies": {

test.js

@@ -1,5 +1,5 @@
-const test = require('ava');
-const open = require('.');
+import test from 'ava';
+import {open} from './index.js';
 const {openApp} = open;
 
 // Tests only checks that opening doesn't return an error