feat: simplify multi scan check with a function

src/cli/commands/monitor/index.ts

@@ -43,6 +43,7 @@ import {
   execShell,
 } from '../../../lib/monitor/dev-count-analysis';
 import { FailedToRunTestError, MonitorError } from '../../../lib/errors';
+import { isMultiProjectScan } from '../../../lib/is-multi-project-scan';
 
 const SEPARATOR = '\n-------------------------------------------------------\n';
 const debug = Debug('snyk');
@@ -120,7 +121,7 @@ async function monitor(...args0: MethodArgs): Promise<any> {
       validateMonitorPath(path, options.docker);
       let analysisType = 'all';
       let packageManager;
-      if (options.allProjects || options.yarnWorkspaces) {
+      if (isMultiProjectScan(options)) {
         analysisType = 'all';
       } else if (options.docker) {
         analysisType = 'docker';

src/cli/commands/test/index.ts

@@ -50,6 +50,7 @@ import * as utils from './utils';
 import { getIacDisplayedOutput } from './iac-output';
 import { getEcosystem, testEcosystem } from '../../../lib/ecosystems';
 import { TestLimitReachedError } from '../../../lib/errors';
+import { isMultiProjectScan } from '../../../lib/is-multi-project-scan';
 
 const debug = Debug('snyk-test');
 const SEPARATOR = '\n-------------------------------------------------------\n';
@@ -449,8 +450,7 @@ function displayResult(
   }
   const advertiseAllProjectsCount =
     projectType !== 'gradle' &&
-    !options.allProjects &&
-    !options.yarnWorkspaces &&
+    !isMultiProjectScan(options) &&
     foundProjectCount;
   if (advertiseAllProjectsCount) {
     multiProjAdvice = chalk.bold.white(

src/lib/is-multi-project-scan.ts

@@ -0,0 +1,7 @@
+import { Options, TestOptions, MonitorOptions, PolicyOptions } from './types';
+
+export function isMultiProjectScan(
+  options: Partial<Options & TestOptions & MonitorOptions & PolicyOptions>,
+): boolean {
+  return !!(options.allProjects || options.yarnWorkspaces);
+}

src/lib/options-validator.ts

@@ -2,14 +2,15 @@ import { MonitorOptions, Options, TestOptions } from './types';
 import * as config from './config';
 import { SupportedPackageManagers } from './package-managers';
 import * as reachableVulns from './reachable-vulns';
+import { isMultiProjectScan } from './is-multi-project-scan';
 
 export async function validateOptions(
   options: (Options & TestOptions) | (Options & MonitorOptions),
   packageManager?: SupportedPackageManagers,
 ): Promise<void> {
   if (options.reachableVulns) {
     // Throwing error only in case when both packageManager and allProjects not defined
-    if (!packageManager && !options.allProjects && !options.yarnWorkspaces) {
+    if (!packageManager && !isMultiProjectScan(options)) {
       throw new Error('Could not determine package manager');
     }
     const org = options.org || config.org;

src/lib/reachable-vulns.ts

@@ -12,6 +12,7 @@ import {
   UnsupportedFeatureFlagError,
 } from './errors';
 import { MonitorOptions, Options, TestOptions } from './types';
+import { isMultiProjectScan } from './is-multi-project-scan';
 
 const featureFlag = 'reachableVulns';
 
@@ -36,8 +37,7 @@ export async function validatePayload(
 ): Promise<boolean> {
   if (
     packageManager &&
-    !options.allProjects &&
-    !options.yarnWorkspaces &&
+    !isMultiProjectScan(options) &&
     !REACHABLE_VULNS_SUPPORTED_PACKAGE_MANAGERS.includes(packageManager)
   ) {
     throw new FeatureNotSupportedByPackageManagerError(

src/lib/snyk-test/index.js

@@ -9,6 +9,7 @@ const {
   UnsupportedPackageManagerError,
   NotSupportedIacFileError,
 } = require('../errors');
+const { isMultiProjectScan } = require('../is-multi-project-scan');
 
 async function test(root, options, callback) {
   if (typeof options === 'function') {
@@ -69,8 +70,7 @@ function validateProjectType(options, projectType) {
     if (
       !(
         options.docker ||
-        options.allProjects ||
-        options.yarnWorkspaces ||
+        isMultiProjectScan(options) ||
         pm.SUPPORTED_PACKAGE_MANAGER_NAME[projectType]
       )
     ) {