From 78de83cc74918edc05d951c704651d81243d0e35 Mon Sep 17 00:00:00 2001
From: Rich Trott <rtrott@gmail.com>
Date: Sun, 1 Aug 2021 06:29:20 -0700
Subject: [PATCH] tools: improve error detection in find-inactive-collaborators

Previously, if the script failed to find a collaborator section in the
README file, it would carry on with no results and no error. This
detects the problem and throws an error.

It is also more robust in that it will still work if the emeriti section
ends up getting moved and does not immedaitely follow the collaborator
section.

PR-URL: https://github.com/nodejs/node/pull/39617
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
---
 tools/find-inactive-collaborators.mjs | 22 +++++++++++++++-------
 1 file changed, 15 insertions(+), 7 deletions(-)

diff --git a/tools/find-inactive-collaborators.mjs b/tools/find-inactive-collaborators.mjs
index 282a49dc965308..0d89d2869cf4f7 100755
--- a/tools/find-inactive-collaborators.mjs
+++ b/tools/find-inactive-collaborators.mjs
@@ -64,16 +64,19 @@ async function getCollaboratorsFromReadme() {
     crlfDelay: Infinity,
   });
   const returnedArray = [];
-  let processingCollaborators = false;
+  let foundCollaboratorHeading = false;
   for await (const line of readmeText) {
-    const isCollaborator = processingCollaborators && line.length;
-    if (line === '### Collaborators') {
-      processingCollaborators = true;
-    }
-    if (line === '### Collaborator emeriti') {
-      processingCollaborators = false;
+    // If we've found the collaborator heading already, stop processing at the
+    // next heading.
+    if (foundCollaboratorHeading && line.startsWith('#')) {
       break;
     }
+
+    const isCollaborator = foundCollaboratorHeading && line.length;
+
+    if (line === '### Collaborators') {
+      foundCollaboratorHeading = true;
+    }
     if (line.startsWith('**') && isCollaborator) {
       const [, name, email] = /^\*\*([^*]+)\*\* &lt;(.+)&gt;/.exec(line);
       const mailmap = await runGitCommand(
@@ -89,6 +92,11 @@ async function getCollaboratorsFromReadme() {
       });
     }
   }
+
+  if (!foundCollaboratorHeading) {
+    throw new Error('Could not find Collaborator section of README');
+  }
+
   return returnedArray;
 }