Merge pull request #1116 from NativeScript/sd/tsc

`tns install <module>` command should do npm install "nativescript-dev-<module>"

Author: tailsu

Gruntfile.js

@@ -57,7 +57,7 @@ module.exports = function(grunt) {
 				}
 			},
 		},
-		
+
 		tslint: {
             build: {
                 files: {
@@ -68,7 +68,7 @@ module.exports = function(grunt) {
                 }
             }
         },
-		
+
 		watch: {
 			devall: {
 				files: ["lib/**/*.ts", 'test/**/*.ts', "!lib/common/node_modules/**/*.ts"],
@@ -80,6 +80,16 @@ module.exports = function(grunt) {
 					atBegin: true,
 					interrupt: true
 				}
+			},
+			ts: {
+				files: ["lib/**/*.ts", 'test/**/*.ts', "!lib/common/node_modules/**/*.ts"],
+				tasks: [
+					'ts:devall'
+				],
+				options: {
+					atBegin: true,
+					interrupt: true
+				}
 			}
 		},
 
@@ -106,7 +116,7 @@ module.exports = function(grunt) {
 			npm_test: {
 				command: "npm test"
 			}
-	
+
 		},
 
 		copy: {

docs/man_pages/project/configuration/install.md

@@ -3,18 +3,24 @@ install
 
 Usage | Synopsis
 ---|---
-General | `$ tns install [--path]`
+Install all dependencies | `$ tns install [--path]`
+Install a selected npm module | `$ tns install <Module>`
+Enable TypeScript support | `$ tns install typescript`
 
-Installs all platforms and dependencies described in the `package.json` file in the current directory.
+Installs all dependencies described in a valid `package.json` or installs a selected NativeScript development module as a dev dependency.
 
-<% if(isHtml) { %> 
-The `package.json` file must be a valid `package.json` describing the configuration of a NativeScript project. If missing or corrupted, you can recreate the file by running `$ tns init` in the directory of a NativeScript project. 
+<% if(isHtml) { %>
+The `package.json` file must be a valid `package.json` describing the configuration of a NativeScript project. If missing or corrupted, you can recreate the file by running `$ tns init` in the directory of a NativeScript project.
 <% } %>
 
 ### Options
 * `--path` - Specifies the directory which contains the `package.json` file, if different from the current directory.
 
-<% if(isHtml) { %> 
+### Arguments
+* `<Module>` - Specifies a NativeScript development module by path to a local directory containing a valid npm module or by name in the npm registry.<% if(isHtml) { %> When a `<Module>` is specified, this command adds the module as a dev dependency in your `package.json`.
+
+> **TIP:** When installing a module from the npm registry, you can specify it by full name or suffix. NativeScript development modules are published in the npm registry with the following name format: `nativescript-dev-<suffix>`.
+
 ### Related Commands
 
 Command | Description
@@ -24,4 +30,4 @@ Command | Description
 [platform update](platform-update.html) | Updates the NativeScript runtime for the specified platform.
 [platform](platform.html) | Lists all platforms that the project currently targets.
 [prepare](prepare.html) | Copies common and relevant platform-specific content from the app directory to the subdirectory for the selected target platform in the platforms directory.
-<% } %>
+<% } %>

lib/commands/install.ts

@@ -1,19 +1,28 @@
 ///<reference path="../.d.ts"/>
 "use strict";
 
+import * as path from 'path';
+
 export class InstallCommand implements ICommand {
 	constructor(private $platformsData: IPlatformsData,
 		private $platformService: IPlatformService,
 		private $projectData: IProjectData,
 		private $projectDataService: IProjectDataService,
 		private $pluginsService: IPluginsService,
-		private $logger: ILogger) { }
+		private $logger: ILogger,
+		private $fs: IFileSystem,
+		private $stringParameter: ICommandParameter,
+		private $npm: INodePackageManager) { }
 
 	public enableHooks = false;
 
-	public allowedParameters: ICommandParameter[] = [];
+	public allowedParameters: ICommandParameter[] = [this.$stringParameter];
 
 	public execute(args: string[]): IFuture<void> {
+		return args[0] ? this.installModule(args[0]) : this.installProjectDependencies();
+	}
+
+	private installProjectDependencies(): IFuture<void> {
 		return (() => {
 			let error: string = "";
 
@@ -23,19 +32,33 @@ export class InstallCommand implements ICommand {
 			_.each(this.$platformsData.platformsNames, platform => {
 				let platformData = this.$platformsData.getPlatformData(platform);
 				let frameworkPackageData = this.$projectDataService.getValue(platformData.frameworkPackageName).wait();
-				if(frameworkPackageData && frameworkPackageData.version) {
+				if (frameworkPackageData && frameworkPackageData.version) {
 					try {
 						this.$platformService.addPlatforms([`${platform}@${frameworkPackageData.version}`]).wait();
-					} catch(err) {
+					} catch (err) {
 						error += err;
 					}
 				}
 			});
 
-			if(error) {
+			if (error) {
 				this.$logger.error(error);
 			}
+		}).future<void>()();
+	}
+
+	private installModule(moduleName: string): IFuture<void> {
+		return (() => {
+			let projectDir = this.$projectData.projectDir;
+			process.env['TNS_PROJECT_DIR'] = projectDir;
+			process.env['TNS_HOOKS_DIR'] = path.join(projectDir, 'hooks');
+
+			let devPrefix = 'nativescript-dev-';
+			if (!this.$fs.exists(moduleName).wait() && moduleName.indexOf(devPrefix) !== 0) {
+				moduleName = devPrefix + moduleName;
+			}
 
+			this.$npm.install(moduleName, projectDir, { 'save-dev': true }).wait();
 		}).future<void>()();
 	}
 }

lib/common

@@ -37,6 +37,10 @@ export class StaticConfig extends staticConfigBaseLibPath.StaticConfigBase imple
 		this.RESOURCE_DIR_PATH = path.join(this.RESOURCE_DIR_PATH, "../../resources");
 	}
 
+	public get disableHooks() {
+		return true;
+	}
+
 	public get SYS_REQUIREMENTS_LINK(): string {
 		let linkToSysRequirements: string;
 		switch(process.platform) {

lib/config.ts

@@ -19,6 +19,7 @@ export class PlatformService implements IPlatformService {
 		private $projectData: IProjectData,
 		private $projectDataService: IProjectDataService,
 		private $prompter: IPrompter,
+		private $hooksService: IHooksService,
 		private $commandsService: ICommandsService,
 		private $options: IOptions,
 		private $broccoliBuilder: IBroccoliBuilder,
@@ -147,6 +148,7 @@ export class PlatformService implements IPlatformService {
 		}).future<string[]>()();
 	}
 
+	@helpers.hook('prepare')
 	public preparePlatform(platform: string): IFuture<void> {
 		return (() => {
 			this.validatePlatform(platform);

lib/services/platform-service.ts

@@ -120,6 +120,7 @@ function createTestInjector() {
 		}
 	});
 	testInjector.register("projectFilesManager", ProjectFilesManagerLib.ProjectFilesManager);
+	testInjector.register("hooksService", stubs.HooksServiceStub);
 
 	return testInjector;
 }

test/platform-commands.ts

@@ -52,6 +52,7 @@ function createTestInjector() {
 		}
 	});
 	testInjector.register("projectFilesManager", ProjectFilesManagerLib.ProjectFilesManager);
+	testInjector.register("hooksService", stubs.HooksServiceStub);
 
 	return testInjector;
 }

test/platform-service.ts

@@ -383,13 +383,11 @@ export class ProjectTemplatesService implements IProjectTemplatesService {
 }
 
 export class HooksServiceStub implements IHooksService {
-	initialize(commandName: string): void {
-	}
-	executeBeforeHooks(): IFuture<void> {
-		return (() => { }).future<void>()();
+	executeBeforeHooks(commandName: string): IFuture<void> {
+		return Future.fromResult();
 	}
-	executeAfterHooks(): IFuture<void> {
-		return (() => { }).future<void>()();
+	executeAfterHooks(commandName: string): IFuture<void> {
+		return Future.fromResult();
 	}
 }