feat: initial implementation of pods override

Author: KristianDD

lib/controllers/prepare-controller.ts

@@ -3,7 +3,7 @@ import { hook } from "../common/helpers";
 import { performanceLog, cache } from "../common/decorators";
 import { EventEmitter } from "events";
 import * as path from "path";
-import { PREPARE_READY_EVENT_NAME, WEBPACK_COMPILATION_COMPLETE, PACKAGE_JSON_FILE_NAME, PLATFORMS_DIR_NAME, TrackActionNames, AnalyticsEventLabelDelimiter } from "../constants";
+import { PREPARE_READY_EVENT_NAME, WEBPACK_COMPILATION_COMPLETE, PACKAGE_JSON_FILE_NAME, PLATFORMS_DIR_NAME, TrackActionNames, AnalyticsEventLabelDelimiter, CONFIG_NS_FILE_NAME } from "../constants";
 interface IPlatformWatcherData {
 	hasWebpackCompilerProcess: boolean;
 	nativeFilesWatcher: choki.FSWatcher;
@@ -189,6 +189,7 @@ export class PrepareController extends EventEmitter {
 
 		const patterns = [
 			path.join(projectData.projectDir, PACKAGE_JSON_FILE_NAME),
+			path.join(projectData.projectDir, CONFIG_NS_FILE_NAME),
 			path.join(projectData.getAppDirectoryPath(), PACKAGE_JSON_FILE_NAME),
 			path.join(projectData.getAppResourcesRelativeDirectoryPath(), platformData.normalizedPlatformName),
 		]

lib/definitions/platform.d.ts

@@ -46,7 +46,13 @@ interface IPlatformsDataService {
 }
 
 interface INodeModulesBuilder {
-	prepareNodeModules(platformData: IPlatformData, projectData: IProjectData): Promise<void>;
+	prepareNodeModules(prepareNodeModulesData: IPrepareNodeModulesData): Promise<void>;
+}
+
+interface IPrepareNodeModulesData {
+	platformData: IPlatformData;
+	projectData: IProjectData;
+	forcePluginNativePrepare: boolean;
 }
 
 interface INodeModulesDependenciesBuilder {

lib/definitions/plugins.d.ts

@@ -12,11 +12,18 @@ interface IPluginsService {
 	 * @returns {IPackageJsonDepedenciesResult}
 	 */
 	getDependenciesFromPackageJson(projectDir: string): IPackageJsonDepedenciesResult;
-	preparePluginNativeCode(pluginData: IPluginData, platform: string, projectData: IProjectData): Promise<void>;
+	preparePluginNativeCode(preparePluginNativeCodeData: IPreparePluginNativeCodeData): Promise<void>;
 	convertToPluginData(cacheData: any, projectDir: string): IPluginData;
 	isNativeScriptPlugin(pluginPackageJsonPath: string): boolean;
 }
 
+interface IPreparePluginNativeCodeData {
+	pluginData: IPluginData;
+	platform: string;
+	projectData: IProjectData;
+	forcePluginNativePrepare: boolean;
+}
+
 interface IPackageJsonDepedenciesResult {
 	dependencies: IBasePluginData[],
 	devDependencies?: IBasePluginData[]

lib/definitions/project-changes.d.ts

@@ -17,6 +17,7 @@ interface IProjectChangesInfo extends IAddedNativePlatform {
 	configChanged: boolean;
 	nativeChanged: boolean;
 	signingChanged: boolean;
+	nsConfigChanged: boolean;
 
 	readonly hasChanges: boolean;
 	readonly changesRequireBuild: boolean;

lib/definitions/project.d.ts

@@ -75,6 +75,7 @@ interface INsConfig {
 	appResourcesPath?: string;
 	shared?: boolean;
 	previewAppSchema?: string;
+	overridePods?: string
 }
 
 interface IProjectData extends ICreateProjectData {
@@ -406,10 +407,10 @@ interface ICocoaPodsService {
 	 * @param {string} moduleName The module which the Podfile is from.
 	 * @param {string} podfilePath The path to the podfile.
 	 * @param {IProjectData} projectData Information about the project.
-	 * @param {string} nativeProjectPath Path to the native Xcode project.
+	 * @param {IPlatformData} platformData Information about the platform.
 	 * @returns {Promise<void>}
 	 */
-	applyPodfileToProject(moduleName: string, podfilePath: string, projectData: IProjectData, nativeProjectPath: string): Promise<void>;
+	applyPodfileToProject(moduleName: string, podfilePath: string, projectData: IProjectData, platformData: IPlatformData): Promise<void>;
 
 	/**
 	 * Gives the path to the plugin's Podfile.

lib/services/cocoapods-service.ts

@@ -1,20 +1,22 @@
 import { EOL } from "os";
 import * as path from "path";
 import { PluginNativeDirNames, PODFILE_NAME, NS_BASE_PODFILE } from "../constants";
-import { regExpEscape } from "../common/helpers";
+import { regExpEscape, getHash } from "../common/helpers";
 
 export class CocoaPodsService implements ICocoaPodsService {
 	private static PODFILE_POST_INSTALL_SECTION_NAME = "post_install";
 	private static INSTALLER_BLOCK_PARAMETER_NAME = "installer";
-
+	private getCocoaPodsFromPodfile: Function;
 	constructor(
 		private $cocoaPodsPlatformManager: ICocoaPodsPlatformManager,
 		private $fs: IFileSystem,
 		private $childProcess: IChildProcess,
 		private $errors: IErrors,
 		private $logger: ILogger,
 		private $config: IConfiguration,
-		private $xcconfigService: IXcconfigService) { }
+		private $xcconfigService: IXcconfigService) {
+			this.getCocoaPodsFromPodfile = _.memoize(this._getCocoaPodsFromPodfile, getHash);
+		}
 
 	public getPodfileHeader(targetName: string): string {
 		return `use_frameworks!${EOL}${EOL}target "${targetName}" do${EOL}`;
@@ -35,7 +37,11 @@ export class CocoaPodsService implements ICocoaPodsService {
 		const podInstallResult = await this.$childProcess.spawnFromEvent(podTool, ["install"], "close", { cwd: projectRoot, stdio: ['pipe', process.stdout, process.stdout] }, { throwError: false });
 
 		if (podInstallResult.exitCode !== 0) {
-			this.$errors.fail(`'${podTool} install' command failed.${podInstallResult.stderr ? " Error is: " + podInstallResult.stderr : ""}`);
+			// https://github.com/CocoaPods/CocoaPods/blob/92aaf0f1120d32f3487960b485fb69fcaf61486c/lib/cocoapods/resolver.rb#L498
+			// TODO add article
+			const versionResolutionHint = podInstallResult.exitCode === 31 ? `For more information on resolving CocoaPod issues in NativeScript read.` : "";
+			this.$errors.fail(`'${podTool} install' command failed.${podInstallResult.stderr ? " Error is: " + podInstallResult.stderr : ""}
+${versionResolutionHint}`);
 		}
 
 		return podInstallResult;
@@ -59,17 +65,18 @@ export class CocoaPodsService implements ICocoaPodsService {
 		const mainPodfilePath = path.join(projectData.appResourcesDirectoryPath, normalizedPlatformName, PODFILE_NAME);
 		const projectPodfilePath = this.getProjectPodfilePath(projectRoot);
 		if (this.$fs.exists(projectPodfilePath) || this.$fs.exists(mainPodfilePath)) {
-			await this.applyPodfileToProject(NS_BASE_PODFILE, mainPodfilePath, projectData, projectRoot);
+			await this.applyPodfileToProject(NS_BASE_PODFILE, mainPodfilePath, projectData, platformData);
 		}
 	}
 
-	public async applyPodfileToProject(moduleName: string, podfilePath: string, projectData: IProjectData, nativeProjectPath: string): Promise<void> {
+	public async applyPodfileToProject(moduleName: string, podfilePath: string, projectData: IProjectData, platformData: IPlatformData): Promise<void> {
+		const nativeProjectPath = platformData.projectRoot;
 		if (!this.$fs.exists(podfilePath)) {
 			this.removePodfileFromProject(moduleName, podfilePath, projectData, nativeProjectPath);
 			return;
 		}
 
-		const { podfileContent, replacedFunctions, podfilePlatformData } = this.buildPodfileContent(podfilePath, moduleName);
+		const { podfileContent, replacedFunctions, podfilePlatformData } = this.buildPodfileContent(podfilePath, moduleName, projectData, platformData);
 		const pathToProjectPodfile = this.getProjectPodfilePath(nativeProjectPath);
 		const projectPodfileContent = this.$fs.exists(pathToProjectPodfile) ? this.$fs.readText(pathToProjectPodfile).trim() : "";
 
@@ -86,6 +93,10 @@ export class CocoaPodsService implements ICocoaPodsService {
 				finalPodfileContent = this.$cocoaPodsPlatformManager.addPlatformSection(projectData, podfilePlatformData, finalPodfileContent);
 			}
 
+			if (this.isMainPodFile(podfilePath, projectData, platformData) && projectData.nsConfig.overridePods) {
+				finalPodfileContent = this.overridePodsFromFile(finalPodfileContent, projectData, platformData);
+			}
+
 			finalPodfileContent = `${finalPodfileContent.trim()}${EOL}${EOL}${podfileContent.trim()}${EOL}`;
 			this.saveProjectPodfile(projectData, finalPodfileContent, nativeProjectPath);
 		}
@@ -222,10 +233,17 @@ export class CocoaPodsService implements ICocoaPodsService {
 		return `${CocoaPodsService.PODFILE_POST_INSTALL_SECTION_NAME} do |${CocoaPodsService.INSTALLER_BLOCK_PARAMETER_NAME}|${EOL}`;
 	}
 
-	private buildPodfileContent(pluginPodFilePath: string, pluginName: string): { podfileContent: string, replacedFunctions: IRubyFunction[], podfilePlatformData: IPodfilePlatformData } {
+	private buildPodfileContent(pluginPodFilePath: string, pluginName: string, projectData: IProjectData, platformData: IPlatformData): { podfileContent: string, replacedFunctions: IRubyFunction[], podfilePlatformData: IPodfilePlatformData } {
+		const mainPodfilePath = this.getMainPodFilePath(projectData, platformData);
 		const pluginPodfileContent = this.$fs.readText(pluginPodFilePath);
 		const data = this.replaceHookContent(CocoaPodsService.PODFILE_POST_INSTALL_SECTION_NAME, pluginPodfileContent, pluginName);
-		const { replacedContent, podfilePlatformData } = this.$cocoaPodsPlatformManager.replacePlatformRow(data.replacedContent, pluginPodFilePath);
+		const cocoapodsData = this.$cocoaPodsPlatformManager.replacePlatformRow(data.replacedContent, pluginPodFilePath);
+		const podfilePlatformData  = cocoapodsData.podfilePlatformData;
+		let replacedContent = cocoapodsData.replacedContent;
+
+		if (projectData.nsConfig.overridePods && mainPodfilePath !== pluginPodFilePath) {
+			replacedContent = this.overridePodsFromFile(replacedContent, projectData, platformData);
+		}
 
 		return {
 			podfileContent: `${this.getPluginPodfileHeader(pluginPodFilePath)}${EOL}${replacedContent}${EOL}${this.getPluginPodfileEnd()}`,
@@ -234,6 +252,43 @@ export class CocoaPodsService implements ICocoaPodsService {
 		};
 	}
 
+	private getMainPodFilePath(projectData: IProjectData, platformData: IPlatformData): string {
+		return path.join(projectData.appResourcesDirectoryPath, platformData.normalizedPlatformName, PODFILE_NAME);
+	}
+
+	private isMainPodFile(podFilePath: string, projectData: IProjectData, platformData: IPlatformData): boolean {
+		const mainPodfilePath = this.getMainPodFilePath(projectData, platformData);
+
+		return podFilePath === mainPodfilePath;
+	}
+
+	private overridePodsFromFile(podfileContent: string, projectData: IProjectData, platformData: IPlatformData): string {
+		const mainPodfilePath = this.getMainPodFilePath(projectData, platformData);
+		const mainPodfileContent = this.$fs.readText(mainPodfilePath);
+		const pods = this.getCocoaPodsFromPodfile(mainPodfileContent);
+		_.forEach(pods, pod => {
+			podfileContent = podfileContent.replace(new RegExp(`^[ ]*pod\\s*["']${pod}['"].*$`, "gm"), '#$&');
+		});
+
+		return podfileContent;
+	}
+
+	private _getCocoaPodsFromPodfile(podfileContent: string): Array<string> {
+		const pods = [];
+		const podsRegex = /^\s*pod\s*["'](.*?)['"].*$/gm;
+
+		let match = podsRegex.exec(podfileContent);
+		while (match != null) {
+			const podName: string = match[1];
+			if (podName) {
+				pods.push(podName);
+			}
+
+			match = podsRegex.exec(podfileContent);
+		}
+
+		return pods;
+	}
 }
 
 $injector.register("cocoapodsService", CocoaPodsService);

lib/services/ios-project-service.ts

@@ -495,9 +495,8 @@ export class IOSProjectService extends projectServiceBaseLib.PlatformProjectServ
 		await this.prepareResources(pluginPlatformsFolderPath, pluginData, projectData);
 		await this.prepareFrameworks(pluginPlatformsFolderPath, pluginData, projectData);
 		await this.prepareStaticLibs(pluginPlatformsFolderPath, pluginData, projectData);
-
-		const projectRoot = this.getPlatformData(projectData).projectRoot;
-		await this.$cocoapodsService.applyPodfileToProject(pluginData.name, this.$cocoapodsService.getPluginPodfilePath(pluginData), projectData, projectRoot);
+		const platformData = this.getPlatformData(projectData);
+		await this.$cocoapodsService.applyPodfileToProject(pluginData.name, this.$cocoapodsService.getPluginPodfilePath(pluginData), projectData, platformData);
 	}
 
 	public async removePluginNativeCode(pluginData: IPluginData, projectData: IProjectData): Promise<void> {

lib/services/platform/prepare-native-platform-service.ts

@@ -24,7 +24,7 @@ export class PrepareNativePlatformService implements IPrepareNativePlatformServi
 		const hasConfigChange = !changesInfo || changesInfo.configChanged;
 		const hasChangesRequirePrepare = !changesInfo || changesInfo.changesRequirePrepare;
 
-		const hasChanges = hasNativeModulesChange || hasConfigChange || hasChangesRequirePrepare;
+		const hasChanges = hasNativeModulesChange || hasConfigChange || hasChangesRequirePrepare || changesInfo.nsConfigChanged;
 
 		if (changesInfo.hasChanges) {
 			await this.cleanProject(platformData, { release });
@@ -36,8 +36,8 @@ export class PrepareNativePlatformService implements IPrepareNativePlatformServi
 			await platformData.platformProjectService.prepareProject(projectData, prepareData);
 		}
 
-		if (hasNativeModulesChange) {
-			await this.$nodeModulesBuilder.prepareNodeModules(platformData, projectData);
+		if (hasNativeModulesChange || changesInfo.nsConfigChanged) {
+			await this.$nodeModulesBuilder.prepareNodeModules({platformData, projectData, forcePluginNativePrepare: changesInfo.nsConfigChanged});
 		}
 
 		if (hasNativeModulesChange || hasConfigChange) {

lib/services/plugins-service.ts

@@ -114,7 +114,7 @@ export class PluginsService implements IPluginsService {
 		}
 	}
 
-	public async preparePluginNativeCode(pluginData: IPluginData, platform: string, projectData: IProjectData): Promise<void> {
+	public async preparePluginNativeCode({pluginData, platform, projectData, forcePluginNativePrepare}: IPreparePluginNativeCodeData): Promise<void> {
 		const platformData = this.$platformsDataService.getPlatformData(platform, projectData);
 		pluginData.pluginPlatformsFolderPath = (_platform: string) => path.join(pluginData.fullPath, "platforms", _platform.toLowerCase());
 
@@ -126,7 +126,7 @@ export class PluginsService implements IPluginsService {
 			const oldPluginNativeHashes = allPluginsNativeHashes[pluginData.name];
 			const currentPluginNativeHashes = await this.getPluginNativeHashes(pluginPlatformsFolderPath);
 
-			if (!oldPluginNativeHashes || this.$filesHashService.hasChangesInShasums(oldPluginNativeHashes, currentPluginNativeHashes)) {
+			if (forcePluginNativePrepare || !oldPluginNativeHashes || this.$filesHashService.hasChangesInShasums(oldPluginNativeHashes, currentPluginNativeHashes)) {
 				await platformData.platformProjectService.preparePluginNativeCode(pluginData, projectData);
 				this.setPluginNativeHashes({
 					pathToPluginsBuildFile,

lib/services/project-changes-service.ts

@@ -1,5 +1,5 @@
 import * as path from "path";
-import { NativePlatformStatus, PACKAGE_JSON_FILE_NAME, APP_GRADLE_FILE_NAME, BUILD_XCCONFIG_FILE_NAME, PLATFORMS_DIR_NAME } from "../constants";
+import { NativePlatformStatus, PACKAGE_JSON_FILE_NAME, APP_GRADLE_FILE_NAME, BUILD_XCCONFIG_FILE_NAME, PLATFORMS_DIR_NAME, CONFIG_NS_FILE_NAME } from "../constants";
 import { getHash, hook } from "../common/helpers";
 
 const prepareInfoFileName = ".nsprepareinfo";
@@ -8,6 +8,7 @@ class ProjectChangesInfo implements IProjectChangesInfo {
 
 	public appResourcesChanged: boolean;
 	public configChanged: boolean;
+	public nsConfigChanged: boolean;
 	public nativeChanged: boolean;
 	public signingChanged: boolean;
 	public nativePlatformStatus: NativePlatformStatus;
@@ -70,6 +71,10 @@ export class ProjectChangesService implements IProjectChangesService {
 				this._changesInfo.nativeChanged = this.isProjectFileChanged(projectData.projectDir, platformData);
 			}
 
+			// If this causes too much rebuilds of the plugins or uncecessary builds for Android, move overrideCocoapods to prepareInfo.
+			this._changesInfo.nsConfigChanged = this.filesChanged([path.join(projectData.projectDir, CONFIG_NS_FILE_NAME)]);
+			this._changesInfo.nativeChanged = this._changesInfo.nativeChanged || this._changesInfo.nsConfigChanged;
+
 			this.$logger.trace(`Set nativeChanged to ${this._changesInfo.nativeChanged}.`);
 
 			if (platformData.platformNameLowerCase === this.$devicePlatformsConstants.iOS.toLowerCase()) {
@@ -183,6 +188,7 @@ export class ProjectChangesService implements IProjectChangesService {
 		this._changesInfo.appResourcesChanged = true;
 		this._changesInfo.configChanged = true;
 		this._changesInfo.nativeChanged = true;
+		this._changesInfo.nsConfigChanged = true;
 		return true;
 	}
 

lib/tools/node-modules/node-modules-builder.ts

@@ -5,7 +5,7 @@ export class NodeModulesBuilder implements INodeModulesBuilder {
 		private $pluginsService: IPluginsService
 	) { }
 
-	public async prepareNodeModules(platformData: IPlatformData, projectData: IProjectData): Promise<void> {
+	public async prepareNodeModules({platformData , projectData, forcePluginNativePrepare}: IPrepareNodeModulesData): Promise<void> {
 		const dependencies = this.$nodeModulesDependenciesBuilder.getProductionDependencies(projectData.projectDir);
 		if (_.isEmpty(dependencies)) {
 			return;
@@ -19,7 +19,7 @@ export class NodeModulesBuilder implements INodeModulesBuilder {
 			if (isPlugin) {
 				this.$logger.debug(`Successfully prepared plugin ${dependency.name} for ${platformData.normalizedPlatformName.toLowerCase()}.`);
 				const pluginData = this.$pluginsService.convertToPluginData(dependency, projectData.projectDir);
-				await this.$pluginsService.preparePluginNativeCode(pluginData, platformData.normalizedPlatformName.toLowerCase(), projectData);
+				await this.$pluginsService.preparePluginNativeCode({pluginData, platform: platformData.normalizedPlatformName.toLowerCase(), projectData, forcePluginNativePrepare});
 			}
 		}
 	}