nativescript 8.8.0 → 8.8.1

package/lib/common/verify-node-version.js

@@ -1,10 +1,15 @@
 "use strict";
+// This function must be separate to avoid dependencies on C++ modules - it must execute precisely when other functions cannot
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.getNodeWarning = exports.verifyNodeVersion = void 0;
 const color_1 = require("../color");
+// Use only ES5 code here - pure JavaScript can be executed with any Node.js version (even 0.10, 0.12).
+/* tslint:disable:no-var-keyword no-var-requires prefer-const*/
 var os = require("os");
 var semver = require("semver");
 var util = require("util");
+// These versions cannot be used with CLI due to bugs in the node itself.
+// We are absolutely sure we cannot work with them, so inform the user if he is trying to use any of them and exit the process.
 var versionsCausingFailure = ["0.10.34", "4.0.0", "4.2.0", "5.0.0"];
 var minimumRequiredVersion = "8.0.0";
 function getNodeVersionOpts() {
@@ -69,10 +74,12 @@ function getNodeWarning() {
         if (warningMessage) {
             nodeWarn = {
                 message: warningMessage,
-                severity: "medium",
+                severity: "medium" /* SystemWarningsSeverity.medium */,
             };
         }
     }
     return nodeWarn;
 }
 exports.getNodeWarning = getNodeWarning;
+/* tslint:enable */
+//# sourceMappingURL=verify-node-version.js.map

package/lib/common/yok.js

@@ -16,6 +16,13 @@ const helpers_1 = require("./helpers");
 const constants_1 = require("./constants");
 let indent = "";
 function trace(formatStr, ...args) {
+    // uncomment following lines when debugging dependency injection
+    // const items: any[] = [];
+    // for (let _i = 1; _i < arguments.length; _i++) {
+    // 	items[_i - 1] = arguments[_i];
+    // }
+    // const util = require("util");
+    // console.log(util.format.apply(util, [indent + formatStr].concat(args)));
 }
 function pushIndent() {
     indent += "  ";
@@ -33,6 +40,7 @@ function forEachName(names, action) {
 }
 function register(...rest) {
     return function (target) {
+        // TODO: Check if 'rest' has more arguments that have to be registered
         exports.injector.register(rest[0], target);
     };
 }
@@ -52,7 +60,7 @@ class Yok {
     }
     requireCommand(names, file) {
         forEachName(names, (commandName) => {
-            const commands = commandName.split("|");
+            const commands = commandName.split("|" /* CommandsDelimiters.HierarchicalCommand */);
             if (commands.length > 1) {
                 if (_.startsWith(commands[1], "*") &&
                     this.modules[this.createCommandName(commands[0])]) {
@@ -62,7 +70,7 @@ class Yok {
                 if (!this.hierarchicalCommands[parentCommandName]) {
                     this.hierarchicalCommands[parentCommandName] = [];
                 }
-                this.hierarchicalCommands[parentCommandName].push(_.tail(commands).join("|"));
+                this.hierarchicalCommands[parentCommandName].push(_.tail(commands).join("|" /* CommandsDelimiters.HierarchicalCommand */));
             }
             if (commands.length > 1 &&
                 !this.modules[this.createCommandName(commands[0])]) {
@@ -133,7 +141,7 @@ class Yok {
     }
     registerCommand(names, resolver) {
         forEachName(names, (name) => {
-            const commands = name.split("|");
+            const commands = name.split("|" /* CommandsDelimiters.HierarchicalCommand */);
             this.register(this.createCommandName(name), resolver);
             if (commands.length > 1) {
                 this.createHierarchicalCommand(commands[0]);
@@ -145,7 +153,7 @@ class Yok {
     }
     getDefaultCommand(name, commandArguments) {
         const subCommands = this.hierarchicalCommands[name];
-        const defaultCommand = _.find(subCommands, (command) => _.some(command.split("|"), (c) => _.startsWith(c, "*")));
+        const defaultCommand = _.find(subCommands, (command) => _.some(command.split("|" /* CommandsDelimiters.HierarchicalCommand */), (c) => _.startsWith(c, "*" /* CommandsDelimiters.DefaultCommandSymbol */)));
         return defaultCommand;
     }
     buildHierarchicalCommand(parentCommandName, commandLineArguments) {
@@ -161,7 +169,7 @@ class Yok {
             remainingArguments = _.tail(remainingArguments);
             if ((matchingSubCommandName = _.find(subCommands, (sc) => sc === currentSubCommandName ||
                 sc ===
-                    `${"*"}${currentSubCommandName}`))) {
+                    `${"*" /* CommandsDelimiters.DefaultCommandSymbol */}${currentSubCommandName}`))) {
                 finalSubCommandName = matchingSubCommandName;
                 finalRemainingArguments = remainingArguments;
             }
@@ -170,8 +178,8 @@ class Yok {
             finalSubCommandName =
                 this.getDefaultCommand(parentCommandName, commandLineArguments) || "";
             finalRemainingArguments = _.difference(commandLineArguments, finalSubCommandName
-                .split("|")
-                .map((command) => _.startsWith(command, "*")
+                .split("|" /* CommandsDelimiters.HierarchicalCommand */)
+                .map((command) => _.startsWith(command, "*" /* CommandsDelimiters.DefaultCommandSymbol */)
                 ? command.substr(1)
                 : command));
         }
@@ -202,7 +210,9 @@ class Yok {
                             commandName = defaultCommand
                                 ? this.getHierarchicalCommandName(name, defaultCommand)
                                 : "help";
-                            if (_.includes(this.hierarchicalCommands[name], "*" + args[0])) {
+                            // If we'll execute the default command, but it's full name had been written by the user
+                            // for example "tns run ios", we have to remove the "ios" option from the arguments that we'll pass to the command.
+                            if (_.includes(this.hierarchicalCommands[name], "*" /* CommandsDelimiters.DefaultCommandSymbol */ + args[0])) {
                                 commandArguments = _.tail(args);
                             }
                             else {
@@ -211,11 +221,13 @@ class Yok {
                         }
                     }
                     else {
+                        //Execute only default command without arguments
                         if (defaultCommand) {
                             commandName = this.getHierarchicalCommandName(name, defaultCommand);
                         }
                         else {
                             commandName = "help";
+                            // Show command-line help
                             const options = this.resolve("options");
                             options.help = true;
                         }
@@ -227,7 +239,7 @@ class Yok {
         exports.injector.registerCommand(name, factory);
     }
     getHierarchicalCommandName(parentCommandName, subCommandName) {
-        return [parentCommandName, subCommandName].join("|");
+        return [parentCommandName, subCommandName].join("|" /* CommandsDelimiters.HierarchicalCommand */);
     }
     isValidHierarchicalCommand(commandName, commandArguments) {
         return __awaiter(this, void 0, void 0, function* () {
@@ -236,6 +248,8 @@ class Yok {
                 if (subCommands) {
                     const fullCommandName = this.buildHierarchicalCommand(commandName, commandArguments);
                     if (!fullCommandName) {
+                        // In case buildHierarchicalCommand doesn't find a valid command
+                        // there isn't a valid command or default with those arguments
                         const errors = exports.injector.resolve("errors");
                         errors.failWithHelp(constants_1.ERROR_NO_VALID_SUBCOMMAND_FORMAT, commandName);
                     }
@@ -246,8 +260,8 @@ class Yok {
         });
     }
     isDefaultCommand(commandName) {
-        return (commandName.indexOf("*") > 0 &&
-            commandName.indexOf("|") > 0);
+        return (commandName.indexOf("*" /* CommandsDelimiters.DefaultCommandSymbol */) > 0 &&
+            commandName.indexOf("|" /* CommandsDelimiters.HierarchicalCommand */) > 0);
     }
     register(name, resolver, shared) {
         shared = shared === undefined ? true : shared;
@@ -294,6 +308,12 @@ class Yok {
             return this.resolveByName(param, ctorArguments);
         }
     }
+    /* Regex to match dynamic calls in the following format:
+        #{moduleName.functionName} or
+        #{moduleName.functionName(param1)} or
+        #{moduleName.functionName(param1, param2)} - multiple parameters separated with comma are supported
+        Check dynamicCall method for sample usage of this regular expression and see how to determine the passed parameters
+    */
     get dynamicCallRegex() {
         return /#{([^.]+)\.([^}]+?)(\((.+)\))*}/;
     }
@@ -424,3 +444,4 @@ function setGlobalInjector(inj) {
     return inj;
 }
 exports.setGlobalInjector = setGlobalInjector;
+//# sourceMappingURL=yok.js.map

package/lib/config.js

@@ -16,8 +16,10 @@ const os = require("os");
 const _ = require("lodash");
 const yok_1 = require("./common/yok");
 class Configuration {
+    /*don't require logger and everything that has logger as dependency in config.js due to cyclic dependency*/
     constructor($fs) {
         this.$fs = $fs;
+        // User specific config
         this.DEBUG = false;
         this.ANDROID_DEBUG_UI = null;
         this.USE_POD_SANDBOX = false;
@@ -56,6 +58,7 @@ class StaticConfig {
         this._userAgent = null;
     }
     get disableCommandHooks() {
+        // Never set this to false because it will duplicate execution of hooks realized through method decoration
         return true;
     }
     get HTML_CLI_HELPERS_DIR() {
@@ -100,6 +103,7 @@ class StaticConfig {
         return __awaiter(this, void 0, void 0, function* () {
             const $childProcess = this.$injector.resolve("$childProcess");
             try {
+                // Do NOT use the adb wrapper because it will end blow up with Segmentation fault because the wrapper uses this method!!!
                 const proc = yield $childProcess.spawnFromEvent("adb", ["version"], "exit", undefined, { throwError: false });
                 if (proc.stderr) {
                     return yield this.spawnPrivateAdb();
@@ -113,21 +117,37 @@ class StaticConfig {
             return "adb";
         });
     }
+    /*
+        Problem:
+        1. Adb forks itself as a server which keeps running until adb kill-server is invoked or crashes
+        2. On Windows running processes lock their image files due to memory mapping. Locked files prevent their parent directories from deletion and cannot be overwritten.
+        3. Update and uninstall scenarios are broken
+        Solution:
+        - Copy adb and associated files into a temporary directory. Let this copy of adb run persistently
+        - On Posix OSes, immediately delete the file to not take file space
+        - Tie common lib version to updates of adb, so that when we integrate a newer adb we can use it
+        - Adb is named differently on OSes and may have additional files. The code is hairy to accommodate these differences
+     */
     spawnPrivateAdb() {
         return __awaiter(this, void 0, void 0, function* () {
             const $fs = this.$injector.resolve("$fs"), $childProcess = this.$injector.resolve("$childProcess"), $hostInfo = this.$injector.resolve("$hostInfo");
+            // prepare the directory to host our copy of adb
             const defaultAdbDirPath = path.join(__dirname, "common", "resources", "platform-tools", "android", process.platform);
             const pathToPackageJson = path.join(__dirname, "..", "package.json");
             const nsCliVersion = require(pathToPackageJson).version;
             const tmpDir = path.join(os.tmpdir(), `nativescript-cli-${nsCliVersion}`);
             $fs.createDirectory(tmpDir);
+            // copy the adb and associated files
             const targetAdb = path.join(tmpDir, "adb");
+            // In case directory is missing or it's empty, copy the new adb
             if (!$fs.exists(tmpDir) || !$fs.readDirectory(tmpDir).length) {
-                shelljs.cp(path.join(defaultAdbDirPath, "*"), tmpDir);
+                shelljs.cp(path.join(defaultAdbDirPath, "*"), tmpDir); // deliberately ignore copy errors
+                // adb loses its executable bit when packed inside electron asar file. Manually fix the issue
                 if (!$hostInfo.isWindows) {
                     shelljs.chmod("+x", targetAdb);
                 }
             }
+            // let adb start its global server
             yield $childProcess.spawnFromEvent(targetAdb, ["start-server"], "exit");
             return targetAdb;
         });
@@ -135,3 +155,4 @@ class StaticConfig {
 }
 exports.StaticConfig = StaticConfig;
 yok_1.injector.register("staticConfig", StaticConfig);
+//# sourceMappingURL=config.js.map

package/lib/constants-provider.js

@@ -10,3 +10,4 @@ class Constants {
 }
 exports.Constants = Constants;
 yok_1.injector.register("constants", Constants);
+//# sourceMappingURL=constants-provider.js.map

package/lib/constants.js

@@ -146,6 +146,7 @@ exports.RESERVED_TEMPLATE_NAMES = {
     solid: "@nativescript/template-blank-solid",
     solidjs: "@nativescript/template-blank-solid",
     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",
@@ -221,6 +222,9 @@ DebugCommandErrors.NO_DEVICES_EMULATORS_FOUND_FOR_OPTIONS = "Unable to find devi
 DebugCommandErrors.UNSUPPORTED_DEVICE_OS_FOR_DEBUGGING = "Unsupported device OS for debugging";
 exports.DebugCommandErrors = DebugCommandErrors;
 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 {
@@ -237,6 +241,7 @@ AssetConstants.sizeDelimiter = "x";
 AssetConstants.defaultScale = 1;
 AssetConstants.defaultOverlayImageScale = 0.8;
 exports.AssetConstants = AssetConstants;
+// https://en.wikipedia.org/wiki/Darwin_(operating_system)#Release_history
 class MacOSVersions {
 }
 MacOSVersions.Sierra = "10.12";
@@ -246,6 +251,14 @@ MacOSVersions.Catalina = "10.15";
 exports.MacOSVersions = MacOSVersions;
 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 {
 }
 Hooks.createProject = "createProject";
@@ -286,6 +299,7 @@ AndroidAppBundleMessages.ANDROID_APP_BUNDLE_PUBLISH_DOCS_MESSAGE = "How to use A
 exports.AndroidAppBundleMessages = AndroidAppBundleMessages;
 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",
@@ -322,14 +336,52 @@ var DeviceConnectionType;
 })(DeviceConnectionType = exports.DeviceConnectionType || (exports.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 || (exports.LoggerLevel = {}));
 var LoggerConfigData;
@@ -347,3 +399,4 @@ var PackageManagers;
     PackageManagers["yarn2"] = "yarn2";
     PackageManagers["bun"] = "bun";
 })(PackageManagers = exports.PackageManagers || (exports.PackageManagers = {}));
+//# sourceMappingURL=constants.js.map

package/lib/controllers/build-controller.js

@@ -47,7 +47,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;
@@ -57,8 +57,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) {
                 yield platformData.platformProjectService.cleanProject(platformData.projectRoot);
@@ -131,3 +131,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

@@ -50,13 +50,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.`);
             }
             yield 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 (!(yield device.applicationManager.isApplicationInstalled(debugData.applicationIdentifier))) {
@@ -101,6 +101,7 @@ class DebugController extends events_1.EventEmitter {
     }
     attachDebugger(attachDebuggerData) {
         return __awaiter(this, void 0, void 0, function* () {
+            // Default values
             if (attachDebuggerData.debugOptions) {
                 attachDebuggerData.debugOptions.chrome =
                     attachDebuggerData.debugOptions.chrome === undefined
@@ -119,6 +120,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 = yield this.startDebug(debugData);
             const result = this.printDebugInformation(debugInfo, attachDebuggerData.debugOptions.forceDebuggerAttachedEvent);
             return result;
@@ -233,3 +237,4 @@ __decorate([
 ], DebugController.prototype, "enableDebuggingCoreWithoutWaitingCurrentAction", null);
 exports.DebugController = DebugController;
 yok_1.injector.register("debugController", DebugController);
+//# sourceMappingURL=debug-controller.js.map

package/lib/controllers/deploy-controller.js

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