fix(yarn): handle global for yarn 1.x only

unjs · Mar 3, 2024 · f5e4a39 · f5e4a39
1 parent 15aaee8
commit f5e4a39
Showing 1 changed file with 9 additions and 5 deletions.
diff --git a/src/api.ts b/src/api.ts
@@ -49,6 +49,11 @@ export async function addDependency(
     resolvedOptions.packageManager.name === "yarn"
       ? [
           ...getWorkspaceArgs(resolvedOptions),
+          // Global is not supported in berry: yarnpkg/berry#821
+          resolvedOptions.global &&
+          resolvedOptions.packageManager.majorVersion === "1"
+            ? "global"
+            : "",
           "add",
           resolvedOptions.dev ? "-D" : "",
           ...names,
@@ -108,11 +113,10 @@ export async function removeDependency(
   const args = (
     resolvedOptions.packageManager.name === "yarn"
       ? [
-          // prettier-ignore
-          resolvedOptions.global
-            ? (resolvedOptions.packageManager.majorVersion === "1"
-              ? "dlx"
-              : "global")
+          // Global is not supported in berry: yarnpkg/berry#821
+          resolvedOptions.global &&
+          resolvedOptions.packageManager.majorVersion === "1"
+            ? "global"
             : "",
           ...getWorkspaceArgs(resolvedOptions),
           "remove",