Simplify getModulePackageJSON usage

Author: valentinpalkovic

code/core/src/common/js-package-manager/BUNProxy.ts

@@ -87,12 +87,12 @@ export class BUNProxy extends JsPackageManager {
     return `bunx ${pkg}${specifier ? `@${specifier}` : ''} ${args.join(' ')}`;
   }
 
-  public async getModulePackageJSON(
-    packageName: string,
-    cwd = this.cwd
-  ): Promise<PackageJson | null> {
+  public async getModulePackageJSON(packageName: string): Promise<PackageJson | null> {
     const wantedPath = join('node_modules', packageName, 'package.json');
-    const packageJsonPath = find.up(wantedPath, { cwd, last: getProjectRoot() });
+    const packageJsonPath = find.up(wantedPath, {
+      cwd: this.primaryPackageJson.operationDir,
+      last: getProjectRoot(),
+    });
 
     if (!packageJsonPath) {
       return null;

code/core/src/common/js-package-manager/NPMProxy.ts

@@ -76,9 +76,12 @@ export class NPMProxy extends JsPackageManager {
     return `npm run ${command}`;
   }
 
-  async getModulePackageJSON(packageName: string, cwd = this.cwd): Promise<PackageJson | null> {
+  async getModulePackageJSON(packageName: string): Promise<PackageJson | null> {
     const wantedPath = join('node_modules', packageName, 'package.json');
-    const packageJsonPath = find.up(wantedPath, { cwd, last: getProjectRoot() });
+    const packageJsonPath = find.up(wantedPath, {
+      cwd: this.primaryPackageJson.operationDir,
+      last: getProjectRoot(),
+    });
 
     if (!packageJsonPath) {
       return null;

code/core/src/common/js-package-manager/PNPMProxy.ts

@@ -127,12 +127,9 @@ export class PNPMProxy extends JsPackageManager {
   }
 
   // TODO: Remove pnp compatibility code in SB11
-  public async getModulePackageJSON(
-    packageName: string,
-    cwd = this.cwd
-  ): Promise<PackageJson | null> {
+  public async getModulePackageJSON(packageName: string): Promise<PackageJson | null> {
     const pnpapiPath = find.any(['.pnp.js', '.pnp.cjs'], {
-      cwd,
+      cwd: this.primaryPackageJson.operationDir,
       last: getProjectRoot(),
     });
 

code/core/src/common/js-package-manager/Yarn1Proxy.ts

@@ -76,12 +76,12 @@ export class Yarn1Proxy extends JsPackageManager {
     });
   }
 
-  public async getModulePackageJSON(
-    packageName: string,
-    cwd = this.cwd
-  ): Promise<PackageJson | null> {
+  public async getModulePackageJSON(packageName: string): Promise<PackageJson | null> {
     const wantedPath = join('node_modules', packageName, 'package.json');
-    const packageJsonPath = find.up(wantedPath, { cwd, last: getProjectRoot() });
+    const packageJsonPath = find.up(wantedPath, {
+      cwd: this.primaryPackageJson.operationDir,
+      last: getProjectRoot(),
+    });
 
     if (!packageJsonPath) {
       return null;

code/core/src/common/js-package-manager/Yarn2Proxy.ts

@@ -146,9 +146,9 @@ export class Yarn2Proxy extends JsPackageManager {
   }
 
   // TODO: Remove pnp compatibility code in SB11
-  async getModulePackageJSON(packageName: string, cwd = this.cwd): Promise<PackageJson | null> {
+  async getModulePackageJSON(packageName: string): Promise<PackageJson | null> {
     const pnpapiPath = find.any(['.pnp.js', '.pnp.cjs'], {
-      cwd,
+      cwd: this.primaryPackageJson.operationDir,
       last: getProjectRoot(),
     });