feat(ci): request reviews from API experts

Author: avivkeller

.github/CODEOWNERS

@@ -38,7 +38,3 @@ docs @nodejs/nodejs-website @nodejs/web-infra
 # Node.js Release Blog Posts
 apps/site/pages/en/blog/release @nodejs/releasers
 apps/site/pages/en/blog/announcements @nodejs/releasers
-
-# Specific content
-apps/site/pages/en/learn/getting-started/security-best-practices.md @nodejs/security-wg
-apps/site/pages/en/learn/typescript @nodejs/typescript

.github/reviewers.json

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

.github/scripts/get-reviewers.mjs

@@ -0,0 +1,67 @@
+import fs from 'fs';
+
+export default async function (core, filesList) {
+  // Load reviewers mapping
+  const reviewersMap = JSON.parse(
+    fs.readFileSync('.github/reviewers.json', 'utf8')
+  );
+
+  // Parse the file list from the input
+  const files = filesList.trim().split(' ').filter(Boolean);
+
+  // Track reviewers we need to mention
+  const reviewersToMention = new Set();
+
+  // Process each file
+  for (const file of files) {
+    const content = fs.readFileSync(file, 'utf8');
+    const apiValues = extractApisFromFrontmatter(content);
+
+    if (apiValues && apiValues.length > 0) {
+      for (const apiValue of apiValues) {
+        if (reviewersMap[apiValue]) {
+          reviewersToMention.add(`- [ ] ${reviewersMap[apiValue]}`);
+        }
+      }
+    }
+  }
+
+  // Set outputs
+  const reviewersList = Array.from(reviewersToMention);
+  core.setOutput('found', reviewersList.length > 0);
+  core.setOutput('reviewers', reviewersList.join('\n'));
+}
+
+/**
+ * Extract API values from frontmatter
+ * @returns {string[]} Array of API values found in frontmatter
+ */
+function extractApisFromFrontmatter(content) {
+  if (!content.trimStart().startsWith('---')) {
+    return [];
+  }
+
+  const firstMarker = content.indexOf('---');
+  const secondMarker = content.indexOf('---', firstMarker + 3);
+
+  if (firstMarker === -1 || secondMarker === -1) {
+    return [];
+  }
+
+  const frontmatterContent = content.substring(firstMarker + 3, secondMarker);
+
+  // Find the api: line
+  const apiLineRegex = /^\s*api\s*:\s*(.+)$/m;
+  const match = apiLineRegex.exec(frontmatterContent);
+
+  if (!match) return [];
+
+  // Extract all API values, handling various formats
+  // This splits by commas and trims each value
+  const apiValues = match[1]
+    .split(',')
+    .map(value => value.trim())
+    .filter(Boolean);
+
+  return apiValues;
+}

.github/workflows/request-review.yml

@@ -0,0 +1,74 @@
+name: Request Reviewers
+
+on:
+  # DEMO
+  pull_request:
+    branches:
+      - main
+      # TMP
+      - add-reviewer-support
+    types:
+      - opened
+      - reopened
+      - synchronize
+    paths:
+      - "apps/site/pages/en/**/*.md"
+      - "apps/site/pages/en/**/*.mdx"
+
+# Cancel any runs on the same branch
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
+
+permissions:
+  actions: read
+
+jobs:
+  request-review:
+    name: Request Review from API Experts
+    runs-on: ubuntu-latest
+
+    permissions:
+      # This permission is required by `thollander/actions-comment-pull-request`
+      pull-requests: write
+
+    steps:
+      - name: Harden Runner
+        uses: step-security/harden-runner@6c439dc8bdf85cadbbce9ed30d1c7b959517bc49 # v2.12.2
+        with:
+          egress-policy: audit
+
+      - name: Checkout Repo
+        uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
+        with:
+          fetch-depth: 0
+
+      - name: Get changed files
+        id: changed-files
+        run: |
+          FILES=$(git diff --name-only $BASE $HEAD | grep -E '^apps/site/pages/en/.*\.(md|mdx)$')
+          echo $FILES
+          echo "files<<EOF" >> $GITHUB_OUTPUT
+          echo $FILES >> $GITHUB_OUTPUT
+          echo "EOF" >> $GITHUB_OUTPUT
+        env:
+          BASE: ${{ github.event.pull_request.base.sha }}
+          HEAD: ${{ github.event.pull_request.head.sha }}
+
+      - name: Get Reviewers
+        id: reviewers
+        uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
+        with:
+          script: |
+            const { default: reviewers } = await import("${{github.workspace}}/.github/scripts/get-reviewers.mjs");
+            reviewers(core, 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

apps/site/pages/en/learn/asynchronous-work/asynchronous-flow-control.md

@@ -2,6 +2,7 @@
 title: Asynchronous flow control
 layout: learn
 authors: aug2uag, ovflowd
+api: timers, events
 ---
 
 # Asynchronous flow control

apps/site/pages/en/learn/asynchronous-work/discover-javascript-timers.md

@@ -2,6 +2,7 @@
 title: Discover JavaScript Timers
 layout: learn
 authors: flaviocopes, MylesBorins, LaRuaNa, amiller-gh, ahmadawais, ovflowd
+api: timers
 ---
 
 # Discover JavaScript Timers

apps/site/pages/en/learn/asynchronous-work/dont-block-the-event-loop.md

@@ -1,6 +1,7 @@
 ---
 title: Don't Block the Event Loop (or the Worker Pool)
 layout: learn
+api: events, workers
 ---
 
 # Don't Block the Event Loop (or the Worker Pool)

apps/site/pages/en/learn/asynchronous-work/event-loop-timers-and-nexttick.md

@@ -1,6 +1,7 @@
 ---
 title: The Node.js Event Loop
 layout: learn
+api: timers, events
 ---
 
 # The Node.js Event Loop

apps/site/pages/en/learn/asynchronous-work/overview-of-blocking-vs-non-blocking.md

@@ -2,6 +2,7 @@
 title: Overview of Blocking vs Non-Blocking
 layout: learn
 authors: ovflowd, HassanBahati
+api: events
 ---
 
 # Overview of Blocking vs Non-Blocking

apps/site/pages/en/learn/asynchronous-work/the-nodejs-event-emitter.md

@@ -2,6 +2,7 @@
 title: The Node.js Event emitter
 layout: learn
 authors: flaviocopes, MylesBorins, fhemberger, LaRuaNa, ahmadawais, ovflowd
+api: events
 ---
 
 # The Node.js Event emitter