fix(test, monitor): handle error from pnpm FF check (#5348)

snyk · Jun 27, 2024 · 4f061c9 · 4f061c9
2 parents 715c5bf + 68ffb2b
commit 4f061c9
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 3 deletions.
diff --git a/src/cli/commands/monitor/index.ts b/src/cli/commands/monitor/index.ts
@@ -159,7 +159,15 @@ export default async function monitor(...args0: MethodArgs): Promise<any> {
     );
   }
 
-  const hasPnpmSupport = await hasFeatureFlag(PNPM_FEATURE_FLAG, options);
+  let hasPnpmSupport = false;
+  try {
+    hasPnpmSupport = (await hasFeatureFlag(
+      PNPM_FEATURE_FLAG,
+      options,
+    )) as boolean;
+  } catch (err) {
+    hasPnpmSupport = false;
+  }
 
   const featureFlags = hasPnpmSupport
     ? new Set<string>([PNPM_FEATURE_FLAG])

diff --git a/src/lib/snyk-test/index.js b/src/lib/snyk-test/index.js
@@ -29,9 +29,13 @@ async function test(root, options, callback) {
 }
 
 async function executeTest(root, options) {
+  let hasPnpmSupport = false;
+  try {
+    hasPnpmSupport = await hasFeatureFlag(PNPM_FEATURE_FLAG, options);
+  } catch (err) {
+    hasPnpmSupport = false;
+  }
   try {
-    const hasPnpmSupport = await hasFeatureFlag(PNPM_FEATURE_FLAG, options);
-
     const featureFlags = hasPnpmSupport
       ? new Set([PNPM_FEATURE_FLAG])
       : new Set([]);