feat: yarn2+ support

Author: farfromrefug

lib/bootstrap.ts

@@ -218,6 +218,7 @@ injector.requireCommand("setup|*", "./commands/setup");
 injector.requirePublic("packageManager", "./package-manager");
 injector.requirePublic("npm", "./node-package-manager");
 injector.requirePublic("yarn", "./yarn-package-manager");
+injector.requirePublic("yarn2", "./yarn2-package-manager");
 injector.requirePublic("pnpm", "./pnpm-package-manager");
 injector.requireCommand(
 	"package-manager|*get",

lib/commands/preview.ts

@@ -51,6 +51,7 @@ export class PreviewCommand implements ICommand {
 					installCommand = "npm install --save-dev @nativescript/preview-cli";
 					break;
 				case PackageManagers.yarn:
+				case PackageManagers.yarn2:
 					installCommand = "yarn add -D @nativescript/preview-cli";
 					break;
 				case PackageManagers.pnpm:

lib/common/dispatchers.ts

@@ -124,6 +124,7 @@ export class CommandDispatcher implements ICommandDispatcher {
 				updateCommand = "npm i -g nativescript";
 				break;
 			case PackageManagers.yarn:
+			case PackageManagers.yarn2:
 				updateCommand = "yarn global add nativescript";
 				break;
 			case PackageManagers.pnpm:

lib/constants.ts

@@ -492,4 +492,5 @@ export enum PackageManagers {
 	npm = "npm",
 	pnpm = "pnpm",
 	yarn = "yarn",
+	yarn2 = "yarn2",
 }

lib/declarations.d.ts

@@ -663,6 +663,7 @@ interface IOptions
 	frameworkName: string;
 	frameworkVersion: string;
 	yarn: string;
+	yarn2: string;
 	pnpm: string;
 	ipa: string;
 	tsc: boolean;

lib/options.ts

@@ -136,6 +136,7 @@ export class Options {
 			ts: { type: OptionType.Boolean, hasSensitiveValue: false },
 			typescript: { type: OptionType.Boolean, hasSensitiveValue: false },
 			yarn: { type: OptionType.Boolean, hasSensitiveValue: false },
+			yarn2: { type: OptionType.Boolean, hasSensitiveValue: false },
 			pnpm: { type: OptionType.Boolean, hasSensitiveValue: false },
 			androidTypings: { type: OptionType.Boolean, hasSensitiveValue: false },
 			bundle: { type: OptionType.String, hasSensitiveValue: false },

lib/package-manager.ts

@@ -26,6 +26,7 @@ export class PackageManager implements IPackageManager {
 		private $npm: INodePackageManager,
 		private $options: IOptions,
 		private $yarn: INodePackageManager,
+		private $yarn2: INodePackageManager,
 		private $pnpm: INodePackageManager,
 		private $logger: ILogger,
 		private $userSettingsService: IUserSettingsService,
@@ -164,6 +165,9 @@ export class PackageManager implements IPackageManager {
 		if (pm === PackageManagers.yarn || this.$options.yarn) {
 			this._packageManagerName = PackageManagers.yarn;
 			return this.$yarn;
+		} if (pm === PackageManagers.yarn2 || this.$options.yarn2) {
+			this._packageManagerName = PackageManagers.yarn2;
+			return this.$yarn2;
 		} else if (pm === PackageManagers.pnpm || this.$options.pnpm) {
 			this._packageManagerName = PackageManagers.pnpm;
 			return this.$pnpm;

lib/yarn2-package-manager.ts

@@ -0,0 +1,150 @@
+import * as path from "path";
+import * as _ from "lodash";
+import { BasePackageManager } from "./base-package-manager";
+import { exported } from "./common/decorators";
+import {
+	INodePackageManagerInstallOptions,
+	INpmInstallResultInfo,
+	INpmsResult,
+} from "./declarations";
+import {
+	IChildProcess,
+	IErrors,
+	IFileSystem,
+	IHostInfo,
+	Server,
+	IDictionary,
+} from "./common/declarations";
+import { injector } from "./common/yok";
+
+export class Yarn2PackageManager extends BasePackageManager {
+	constructor(
+		$childProcess: IChildProcess,
+		private $errors: IErrors,
+		$fs: IFileSystem,
+		$hostInfo: IHostInfo,
+		private $httpClient: Server.IHttpClient,
+		private $logger: ILogger,
+		$pacoteService: IPacoteService
+	) {
+		super($childProcess, $fs, $hostInfo, $pacoteService, "yarn2");
+	}
+
+	@exported("yarn2")
+	public async install(
+		packageName: string,
+		pathToSave: string,
+		config: INodePackageManagerInstallOptions
+	): Promise<INpmInstallResultInfo> {
+		if (config.disableNpmInstall) {
+			return;
+		}
+		if (config.ignoreScripts) {
+			config["ignore-scripts"] = true;
+		}
+
+		const packageJsonPath = path.join(pathToSave, "package.json");
+		const jsonContentBefore = this.$fs.readJson(packageJsonPath);
+
+		const flags = this.getFlagsString(config, true);
+		let params = [];
+		const isInstallingAllDependencies = packageName === pathToSave;
+		if (!isInstallingAllDependencies) {
+			params.push("add", packageName);
+		}
+
+		params = params.concat(flags);
+		const cwd = pathToSave;
+
+		try {
+			const result = await this.processPackageManagerInstall(
+				packageName,
+				params,
+				{ cwd, isInstallingAllDependencies }
+			);
+			return result;
+		} catch (e) {
+			this.$fs.writeJson(packageJsonPath, jsonContentBefore);
+			throw e;
+		}
+	}
+
+	@exported("yarn2")
+	public uninstall(
+		packageName: string,
+		config?: IDictionary<string | boolean>,
+		cwd?: string
+	): Promise<string> {
+		const flags = this.getFlagsString(config, false);
+		return this.$childProcess.exec(`yarn remove ${packageName} ${flags}`, {
+			cwd,
+		});
+	}
+
+	@exported("yarn2")
+	public async view(packageName: string, config: Object): Promise<any> {
+		const wrappedConfig = _.extend({}, config, { json: true });
+
+		const flags = this.getFlagsString(wrappedConfig, false);
+		let viewResult: any;
+		try {
+			viewResult = await this.$childProcess.exec(
+				`yarn info ${packageName} ${flags}`
+			);
+		} catch (e) {
+			this.$errors.fail(e.message);
+		}
+
+		try {
+			const result = JSON.parse(viewResult);
+			return result.data;
+		} catch (err) {
+			return null;
+		}
+	}
+
+	@exported("yarn2")
+	public search(
+		filter: string[],
+		config: IDictionary<string | boolean>
+	): Promise<string> {
+		this.$errors.fail(
+			"Method not implemented. Yarn does not support searching for packages in the registry."
+		);
+		return null;
+	}
+
+	public async searchNpms(keyword: string): Promise<INpmsResult> {
+		const httpRequestResult = await this.$httpClient.httpRequest(
+			`https://api.npms.io/v2/search?q=keywords:${keyword}`
+		);
+		const result: INpmsResult = JSON.parse(httpRequestResult.body);
+		return result;
+	}
+
+	@exported("yarn2")
+	public async getRegistryPackageData(packageName: string): Promise<any> {
+		const registry = await this.$childProcess.exec(`yarn config get npmRegistryServer`);
+		const url = `${registry.trim()}/${packageName}`;
+		this.$logger.trace(
+			`Trying to get data from yarn registry for package ${packageName}, url is: ${url}`
+		);
+		const responseData = (await this.$httpClient.httpRequest(url)).body;
+		this.$logger.trace(
+			`Successfully received data from yarn registry for package ${packageName}. Response data is: ${responseData}`
+		);
+		const jsonData = JSON.parse(responseData);
+		this.$logger.trace(
+			`Successfully parsed data from yarn registry for package ${packageName}.`
+		);
+		return jsonData;
+	}
+
+	@exported("yarn2")
+	public async getCachePath(): Promise<string> {
+		const result = await this.$childProcess.exec(`yarn config get cacheFolder`);
+		return result.toString().trim();
+	}
+}
+
+injector.register("yarn2", Yarn2PackageManager);

test/controllers/add-platform-controller.ts

@@ -8,6 +8,7 @@ import { AddPlaformErrors } from "../../lib/constants";
 import { PackageManager } from "../../lib/package-manager";
 import { NodePackageManager } from "../../lib/node-package-manager";
 import { YarnPackageManager } from "../../lib/yarn-package-manager";
+import { Yarn2PackageManager } from "../../lib/yarn2-package-manager";
 import { PnpmPackageManager } from "../../lib/pnpm-package-manager";
 import { MobileHelper } from "../../lib/common/mobile/mobile-helper";
 
@@ -29,6 +30,7 @@ function createInjector(data?: { latestFrameworkVersion: string }) {
 	injector.register("packageManager", PackageManager);
 	injector.register("npm", NodePackageManager);
 	injector.register("yarn", YarnPackageManager);
+	injector.register("yarn2", Yarn2PackageManager);
 	injector.register("pnpm", PnpmPackageManager);
 	injector.register("userSettingsService", {
 		getSettingValue: async (settingName: string): Promise<void> => undefined,

test/package-installation-manager.ts

@@ -6,6 +6,7 @@ import * as HostInfoLib from "../lib/common/host-info";
 import * as LoggerLib from "../lib/common/logger/logger";
 import * as NpmLib from "../lib/node-package-manager";
 import * as YarnLib from "../lib/yarn-package-manager";
+import * as Yarn2Lib from "../lib/yarn2-package-manager";
 import * as PnpmLib from "../lib/pnpm-package-manager";
 import * as PackageManagerLib from "../lib/package-manager";
 import * as PackageInstallationManagerLib from "../lib/package-installation-manager";
@@ -46,6 +47,7 @@ function createTestInjector(): IInjector {
 	});
 	testInjector.register("npm", NpmLib.NodePackageManager);
 	testInjector.register("yarn", YarnLib.YarnPackageManager);
+	testInjector.register("yarn2", Yarn2Lib.Yarn2PackageManager);
 	testInjector.register("pnpm", PnpmLib.PnpmPackageManager);
 	testInjector.register("packageManager", PackageManagerLib.PackageManager);
 	testInjector.register("projectConfigService", ProjectConfigServiceStub);

test/plugins-service.ts

@@ -4,6 +4,7 @@ import { PackageManager } from "../lib/package-manager";
 import { PackageInstallationManager } from "../lib/package-installation-manager";
 import { NodePackageManager } from "../lib/node-package-manager";
 import { YarnPackageManager } from "../lib/yarn-package-manager";
+import { Yarn2PackageManager } from "../lib/yarn2-package-manager";
 import { PnpmPackageManager } from "../lib/pnpm-package-manager";
 import { ProjectData } from "../lib/project-data";
 import { ChildProcess } from "../lib/common/child-process";
@@ -74,6 +75,7 @@ function createTestInjector() {
 	);
 	testInjector.register("npm", NodePackageManager);
 	testInjector.register("yarn", YarnPackageManager);
+	testInjector.register("yarn2", Yarn2PackageManager);
 	testInjector.register("pnpm", PnpmPackageManager);
 	testInjector.register("fs", FileSystem);
 	// const fileSystemStub = new stubs.FileSystemStub();

test/services/extensibility-service.ts

@@ -5,6 +5,7 @@ import { assert } from "chai";
 import { NodePackageManager } from "../../lib/node-package-manager";
 import { PackageManager } from "../../lib/package-manager";
 import { YarnPackageManager } from "../../lib/yarn-package-manager";
+import { Yarn2PackageManager } from "../../lib/yarn2-package-manager";
 import { PnpmPackageManager } from "../../lib/pnpm-package-manager";
 import * as constants from "../../lib/constants";
 import { ChildProcess } from "../../lib/common/child-process";
@@ -75,6 +76,7 @@ describe("extensibilityService", () => {
 		});
 		testInjector.register("npm", NodePackageManager);
 		testInjector.register("yarn", YarnPackageManager);
+		testInjector.register("yarn2", Yarn2PackageManager);
 		testInjector.register("pnpm", PnpmPackageManager);
 		testInjector.register("settingsService", SettingsService);
 		testInjector.register("requireService", {