commander 1.1.1 → 1.3.2

package/History.md

@@ -1,4 +1,26 @@
 
+1.3.2 / 2013-07-18 
+==================
+
+ * add support for sub-commands to co-exist with the original command
+
+1.3.1 / 2013-07-18 
+==================
+
+ * add quick .runningCommand hack so you can opt-out of other logic when running a sub command
+
+1.3.0 / 2013-07-09 
+==================
+
+ * add EACCES error handling
+ * fix sub-command --help
+
+1.2.0 / 2013-06-13 
+==================
+
+ * allow "-" hyphen as an option argument
+ * support for RegExp coercion
+
 1.1.1 / 2012-11-20 
 ==================
 

package/Readme.md

@@ -31,7 +31,7 @@ program
 
 console.log('you ordered a pizza with:');
 if (program.peppers) console.log('  - peppers');
-if (program.pineapple) console.log('  - pineappe');
+if (program.pineapple) console.log('  - pineapple');
 if (program.bbq) console.log('  - bbq');
 console.log('  - %s cheese', program.cheese);
 ```
@@ -51,7 +51,7 @@ console.log('  - %s cheese', program.cheese);
 
      -V, --version        output the version number
      -p, --peppers        Add peppers
-     -P, --pineapple      Add pineappe
+     -P, --pineapple      Add pineapple
      -b, --bbq            Add bbq sauce
      -c, --cheese <type>  Add the specified type of cheese [marble]
      -h, --help           output usage information
@@ -185,6 +185,12 @@ program.prompt('Birthdate: ', Date, function(date){
 });
 ```
 
+```js
+program.prompt('Email: ', /^.+@.+\..+$/, function(email){
+  console.log('email: %j', email);
+});
+```
+
 ## .password(msg[, mask], fn)
 
 Prompt for password without echoing:
@@ -267,4 +273,4 @@ MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
 IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY
 CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
 TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
-SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

package/index.js

@@ -91,6 +91,7 @@ Option.prototype.is = function(arg){
 function Command(name) {
   this.commands = [];
   this.options = [];
+  this._execs = [];
   this._args = [];
   this._name = name;
 }
@@ -155,6 +156,7 @@ Command.prototype.command = function(name, desc){
   var cmd = new Command(args.shift());
   if (desc) cmd.description(desc);
   if (desc) this.executables = true;
+  if (desc) this._execs[cmd._name] = true;
   this.commands.push(cmd);
   cmd.parseExpectedArgs(args);
   cmd.parent = this;
@@ -372,10 +374,13 @@ Command.prototype.parse = function(argv){
   var parsed = this.parseOptions(this.normalize(argv.slice(2)));
   var args = this.args = parsed.args;
  
-  // executable sub-commands, skip .parseArgs()
-  if (this.executables) return this.executeSubCommand(argv, args, parsed.unknown);
+  var result = this.parseArgs(this.args, parsed.unknown);
 
-  return this.parseArgs(this.args, parsed.unknown);
+  // executable sub-commands
+  var name = result.args[0];
+  if (this._execs[name]) return this.executeSubCommand(argv, args, parsed.unknown);
+
+  return result;
 };
 
 /**
@@ -405,16 +410,19 @@ Command.prototype.executeSubCommand = function(argv, args, unknown) {
 
   // check for ./<bin> first
   var local = path.join(dir, bin);
-  if (exists(local)) bin = local;
 
   // run it
   args = args.slice(1);
-  var proc = spawn(bin, args, { stdio: 'inherit', customFds: [0, 1, 2] });
-  proc.on('exit', function(code){
-    if (code == 127) {
-      console.error('\n  %s(1) does not exist\n', bin);
+  var proc = spawn(local, args, { stdio: 'inherit', customFds: [0, 1, 2] });
+  proc.on('error', function(err){
+    if (err.code == "ENOENT") {
+      console.error('\n  %s(1) does not exist, try --help\n', bin);
+    } else if (err.code == "EACCES") {
+      console.error('\n  %s(1) not executable. try chmod or run with root\n', bin);
     }
   });
+
+  this.runningCommand = proc;
 };
 
 /**
@@ -543,12 +551,12 @@ Command.prototype.parseOptions = function(argv){
       if (option.required) {
         arg = argv[++i];
         if (null == arg) return this.optionMissingArgument(option);
-        if ('-' == arg[0]) return this.optionMissingArgument(option, arg);
+        if ('-' == arg[0] && '-' != arg) return this.optionMissingArgument(option, arg);
         this.emit(option.name(), arg);
       // optional arg
       } else if (option.optional) {
         arg = argv[i+1];
-        if (null == arg || '-' == arg[0]) {
+        if (null == arg || ('-' == arg[0] && '-' != arg)) {
           arg = null;
         } else {
           ++i;
@@ -811,6 +819,25 @@ Command.prototype.promptForDate = function(str, fn){
   });
 };
 
+
+/**
+ * Prompt for a `Regular Expression`.
+ *
+ * @param {String} str
+ * @param {Object} pattern regular expression object to test
+ * @param {Function} fn
+ * @api private
+ */
+
+Command.prototype.promptForRegexp = function(str, pattern, fn){
+  var self = this;
+  this.promptSingleLine(str, function parseRegexp(val){
+    if(!pattern.test(val)) return self.promptSingleLine(str + '(regular expression mismatch) ', parseRegexp);
+    fn(val);
+  });
+};
+
+
 /**
  * Single-line prompt.
  *
@@ -820,7 +847,10 @@ Command.prototype.promptForDate = function(str, fn){
  */
 
 Command.prototype.promptSingleLine = function(str, fn){
-  if ('function' == typeof arguments[2]) {
+  // determine if the 2nd argument is a regular expression
+  if (arguments[1].global !== undefined && arguments[1].multiline !== undefined) {
+    return this.promptForRegexp(str, arguments[1], arguments[2]);
+  } else if ('function' == typeof arguments[2]) {
     return this['promptFor' + (fn.name || fn)](str, arguments[2]);
   }
 
@@ -879,7 +909,6 @@ Command.prototype.promptMultiLine = function(str, fn){
 
 Command.prototype.prompt = function(str, fn){
   var self = this;
-
   if ('string' == typeof str) {
     if (/ $/.test(str)) return this.promptSingleLine.apply(this, arguments);
     this.promptMultiLine(str, fn);

package/package.json

@@ -1,10 +1,10 @@
 {
     "name": "commander"
-  , "version": "1.1.1"
+  , "version": "1.3.2"
   , "description": "the complete solution for node.js command-line programs"
   , "keywords": ["command", "option", "parser", "prompt", "stdin"]
   , "author": "TJ Holowaychuk <tj@vision-media.ca>"
-  , "repository": { "type": "git", "url": "https://github.com/visionmedia/commander.js.git" }  
+  , "repository": { "type": "git", "url": "https://github.com/visionmedia/commander.js.git" }
   , "dependencies": { "keypress": "0.1.x"}
   , "devDependencies": { "should": ">= 0.0.1" }
   , "scripts": { "test": "make test" }

package/.npmignore

@@ -1,4 +0,0 @@
-support
-test
-examples
-*.sock

package/.travis.yml

@@ -1,4 +0,0 @@
-language: node_js
-node_js:
-  - 0.4
-  - 0.6

package/Makefile

@@ -1,7 +0,0 @@
-
-TESTS = $(shell find test/test.*.js)
-
-test:
-	@./test/run $(TESTS)
-
-.PHONY: test