feat: add run --global option

Author: jdanil

.github/actions/prepare/action.yml

@@ -16,7 +16,7 @@ runs:
     - name: 'Build the standard bundle'
       if: steps.cache-build.outputs.cache-hit != 'true'
       run: |
-        node ./scripts/run-yarn.js build:cli
+        node ./scripts/run-yarn.js run --global build:cli
       shell: bash
     #endregion
 

.github/workflows/integration-workflow.yml

@@ -159,10 +159,10 @@ jobs:
     - name: 'Build bundle & plugins'
       run: |
         node --version
-        node ./scripts/run-yarn.js build:pnp:hook
-        node ./scripts/run-yarn.js build:cli
+        node ./scripts/run-yarn.js run --global build:pnp:hook
+        node ./scripts/run-yarn.js run --global build:cli
         mv ./packages/yarnpkg-cli/bundles/yarn.js ./packages/yarnpkg-cli/bundles/yarn-min.js
-        node ./scripts/run-yarn.js build:cli --no-minify
+        node ./scripts/run-yarn.js run --global build:cli --no-minify
       shell: bash
 
     - uses: actions/upload-artifact@v2
@@ -174,7 +174,7 @@ jobs:
 
     - name: 'Build vscode-zipfs'
       run: |
-        node ./scripts/run-yarn.js package:vscode-zipfs
+        node ./scripts/run-yarn.js run --global package:vscode-zipfs
       if: |
         success() || failure()
 

.github/workflows/package-workflow.yml

@@ -27,7 +27,7 @@ jobs:
     - name: 'Build the standard bundle'
       run: |
         set -ex
-        node ./scripts/run-yarn.js build:cli
+        node ./scripts/run-yarn.js run --global build:cli
 
     - name: 'Build the dist directory'
       run: |

.yarn/versions/c5422023.yml

@@ -0,0 +1,12 @@
+releases:
+  "@yarnpkg/plugin-essentials": major
+  "@yarnpkg/plugin-workspace-tools": major
+
+declined:
+  - "@yarnpkg/plugin-compat"
+  - "@yarnpkg/plugin-github"
+  - "@yarnpkg/fslib"
+  - "@yarnpkg/nm"
+  - "@yarnpkg/parsers"
+  - "@yarnpkg/pnpify"
+  - "@yarnpkg/shell"

CHANGELOG.md

@@ -16,6 +16,7 @@ Yarn now accepts sponsorships! Please give a look at our [OpenCollective](https:
 - Some legacy layers have been sunset:
   - Plugins cannot access the Clipanion 2 APIs anymore (upgrade to [Clipanion 3](https://github.com/arcanis/clipanion))
   - Plugins cannot access the internal copy of Yup anymore (use [Typanion](https://github.com/arcanis/typanion) instead)
+- Scripts containing a colon are no longer implicitly global. The `--global` option should be used instead.
 
 ### **API Changes**
 

constraints.pro

@@ -69,7 +69,7 @@ gen_enforced_field(WorkspaceCwd, 'scripts.update-local', '<any value>') :-
 inline_compile('@yarnpkg/eslint-config').
 inline_compile('@yarnpkg/libui').
 
-gen_enforced_field(WorkspaceCwd, 'scripts.prepack', 'run build:compile "$(pwd)"') :-
+gen_enforced_field(WorkspaceCwd, 'scripts.prepack', 'run --global build:compile "$(pwd)"') :-
   workspace(WorkspaceCwd),
   % This package is built using Rollup, so we allow it to configure its build scripts itself
     \+ workspace_ident(WorkspaceCwd, '@yarnpkg/pnp'),

packages/acceptance-tests/package.json

@@ -13,7 +13,7 @@
     "pkg-tests-core": "workspace:^"
   },
   "scripts": {
-    "test:integration": "run test:unit --config \"$(pwd)/jest.config.js\""
+    "test:integration": "run --global test:unit --config \"$(pwd)/jest.config.js\""
   },
   "repository": {
     "type": "git",

packages/plugin-compat/package.json

@@ -29,8 +29,8 @@
   },
   "scripts": {
     "postpack": "rm -rf lib",
-    "prepack": "run build:compile \"$(pwd)\"",
-    "test:plugin-compat": "run test:unit --config \"$(pwd)/jest.config.js\"",
+    "prepack": "run --global build:compile \"$(pwd)\"",
+    "test:plugin-compat": "run --global test:unit --config \"$(pwd)/jest.config.js\"",
     "debug:patch": "node -r @yarnpkg/monorepo/scripts/setup-ts-execution extra/debugPatch \"$INIT_CWD\""
   },
   "publishConfig": {

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

@@ -56,6 +56,10 @@ export default class RunCommand extends BaseCommand {
     description: `Forwarded to the underlying Node process when executing a binary`,
   });
 
+  global = Option.Boolean(`--global`, false, {
+    description: `Check all workspaces for ones that declare the script uniquely`,
+  });
+
   // The v1 used to print the Yarn version header when using "yarn run", which
   // was messing with the output of things like `--version` & co. We don't do
   // this anymore, but many workflows use `yarn run --silent` to make sure that
@@ -123,13 +127,13 @@ export default class RunCommand extends BaseCommand {
 
     // When it fails, we try to check whether it's a global script (ie we look
     // into all the workspaces to find one that exports this script). We only do
-    // this if the script name contains a colon character (":"), and we skip
+    // this if the `--global` option is passed, and we skip
     // this logic if multiple workspaces share the same script name.
     //
     // We also disable this logic for packages coming from third-parties (ie
     // not workspaces). No particular reason except maybe security concerns.
 
-    if (!this.topLevel && !this.binariesOnly && workspace && this.scriptName.includes(`:`)) {
+    if (!this.topLevel && !this.binariesOnly && workspace && this.global) {
       const candidateWorkspaces = await Promise.all(project.workspaces.map(async workspace => {
         return workspace.manifest.scripts.has(this.scriptName) ? workspace : null;
       }));

packages/plugin-github/package.json

@@ -16,9 +16,9 @@
     "@yarnpkg/plugin-git": "workspace:^"
   },
   "scripts": {
-    "test": "run test:unit packages/plugin-github",
+    "test": "run --global test:unit packages/plugin-github",
     "postpack": "rm -rf lib",
-    "prepack": "run build:compile \"$(pwd)\""
+    "prepack": "run --global build:compile \"$(pwd)\""
   },
   "repository": {
     "type": "git",