tools: lint README lists more strictly

PR-URL: #55625 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]>

Author: aduh95

README.md

@@ -409,7 +409,7 @@ For information about the governance of the Node.js project, see
 **Filip Skokan** <<[email protected]>> (he/him)
 *[pimterry](https://github.com/pimterry) -
 **Tim Perry** <<[email protected]>> (he/him)
-*[pmarchini](https://github.com/pmarchini)
+*[pmarchini](https://github.com/pmarchini) -
 **Pietro Marchini** <<[email protected]>> (he/him)
 *[Qard](https://github.com/Qard) -
 **Stephen Belanger** <<[email protected]>> (he/him)
@@ -515,7 +515,7 @@ For information about the governance of the Node.js project, see
 **Hitesh Kanwathirtha** <<[email protected]>> (he/him)
 *[dmabupt](https://github.com/dmabupt) -
 **Xu Meng** <<[email protected]>> (he/him)
-*[dnlup](https://github.com/dnlup)
+*[dnlup](https://github.com/dnlup) -
 **dnlup** <<[email protected]>>
 *[eljefedelrodeodeljefe](https://github.com/eljefedelrodeodeljefe) -
 **Robert Jefe Lindstaedt** <<[email protected]>>
@@ -757,7 +757,7 @@ maintaining the Node.js project.
 **Mert Can Altin** <<[email protected]>>
 *[preveen-stack](https://github.com/preveen-stack) -
 **Preveen Padmanabhan** <<[email protected]>> (he/him)
-*[RedYetiDev](https://github.com/redyetidev) -
+*[RedYetiDev](https://github.com/RedYetiDev) -
 **Aviv Keller** <<[email protected]>> (they/them)
 *[VoltrexKeyva](https://github.com/VoltrexKeyva) -
 **Mohammed Keyvanzadeh** <<[email protected]>> (he/him)

tools/lint-readme-lists.mjs

@@ -6,6 +6,9 @@ import assert from 'node:assert'
 import{open}from'node:fs/promises';
 import{argv}from'node:process';
 
+constghHandleLine=/^\*\[(.+)\]\(https:\/\/github\.com\/\1\)-$/;
+constmemberInfoLine=/^{2}\*\*[^*]+\*\*<<[^@]+@.+\.[a-z]+>>(\(\w+(\/[^)/]+)+\))?(-\[Supportme\]\(.+\))?$/;
+
 constlists={
 '__proto__': null,
 
@@ -26,12 +29,19 @@ const tscMembers = new Set();
 constreadme=awaitopen(newURL('../README.md',import.meta.url),'r');
 
 letcurrentList=null;
+letpreviousGithubHandleInfoRequired;
 letpreviousGithubHandle;
 letlineNumber=0;
 
 forawait(constlineofreadme.readLines()){
 lineNumber++;
-if(line.startsWith('### ')){
+if(previousGithubHandleInfoRequired){
+if(!memberInfoLine.test(line)){
+thrownewError(`${previousGithubHandleInfoRequired} info are not formatted correctly (README.md:${lineNumber})`);
+}
+previousGithubHandle=previousGithubHandleInfoRequired;
+previousGithubHandleInfoRequired=null;
+}elseif(line.startsWith('### ')){
 currentList=line.slice(4);
 previousGithubHandle=null;
 }elseif(line.startsWith('#### ')){
@@ -49,6 +59,10 @@ for await (const line of readme.readLines()){
 );
 }
 
+if(!ghHandleLine.test(line)){
+thrownewError(`${currentGithubHandle} is not formatted correctly (README.md:${lineNumber})`);
+}
+
 if(
 currentList==='TSC voting members'||
 currentList==='TSC regular members'
@@ -60,7 +74,7 @@ for await (const line of readme.readLines()){
 if(lists[currentList]){
 (actualMembers[lists[currentList]]??=newSet()).add(currentGithubHandle);
 }
-previousGithubHandle=currentGithubHandleLowerCase;
+previousGithubHandleInfoRequired=currentGithubHandleLowerCase;
 }
 }
 console.info('Lists are in the alphabetical order.');