cleanup: Consolidate code a bit.

Author: varungandhi-src

src/CommandLineOptions.ts

@@ -38,7 +38,7 @@ export function mainCommand(
     .option('--yarn-workspaces', 'whether to index all yarn workspaces', false)
     .option(
       '--yarn-berry-workspaces',
-      'whether to index all yarn v3 workspaces',
+      '(deprecated) use --yarn-workspaces instead',
       false
     )
     .option(

src/main.ts

@@ -31,9 +31,9 @@ export function indexCommand(
   options: MultiProjectOptions
 ): void {
   if (options.yarnWorkspaces) {
-    projects.push(...listYarnWorkspaces(options.cwd))
+    projects.push(...listYarnWorkspaces(options.cwd, 'tryYarn1'))
   } else if (options.yarnBerryWorkspaces) {
-    projects.push(...listYarnBerryWorkspaces(options.cwd))
+    projects.push(...listYarnWorkspaces(options.cwd, 'yarn2Plus'))
   } else if (projects.length === 0) {
     projects.push(options.cwd)
   }
@@ -200,52 +200,30 @@ function defaultCompilerOptions(configFileName?: string): ts.CompilerOptions {
   return options
 }
 
-function listYarnBerryWorkspaces(directory: string): string[] {
-  const result: string[] = []
-  const lines = child_process
-    .execSync('yarn workspaces list --json', {
-      cwd: directory,
-      encoding: 'utf-8',
-      maxBuffer: 1024 * 1024 * 5, // 5MB
-    })
-    .split('\n')
-  for (const line of lines) {
-    if (!line) {
-      continue
-    }
-    const location = 'location'
-    const json = JSON.parse(line)
-    if (json[location] !== undefined) {
-      // eslint-disable-next-line @typescript-eslint/no-unsafe-argument, @typescript-eslint/no-unsafe-member-access
-      result.push(path.join(directory, json[location]))
-    }
-  }
-  return result
-}
-
-function listYarnWorkspaces(directory: string): string[] {
+function listYarnWorkspaces(directory: string, yarnVersion: 'tryYarn1' | 'yarn2Plus'): string[] {
   const runYarn = (cmd: string): string =>
     child_process.execSync(cmd, {
       cwd: directory,
       encoding: 'utf-8',
       maxBuffer: 1024 * 1024 * 5, // 5MB
     })
   const result: string[] = []
-  try {
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
-    const json = JSON.parse(
-      // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
-      JSON.parse(runYarn('yarn --silent --json workspaces info')).data
-    )
-    for (const key of Object.keys(json)) {
-      const location = 'location'
-      // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
-      if (json[key][location] !== undefined) {
+  const yarn1WorkspaceInfo = (): void => {
+      // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
+      const json = JSON.parse(
         // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
-        result.push(path.join(directory, json[key][location]))
+        JSON.parse(runYarn('yarn --silent --json workspaces info')).data
+      )
+      for (const key of Object.keys(json)) {
+        const location = 'location'
+        // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+        if (json[key][location] !== undefined) {
+          // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+          result.push(path.join(directory, json[key][location]))
+        }
       }
-    }
-  } catch {
+  };
+  const yarn2PlusWorkspaceInfo = (): void => {
     const jsonLines = runYarn('yarn --json workspaces list').split(
       /\r?\n|\r|\n/g
     )
@@ -262,5 +240,14 @@ function listYarnWorkspaces(directory: string): string[] {
       }
     }
   }
+  if (yarnVersion === 'tryYarn1') {
+    try {
+      yarn1WorkspaceInfo()
+    } catch {
+      yarn2PlusWorkspaceInfo()
+    }
+  } else {
+    yarn2PlusWorkspaceInfo()
+  }
   return result
 }