nativescript 9.0.4-dev.1 → 9.0.4-dev.3

package/lib/nativescript-cli.js

@@ -6,6 +6,7 @@ const shelljs = require("shelljs");
 shelljs.config.silent = true;
 shelljs.config.fatal = true;
 if (process.platform === "win32") {
+    // Later versions of shelljs do not process globs with \ path delimiters correctly, for windows change to /
     const realcp = shelljs.cp;
     shelljs.cp = (...args) => {
         if (args.length === 3) {
@@ -37,7 +38,7 @@ const errors_1 = require("./common/errors");
 const helpers_1 = require("./common/helpers");
 const yok_1 = require("./common/yok");
 const color_1 = require("./color");
-(0, errors_1.installUncaughtExceptionListener)(process.exit.bind(process, 120));
+(0, errors_1.installUncaughtExceptionListener)(process.exit.bind(process, 120 /* ErrorCodes.UNCAUGHT */));
 const logger = yok_1.injector.resolve("logger");
 exports.originalProcessOn = process.on.bind(process);
 process.on = (event, listener) => {
@@ -51,8 +52,10 @@ process.on = (event, listener) => {
         return (0, exports.originalProcessOn)(event, listener);
     }
 };
+/* tslint:disable:no-floating-promises */
 (async () => {
     if (process.argv.includes("--get-yargs-completions")) {
+        // This is a special case when we want to get the yargs completions as fast as possible...
         yok_1.injector.resolve("$options");
         return;
     }
@@ -69,6 +72,13 @@ process.on = (event, listener) => {
         logger.trace("Unable to load extensions. Error is: ", err);
     }
     const commandDispatcher = yok_1.injector.resolve("commandDispatcher");
+    // unused...
+    // const messages: IMessagesService = injector.resolve("$messagesService");
+    // messages.pathsToMessageJsonFiles = [
+    // 	/* Place client-specific json message file paths here */
+    // ];
     await commandDispatcher.dispatchCommand();
     yok_1.injector.dispose();
 })();
+/* tslint:enable:no-floating-promises */
+//# sourceMappingURL=nativescript-cli.js.map

package/lib/node/pbxproj-dom-xcode.js

@@ -3,3 +3,4 @@ Object.defineProperty(exports, "__esModule", { value: true });
 const pbxprojDomXcodeModule = require("pbxproj-dom/xcode");
 const yok_1 = require("../common/yok");
 yok_1.injector.register("pbxprojDomXcode", pbxprojDomXcodeModule);
+//# sourceMappingURL=pbxproj-dom-xcode.js.map

package/lib/node/xcode.js

@@ -5,3 +5,4 @@ const xcode = require("nativescript-dev-xcode");
 exports.xcode = xcode;
 const yok_1 = require("../common/yok");
 yok_1.injector.register("xcode", xcode);
+//# sourceMappingURL=xcode.js.map

package/lib/node-package-manager.js

@@ -37,8 +37,12 @@ class NodePackageManager extends base_package_manager_1.BasePackageManager {
         }
         params = params.concat(flags);
         const cwd = pathToSave;
+        // Npm creates `etc` directory in installation dir when --prefix is passed
+        // https://github.com/npm/npm/issues/11486
+        // we should delete it if it was created because of us
         const etcDirectoryLocation = (0, path_1.join)(cwd, "etc");
         const etcExistsPriorToInstallation = this.$fs.exists(etcDirectoryLocation);
+        //TODO: plamen5kov: workaround is here for a reason (remove whole file later)
         if (config.path) {
             let relativePathFromCwdToSource = "";
             if (config.frameworkPath) {
@@ -54,9 +58,13 @@ class NodePackageManager extends base_package_manager_1.BasePackageManager {
         }
         catch (err) {
             if (err.message && err.message.indexOf("EPEERINVALID") !== -1) {
+                // Not installed peer dependencies are treated by npm 2 as errors, but npm 3 treats them as warnings.
+                // We'll show them as warnings and let the user install them in case they are needed.
                 this.$logger.warn(err.message);
             }
             else {
+                // All other errors should be handled by the caller code.
+                // Revert package.json contents to preserve valid state
                 this.$fs.writeJson(packageJsonPath, jsonContentBefore);
                 throw err;
             }
@@ -78,7 +86,7 @@ class NodePackageManager extends base_package_manager_1.BasePackageManager {
         return this.$childProcess.exec(`npm search ${filter.join(" ")} ${flags}`);
     }
     async view(packageName, config) {
-        const wrappedConfig = _.extend({}, config, { json: true });
+        const wrappedConfig = _.extend({}, config, { json: true }); // always require view response as JSON
         const flags = this.getFlagsString(wrappedConfig, false);
         let viewResult;
         try {
@@ -95,6 +103,7 @@ class NodePackageManager extends base_package_manager_1.BasePackageManager {
         }
     }
     async searchNpms(keyword) {
+        // TODO: Fix the generation of url - in case it contains @ or / , the call may fail.
         const httpRequestResult = await this.$httpClient.httpRequest(`https://api.npms.io/v2/search?q=keywords:${keyword}`);
         const result = JSON.parse(httpRequestResult.body);
         return result;
@@ -131,3 +140,4 @@ __decorate([
     (0, decorators_1.cache)()
 ], NodePackageManager.prototype, "getCachePath", null);
 yok_1.injector.register("npm", NodePackageManager);
+//# sourceMappingURL=node-package-manager.js.map

package/lib/options.js

@@ -14,6 +14,7 @@ class Options {
             this.setArgv();
         }
         this.argv.bundle = "webpack";
+        // Check if the user has explicitly provide --hmr and --release options from command line
         if (this.initialArgv.release && this.initialArgv.hmr) {
             this.$errors.fail("The options --release and --hmr cannot be used simultaneously.");
         }
@@ -21,6 +22,8 @@ class Options {
             this.argv.hmr = !this.argv.release;
         }
         if (this.argv.debugBrk) {
+            // we cannot use HMR along with debug-brk because we have to restart the app
+            // on each livesync in order to stop and allow debugging on app start
             this.argv.hmr = false;
         }
         if (this.argv.justlaunch) {
@@ -38,17 +41,18 @@ class Options {
             "timeout",
             "_",
             "$0",
-        ];
+        ]; // These options shouldn't be validated
         this.globalOptions = {
-            log: { type: "string", hasSensitiveValue: false },
-            verbose: { type: "boolean", hasSensitiveValue: false },
-            version: { type: "boolean", alias: "v", hasSensitiveValue: false },
-            help: { type: "boolean", alias: "h", hasSensitiveValue: false },
-            profileDir: { type: "string", hasSensitiveValue: true },
-            analyticsClient: { type: "string", hasSensitiveValue: false },
-            path: { type: "string", alias: "p", hasSensitiveValue: true },
-            config: { type: "string", alias: "c", hasSensitiveValue: true },
-            _: { type: "string", hasSensitiveValue: false },
+            log: { type: "string" /* OptionType.String */, hasSensitiveValue: false },
+            verbose: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            version: { type: "boolean" /* OptionType.Boolean */, alias: "v", hasSensitiveValue: false },
+            help: { type: "boolean" /* OptionType.Boolean */, alias: "h", hasSensitiveValue: false },
+            profileDir: { type: "string" /* OptionType.String */, hasSensitiveValue: true },
+            analyticsClient: { type: "string" /* OptionType.String */, hasSensitiveValue: false },
+            path: { type: "string" /* OptionType.String */, alias: "p", hasSensitiveValue: true },
+            config: { type: "string" /* OptionType.String */, alias: "c", hasSensitiveValue: true },
+            // This will parse all non-hyphenated values as strings.
+            _: { type: "string" /* OptionType.String */, hasSensitiveValue: false },
         };
         this.options = _.extend({}, this.commonOptions, this.globalOptions);
         this.setArgv();
@@ -64,163 +68,164 @@ class Options {
     }
     get commonOptions() {
         return {
-            ipa: { type: "string", hasSensitiveValue: true },
-            frameworkPath: { type: "string", hasSensitiveValue: true },
-            frameworkName: { type: "string", hasSensitiveValue: false },
-            framework: { type: "string", hasSensitiveValue: false },
-            frameworkVersion: { type: "string", hasSensitiveValue: false },
-            forDevice: { type: "boolean", hasSensitiveValue: false },
+            ipa: { type: "string" /* OptionType.String */, hasSensitiveValue: true },
+            frameworkPath: { type: "string" /* OptionType.String */, hasSensitiveValue: true },
+            frameworkName: { type: "string" /* OptionType.String */, hasSensitiveValue: false },
+            framework: { type: "string" /* OptionType.String */, hasSensitiveValue: false },
+            frameworkVersion: { type: "string" /* OptionType.String */, hasSensitiveValue: false },
+            forDevice: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
             iCloudContainerEnvironment: {
-                type: "string",
+                type: "string" /* OptionType.String */,
                 hasSensitiveValue: false,
             },
-            provision: { type: "object", hasSensitiveValue: true },
+            provision: { type: "object" /* OptionType.Object */, hasSensitiveValue: true },
             client: {
-                type: "boolean",
+                type: "boolean" /* OptionType.Boolean */,
                 default: true,
                 hasSensitiveValue: false,
             },
-            env: { type: "object", hasSensitiveValue: false },
-            production: { type: "boolean", hasSensitiveValue: false },
-            debugTransport: { type: "boolean", hasSensitiveValue: false },
-            keyStorePath: { type: "string", hasSensitiveValue: true },
-            keyStorePassword: { type: "string", hasSensitiveValue: true },
-            keyStoreAlias: { type: "string", hasSensitiveValue: true },
+            env: { type: "object" /* OptionType.Object */, hasSensitiveValue: false },
+            production: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            debugTransport: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            keyStorePath: { type: "string" /* OptionType.String */, hasSensitiveValue: true },
+            keyStorePassword: { type: "string" /* OptionType.String */, hasSensitiveValue: true },
+            keyStoreAlias: { type: "string" /* OptionType.String */, hasSensitiveValue: true },
             keyStoreAliasPassword: {
-                type: "string",
+                type: "string" /* OptionType.String */,
                 hasSensitiveValue: true,
             },
-            ignoreScripts: { type: "boolean", hasSensitiveValue: false },
-            disableNpmInstall: { type: "boolean", hasSensitiveValue: false },
-            compileSdk: { type: "number", hasSensitiveValue: false },
-            port: { type: "number", hasSensitiveValue: false },
-            copyTo: { type: "string", hasSensitiveValue: true },
-            js: { type: "boolean", hasSensitiveValue: false },
-            javascript: { type: "boolean", hasSensitiveValue: false },
-            ng: { type: "boolean", hasSensitiveValue: false },
-            angular: { type: "boolean", hasSensitiveValue: false },
-            react: { type: "boolean", hasSensitiveValue: false },
-            reactjs: { type: "boolean", hasSensitiveValue: false },
-            vue: { type: "boolean", hasSensitiveValue: false },
-            vuejs: { type: "boolean", hasSensitiveValue: false },
-            svelte: { type: "boolean", hasSensitiveValue: false },
-            vision: { type: "boolean", hasSensitiveValue: false },
-            "vision-ng": { type: "boolean", hasSensitiveValue: false },
-            "vision-react": { type: "boolean", hasSensitiveValue: false },
-            "vision-solid": { type: "boolean", hasSensitiveValue: false },
-            "vision-svelte": { type: "boolean", hasSensitiveValue: false },
-            "vision-vue": { type: "boolean", hasSensitiveValue: false },
-            tsc: { type: "boolean", hasSensitiveValue: false },
-            ts: { type: "boolean", hasSensitiveValue: false },
-            typescript: { type: "boolean", hasSensitiveValue: false },
-            yarn: { type: "boolean", hasSensitiveValue: false },
-            yarn2: { type: "boolean", hasSensitiveValue: false },
-            pnpm: { type: "boolean", hasSensitiveValue: false },
-            androidTypings: { type: "boolean", hasSensitiveValue: false },
-            bundle: { type: "string", hasSensitiveValue: false },
-            all: { type: "boolean", hasSensitiveValue: false },
-            teamId: { type: "object", hasSensitiveValue: true },
-            chrome: { type: "boolean", hasSensitiveValue: false },
-            inspector: { type: "boolean", hasSensitiveValue: false },
-            clean: { type: "boolean", hasSensitiveValue: false },
+            ignoreScripts: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            disableNpmInstall: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            compileSdk: { type: "number" /* OptionType.Number */, hasSensitiveValue: false },
+            port: { type: "number" /* OptionType.Number */, hasSensitiveValue: false },
+            copyTo: { type: "string" /* OptionType.String */, hasSensitiveValue: true },
+            js: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            javascript: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            ng: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            angular: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            react: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            reactjs: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            vue: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            vuejs: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            svelte: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            vision: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            "vision-ng": { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            "vision-react": { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            "vision-solid": { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            "vision-svelte": { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            "vision-vue": { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            tsc: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            ts: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            typescript: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            yarn: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            yarn2: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            pnpm: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            androidTypings: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            bundle: { type: "string" /* OptionType.String */, hasSensitiveValue: false },
+            all: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            teamId: { type: "object" /* OptionType.Object */, hasSensitiveValue: true },
+            chrome: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            inspector: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            clean: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
             watch: {
-                type: "boolean",
+                type: "boolean" /* OptionType.Boolean */,
                 default: true,
                 hasSensitiveValue: false,
             },
-            background: { type: "string", hasSensitiveValue: false },
-            username: { type: "string", hasSensitiveValue: true },
-            pluginName: { type: "string", hasSensitiveValue: false },
+            background: { type: "string" /* OptionType.String */, hasSensitiveValue: false },
+            username: { type: "string" /* OptionType.String */, hasSensitiveValue: true },
+            pluginName: { type: "string" /* OptionType.String */, hasSensitiveValue: false },
             includeTypeScriptDemo: {
-                type: "string",
+                type: "string" /* OptionType.String */,
                 hasSensitiveValue: false,
             },
-            includeAngularDemo: { type: "string", hasSensitiveValue: false },
+            includeAngularDemo: { type: "string" /* OptionType.String */, hasSensitiveValue: false },
             hmr: {
-                type: "boolean",
+                type: "boolean" /* OptionType.Boolean */,
                 hasSensitiveValue: false,
                 default: true,
             },
             collection: {
-                type: "string",
+                type: "string" /* OptionType.String */,
                 alias: "c",
                 hasSensitiveValue: false,
             },
-            json: { type: "boolean", hasSensitiveValue: false },
-            avd: { type: "string", hasSensitiveValue: true },
-            config: { type: "array", hasSensitiveValue: false },
+            json: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            avd: { type: "string" /* OptionType.String */, hasSensitiveValue: true },
+            // check not used
+            config: { type: "array" /* OptionType.Array */, hasSensitiveValue: false },
             insecure: {
-                type: "boolean",
+                type: "boolean" /* OptionType.Boolean */,
                 alias: "k",
                 hasSensitiveValue: false,
             },
-            debug: { type: "boolean", alias: "d", hasSensitiveValue: false },
-            timeout: { type: "string", hasSensitiveValue: false },
-            device: { type: "string", hasSensitiveValue: true },
-            availableDevices: { type: "boolean", hasSensitiveValue: false },
-            appid: { type: "string", hasSensitiveValue: true },
-            geny: { type: "string", hasSensitiveValue: true },
-            debugBrk: { type: "boolean", hasSensitiveValue: false },
-            debugPort: { type: "number", hasSensitiveValue: false },
-            start: { type: "boolean", hasSensitiveValue: false },
-            stop: { type: "boolean", hasSensitiveValue: false },
-            ddi: { type: "string", hasSensitiveValue: true },
-            justlaunch: { type: "boolean", hasSensitiveValue: false },
-            file: { type: "string", hasSensitiveValue: true },
-            force: { type: "boolean", alias: "f", hasSensitiveValue: false },
-            emulator: { type: "boolean", hasSensitiveValue: false },
-            simulator: { type: "boolean", hasSensitiveValue: false },
-            sdk: { type: "string", hasSensitiveValue: false },
-            template: { type: "string", hasSensitiveValue: true },
-            certificate: { type: "string", hasSensitiveValue: true },
-            certificatePassword: { type: "string", hasSensitiveValue: true },
+            debug: { type: "boolean" /* OptionType.Boolean */, alias: "d", hasSensitiveValue: false },
+            timeout: { type: "string" /* OptionType.String */, hasSensitiveValue: false },
+            device: { type: "string" /* OptionType.String */, hasSensitiveValue: true },
+            availableDevices: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            appid: { type: "string" /* OptionType.String */, hasSensitiveValue: true },
+            geny: { type: "string" /* OptionType.String */, hasSensitiveValue: true },
+            debugBrk: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            debugPort: { type: "number" /* OptionType.Number */, hasSensitiveValue: false },
+            start: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            stop: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            ddi: { type: "string" /* OptionType.String */, hasSensitiveValue: true }, // the path to developer  disk image
+            justlaunch: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            file: { type: "string" /* OptionType.String */, hasSensitiveValue: true },
+            force: { type: "boolean" /* OptionType.Boolean */, alias: "f", hasSensitiveValue: false },
+            emulator: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            simulator: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            sdk: { type: "string" /* OptionType.String */, hasSensitiveValue: false },
+            template: { type: "string" /* OptionType.String */, hasSensitiveValue: true },
+            certificate: { type: "string" /* OptionType.String */, hasSensitiveValue: true },
+            certificatePassword: { type: "string" /* OptionType.String */, hasSensitiveValue: true },
             release: {
-                type: "boolean",
+                type: "boolean" /* OptionType.Boolean */,
                 alias: "r",
                 hasSensitiveValue: false,
             },
-            markingMode: { type: "boolean", hasSensitiveValue: false },
-            var: { type: "object", hasSensitiveValue: true },
-            default: { type: "boolean", hasSensitiveValue: false },
-            count: { type: "number", hasSensitiveValue: false },
-            analyticsLogFile: { type: "string", hasSensitiveValue: true },
-            disableAnalytics: { type: "boolean", hasSensitiveValue: false },
-            cleanupLogFile: { type: "string", hasSensitiveValue: true },
+            markingMode: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            var: { type: "object" /* OptionType.Object */, hasSensitiveValue: true },
+            default: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            count: { type: "number" /* OptionType.Number */, hasSensitiveValue: false },
+            analyticsLogFile: { type: "string" /* OptionType.String */, hasSensitiveValue: true },
+            disableAnalytics: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            cleanupLogFile: { type: "string" /* OptionType.String */, hasSensitiveValue: true },
             hooks: {
-                type: "boolean",
+                type: "boolean" /* OptionType.Boolean */,
                 default: true,
                 hasSensitiveValue: false,
             },
             link: {
-                type: "boolean",
+                type: "boolean" /* OptionType.Boolean */,
                 default: false,
                 hasSensitiveValue: false,
             },
-            gradlePath: { type: "string", hasSensitiveValue: false },
-            gradleArgs: { type: "string", hasSensitiveValue: false },
-            hostProjectPath: { type: "string", hasSensitiveValue: false },
+            gradlePath: { type: "string" /* OptionType.String */, hasSensitiveValue: false },
+            gradleArgs: { type: "string" /* OptionType.String */, hasSensitiveValue: false },
+            hostProjectPath: { type: "string" /* OptionType.String */, hasSensitiveValue: false },
             hostProjectModuleName: {
-                type: "string",
+                type: "string" /* OptionType.String */,
                 hasSensitiveValue: false,
                 default: constants_1.APP_FOLDER_NAME,
             },
-            aab: { type: "boolean", hasSensitiveValue: false },
-            performance: { type: "object", hasSensitiveValue: true },
+            aab: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            performance: { type: "object" /* OptionType.Object */, hasSensitiveValue: true },
             appleApplicationSpecificPassword: {
-                type: "string",
+                type: "string" /* OptionType.String */,
                 hasSensitiveValue: true,
             },
-            appleSessionBase64: { type: "string", hasSensitiveValue: true },
-            jar: { type: "string", hasSensitiveValue: true },
-            aar: { type: "string", hasSensitiveValue: true },
-            filter: { type: "string", hasSensitiveValue: true },
+            appleSessionBase64: { type: "string" /* OptionType.String */, hasSensitiveValue: true },
+            jar: { type: "string" /* OptionType.String */, hasSensitiveValue: true },
+            aar: { type: "string" /* OptionType.String */, hasSensitiveValue: true },
+            filter: { type: "string" /* OptionType.String */, hasSensitiveValue: true },
             git: {
-                type: "boolean",
+                type: "boolean" /* OptionType.Boolean */,
                 hasSensitiveValue: false,
                 default: true,
             },
-            dryRun: { type: "boolean", hasSensitiveValue: false },
-            uniqueBundle: { type: "boolean", hasSensitiveValue: false },
+            dryRun: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
+            uniqueBundle: { type: "boolean" /* OptionType.Boolean */, hasSensitiveValue: false },
         };
     }
     get optionNames() {
@@ -238,6 +243,7 @@ class Options {
             parsed[optionName] = this.getOptionValue(optionName);
         }
         _.each(parsed, (value, originalOptionName) => {
+            // when this.options are passed to yargs, it returns all of them and the ones that are not part of process.argv are set to undefined.
             if (value === undefined) {
                 return;
             }
@@ -247,14 +253,14 @@ class Options {
                     this.$errors.failWithHelp(`The option '${originalOptionName}' is not supported.`);
                 }
                 const optionType = this.getOptionType(optionName), optionValue = parsed[optionName];
-                if (_.isArray(optionValue) && optionType !== "array") {
+                if (_.isArray(optionValue) && optionType !== "array" /* OptionType.Array */) {
                     this.$errors.failWithHelp("The '%s' option requires a single value.", originalOptionName);
                 }
-                else if (optionType === "string" &&
+                else if (optionType === "string" /* OptionType.String */ &&
                     helpers.isNullOrWhitespace(optionValue)) {
                     this.$errors.failWithHelp("The option '%s' requires non-empty value.", originalOptionName);
                 }
-                else if (optionType === "array" &&
+                else if (optionType === "array" /* OptionType.Array */ &&
                     optionValue.length === 0) {
                     this.$errors.failWithHelp(`The option '${originalOptionName}' requires one or more values, separated by a space.`);
                 }
@@ -282,9 +288,16 @@ class Options {
         }
         return true;
     }
+    // If you pass value with dash, yargs adds it to yargs.argv in two ways:
+    // with dash and without dash, replacing first symbol after it with its toUpper equivalent
+    // ex, "$ <cli name> emulate android --profile-dir" will add profile-dir to yargs.argv as profile-dir and profileDir
+    // IMPORTANT: In your code, it is better to use the value without dashes (profileDir in the example).
+    // This way your code will work in case "$ <cli name> emulate android --profile-dir" or "$ <cli name> emulate android --profileDir" is used by user.
     getNonDashedOptionName(optionName) {
         const matchUpperCaseLetters = optionName.match(Options.NONDASHED_OPTION_REGEX);
         if (matchUpperCaseLetters) {
+            // get here if option with upperCase letter is specified, for example profileDir
+            // check if in knownOptions we have its kebabCase presentation
             const secondaryOptionName = matchUpperCaseLetters[1] +
                 matchUpperCaseLetters[2].toUpperCase() +
                 matchUpperCaseLetters[3] || "";
@@ -313,15 +326,18 @@ class Options {
             const args = argv._.slice(1);
             const commands = yok_1.injector
                 .getRegisteredCommandsNames(false)
-                .filter((c) => c != "/?");
+                .filter((c) => c != "/?"); // remove the /? command, looks weird... :D
             const currentDepth = args.length > 0 ? args.length - 1 : 0;
             const current = (_a = current_ !== null && current_ !== void 0 ? current_ : args[currentDepth]) !== null && _a !== void 0 ? _a : "";
+            // split all commands into their components ie. "device|list" => ["device", "list"]
             const matchGroups = commands.map((c) => c.split("|"));
+            // find all commands that match the current depth and all the previous args
             const possibleMatches = matchGroups.filter((group) => {
                 return group.slice(0, currentDepth).every((g, i) => {
                     return g === args[i] || args[i].at(0) === "-";
                 });
             });
+            // filter out duplicates
             const completions = [
                 ...new Set(possibleMatches
                     .map((match) => {
@@ -329,21 +345,28 @@ class Options {
                 })
                     .filter(Boolean)),
             ];
+            // autocomplete long -- options
             if (current.startsWith("--")) {
                 return this.optionNames.filter((o) => o !== "_").map((o) => `--${o}`);
             }
+            // autocomple short - options
             if (current.startsWith("-")) {
                 return this.shorthands.map((o) => `-${o}`);
             }
+            // autocomplete matched completions
             return completions;
         });
         this.initialArgv = parsed.argv;
         this.argv = parsed.options(opts).argv;
+        // For backwards compatibility
+        // Previously profileDir had a default option and calling `this.$options.profileDir` always returned valid result.
+        // Now the profileDir should be used from $settingsService, but ensure the `this.$options.profileDir` returns the same value.
         this.$settingsService.setSettings({
             profileDir: this.argv.profileDir,
         });
         this.argv.profileDir = this.argv["profile-dir"] =
             this.$settingsService.getProfileDir();
+        // if justlaunch is set, it takes precedence over the --watch flag and the default true value
         if (this.argv.justlaunch) {
             this.argv.watch = false;
         }
@@ -359,6 +382,7 @@ class Options {
         if (this.argv.javascript) {
             this.argv.js = true;
         }
+        // alias --simulator to --emulator
         if (this.argv.simulator) {
             this.argv.emulator = this.argv.simulator;
         }
@@ -383,3 +407,4 @@ exports.Options = Options;
 Options.DASHED_OPTION_REGEX = /(.+?)([A-Z])(.*)/;
 Options.NONDASHED_OPTION_REGEX = /(.+?)[-]([a-zA-Z])(.*)/;
 yok_1.injector.register("options", Options);
+//# sourceMappingURL=options.js.map

package/lib/package-installation-manager.js

@@ -24,6 +24,7 @@ class PackageInstallationManager {
     async getLatestCompatibleVersion(packageName, referenceVersion) {
         referenceVersion = referenceVersion || this.$staticConfig.version;
         const isPreReleaseVersion = semver.prerelease(referenceVersion) !== null;
+        // if the user has some v.v.v-prerelease-xx.xx pre-release version, include pre-release versions in the search query.
         const compatibleVersionRange = isPreReleaseVersion
             ? `~${referenceVersion}`
             : `~${semver.major(referenceVersion)}.${semver.minor(referenceVersion)}.0`;
@@ -105,13 +106,14 @@ class PackageInstallationManager {
     }
     async getInspectorFromCache(inspectorNpmPackageName, projectDir) {
         const inspectorPath = path.join(projectDir, constants.NODE_MODULES_FOLDER_NAME, inspectorNpmPackageName);
+        // local installation takes precedence over cache
         if (this.inspectorAlreadyInstalled(inspectorPath)) {
             return inspectorPath;
         }
         const cachePath = this.getInspectorCachePath();
         this.prepareCacheDir(cachePath);
         const pathToPackageInCache = path.join(cachePath, constants.NODE_MODULES_FOLDER_NAME, inspectorNpmPackageName);
-        const iOSFrameworkNSValue = this.$projectDataService.getRuntimePackage(projectDir, "ios");
+        const iOSFrameworkNSValue = this.$projectDataService.getRuntimePackage(projectDir, "ios" /* constants.PlatformTypes.ios */);
         const version = await this.getLatestCompatibleVersion(inspectorNpmPackageName, iOSFrameworkNSValue.version);
         let shouldInstall = !this.$fs.exists(pathToPackageInCache);
         if (!shouldInstall) {
@@ -172,6 +174,10 @@ class PackageInstallationManager {
         }
         return await this.$packageManager.install(packageName, pathToSave, npmOptions);
     }
+    /**
+     * This function must not be used with packageName being a URL or local file,
+     * because npm view doens't work with those
+     */
     async getVersion(packageName, version) {
         var _a;
         let data = await this.$packageManager.view(packageName, {
@@ -184,3 +190,4 @@ class PackageInstallationManager {
 }
 exports.PackageInstallationManager = PackageInstallationManager;
 yok_1.injector.register("packageInstallationManager", PackageInstallationManager);
+//# sourceMappingURL=package-installation-manager.js.map

package/lib/package-manager.js

@@ -92,6 +92,7 @@ class PackageManager {
             }
         }
         catch (err) {
+            // ignore error, but log info
             this.$logger.trace("Tried to read cli.packageManager from project config and failed. Error is: ", err);
         }
         if (pm === constants_1.PackageManagers.yarn || this.$options.yarn) {
@@ -159,3 +160,4 @@ __decorate([
     (0, decorators_1.invokeInit)()
 ], PackageManager.prototype, "getCachePath", null);
 yok_1.injector.register("packageManager", PackageManager);
+//# sourceMappingURL=package-manager.js.map

package/lib/platform-command-param.js

@@ -16,3 +16,4 @@ class PlatformCommandParameter {
 }
 exports.PlatformCommandParameter = PlatformCommandParameter;
 yok_1.injector.register("platformCommandParameter", PlatformCommandParameter);
+//# sourceMappingURL=platform-command-param.js.map

package/lib/pnpm-package-manager.js

@@ -24,13 +24,14 @@ class PnpmPackageManager extends base_package_manager_1.BasePackageManager {
         if (config.disableNpmInstall) {
             return;
         }
-        delete config.dev;
+        delete config.dev; // temporary fix for unsupported yarn flag
         if (config.ignoreScripts) {
             config["ignore-scripts"] = true;
         }
         const packageJsonPath = path.join(pathToSave, "package.json");
         const jsonContentBefore = this.$fs.readJson(packageJsonPath);
         const flags = this.getFlagsString(config, true);
+        // With pnpm we need to install as "flat" or some imports wont be found
         let params = ["i", "--shamefully-hoist"];
         const isInstallingAllDependencies = packageName === pathToSave;
         if (!isInstallingAllDependencies) {
@@ -48,6 +49,7 @@ class PnpmPackageManager extends base_package_manager_1.BasePackageManager {
         }
     }
     uninstall(packageName, config, cwd) {
+        // pnpm does not want save option in remove. It saves it by default
         delete config["save"];
         const flags = this.getFlagsString(config, false);
         return this.$childProcess.exec(`pnpm remove ${packageName} ${flags}`, {
@@ -115,3 +117,4 @@ __decorate([
     (0, decorators_1.exported)("pnpm")
 ], PnpmPackageManager.prototype, "getCachePath", null);
 yok_1.injector.register("pnpm", PnpmPackageManager);
+//# sourceMappingURL=pnpm-package-manager.js.map