fix: pass mode to yarn up --recursive

.yarn/versions/5ee1a872.yml

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

packages/acceptance-tests/pkg-tests-specs/sources/commands/__snapshots__/up.test.ts.snap

@@ -0,0 +1,29 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`Commands up it should skip build scripts when using --mode=skip-build (recursive) 1`] = `
+"➤ YN0000: · Yarn 0.0.0
+➤ YN0000: ┌ Resolution step
+➤ YN0085: │ + no-deps-scripted@npm:1.0.0
+➤ YN0000: └ Completed
+➤ YN0000: ┌ Fetch step
+➤ YN0013: │ A package was added to the cache (+ 2.07 KiB).
+➤ YN0000: └ Completed
+➤ YN0000: ┌ Link step
+➤ YN0000: └ Completed
+➤ YN0000: · Done
+"
+`;
+
+exports[`Commands up it should skip build scripts when using --mode=skip-build 1`] = `
+"➤ YN0000: · Yarn 0.0.0
+➤ YN0000: ┌ Resolution step
+➤ YN0085: │ + no-deps-scripted@npm:1.0.0
+➤ YN0000: └ Completed
+➤ YN0000: ┌ Fetch step
+➤ YN0013: │ A package was added to the cache (+ 2.07 KiB).
+➤ YN0000: └ Completed
+➤ YN0000: ┌ Link step
+➤ YN0000: └ Completed
+➤ YN0000: · Done
+"
+`;

packages/acceptance-tests/pkg-tests-specs/sources/commands/install.test.js

@@ -28,7 +28,7 @@ describe(`Commands`, () => {
       }, async ({path, run, source}) => {
         const {stdout} = await run(`install`, `--inline-builds`, `--mode=skip-build`);
 
-        await expect(stdout).toMatchSnapshot();
+        expect(stdout).toMatchSnapshot();
       }),
     );
 

packages/acceptance-tests/pkg-tests-specs/sources/commands/up.test.ts

@@ -119,7 +119,7 @@ describe(`Commands`, () => {
           [`no-deps`]: `1.0.0`,
         },
       }, async ({path, run, source}) => {
-        await run(`add`, `-F`, `no-deps@latest`);
+        await run(`up`, `-F`, `no-deps@latest`);
 
         await expect(xfs.readJsonPromise(ppath.join(path, Filename.manifest))).resolves.toMatchObject({
           dependencies: {
@@ -128,5 +128,41 @@ describe(`Commands`, () => {
         });
       }),
     );
+
+    test(
+      `it should skip build scripts when using --mode=skip-build`,
+      makeTemporaryEnv({
+        dependencies: {
+          [`no-deps-scripted`]: `1.0.0`,
+        },
+      }, async ({path, run, source}) => {
+        const {stdout} = await run(`up`, `no-deps-scripted`, `--mode=skip-build`, {
+          env: {
+            YARN_ENABLE_INLINE_BUILDS: `1`,
+          },
+        });
+
+        expect(stdout).not.toContain(`must be built`);
+        expect(stdout).toMatchSnapshot();
+      }),
+    );
+
+    test(
+      `it should skip build scripts when using --mode=skip-build (recursive)`,
+      makeTemporaryEnv({
+        dependencies: {
+          [`no-deps-scripted`]: `1.0.0`,
+        },
+      }, async ({path, run, source}) => {
+        const {stdout} = await run(`up`, `no-deps-scripted`, `--recursive`, `--mode=skip-build`, {
+          env: {
+            YARN_ENABLE_INLINE_BUILDS: `1`,
+          },
+        });
+
+        expect(stdout).not.toContain(`must be built`);
+        expect(stdout).toMatchSnapshot();
+      }),
+    );
   });
 });

packages/plugin-essentials/sources/commands/up.ts

@@ -145,6 +145,7 @@ export default class UpCommand extends BaseCommand {
       stdout: this.context.stdout,
     }, {
       cache,
+      mode: this.mode,
     });
   }