fix: replace minimist with mri to avoid number conversion issue

Previously "npm makes aurelia 123" will fail because minimist converts
string "123" into number 123, this is very annoying. Switch to mri to
bypass this unwanted feature. Thank @kalle for the bug report.

lib/get-opts.js

@@ -1,12 +1,16 @@
 /* eslint-disable no-console */
-const minimist = require('minimist');
+const mri = require('mri');
 const camelCase = require('lodash.camelcase');
 
 module.exports = function(argv, help = '') {
-  const options = minimist(argv, {
+  const options = mri(argv, {
     alias: {s: 'select', h: 'help'},
     string: ['select'],
-    boolean: ['help', 'here']
+    boolean: ['help', 'here'],
+    default: {
+      help: false,
+      here: false
+    }
   });
 
   if (options.help) {

package.json

@@ -52,8 +52,8 @@
     "isutf8": "^3.1.1",
     "lodash.camelcase": "^4.3.0",
     "lodash.mergewith": "^4.6.2",
-    "minimist": "^1.2.5",
     "mock-fs": "^4.13.0",
+    "mri": "^1.1.6",
     "nyc": "^15.1.0",
     "sisteransi": "^1.0.5",
     "standard-changelog": "^2.0.27",

test/get-opts.spec.js

@@ -109,6 +109,16 @@ test('getOpts gets predefined properties', t => {
     here: false
   });
 
+  // minimist converts "123" to number 123.
+  // We switched to mri to avoid that issue.
+  t.deepEqual(getOpts(['a', '123']), {
+    supplier: 'a',
+    predefinedProperties: {name: '123'},
+    preselectedFeatures: [],
+    unattended: false,
+    here: false
+  });
+
   t.deepEqual(getOpts(['bitbucket:name/a', 'app', '--foo-bar', 'loo']), {
     supplier: 'bitbucket:name/a',
     predefinedProperties: {name: 'app', 'foo-bar': 'loo', 'fooBar': 'loo'},