tools: clarify README linter error message

joyeecheung · aduh95 · commit 195d6038dc3f · 2025-07-26T01:32:06.000+02:00
PR-URL: #59160 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Richard Lau <richard.lau@ibm.com>
diff --git a/tools/lint-readme-lists.mjs b/tools/lint-readme-lists.mjs
@@ -6,7 +6,7 @@ import assert from 'node:assert';
 import{open } from 'node:fs/promises';
 import{argv } from 'node:process';
 
-const ghHandleLine = /^\* \[(.+)\]\(https:\/\/github\.com\/\1\) -$/;
+const ghHandleLine = /^\* \[(.+)\]\(https:\/\/github\.com\/(.+)\) -$/;
 const memberInfoLine = /^{2}\*\*[^*]+\*\* <<[^@]+@.+\.[a-z]+>>( \(\w+(\/[^)/]+)+\))?( - \[Support me\]\(.+\))?$/;
 
 const lists ={
@@ -59,9 +59,11 @@ for await (const line of readme.readLines()){
 );
 }
 
- if (!ghHandleLine.test(line)){
- throw new Error(`${currentGithubHandle} is not formatted correctly (README.md:${lineNumber})`);
+ const match = line.match(ghHandleLine);
+ if (!match){
+ throw new Error(`${line} should match ${ghHandleLine} (README.md:${lineNumber})`);
 }
+ assert.strictEqual(match[1], match[2], `GitHub handle does not match the URL (README.md:${lineNumber})`);
 
 if (
 currentList === 'TSC voting members' ||

-Original file line number
+Diff line change
 import{open}from'node:fs/promises';
 import{argv}from'node:process';
 -constghHandleLine=/^\*\[(.+)\]\(https:\/\/github\.com\/\1\)-$/;
 +constghHandleLine=/^\*\[(.+)\]\(https:\/\/github\.com\/(.+)\)-$/;
 constmemberInfoLine=/^{2}\*\*[^*]+\*\*<<[^@]+@.+\.[a-z]+>>(\(\w+(\/[^)/]+)+\))?(-\[Supportme\]\(.+\))?$/;
 constlists={
 );
+}
 -if(!ghHandleLine.test(line)){
 -thrownewError(`${currentGithubHandle} is not formatted correctly (README.md:${lineNumber})`);
 +constmatch=line.match(ghHandleLine);
 +if(!match){
 +thrownewError(`${line} should match ${ghHandleLine} (README.md:${lineNumber})`);
+}
 +assert.strictEqual(match[1],match[2],`GitHub handle does not match the URL (README.md:${lineNumber})`);
 if(
 currentList==='TSC voting members'||