merge all rev-manifest.json assets to one rev-manifest.json

[miladmeidanshahi]

Description

Hi, how can i merge all assets to one rev-manifest.json for example css files js files to one rev-manifest.json and rev-del work correctly in this situation.

here is asset directory

static/dist/css/*.css
static/dist/js/*.js

rev-manifest.json files for all assets

static/dist/rev-manifest.json

{
  "test.js": "test-b159db5fed.min.js",
  "test.css": "test-b159db5fed.min.css",
  ...
}

and finally i think the gulp task must be something like this

for javascript files

        .pipe(rev())
        .pipe(gulp.dest(statics + 'dist/js'))
        .pipe(rev.manifest({ merge: true }))
        .pipe(revDel({ dest: statics + 'dist/js' }))
        // hear save the rev-manifest.json to static/dist/rev-manifest.json not separate for each folder 
        .pipe(gulp.dest(statics + 'dist'))

for css files

        .pipe(rev())
        .pipe(gulp.dest(statics + 'dist/css'))
        .pipe(rev.manifest({ merge: true }))
        .pipe(revDel({ dest: statics + 'dist/css' }))
        // hear save the rev-manifest.json to static/dist/rev-manifest.json not separate for each folder 
        .pipe(gulp.dest(statics + 'dist'))

the important step is revdel should work fine

[prime89]

The merge option does not work on multi-process

[Saturate]

You can write another gulp task to merge all the manifests after all parallel tasks are done. Something like this should do:

import fs from 'fs';
import log from 'fancy-log';
import chalk from 'chalk';

const options = {
	finalManifest: './dist/asset-manifest.json'
};

function readManifest(manifestPath) {
	return new Promise((resolve, reject) => {
		fs.readFile(manifestPath, 'utf8', (readErr, data) => {
			let jsonData;

			if (readErr) {
				console.error(readErr);
				return reject(readErr);
			}

			try {
				jsonData = JSON.parse(data);
			} catch (parseError) {
				console.error('Parse Error: ', manifestPath, data, parseError);
				return reject(parseError);
			}

			resolve(jsonData);

			return data;
		});
	});
}

function mergeManifest(done) {
	return Promise.all([
		readManifest('./dist/css/rev-manifest.json'),
		readManifest('./dist/js/rev-manifest.json')
	]).then((values) => {
		const mergedManifest = Object.assign(...values);

		log(chalk.cyan('Writing merged assest manifest') + chalk.grey(` (${options.finalManifest})`));

		fs.writeFile(options.finalManifest, JSON.stringify(mergedManifest, null, '\t'), (err) => {
			if (err) throw err;
			done();
		});
	}).catch((err) => {
		console.error(err);
		throw err;
	});
}

export default mergeManifest;