perf: remove unnecessary realpath on --cwd

.yarn/versions/10553996.yml

@@ -0,0 +1,23 @@
+releases:
+  "@yarnpkg/cli": patch
+
+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-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/_entry.test.ts

@@ -1,7 +1,7 @@
-export {};
+import {PortablePath, npath, xfs} from '@yarnpkg/fslib';
 
 describe(`Entry`, () => {
-  describe(`version option`, () => {
+  describe(`--version`, () => {
     test(
       `it should print the version from the package.json when given --version`,
       makeTemporaryEnv({}, async ({path, run, source}) => {
@@ -18,4 +18,82 @@ describe(`Entry`, () => {
       }),
     );
   });
+
+  describe(`--cwd`, () => {
+    test(`should use the specified --cwd (relative path)`, makeTemporaryEnv({}, async ({path, run, source}) => {
+      await run(`install`);
+
+      await xfs.mkdirPromise(`${path}/packages` as PortablePath);
+
+      await expect(run(`--cwd`, `packages`, `exec`, `pwd`)).resolves.toMatchObject({
+        stdout: `${npath.fromPortablePath(`${path}/packages`)}\n`,
+      });
+    }));
+
+    test(`should use the specified --cwd (absolute path)`, makeTemporaryEnv({}, async ({path, run, source}) => {
+      await run(`install`);
+
+      await xfs.mkdirPromise(`${path}/packages` as PortablePath);
+
+      await expect(run(`--cwd`, npath.fromPortablePath(`${path}/packages`), `exec`, `pwd`)).resolves.toMatchObject({
+        stdout: `${npath.fromPortablePath(`${path}/packages`)}\n`,
+      });
+    }));
+
+    test(`should use the specified --cwd (relative symlink)`, makeTemporaryEnv({}, async ({path, run, source}) => {
+      await run(`install`);
+
+      await xfs.mkdirPromise(`${path}/packages` as PortablePath);
+      await xfs.symlinkPromise(`${path}/packages` as PortablePath, `${path}/modules` as PortablePath);
+
+      await expect(run(`--cwd`, `modules`, `exec`, `pwd`)).resolves.toMatchObject({
+        stdout: `${npath.fromPortablePath(`${path}/modules`)}\n`,
+      });
+    }));
+
+    test(`should use the specified --cwd (absolute symlink)`, makeTemporaryEnv({}, async ({path, run, source}) => {
+      await run(`install`);
+
+      await xfs.mkdirPromise(`${path}/packages` as PortablePath);
+      await xfs.symlinkPromise(`${path}/packages` as PortablePath, `${path}/modules` as PortablePath);
+
+      await expect(run(`--cwd`, npath.fromPortablePath(`${path}/modules`), `exec`, `pwd`)).resolves.toMatchObject({
+        stdout: `${npath.fromPortablePath(`${path}/modules`)}\n`,
+      });
+    }));
+
+    test(`should use the specified --cwd (bound relative path)`, makeTemporaryEnv({}, async ({path, run, source}) => {
+      await run(`install`);
+
+      await xfs.mkdirPromise(`${path}/packages` as PortablePath);
+
+      await expect(run(`--cwd=packages`, `exec`, `pwd`)).resolves.toMatchObject({
+        stdout: `${npath.fromPortablePath(`${path}/packages`)}\n`,
+      });
+    }));
+
+    test(`should use the specified --cwd (bound absolute path)`, makeTemporaryEnv({}, async ({path, run, source}) => {
+      await run(`install`);
+
+      await xfs.mkdirPromise(`${path}/packages` as PortablePath);
+
+      await expect(run(`--cwd=${npath.fromPortablePath(`${path}/packages`)}`, `exec`, `pwd`)).resolves.toMatchObject({
+        stdout: `${npath.fromPortablePath(`${path}/packages`)}\n`,
+      });
+    }));
+
+    test(`should use the specified --cwd (inside script)`, makeTemporaryEnv({
+      scripts: {
+        foo: `yarn --cwd=packages exec pwd`,
+      },
+    }, async ({path, run, source}) => {
+      await run(`install`);
+
+      await xfs.mkdirPromise(`${path}/packages` as PortablePath);
+
+      await expect(run(`run`, `foo`)).resolves.toMatchObject({
+        stdout: `${npath.fromPortablePath(`${path}/packages`)}\n`,
+      });
+    }));
+  });
 });

packages/yarnpkg-cli/sources/lib.ts

@@ -94,14 +94,14 @@ function checkCwd(cli: YarnCli, argv: Array<string>) {
 
   let postCwdArgv = argv;
   if (argv.length >= 2 && argv[0] === `--cwd`) {
-    cwd = xfs.realpathSync(npath.toPortablePath(argv[1]));
+    cwd = npath.toPortablePath(argv[1]);
     postCwdArgv = argv.slice(2);
   } else if (argv.length >= 1 && argv[0].startsWith(`--cwd=`)) {
-    cwd = xfs.realpathSync(npath.toPortablePath(argv[0].slice(6)));
+    cwd = npath.toPortablePath(argv[0].slice(6));
     postCwdArgv = argv.slice(1);
   }
 
-  cli.defaultContext.cwd = cwd ?? ppath.cwd();
+  cli.defaultContext.cwd = ppath.resolve(cwd ?? ppath.cwd());
   return postCwdArgv;
 }