exclude jetify bot, skip if it's an internally created pr

Author: Lagoja

.github/workflows/random-reviewer-assignment.yml

@@ -19,6 +19,7 @@ jobs:
         with:
           script: |
             const TRIAGE_USERNAME = 'Lagoja';
+            const EXCLUDE_USERNAMES = ['jetpack-io-bot'];
 
             try {
               const authenticatedUser = await github.rest.users.getAuthenticated();
@@ -31,19 +32,28 @@ jobs:
               const prAuthor = context.payload.pull_request.user.login.toLowerCase();
               const prAuthorId = context.payload.pull_request.user.id;
               const authenticatedUserLower = authenticatedUser.data.login.toLowerCase();
+
+              // If the PR author is already a member of the team, we can skip random assignment
+              const isPrAuthorInTeam = teamMembers.data.some(member => 
+                member.login.toLowerCase() === prAuthor && member.id === prAuthorId
+              );
+
+              if (isPrAuthorInTeam) {
+                console.log(`PR author ${prAuthor} is already a team member, skipping random assignment.`);
+                return;
+              }
               
               // Get eligible reviewers (excluding PR author, authenticated user, and lagoja)
               const eligibleReviewers = teamMembers.data
                 .filter(member => {
                   const loginLower = member.login.toLowerCase();
                   
-                  // Exclude PR author by both ID and username
-                  const isNotAuthor = (prAuthorId && member.id !== prAuthorId) && loginLower !== prAuthor;
                   // Exclude authenticated user
                   const isNotAuthenticatedUser = member.id !== authenticatedUser.data.id;
                   const isNotTriage = loginLower !== TRIAGE_USERNAME.toLowerCase();
-                  
-                  return isNotAuthor && isNotAuthenticatedUser && isNotTriage;
+                  const isNotExcludedUsername = !EXCLUDE_USERNAMES.includes(loginLower);
+
+                  return isNotAuthenticatedUser && isNotTriage && isNotExcludedUsername;
                 })
                 .map(member => member.login);