feat!: dedupe-peer-dependents is true by default (#6185)

.changeset/dry-singers-drop.md

@@ -0,0 +1,6 @@
+---
+"@pnpm/core": major
+"pnpm": major
+---
+
+`dedupe-peer-dependents` is `true` by default.

config/config/src/index.ts

@@ -186,7 +186,7 @@ export async function getConfig (
     bail: true,
     color: 'auto',
     'deploy-all-files': false,
-    'dedupe-peer-dependents': false,
+    'dedupe-peer-dependents': true,
     'enable-modules-dir': true,
     'extend-node-path': true,
     'fetch-retries': 2,

pkg-manager/core/src/install/extendInstallOptions.ts

@@ -206,7 +206,7 @@ const defaults = async (opts: InstallOptions) => {
     modulesCacheMaxAge: 7 * 24 * 60,
     resolveSymlinksInInjectedDirs: false,
     dedupeDirectDeps: true,
-    dedupePeerDependents: false,
+    dedupePeerDependents: true,
     resolvePeersFromWorkspaceRoot: true,
     extendNodePath: true,
   } as StrictInstallOptions

pkg-manager/core/test/install/multipleImporters.ts

@@ -1465,6 +1465,7 @@ test('resolve a subdependency from the workspace and use it as a peer', async ()
   await mutateModules(importers, await testDefaults({
     allProjects,
     autoInstallPeers: false,
+    dedupePeerDependents: false,
     linkWorkspacePackagesDepth: Infinity,
     strictPeerDependencies: false,
     workspacePackages,

pkg-manager/core/test/install/peerDependencies.ts

@@ -59,6 +59,7 @@ test('nothing is needlessly removed from node_modules', async () => {
   prepareEmpty()
   const opts = await testDefaults({
     autoInstallPeers: false,
+    dedupePeerDependents: false,
     modulesCacheMaxAge: 0,
     strictPeerDependencies: false,
   })
@@ -151,7 +152,13 @@ test('the right peer dependency is used in every workspace package', async () =>
       rootDir: path.resolve('project-2'),
     },
   ]
-  await mutateModules(importers, await testDefaults({ allProjects, autoInstallPeers: false, lockfileOnly: true, strictPeerDependencies: false }))
+  await mutateModules(importers, await testDefaults({
+    allProjects,
+    autoInstallPeers: false,
+    dedupePeerDependents: false,
+    lockfileOnly: true,
+    strictPeerDependencies: false,
+  }))
 
   const lockfile = await readYamlFile<Lockfile>(path.resolve(WANTED_LOCKFILE))
 

pnpm/test/recursive/misc.ts

@@ -460,7 +460,8 @@ test('set recursive-install to false in .npmrc would disable recursive install i
 
   process.chdir('workspace')
   await fs.writeFile('pnpm-workspace.yaml', '', 'utf8')
-  await fs.writeFile('.npmrc', 'recursive-install = false', 'utf8')
+  await fs.writeFile('.npmrc', `recursive-install = false
+dedupe-peer-dependents = false`, 'utf8')
 
   process.chdir('project-1')
   await execPnpm(['install'])