fix(help): Display other commands when default command is present (#64)

mattallty · Oct 20, 2017 · 8df56e8 · 8df56e8
1 parent cc253d9
commit 8df56e8
Showing 1 changed file with 16 additions and 12 deletions.
diff --git a/lib/help.js b/lib/help.js
@@ -37,7 +37,7 @@ class Help {
      
    ${this._getUsage(command)}`;
 
-    if (!command && this._program._commands.length > 1) {
+    if (!command || command.name() === '' && this._program._commands.length > 1) {
       help += "\n\n   " + this._getCommands();
     }
 
@@ -52,11 +52,14 @@ class Help {
 
   _getCommands() {
     const commandTable = this._getSimpleTable();
-    this._program._commands.forEach(cmd => {
-      commandTable.push(
-        [chalk.magenta(cmd.getSynopsis()), cmd.description()]
-      );
-    });
+    this._program._commands
+      // don't include default command
+      .filter((c) => c.name() !== '')
+      .forEach(cmd => {
+        commandTable.push(
+          [chalk.magenta(cmd.getSynopsis()), cmd.description()]
+        );
+      });
     commandTable.push([chalk.magenta('help <command>'), 'Display help for a specific command']);
     return chalk.bold('COMMANDS') + "\n\n" + colorize(commandTable.toString());
   }
@@ -112,14 +115,15 @@ class Help {
   }
 
 
-
   _getSimpleTable() {
     return new Table({
-      chars: { 'top': '' , 'top-mid': '' , 'top-left': '' , 'top-right': ''
-        , 'bottom': '' , 'bottom-mid': '' , 'bottom-left': '' , 'bottom-right': ''
-        , 'left': '' , 'left-mid': '' , 'mid': '' , 'mid-mid': ''
-        , 'right': '' , 'right-mid': '' , 'middle': ' ' },
-      style: { 'padding-left': 5, 'padding-right': 0 }
+      chars: {
+        'top': '', 'top-mid': '', 'top-left': '', 'top-right': ''
+        , 'bottom': '', 'bottom-mid': '', 'bottom-left': '', 'bottom-right': ''
+        , 'left': '', 'left-mid': '', 'mid': '', 'mid-mid': ''
+        , 'right': '', 'right-mid': '', 'middle': ' '
+      },
+      style: {'padding-left': 5, 'padding-right': 0}
     });
   }