use shell-env module

index.js

@@ -2,18 +2,13 @@
 const childProcess = require('child_process');
 const execa = require('execa');
 const stripAnsi = require('strip-ansi');
-const defaultShell = require('default-shell');
+const shellEnv = require('shell-env');
 const user = process.env.USER;
 const opts = {encoding: 'utf8'};
 
-function clean(str) {
-	return stripAnsi(str.trim()).split('\n').pop();
-}
-
-function pathFromShell() {
-	return execa(defaultShell, ['-ic', 'echo "$PATH"'])
-		.then(x => clean(x.stdout) || '')
-		.catch(() => '');
+function parseEnv(env) {
+	const pathLine = stripAnsi(env.trim()).split('\n').filter(x => /^PATH=/.test(x.trim()))[0];
+	return (pathLine && pathLine.split('=')[1]) || '';
 }
 
 function pathFromSudo() {
@@ -22,12 +17,6 @@ function pathFromSudo() {
 		.catch(() => '');
 }
 
-function pathFromShellSync() {
-	// TODO: use `execa` → https://github.com/sindresorhus/execa/issues/7
-	const stdout = childProcess.execFileSync(defaultShell, ['-ic', 'echo "$PATH"'], opts);
-	return clean(stdout) || '';
-}
-
 function pathFromSudoSync() {
 	try {
 		// TODO: use `execa` → https://github.com/sindresorhus/execa/issues/7
@@ -38,11 +27,6 @@ function pathFromSudoSync() {
 	}
 }
 
-function parseEnv(env) {
-	const pathLine = stripAnsi(env.trim()).split('\n').filter(x => /^PATH=/.test(x.trim()))[0];
-	return (pathLine && pathLine.split('=')[1]) || '';
-}
-
 function longest(arr) {
 	return arr.reduce((a, b) => a.split(':').length > b.split(':').length ? a : b);
 }
@@ -53,10 +37,9 @@ module.exports = () => {
 	}
 
 	return Promise.all([
-		pathFromShell(),
-		pathFromSudo(),
-		process.env.PATH
-	]).then(x => longest(x));
+		shellEnv().then(x => x.PATH),
+		pathFromSudo()
+	]).then(longest);
 };
 
 module.exports.sync = () => {
@@ -65,8 +48,7 @@ module.exports.sync = () => {
 	}
 
 	return longest([
-		pathFromShellSync(),
-		pathFromSudoSync(),
-		process.env.PATH
+		shellEnv.sync().PATH,
+		pathFromSudoSync()
 	]);
 };

package.json

@@ -36,8 +36,8 @@
     "nwjs"
   ],
   "dependencies": {
-    "default-shell": "^1.0.0",
     "execa": "^0.2.2",
+    "shell-env": "^0.1.0",
     "strip-ansi": "^3.0.0"
   },
   "devDependencies": {

test.js

@@ -1,15 +1,14 @@
-import os from 'os';
 import test from 'ava';
 import m from './';
 
 test('async', async t => {
 	const PATH = await m();
 	t.true(PATH.includes('/usr/bin'));
-	t.false(PATH.includes(os.EOL));
+	t.false(PATH.includes('\n'));
 });
 
 test('sync', t => {
 	const PATH = m.sync();
 	t.true(PATH.includes('/usr/bin'));
-	t.false(PATH.includes(os.EOL));
+	t.false(PATH.includes('\n'));
 });