tools: refactor update-authors.js to ESM

PR-URL: #43098 Reviewed-By: Antoine du Hamel <[email protected]>

Author: F3n67u

.github/workflows/authors.yml

@@ -15,7 +15,7 @@ jobs:
 with:
 fetch-depth: '0'# This is required to actually get all the authors
 persist-credentials: false
- - run: tools/update-authors.js# Run the AUTHORS tool
+ - run: tools/update-authors.mjs# Run the AUTHORS tool
  - uses: gr2m/create-or-update-pull-request-action@v1 # Create a PR or update the Action's existing PR
 env:
 GITHUB_TOKEN: ${{secrets.GH_USER_TOKEN }}
@@ -24,7 +24,7 @@ jobs:
 body: >
  Here are some new additions to the AUTHORS file.
  This is an automatically generated PR by the
- `authors.yml` GitHub Action, which runs `tools/update-authors.js`.
+ `authors.yml` GitHub Action, which runs `tools/update-authors.mjs`.
 branch: actions/authors-update # Custom branch *just* for this Action.
 commit-message: 'meta: update AUTHORS'
 labels: meta

AUTHORS

@@ -3460,4 +3460,4 @@ William Marlow <[email protected]>
 Keyhan Vakil <[email protected]>
 Feng Yu <[email protected]>
 
-# Generated by tools/update-authors.js
+# Generated by tools/update-authors.mjs

tools/update-authors.js tools/update-authors.mjstools/update-authors.js renamed to tools/update-authors.mjs

@@ -1,11 +1,9 @@
 #!/usr/bin/env node
-// Usage: tools/update-author.js [--dry]
+// Usage: tools/update-author.mjs [--dry]
 // Passing --dry will redirect output to stdout rather than write to 'AUTHORS'.
-'use strict';
-const{ spawn }=require('child_process');
-constpath=require('path');
-constfs=require('fs');
-constreadline=require('readline');
+import{spawn}from'node:child_process';
+importfsfrom'node:fs';
+importreadlinefrom'node:readline';
 
 classCaseIndifferentMap{
 _map=newMap();
@@ -33,7 +31,7 @@ output.write('# Authors ordered by first contribution.\n\n');
 
 constmailmap=newCaseIndifferentMap();
 {
-constlines=fs.readFileSync(path.resolve(__dirname,'../','.mailmap'),
+constlines=fs.readFileSync(newURL('../.mailmap',import.meta.url),
 {encoding: 'utf8'}).split('\n');
 for(letlineoflines){
 line=line.trim();
@@ -55,7 +53,7 @@ const mailmap = new CaseIndifferentMap();
 
 constpreviousAuthors=newCaseIndifferentMap();
 {
-constlines=fs.readFileSync(path.resolve(__dirname,'../','AUTHORS'),
+constlines=fs.readFileSync(newURL('../AUTHORS',import.meta.url),
 {encoding: 'utf8'}).split('\n');
 for(letlineoflines){
 line=line.trim();
@@ -85,9 +83,9 @@ const seen = new Set();
 // by GitHub now.
 constauthorRe=
 /(^Author:|^Co-authored-by:)\s+(?<author>[^<]+)\s+(?<email><[^>]+>)/i;
-rl.on('line',(line)=>{
+forawait(constlineofrl){
 constmatch=line.match(authorRe);
-if(!match)return;
+if(!match)continue;
 
 let{ author, email }=match.groups;
 constemailLower=email.toLowerCase();
@@ -99,7 +97,7 @@ rl.on('line', (line) =>{
 }
 
 if(seen.has(email)){
-return;
+continue;
 }
 
 seen.add(email);
@@ -109,8 +107,6 @@ rl.on('line', (line) =>{
 console.warn('Author name already in AUTHORS file. Possible duplicate:');
 console.warn(` ${author}${email}`);
 }
-});
+}
 
-rl.on('close',()=>{
-output.end('\n# Generated by tools/update-authors.js\n');
-});
+output.end('\n# Generated by tools/update-authors.mjs\n');