feat(pnp)!: default pnpFallbackMode to none

.yarn/versions/7a4f72bf.yml

@@ -0,0 +1,23 @@
+releases:
+  "@yarnpkg/cli": major
+  "@yarnpkg/plugin-pnp": major
+
+declined:
+  - "@yarnpkg/plugin-compat"
+  - "@yarnpkg/plugin-constraints"
+  - "@yarnpkg/plugin-dlx"
+  - "@yarnpkg/plugin-essentials"
+  - "@yarnpkg/plugin-init"
+  - "@yarnpkg/plugin-interactive-tools"
+  - "@yarnpkg/plugin-nm"
+  - "@yarnpkg/plugin-npm-cli"
+  - "@yarnpkg/plugin-pack"
+  - "@yarnpkg/plugin-patch"
+  - "@yarnpkg/plugin-pnpm"
+  - "@yarnpkg/plugin-stage"
+  - "@yarnpkg/plugin-typescript"
+  - "@yarnpkg/plugin-version"
+  - "@yarnpkg/plugin-workspace-tools"
+  - "@yarnpkg/builder"
+  - "@yarnpkg/core"
+  - "@yarnpkg/doctor"

.yarnrc.yml

@@ -5,6 +5,7 @@ changesetIgnorePatterns:
   - packages/*/tests/**
   - packages/plugin-compat/extra/**
   - packages/gatsby/**
+  - packages/docusaurus/**
 
 compressionLevel: mixed
 

packages/acceptance-tests/pkg-tests-specs/sources/pnp.test.js

@@ -125,9 +125,7 @@ describe(`Plug'n'Play`, () => {
         dependencies: {[`various-requires`]: `1.0.0`, [`no-deps`]: `1.0.0`},
       },
       {
-        // By default tests are executed with the fallback disabled; this
-        // setting forces this test to execute in the default mode instead
-        pnpFallbackMode: undefined,
+        pnpFallbackMode: `dependencies-only`,
       },
       async ({path, run, source}) => {
         await run(`install`);
@@ -147,9 +145,7 @@ describe(`Plug'n'Play`, () => {
         dependencies: {[`various-requires`]: `1.0.0`, [`no-deps`]: `1.0.0`},
       },
       {
-        // By default tests are executed with the fallback disabled; this
-        // setting forces this test to execute in the default mode instead
-        pnpFallbackMode: undefined,
+        pnpFallbackMode: `dependencies-only`,
       },
       async ({path, run, source}) => {
         await run(`install`);
@@ -170,9 +166,7 @@ describe(`Plug'n'Play`, () => {
         dependencies: {[`no-deps`]: `1.0.0`},
       },
       {
-        // By default tests are executed with the fallback disabled; this
-        // setting forces this test to execute in the default mode instead
-        pnpFallbackMode: undefined,
+        pnpFallbackMode: `dependencies-only`,
       },
       async ({path, run, source}) => {
         await writeJson(`${path}/packages/workspace-a/package.json`, {

packages/acceptance-tests/pkg-tests-specs/sources/protocols/links.test.js

@@ -113,8 +113,6 @@ describe(`Protocols`, () => {
           [`no-deps`]: `1.0.0`,
           [`foo`]: `link:./my-dir`,
         },
-      }, {
-        pnpFallbackMode: `none`,
       }, async ({path, run, source}) => {
         await xfs.mkdirPromise(`${path}/my-dir`);
         await xfs.writeFilePromise(`${path}/my-dir/index.js`, `module.exports = require('no-deps');\n`);

packages/docusaurus/static/configuration/yarnrc.json

@@ -621,9 +621,9 @@
     },
     "pnpFallbackMode": {
       "_package": "@yarnpkg/plugin-pnp",
-      "description": "Enumeration whose values (`none`, `dependencies-only`, `all`) define in which capacity should the PnP hook allow packages to rely on the builtin fallback mechanism. In `dependencies-only` mode (the default), your workspaces aren't allowed to use it.",
+      "description": "Enumeration whose values (`none`, `dependencies-only`, `all`) define in which capacity should the PnP hook allow packages to rely on the builtin fallback mechanism. In `dependencies-only` mode, your workspaces aren't allowed to use it.",
       "enum": ["none", "dependencies-only", "all"],
-      "default": "dependencies-only"
+      "default": "none"
     },
     "pnpIgnorePatterns": {
       "_package": "@yarnpkg/plugin-pnp",

packages/gatsby/static/configuration/yarnrc.json

@@ -645,9 +645,9 @@
     },
     "pnpFallbackMode": {
       "_package": "@yarnpkg/plugin-pnp",
-      "description": "Enumeration whose values (`none`, `dependencies-only`, `all`) define in which capacity should the PnP hook allow packages to rely on the builtin fallback mechanism. In `dependencies-only` mode (the default), your workspaces aren't allowed to use it.",
+      "description": "Enumeration whose values (`none`, `dependencies-only`, `all`) define in which capacity should the PnP hook allow packages to rely on the builtin fallback mechanism. In `dependencies-only` mode, your workspaces aren't allowed to use it.",
       "enum": ["none", "dependencies-only", "all"],
-      "default": "dependencies-only"
+      "default": "none"
     },
     "pnpIgnorePatterns": {
       "_package": "@yarnpkg/plugin-pnp",

packages/plugin-pnp/sources/index.ts

@@ -122,7 +122,7 @@ const plugin: Plugin<CoreHooks & StageHooks> = {
     pnpFallbackMode: {
       description: `If true, the generated PnP loader will follow the top-level fallback rule`,
       type: SettingsType.STRING,
-      default: `dependencies-only`,
+      default: `none`,
     },
     pnpUnpluggedFolder: {
       description: `Folder where the unplugged packages must be stored`,

scripts/e2e-setup-ci.sh

@@ -25,9 +25,6 @@ export YARN_ENABLE_INLINE_BUILDS=1
 # We want to allow installs to modify the lockfile
 export YARN_ENABLE_IMMUTABLE_INSTALLS=0
 
-# We want to make sure the projects work in a monorepo
-export YARN_PNP_FALLBACK_MODE=none
-
 # Otherwise git commit doesn't work, and some tools require it
 git config --global user.email "[email protected]"
 git config --global user.name "John Doe"