request reviews instead of commenting

Author: avivkeller

.github/reviewers.json

@@ -1,13 +1,13 @@
 {
-  "fs": "@nodejs/fs",
-  "timers": "@nodejs/timers",
-  "events": "@nodejs/events",
-  "repl": "@nodejs/repl",
-  "diagnostics": "@nodejs/diagnostics",
-  "undici": "@nodejs/undici",
-  "npm": "@nodejs/npm",
-  "security": "@nodejs/security-wg",
-  "n-api": "@nodejs/addon-api",
-  "test": "@nodejs/test_runner",
-  "typescript": "@nodejs/typescript"
+  "fs": "fs",
+  "timers": "timers",
+  "events": "events",
+  "repl": "repl",
+  "diagnostics": "diagnostics",
+  "undici": "undici",
+  "npm": "npm",
+  "security": "security-wg",
+  "n-api": "addon-api",
+  "test": "test_runner",
+  "typescript": "typescript"
 }

.github/scripts/get-reviewers.mjs

@@ -1,6 +1,6 @@
 import fs from 'fs';
 
-export default async function (core, filesList) {
+export default async function (github, context, filesList) {
   // Load reviewers mapping
   const reviewersMap = JSON.parse(
     fs.readFileSync('.github/reviewers.json', 'utf8')
@@ -10,7 +10,7 @@ export default async function (core, filesList) {
   const files = filesList.trim().split(' ').filter(Boolean);
 
   // Track reviewers we need to mention
-  const reviewersToMention = new Set();
+  const reviewers = new Set();
 
   // Process each file
   for (const file of files) {
@@ -20,16 +20,19 @@ export default async function (core, filesList) {
     if (apiValues && apiValues.length > 0) {
       for (const apiValue of apiValues) {
         if (reviewersMap[apiValue]) {
-          reviewersToMention.add(`- [ ] ${reviewersMap[apiValue]}`);
+          reviewers.add(reviewersMap[apiValue]);
         }
       }
     }
   }
 
-  // Set outputs
-  const reviewersList = Array.from(reviewersToMention);
-  core.setOutput('found', reviewersList.length > 0);
-  core.setOutput('reviewers', reviewersList.join('\n'));
+  // Request Review
+  github.rest.pulls.requestReviewers({
+    owner: context.repo.owner,
+    repo: context.repo.repo,
+    pull_number: context.pull_request.number,
+    team_reviewers: Array.from(reviewers),
+  });
 }
 
 /**

.github/workflows/request-review.yml

@@ -25,7 +25,7 @@ jobs:
     runs-on: ubuntu-latest
 
     permissions:
-      # This permission is required by `thollander/actions-comment-pull-request`
+      # This permission is required to request reviews
       pull-requests: write
 
     steps:
@@ -57,14 +57,6 @@ jobs:
         with:
           script: |
             const { default: reviewers } = await import("${{github.workspace}}/.github/scripts/get-reviewers.mjs");
-            reviewers(core, process.env.PR_FILES);
+            reviewers(github, context, process.env.PR_FILES);
         env:
           PR_FILES: ${{ steps.changed-files.outputs.files }}
-
-      - uses: thollander/actions-comment-pull-request@e2c37e53a7d2227b61585343765f73a9ca57eda9 # v3.0.0
-        if: steps.reviewers.outputs.found == 'true'
-        with:
-          message: |
-            Requesting a review from the following teams:
-            ${{ steps.reviewers.outputs.reviewers }}
-          comment-tag: request_review