From 701638e196c0679385ee623422c8a4b42b9efa36 Mon Sep 17 00:00:00 2001 From: Valentin Palkovic Date: Wed, 5 Apr 2023 14:11:21 +0200 Subject: [PATCH] Add platform-specific null piping for stderr for npm list command --- code/lib/cli/src/js-package-manager/NPMProxy.ts | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/code/lib/cli/src/js-package-manager/NPMProxy.ts b/code/lib/cli/src/js-package-manager/NPMProxy.ts index db79d2b3f320..709b6703eef8 100644 --- a/code/lib/cli/src/js-package-manager/NPMProxy.ts +++ b/code/lib/cli/src/js-package-manager/NPMProxy.ts @@ -1,4 +1,5 @@ import sort from 'semver/functions/sort'; +import { platform } from 'os'; import { JsPackageManager } from './JsPackageManager'; import type { PackageJson } from './PackageJson'; import type { InstallationMetadata, PackageMetadata } from './types'; @@ -51,7 +52,8 @@ export class NPMProxy extends JsPackageManager { } public findInstallations() { - const commandResult = this.executeCommand('npm', ['ls', '--json', '--depth=99']); + const pipeToNull = platform() === 'win32' ? '2>NUL' : '2>/dev/null'; + const commandResult = this.executeCommand('npm', ['ls', '--json', '--depth=99', pipeToNull]); try { const parsedOutput = JSON.parse(commandResult);