fix(libs): 3rd party library install fixes

Close angular#160

Author: jkuri

addon/ng2/commands/install.js

@@ -7,7 +7,7 @@ var LibInstallTask  = require('../tasks/lib-install');
 
 module.exports = Command.extend({
   name: 'install',
-  description: 'Adds 3rd party libraries to existing project',
+  description: 'Adds 3rd party library to existing project',
   works: 'insideProject',
   availableOptions: [
     { name: 'skip-injection', type: Boolean, default: false, aliases: ['s'] },
@@ -16,7 +16,7 @@ module.exports = Command.extend({
   run: function (commandOptions, rawArgs) {
     if (!rawArgs.length) {
       var msg = 'The `ng install` command must take an argument with ' +
-        'at least one package name.';
+        'a package name.';
 
       return Promise.reject(new SilentError(msg));
     }

addon/ng2/tasks/lib-install.js

@@ -12,7 +12,6 @@ var _           = require('lodash');
 var glob        = require('glob');
 var appRoot     = path.resolve('./src');
 var nodeModules = path.resolve('./node_modules');
-var checkDirs   = ['components', 'providers', 'directives', 'pipes'];
 
 module.exports = Task.extend({
   command: '',
@@ -46,17 +45,27 @@ module.exports = Task.extend({
           return this.announceOKCompletion();
         }
 
-        return this.installProcedure(this.autoInjection)
+        if (this.autoInjection) {
+          var pkg = this.packages[0];
+          
+          if (existsSync(path.resolve(process.cwd(), 'src', 'app.ts'))) {
+            var entryPoint = path.resolve(process.cwd(), 'src', 'app.ts');
+            var packageData = this.parseFile(pkg);
+            this.importInBootstrap(entryPoint, pkg, packageData);
+          }
+
+          return this.announceOKCompletion();
+        }
+
+        return this.installProcedure()
           .then(function(resp) {
             return this.announceOKCompletion();
           }.bind(this));
       }.bind(this));
   },
 
-  installProcedure: function(autoInjection) {
-    var allPackages         = {};
-    allPackages.toUninstall = [];
-    allPackages.toProcess   = [];
+  installProcedure: function() {
+    var allPackages = { toUninstall: [], toProcess: [] };
     var pkgSrcPath;
     var pkgDirs;
 
@@ -76,46 +85,35 @@ module.exports = Task.extend({
 
           return npm('uninstall', allPackages.toUninstall, this.npmOptions, this.npm)
             .then(function() {
-              return this.processPackages(allPackages.toProcess, autoInjection)
+              return this.processPackages(allPackages.toProcess)
                 .then(function(resp) {
-                  resolve(resp);
+                  return resolve(resp);
                 });
             }.bind(this));
         }
         else {
-          return this.processPackages(allPackages.toProcess, autoInjection)
+          return this.processPackages(allPackages.toProcess)
             .then(function(resp) {
-              resolve(resp);
+              return resolve(resp);
             });
         }
 
       }.bind(this));
     }.bind(this));
   },
 
-  processPackages: function (packages, autoInjection) {
+  processPackages: function (packages) {
     if (!packages.length) {
       this.ui.writeLine(chalk.yellow('No package to process. Quitting.'));
       return Promise.resolve();
     } else {
       this.ui.writeLine(chalk.green('Package successfully installed.'));
-      this.ui.writeLine('');
-
-      if (autoInjection) {
-        if (existsSync(path.resolve(process.cwd(), 'src', 'app.ts'))) {
-          var entryPoint = path.resolve(process.cwd(), 'src', 'app.ts');
-          var packageData = this.parseFile(this.packages[0]);
-          this.importInBootstrap(entryPoint, this.packages[0], packageData);
-        }
-
-        return Promise.resolve(); 
-      }
 
       return this.ui.prompt({
         type: 'text',
         name: 'confirm',
-        message: 'Would you like to inject the installed packages into your app automatically? (N/y)',
-        default: function() { return 'N'; },
+        message: 'Inject the installed package into your app? (Y/n)',
+        default: function() { return 'Y'; },
         validate: function(value) {
           return /[YN]/i.test(value) ? true : 'Enter Y(es) or N(o)';
         }
@@ -125,79 +123,54 @@ module.exports = Task.extend({
           return Promise.resolve();
         }
         else {
-          return this.ui.prompt({
-            type: 'text',
-            name: 'entry',
-            message: 'What is the path to the file which bootstraps your app?',
-            default: function() {
-              if (existsSync(path.join(appRoot, 'app.ts'))) {
-                return path.join(appRoot, 'app.ts');
-              } else {
-                var possibleFiles = glob.sync(appRoot, '*.ts');
-                if (possibleFiles.length) {
-                  return possibleFiles[0];
-                } else {
-                  return '';
-                }
-              }
-            },
-            validate: function(val) {
-              return (existsSync(val)) ? true : 'File not exists.';
-            }
-          })
-          .then(function(boot) {
-            var entryPoint = boot.entry;
-            return packages.reduce(function(promise, packageName) {
-              return promise.then(function() {
-                return this.parsePackage(packageName, entryPoint, autoInjection);
-              }.bind(this));
-            }.bind(this), Promise.resolve());
-          }.bind(this));
+          return this.parsePackage(packages[0]);
         }
       }.bind(this));
-        
     }
   },
 
-  parsePackage: function (packageName, entryPoint, autoInjection) {
+  parsePackage: function (packageName, entryPoint) {
     return new Promise(function(resolve, reject) {
       var packageData;
 
-      if (autoInjection) {
-        packageData = this.parseFile(packageName);
-        this.importInBootstrap(entryPoint, packageName, packageData);
-        return resolve();
-      }
-
-      this.ui.writeLine('');
-
-      var msg = 'Would you like to customize the injection of ' + 
-      chalk.yellow(path.basename(packageName)) + '? (N/y)';
+      var msg = 'Customize the injection of ' + 
+      chalk.yellow(path.basename(packageName)) + '? (Y/n)';
 
       return this.ui.prompt({
         type: 'input',
         name: 'option',
         message: msg,
-        default: function () { return 'N'; },
+        default: function () { return 'Y'; },
         validate: function(value) {
           return /[YN]/i.test(value) ? true : 'Enter Y(es) or N(o)';
         }
       })
       .then(function(answer) {
         if (answer.option.toLowerCase().substring(0, 1) === 'y') {
-          packageData = this.parseFile(packageName);
-          this.importInBootstrap(entryPoint, packageName, packageData);
-
           return this.customImport(packageName)
             .then(function() {
-              return resolve();
-            });
+              packageData = this.parseFile(packageName);
+              if (packageData.Provider && packageData.Provider.length) {
+                return this.importInBootstrapPrompt(packageName, packageData)
+                  .then(function() {
+                    resolve();
+                  });
+              } else {
+                return resolve();
+              }
+            }.bind(this));
         }
         else {
           packageData = this.parseFile(packageName);
-          this.importInBootstrap(entryPoint, packageName, packageData);
-
-          return resolve();
+          if (packageData.Provider && packageData.Provider.length) {
+            return this.importInBootstrapPrompt(packageName, packageData)
+              .then(function() {
+                resolve();
+              });
+          }
+          else {
+            return resolve();
+          }
         }
       }.bind(this));
     }.bind(this));
@@ -292,9 +265,11 @@ module.exports = Task.extend({
                             possibleFiles[fileIndex]);
 
             this.ui.writeLine(chalk.green('Successfully injected.'));
-            this.ui.writeLine('');
 
-            this.customImport(packageName);
+            return this.customImport(packageName)
+              .then(function() {
+                resolve();
+              });
 
           }.bind(this));
         }.bind(this));
@@ -388,11 +363,54 @@ module.exports = Task.extend({
 
   },
 
+  importInBootstrapPrompt: function (packageName, packageData) {
+    return new Promise(function(resolve, reject) {
+      return this.ui.prompt({
+        type: 'text',
+        name: 'option',
+        message: 'Inject providers into bootstrap script? (Y/n)',
+        default: function () { return 'Y'; },
+        validate: function(value) {
+          return /[YN]/i.test(value) ? true : 'Enter Y(es) or N(o)';
+        }
+      })
+      .then(function(answer) {
+        if (answer.option.toLowerCase().substring(0, 1) === 'y') {
+          return this.ui.prompt({
+            type: 'text',
+            name: 'entry',
+            message: 'Path to the file which bootstraps your app?',
+            default: function() {
+              if (existsSync(path.join(appRoot, 'app.ts'))) {
+                return path.join(appRoot, 'app.ts');
+              } else {
+                var possibleFiles = glob.sync(appRoot, '*.ts');
+                if (possibleFiles.length) {
+                  return possibleFiles[0];
+                } else {
+                  return '';
+                }
+              }
+            },
+            validate: function(val) {
+              return (existsSync(val)) ? true : 'File not exists.';
+            }
+          })
+          .then(function(answer) {
+            var entryPoint = answer.entry;
+            this.importInBootstrap(entryPoint, packageName, packageData);
+            return resolve();
+          }.bind(this));
+        }
+        else {
+          return resolve();
+        }
+      }.bind(this));
+    }.bind(this));
+  },
+
   importInBootstrap: function(entryPoint, packageName, packageData) {
     var providers = packageData.Provider;
-    var directives = packageData.Directive;
-    var pipes = packageData.Pipe;
-    var all = [].concat(providers).concat(directives).concat(pipes);
     var contents = fs.readFileSync(entryPoint, 'utf8');
     var contentsArr = contents.split('\n');
     var lastIndex;
@@ -409,10 +427,10 @@ module.exports = Task.extend({
       return false;
     }
 
-    var imports = 'import {' + all.join(',') + '} from \'' + packageName + '\';';
+    var imports = 'import {' + providers.join(',') + '} from \'' + packageName + '\';';
 
     if (imports.length > 100) {
-      imports = 'import {\n' + all.join('  ,\n') + '}\n from \'' + packageName + '\';';
+      imports = 'import {\n' + providers.join('  ,\n') + '}\n from \'' + packageName + '\';';
     }
 
     contentsArr.forEach(function(line, index) {
@@ -456,7 +474,7 @@ module.exports = Task.extend({
     contentsArr.splice(lastIndex + 1, 0, imports);
 
     fs.writeFileSync(entryPoint, contentsArr.join('\n'), 'utf8');
-    this.ui.writeLine(chalk.green('Package imported in', entryPoint));
+    this.ui.writeLine(chalk.green('Providers imported in', entryPoint));
   },
 
   injectItem: function(type, name, file) {
@@ -465,6 +483,10 @@ module.exports = Task.extend({
     var replace;
     var match;
 
+    if (type === 'component') {
+      type = 'directive';
+    }
+
     contentsArr.forEach(function(line, index) {
       var regExp = new RegExp(type);
       if (regExp.test(line)) {
@@ -491,13 +513,17 @@ module.exports = Task.extend({
     var contents = fs.readFileSync(packagePath, 'utf8');
     var data = {};
 
+    data.Component = [];
     data.Directive = [];
     data.Pipe = [];
     data.Provider = [];
     data.styleUrl = [];
 
     var contentsArr = contents.split('\n');
     contentsArr.forEach(function(line, index) {
+      if (/components:/.test(line)) {
+        data.Component = this.parseData(line);
+      }
       if (/directives:/.test(line)) {
         data.Directive = this.parseData(line);
       }
@@ -530,8 +556,6 @@ module.exports = Task.extend({
   },
 
   checkIfPackageIsAuthentic: function (pkgName) {
-    return true;
-
     if (!existsSync(path.join(nodeModules, pkgName))) {
       return false;
     }
@@ -548,7 +572,6 @@ module.exports = Task.extend({
   },
 
   announceOKCompletion: function() {
-    this.ui.writeLine('');
     this.completionOKMessage = 'Done.';
     this.ui.writeLine(chalk.green(this.completionOKMessage));
     this.done();

addon/ng2/tasks/lib-uninstall.js

@@ -64,12 +64,12 @@ module.exports = Task.extend({
   },
 
   uninstallProcedure: function() {
-    this.ui.writeLine(chalk.yellow('Uninstalled packages:', this.packages.join(',')));
+    this.ui.writeLine(chalk.yellow('Uninstalled package:', this.packages.join(',')));
 
     return this.ui.prompt({
       type: 'input',
       name: 'remove',
-      message: 'Do you want to automatically remove packages from you app?',
+      message: 'Do you want to automatically remove package from you app?',
       default: function() { return 'Y'; },
       validate: function(value) {
         return /[YN]/i.test(value) ? true : 'Enter Y(es) or N(o)';
@@ -127,8 +127,6 @@ module.exports = Task.extend({
   },
 
   removePackagesFromApp: function() {
-    this.ui.writeLine('');
-
     this.removedPackagesData.forEach(function(packageData) {
       var files = glob.sync(process.cwd() +  '/src/**/*.ts');
       var contents;
@@ -205,7 +203,6 @@ module.exports = Task.extend({
   },
 
   announceOKCompletion: function() {
-    this.ui.writeLine('');
     this.completionOKMessage = 'Done.';
     this.ui.writeLine(chalk.green(this.completionOKMessage));
     this.done();

lib/broccoli/angular2-app.js

@@ -56,7 +56,8 @@ Angular2App.prototype.toTree = function() {
     });
 
     var thirdPartyJsTree = new Funnel('node_modules', {
-      include: ['ng2*/bundles/*.js']
+      include: ['ng2*/bundles/*.js'],
+      exclude: ['ng2*/bundles/*.min.js', 'ng2*/bundles/*.standalone.js'],
     });
 
     var thirdPartyJs = new Concat(thirdPartyJsTree, {