Tooling: Fix jetpack cli link and unlink (#42130)

* Fix cli link/unlink

* Add comments

Author: tbradsha

tools/cli/commands/cli.js

@@ -47,7 +47,9 @@ function cliLink( options ) {
 						[
 							{
 								title: chalkJetpackGreen( `Enabling global access to the CLI` ),
-								task: () => command( 'pnpm link', options.v, path.resolve( 'tools/cli' ) ),
+								// Theoretically we shouldn't need the --global as of v10.0, but there was a bug until 10.5.
+								// It shouldn't hurt to leave this.
+								task: () => command( 'pnpm link --global', options.v, path.resolve( 'tools/cli' ) ),
 							},
 						],
 						opts
@@ -87,7 +89,9 @@ function cliUnlink( options ) {
 						[
 							{
 								title: chalkJetpackGreen( `Removing global access to the CLI` ),
-								task: () => command( 'pnpm unlink', options.v, path.resolve( 'tools/cli' ) ),
+								// In theory unlink should work, but it doesn't (and there are dozens of issues related
+								// to link/unlink in the pnpm repo. This works well.
+								task: () => command( 'pnpm uninstall --global jetpack-cli', options.v ),
 							},
 						],
 						opts