From 7936a00c1eca818f5a844ed73bfc915defa5626a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Mat=C4=9Bj=20Chalk?= Date: Wed, 5 Mar 2025 16:01:03 +0100 Subject: [PATCH] fix(plugin-js-packages): ignore non-empty stderr --- packages/plugin-js-packages/src/lib/runner/index.ts | 13 ++----------- 1 file changed, 2 insertions(+), 11 deletions(-) diff --git a/packages/plugin-js-packages/src/lib/runner/index.ts b/packages/plugin-js-packages/src/lib/runner/index.ts index e2784e30e..f8bca562d 100644 --- a/packages/plugin-js-packages/src/lib/runner/index.ts +++ b/packages/plugin-js-packages/src/lib/runner/index.ts @@ -78,18 +78,13 @@ async function processOutdated( packageJsonPaths: PackageJsonPaths, ) { const pm = packageManagers[id]; - const { stdout, stderr } = await executeProcess({ + const { stdout } = await executeProcess({ command: pm.command, args: pm.outdated.commandArgs, cwd: process.cwd(), ignoreExitCode: true, // outdated returns exit code 1 when outdated dependencies are found }); - // Successful outdated check has empty stderr - if (stderr) { - throw new Error(`JS packages plugin: outdated error: ${stderr}`); - } - // Locate all package.json files in the repository if not provided const finalPaths = Array.isArray(packageJsonPaths) ? packageJsonPaths @@ -122,16 +117,12 @@ async function processAudit( const auditResults = await Promise.allSettled( compatibleAuditDepGroups.map( async (depGroup): Promise<[DependencyGroup, AuditResult]> => { - const { stdout, stderr } = await executeProcess({ + const { stdout } = await executeProcess({ command: pm.command, args: pm.audit.getCommandArgs(depGroup), cwd: process.cwd(), ignoreExitCode: pm.audit.ignoreExitCode, }); - // Successful audit check has empty stderr - if (stderr) { - throw new Error(`JS packages plugin: audit error: ${stderr}`); - } return [depGroup, pm.audit.unifyResult(stdout)]; }, ),