fix: change installOptions to use npm in yarn PM (electron-userland#9197)

fix electron-userland#9187
electron-userland#9169

Author: beyondkmp

.changeset/afraid-icons-yawn.md

@@ -0,0 +1,5 @@
+---
+"app-builder-lib": patch
+---
+
+fix: change installOptions to use npm in yarn PM

packages/app-builder-lib/src/node-module-collector/packageManager.ts

@@ -77,7 +77,10 @@ export function detectPackageManager(cwd: string): PM {
   if (npm) detected.push(PM.NPM)
 
   if (detected.length === 1) {
-    return detected[0] === PM.YARN ? detectPackageManagerByEnv() : detected[0]
+    if (detected[0] === PM.YARN) {
+      return detectPackageManagerByEnv() === PM.YARN_BERRY ? PM.YARN_BERRY : PM.YARN
+    }
+    return detected[0]
   }
 
   // fallback: multiple lockfiles or none

packages/app-builder-lib/src/node-module-collector/yarnNodeModulesCollector.ts

@@ -7,5 +7,6 @@ export class YarnNodeModulesCollector extends NpmNodeModulesCollector {
   }
 
   // note: do not override instance-var `pmCommand`. We explicitly use npm for the json payload
-  public readonly installOptions = { manager: PM.YARN, lockfile: "yarn.lock" }
+  // yarn list cannot get the detailed dependencies path info, But we can use npm to get all dependencies path info
+  public readonly installOptions = { manager: PM.NPM, lockfile: "yarn.lock" }
 }