Remove slow review label after processing commands to avoid race (#24765

)

scripts/ci/pr-bot/processPrUpdate.ts

@@ -75,20 +75,19 @@ async function processPrComment(
   const commentContents = payload.comment.body;
   const commentAuthor = payload.sender.login;
   const pullAuthor = getPullAuthorFromPayload(payload);
+  console.log(commentContents);
+  const processedCommand = await processCommand(
+    payload,
+    commentAuthor,
+    commentContents,
+    stateClient,
+    reviewerConfig
+  );
   // If there's been a comment by a non-author, we can remove the slow review label
   if (commentAuthor !== pullAuthor && commentAuthor !== BOT_NAME) {
     await removeSlowReviewLabel(payload);
   }
-  console.log(commentContents);
-  if (
-    await processCommand(
-      payload,
-      commentAuthor,
-      commentContents,
-      stateClient,
-      reviewerConfig
-    )
-  ) {
+  if (processedCommand) {
     // If we've processed a command, don't worry about trying to change the attention set.
     // This is not a meaningful push or comment from the author.
     console.log("Processed command");

scripts/ci/pr-bot/shared/persistentState.ts

@@ -40,14 +40,20 @@ async function commitStateToRepo() {
     );
   }
   // Print changes for observability
-  await exec.exec("git status", [], {ignoreReturnCode: true});
+  await exec.exec("git status", [], { ignoreReturnCode: true });
   await exec.exec("git add state/*");
-  const changes = await exec.exec("git diff --quiet --cached origin/pr-bot-state state", [], {ignoreReturnCode: true});
+  const changes = await exec.exec(
+    "git diff --quiet --cached origin/pr-bot-state state",
+    [],
+    { ignoreReturnCode: true }
+  );
   if (changes == 1) {
     await exec.exec(`git commit -m "Updating config from bot" --allow-empty`);
     await exec.exec("git push origin pr-bot-state");
   } else {
-    console.log("Skipping updating state branch since there are no changes to commit");
+    console.log(
+      "Skipping updating state branch since there are no changes to commit"
+    );
   }
 }