feat: let shell be false

Author: antongolub

src/main/ts/spawn.ts

@@ -64,7 +64,7 @@ export interface TSpawnCtxNormalized {
   on:         Partial<TSpawnListeners>
   ac:         AbortController
   signal:     AbortController['signal']
-  shell:      string | true | undefined
+  shell:      string | boolean | undefined
   spawn:      typeof cp.spawn
   spawnSync:  typeof cp.spawnSync
   spawnOpts:  Record<string, any>

src/main/ts/x.ts

@@ -101,7 +101,8 @@ export const $: TShell = function(this: any, pieces?: any, ...args: any): any {
 }
 
 const ignite = (preset: any, pieces: TemplateStringsArray, ...args: any[]) => {
-  const cmd = buildCmd(preset.quote || quote, pieces as TemplateStringsArray, args)
+  const _quote = preset.quote || (preset.shell === false ? (arg: string) => arg : quote)
+  const cmd = buildCmd(_quote, pieces as TemplateStringsArray, args)
   const input = parseInput(preset.input)
   const run = cmd instanceof Promise
     ? (cb: TVoidCallback, ctx: TShellCtx) => cmd.then((cmd) => { ctx.cmd = cmd; cb() })

src/test/ts/x.test.ts

@@ -28,6 +28,9 @@ describe('$()', () => {
       console.error(error)
       assert.ok((error as Error).message.includes('exit code: 2 (Misuse of shell builtins)'))
     }
+
+    const err = await $`exit 2`.catch((error) => error)
+    assert.ok(err.message.includes('exit code: 2 (Misuse of shell builtins)'))
   })
 
   it('supports sync flow', () => {
@@ -74,6 +77,14 @@ describe('$()', () => {
     $({stdio: 'ignore', sync: true})`ls`
   })
 
+  it('works without shell', async () => {
+    const o1 = await $({shell: true})`exit 2 | exit 0`
+    const o2 = await $({shell: false, nothrow: true})`exit 1 | exit 0`
+
+    assert.equal(o1.status, 0)
+    assert.equal(o2.status, -2)
+  })
+
   it('supports presets', () => {
     const $$ = $({sync: true, cmd: 'echo foo'})
     const $$$ = $$({cmd: 'echo bar'})

target/cjs/index.cjs

@@ -141,7 +141,8 @@ var $ = function(pieces, ...args) {
   return (...args2) => $.apply(self ? (0, import_util4.assign)(self, pieces) : pieces, args2);
 };
 var ignite = (preset, pieces, ...args) => {
-  const cmd = (0, import_util4.buildCmd)(preset.quote || import_util4.quote, pieces, args);
+  const _quote = preset.quote || (preset.shell === false ? (arg) => arg : import_util4.quote);
+  const cmd = (0, import_util4.buildCmd)(_quote, pieces, args);
   const input = (0, import_util4.parseInput)(preset.input);
   const run = cmd instanceof Promise ? (cb, ctx) => cmd.then((cmd2) => {
     ctx.cmd = cmd2;

target/dts/spawn.d.ts

@@ -56,7 +56,7 @@ export interface TSpawnCtxNormalized {
     on: Partial<TSpawnListeners>;
     ac: AbortController;
     signal: AbortController['signal'];
-    shell: string | true | undefined;
+    shell: string | boolean | undefined;
     spawn: typeof cp.spawn;
     spawnSync: typeof cp.spawnSync;
     spawnOpts: Record<string, any>;
@@ -81,7 +81,7 @@ export declare const buildSpawnOpts: ({ spawnOpts, stdio, cwd, shell, input, env
     env: Record<string, string | undefined>;
     cwd: string;
     stdio: cp.StdioOptions;
-    shell: string | true | undefined;
+    shell: string | boolean | undefined;
     input: string | Buffer;
     windowsHide: boolean;
     detached: boolean;

target/esm/index.mjs

@@ -136,7 +136,8 @@ var $ = function(pieces, ...args) {
   return (...args2) => $.apply(self ? assign4(self, pieces) : pieces, args2);
 };
 var ignite = (preset, pieces, ...args) => {
-  const cmd = buildCmd(preset.quote || quote, pieces, args);
+  const _quote = preset.quote || (preset.shell === false ? (arg) => arg : quote);
+  const cmd = buildCmd(_quote, pieces, args);
   const input = parseInput(preset.input);
   const run = cmd instanceof Promise ? (cb, ctx) => cmd.then((cmd2) => {
     ctx.cmd = cmd2;