feat: introduce initializeService

Introduce initializeService which can be used to setup some specifics of CLI (currently the logger) and to print all required warnings. At the moment, whenever we want to introduce new warnings, we need to introduce new methods that we call from CLI and when it is used as a library.
With the new solution all such warnings can be placed in the initialize method and it will automatically show those warnings.

Author: rosen-vladimirov

lib/bootstrap.ts

@@ -199,3 +199,5 @@ $injector.requirePublic("cleanupService", "./services/cleanup-service");
 $injector.require("applePortalSessionService", "./services/apple-portal/apple-portal-session-service");
 $injector.require("applePortalCookieService", "./services/apple-portal/apple-portal-cookie-service");
 $injector.require("applePortalApplicationService", "./services/apple-portal/apple-portal-application-service");
+
+$injector.requirePublicClass("initializeService", "./services/initialize-service");

lib/definitions/initialize-service.d.ts

@@ -0,0 +1,3 @@
+interface IInitializeService {
+	initialize(initOpts?: { loggerOptions?: ILoggerOptions }): Promise<void>;
+}

lib/nativescript-cli.ts

@@ -1,6 +1,5 @@
 require("./bootstrap");
 
-import { EOL } from "os";
 import * as shelljs from "shelljs";
 shelljs.config.silent = true;
 shelljs.config.fatal = true;
@@ -25,24 +24,18 @@ process.on = (event: string, listener: any): any => {
 	const err: IErrors = $injector.resolve("$errors");
 	err.printCallStack = config.DEBUG;
 
+	const $options = $injector.resolve<IOptions>("options");
+
+	const $initializeService = $injector.resolve<IInitializeService>("initializeService");
+	await $initializeService.initialize({ loggerOptions: { level: <any>$options.log } });
+
 	const extensibilityService: IExtensibilityService = $injector.resolve("extensibilityService");
 	try {
 		await settlePromises<IExtensionData>(extensibilityService.loadExtensions());
 	} catch (err) {
 		logger.trace("Unable to load extensions. Error is: ", err);
 	}
 
-	const $sysInfo = $injector.resolve<ISysInfo>("sysInfo");
-	const macOSWarning = await $sysInfo.getMacOSWarningMessage();
-	if (macOSWarning) {
-		const message = `${EOL}${macOSWarning.message}${EOL}`;
-		if (macOSWarning.severity === SystemWarningsSeverity.high) {
-			logger.printOnStderr(message.red.bold);
-		} else {
-			logger.warn(message);
-		}
-	}
-
 	const commandDispatcher: ICommandDispatcher = $injector.resolve("commandDispatcher");
 
 	const messages: IMessagesService = $injector.resolve("$messagesService");

lib/services/initialize-service.ts

@@ -0,0 +1,30 @@
+import { EOL } from "os";
+
+export class InitializeService implements IInitializeService {
+	// NOTE: Do not inject anything here, use $injector.resolve in the code
+	// Injecting something may lead to logger initialization, but we want to initialize it from here.
+	constructor(private $injector: IInjector) { }
+
+	public async initialize(initOpts?: { loggerOptions?: ILoggerOptions }): Promise<void> {
+		initOpts = initOpts || {};
+		const $logger = this.$injector.resolve<ILogger>("logger");
+		$logger.initialize(initOpts.loggerOptions);
+
+		await this.showWarnings($logger);
+	}
+
+	private async showWarnings($logger: ILogger): Promise<void> {
+		const $sysInfo = $injector.resolve<ISysInfo>("sysInfo");
+		const macOSWarning = await $sysInfo.getMacOSWarningMessage();
+		if (macOSWarning) {
+			const message = `${EOL}${macOSWarning.message}${EOL}`;
+			if (macOSWarning.severity === SystemWarningsSeverity.high) {
+				$logger.printOnStderr(message.red.bold);
+			} else {
+				$logger.warn(message);
+			}
+		}
+	}
+}
+
+$injector.register("initializeService", InitializeService);

test/nativescript-cli-lib.ts

@@ -67,6 +67,9 @@ describe("nativescript-cli-lib", () => {
 			"initialize",
 			"getLevel",
 			"info"
+		],
+		initializeService: [
+			"initialize"
 		]
 	};