Try using yargs instead of optimits

This is me poking at the issue of optimits not being maintained any
more[0] and having the security issue because of minimist[1]. I’m not
sure it’s going to be yargs in the end, it’s just what optimits
developer recommends as an alternative.

[0]: https://www.npmjs.com/package/optimist
[1]: https://www.npmjs.com/advisories/1179

Signed-off-by: Vlad GURDIGA <[email protected]>

Author: gurdiga

lib/commands/run.js

@@ -1,7 +1,7 @@
 'use strict';
 
 var log = require('db-migrate-shared').log;
-var optimist = require('optimist');
+var argParser = require('yargs');
 var load = require('./');
 var transition = load('transition');
 
@@ -117,7 +117,7 @@ function run (internals, config) {
           'Invalid Action: Must be [up|down|check|create|reset|sync|' +
             'db|transition].'
         );
-        optimist.showHelp();
+        argParser.showHelp();
         process.exit(1);
       }
       break;

lib/commands/set-default-argv.js

@@ -1,4 +1,4 @@
-var optimist = require('optimist');
+var argParser = require('yargs');
 var log = require('db-migrate-shared').log;
 
 module.exports = function (internals, isModule) {
@@ -20,7 +20,7 @@ module.exports = function (internals, isModule) {
   };
 
   if (!isModule) {
-    internals.argv = optimist
+    internals.argv = argParser
       .default(defaultConfig)
       .usage(
         'Usage: db-migrate [up|down|check|reset|sync|create|db|transition] ' +
@@ -110,7 +110,8 @@ module.exports = function (internals, isModule) {
       .describe('ignore-completed-migrations', 'Start at the first migration')
       .boolean('ignore-completed-migrations')
       .describe('log-level', 'Set the log-level, for example sql|warn')
-      .string('log-level');
+      .string('log-level')
+      .parse();
   } else {
     const _internalsArgv = Object.assign(defaultConfig, internals.cmdOptions);
     internals.argv = {
@@ -128,7 +129,7 @@ module.exports = function (internals, isModule) {
     plugin.forEach(function (plugin) {
       // var configs = plugin['init:cli:config:hook']();
       // if (!configs) return;
-      // hook not yet used, we look into migrating away from optimist first
+      // hook not yet used, we look into migrating away from argParser first
     });
   }
 
@@ -142,7 +143,7 @@ module.exports = function (internals, isModule) {
   }
 
   if (!isModule && (internals.argv.help || internals.argv._.length === 0)) {
-    optimist.showHelp();
+    argParser.showHelp();
     process.exit(1);
   }