Fix update script (#242)

Author: fisker

scripts/get-builtin-globals.mjs

@@ -25,20 +25,6 @@ const getText = async url => {
 	return text;
 };
 
-const any = async asyncFunctions => {
-	const errors = [];
-	for (const function_ of asyncFunctions) {
-		try {
-			// eslint-disable-next-line no-await-in-loop
-			return await function_();
-		} catch (error) {
-			errors.push(error);
-		}
-	}
-
-	throw new AggregateError(errors, 'All failed.');
-};
-
 const getSpecification = async () => {
 	let stat;
 
@@ -54,7 +40,7 @@ const getSpecification = async () => {
 		await fs.rm(CACHE_FILE);
 	}
 
-	const text = await any(SPECIFICATION_URLS.map(url => () => getText(url)));
+	const text = await Promise.any(SPECIFICATION_URLS.map(url => getText(url)));
 
 	await fs.mkdir(new URL('./', CACHE_FILE), {recursive: true});
 	await fs.writeFile(CACHE_FILE, text);

scripts/update.mjs

@@ -12,6 +12,7 @@ const ALL_JOBS = [
 	{
 		environment: 'builtin',
 		getGlobals: getBuiltinGlobals,
+		incremental: false,
 	},
 	{
 		environment: 'nodeBuiltin',
@@ -42,7 +43,7 @@ async function run(options) {
 		? ALL_JOBS.filter(job => job.environment === options.environment)
 		: ALL_JOBS;
 
-	for (const {environment, getGlobals, incremental = false} of jobs) {
+	for (const {environment, getGlobals, incremental = true} of jobs) {
 		const {
 			added,
 			removed,
@@ -52,7 +53,7 @@ async function run(options) {
 			environment,
 			getGlobals,
 			dryRun: options.dry,
-			incremental: incremental ?? !options.clean,
+			incremental: options.clean ? false : incremental,
 		});
 
 		console.log(`✅ ${environment} globals updated.`);