nativescript 9.0.4-dev.1 → 9.0.4-dev.2

package/lib/constants.js

@@ -149,6 +149,7 @@ exports.RESERVED_TEMPLATE_NAMES = {
     solidjs: "@nativescript/template-blank-solid",
     solidts: "@nativescript/template-blank-solid-ts",
     svelte: "@nativescript/template-blank-svelte",
+    // vision templates
     vision: "@nativescript/template-hello-world-ts-vision",
     "vision-vue": "@nativescript/template-blank-vue-vision",
     "vision-ng": "@nativescript/template-hello-world-ng-vision",
@@ -224,6 +225,9 @@ DebugCommandErrors.UNABLE_TO_USE_FOR_DEVICE_AND_EMULATOR = "The options --for-de
 DebugCommandErrors.NO_DEVICES_EMULATORS_FOUND_FOR_OPTIONS = "Unable to find device or emulator for specified options.";
 DebugCommandErrors.UNSUPPORTED_DEVICE_OS_FOR_DEBUGGING = "Unsupported device OS for debugging";
 exports.AnalyticsEventLabelDelimiter = "__";
+/**
+ * Used in ProjectDataService to concatenate the names of the properties inside nativescript key of package.json.
+ */
 exports.NATIVESCRIPT_PROPS_INTERNAL_DELIMITER = "**|__**";
 exports.CLI_RESOURCES_DIR_NAME = "resources";
 class AssetConstants {
@@ -240,6 +244,7 @@ AssetConstants.assets = "assets";
 AssetConstants.sizeDelimiter = "x";
 AssetConstants.defaultScale = 1;
 AssetConstants.defaultOverlayImageScale = 0.8;
+// https://en.wikipedia.org/wiki/Darwin_(operating_system)#Release_history
 class MacOSVersions {
 }
 exports.MacOSVersions = MacOSVersions;
@@ -249,6 +254,14 @@ MacOSVersions.Mojave = "10.14";
 MacOSVersions.Catalina = "10.15";
 exports.MacOSDeprecationStringFormat = "NativeScript does not support macOS %s and some functionality may not work. Please, upgrade to the latest macOS version.";
 exports.XcodeDeprecationStringFormat = "The current Xcode version %s will not be supported in the next release of NativeScript. Consider updating your Xcode to latest official version.";
+// export class TemplateVersions {
+// 	public static v1 = "v1";
+// 	public static v2 = "v2";
+// }
+// export class ProjectTemplateErrors {
+// 	public static InvalidTemplateVersionStringFormat =
+// 		"The template '%s' has a NativeScript version '%s' that is not supported. Unable to create project from it.";
+// }
 class Hooks {
 }
 exports.Hooks = Hooks;
@@ -289,6 +302,7 @@ AndroidAppBundleMessages.ANDROID_APP_BUNDLE_DOCS_MESSAGE = "What is Android App
 AndroidAppBundleMessages.ANDROID_APP_BUNDLE_PUBLISH_DOCS_MESSAGE = "How to use Android App Bundle for publishing: https://docs.nativescript.org/tooling/publishing/publishing-android-apps#android-app-bundle";
 exports.RunOnDeviceEvents = {
     runOnDeviceStopped: "runOnDeviceStopped",
+    // In case we name it error, EventEmitter expects instance of Error to be raised and will also raise uncaught exception in case there's no handler
     runOnDeviceError: "runOnDeviceError",
     runOnDeviceExecuted: "runOnDeviceExecuted",
     runOnDeviceStarted: "runOnDeviceStarted",
@@ -325,14 +339,52 @@ var DeviceConnectionType;
 })(DeviceConnectionType || (exports.DeviceConnectionType = DeviceConnectionType = {}));
 var LoggerLevel;
 (function (LoggerLevel) {
+    /**
+     * Show all log messages.
+     * Log levels are used to assign importance to log messages, with the integer value being used to sort them.
+     * If you do not specify anything in your configuration, the default values are used (ALL < TRACE < DEBUG < INFO < WARN < ERROR < FATAL < MARK < OFF)
+     */
     LoggerLevel["ALL"] = "ALL";
+    /**
+     * Log levels are used to assign importance to log messages, with the integer value being used to sort them.
+     * If you do not specify anything in your configuration, the default values are used (ALL < TRACE < DEBUG < INFO < WARN < ERROR < FATAL < MARK < OFF)
+     */
     LoggerLevel["TRACE"] = "TRACE";
+    /**
+     * Log levels are used to assign importance to log messages, with the integer value being used to sort them.
+     * If you do not specify anything in your configuration, the default values are used (ALL < TRACE < DEBUG < INFO < WARN < ERROR < FATAL < MARK < OFF)
+     */
     LoggerLevel["DEBUG"] = "DEBUG";
+    /**
+     * Log levels are used to assign importance to log messages, with the integer value being used to sort them.
+     * If you do not specify anything in your configuration, the default values are used (ALL < TRACE < DEBUG < INFO < WARN < ERROR < FATAL < MARK < OFF)
+     */
     LoggerLevel["INFO"] = "INFO";
+    /**
+     * Log levels are used to assign importance to log messages, with the integer value being used to sort them.
+     * If you do not specify anything in your configuration, the default values are used (ALL < TRACE < DEBUG < INFO < WARN < ERROR < FATAL < MARK < OFF)
+     */
     LoggerLevel["WARN"] = "WARN";
+    /**
+     * Log levels are used to assign importance to log messages, with the integer value being used to sort them.
+     * If you do not specify anything in your configuration, the default values are used (ALL < TRACE < DEBUG < INFO < WARN < ERROR < FATAL < MARK < OFF)
+     */
     LoggerLevel["ERROR"] = "ERROR";
+    /**
+     * Log levels are used to assign importance to log messages, with the integer value being used to sort them.
+     * If you do not specify anything in your configuration, the default values are used (ALL < TRACE < DEBUG < INFO < WARN < ERROR < FATAL < MARK < OFF)
+     */
     LoggerLevel["FATAL"] = "FATAL";
+    /**
+     * Log levels are used to assign importance to log messages, with the integer value being used to sort them.
+     * If you do not specify anything in your configuration, the default values are used (ALL < TRACE < DEBUG < INFO < WARN < ERROR < FATAL < MARK < OFF)
+     */
     LoggerLevel["MARK"] = "MARK";
+    /**
+     * Disable all logging.
+     * Log levels are used to assign importance to log messages, with the integer value being used to sort them.
+     * If you do not specify anything in your configuration, the default values are used (ALL < TRACE < DEBUG < INFO < WARN < ERROR < FATAL < MARK < OFF)
+     */
     LoggerLevel["OFF"] = "OFF";
 })(LoggerLevel || (exports.LoggerLevel = LoggerLevel = {}));
 var LoggerConfigData;
@@ -350,3 +402,4 @@ var PackageManagers;
     PackageManagers["yarn2"] = "yarn2";
     PackageManagers["bun"] = "bun";
 })(PackageManagers || (exports.PackageManagers = PackageManagers = {}));
+//# sourceMappingURL=constants.js.map

package/lib/controllers/build-controller.js

@@ -35,7 +35,7 @@ class BuildController extends events_1.EventEmitter {
         const platform = buildData.platform.toLowerCase();
         const projectData = this.$projectDataService.getProjectData(buildData.projectDir);
         const platformData = this.$platformsDataService.getPlatformData(platform, projectData);
-        const action = "Build";
+        const action = "Build" /* constants.TrackActionNames.Build */;
         const isForDevice = this.$mobileHelper.isAndroidPlatform(platform)
             ? null
             : buildData && buildData.buildForDevice;
@@ -45,8 +45,8 @@ class BuildController extends events_1.EventEmitter {
             platform,
             projectDir: projectData.projectDir,
             additionalData: `${buildData.release ? constants_1.Configurations.Release : constants_1.Configurations.Debug}_${buildData.clean
-                ? "Clean"
-                : "Incremental"}`,
+                ? "Clean" /* constants.BuildStates.Clean */
+                : "Incremental" /* constants.BuildStates.Incremental */}`,
         });
         if (buildData.clean) {
             await platformData.platformProjectService.cleanProject(platformData.projectRoot);
@@ -114,3 +114,4 @@ class BuildController extends events_1.EventEmitter {
 }
 exports.BuildController = BuildController;
 yok_1.injector.register("buildController", BuildController);
+//# sourceMappingURL=build-controller.js.map

package/lib/controllers/debug-controller.js

@@ -40,13 +40,13 @@ class DebugController extends events_1.EventEmitter {
             this.$errors.fail(`The device with identifier ${debugData.deviceIdentifier} is unreachable. Make sure it is Trusted and try again.`);
         }
         await this.$analyticsService.trackEventActionInGoogleAnalytics({
-            action: "Debug",
+            action: "Debug" /* TrackActionNames.Debug */,
             device,
             additionalData: this.$mobileHelper.isiOSPlatform(device.deviceInfo.platform) &&
                 options &&
                 options.inspector
-                ? "Inspector"
-                : "Chrome",
+                ? "Inspector" /* DebugTools.Inspector */
+                : "Chrome" /* DebugTools.Chrome */,
             projectDir: debugData.projectDir,
         });
         if (!(await device.applicationManager.isApplicationInstalled(debugData.applicationIdentifier))) {
@@ -87,6 +87,7 @@ class DebugController extends events_1.EventEmitter {
         }
     }
     async attachDebugger(attachDebuggerData) {
+        // Default values
         if (attachDebuggerData.debugOptions) {
             attachDebuggerData.debugOptions.chrome =
                 attachDebuggerData.debugOptions.chrome === undefined
@@ -105,6 +106,9 @@ class DebugController extends events_1.EventEmitter {
         }
         const projectData = this.$projectDataService.getProjectData(attachDebuggerData.projectDir);
         const debugData = this.$debugDataService.getDebugData(attachDebuggerData.deviceIdentifier, projectData, attachDebuggerData.debugOptions);
+        // const platformData = this.$platformsDataService.getPlatformData(settings.platform, projectData);
+        // Of the properties below only `buildForDevice` and `release` are currently used.
+        // Leaving the others with placeholder values so that they may not be forgotten in future implementations.
         const debugInfo = await this.startDebug(debugData);
         const result = this.printDebugInformation(debugInfo, attachDebuggerData.debugOptions.forceDebuggerAttachedEvent);
         return result;
@@ -212,3 +216,4 @@ __decorate([
     (0, decorators_1.performanceLog)()
 ], DebugController.prototype, "enableDebuggingCoreWithoutWaitingCurrentAction", null);
 yok_1.injector.register("debugController", DebugController);
+//# sourceMappingURL=debug-controller.js.map

package/lib/controllers/deploy-controller.js

@@ -28,3 +28,4 @@ class DeployController {
 }
 exports.DeployController = DeployController;
 yok_1.injector.register("deployController", DeployController);
+//# sourceMappingURL=deploy-controller.js.map

package/lib/controllers/migrate-controller.js

@@ -13,8 +13,12 @@ const yok_1 = require("../common/yok");
 const fs = require("fs");
 const os_1 = require("os");
 const color_1 = require("../color");
+// const wait: (ms: number) => Promise<void> = (ms: number = 1000) =>
+// 	new Promise((resolve) => setTimeout(resolve, ms));
 class MigrateController extends update_controller_base_1.UpdateControllerBase {
-    constructor($fs, $platformCommandHelper, $platformsDataService, $packageInstallationManager, $packageManager, $pacoteService, $logger, $errors, $pluginsService, $projectDataService, $projectConfigService, $options, $resources, $injector, $settingsService, $staticConfig, $terminalSpinnerService, $projectCleanupService, $projectBackupService, $childProcess) {
+    constructor($fs, $platformCommandHelper, $platformsDataService, $packageInstallationManager, $packageManager, $pacoteService, 
+    // private $androidResourcesMigrationService: IAndroidResourcesMigrationService,
+    $logger, $errors, $pluginsService, $projectDataService, $projectConfigService, $options, $resources, $injector, $settingsService, $staticConfig, $terminalSpinnerService, $projectCleanupService, $projectBackupService, $childProcess) {
         super($fs, $platformCommandHelper, $platformsDataService, $packageInstallationManager, $packageManager, $pacoteService);
         this.$fs = $fs;
         this.$platformCommandHelper = $platformCommandHelper;
@@ -152,15 +156,18 @@ class MigrateController extends update_controller_base_1.UpdateControllerBase {
             {
                 packageName: "node-sass",
                 replaceWith: "sass",
-                minVersion: "0.0.0",
+                minVersion: "0.0.0", // ignore
                 isDev: true,
+                // shouldRemove: true,
             },
             {
                 packageName: "sass",
-                minVersion: "0.0.0",
+                minVersion: "0.0.0", // ignore
                 desiredVersion: "^1.49.9",
                 isDev: true,
+                // shouldRemove: true,
             },
+            // runtimes
             {
                 packageName: "tns-ios",
                 minVersion: "6.5.3",
@@ -203,8 +210,10 @@ class MigrateController extends update_controller_base_1.UpdateControllerBase {
                 remainingPlatforms.push(platform);
                 continue;
             }
+            // should only run in loose mode...
             const cachedResult = await this.getCachedShouldMigrate(projectDir, platform);
             this.$logger.trace(`Got cached result for shouldMigrate for platform: ${platform}: ${cachedResult}`);
+            // the cached result is only used if it's false, otherwise we need to check again
             if (cachedResult !== false) {
                 remainingPlatforms.push(platform);
             }
@@ -216,6 +225,7 @@ class MigrateController extends update_controller_base_1.UpdateControllerBase {
                 loose,
             });
             this.$logger.trace(`Executed shouldMigrate for platforms: ${remainingPlatforms}. Result is: ${shouldMigrate}`);
+            // only cache results if running in loose mode
             if (!shouldMigrate && loose) {
                 for (const remainingPlatform of remainingPlatforms) {
                     await this.setCachedShouldMigrate(projectDir, remainingPlatform);
@@ -242,37 +252,46 @@ class MigrateController extends update_controller_base_1.UpdateControllerBase {
             platforms,
             loose: loose,
         });
+        // ensure in git repo and require --force if not (for safety)
+        // ensure git branch is clean
         const canMigrate = await this.ensureGitCleanOrForce(projectDir);
         if (!canMigrate) {
             this.spinner.fail("Pre-Migration verification failed");
             return;
         }
         this.spinner.succeed("Pre-Migration verification complete");
+        // back up project files and folders
         this.spinner.info("Backing up project files before migration");
         const backup = await this.backupProject(projectDir);
         this.spinner.succeed("Project files have been backed up");
+        // clean up project files
         this.spinner.info("Cleaning up project files before migration");
         await this.cleanUpProject(projectData);
         this.spinner.succeed("Project files have been cleaned up");
+        // clean up artifacts
         this.spinner.info("Cleaning up old artifacts");
         await this.handleAutoGeneratedFiles(backup, projectData);
         this.spinner.succeed("Cleaned old artifacts");
         const newConfigPath = path.resolve(projectDir, "nativescript.config.ts");
         if (!this.$fs.exists(newConfigPath)) {
+            // migrate configs
             this.spinner.info(`Migrating project to use ${color_1.color.green("nativescript.config.ts")}`);
             await this.migrateConfigs(projectDir);
             this.spinner.succeed(`Project has been migrated to use ${color_1.color.green("nativescript.config.ts")}`);
         }
+        // update dependencies
         this.spinner.info("Updating project dependencies");
         await this.migrateDependencies(projectData, platforms, loose);
         this.spinner.succeed("Project dependencies have been updated");
         const isAngular = this.hasDependency({
             packageName: "@nativescript/angular",
         }, projectData);
+        // ensure polyfills.ts exists in angular projects
         let polyfillsPath;
         if (isAngular) {
             polyfillsPath = await this.checkOrCreatePolyfillsTS(projectData);
         }
+        // update tsconfig
         const tsConfigPath = path.resolve(projectDir, "tsconfig.json");
         if (this.$fs.exists(tsConfigPath)) {
             this.spinner.info(`Updating ${color_1.color.yellow("tsconfig.json")}`);
@@ -284,11 +303,19 @@ class MigrateController extends update_controller_base_1.UpdateControllerBase {
             this.spinner.succeed(`Updated ${color_1.color.yellow("tsconfig.json")}`);
         }
         await this.migrateWebpack5(projectDir, projectData);
+        // run @nativescript/eslint over codebase
         await this.runESLint(projectDir);
         this.spinner.succeed("Migration complete.");
         this.$logger.info("");
         this.$logger.printMarkdown("Project has been successfully migrated. The next step is to run `ns run <platform>` to ensure everything is working properly." +
             "\n\nPlease note that you may need additional changes to complete the migration.");
+        // print markdown for next steps:
+        // if no runtime has been added, print a message that it will be added when they run ns run <platform>
+        // if all is good, run ns migrate clean to clean up backup folders
+        // in case of failure, print diagnostic data: what failed and why
+        // restore all files - or perhaps let the user sort it out
+        // or ns migrate restore - to restore from pre-migration backup
+        // for some known cases, print suggestions perhaps
     }
     async _shouldMigrate({ projectDir, platforms, loose, }) {
         const isMigrate = _.get(this.$options, "argv._[0]") === "migrate";
@@ -305,6 +332,7 @@ class MigrateController extends update_controller_base_1.UpdateControllerBase {
                 if (dependency.shouldAddIfMissing) {
                     this.$logger.trace(`${shouldMigrateCommonMessage}'${dependency.packageName}' is missing.`);
                     if (loose) {
+                        // in loose mode we ignore missing dependencies
                         continue;
                     }
                     return true;
@@ -320,6 +348,7 @@ class MigrateController extends update_controller_base_1.UpdateControllerBase {
             }
             if (dependency.replaceWith || dependency.shouldRemove) {
                 this.$logger.trace(`${shouldMigrateCommonMessage}'${dependency.packageName}' is deprecated.`);
+                // in loose mode we ignore deprecated dependencies
                 if (loose) {
                     continue;
                 }
@@ -362,11 +391,32 @@ class MigrateController extends update_controller_base_1.UpdateControllerBase {
         const projectPackageJsonFilePath = path.join(projectDir, constants.PACKAGE_JSON_FILE_NAME);
         return await this.$fs.getFileShasum(projectPackageJsonFilePath);
     }
+    // private async migrateOldAndroidAppResources(
+    // 	projectData: IProjectData,
+    // 	backupDir: string
+    // ) {
+    // 	const appResourcesPath = projectData.getAppResourcesDirectoryPath();
+    // 	if (!this.$androidResourcesMigrationService.hasMigrated(appResourcesPath)) {
+    // 		this.spinner.info("Migrate old Android App_Resources structure.");
+    // 		try {
+    // 			await this.$androidResourcesMigrationService.migrate(
+    // 				appResourcesPath,
+    // 				backupDir
+    // 			);
+    // 		} catch (error) {
+    // 			this.$logger.warn(
+    // 				"Migrate old Android App_Resources structure failed: ",
+    // 				error.message
+    // 			);
+    // 		}
+    // 	}
+    // }
     async ensureGitCleanOrForce(projectDir) {
         const git = (0, simple_git_1.default)(projectDir);
         const isGit = await git.checkIsRepo();
         const isForce = this.$options.force;
         if (!isGit) {
+            // not a git repo and no --force
             if (!isForce) {
                 this.$logger.printMarkdown(`Running \`ns migrate\` in a non-git project is not recommended. If you want to skip this check run \`ns migrate --force\`.`);
                 this.$errors.fail("Not in Git repo.");
@@ -413,6 +463,7 @@ class MigrateController extends update_controller_base_1.UpdateControllerBase {
         const { dependencies, devDependencies } = await this.$pluginsService.getDependenciesFromPackageJson(projectData.projectDir);
         const hasSchematics = [...dependencies, ...devDependencies].find((p) => p.name === "@nativescript/schematics");
         if (!hasSchematics) {
+            // clean tsconfig.tns.json if not in a shared project
             await this.$projectCleanupService.clean([
                 constants.TSCCONFIG_TNS_JSON_NAME,
             ]);
@@ -459,12 +510,14 @@ class MigrateController extends update_controller_base_1.UpdateControllerBase {
         });
     }
     isOutdatedVersion(current, target, loose) {
+        // in loose mode, a falsy version is not considered outdated
         if (!current && loose) {
             return false;
         }
         const installed = semver.coerce(current);
         const min = semver.coerce(target.minVersion);
         const desired = semver.coerce(target.desiredVersion);
+        // in loose mode we check if we satisfy the min version
         if (loose) {
             if (!installed || !min) {
                 return false;
@@ -474,6 +527,7 @@ class MigrateController extends update_controller_base_1.UpdateControllerBase {
         if (!installed || !desired) {
             return true;
         }
+        // otherwise we compare with the desired version
         return semver.lt(installed, desired);
     }
     detectAppPath(projectDir, configData) {
@@ -536,6 +590,7 @@ class MigrateController extends update_controller_base_1.UpdateControllerBase {
     async migrateDependency(dependency, projectData, loose) {
         var _a, _b, _c, _d, _e;
         const hasDependency = this.hasDependency(dependency, projectData);
+        // show warning if needed
         if (hasDependency && dependency.warning) {
             this.$logger.warn(dependency.warning);
         }
@@ -551,7 +606,9 @@ class MigrateController extends update_controller_base_1.UpdateControllerBase {
             return;
         }
         if (dependency.replaceWith || dependency.shouldRemove) {
+            // remove
             this.$pluginsService.removeFromPackageJson(dependency.packageName, projectData.projectDir);
+            // no replacement required - we're done
             if (!dependency.replaceWith) {
                 return;
             }
@@ -560,6 +617,7 @@ class MigrateController extends update_controller_base_1.UpdateControllerBase {
                 this.$errors.fail("Failed to find replacement dependency.");
             }
             const version = (_d = (_c = (_b = replacementDep.desiredVersion) !== null && _b !== void 0 ? _b : replacementDep.minVersion) !== null && _c !== void 0 ? _c : dependency.desiredVersion) !== null && _d !== void 0 ? _d : dependency.minVersion;
+            // add replacement dependency
             this.$pluginsService.addToPackageJson(replacementDep.packageName, version, replacementDep.isDev, projectData.projectDir);
             this.spinner.clear();
             this.$logger.info(`  - ${color_1.color.yellow(dependency.packageName)} has been replaced with ${color_1.color.cyan(replacementDep.packageName)} ${color_1.color.green(version)}`);
@@ -579,40 +637,57 @@ class MigrateController extends update_controller_base_1.UpdateControllerBase {
     }
     async migrateConfigs(projectDir) {
         const projectData = this.$projectDataService.getProjectData(projectDir);
+        // package.json
         const rootPackageJsonPath = path.resolve(projectDir, constants.PACKAGE_JSON_FILE_NAME);
+        // nested package.json
         const embeddedPackageJsonPath = path.resolve(projectData.projectDir, projectData.getAppDirectoryRelativePath(), constants.PACKAGE_JSON_FILE_NAME);
+        // nsconfig.json
         const legacyNsConfigPath = path.resolve(projectData.projectDir, constants.CONFIG_NS_FILE_NAME);
         let rootPackageJsonData = {};
         if (this.$fs.exists(rootPackageJsonPath)) {
             rootPackageJsonData = this.$fs.readJson(rootPackageJsonPath);
         }
+        // write the default config unless it already exists
         const newConfigPath = this.$projectConfigService.writeDefaultConfig(projectData.projectDir);
+        // force legacy config mode
         this.$projectConfigService.setForceUsingLegacyConfig(true);
+        // all different sources are combined into configData (nested package.json, nsconfig and root package.json[nativescript])
         const configData = this.$projectConfigService.readConfig(projectData.projectDir);
+        // we no longer want to force legacy config mode
         this.$projectConfigService.setForceUsingLegacyConfig(false);
+        // move main key into root package.json
         if (configData.main) {
             rootPackageJsonData.main = configData.main;
             delete configData.main;
         }
+        // detect appPath and App_Resources path
         configData.appPath = this.detectAppPath(projectDir, configData);
         configData.appResourcesPath = this.detectAppResourcesPath(projectDir, configData);
+        // delete nativescript key from root package.json
         if (rootPackageJsonData.nativescript) {
             delete rootPackageJsonData.nativescript;
         }
+        // force the config service to use nativescript.config.ts
         this.$projectConfigService.setForceUsingNewConfig(true);
-        const hasUpdatedConfigSuccessfully = await this.$projectConfigService.setValue("", configData);
+        // migrate data into nativescript.config.ts
+        const hasUpdatedConfigSuccessfully = await this.$projectConfigService.setValue("", // root
+        configData);
         if (!hasUpdatedConfigSuccessfully) {
             if (typeof newConfigPath === "string") {
+                // only clean the config if it was created by the migration script
                 await this.$projectCleanupService.cleanPath(newConfigPath);
             }
             this.$errors.fail(`Failed to migrate project to use ${constants.CONFIG_FILE_NAME_TS}. One or more values could not be updated.`);
         }
+        // save root package.json
         this.$fs.writeJson(rootPackageJsonPath, rootPackageJsonData);
+        // delete migrated files
         await this.$projectCleanupService.cleanPath(embeddedPackageJsonPath);
         await this.$projectCleanupService.cleanPath(legacyNsConfigPath);
         return true;
     }
     async migrateUnitTestRunner(projectData, migrationBackupDirPath) {
+        // Migrate karma.conf.js
         const pathToKarmaConfig = path.join(migrationBackupDirPath, constants.KARMA_CONFIG_NAME);
         if (this.$fs.exists(pathToKarmaConfig)) {
             const oldKarmaContent = this.$fs.readText(pathToKarmaConfig);
@@ -630,6 +705,7 @@ class MigrateController extends update_controller_base_1.UpdateControllerBase {
             });
             this.$fs.writeFile(path.join(projectData.projectDir, constants.KARMA_CONFIG_NAME), karmaConf);
         }
+        // Dependencies to migrate
         const dependencies = [
             {
                 packageName: "karma-webpack",
@@ -671,6 +747,7 @@ class MigrateController extends update_controller_base_1.UpdateControllerBase {
     async migrateTSConfig({ tsConfigPath, isAngular, polyfillsPath, }) {
         try {
             const configContents = this.$fs.readJson(tsConfigPath);
+            // update
             configContents.compilerOptions = configContents.compilerOptions || {};
             configContents.compilerOptions.target = "es2020";
             configContents.compilerOptions.module = "esnext";
@@ -681,6 +758,7 @@ class MigrateController extends update_controller_base_1.UpdateControllerBase {
                 ...new Set([...(configContents.compilerOptions.lib || []), "ESNext"]),
             ];
             if (isAngular) {
+                // make sure polyfills.ts is in files
                 if (configContents.files) {
                     configContents.files = [
                         ...new Set([
@@ -712,8 +790,10 @@ class MigrateController extends update_controller_base_1.UpdateControllerBase {
             return "./" + path.relative(projectDir, polyfillsPath);
         }
         const tempDir = fs.mkdtempSync(path.join((0, os_1.tmpdir)(), "migrate-angular-polyfills-"));
+        // get from default angular template
         await this.$pacoteService.extractPackage(constants.RESERVED_TEMPLATE_NAMES["angular"], tempDir);
         this.$fs.copyFile(path.resolve(tempDir, "src/polyfills.ts"), possiblePaths[0]);
+        // clean up temp project
         this.$fs.deleteDirectory(tempDir);
         this.spinner.succeed(`Created fresh ${color_1.color.cyan("polyfills.ts")}`);
         return "./" + path.relative(projectDir, possiblePaths[0]);
@@ -782,6 +862,7 @@ class MigrateController extends update_controller_base_1.UpdateControllerBase {
                 desiredVersion: "~0.15.0",
                 shouldAddIfMissing: true,
             },
+            // devDependencies
             {
                 packageName: "@angular/cli",
                 minVersion,
@@ -852,10 +933,12 @@ class MigrateController extends update_controller_base_1.UpdateControllerBase {
                 packageName: "@babel/preset-env",
                 shouldRemove: true,
             },
+            // remove any version of vue
             {
                 packageName: "vue",
                 shouldRemove: true,
             },
+            // add latest
             {
                 packageName: "vue",
                 desiredVersion: "2.6.12",
@@ -951,6 +1034,7 @@ class MigrateController extends update_controller_base_1.UpdateControllerBase {
                 return;
             }
         }
+        // clean old config before generating new one
         await this.$projectCleanupService.clean(["webpack.config.js"]);
         this.spinner.info(`Initializing new ${color_1.color.yellow("webpack.config.js")}`);
         const { desiredVersion: webpackVersion } = this.migrationDependencies.find((dep) => dep.packageName === constants.WEBPACK_PLUGIN_NAME);
@@ -1025,6 +1109,7 @@ class MigrateController extends update_controller_base_1.UpdateControllerBase {
     }
 }
 exports.MigrateController = MigrateController;
+// static readonly typescriptPackageName: string = "typescript";
 MigrateController.backupFolderName = ".migration_backup";
 MigrateController.pathsToBackup = [
     constants.LIB_DIR_NAME,
@@ -1037,3 +1122,4 @@ MigrateController.pathsToBackup = [
     constants.CONFIG_NS_FILE_NAME,
 ];
 yok_1.injector.register("migrateController", MigrateController);
+//# sourceMappingURL=migrate-controller.js.map

package/lib/controllers/platform-controller.js

@@ -84,9 +84,13 @@ class PlatformController {
                 desiredRuntimePackage.version = version;
             }
             if (!desiredRuntimePackage.version) {
+                // if no version is explicitly added, then we use the latest
                 desiredRuntimePackage.version =
                     await this.$packageInstallationManager.getLatestCompatibleVersion(desiredRuntimePackage.name);
             }
+            // const currentPlatformData = this.$projectDataService.getNSValue(projectData.projectDir, platformData.frameworkPackageName);
+            // version = (currentPlatformData && currentPlatformData.version) ||
+            // 	await this.$packageInstallationManager.getLatestCompatibleVersion(platformData.frameworkPackageName);
             result = `${desiredRuntimePackage.name}@${desiredRuntimePackage.version}`;
         }
         return result;
@@ -98,11 +102,14 @@ class PlatformController {
         const prepareInfo = this.$projectChangesService.getPrepareInfo(platformData);
         const requiresNativePlatformAdd = prepareInfo &&
             prepareInfo.nativePlatformStatus ===
-                "1";
+                "1" /* NativePlatformStatus.requiresPlatformAdd */;
         const shouldAddPlatform = !hasPlatformDirectory ||
             (shouldAddNativePlatform && requiresNativePlatformAdd);
         if (hasPlatformDirectory && !shouldAddPlatform) {
             const platformDirectoryItemCount = this.$fs.readDirectory(path.join(projectData.platformsDir, platformName)).length;
+            // 2 is a magic number to approximate a valid platform folder
+            // any valid platform should contain at least 2 files/folders
+            // we choose 2 to avoid false-positives due to system files like .DS_Store etc.
             if (platformDirectoryItemCount <= 2) {
                 this.$logger.warn(`The platforms/${platformName} folder appears to be invalid. If the build fails, run 'ns clean' and rebuild the app.`, { wrapMessageWithBorders: true });
             }
@@ -112,3 +119,4 @@ class PlatformController {
 }
 exports.PlatformController = PlatformController;
 yok_1.injector.register("platformController", PlatformController);
+//# sourceMappingURL=platform-controller.js.map

package/lib/controllers/prepare-controller.js

@@ -76,6 +76,10 @@ class PrepareController extends events_1.EventEmitter {
         await this.$platformController.addPlatformIfNeeded(prepareData, projectData);
         await this.trackRuntimeVersion(prepareData.platform, projectData);
         this.$logger.info("Preparing project...");
+        // we need to mark the ~/package.json (used by core modules)
+        // as external for us to be able to write the config to it
+        // in writeRuntimePackageJson() below, because otherwise
+        // webpack will inline it into the bundle/vendor chunks
         prepareData.env = prepareData.env || {};
         prepareData.env.externals = prepareData.env.externals || [];
         prepareData.env.externals.push("~/package.json");
@@ -116,8 +120,8 @@ class PrepareController extends events_1.EventEmitter {
                 },
             };
         }
-        await this.startJSWatcherWithPrepare(platformData, projectData, prepareData);
-        const hasNativeChanges = await this.startNativeWatcherWithPrepare(platformData, projectData, prepareData);
+        await this.startJSWatcherWithPrepare(platformData, projectData, prepareData); // -> start watcher + initial compilation
+        const hasNativeChanges = await this.startNativeWatcherWithPrepare(platformData, projectData, prepareData); // -> start watcher + initial prepare
         const result = {
             platform: platformData.platformNameLowerCase,
             hasNativeChanges,
@@ -126,6 +130,7 @@ class PrepareController extends events_1.EventEmitter {
         if (hasPersistedDataWithNativeChanges) {
             result.hasNativeChanges = true;
         }
+        // TODO: Do not persist this in `this` context. Also it should be per platform.
         this.isInitialPrepareReady = true;
         if (this.persistedData && this.persistedData.length) {
             this.emitPrepareEvent({
@@ -178,7 +183,7 @@ class PrepareController extends events_1.EventEmitter {
                 pollInterval: 100,
                 stabilityThreshold: 500,
             },
-            ignored: ["**/.*", ".*"],
+            ignored: ["**/.*", ".*"], // hidden files
         };
         const watcher = (0, chokidar_1.watch)(patterns, watcherOptions).on("all", async (event, filePath) => {
             if (this.isFileWatcherPaused())
@@ -220,6 +225,9 @@ class PrepareController extends events_1.EventEmitter {
             .concat(pluginsPackageJsonFiles);
         return patterns;
     }
+    /**
+     * TODO: move this logic to the webpack side of things - WIP and deprecate here with a webpack version check...
+     */
     async writeRuntimePackageJson(projectData, platformData, prepareData = null) {
         const configInfo = this.$projectConfigService.detectProjectConfigs(projectData.projectDir);
         if (configInfo.usingNSConfig) {
@@ -252,7 +260,11 @@ class PrepareController extends events_1.EventEmitter {
             packagePath = path.join(platformData.projectRoot, this.$options.hostProjectModuleName, "src", this.$options.hostProjectPath ? "nativescript" : "main", "assets", "app", "package.json");
         }
         try {
+            // this will read the package.json that is already emitted by
+            // the GenerateNativeScriptEntryPointsPlugin webpack plugin
             const emittedPackageData = this.$fs.readJson(packagePath);
+            // since ns7 we only care about the main key from the emitted
+            // package.json, the rest is coming from the new config.
             if (emittedPackageData === null || emittedPackageData === void 0 ? void 0 : emittedPackageData.main) {
                 packageData.main = emittedPackageData.main;
             }
@@ -280,7 +292,7 @@ class PrepareController extends events_1.EventEmitter {
             return;
         }
         await this.$analyticsService.trackEventActionInGoogleAnalytics({
-            action: "Using Runtime Version",
+            action: "Using Runtime Version" /* TrackActionNames.UsingRuntimeVersion */,
             additionalData: `${platform.toLowerCase()}${constants_1.AnalyticsEventLabelDelimiter}${version}`,
         });
     }
@@ -325,3 +337,4 @@ __decorate([
     (0, decorators_1.cache)()
 ], PrepareController.prototype, "trackRuntimeVersion", null);
 yok_1.injector.register("prepareController", PrepareController);
+//# sourceMappingURL=prepare-controller.js.map