feat(bin): two bins (shell, non-shell)

Revert the default bin (cross-env) to its v3 behavior (not using the shell option). Add a new
(cross-env-shell)  which uses the shell option.

BREAKING CHANGE: Scripts using quotes or escape sequences will see a difference in behavior.
Switching to the second bin should resolve any issue.

Closes kentcdodds#99.

src/bin/cross-env-shell.js

@@ -0,0 +1,5 @@
+#!/usr/bin/env node
+
+const crossEnv = require('..')
+
+crossEnv(process.argv.slice(2), {shell: true})

src/index.js

@@ -6,15 +6,15 @@ module.exports = crossEnv
 
 const envSetterRegex = /(\w+)=('(.+)'|"(.+)"|(.+))/
 
-function crossEnv(args) {
+function crossEnv(args, options = {}) {
   const [envSetters, command, commandArgs] = parseCommand(args)
   if (command) {
     const proc = spawn(
       commandConvert(command),
       commandArgs.map(commandConvert),
       {
         stdio: 'inherit',
-        shell: true,
+        shell: options.shell,
         env: getEnvVars(envSetters),
       },
     )

src/index.test.js

@@ -44,7 +44,9 @@ it(`should handle equality signs in quoted strings`, () => {
 })
 
 it(`should handle quoted scripts`, () => {
-  crossEnv(['GREETING=Hi', 'NAME=Joe', 'echo $GREETING && echo $NAME'])
+  crossEnv(['GREETING=Hi', 'NAME=Joe', 'echo $GREETING && echo $NAME'], {
+    shell: true,
+  })
   expect(
     crossSpawnMock.spawn,
   ).toHaveBeenCalledWith('echo $GREETING && echo $NAME', [], {
@@ -94,7 +96,7 @@ function testEnvSetting(expected, ...envSettings) {
   expect(crossSpawnMock.spawn).toHaveBeenCalledTimes(1)
   expect(crossSpawnMock.spawn).toHaveBeenCalledWith('echo', ['hello world'], {
     stdio: 'inherit',
-    shell: true,
+    shell: undefined,
     env: Object.assign({}, process.env, env),
   })