From c9f6bd05c9cb2e4934a9b54333db3988ef67bf26 Mon Sep 17 00:00:00 2001 From: Maurizio Casimirri Date: Thu, 22 Feb 2024 12:48:20 +0100 Subject: [PATCH 1/2] chore: hide confusing snyk output --- scripts/snyk-test.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/snyk-test.js b/scripts/snyk-test.js index 1655e05b88b..61d06217bb1 100644 --- a/scripts/snyk-test.js +++ b/scripts/snyk-test.js @@ -24,7 +24,7 @@ async function snykTest(cwd) { '--dev', `--json-file-output=${tmpPath}`, ], - { cwd, stdio: 'inherit' } + { cwd } ); } catch (err) { console.warn(err); From 6e36a66c3e5f47aa7894b9f88460b77b5048c6b3 Mon Sep 17 00:00:00 2001 From: Maurizio Casimirri Date: Thu, 22 Feb 2024 16:57:07 +0100 Subject: [PATCH 2/2] actually suppress output unless the snyk test fails --- scripts/snyk-test.js | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/scripts/snyk-test.js b/scripts/snyk-test.js index 61d06217bb1..a8b11655d8a 100644 --- a/scripts/snyk-test.js +++ b/scripts/snyk-test.js @@ -9,6 +9,8 @@ const execFile = promisify(childProcess.execFile); async function snykTest(cwd) { const tmpPath = path.join(os.tmpdir(), 'tempfile-' + Date.now()); + let execErr; + try { console.info(`testing ${cwd} ...`); await fs.mkdir(path.join(cwd, `node_modules`), { recursive: true }); @@ -27,14 +29,16 @@ async function snykTest(cwd) { { cwd } ); } catch (err) { - console.warn(err); + execErr = err; } const res = JSON.parse(await fs.readFile(tmpPath)); console.info(`testing ${cwd} done.`); return res; } catch (err) { - console.error(`testing ${cwd} failed. ${err.message}`); + console.error( + `testing ${cwd} failed. ${err.message}. Exec error: ${execErr}` + ); } finally { try { await fs.rm(tmpPath);