Switched generator commands to a more commander-compliant syntax

Author: begedin

lib/cli/build.js

@@ -3,15 +3,14 @@
 var program = require('commander');
 
 var dasherize = require('../utils/string').dasherize;
-var Promise = require('../ext/promise');
 var ui = require('../ui');
 
 program
   .usage('<addonName>')
   .arguments('<addonName>')
   .action(function(addonName) {
     if (addonName) {
-      require('../tasks/build')({ addonName: addonName }).catch(ui.error);
+      require('../tasks/build')({ addonName: dasherize(addonName) }).catch(ui.error);
     }
   })
   .parse(process.argv);

lib/cli/component.js

@@ -3,34 +3,19 @@
 var program = require('commander');
 
 var dasherize = require('../utils/string').dasherize;
-var Promise = require('../ext/promise');
 var ui = require('../ui');
 
-function validateOptions(options) {
-  return new Promise(function(resolve, reject) {
-    if (!options.addonName) {
-      reject('You must provide a name of the component you wisth to generate.');
-    } else {
-      resolve(options);
+program
+  .arguments('<addonName>')
+  .action(function(addonName) {
+    if (addonName) {
+      require('../tasks/component')({ addonName: dasherize(addonName) }).catch(ui.error);
     }
   });
-}
-
-function run(options) {
-  return require('../tasks/component')(options);
-}
 
 program
-  .arguments('<addonName>')
-  .action(function(addonName) {
-    program.addonName = addonName;
-  })
   .parse(process.argv);
 
-var options = {
-  addonName: program.addonName ? dasherize(program.addonName) : null
-};
-
-return validateOptions(options)
-  .then(run)
-  .catch(ui.error);
+if (process.argv.length < 3) {
+  program.outputHelp();
+}

lib/cli/helper.js

@@ -3,34 +3,19 @@
 var program = require('commander');
 
 var dasherize = require('../utils/string').dasherize;
-var Promise = require('../ext/promise');
 var ui = require('../ui');
 
-function validateOptions(options) {
-  return new Promise(function(resolve, reject) {
-    if (!options.addonName) {
-      reject('You must provide a name of the helper you wisth to generate.');
-    } else {
-      resolve(options);
+program
+  .arguments('<addonName>')
+  .action(function(addonName) {
+    if (addonName) {
+      require('../tasks/helper')({ addonName: dasherize(addonName) }).catch(ui.error);
     }
   });
-}
-
-function run(options) {
-  return require('../tasks/helper')(options);
-}
 
 program
-  .arguments('<addonName>')
-  .action(function(addonName) {
-    program.addonName = addonName;
-  })
   .parse(process.argv);
 
-var options = {
-  addonName: program.addonName ? dasherize(program.addonName) : null
-};
-
-return validateOptions(options)
-  .then(run)
-  .catch(ui.error);
+if (process.argv.length < 3) {
+  program.outputHelp();
+}

lib/cli/library.js

@@ -3,34 +3,19 @@
 var program = require('commander');
 
 var dasherize = require('../utils/string').dasherize;
-var Promise = require('../ext/promise');
 var ui = require('../ui');
 
-function validateOptions(options) {
-  return new Promise(function(resolve, reject) {
-    if (!options.addonName) {
-      reject('You must provide a name of the library you wisth to generate.');
-    } else {
-      resolve(options);
+program
+  .arguments('<addonName>')
+  .action(function(addonName) {
+    if (addonName) {
+      require('../tasks/library')({ addonName: dasherize(addonName) }).catch(ui.error);
     }
   });
-}
-
-function run(options) {
-  return require('../tasks/library')(options);
-}
 
 program
-  .arguments('<addonName>')
-  .action(function(addonName) {
-    program.addonName = addonName;
-  })
   .parse(process.argv);
 
-var options = {
-  addonName: program.addonName ? dasherize(program.addonName) : null
-};
-
-return validateOptions(options)
-  .then(run)
-  .catch(ui.error);
+if (process.argv.length < 3) {
+  program.outputHelp();
+}