[ci] more logging in docs_pr_nudge.yaml (#17588)

Author: blinkov

.github/workflows/docs_pr_nudge.yaml

@@ -22,8 +22,8 @@ jobs:
     env:
       AUTHOR_RESPONSE_THRESHOLD_HOURS: 24         # when to nudge reviewers after author
       REVIEWER_RESPONSE_THRESHOLD_DAYS: 7         # when to nudge author after reviewer
-      REVIEWER_NUDGE_COOLDOWN_HOURS: 24           # min hours between reviewer‑nudges
-      AUTHOR_NUDGE_COOLDOWN_DAYS: 7               # min days between author‑nudges
+      REVIEWER_NUDGE_COOLDOWN_HOURS: 24           # min hours between reviewer-nudges
+      AUTHOR_NUDGE_COOLDOWN_DAYS: 7               # min days between author-nudges
     steps:
       - uses: actions/github-script@v7
 
@@ -48,13 +48,13 @@ jobs:
               (month === 5  && date <= 7) ||
               (month === 1  && date <= 7)
             ) {
-              console.log('Skip run: weekend or first week of May/year');
+              console.log(`Skip run: weekend or first-week (month=${month}, date=${date}, day=${day})`);
               return;
             }
 
             const isWeekend = d => d.getUTCDay() === 0 || d.getUTCDay() === 6;
 
-            // sum only business‑hours between two dates
+            // sum only business-hours between two dates
             function businessMsBetween(start, end) {
               let ms = 0;
               let cur = new Date(start);
@@ -65,7 +65,7 @@ jobs:
               return ms;
             }
 
-            // count only business‑days between two dates
+            // count only business-days between two dates
             function businessDaysBetween(start, end) {
               let days = 0;
               let cur = new Date(start);
@@ -79,7 +79,7 @@ jobs:
               return days;
             }
 
-            // 1) search open, non‑draft docs PRs
+            // 1) search open, non-draft docs PRs
             const q = [
               `repo:${owner}/${repo}`,
               `type:pr`,
@@ -158,14 +158,20 @@ jobs:
               ) {
                 const { data: comment } = await github.rest.issues.createComment({
                   owner, repo, issue_number: prNum,
-                  body: `Hey ${assignees.map(a=>`@${a}`).join(', ')}, it has been ${hoursSinceAuthor} business‑hours since the author's last update, could you please review? ${BOT_MARKER}`
+                  body: `Hey ${assignees.map(a=>`@${a}`).join(', ')}, it has been ${hoursSinceAuthor} business-hours since the author's last update, could you please review? ${BOT_MARKER}`
                 });
                 console.log(`Comment posted on PR #${prNum}: ${comment.html_url}`);
                 continue;
+              } else {
+                const reasons = [];
+                if (!(lastAuthorActivity > lastReviewerActivity))        reasons.push(`lastAuthor(${lastAuthorActivity.toISOString()})≤lastReviewer(${lastReviewerActivity.toISOString()})`);
+                if (!(msSinceAuthor > authorThresholdH * 3600e3))        reasons.push(`hoursSinceAuthor=${hoursSinceAuthor}h≤${authorThresholdH}h`);
+                if (!(msSinceLastRevNudge > reviewerCooldownH * 3600e3)) reasons.push(`hoursSinceLastNudge=${Math.floor(msSinceLastRevNudge/3600e3)}h≤${reviewerCooldownH}h`);
+                console.log(`Reviewer nudge skipped: ${reasons.join('; ')}`);
               }
 
               // RULE 2: nudge author
-              const daysSinceReviewer    = businessDaysBetween(lastReviewerActivity, now);
+              const daysSinceReviewer      = businessDaysBetween(lastReviewerActivity, now);
               const daysSinceLastAuthNudge = businessDaysBetween(lastNudgeTime, now);
 
               if (
@@ -175,8 +181,14 @@ jobs:
               ) {
                 const { data: comment } = await github.rest.issues.createComment({
                   owner, repo, issue_number: prNum,
-                  body: `Heads‑up: it's been ${daysSinceReviewer} business‑days since a reviewer comment. @${author}, any updates? ${assignees.map(a=>`@${a}`).join(' ')}, please check the status with the author. ${BOT_MARKER}`
+                  body: `Heads-up: it's been ${daysSinceReviewer} business-days since a reviewer comment. @${author}, any updates? ${assignees.map(a=>`@${a}`).join(' ')}, please check the status with the author. ${BOT_MARKER}`
                 });
                 console.log(`Comment posted on PR #${prNum}: ${comment.html_url}`);
+              } else {
+                const reasons = [];
+                if (!(lastReviewerActivity > lastAuthorActivity))     reasons.push(`lastReviewer(${lastReviewerActivity.toISOString()})≤lastAuthor(${lastAuthorActivity.toISOString()})`);
+                if (!(daysSinceReviewer > reviewerThresholdD))        reasons.push(`daysSinceReviewer=${daysSinceReviewer}d≤${reviewerThresholdD}d`);
+                if (!(daysSinceLastAuthNudge > authorCooldownD))      reasons.push(`daysSinceLastAuthNudge=${daysSinceLastAuthNudge}d≤${authorCooldownD}d`);
+                console.log(`Author nudge skipped: ${reasons.join('; ')}`);
               }
             }