From 471f2ac4da990ccd6bae634c60d0d069842b4234 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Juan=20de=20Dios=20Mart=C3=ADnez=20Vallejo?= Date: Mon, 24 Mar 2025 21:21:28 +0100 Subject: [PATCH] fix: remove --legacy-peer-deps flag on install --- lib/bun-package-manager.ts | 5 +---- lib/node-package-manager.ts | 2 +- 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/lib/bun-package-manager.ts b/lib/bun-package-manager.ts index b2d5b8f842..cfd5ffc057 100644 --- a/lib/bun-package-manager.ts +++ b/lib/bun-package-manager.ts @@ -47,10 +47,7 @@ export class BunPackageManager extends BasePackageManager { const jsonContentBefore = this.$fs.readJson(packageJsonPath); const flags = this.getFlagsString(config, true); - // TODO: Confirm desired behavior. The npm version uses --legacy-peer-deps - // by default, we could use `--no-peer` for Bun if similar is needed; the - // pnpm version uses `--shamefully-hoist`, but Bun has no similar flag. - let params = ["install", "--legacy-peer-deps"]; + let params = ["install"]; const isInstallingAllDependencies = packageName === pathToSave; if (!isInstallingAllDependencies) { params.push(packageName); diff --git a/lib/node-package-manager.ts b/lib/node-package-manager.ts index 254ff7e5e5..bf63cae523 100644 --- a/lib/node-package-manager.ts +++ b/lib/node-package-manager.ts @@ -47,7 +47,7 @@ export class NodePackageManager extends BasePackageManager { const jsonContentBefore = this.$fs.readJson(packageJsonPath); const flags = this.getFlagsString(config, true); - let params = ["install", "--legacy-peer-deps"]; + let params = ["install"]; const isInstallingAllDependencies = packageName === pathToSave; if (!isInstallingAllDependencies) { params.push(packageName);