feat: enable hmr by default for new projects

Author: Fatme

lib/common/services/commands-service.ts

@@ -27,7 +27,10 @@ export class CommandsService implements ICommandsService {
 		private $staticConfig: Config.IStaticConfig,
 		private $helpService: IHelpService,
 		private $extensibilityService: IExtensibilityService,
-		private $optionsTracker: IOptionsTracker) {
+		private $optionsTracker: IOptionsTracker,
+		private $projectDataService: IProjectDataService) {
+			const projectData = this.$projectDataService.getProjectData();
+			this.$options.setupOptions(projectData);
 	}
 
 	public allCommands(opts: { includeDevCommands: boolean }): string[] {

lib/declarations.d.ts

@@ -568,6 +568,7 @@ interface IOptions extends IRelease, IDeviceIdentifier, IJustLaunch, IAvd, IAvai
 	link: boolean;
 	analyticsLogFile: string;
 	performance: Object;
+	setupOptions(projectData: IProjectData): void;
 }
 
 interface IEnvOptions {

lib/definitions/project.d.ts

@@ -74,6 +74,7 @@ interface INsConfig {
 	appPath?: string;
 	appResourcesPath?: string;
 	shared?: boolean;
+	isHmrEnabledByDefault?: boolean;
 }
 
 interface IProjectData extends ICreateProjectData {
@@ -99,6 +100,11 @@ interface IProjectData extends ICreateProjectData {
 	 */
 	isShared: boolean;
 
+	/**
+	 * Defines if the project has hmr enabled by default
+	 */
+	isHmrEnabledByDefault: boolean;
+
 	/**
 	 * Initializes project data with the given project directory. If none supplied defaults to --path option or cwd.
 	 * @param {string} projectDir Project root directory.

lib/options.ts

@@ -21,6 +21,26 @@ export class Options {
 
 	public options: IDictionary<IDashedOption>;
 
+	public setupOptions(projectData: IProjectData): void {
+		if (this.argv.release && this.argv.hmr) {
+			this.$errors.failWithoutHelp("The options --release and --hmr cannot be used simultaneously.");
+		}
+
+		// HACK: temporary solution for 5.3.0 release (until the webpack only feature)
+		const parsed = require("yargs-parser")(process.argv.slice(2), { 'boolean-negation': false });
+		// --no-hmr -> hmr: false or --hmr false -> hmr: 'false'
+		const noHmr = parsed && (parsed.hmr === false || parsed.hmr === 'false');
+		if (noHmr) {
+			this.argv.hmr = false;
+			return;
+		}
+
+		if (projectData.isHmrEnabledByDefault) {
+			this.argv.bundle = this.argv.bundle !== undefined ? this.argv.bundle : "webpack";
+			this.argv.hmr = !this.argv.release;
+		}
+	}
+
 	constructor(private $errors: IErrors,
 		private $staticConfig: Config.IStaticConfig,
 		private $settingsService: ISettingsService) {

lib/project-data.ts

@@ -61,6 +61,7 @@ export class ProjectData implements IProjectData {
 	public buildXcconfigPath: string;
 	public podfilePath: string;
 	public isShared: boolean;
+	public isHmrEnabledByDefault: boolean;
 
 	constructor(private $fs: IFileSystem,
 		private $errors: IErrors,
@@ -135,6 +136,7 @@ export class ProjectData implements IProjectData {
 			this.buildXcconfigPath = path.join(this.appResourcesDirectoryPath, this.$devicePlatformsConstants.iOS, constants.BUILD_XCCONFIG_FILE_NAME);
 			this.podfilePath = path.join(this.appResourcesDirectoryPath, this.$devicePlatformsConstants.iOS, constants.PODFILE_NAME);
 			this.isShared = !!(this.nsConfig && this.nsConfig.shared);
+			this.isHmrEnabledByDefault = !!(this.nsConfig && this.nsConfig.isHmrEnabledByDefault);
 			return;
 		}