Clean up utils

Author: moonglum

lib/index.js

@@ -3,9 +3,8 @@
 let server = require("./server");
 let { pluginsByBucket } = require("./plugins");
 let { AssetManager } = require("./manager");
-let { resolvePath } = require("./util/resolve");
-let { abort, repr } = require("./util");
-let { SerializedRunner } = require("./util/runner");
+let { abort, repr, resolvePath } = require("./util");
+let { SerializedRunner } = require("./runner");
 let browserslist = require("browserslist");
 
 exports.faucetDispatch = async function faucetDispatch(referenceDir, config,

lib/manager.js

@@ -2,9 +2,9 @@
 
 let { Manifest } = require("./manifest");
 let { createFile } = require("./util/files");
-let { resolvePath } = require("./util/resolve");
-let { reportFileStatus, abort, generateFingerprint } = require("./util");
+let { reportFileStatus, abort, resolvePath } = require("./util");
 let path = require("path");
+let crypto = require("crypto");
 
 exports.AssetManager = class AssetManager {
 	constructor(referenceDir, { manifestConfig, fingerprint, exitOnError } = {}) {
@@ -58,3 +58,20 @@ exports.AssetManager = class AssetManager {
 		return this.manifest.set(originalPath, actualPath, targetDir);
 	}
 };
+
+function generateFingerprint(filepath, data) {
+	let filename = path.basename(filepath);
+	let ext = filename.indexOf(".") === -1 ? "" : "." + filename.split(".").pop();
+	let name = ext.length === 0 ? filename : path.basename(filepath, ext);
+	let hash = generateHash(data);
+	return path.join(path.dirname(filepath), `${name}-${hash}${ext}`);
+}
+
+// exported for testing
+exports.generateFingerprint = generateFingerprint;
+
+function generateHash(str) {
+	let hash = crypto.createHash("md5");
+	hash.update(str);
+	return hash.digest("hex");
+}

lib/manifest.js

@@ -1,8 +1,7 @@
 "use strict";
 
 let { createFile } = require("./util/files");
-let { resolvePath } = require("./util/resolve");
-let { abort } = require("./util");
+let { abort, resolvePath } = require("./util");
 let path = require("path");
 
 exports.Manifest = class Manifest {

lib/util/runner.js renamed to lib/runner.js

@@ -1,10 +1,7 @@
 "use strict";
 
+let fs = require("fs");
 let path = require("path");
-let crypto = require("crypto");
-
-exports.abort = abort;
-exports.repr = repr;
 
 // reports success or failure for a given file path (typically regarding
 // compilation or write operations)
@@ -26,28 +23,39 @@ exports.loadExtension = async (pkg, errorMessage, supplier = pkg) => {
 	}
 };
 
-exports.generateFingerprint = (filepath, data) => {
-	let filename = path.basename(filepath);
-	let ext = filename.indexOf(".") === -1 ? "" : "." + filename.split(".").pop();
-	let name = ext.length === 0 ? filename : path.basename(filepath, ext);
-	let hash = generateHash(data);
-	return path.join(path.dirname(filepath), `${name}-${hash}${ext}`);
-};
-
 function abort(msg, code = 1) {
 	console.error(msg);
 	process.exit(code);
 }
+exports.abort = abort;
 
 function repr(value, jsonify = true) {
 	if(jsonify) {
 		value = JSON.stringify(value);
 	}
 	return `\`${value}\``;
 }
+exports.repr = repr;
 
-function generateHash(str) {
-	let hash = crypto.createHash("md5");
-	hash.update(str);
-	return hash.digest("hex");
-}
+exports.resolvePath = (filepath, referenceDir, { enforceRelative } = {}) => {
+	if(/^\.?\.\//.test(filepath)) { // starts with `./` or `../`
+		return path.resolve(referenceDir, filepath);
+	} else if(enforceRelative) {
+		abort(`ERROR: path must be relative: ${repr(filepath)}`);
+	} else { // attempt via Node resolution algorithm
+		try {
+			return require.resolve(filepath, { paths: [referenceDir] });
+		} catch(err) {
+			// attempt to resolve non-JavaScript package references by relying
+			// on typical package paths (simplistic approximation of Node's
+			// resolution algorithm)
+			let resolved = path.resolve(referenceDir, "node_modules", filepath);
+			try {
+				fs.statSync(resolved); // ensures file/directory exists
+			} catch(_err) {
+				abort(`ERROR: could not resolve ${repr(filepath)}`);
+			}
+			return resolved;
+		}
+	}
+};

lib/util/files/index.js renamed to lib/util/files.js

@@ -1,6 +1,6 @@
 "use strict";
 
-let { AssetManager } = require("../lib/manager");
+let { AssetManager, generateFingerprint } = require("../lib/manager");
 let { describe, it, before, after } = require("node:test");
 let path = require("path");
 let assert = require("assert");
@@ -51,3 +51,18 @@ describe("asset manager", () => {
 		}, /exit 1/);
 	});
 });
+
+describe("fingerprinting", () => {
+	it("generates a content-dependent hash", () => {
+		let fingerprint = generateFingerprint("/path/to/foo.js", "lorem ipsum");
+		assertSame(fingerprint, "/path/to/foo-80a751fde577028640c419000e33eba6.js");
+
+		fingerprint = generateFingerprint("/path/to/bar.js", "dolor sit amet");
+		assertSame(fingerprint, "/path/to/bar-7afed6210e0b8fce023f06abd4490fa0.js");
+	});
+
+	it("supports files without extension", () => {
+		let fingerprint = generateFingerprint("/path/to/baz", "lipsum");
+		assertSame(fingerprint, "/path/to/baz-8047cfaac755e5c7f77af066123980a5");
+	});
+});

lib/util/files/finder.js

@@ -1,6 +1,6 @@
 "use strict";
 
-let { SerializedRunner } = require("../lib/util/runner");
+let { SerializedRunner } = require("../lib/runner");
 let { describe, it } = require("node:test");
 let { strictEqual: assertSame, deepStrictEqual: assertDeep } = require("assert");