used build script instead of multiple npm tasks for build & avoid unneeded jison wrapper code

Author: russaa

package.json

@@ -27,20 +27,20 @@
   },
   "dependencies": {
     "JSV": ">= 4.0.x",
-    "chalk": "^4.1.0"
+    "chalk": "^4.1.0",
     "lodash": "^4.17.15"
   },
   "devDependencies": {
+    "fs-extra": "^9.0.1",
     "jison": "^0.4.18",
     "test": "*",
     "uglify-js": "^3.10.0"
   },
   "scripts": {
-    "build": "npm run build:jison && npm run build:mv && npm run build:web && npm run build:webmin",
-    "build:jison": "jison src/jsonlint.y src/jsonlint.l",
-    "build:mv": "node -e require('fs').renameSync('jsonlint.js','lib/jsonlint-ext.js')",
-    "build:web": "node scripts/bundle.js > web/jsonlint-ext.js",
-    "build:webmin": "node scripts/bundle.js | uglifyjs > web/jsonlint-ext.min.js",
+    "build": "node scripts/build.js",
+    "build:jison": "node scripts/build-jison.js src/jsonlint.y src/jsonlint.l",
+    "build:lib": "node scripts/bundle.js jsonlint.js lib/jsonlint-ext.js web/jsonlint-ext.js",
+    "build:rm": "node -e require('fs').unlinkSync('jsonlint.js')",
     "test": "node test/all-tests.js"
   },
   "optionalDependencies": {}

scripts/build-jison.js

@@ -0,0 +1,39 @@
+
+var fs = require('fs-extra');
+var path = require('path');
+var cli = require('jison/lib/cli');
+
+function processGrammar(raw, lex, opts) {
+    var grammar,
+    parser;
+    grammar = cli.processGrammars(raw, lex, opts.json);
+    parser = cli.generateParserString(opts, grammar);
+    return parser;
+}
+
+function buildGrammer (rawFile, lexFile, options){
+    return Promise.all([
+        fs.readFile(rawFile, 'utf-8'),
+        fs.readFile(lexFile, 'utf-8')
+    ]).then(function(files){
+        var opts = Object.assign({
+            moduleName: 'jsonlint',
+            'module-type': 'js'
+        }, options || {});
+        return processGrammar(files[0], files[1], opts);
+    })
+}
+
+module.exports = {
+    build: buildGrammer
+}
+
+if (require.main === module) {
+    var rawFile = path.resolve(__dirname, '..', process.argv[2]);
+    var lexFile = path.resolve(__dirname, '..', process.argv[3]);
+
+    buildGrammer(rawFile, lexFile).then(function(parser){
+        var outFile = path.resolve(__dirname, '..', 'jsonlint.js')
+        return fs.writeFile(outFile, parser, 'utf-8');
+    });
+}

scripts/build.js

@@ -0,0 +1,26 @@
+
+var fs = require('fs-extra');
+var path = require('path');
+
+var buildJison = require('./build-jison');
+var bundle = require('./bundle');
+
+var rawFile = path.resolve(__dirname, '..', 'src/jsonlint.y');
+var lexFile = path.resolve(__dirname, '..', 'src/jsonlint.l');
+
+buildJison.build(rawFile, lexFile).then(function(parser){
+    var libFile = path.resolve(__dirname, '..', 'lib/jsonlint-ext.js');
+    var webFile = path.resolve(__dirname, '..', 'web/jsonlint-ext.js');
+
+    bundle.bundleTo(parser, [libFile, webFile]).then(function(bundledCode){
+
+        var minFileName = 'jsonlint-ext.min.js'
+        var minFile = path.resolve(__dirname, '..', 'web', minFileName);
+        var minified = bundle.minifySync(bundledCode, minFileName);
+
+        return Promise.all([
+            fs.writeFile(minFile, minified.code, 'utf-8'),
+            fs.writeFile(minFile + '.map', minified.map, 'utf-8')
+        ])
+    });
+})

scripts/bundle.js

@@ -1,39 +1,62 @@
-var fs = require('fs');
+
+var fs = require('fs-extra');
 var path = require('path');
+var uglify = require('uglify-js');
 
 var template = require('./template');
 
 function bundle(srcPath){
 
-    var srcFile = fs.existsSync(srcPath)? fs.readFileSync(srcPath, 'utf8') : srcPath;
+    var promise = fs.existsSync(srcPath)? fs.readFile(srcPath, 'utf8') : Promise.resolve(srcPath);
 
-    var source = template.preamble + template.umdHeader +
-        srcFile +
-        template.moduleExports + template.umdFooter;
+    return promise.then(function(srcFile){
+        var source = template.preamble + template.umdHeader +
+            srcFile +
+            template.moduleExports + template.umdFooter;
 
-    return source;
+        return source;
+    });
 }
 
 function bundleTo(srcPath, targets){
-    var source = bundle(srcPath);
-    var t;
-    for(var i=0,size=targets.length; i < size; ++i){
-        t = targets[i];
-        fs.writeFileSync(path.resolve(__dirname, '..', t), source);
-    }
+    return bundle(srcPath).then(function(source){
+        var t, files = [];
+        if(!Array.isArray(targets)){
+            targets = [targets];
+        }
+        for(var i=0,size=targets.length; i < size; ++i){
+            t = targets[i];
+            files.push(fs.writeFile(path.resolve(__dirname, '..', t), source));
+        }
+        return Promise.all(files).then(function(){
+            return source;
+        });
+    });
+}
+
+function minifySync(code, targetFileName){
+    return uglify.minify(code, {
+        sourceMap: {
+            filename: targetFileName,
+            url: targetFileName + '.map'
+        }
+    });
 }
 
 module.exports = {
     bundle: bundle,
-    bundleTo: bundleTo
+    bundleTo: bundleTo,
+    minifySync: minifySync
 }
 
 if (require.main === module) {
 
     if(process.argv.length === 2){
 
         var srcPath = path.resolve(__dirname, '..', 'jsonlint.js');
-        console.log(bundle(srcPath));
+        bundle(srcPath).then(function(src){
+            console.log(src);
+        });
 
     } else {