From 104c771eb02508e1f672c43805f1df6c9e4da215 Mon Sep 17 00:00:00 2001
From: Radoslav Dimitrov <radoslav@stacklok.com>
Date: Mon, 11 May 2026 17:44:04 +0300
Subject: [PATCH] Drop 5-reviewer cap in upstream-release-docs

Attempt assignment for every non-bot contributor in the release range
instead of stopping after 5. The cap was systematically pushing
contributors whose login sorts late alphabetically (e.g. tgrunnagle,
yrobla) into the mention block without an assignment attempt on
larger releases, which surfaced as inconsistent reviewer lists from
one release to the next.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 .github/workflows/upstream-release-docs.yml | 12 +-----------
 1 file changed, 1 insertion(+), 11 deletions(-)

diff --git a/.github/workflows/upstream-release-docs.yml b/.github/workflows/upstream-release-docs.yml
index 63984f8f..a10bae4e 100644
--- a/.github/workflows/upstream-release-docs.yml
+++ b/.github/workflows/upstream-release-docs.yml
@@ -1019,20 +1019,10 @@ jobs:
           # Per-user attempts sidestep both issues: the authoritative
           # answer is "does GitHub accept this as a reviewer right now"
           # and we ask the API that question directly.
-          #
-          # Cap attempts at 5 to avoid review fatigue on big releases.
-          TRIED=0
           ASSIGN_LIST=""
           MENTION_LIST=""
           while IFS= read -r login; do
             [ -z "$login" ] && continue
-            if [ "$TRIED" -ge 5 ]; then
-              # Over the review-fatigue cap -- mention any additional
-              # contributors instead of trying to assign them.
-              MENTION_LIST="${MENTION_LIST:+$MENTION_LIST }@$login"
-              continue
-            fi
-            TRIED=$((TRIED + 1))
             if gh pr edit "$PR_NUMBER" --add-reviewer "$login" 2>/dev/null; then
               ASSIGN_LIST="${ASSIGN_LIST:+$ASSIGN_LIST,}$login"
               echo "Assigned: $login"
@@ -1049,7 +1039,7 @@ jobs:
           {
             echo "mention_block<<MENTION_EOF"
             if [ -n "$MENTION_LIST" ]; then
-              echo "Release contributors we couldn't auto-assign as reviewers (review fatigue cap or GitHub rejected the assignment). Mentioning them so they see the PR documenting their work:"
+              echo "Release contributors we couldn't auto-assign as reviewers (GitHub rejected the assignment). Mentioning them so they see the PR documenting their work:"
               echo ""
               echo "$MENTION_LIST"
             fi