Make --raw compatible with --hide (#486)

Author: PoQuatre

bin/concurrently.spec.ts

@@ -234,6 +234,22 @@ describe('--hide', () => {
         expect(lines).toContainEqual(expect.stringContaining('foo'));
         expect(lines).not.toContainEqual(expect.stringContaining('bar'));
     });
+
+    it('hides the output of a process by its index in raw mode', async () => {
+        const lines = await run('--hide 1 --raw "echo foo" "echo bar"').getLogLines();
+
+        expect(lines).toHaveLength(1);
+        expect(lines).toContainEqual(expect.stringContaining('foo'));
+        expect(lines).not.toContainEqual(expect.stringContaining('bar'));
+    });
+
+    it('hides the output of a process by its name in raw mode', async () => {
+        const lines = await run('-n foo,bar --hide bar --raw "echo foo" "echo bar"').getLogLines();
+
+        expect(lines).toHaveLength(1);
+        expect(lines).toContainEqual(expect.stringContaining('foo'));
+        expect(lines).not.toContainEqual(expect.stringContaining('bar'));
+    });
 });
 
 describe('--group', () => {

src/concurrently.ts

@@ -4,7 +4,14 @@ import { cpus } from 'os';
 import { Writable } from 'stream';
 import treeKill from 'tree-kill';
 
-import { CloseEvent, Command, CommandInfo, KillProcess, SpawnCommand } from './command';
+import {
+    CloseEvent,
+    Command,
+    CommandIdentifier,
+    CommandInfo,
+    KillProcess,
+    SpawnCommand,
+} from './command';
 import { CommandParser } from './command-parser/command-parser';
 import { ExpandArguments } from './command-parser/expand-arguments';
 import { ExpandNpmShortcut } from './command-parser/expand-npm-shortcut';
@@ -94,6 +101,11 @@ export type ConcurrentlyOptions = {
      */
     raw?: boolean;
 
+    /**
+     * Which command(s) should have their output hidden.
+     */
+    hide?: CommandIdentifier | CommandIdentifier[];
+
     /**
      * The current working directory of commands which didn't specify one.
      * Defaults to `process.cwd()`.
@@ -169,6 +181,13 @@ export function concurrently(
         commandParsers.push(new ExpandArguments(options.additionalArguments));
     }
 
+    // To avoid empty strings from hiding the output of commands that don't have a name,
+    // keep in the list of commands to hide only strings with some length.
+    // This might happen through the CLI when no `--hide` argument is specified, for example.
+    const hide = _.castArray(options.hide)
+        .filter((name) => name || name === 0)
+        .map(String);
+
     let commands = _(baseCommands)
         .map(mapToCommandInfo)
         .flatMap((command) => parseCommand(command, commandParsers))
@@ -183,6 +202,7 @@ export function concurrently(
                     raw: command.raw ?? options.raw,
                     env: command.env,
                     cwd: command.cwd || options.cwd,
+                    hide: hide.includes(String(index)) || hide.includes(command.name),
                 }),
                 options.spawn,
                 options.kill,

src/index.ts

@@ -110,6 +110,7 @@ export function concurrently(
         raw: options.raw,
         successCondition: options.successCondition,
         cwd: options.cwd,
+        hide: options.hide,
         logger,
         outputStream: options.outputStream || process.stdout,
         group: options.group,

src/spawn.spec.ts

@@ -31,6 +31,10 @@ describe('getSpawnOpts()', () => {
         expect(getSpawnOpts({ raw: true }).stdio).toBe('inherit');
     });
 
+    it('unsets stdio when raw and hide', () => {
+        expect(getSpawnOpts({ raw: true, hide: true }).stdio).toBeUndefined();
+    });
+
     it('merges FORCE_COLOR into env vars if color supported', () => {
         const process = { ...baseProcess, env: { foo: 'bar' } };
         expect(getSpawnOpts({ process, colorSupport: false }).env).toEqual(process.env);

src/spawn.ts

@@ -28,6 +28,7 @@ export const getSpawnOpts = ({
     process = global.process,
     raw = false,
     env = {},
+    hide = false,
 }: {
     /**
      * What the color support of the spawned processes should be.
@@ -58,9 +59,15 @@ export const getSpawnOpts = ({
      * Map of custom environment variables to include in the spawn options.
      */
     env?: Record<string, unknown>;
+
+    /**
+     * Whether to hide the standard output.
+     * Defaults to false.
+     */
+    hide?: boolean;
 }): SpawnOptions => ({
     cwd: cwd || process.cwd(),
-    ...(raw && { stdio: 'inherit' as const }),
+    ...(raw && !hide && { stdio: 'inherit' as const }),
     ...(/^win/.test(process.platform) && { detached: false }),
     env: {
         ...(colorSupport ? { FORCE_COLOR: colorSupport.level.toString() } : {}),