refactor: introduce shouldprepare hook

Author: Dimitar Kerezov

lib/definitions/platform.d.ts

@@ -78,6 +78,14 @@ interface IPlatformService extends NodeJS.EventEmitter {
 	 */
 	shouldInstall(device: Mobile.IDevice, projectData: IProjectData, outputPath?: string): Promise<boolean>;
 
+	/**
+	 * Determines whether the project should undergo the prepare process.
+	 * @param {IPreparePlatformInfo} platformInfo Options to control the preparation.
+	 * @param {IProjectChangesInfo} @optional changesInfo Info about the current changes in the project.
+	 * @returns {Promise<boolean>} true indicates that the project should be prepared.
+	 */
+	shouldPrepare(platformInfo: IPreparePlatformInfo, changesInfo?: IProjectChangesInfo): Promise<boolean>
+
 	/**
 	 * Installs the application on specified device.
 	 * When finishes, saves .nsbuildinfo in application root folder to indicate the prepare that was used to build the app.

lib/services/platform-service.ts

@@ -187,13 +187,25 @@ export class PlatformService extends EventEmitter implements IPlatformService {
 		return _.filter(this.$platformsData.platformsNames, p => { return this.isPlatformPrepared(p, projectData); });
 	}
 
-	public async preparePlatform(platformInfo: IPreparePlatformInfo): Promise<boolean> {
+	@helpers.hook('shouldPrepare')
+	public async shouldPrepare(platformInfo: IPreparePlatformInfo, changesInfo?: IProjectChangesInfo): Promise<boolean> {
+		changesInfo = changesInfo || await this.getChangesInfo(platformInfo);
+		const requiresNativePrepare = (!platformInfo.nativePrepare || !platformInfo.nativePrepare.skipNativePrepare) && changesInfo.nativePlatformStatus === constants.NativePlatformStatus.requiresPrepare;
+
+		return changesInfo.hasChanges || requiresNativePrepare;
+	}
+
+	private async getChangesInfo(platformInfo: IPreparePlatformInfo): Promise<IProjectChangesInfo> {
 		const platformData = this.$platformsData.getPlatformData(platformInfo.platform, platformInfo.projectData);
 
-		const changesInfo = await this.initialPrepare(platformInfo.platform, platformData, platformInfo.appFilesUpdaterOptions, platformInfo.platformTemplate, platformInfo.projectData, platformInfo.config, platformInfo.nativePrepare, platformInfo);
-		const requiresNativePrepare = (!platformInfo.nativePrepare || !platformInfo.nativePrepare.skipNativePrepare) && changesInfo.nativePlatformStatus === constants.NativePlatformStatus.requiresPrepare;
+		return this.initialPrepare(platformInfo.platform, platformData, platformInfo.appFilesUpdaterOptions, platformInfo.platformTemplate, platformInfo.projectData, platformInfo.config, platformInfo.nativePrepare, platformInfo);
+	}
+
+	public async preparePlatform(platformInfo: IPreparePlatformInfo): Promise<boolean> {
+		const changesInfo = await this.getChangesInfo(platformInfo);
+		const shouldPrepare = await this.shouldPrepare(platformInfo, changesInfo);
 
-		if (changesInfo.hasChanges || requiresNativePrepare) {
+		if (shouldPrepare) {
 			// Always clear up the app directory in platforms if `--bundle` value has changed in between builds or is passed in general
 			// this is done as user has full control over what goes in platforms when `--bundle` is passed
 			// and we may end up with duplicate symbols which would fail the build

test/stubs.ts

@@ -613,6 +613,9 @@ export class CommandsService implements ICommandsService {
 }
 
 export class PlatformServiceStub extends EventEmitter implements IPlatformService {
+	public shouldPrepare(): Promise<boolean> {
+		return Promise.resolve(true);
+	}
 
 	public validateOptions(): Promise<boolean> {
 		return Promise.resolve(true);