Merge pull request #14536 from guardian/ravi/remove-load-json-file

Replace `load-json-file` with native node method

Author: arelra

dotcom-rendering/package.json

@@ -137,7 +137,6 @@
 		"jest": "29.7.0",
 		"jest-environment-jsdom": "29.7.0",
 		"jsdom": "22.1.0",
-		"load-json-file": "6.2.0",
 		"lodash.debounce": "4.0.8",
 		"log4js": "6.9.1",
 		"lz-string": "1.5.0",

dotcom-rendering/scripts/test/build-check.js

@@ -5,10 +5,22 @@
 // 1. That the manifest files are output
 // 2. That the manifest files contain at least the entry points under the expected property
 
+const { readFile } = require('node:fs').promises;
 const find = require('find');
-const loadJsonFile = require('load-json-file');
 const { BUILD_VARIANT } = require('../../webpack/bundles');
 
+/**
+ * Loads a JSON file.
+ * Inspired by https://github.com/sindresorhus/load-json-file/blob/de8256b9010db73c75a1e2036ff96025e94c0b6e/index.js#L6
+ * @param {string} filePath The path to the JSON file.
+ * @returns {Promise<Object>} The parsed JSON object.
+ */
+async function loadJsonFile(filePath) {
+	const buffer = await readFile(filePath);
+	// Unlike `buffer.toString()` and `fs.readFile(path, 'utf8')`, `TextDecoder` will remove BOM.
+	return JSON.parse(new TextDecoder().decode(buffer));
+}
+
 const errorAndThrow = (error) => {
 	console.error(error);
 	throw new Error(error);