commander instead of yargs

package.json

@@ -1,6 +1,6 @@
 {
   "name": "zimmer",
-  "version": "0.0.2",
+  "version": "0.1.0",
   "description": "Zim file packer",
   "main": "zimmer.js",
   "scripts": {
@@ -8,28 +8,36 @@
   },
   "private": true,
   "keywords": [
+    "mediawiki",
     "zim",
     "pack"
   ],
   "dependencies": {
     "bluebird": "*",
     "cheerio": "*",
-    "child-process": "^1.0.2",
+    "child-process": "*",
     "csv-parse": "*",
-    "csv-stringify": "*",
+    "encodeurl": "^1.0.1",
     "expand-home-dir": "*",
     "fs-extra": "^3.0.1",
     "generic-pool": "^3.1.7",
+    "iconv-lite": "^0.4.17",
+    "langs": "^2.0.0",
     "lzma-native": "*",
     "mime-db": "*",
     "mime-types": "*",
     "mmmagic": "*",
     "mozjpeg": "*",
     "mz": "^2.6.0",
+    "quick-lru": "^1.0.0",
+    "request": "^2.81.0",
+    "request-promise": "^4.2.1",
+    "sanitize-filename": "^1.6.1",
     "sharp": "^0.17.3",
     "sqlite": "^2.8.0",
+    "sqlite3": "*",
     "uuid": "*",
-    "yargs": "*"
+    "commander": "^2.11.0"
   },
   "author": "Vadim Shlykahov",
   "license": "ISC"

zimmer.js

@@ -29,20 +29,21 @@ SOFTWARE.
 
 */
 
+const packageInfo = require('./package.json');
 const Promise = require( 'bluebird' )
 const os = require( 'os' )
-const process = require( 'process' )
-const yargs = require( 'yargs' )
-const util = require( 'util' )
-const fs = require( 'fs-extra' )
 const osPath = require( 'path' )
+const process = require( 'process' )
+const url = require( 'url' )
+const crypto = require( "crypto" )
+
+const argv = require('commander')
+const fs = require( 'fs-extra' )
 const expandHomeDir = require( 'expand-home-dir' )
 const lzma = require( 'lzma-native' )
 const cheerio = require('cheerio')
 
-const url = require( 'url' )
 const uuid = require( "uuid" )
-const crypto = require( "crypto" )
 const csvParse = require( 'csv-parse' )
 const csvParseSync = require( 'csv-parse/lib/sync' )
 const zlib = require( 'mz/zlib' )
@@ -61,8 +62,6 @@ const mimeMagic = new mmmagic.Magic( mmmagic.MAGIC_MIME_TYPE )
 
 const cpuCount = os.cpus().length
 
-var argv
-
 var srcPath
 var outPath
 var out // output file writer
@@ -1552,42 +1551,40 @@ function core () {
 // -i, --withFullTextIndex index the content and add it to the ZIM.
 
 function main () {
-    argv = yargs.usage( 'Pack a directory into a zim file\nUsage: $0'
+
-               + '\nExample: $0 ' )
+    argv
-        //~ .boolean( 'optimg' )
+    .version( packageInfo.version )
-        .options({
+    .description( 'Pack a directory into a zim file' )
-            'w': {alias: 'welcome', default: 'index.htm'},
+    .arguments( '<source-directory> [output...]' )
-            'f': {alias: 'favicon', default: 'favicon.png'},
+    // Mandatory arguments:
-            'l': {alias: 'language', default: 'eng'},
+    .option( '-w, --welcome <page>', 'path of default/main HTML page. The path must be relative to HTML_DIRECTORY', 'index.htm' )
-            't': {alias: 'title', default: ''},
+    .option( '-f, --favicon <file>', 'path of ZIM file favicon. The path must be relative to HTML_DIRECTORY and the image a 48x48 PNG', 'favicon.png' )
-            'd': {alias: 'description', default: ''},
+    .option( '-l, --language <id>', 'language code of the content in ISO639-3', 'eng' )
-            'c': {alias: 'creator', default: ''},
+    .option( '-t, --title <title>', 'title of the ZIM file', '' )
-            'p': {alias: 'publisher', default: ''},
+    .option( '-d, --description <text>', 'short description of the content', '' )
-            'v': {alias: 'verbose', type: 'boolean', default: false},
+    .option( '-c, --creator  <text>', 'creator(s) of the content', '' )
-            'm': {alias: 'minChunkSize', type: 'number', default: ClusterSizeThreshold / 1024},
+    .option( '-p, --publisher  <text>', 'creator of the ZIM file itself', '' )
-            'x': {alias: 'inflateHtml', type: 'boolean', default: false},
+    // Optional arguments:
-            'u': {alias: 'uniqueNamespace', type: 'boolean', default: false},
+    .option( '-v, --verbose', 'print processing details on STDOUT' )
-            //~ 'r': {alias: 'redirects', default: 'redirects.csv'},
+    .option( '-m, --minChunkSize <size>', 'number of bytes per ZIM cluster (default: 2048)', parseInt, 2048 )
-            'r': {alias: 'redirects', default: ''},
+    .option( '-x, --inflateHtml', 'try to inflate HTML files before packing (*.html, *.htm, ...)' )
-            //~ 'i': {alias: 'withFullTextIndex', type:'boolean', default: false},
+    .option( '-u, --uniqueNamespace', 'put everything in the same namespace "A". Might be necessary to avoid problems with dynamic/javascript data loading' )
-            'h': {alias: 'help'},
+    .option( '-r, --redirects <path>', 'path to the CSV file with the list of redirects (url, title, target_url tab separated)', '' )
-            'optimg': {type: 'boolean', default: false},
+    //~ .option( '-i, --withFullTextIndex', 'index the content and add it to the ZIM' )
-            'jpegquality': {type: 'number', default: 60},
+    .option( '--optimg', 'optimise images' )
-        })
+    .option( '--jpegquality <factor>', 'JPEG quality', parseInt, 60 )
-        .help( 'help' )
+    .parse( process.argv )
-        //~ .strict()
-        .argv
 
     log( argv )
 
-    var pargs = argv._
+    var args = argv.args
 
-    while ( pargs[ 0 ] == '' ) // if mwoffliner prepends with empty extra parameter(s)
+    while ( args[ 0 ] == '' ) // if mwoffliner prepends with empty extra parameter(s)
-        pargs.shift()
+        args.shift()
 
-    srcPath = expandHomeDir( pargs[ 0 ])
+    srcPath = expandHomeDir( args[ 0 ])
-    if ( argv._[ 1 ])
+    if ( args[ 1 ])
-        outPath = expandHomeDir( pargs[ 1 ])
+        outPath = expandHomeDir( args[ 1 ])
     else {
         var parsed = osPath.parse( srcPath )
         outPath = parsed.base + '.zim'